microsoft / microsoft/TypeScript
False-positive `unintentional comparison` error with closures
- Langage dominant
- Go
- Étoiles
- 111k
- Forks
- 14.3k
- Merge moyen
- 2 j 4 h
- PR mergées (30 j)
- 132
Description
### 🔎 Search Terms
unintentinal, comparison, 2367, overlap
### 🕗 Version & Regression Information
- typescript: `v7.0.2`
- node: `v24.14.1`
- pnpm: `v12.3.4`
### ⏯ Playground Link
_No response_
### 💻 Code
```ts
let n: "a" | "b" = "a";
const updateN = () => {
n = "b";
};
// This comparison appears to be unintentional because the types '"a"' and '"b"' have no overlap. [2367]
if (n === "b") {}
updateN();
// This comparison appears to be unintentional because the types '"a"' and '"b"' have no overlap. [2367]
if (n === "b") {}
n = "b";
// No error
if (n === "b") {}
```
### 🙁 Actual behavior
If a variable of type `"a" | "b"` is changed in a function with closure, the LSP fails to catch the change, and assumes that it was never changed.
### 🙂 Expected behavior
No error warning, since code works as expected
### Additional information about the issue
_No response_
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Reproduisez l’exemple de closure de l’issue avec TypeScript v7.0.2 et examinez le chemin du type-checker qui émet diagnostic 2367. Suivez la manière dont le narrowing du flux de contrôle est géré à travers updateN, puis ajoutez un test de régression couvrant les deux comparaisons ; le travail sera terminé lorsque la comparaison après l’appel du closure ne produira aucun avertissement de faux positif, tout en continuant à vérifier correctement l’exemple.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- compilers
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- Active
- Clarté
- Plutôt claire
- Accessibilité débutants
- 55/100