LuaLS / LuaLS/lua-language-server
Table literals aren't properly checked for field injections and missing fields
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?
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
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 con la ruta de comprobación de tipos de Lua para el ejemplo reproducido de Tile e inspecciona cómo se validan los literales de tabla en las sentencias return y en las variables locales con tipo. Reproduce el problema en la extensión de VS Code y, a continuación, verifica que los campos inyectados produzcan advertencias y que la tabla devuelta informe tanto del campo name que falta como del campo adicional f.
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
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100