Uninitialized local check still permits provably erroneous use-before-assign in closure

🔎 Search Terms

uninitialized, local, use before assign, control flow

🕗 Version & Regression Information

• This changed in commit or PR https://github.com/microsoft/TypeScript/pull/55887

⏯ Playground Link

https://www.typescriptlang.org/play/?ts=5.7.0-dev.20240925#code/DYUwLgBAZg9jBcEDOYBOBLAdgcwNwCh8BjGTFZcAMTggF4IAKASkQDcZ0ATOgPggG98ECOiiNYMAHSgcYABYQAPBACMTAUOHQa9AOQTdBYQF98xgvhAAPAA4xUkfqaA

💻 Code

let foo: string;

const setFoo = (): void => {
  if (foo.length < 1) {
    foo = 'foo';
  }
};

export {}

🙁 Actual behavior

No TS Error

🙂 Expected behavior

Variable 'foo' is used before being assigned.(2454)

Additional information about the issue

(this error is correctly reported if we never assign at all, as below)

let foo: string;

const setFoo = (): void => {
  if (foo.length < 1) {
  }
};

export {}

Related, https://github.com/microsoft/TypeScript/issues/60064.

Otherwise this can be solved by not considering assignments which occur unconditionally after a read.

Inizia con la riproduzione collegata di TypeScript Playground e confrontala con l’esempio aggiuntivo che segnala già diagnostic 2454. Leggi la modifica al control-flow in PR 55887 e l’issue correlata 60064; il lavoro è completato quando la lettura dalla closure segnala use-before-assign senza interrompere il comportamento esistente della diagnostica no-assignment.