@agent-native/core 0.90.8 → 0.90.9

package/corpus/README.md

@@ -27,5 +27,5 @@ rg -n "defineAction|useActionQuery" node_modules/@agent-native/core/corpus
 
 ## Generated Counts
 
-- core files: 2144
+- core files: 2145
 - template files: 4835

package/corpus/core/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @agent-native/core
 
+## 0.90.9
+
+### Patch Changes
+
+- a1db610: Fix Netlify single-template deploy previews by keeping Nitro's `preferStatic` true (so `/assets/*` is served from `dist`) and stripping the harmful default-function URL rewrite that is incompatible with `config.path: "/*"`.
+
 ## 0.90.8
 
 ### Patch Changes

package/corpus/core/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-native/core",
-  "version": "0.90.8",
+  "version": "0.90.9",
   "description": "Framework for agent-native application development — where AI agents and UI share SQL state, actions, and context",
   "homepage": "https://github.com/BuilderIO/agent-native#readme",
   "bugs": {

package/corpus/core/src/cli/deploy-build.ts

@@ -0,0 +1,50 @@
+import fs from "fs";
+import path from "path";
+
+export interface DeployPostBuildInvocation {
+  command: string;
+  args: string[];
+  scriptPath: string;
+}
+
+export function hasDeployPreset(env: NodeJS.ProcessEnv = process.env): boolean {
+  return Boolean(env.NITRO_PRESET?.trim());
+}
+
+export function resolveDeployPostBuildInvocation({
+  cliDir,
+  env = process.env,
+  findTsxBin,
+}: {
+  cliDir: string;
+  env?: NodeJS.ProcessEnv;
+  findTsxBin: () => string;
+}): DeployPostBuildInvocation | undefined {
+  const builtDeployBuild = path.resolve(cliDir, "../deploy/build.js");
+  if (fs.existsSync(builtDeployBuild)) {
+    return {
+      command: "node",
+      args: [builtDeployBuild],
+      scriptPath: builtDeployBuild,
+    };
+  }
+
+  const sourceDeployBuild = path.resolve(cliDir, "../deploy/build.ts");
+  if (fs.existsSync(sourceDeployBuild)) {
+    return {
+      command: findTsxBin(),
+      args: [sourceDeployBuild],
+      scriptPath: sourceDeployBuild,
+    };
+  }
+
+  if (hasDeployPreset(env)) {
+    throw new Error(
+      `[build] Deploy build script not found. Expected ${builtDeployBuild} ` +
+        `or ${sourceDeployBuild}; refusing to publish an incomplete ` +
+        `NITRO_PRESET=${env.NITRO_PRESET} build.`,
+    );
+  }
+
+  return undefined;
+}

package/corpus/core/src/cli/index.ts

@@ -8,6 +8,8 @@ import { fileURLToPath } from "url";
 
 import * as Sentry from "@sentry/node";
 
+import { resolveDeployPostBuildInvocation } from "./deploy-build.js";
+
 // Resolve version once at module scope — used by both --version and --help
 let _version = "unknown";
 try {
@@ -577,15 +579,18 @@ switch (command) {
       // `agent-native start` and for serverless presets.
       if (isReactRouterFramework()) {
         const __dirname = path.dirname(fileURLToPath(import.meta.url));
-        const deployBuild = path.resolve(__dirname, "../deploy/build.js");
-        if (fs.existsSync(deployBuild)) {
-          await runBuildStep("node", [deployBuild], {
+        const deployBuild = resolveDeployPostBuildInvocation({
+          cliDir: __dirname,
+          findTsxBin,
+        });
+        if (deployBuild) {
+          await runBuildStep(deployBuild.command, deployBuild.args, {
             label: "deploy-build",
             env: process.env,
           });
         } else {
           console.warn(
-            `[build] Deploy build script not found at ${deployBuild}. Skipping post-build step.`,
+            "[build] Deploy build script not found and no deploy preset is configured. Skipping post-build step.",
           );
         }
       }

package/corpus/core/src/deploy/build.ts

@@ -2427,6 +2427,16 @@ export const config = {
   );
 }
 
+/**
+ * Nitro's Netlify preset can emit a harmful fallback rewrite to
+ * `/.netlify/functions/server`. With `config.path: "/*"`, that default URL is
+ * removed, so the rewrite publishes platform 404s. Single-template deploys keep
+ * Nitro's `preferStatic: true` so hashed `/assets/*` files in dist win before
+ * the SSR catch-all runs.
+ */
+const NETLIFY_DEFAULT_FUNCTION_URL_REDIRECT =
+  "/* /.netlify/functions/server 200";
+
 export function assertSingleTemplateNetlifyBuildOutput(
   projectCwd: string,
 ): void {
@@ -2440,21 +2450,31 @@ export function assertSingleTemplateNetlifyBuildOutput(
 
   if (!fs.existsSync(publishDir)) {
     failures.push("missing publish directory: dist");
-  } else if (!fs.existsSync(redirectsPath)) {
-    failures.push("missing Netlify fallback redirects file: dist/_redirects");
   } else {
+    const assetsDir = path.join(publishDir, "assets");
+    if (
+      !fs.existsSync(assetsDir) ||
+      fs.readdirSync(assetsDir).every((name) => name.startsWith("."))
+    ) {
+      failures.push(
+        "dist/assets is missing hashed client assets — the publish dir would load an infinite spinner",
+      );
+    }
+  }
+
+  if (fs.existsSync(publishDir) && fs.existsSync(redirectsPath)) {
     const redirects = fs.readFileSync(redirectsPath, "utf-8");
     if (
-      !redirects
+      redirects
         .split(/\r?\n/)
         .some(
           (line) =>
             line.trim().replace(/\s+/g, " ") ===
-            "/* /.netlify/functions/server 200",
+            NETLIFY_DEFAULT_FUNCTION_URL_REDIRECT,
         )
     ) {
       failures.push(
-        'dist/_redirects is missing "/* /.netlify/functions/server 200"',
+        'dist/_redirects must not contain "/* /.netlify/functions/server 200" — Nitro\'s custom config.path: "/*" removes that default function URL',
       );
     }
   }
@@ -2492,6 +2512,11 @@ export function assertSingleTemplateNetlifyBuildOutput(
         "Netlify server entry does not reference the generated main.mjs bundle",
       );
     }
+    if (!/\bpreferStatic:\s*true\b/.test(serverEntry)) {
+      failures.push(
+        "Netlify server entry must keep preferStatic: true so /assets/* is served from dist before the SSR catch-all",
+      );
+    }
   }
 
   if (isDurableBackgroundDeployEnabled()) {
@@ -2544,37 +2569,49 @@ export function assertSingleTemplateNetlifyBuildOutput(
   );
 }
 
+/**
+ * Strip the harmful single-template catch-all rewrite that points at
+ * `/.netlify/functions/server`. Nitro declares `config.path: "/*"`, which
+ * removes the default function URL, so rewriting to that URL publishes
+ * Netlify platform 404s. Preserve any real redirects from `public/_redirects`.
+ */
 export function writeSingleTemplateNetlifyRedirects(projectCwd: string): void {
   const publishDir = path.join(projectCwd, "dist");
-  if (!fs.existsSync(publishDir)) return;
-
   const redirectsPath = path.join(publishDir, "_redirects");
-  const existing = fs.existsSync(redirectsPath)
-    ? fs.readFileSync(redirectsPath, "utf-8").trimEnd()
-    : "";
-  const lines = existing ? existing.split(/\r?\n/) : [];
-
-  function hasRedirect(line: string): boolean {
-    return lines.some(
-      (existingLine) => existingLine.trim().replace(/\s+/g, " ") === line,
-    );
-  }
+  if (!fs.existsSync(redirectsPath)) return;
 
-  const fallbackRules = [
-    "# Generated by agent-native build for Netlify single-template deploys",
-    "# Static files are served first; dynamic routes fall through to the server function.",
-    "/* /.netlify/functions/server 200",
-  ];
+  const existing = fs.readFileSync(redirectsPath, "utf-8");
+  const kept: string[] = [];
+  let removed = 0;
 
-  for (const line of fallbackRules) {
-    if (line.startsWith("#") || !hasRedirect(line)) {
-      lines.push(line);
+  for (const line of existing.split(/\r?\n/)) {
+    const normalized = line.trim().replace(/\s+/g, " ");
+    if (
+      normalized === NETLIFY_DEFAULT_FUNCTION_URL_REDIRECT ||
+      normalized ===
+        "# Generated by agent-native build for Netlify single-template deploys" ||
+      normalized ===
+        "# Static files are served first; dynamic routes fall through to the server function."
+    ) {
+      removed += 1;
+      continue;
     }
+    kept.push(line);
+  }
+
+  while (kept.length > 0 && kept[kept.length - 1].trim() === "") {
+    kept.pop();
   }
 
-  fs.writeFileSync(redirectsPath, lines.join("\n") + "\n");
+  if (removed === 0) return;
+
+  if (kept.every((line) => line.trim() === "")) {
+    fs.rmSync(redirectsPath, { force: true });
+  } else {
+    fs.writeFileSync(redirectsPath, kept.join("\n").trimEnd() + "\n");
+  }
   console.log(
-    "[deploy] Wrote Netlify fallback redirect /* -> /.netlify/functions/server.",
+    '[deploy] Removed Netlify fallback rewrite to /.netlify/functions/server (incompatible with Nitro config.path: "/*").',
   );
 }
 

package/dist/cli/deploy-build.d.ts

@@ -0,0 +1,12 @@
+export interface DeployPostBuildInvocation {
+    command: string;
+    args: string[];
+    scriptPath: string;
+}
+export declare function hasDeployPreset(env?: NodeJS.ProcessEnv): boolean;
+export declare function resolveDeployPostBuildInvocation({ cliDir, env, findTsxBin, }: {
+    cliDir: string;
+    env?: NodeJS.ProcessEnv;
+    findTsxBin: () => string;
+}): DeployPostBuildInvocation | undefined;
+//# sourceMappingURL=deploy-build.d.ts.map

package/dist/cli/deploy-build.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"deploy-build.d.ts","sourceRoot":"","sources":["../../src/cli/deploy-build.ts"],"names":[],"mappings":"AAGA,MAAM,WAAW,yBAAyB;IACxC,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,EAAE,CAAC;IACf,UAAU,EAAE,MAAM,CAAC;CACpB;AAED,wBAAgB,eAAe,CAAC,GAAG,GAAE,MAAM,CAAC,UAAwB,GAAG,OAAO,CAE7E;AAED,wBAAgB,gCAAgC,CAAC,EAC/C,MAAM,EACN,GAAiB,EACjB,UAAU,GACX,EAAE;IACD,MAAM,EAAE,MAAM,CAAC;IACf,GAAG,CAAC,EAAE,MAAM,CAAC,UAAU,CAAC;IACxB,UAAU,EAAE,MAAM,MAAM,CAAC;CAC1B,GAAG,yBAAyB,GAAG,SAAS,CA4BxC"}

package/dist/cli/deploy-build.js

@@ -0,0 +1,30 @@
+import fs from "fs";
+import path from "path";
+export function hasDeployPreset(env = process.env) {
+    return Boolean(env.NITRO_PRESET?.trim());
+}
+export function resolveDeployPostBuildInvocation({ cliDir, env = process.env, findTsxBin, }) {
+    const builtDeployBuild = path.resolve(cliDir, "../deploy/build.js");
+    if (fs.existsSync(builtDeployBuild)) {
+        return {
+            command: "node",
+            args: [builtDeployBuild],
+            scriptPath: builtDeployBuild,
+        };
+    }
+    const sourceDeployBuild = path.resolve(cliDir, "../deploy/build.ts");
+    if (fs.existsSync(sourceDeployBuild)) {
+        return {
+            command: findTsxBin(),
+            args: [sourceDeployBuild],
+            scriptPath: sourceDeployBuild,
+        };
+    }
+    if (hasDeployPreset(env)) {
+        throw new Error(`[build] Deploy build script not found. Expected ${builtDeployBuild} ` +
+            `or ${sourceDeployBuild}; refusing to publish an incomplete ` +
+            `NITRO_PRESET=${env.NITRO_PRESET} build.`);
+    }
+    return undefined;
+}
+//# sourceMappingURL=deploy-build.js.map

package/dist/cli/deploy-build.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"deploy-build.js","sourceRoot":"","sources":["../../src/cli/deploy-build.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,IAAI,CAAC;AACpB,OAAO,IAAI,MAAM,MAAM,CAAC;AAQxB,MAAM,UAAU,eAAe,CAAC,GAAG,GAAsB,OAAO,CAAC,GAAG;IAClE,OAAO,OAAO,CAAC,GAAG,CAAC,YAAY,EAAE,IAAI,EAAE,CAAC,CAAC;AAC3C,CAAC;AAED,MAAM,UAAU,gCAAgC,CAAC,EAC/C,MAAM,EACN,GAAG,GAAG,OAAO,CAAC,GAAG,EACjB,UAAU,GAKX;IACC,MAAM,gBAAgB,GAAG,IAAI,CAAC,OAAO,CAAC,MAAM,EAAE,oBAAoB,CAAC,CAAC;IACpE,IAAI,EAAE,CAAC,UAAU,CAAC,gBAAgB,CAAC,EAAE,CAAC;QACpC,OAAO;YACL,OAAO,EAAE,MAAM;YACf,IAAI,EAAE,CAAC,gBAAgB,CAAC;YACxB,UAAU,EAAE,gBAAgB;SAC7B,CAAC;IACJ,CAAC;IAED,MAAM,iBAAiB,GAAG,IAAI,CAAC,OAAO,CAAC,MAAM,EAAE,oBAAoB,CAAC,CAAC;IACrE,IAAI,EAAE,CAAC,UAAU,CAAC,iBAAiB,CAAC,EAAE,CAAC;QACrC,OAAO;YACL,OAAO,EAAE,UAAU,EAAE;YACrB,IAAI,EAAE,CAAC,iBAAiB,CAAC;YACzB,UAAU,EAAE,iBAAiB;SAC9B,CAAC;IACJ,CAAC;IAED,IAAI,eAAe,CAAC,GAAG,CAAC,EAAE,CAAC;QACzB,MAAM,IAAI,KAAK,CACb,mDAAmD,gBAAgB,GAAG;YACpE,MAAM,iBAAiB,sCAAsC;YAC7D,gBAAgB,GAAG,CAAC,YAAY,SAAS,CAC5C,CAAC;IACJ,CAAC;IAED,OAAO,SAAS,CAAC;AACnB,CAAC","sourcesContent":["import fs from \"fs\";\nimport path from \"path\";\n\nexport interface DeployPostBuildInvocation {\n  command: string;\n  args: string[];\n  scriptPath: string;\n}\n\nexport function hasDeployPreset(env: NodeJS.ProcessEnv = process.env): boolean {\n  return Boolean(env.NITRO_PRESET?.trim());\n}\n\nexport function resolveDeployPostBuildInvocation({\n  cliDir,\n  env = process.env,\n  findTsxBin,\n}: {\n  cliDir: string;\n  env?: NodeJS.ProcessEnv;\n  findTsxBin: () => string;\n}): DeployPostBuildInvocation | undefined {\n  const builtDeployBuild = path.resolve(cliDir, \"../deploy/build.js\");\n  if (fs.existsSync(builtDeployBuild)) {\n    return {\n      command: \"node\",\n      args: [builtDeployBuild],\n      scriptPath: builtDeployBuild,\n    };\n  }\n\n  const sourceDeployBuild = path.resolve(cliDir, \"../deploy/build.ts\");\n  if (fs.existsSync(sourceDeployBuild)) {\n    return {\n      command: findTsxBin(),\n      args: [sourceDeployBuild],\n      scriptPath: sourceDeployBuild,\n    };\n  }\n\n  if (hasDeployPreset(env)) {\n    throw new Error(\n      `[build] Deploy build script not found. Expected ${builtDeployBuild} ` +\n        `or ${sourceDeployBuild}; refusing to publish an incomplete ` +\n        `NITRO_PRESET=${env.NITRO_PRESET} build.`,\n    );\n  }\n\n  return undefined;\n}\n"]}

package/dist/cli/index.js

@@ -5,6 +5,7 @@ import { createRequire } from "module";
 import path from "path";
 import { fileURLToPath } from "url";
 import * as Sentry from "@sentry/node";
+import { resolveDeployPostBuildInvocation } from "./deploy-build.js";
 // Resolve version once at module scope — used by both --version and --help
 let _version = "unknown";
 try {
@@ -504,15 +505,18 @@ switch (command) {
             // `agent-native start` and for serverless presets.
             if (isReactRouterFramework()) {
                 const __dirname = path.dirname(fileURLToPath(import.meta.url));
-                const deployBuild = path.resolve(__dirname, "../deploy/build.js");
-                if (fs.existsSync(deployBuild)) {
-                    await runBuildStep("node", [deployBuild], {
+                const deployBuild = resolveDeployPostBuildInvocation({
+                    cliDir: __dirname,
+                    findTsxBin,
+                });
+                if (deployBuild) {
+                    await runBuildStep(deployBuild.command, deployBuild.args, {
                         label: "deploy-build",
                         env: process.env,
                     });
                 }
                 else {
-                    console.warn(`[build] Deploy build script not found at ${deployBuild}. Skipping post-build step.`);
+                    console.warn("[build] Deploy build script not found and no deploy preset is configured. Skipping post-build step.");
                 }
             }
             console.log("\nBuild complete.");