rollup-plugin-iife-split 0.0.2 → 0.0.3

package/dist/index.d.ts

@@ -39,9 +39,11 @@ interface IifeSplitOptions {
      */
     unshared?: (id: string) => boolean;
     /**
-     * Enable debug logging to see intermediate transformation steps.
+     * Directory to write intermediate files for debugging.
+     * If set, writes ESM files before IIFE conversion to help diagnose issues.
+     * Example: './debug-output'
      */
-    debug?: boolean;
+    debugDir?: string;
 }
 
 declare function iifeSplit(options: IifeSplitOptions): Plugin;

package/dist/index.js

@@ -158,18 +158,8 @@ function destructureSharedParameter(code, mappings, parse) {
   return ms.toString();
 }
 async function convertToIife(options) {
-  const { code, globalName, globals, sharedGlobalPath, sharedChunkFileName, parse, debug } = options;
+  const { code, globalName, globals, sharedGlobalPath, sharedChunkFileName, parse } = options;
   const importMappings = sharedGlobalPath ? extractSharedImportMappings(code, parse) : [];
-  if (debug && sharedGlobalPath) {
-    console.log("\n=== DEBUG convertToIife ===");
-    console.log("globalName:", globalName);
-    console.log("sharedGlobalPath:", sharedGlobalPath);
-    console.log("sharedChunkFileName:", sharedChunkFileName);
-    console.log("--- ESM code (first 500 chars) ---");
-    console.log(code.slice(0, 500));
-    console.log("--- Import mappings ---");
-    console.log(importMappings);
-  }
   const rollupGlobals = (id) => {
     if (sharedGlobalPath) {
       if (id.includes(SHARED_CHUNK_NAME)) {
@@ -199,18 +189,9 @@ async function convertToIife(options) {
   });
   await bundle.close();
   let result = output[0].code;
-  if (debug && sharedGlobalPath) {
-    console.log("--- IIFE before destructuring (first 800 chars) ---");
-    console.log(result.slice(0, 800));
-  }
   if (sharedGlobalPath && importMappings.length > 0) {
     result = stripNamespaceGuards(result);
     result = destructureSharedParameter(result, importMappings, parse);
-    if (debug) {
-      console.log("--- IIFE after destructuring (first 800 chars) ---");
-      console.log(result.slice(0, 800));
-      console.log("=== END DEBUG ===\n");
-    }
   } else if (globalName && !globalName.includes(".")) {
   } else if (globalName) {
     result = stripNamespaceGuards(result);
@@ -347,6 +328,7 @@ function removeSharedImportsAndRewriteRefs(code, sharedChunkFileName, sharedExpo
   const ast = parse(code);
   const s = new MagicString2(code);
   const namespaceNames = /* @__PURE__ */ new Set();
+  const namedImportRenames = /* @__PURE__ */ new Map();
   walk2(ast, {
     enter(node) {
       if (node.type === "ImportDeclaration") {
@@ -356,6 +338,19 @@ function removeSharedImportsAndRewriteRefs(code, sharedChunkFileName, sharedExpo
           for (const spec of importNode.specifiers) {
             if (spec.type === "ImportNamespaceSpecifier") {
               namespaceNames.add(spec.local.name);
+            } else if (spec.type === "ImportSpecifier" && spec.imported) {
+              const exportedName = spec.imported.name;
+              const localAlias = spec.local.name;
+              const actualLocal = sharedExportToLocal.get(exportedName) ?? exportedName;
+              if (localAlias !== actualLocal) {
+                namedImportRenames.set(localAlias, actualLocal);
+              }
+            } else if (spec.type === "ImportDefaultSpecifier") {
+              const localAlias = spec.local.name;
+              const actualLocal = sharedExportToLocal.get("default") ?? "__shared_default__";
+              if (localAlias !== actualLocal) {
+                namedImportRenames.set(localAlias, actualLocal);
+              }
             }
           }
         }
@@ -369,6 +364,7 @@ function removeSharedImportsAndRewriteRefs(code, sharedChunkFileName, sharedExpo
         const source = importNode.source.value;
         if (typeof source === "string" && isSharedChunkSource(source, sharedChunkFileName)) {
           s.remove(importNode.start, importNode.end);
+          this.skip();
         }
       }
       if (node.type === "ExportNamedDeclaration") {
@@ -388,6 +384,7 @@ function removeSharedImportsAndRewriteRefs(code, sharedChunkFileName, sharedExpo
               }
             }
             s.overwrite(exportNode.start, exportNode.end, `export { ${exportParts.join(", ")} };`);
+            this.skip();
           }
         }
       }
@@ -397,6 +394,14 @@ function removeSharedImportsAndRewriteRefs(code, sharedChunkFileName, sharedExpo
           const propertyName = memberNode.property.name;
           const localName = sharedExportToLocal.get(propertyName) ?? propertyName;
           s.overwrite(memberNode.start, memberNode.end, localName);
+          this.skip();
+        }
+      }
+      if (node.type === "Identifier" && namedImportRenames.size > 0) {
+        const id = node;
+        const newName = namedImportRenames.get(id.name);
+        if (newName) {
+          s.overwrite(id.start, id.end, newName);
         }
       }
     }
@@ -494,7 +499,7 @@ function removeChunkImportsAndRewriteRefs(code, chunkFileName, exportToLocal, pa
 function chunkImportsFrom(chunk, sourceChunkFileName) {
   return chunk.imports.some((imp) => isChunkSource(imp, sourceChunkFileName));
 }
-function mergeUnsharedIntoImporters(unsharedChunk, entryChunks, parse, debug) {
+function mergeUnsharedIntoImporters(unsharedChunk, entryChunks, parse) {
   const { exports: unsharedExports, hasDefault } = extractExports(unsharedChunk.code, parse);
   const exportToLocal = /* @__PURE__ */ new Map();
   for (const exp of unsharedExports) {
@@ -534,16 +539,12 @@ function mergeUnsharedIntoImporters(unsharedChunk, entryChunks, parse, debug) {
       localExportToLocal,
       parse
     );
-    entry.code = [
-      debug && `// === Inlined from ${unsharedChunk.name} (duplicated by rollup-plugin-iife-split) ===`,
-      codeToInline.trim(),
-      "",
-      debug && "// === Entry code ===",
-      entryWithoutImports.trim()
-    ].filter((line) => line !== false).join("\n");
+    entry.code = `${codeToInline.trim()}
+
+${entryWithoutImports.trim()}`;
   }
 }
-function mergeSharedIntoPrimary(primaryChunk, sharedChunk, sharedProperty, neededExports, parse, debug) {
+function mergeSharedIntoPrimary(primaryChunk, sharedChunk, sharedProperty, neededExports, parse) {
   const { exports: sharedExports, hasDefault } = extractExports(sharedChunk.code, parse);
   const sharedDeclarations = extractTopLevelDeclarations(sharedChunk.code, parse);
   const primaryDeclarations = extractTopLevelDeclarations(primaryChunk.code, parse);
@@ -581,21 +582,30 @@ function mergeSharedIntoPrimary(primaryChunk, sharedChunk, sharedProperty, neede
   ];
   const sharedExportObject = `const ${sharedProperty} = { ${sharedExportEntries.join(", ")} };`;
   primaryChunk.code = [
-    debug && "// === Shared code (merged by rollup-plugin-iife-split) ===",
     strippedSharedCode.trim(),
     "",
-    debug && "// === Primary entry code ===",
     primaryWithoutSharedImports.trim(),
     "",
-    debug && "// === Shared exports object ===",
     sharedExportObject,
     `export { ${sharedProperty} };`
-  ].filter((line) => line !== false).join("\n");
+  ].join("\n");
 }
 
 // src/index.ts
+import { writeFileSync, mkdirSync } from "fs";
+import { join } from "path";
 function iifeSplit(options) {
-  const { primary, primaryGlobal, secondaryProps, sharedProp, unshared, debug } = options;
+  const { primary, primaryGlobal, secondaryProps, sharedProp, unshared, debugDir } = options;
+  const sanitizeName = (name) => name.replace(/[/\\]/g, "-");
+  const writeDebugFile = (filename, content) => {
+    if (!debugDir) return;
+    try {
+      mkdirSync(debugDir, { recursive: true });
+      writeFileSync(join(debugDir, filename), content);
+    } catch (e) {
+      console.warn(`[iife-split] Failed to write debug file ${filename}:`, e);
+    }
+  };
   let outputGlobals = {};
   const manualChunks = (id, { getModuleInfo }) => {
     const moduleInfo = getModuleInfo(id);
@@ -625,6 +635,18 @@ function iifeSplit(options) {
     async generateBundle(outputOptions, bundle) {
       const parse = this.parse.bind(this);
       const analysis = analyzeChunks(bundle, primary);
+      if (debugDir) {
+        writeDebugFile("1-primary-original.js", analysis.primaryChunk.code);
+        if (analysis.sharedChunk) {
+          writeDebugFile("1-shared-original.js", analysis.sharedChunk.code);
+        }
+        for (const satellite of analysis.satelliteChunks) {
+          writeDebugFile(`1-satellite-${sanitizeName(satellite.name)}-original.js`, satellite.code);
+        }
+        for (const unshared2 of analysis.unsharedChunks) {
+          writeDebugFile(`1-unshared-${sanitizeName(unshared2.name)}-original.js`, unshared2.code);
+        }
+      }
       const sharedChunkFileName = analysis.sharedChunk?.fileName ?? null;
       if (analysis.sharedChunk) {
         const neededExports = /* @__PURE__ */ new Set();
@@ -634,21 +656,32 @@ function iifeSplit(options) {
             neededExports.add(imp);
           }
         }
+        if (debugDir) {
+          writeDebugFile("2-needed-exports.json", JSON.stringify(Array.from(neededExports), null, 2));
+        }
         mergeSharedIntoPrimary(
           analysis.primaryChunk,
           analysis.sharedChunk,
           sharedProp,
           neededExports,
-          parse,
-          debug
+          parse
         );
         delete bundle[analysis.sharedChunk.fileName];
       }
+      if (debugDir) {
+        writeDebugFile("2-primary-after-shared-merge.js", analysis.primaryChunk.code);
+      }
       const allEntries = [analysis.primaryChunk, ...analysis.satelliteChunks];
       for (const unsharedChunk of analysis.unsharedChunks) {
-        mergeUnsharedIntoImporters(unsharedChunk, allEntries, parse, debug);
+        mergeUnsharedIntoImporters(unsharedChunk, allEntries, parse);
         delete bundle[unsharedChunk.fileName];
       }
+      if (debugDir) {
+        writeDebugFile("3-primary-before-iife.js", analysis.primaryChunk.code);
+        for (const satellite of analysis.satelliteChunks) {
+          writeDebugFile(`3-satellite-${sanitizeName(satellite.name)}-before-iife.js`, satellite.code);
+        }
+      }
       const conversions = [];
       conversions.push(
         convertToIife({
@@ -658,8 +691,7 @@ function iifeSplit(options) {
           sharedGlobalPath: null,
           // Primary doesn't need to import shared
           sharedChunkFileName: null,
-          parse,
-          debug
+          parse
         }).then((code) => {
           analysis.primaryChunk.code = code;
         })
@@ -680,8 +712,7 @@ function iifeSplit(options) {
             globals: outputGlobals,
             sharedGlobalPath: `${primaryGlobal}.${sharedProp}`,
             sharedChunkFileName,
-            parse,
-            debug
+            parse
           }).then((code) => {
             satellite.code = code;
           })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rollup-plugin-iife-split",
-  "version": "0.0.2",
+  "version": "0.0.3",
   "type": "module",
   "description": "Rollup plugin for intelligent IIFE code-splitting",
   "main": "dist/index.js",
@@ -30,11 +30,16 @@
     "bundle"
   ],
   "license": "MIT",
+  "homepage": "https://github.com/arshaw/rollup-plugin-iife-split",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/arshaw/rollup-plugin-iife-split.git"
+  },
   "scripts": {
     "clean": "rm -rf dist",
     "build": "tsup src/index.ts --format esm --dts",
     "typecheck": "tsc --noEmit",
-    "test": "vitest",
-    "test:run": "vitest run"
+    "test": "vitest run",
+    "test:dev": "vitest"
   }
 }