LuaLS / LuaLS/lua-language-server

Optional not removed in if/elseif block for field in filter

Offen
#2,592 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Lua
Sterne
4.4k
Forks
442
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

How are you using the lua-language-server?

Visual Studio Code Extension (sumneko.lua)

Which OS are you using?

Linux

What is the issue affecting?

Type Checking

Expected Behaviour
---@type { n: number? }
local t = { n = 1 }

if t.n then
	_ = t.n -- (field) t.n: number
else
	_ = t.n -- (field) t.n: number?
end
Actual Behaviour
---@type number?
local n = 0

if n then
	_ = n -- local t: number
else
	_ = n -- local t: nil
end

-- but

---@type { n: number? }
local t = { n = 1 }

if t.n then
	_ = t.n -- (field) t.n: number? !!!
else
	_ = t.n -- (field) t.n: number?
end
Additional Notes

I couldn't immediately understand where exactly this type casting is happening

Looks like here
https://github.com/LuaLS/lua-language-server/blob/34ff9d3ca730bc28879ab2b0c1a49f2c5480f9a3/script/vm/tracer.lua#L240-L243
but adding debug log nothing came out

I would be grateful for the information, I could try to fix it myself

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Reproduziere die beiden Narrowing-Beispiele in der Visual Studio Code-Erweiterung und untersuche anschließend script/vm/tracer.lua um die Zeilen 240–243, die im Issue erwähnte Stelle. Als erledigt gilt die Aufgabe, wenn der Feldzugriff t.n im if-Zweig auf number eingegrenzt wird, während er im else-Zweig number? bleibt, entsprechend dem Verhalten bei lokalen Variablen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
lua
Bereich
devtools
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
38/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.