LuaLS / LuaLS/lua-language-server

Missing diagnostic when passing nil-able fields to pairs/ipairs

Ouverte
#2,429 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
Lua
Étoiles
4.4k
Forks
442
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

### 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, Diagnostics/Syntax Checking

### Expected Behaviour

In the following code, I expect to get param-type mismatch for each of the `for` loops (with `pairs` and `ipairs`) because the variable being iterated can be `nil`:

```lua
---@type string[]?
local x1 = {}

for _ in ipairs(x1) do -- param-type-mismatch: Cannot assign `string[]?` to parameter `
break
end

---@type table?
local y1 = {}

for _ in ipairs(y1) do -- param-type-mismatch: Cannot assign `table?` to parameter `
break
end

---@class Class
---@field x string[]?
---@field y table?

---@type Class
local x2 = {}

for _ in ipairs(x2.x) do -- Should warn, but it doesn't
break
end

for _ in pairs(x2.y) do -- Should warn, but it doesn't
break
end
```

### Actual Behaviour

The param-type-mismatch diagnostic is only shown for the first two loops (direct variables without nesting/fields), but not when using fields (the third and fourth for loops).

### Reproduction steps

Copy the code into a Lua file and observe the diagnostics

### Additional Notes

_No response_

### Log File

_No response_

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Reproduisez le problème en copiant l’exemple Lua fourni dans un fichier Lua et en comparant les diagnostics pour les variables directes et les champs imbriqués. Suivez le chemin de vérification des types qui gère les arguments de ipairs et pairs, puis vérifiez que les champs nullable reçoivent le même diagnostic param-type-mismatch que les variables directes.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
lua
Domaine
devtools
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.