LuaLS / LuaLS/lua-language-server
Table literals aren't properly checked for field injections and missing fields
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
I'd expect tables created in the return statement to check for missing fields, and those created outside of them to check for injected fields.
Actual Behaviour
There are no warnings emitted for the attached code.
Reproduction steps
Code to reproduce the issue:
---@class Tile
---@field name string
local Tile = {}
---@return Tile
function Tile.new()
---@type Tile
local d = {
name = 'abc',
f = 10 -- No warning when a field is being injected
}
return {
f = 10 -- No warning even though there's a missing field and a field is being injected
}
end
Note that it won't produce any warnings, regardless of configuration (as far as I can tell).
Additional Notes
No response
Log File
No response
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem Lua-Typprüfpfad für das reproduzierte Tile-Beispiel und untersuche, wie Tabellenliterale in return-Anweisungen und typisierten lokalen Variablen validiert werden. Reproduziere das Problem in der VS Code-Erweiterung und überprüfe anschließend, dass injizierte Felder Warnungen erzeugen und die zurückgegebene Tabelle sowohl das fehlende Feld name als auch das zusätzliche Feld f meldet.
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
- 35/100