microsoft / microsoft/TypeScript
`asserts this is T` does not narrow type of `this`
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
asserts this is Tnarrowassertions- I also checked #59707, but it is unrelated (generic types).
🕗 Version & Regression Information
- Playground:
v5.7.0-dev.20241004 - At least since
v5.6.2
⏯ Playground Link
💻 Code
class A {
onlyDefinedInA() {
console.log("A");
}
assertA(): asserts this is A { }
}
class B {
assertA(): asserts this is A { }
}
type C = A | B;
function assertA(c: C): asserts c is A {
}
function test1(c: C) {
c.assertA();
c.onlyDefinedInA(); // Error: Property 'onlyDefinedInA' does not exist on type 'C'. Property 'onlyDefinedInA' does not exist on type 'B'.
}
function test2(c: C) {
assertA(c);
c.onlyDefinedInA(); // Works!
}
🙁 Actual behavior
Type of c is not narrowed after the method call, producing an error.
🙂 Expected behavior
I expect the member function to to perform type narrowing.
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 di TypeScript Playground e confronta il caso di chiamata al metodo in test1 con l'asserzione autonoma in test2. Traccia come viene gestito asserts this is A dopo una chiamata a un membro, quindi verifica che la chiamata al metodo restringa c in modo che onlyDefinedInA() venga accettato senza causare regressioni nel caso autonomo.
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à
- Tranquilla
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 48/100