@valentinkolb/ssr 0.6.1 → 0.7.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@valentinkolb/ssr",
-  "version": "0.6.1",
+  "version": "0.7.0",
   "description": "Minimal SSR framework for SolidJS and Bun",
   "type": "module",
   "main": "src/index.ts",

package/src/build.ts

@@ -20,8 +20,9 @@ export const buildIslands = async (options: {
   outdir: string;
   verbose: boolean;
   dev?: boolean;
+  external?: string[];
 }): Promise<void> => {
-  const { pattern, outdir, verbose, dev = false } = options;
+  const { pattern, outdir, verbose, dev = false, external } = options;
 
   const totalStart = performance.now();
 
@@ -77,6 +78,7 @@ export const buildIslands = async (options: {
     outdir,
     naming: { entry: "[name].js", chunk: "chunk-[hash].js" },
     target: "browser",
+    external,
     minify: !dev,
     splitting: true,
     sourcemap: dev ? "inline" : "none",
@@ -123,6 +125,51 @@ export const buildIslands = async (options: {
     ],
   });
 
+  // Workaround for Bun bundler bug: duplicate export statements in shared chunks
+  // when splitting: true. Scan chunk files and deduplicate export lines.
+  if (result.success) {
+    const chunkGlob = new Glob("chunk-*.js");
+    for await (const chunkFile of chunkGlob.scan({ cwd: outdir, absolute: true })) {
+      const src = await Bun.file(chunkFile).text();
+      // Collect all export blocks and deduplicate
+      const exportRegex = /^export\s*\{[^}]*\}\s*;?\s*$/gm;
+      const exportBlocks = [...src.matchAll(exportRegex)].map((m) => m[0]);
+      if (exportBlocks.length > 1) {
+        // Parse all exported names from all blocks
+        const seenNames = new Set<string>();
+        const uniqueBlocks: string[] = [];
+        for (const block of exportBlocks) {
+          const names =
+            block
+              .match(/\{([^}]*)\}/)?.[1]
+              ?.split(",")
+              .map((n) => n.trim())
+              .filter(Boolean) ?? [];
+          const newNames = names.filter((n) => !seenNames.has(n));
+          if (newNames.length > 0) {
+            uniqueBlocks.push(`export { ${newNames.join(", ")} };`);
+            newNames.forEach((n) => seenNames.add(n));
+          }
+        }
+        // Replace all export blocks with deduplicated ones
+        let fixed = src;
+        for (const block of exportBlocks) {
+          fixed = fixed.replace(block, "");
+        }
+        // Append single combined export before any debugId comment
+        const debugIdIdx = fixed.lastIndexOf("//# debugId=");
+        const combined = uniqueBlocks.join("\n") + "\n";
+        if (debugIdIdx !== -1) {
+          fixed = fixed.slice(0, debugIdIdx) + combined + fixed.slice(debugIdIdx);
+        } else {
+          fixed = fixed.trimEnd() + "\n" + combined;
+        }
+        await Bun.write(chunkFile, fixed);
+        if (verbose) console.log(`  Fixed duplicate exports in ${chunkFile.split("/").pop()}`);
+      }
+    }
+  }
+
   if (verbose) {
     console.log(`Bundle: ${fmt(performance.now() - bundleStart)}`);
     for (const c of components) {

package/src/index.ts

@@ -29,6 +29,8 @@ export type SsrOptions<T extends object = object> = {
   dev?: boolean;
   /** Enable verbose logging (default: true in prod, false in dev) */
   verbose?: boolean;
+  /** Modules to exclude from the island bundle (passed to Bun.build) */
+  external?: string[];
   /** HTML template function (optional, has default) */
   template?: (
     ctx: {
@@ -85,7 +87,7 @@ export type SsrResult<T extends object> = {
  * ```
  */
 export const createConfig = <T extends object = object>(options: SsrOptions<T> = {}): SsrResult<T> => {
-  const { dev = false, verbose, template } = options;
+  const { dev = false, verbose, external, template } = options;
 
   // Default template if none provided
   const htmlTemplate =
@@ -156,6 +158,7 @@ export const createConfig = <T extends object = object>(options: SsrOptions<T> =
             outdir: islandsOutdir,
             verbose: verbose ?? !dev,
             dev,
+            external,
           });
         };