@pylonsync/functions 0.11.4 → 0.11.6

package/dist/ssr-client-bundler.d.ts

@@ -24,6 +24,14 @@ interface BundleClientMessage {
  * import into a loud build failure that names the offending importer.
  */
 export declare function assertNotServerOnly(specifier: string, importer: string): void;
+/**
+ * A module file → its project-relative, extension-less component path
+ * (`<cwd>/app/blog/page.tsx` → `app/blog/page`). Always "/"-separated, on
+ * Windows too: the result is a manifest key, the input to the SSR boundary
+ * walk, and is spliced into generated `import` specifiers, where a backslash
+ * is a string escape (`"../app\page"` resolves `../apppage`).
+ */
+export declare function componentPathFor(path: any, cwd: string, file: string): string;
 /**
  * Every route-level `loading` module under the app dir, as project-relative
  * component paths (`app/dashboard/loading`), sorted for a stable build.
@@ -108,6 +116,7 @@ export declare function buildClientBundle(appDirRel?: string): Promise<BuildOutp
  * project to need Tailwind installed.
  */
 export declare function buildTailwind(fs: any, path: any, cwd: string, outdir: string, appDirRel: string): Promise<string | null>;
+export declare function _doBuildInner(fs: any, path: any, cwd: string, appDirRel: string): Promise<BuildOutput>;
 /**
  * Return the bundle manifest. If a fresh manifest exists on disk,
  * use it (caching parse output across requests). Otherwise build

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pylonsync/functions",
-  "version": "0.11.4",
+  "version": "0.11.6",
   "description": "TypeScript function runtime for pylon — defines server-side queries, mutations, and actions.",
   "type": "module",
   "main": "src/index.ts",

package/src/ssr-client-bundler.test.ts

@@ -25,6 +25,8 @@ import {
   buildClientBundle,
   buildTailwind,
   assertNotServerOnly,
+  componentPathFor,
+  _doBuildInner,
   discoverLoadingModules,
   generateLoadingRegistry,
   type PylonBundleManifest,
@@ -234,6 +236,55 @@ describe("ssr-client-bundler (Phase 1.5e)", () => {
     }
   });
 
+  test("Windows separators: the build succeeds and every manifest path uses /", async () => {
+    // node:path on Windows returns "\"-separated relative paths. They used to
+    // reach the generated entries as `import Page from "../app\page"`, where
+    // `\p` is a string escape, so Bun.build failed with "Bundle failed" and
+    // every page shipped without hydration (#15).
+    tempDir = makeFixture(
+      {
+        "page.tsx": PAGE_BODY("Home"),
+        "blog/page.tsx": PAGE_BODY("Blog"),
+        "not-found.tsx": PAGE_BODY("Missing"),
+        "loading.tsx": LOADING_BODY,
+      },
+      { "layout.tsx": LAYOUT_BODY },
+    );
+    const winPath = {
+      ...path,
+      relative: (from: string, to: string) =>
+        path.relative(from, to).replace(/\//g, "\\"),
+    };
+
+    const { manifestPath } = await _doBuildInner(fs, winPath, tempDir, "app");
+    const manifest = JSON.parse(
+      fs.readFileSync(manifestPath, "utf8"),
+    ) as PylonBundleManifest;
+
+    expect(Object.keys(manifest.routes).sort()).toEqual([
+      "app/blog/page",
+      "app/not-found",
+      "app/page",
+    ]);
+    expect(manifest.outdir).toBe(".pylon/client-build");
+    for (const route of Object.values(manifest.routes)) {
+      expect(route.file).not.toContain("\\");
+      for (const imp of route.imports) expect(imp).not.toContain("\\");
+    }
+    const entry = fs.readFileSync(
+      path.join(tempDir, ".pylon", "client-entry-app__blog__page.tsx"),
+      "utf8",
+    );
+    expect(entry).toContain('import Page from "../app/blog/page";');
+    expect(entry).toContain('import L0 from "../app/layout";');
+  });
+
+  test("componentPathFor returns a /-separated path for win32 input", () => {
+    expect(
+      componentPathFor(path.win32, "C:\\proj", "C:\\proj\\app\\(shop)\\cart\\page.tsx"),
+    ).toBe("app/(shop)/cart/page");
+  });
+
   test("adding a route grows the manifest by one and stays small per-entry", async () => {
     tempDir = makeFixture(
       {

package/src/ssr-client-bundler.ts

@@ -141,6 +141,20 @@ export function assertNotServerOnly(specifier: string, importer: string): void {
   }
 }
 
+/**
+ * A module file → its project-relative, extension-less component path
+ * (`<cwd>/app/blog/page.tsx` → `app/blog/page`). Always "/"-separated, on
+ * Windows too: the result is a manifest key, the input to the SSR boundary
+ * walk, and is spliced into generated `import` specifiers, where a backslash
+ * is a string escape (`"../app\page"` resolves `../apppage`).
+ */
+export function componentPathFor(path: any, cwd: string, file: string): string {
+  return path
+    .relative(cwd, file)
+    .replace(/\.(tsx?|jsx?)$/, "")
+    .replace(/\\/g, "/");
+}
+
 /**
  * Synchronously walk the route dir (`<appDirRel>` under cwd, e.g.
  * `app` or `web/app`) and return one entry per discovered page, each
@@ -182,7 +196,7 @@ function discoverRoutes(
       .map((n: string) => path.join(dir, n))
       .find((p: string) => fs.existsSync(p));
     const nextLayouts = layoutHere
-      ? [...layouts, path.relative(cwd, layoutHere).replace(/\.(tsx?|jsx?)$/, "")]
+      ? [...layouts, componentPathFor(path, cwd, layoutHere)]
       : layouts;
     const pageHere = ["page.tsx", "page.ts", "page.jsx", "page.js"]
       .map((n: string) => path.join(dir, n))
@@ -190,7 +204,7 @@ function discoverRoutes(
     if (pageHere) {
       pages.push({
         segments: [...segments],
-        component: path.relative(cwd, pageHere).replace(/\.(tsx?|jsx?)$/, ""),
+        component: componentPathFor(path, cwd, pageHere),
         layouts: nextLayouts,
         // e.g. [] → "/", ["p","[slug]"] → "/p/[slug]".
         pattern: "/" + segments.join("/"),
@@ -207,7 +221,7 @@ function discoverRoutes(
       if (bHere) {
         pages.push({
           segments: [...segments],
-          component: path.relative(cwd, bHere).replace(/\.(tsx?|jsx?)$/, ""),
+          component: componentPathFor(path, cwd, bHere),
           layouts: nextLayouts,
         });
       }
@@ -255,9 +269,7 @@ export function discoverLoadingModules(
       .map((n: string) => path.join(dir, n))
       .find((p: string) => fs.existsSync(p));
     if (here) {
-      found.push(
-        path.relative(cwd, here).replace(/\.(tsx?|jsx?)$/, "").replace(/\\/g, "/"),
-      );
+      found.push(componentPathFor(path, cwd, here));
     }
     for (const e of entries) {
       if (!e.isDirectory()) continue;
@@ -1381,7 +1393,9 @@ export async function buildTailwind(
   return stylesName;
 }
 
-async function _doBuildInner(
+// Exported for the Windows-separator regression test (ssr-client-bundler.test.ts),
+// which injects a `path` whose `relative` returns "\"-separated paths.
+export async function _doBuildInner(
   fs: any,
   path: any,
   cwd: string,
@@ -1557,7 +1571,10 @@ async function _doBuildInner(
     //   - chunks (kind === "chunk") — looked up by basename when
     //     scanning entry files for static `import "./chunks/..."`
     //     specifiers.
-    const outdirRel = path.relative(cwd, outdir);
+    // Manifest paths are URL paths under `public_prefix` ("/"-separated). The
+    // runtime's /_pylon/build/ handler rejects any request with a backslash.
+    const toUrlPath = (p: string) => p.replace(/\\/g, "/");
+    const outdirRel = toUrlPath(path.relative(cwd, outdir));
     const entriesByStem = new Map<
       string,
       { absPath: string; relPath: string }
@@ -1568,7 +1585,7 @@ async function _doBuildInner(
     >();
     for (const o of result.outputs) {
       const absPath: string = o.path;
-      const relPath = path.relative(outdir, absPath);
+      const relPath = toUrlPath(path.relative(outdir, absPath));
       const base = path.basename(absPath);
       // Strip `-<hash>.js` to recover the entry source's stem
       // (e.g. `client-entry-app__hello__page`). The hash is