Declare class in namespace should be removed

babel · Jul 20, 2022 · f497498 · f497498
1 parent b807206
commit f497498
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 1 deletion.
diff --git a/packages/babel-plugin-transform-typescript/src/namespace.ts b/packages/babel-plugin-transform-typescript/src/namespace.ts
@@ -185,6 +185,13 @@ function handleNested(
       case "TSEnumDeclaration":
       case "FunctionDeclaration":
       case "ClassDeclaration": {
+        if (
+          subNode.declaration.type === "ClassDeclaration" &&
+          subNode.declaration.declare
+        ) {
+          continue;
+        }
+
         const itemName = subNode.declaration.id.name;
         names.add(itemName);
         namespaceTopLevel.splice(

diff --git a/...s/babel-plugin-transform-typescript/test/fixtures/declarations/nested-namespace/input.mjs b/...s/babel-plugin-transform-typescript/test/fixtures/declarations/nested-namespace/input.mjs
@@ -1,4 +1,4 @@
 ; // Otherwise-empty file
 export declare namespace P {
-  export namespace C {}
+  export namespace C { }
 }
diff --git a/packages/babel-plugin-transform-typescript/test/fixtures/namespace/declare-class/input.ts b/packages/babel-plugin-transform-typescript/test/fixtures/namespace/declare-class/input.ts
@@ -0,0 +1,4 @@
+export namespace A {
+  export declare class B {
+  }
+}
diff --git a/packages/babel-plugin-transform-typescript/test/fixtures/namespace/declare-class/output.mjs b/packages/babel-plugin-transform-typescript/test/fixtures/namespace/declare-class/output.mjs
@@ -0,0 +1,3 @@
+export let A;
+
+(function (_A) {})(A || (A = {}));