@jsenv/plugin-bundling 2.3.0 → 2.4.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/plugin-bundling",
-  "version": "2.3.0",
+  "version": "2.4.1",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -30,7 +30,7 @@
     "@jsenv/sourcemap": "1.1.0",
     "acorn-import-assertions": "1.9.0",
     "lightningcss": "1.21.5",
-    "rollup": "3.26.2"
+    "rollup": "3.26.3"
   },
   "peerDependencies": {
     "@jsenv/core": "*"

package/src/js_module/bundle_js_modules.js

@@ -179,16 +179,35 @@ const rollupPluginJsenv = ({
           });
           return;
         }
+        let preserveSignature;
+        if (strictExports) {
+          preserveSignature = "strict";
+        } else {
+          // When referenced only once we can enable allow-extension
+          // otherwise stick to strict exports to ensure all importers
+          // receive the correct exports
+          let firstStrongRef = null;
+          let hasMoreThanOneStrongRefFromOther = false;
+          for (const referenceFromOther of jsModuleUrlInfo.referenceFromOthersSet) {
+            if (referenceFromOther.isWeak) {
+              continue;
+            }
+            if (firstStrongRef) {
+              hasMoreThanOneStrongRefFromOther = true;
+              break;
+            }
+            firstStrongRef = referenceFromOther;
+          }
+          preserveSignature = hasMoreThanOneStrongRefFromOther
+            ? "strict"
+            : "allow-extension";
+        }
         emitChunk({
           id,
           implicitlyLoadedAfterOneOf: previousNonEntryPointModuleId
             ? [previousNonEntryPointModuleId]
             : null,
-          preserveSignature: strictExports
-            ? "strict"
-            : jsModuleUrlInfo.referenceFromOthersSet.size < 2
-            ? "allow-extension"
-            : "strict",
+          preserveSignature,
         });
         previousNonEntryPointModuleId = id;
       });