@ivogt/rsc-router 0.0.0-experimental.3 → 0.0.0-experimental.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ivogt/rsc-router",
-  "version": "0.0.0-experimental.3",
+  "version": "0.0.0-experimental.5",
   "type": "module",
   "description": "Type-safe RSC router with partial rendering support",
   "author": "Ivo Todorov",

package/src/vite/index.ts

@@ -37,33 +37,80 @@ export { exposeLocationStateId } from "./expose-location-state-id.ts";
  * exists in the tree.
  *
  * By excluding these modules, we ensure a single module instance is used everywhere.
+ *
+ * We include both the scoped package name (@ivogt/rsc-router) and the aliased paths
+ * (rsc-router) because Vite's optimizer runs before alias resolution.
+ */
+/**
+ * esbuild plugin to provide rsc-router:version virtual module during optimization.
+ * This is needed because esbuild runs during Vite's dependency optimization phase,
+ * before Vite's plugin system can handle virtual modules.
  */
+const versionEsbuildPlugin = {
+  name: "rsc-router-version",
+  setup(build: any) {
+    build.onResolve({ filter: /^rsc-router:version$/ }, (args: any) => ({
+      path: args.path,
+      namespace: "rsc-router-virtual",
+    }));
+    build.onLoad({ filter: /.*/, namespace: "rsc-router-virtual" }, () => ({
+      contents: `export const VERSION = "dev";`,
+      loader: "js",
+    }));
+  },
+};
+
+/**
+ * Shared esbuild options for dependency optimization.
+ * Includes the version stub plugin for all environments.
+ */
+const sharedEsbuildOptions = {
+  plugins: [versionEsbuildPlugin],
+};
+
 const RSC_ROUTER_EXCLUDE_DEPS = [
+  // Scoped package paths
   "@ivogt/rsc-router",
   "@ivogt/rsc-router/browser",
   "@ivogt/rsc-router/client",
+  "@ivogt/rsc-router/server",
+  "@ivogt/rsc-router/rsc",
+  "@ivogt/rsc-router/ssr",
   "@ivogt/rsc-router/internal/deps/browser",
   "@ivogt/rsc-router/internal/deps/html-stream-client",
+  "@ivogt/rsc-router/internal/deps/ssr",
+  "@ivogt/rsc-router/internal/deps/rsc",
+  // Aliased paths (before alias resolution)
+  "rsc-router/browser",
+  "rsc-router/client",
+  "rsc-router/server",
+  "rsc-router/rsc",
+  "rsc-router/ssr",
+  "rsc-router/internal/deps/browser",
+  "rsc-router/internal/deps/html-stream-client",
+  "rsc-router/internal/deps/ssr",
+  "rsc-router/internal/deps/rsc",
 ];
 
 /**
- * Plugin to transform CJS react-server-dom vendor file to ESM.
+ * Plugin to transform CJS react-server-dom vendor files to ESM.
  * The @vitejs/plugin-rsc package ships client.browser.js as CommonJS
- * which doesn't work in the browser. This transforms it to ESM re-exports.
+ * which doesn't work in the browser. This transforms both:
+ * 1. The entry point (client.browser.js) to re-export from the CJS file
+ * 2. The actual CJS file content to ESM syntax
  */
 function createCjsToEsmPlugin(): Plugin {
   return {
     name: "rsc-router:cjs-to-esm",
     enforce: "pre",
     transform(code, id) {
-      // Transform the client.browser.js file from CJS to ESM
-      // Match any path containing vendor/react-server-dom/client.browser.js
+      const cleanId = id.split("?")[0];
+
+      // Transform the client.browser.js entry point to re-export from CJS
       if (
-        id.includes("vendor/react-server-dom/client.browser.js") ||
-        id.includes("vendor\\react-server-dom\\client.browser.js")
+        cleanId.includes("vendor/react-server-dom/client.browser.js") ||
+        cleanId.includes("vendor\\react-server-dom\\client.browser.js")
       ) {
-        // The original file uses: module.exports = require('./cjs/...')
-        // Transform to ESM re-export from the development or production CJS file
         const isProd = process.env.NODE_ENV === "production";
         const cjsFile = isProd
           ? "./cjs/react-server-dom-webpack-client.browser.production.js"
@@ -74,6 +121,60 @@ function createCjsToEsmPlugin(): Plugin {
           map: null,
         };
       }
+
+      // Transform the actual CJS files to ESM
+      if (
+        (cleanId.includes("vendor/react-server-dom/cjs/") ||
+          cleanId.includes("vendor\\react-server-dom\\cjs\\")) &&
+        cleanId.includes("client.browser")
+      ) {
+        let transformed = code;
+
+        // Extract the license comment to preserve it
+        const licenseMatch = transformed.match(/^\/\*\*[\s\S]*?\*\//);
+        const license = licenseMatch ? licenseMatch[0] : "";
+        if (license) {
+          transformed = transformed.slice(license.length);
+        }
+
+        // Remove "use strict" and the conditional IIFE wrapper
+        // Pattern: "use strict"; "production" !== process.env.NODE_ENV && (function() { ... })();
+        transformed = transformed.replace(
+          /^\s*["']use strict["'];\s*["']production["']\s*!==\s*process\.env\.NODE_ENV\s*&&\s*\(function\s*\(\)\s*\{/,
+          ""
+        );
+
+        // Remove the closing of the conditional IIFE at the end: })();
+        transformed = transformed.replace(/\}\)\(\);?\s*$/, "");
+
+        // Replace require('react') and require('react-dom') with imports
+        // The pattern spans multiple lines with whitespace
+        transformed = transformed.replace(
+          /var\s+React\s*=\s*require\s*\(\s*["']react["']\s*\)\s*,[\s\n]+ReactDOM\s*=\s*require\s*\(\s*["']react-dom["']\s*\)\s*,/g,
+          'import React from "react";\nimport ReactDOM from "react-dom";\nvar '
+        );
+
+        // Transform exports.xyz = function() to export function xyz()
+        transformed = transformed.replace(
+          /exports\.(\w+)\s*=\s*function\s*\(/g,
+          "export function $1("
+        );
+
+        // Transform exports.xyz = value to export const xyz = value
+        transformed = transformed.replace(
+          /exports\.(\w+)\s*=/g,
+          "export const $1 ="
+        );
+
+        // Reconstruct with license at the top
+        transformed = license + "\n" + transformed;
+
+        return {
+          code: transformed,
+          map: null,
+        };
+      }
+
       return null;
     },
   };
@@ -484,6 +585,7 @@ export async function rscRouter(
           // This ensures the same Context instance is used by both browser entry and RSC proxy modules
           optimizeDeps: {
             exclude: RSC_ROUTER_EXCLUDE_DEPS,
+            esbuildOptions: sharedEsbuildOptions,
           },
           resolve: {
             alias: {
@@ -520,6 +622,7 @@ export async function rscRouter(
               optimizeDeps: {
                 include: ["rsc-html-stream/client"],
                 exclude: RSC_ROUTER_EXCLUDE_DEPS,
+                esbuildOptions: sharedEsbuildOptions,
               },
             },
             ssr: {
@@ -532,6 +635,7 @@ export async function rscRouter(
                 dedupe: ["react", "react-dom"],
               },
               // Pre-bundle SSR entry and React for proper module linking with childEnvironments
+              // Exclude rsc-router modules to ensure same Context instance
               optimizeDeps: {
                 entries: [finalEntries.ssr],
                 include: [
@@ -540,6 +644,14 @@ export async function rscRouter(
                   "react/jsx-runtime",
                   "rsc-html-stream/server",
                 ],
+                exclude: RSC_ROUTER_EXCLUDE_DEPS,
+                esbuildOptions: sharedEsbuildOptions,
+              },
+            },
+            rsc: {
+              // RSC environment also needs esbuild options for version virtual module
+              optimizeDeps: {
+                esbuildOptions: sharedEsbuildOptions,
               },
             },
           },
@@ -605,6 +717,7 @@ export async function rscRouter(
             // This ensures the same Context instance is used by both browser entry and RSC proxy modules
             optimizeDeps: {
               exclude: RSC_ROUTER_EXCLUDE_DEPS,
+              esbuildOptions: sharedEsbuildOptions,
             },
             resolve: {
               alias: {
@@ -639,6 +752,7 @@ export async function rscRouter(
                 // Always exclude rsc-router modules, conditionally add virtual entry
                 optimizeDeps: {
                   exclude: RSC_ROUTER_EXCLUDE_DEPS,
+                  esbuildOptions: sharedEsbuildOptions,
                   ...(useVirtualClient && {
                     // Tell Vite to scan the virtual entry for dependencies
                     entries: [VIRTUAL_IDS.browser],
@@ -652,6 +766,7 @@ export async function rscRouter(
                     // Pre-bundle React for SSR to ensure single instance
                     include: ["react", "react-dom/server.edge", "react/jsx-runtime"],
                     exclude: RSC_ROUTER_EXCLUDE_DEPS,
+                    esbuildOptions: sharedEsbuildOptions,
                   },
                 },
               }),
@@ -661,6 +776,7 @@ export async function rscRouter(
                     entries: [VIRTUAL_IDS.rsc],
                     // Pre-bundle React for RSC to ensure single instance
                     include: ["react", "react/jsx-runtime"],
+                    esbuildOptions: sharedEsbuildOptions,
                   },
                 },
               }),