CFA does not consider pass-by-reference to invalidate assumptions

[Sainan]

🔎 Search Terms

control flow analysis, branch, condition

🕗 Version & Regression Information

Tested in 5.8.2.

⏯ Playground Link

https://www.typescriptlang.org/play/?noUncheckedIndexedAccess=true#code/JYOwLgpgTgZghgYwgAgBIFcC2cTIN4BQyycA5hAPwBcyIWARtANwEC+BBM6ICYwA9rgSCAzlggAKfvQBWNDNhABKfEWTSZAOjIoAvMm4ATCDFARDLdgWEgRYdbPlYcyfXhLkaAFgBMyVizAMMhSstrkKoTENmKYkhpKLNGi9jo0dJiMUK4OWjpMyAD0hcgAcvypuNBQ-FBsBEA

💻 Code

interface Human {
  age?: number;
}

function consume(obj: Human) {
  obj.age = undefined;
}

const obj: Human = { age: 42 };
if (obj.age) {
  consume(obj);
  const age: number = obj.age; // Not an error
}

🙁 Actual behavior

Assignment of const age: number succeeds with undefined.

🙂 Expected behavior

obj.age is assessed as number | undefined, causing the assignment to fail.

Additional information about the issue

No response

[nmain]

Duplicate of #9998

[Sainan]

Fair, although in this case the function is not a black box.

[typescript-bot]

This issue has been marked as "Duplicate" and has seen no recent activity. It has been automatically closed for house-keeping purposes.