-
Notifications
You must be signed in to change notification settings - Fork 13.1k
Description
π Search Terms
Namespace, use before declaration, initialized variable
π Version & Regression Information
- This is the behavior in every version I tried.
β― Playground Link
π» Code
export namespace A{
export namespace A1 {
export const fn = () => "string"
}
export namespace A2{
// should report use before initialization here
export const array = [A3.value]
}
export namespace A3{
export const value = A1.fn()
}
}π Actual behavior
Hi, using typescript V5+, the code provided did not reports any errors, and doesn't work in runtime due to use before initialization.
π Expected behavior
It is expected that TS2448 and TS2454 will be reported.(use before declaration)
These are reported if the namespace keyword is omitted in the code example and it should be the same when namespaces are used.
It seems files with namespaces are always transpiled with the same format using any TS configuration. (everything becomes iife)
Additional information about the issue
Optimistically additional support for namespace will be added, such as not transforming the entire file to iife when namespaces are used- which happen on any configuration.
And maybe even an option to make a code like I've sent work, by isolating these namespaces properly, and resort their order by their variable dependencies between themselves, and report when cycling variable dependency between namespaces exists.
Activity
azerum commentedon Jun 23, 2025
Simpler example where code compiles, but fails at runtime:
Which seems surprising as using
classinstead ofnamespacedoes report the error. I wonder what makes them different with respect to type checking:Copilot commentedon Jun 23, 2025
yarden-app commentedon Jun 23, 2025
I purposely use that code example to complement my "optimistic" request. This will really help with the semantic of the module/file. Like how you would declare greater functions before lesser one in the same file.
I don't want to use ambient namespace and want to avoid declare keyword. Both which do not support nesting properly, which is kind of the main thing about namespace anyway.
Thanks
Copilot commentedon Jun 23, 2025
@RyanCavanaugh Unfortunately I hit an unexpected error while trying to work on issue #61891. I've automatically reported this to GitHub.
You can ask me to try again later by unassigning and then reassigning the issue to me again.
If you want to contact GitHub about this error, please mention the following identifier so they can better serve you:
19689387-b3ca-4037-9cb2-b3fb4268874aSorry for the inconvenience!