@jsenv/plugin-commonjs 2.3.1 → 2.4.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/plugin-commonjs",
-  "version": "2.3.1",
+  "version": "2.4.1",
   "license": "MIT",
   "repository": {
     "type": "git",

package/src/jsenv_plugin_commonjs.js

@@ -1,5 +1,5 @@
 import { URL_META } from "@jsenv/url-meta";
-import { injectQueryParams, urlToExtension } from "@jsenv/urls";
+import { injectQueryParams, urlToExtension, urlToBasename } from "@jsenv/urls";
 import { defaultLookupPackageScope } from "@jsenv/node-esm-resolution";
 
 import { commonJsToJsModule } from "./cjs_to_esm.js";
@@ -15,11 +15,16 @@ export const jsenvPluginCommonJs = ({
 }) => {
   const markAsJsModuleProxy = (reference) => {
     reference.expectedType = "js_module";
-    const packageFileUrl = defaultLookupPackageScope(reference.url);
-    if (packageFileUrl) {
-      reference.filename = `${reference.specifier}${urlToExtension(
-        reference.ownerUrlInfo.url,
-      )}`;
+    const onwerUrlExtension = urlToExtension(reference.ownerUrlInfo.url);
+    const referenceUrlExtension = urlToExtension(reference.url);
+    if (referenceUrlExtension !== onwerUrlExtension) {
+      const packageFileUrl = defaultLookupPackageScope(reference.url);
+      if (packageFileUrl) {
+        const basename = isBareSpecifier(reference.specifier)
+          ? reference.specifier
+          : urlToBasename(reference.url);
+        reference.filename = `${basename}${onwerUrlExtension}`;
+      }
     }
   };
   const turnIntoJsModuleProxy = (reference) => {
@@ -108,3 +113,20 @@ export const jsenvPluginCommonJs = ({
     },
   };
 };
+
+const isBareSpecifier = (specifier) => {
+  if (
+    specifier[0] === "/" ||
+    specifier.startsWith("./") ||
+    specifier.startsWith("../")
+  ) {
+    return false;
+  }
+  try {
+    // eslint-disable-next-line no-new
+    new URL(specifier);
+    return false;
+  } catch (e) {
+    return true;
+  }
+};