LuaLS / LuaLS/lua-language-server

`need-check-nil` when nil check is done in a function

オープン
#1,429 コメント 3 件 リアクション 4 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

enhancement
主要言語
Lua
スター
4.4k
フォーク
442
PR マージ指標
30日以内にマージされた PR はありません

説明

How are you using the lua-language-server?

Visual Studio Code Extension (sumneko.lua)

Which OS are you using?

Windows

What is the issue affecting?

Type Checking

Expected Behaviour

No type checking errors

Actual Behaviour

Need check nil.Lua Diagnostics.(need-check-nil) on the :lower()

Reproduction steps

Simplified case with the issue

---@param str string?
---@return boolean
local function isEmpty(str)
	return str == nil or str == ''
end

---@param input string?
---@return string?
local function myFunc(input)
	if isEmpty(input) then
		return
	end
	return input:lower()
end
Additional Notes

No response

Log File

No response

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず、issue の Lua 例を language server で実行し、need-check-nil 診断を生成する型チェックの経路を調べます。isEmpty 関数の結果が input の絞り込みにどのように影響するかを追跡し、その後、他の場所の nil チェックが引き続き正しく機能する一方で、input:lower() では診断が発生しないことを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
lua
領域
devtools
issue の種類
バグ
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。