refactor(transformer/typescript): correct comment (#15803)

overlookmotel · overlookmotel · commit c167dfa86bbd · 2025-11-17T19:38:37.000Z
Follow-on after #15712. Correct a comment, based on AI review: #15712 (comment)
diff --git a/crates/oxc_transformer/src/typescript/namespace.rs b/crates/oxc_transformer/src/typescript/namespace.rs
@@ -206,8 +206,9 @@ impl<'a> TypeScriptNamespace<'a, '_> {
                     self.handle_nested(decl, /* is_export */ false, &mut new_stmts, None, ctx);
                 }
                 Statement::TSGlobalDeclaration(_) => {
-                    // Remove it
-                    // TODO: Is it even possible for a`TSGlobalDeclaration` to be nested within a `TSModuleDeclaration`?
+                    // Remove it.
+                    // Note: It is legal to have a `TSGlobalDeclaration` nested within a `TSModuleDeclaration`,
+                    // where identifier is a string literal: `declare module 'foo' { global {} }`
                 }
                 Statement::ExportNamedDeclaration(export_decl) => {
                     // NB: `ExportNamedDeclaration` with no declaration (e.g. `export {x}`) is not

Original file line number	Diff line number	Diff line change
`@@ -206,8 +206,9 @@ impl<'a> TypeScriptNamespace<'a, '_> {`
`206`	`206`	`self.handle_nested(decl, /* is_export */ false, &mut new_stmts, None, ctx);`
`207`	`207`	`}`
`208`	`208`	`Statement::TSGlobalDeclaration(_) => {`
`209`		`- // Remove it`
`210`		- // TODO: Is it even possible for a`TSGlobalDeclaration` to be nested within a `TSModuleDeclaration`?
	`209`	`+ // Remove it.`
	`210`	+ // Note: It is legal to have a `TSGlobalDeclaration` nested within a `TSModuleDeclaration`,
	`211`	+ // where identifier is a string literal: `declare module 'foo' { global {} }`
`211`	`212`	`}`
`212`	`213`	`Statement::ExportNamedDeclaration(export_decl) => {`
`213`	`214`	// NB: `ExportNamedDeclaration` with no declaration (e.g. `export {x}`) is not