Do not memoize, since only one is executed

babel · Aug 17, 2023 · 8fce167 · 8fce167
1 parent 8409749
commit 8fce167
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 15 deletions.
diff --git a/packages/babel-plugin-proposal-import-wasm-source-to-fetch/src/index.ts b/packages/babel-plugin-proposal-import-wasm-source-to-fetch/src/index.ts
@@ -9,23 +9,12 @@ export default declare(({ assertVersion, types: t, template }) => {
   function buildFetch(
     path: NodePath<t.ImportDeclaration | t.ImportExpression>,
   ) {
-    let specifier = path.node.source;
-    let specifierRef: t.Expression;
-    if (t.isStringLiteral(specifier)) {
-      specifierRef = specifier;
-    } else {
-      specifierRef = path.scope.generateDeclaredUidIdentifier("specifier");
-      specifier = t.assignmentExpression(
-        "=",
-        t.cloneNode(specifierRef),
-        specifier,
-      );
-    }
+    const specifier = path.node.source;
 
     return template.expression.ast`
       WebAssembly.compileStreaming(fetch(
         import.meta.resolve?.(${specifier}) ??
-        new URL(${t.cloneNode(specifierRef)}, import.meta.url)
+        new URL(${t.cloneNode(specifier)}, import.meta.url)
       ))
     `;
   }

diff --git a/...t-wasm-source-to-fetch/test/fixtures/basic/import-expression-dynamic-specifier/output.mjs b/...t-wasm-source-to-fetch/test/fixtures/basic/import-expression-dynamic-specifier/output.mjs
@@ -1,2 +1 @@
-var _specifier;
-let promise = WebAssembly.compileStreaming(fetch(import.meta.resolve?.(_specifier = getSpecifier()) ?? new URL(_specifier, import.meta.url)));
+let promise = WebAssembly.compileStreaming(fetch(import.meta.resolve?.(getSpecifier()) ?? new URL(getSpecifier(), import.meta.url)));