Fix: fully remove TS nested type-only exported namespaces (#15882)

babel · Aug 24, 2023 · 48d12ef · 48d12ef
1 parent c3f03f6
commit 48d12ef
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 0 deletions.
diff --git a/packages/babel-plugin-transform-typescript/src/namespace.ts b/packages/babel-plugin-transform-typescript/src/namespace.ts
@@ -242,6 +242,9 @@ function handleNested(
               transformed,
             );
           }
+        } else {
+          namespaceTopLevel.splice(i, 1);
+          i--;
         }
       }
     }

diff --git a/packages/babel-plugin-transform-typescript/test/fixtures/namespace/nested-namespace/input.ts b/packages/babel-plugin-transform-typescript/test/fixtures/namespace/nested-namespace/input.ts
@@ -0,0 +1,13 @@
+export namespace A {
+  export namespace B {
+    export interface C {
+      D?: number;
+      E?: number;
+      F?: string;
+    }
+  }
+
+  export enum G {
+    H = 0
+  }
+}
diff --git a/...ges/babel-plugin-transform-typescript/test/fixtures/namespace/nested-namespace/output.mjs b/...ges/babel-plugin-transform-typescript/test/fixtures/namespace/nested-namespace/output.mjs
@@ -0,0 +1,8 @@
+export let A;
+(function (_A) {
+  let G = /*#__PURE__*/function (G) {
+    G[G["H"] = 0] = "H";
+    return G;
+  }({});
+  _A.G = G;
+})(A || (A = {}));
-Original file line number
+Diff line change
@@ Expand Up / @@ -242,6 +242,9 @@ function handleNested( @@
                   transformed,
                 );
               }
+            } else {
+              namespaceTopLevel.splice(i, 1);
+              i--;
             }
           }
         }
@@ Expand Down @@