microsoft / microsoft/TypeScript
Aliased optional chain calls don't narrow like their inline equivalents
Nessuno ha ancora preso questa issue.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.3k
- Merge medio
- 1g 19h
- PR unite (30g)
- 117
Descrizione
🔎 Search Terms
aliased conditions control flow analysis cfa optional chain call
🕗 Version & Regression Information
- This is the behavior in every version I tried
⏯ Playground Link
💻 Code
function resolve1(id: string | undefined) {
const isBundled = id?.includes("_bundled");
if (isBundled) {
const str: string = id; // error
}
}
function resolve2(id: string | undefined) {
if (id?.includes("_bundled")) {
const str: string = id; // ok
}
}
🙁 Actual behavior
resolve1 errors within the if block
🙂 Expected behavior
I'd expect it to be OK. Just like resolve2 is OK with it here - the condition is exactly the same and TS supports Control Flow Analysis of Aliased Conditions and Discriminant since 4.4
Additional information about the issue
No response
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia con la riproduzione collegata in TypeScript Playground e confronta resolve1 con resolve2. Poi traccia l'analisi del flusso di controllo per le condizioni aliasate e le chiamate con optional chaining. Aggiungi o aggiorna un test di regressione che mostri che la condizione aliasata restringe id come la condizione inline e verifica che entrambe le assegnazioni di stringhe abbiano esito positivo.
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à
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 45/100