microsoft / microsoft/TypeScript

False-positive `unintentional comparison` error with closures

Aperta
#64,314 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Duplicate
Lingua principale
Go
Stelle
111k
Fork
14.3k
Merge medio
2g 4h
PR unite (30g)
132

Descrizione

### 🔎 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_

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Riproduci l’esempio di closure nell’issue con TypeScript v7.0.2 e ispeziona il percorso del type-checker che emette diagnostic 2367. Traccia come viene gestito il narrowing del flusso di controllo attraverso updateN, quindi aggiungi un test di regressione che copra entrambi i confronti; il lavoro è completato quando il confronto dopo la chiamata al closure non produce alcun avviso di falso positivo, mentre l’esempio continua a essere verificato correttamente.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
typescript
Ambito
compilers
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Attiva
Chiarezza
Abbastanza chiara
Idoneità per principianti
55/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.