LuaLS / LuaLS/lua-language-server
Accessing deeply nested variable at 'if' statemet cause type collision
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Lua
- Estrellas
- 4.4k
- Forks
- 442
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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
Type checking does not breaks after accessing variable at 'if' statement.
Actual Behaviour
2nd nested variable type became '<type> | unknown' so accessing variable or function which not exists in this class does not cause warning.
Reproduction steps
---@class DemoD
---@field c integer
local DemoD = {}
DemoD.c = 0
---@class DemoC
local DemoC = {}
DemoC.D = DemoD
---@class DemoB
local DemoB = {}
DemoB.C = DemoC
---@class DemoA
local DemoA = {}
DemoA.B = DemoB
---@class DemoRoot
DemoRoot = {}
DemoRoot.A = DemoA
local d = DemoRoot.A.B.C.D
-- if d.c > 1 then end -- if we replace 'DemoRoot.A.B.C.D' with 'd' then F is undefined in all 4 cases
if DemoRoot.A.B.C.D.c > 1 then end
DemoRoot.F() -- warning: F is undefined
DemoRoot.A.F() -- warning: F is undefined
DemoRoot.A.B.F() -- no warning
DemoRoot.A.B.C.F() -- no warning
Additional Notes
No response
Log File
No response
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza reproduciendo el comportamiento informado con el fragmento de Lua del issue y compara las formas directas con las formas que usan variables locales en el comprobador de tipos utilizado por la extensión de Visual Studio Code. La corrección estará completa cuando los miembros no válidos, como DemoRoot.A.B.F() y DemoRoot.A.B.C.F(), produzcan advertencias de miembros no definidos sin romper el acceso anidado válido.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- lua
- Área
- devtools
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100