Microsoft/TypeScript
Vedi su GitHub`const enum` references in the body of nodes with grammar errors are not inlined
Open
#58.209 aperta il 15 apr 2024
BugDomain: JS EmitHelp Wanted
Metriche repository
- Star
- (48.455 star)
- Metriche merge PR
- (Merge medio 6g 17h) (9 PR mergiate in 30 g)
Descrizione
Acknowledgement
- I acknowledge that issues using this template may be closed without further explanation at the maintainer's discretion.
Comment
Which results in output that's more broken than the input.
For example,
const enum A {
a = 1,
b = 2
}
A.a;
return {
f() {
return A.a;
}
}
A top-level return is a grammar error, but is valid in old-style cjs modules at runtime, so is actually a vaguely reasonable grammar error to //@ts-ignore. The issue, however, is that the const enum reference within the body of that return statement -A.a is left as-is rather than being inlined, so at runtime it refers to something which does not exist.
This doesn't seem to be in any way intentional - this is a side effect of getConstantValue in the checker only using a cached resolvedSymbol, rather than ensuring one is available by checking/resolving the access expression first (which is also an issue for emit done before a full checker walk).