feat: use node-loader again (#103)

* feat: use node-loader again

* update CHANGELOG

* missing RSDW node-loader

* update docs

CHANGELOG.md

@@ -3,6 +3,7 @@
 ## [Unreleased]
 ### Changed
 - fix: partially for css modules #98
+- feat: use node-loader again #103
 
 ## [0.13.0] - 2023-07-06
 ### Changed

README.md

@@ -21,7 +21,7 @@ yarn create waku
 ```
 
 ```bash
-pnpm create waku # pnpm not working for now
+pnpm create waku
 ```
 
 ## Work in progress

package.json

@@ -15,6 +15,11 @@
       "require": "./dist/cjs/main.js",
       "default": "./dist/main.js"
     },
+    "./node-loader": {
+      "types": "./dist/node-loader.d.ts",
+      "require": "./dist/cjs/node-loader.js",
+      "default": "./dist/node-loader.js"
+    },
     "./config": {
       "types": "./dist/config.d.ts",
       "require": "./dist/cjs/config.js",

src/lib/builder.ts

@@ -59,11 +59,6 @@ const analyzeEntries = async (entriesFile: string) => {
         (id) => serverEntryFileSet.add(id)
       ),
     ],
-    ssr: {
-      noExternal: /^(?!node:)/,
-      // FIXME this is very adhoc.
-      external: ["react", "minimatch"],
-    },
     resolve: {
       conditions: ["react-server"],
     },
@@ -103,15 +98,6 @@ const buildServerBundle = async (
 ) => {
   const serverBuildOutput = await viteBuild({
     ...configFileConfig,
-    ssr: {
-      noExternal: Object.values(clientEntryFiles).map(
-        // FIXME this might not work with pnpm
-        (fname) =>
-          path
-            .relative(path.join(config.root, "node_modules"), fname)
-            .split("/")[0]!
-      ),
-    },
     resolve: {
       conditions: ["react-server"],
     },

src/lib/middleware/rsc/worker-api.ts

@@ -10,7 +10,14 @@ import type {
 } from "../../../server.js";
 
 const worker = new Worker(new URL("worker-impl.js", import.meta.url), {
-  execArgv: ["--conditions", "react-server"],
+  execArgv: [
+    "--experimental-loader",
+    "waku/node-loader",
+    "--experimental-loader",
+    "react-server-dom-webpack/node-loader",
+    "--conditions",
+    "react-server",
+  ],
 });
 
 export type BuildOutput = {

src/lib/middleware/rsc/worker-impl.ts

@@ -108,11 +108,6 @@ const vitePromise = viteCreateServer({
       parentPort!.postMessage(mesg);
     }),
   ],
-  ssr: {
-    noExternal: /^(?!node:)/,
-    // FIXME this is very adhoc.
-    external: ["react", "minimatch", "react-server-dom-webpack"],
-  },
   resolve: {
     conditions: ["react-server"],
   },

src/node-loader.ts

@@ -0,0 +1,11 @@
+export async function load(url: string, context: any, nextLoad: any) {
+  const result = await nextLoad(url, context, nextLoad);
+  if (result.format === "module") {
+    let { source } = result;
+    if (typeof source !== "string") {
+      source = source.toString();
+    }
+    return { ...result, source };
+  }
+  return result;
+}

website/src/routes/docs/installing.tsx

@@ -2,7 +2,7 @@ import { CodeBlock } from "../../components/Code.js";
 
 const code1 = `npm create waku@latest
 yarn create waku
-pnpm create waku # pnpm not working for now`;
+pnpm create waku`;
 
 export default function Layout() {
   return (