rwsdk 1.0.0-beta.54 → 1.0.0-beta.56

package/dist/lib/e2e/release.mjs

@@ -379,7 +379,7 @@ export async function runRelease(cwd, projectDir, resourceUniqueKey) {
         const stdout = result.stdout;
         // Extract deployment URL from output
         log("Extracting deployment URL from output");
-        const urlMatch = stdout.match(/https:\/\/([a-zA-Z0-9-]+)\.redwoodjs\.workers\.dev/);
+        const urlMatch = stdout.match(/https:\/\/([a-zA-Z0-9-]+)(?:\.[a-zA-Z0-9-]+)?\.workers\.dev/);
         if (!urlMatch || !urlMatch[0]) {
             log("ERROR: Could not extract deployment URL from release output");
             // Log more details about the output for debugging

package/dist/runtime/client/client.js

@@ -64,22 +64,7 @@ export const fetchTransport = (transportContext) => {
                 });
             }
         }
-        // If there's a response handler, check the response first
-        if (transportContext.handleResponse) {
-            const response = await fetchPromise;
-            const shouldContinue = transportContext.handleResponse(response);
-            if (!shouldContinue) {
-                return;
-            }
-            // Continue with the response if handler returned true
-            const streamData = createFromFetch(Promise.resolve(response), {
-                callServer: fetchCallServer,
-            });
-            if (source === "navigation" || source === "action") {
-                transportContext.setRscPayload(streamData);
-            }
-            const result = await streamData;
-            const rawActionResult = result.actionResult;
+        const processActionResponse = (rawActionResult) => {
             if (isActionResponse(rawActionResult)) {
                 const actionResponse = rawActionResult.__rw_action_response;
                 const handledByHook = transportContext.onActionResponse?.(actionResponse) === true;
@@ -90,34 +75,46 @@ export const fetchTransport = (transportContext) => {
                         window.location.href = location;
                         return undefined;
                     }
+                    if (actionResponse.status >= 400) {
+                        throw new Error(`Server function returned status ${actionResponse.status}`);
+                    }
                 }
                 return rawActionResult;
             }
             return rawActionResult;
+        };
+        // If there's a response handler, check the response first
+        if (transportContext.handleResponse) {
+            const response = await fetchPromise;
+            const shouldContinue = transportContext.handleResponse(response);
+            if (!shouldContinue) {
+                return undefined;
+            }
+            // Continue with the response if handler returned true
+            const streamData = createFromFetch(Promise.resolve(response), {
+                callServer: fetchCallServer,
+            });
+            if (source === "navigation" || source === "action") {
+                transportContext.setRscPayload(streamData);
+            }
+            const result = await streamData;
+            return processActionResponse(result.actionResult);
         }
         // Original behavior when no handler is present
-        const streamData = createFromFetch(fetchPromise, {
+        const response = await fetchPromise;
+        const location = response.headers.get("Location");
+        if (response.status >= 300 && response.status < 400 && location) {
+            window.location.href = location;
+            return undefined;
+        }
+        const streamData = createFromFetch(Promise.resolve(response), {
             callServer: fetchCallServer,
         });
         if (source === "navigation" || source === "action") {
             transportContext.setRscPayload(streamData);
         }
         const result = await streamData;
-        const rawActionResult = result.actionResult;
-        if (isActionResponse(rawActionResult)) {
-            const actionResponse = rawActionResult.__rw_action_response;
-            const handledByHook = transportContext.onActionResponse?.(actionResponse) === true;
-            if (!handledByHook) {
-                const location = actionResponse.headers["location"];
-                const isRedirect = actionResponse.status >= 300 && actionResponse.status < 400;
-                if (location && isRedirect) {
-                    window.location.href = location;
-                    return undefined;
-                }
-            }
-            return rawActionResult;
-        }
-        return rawActionResult;
+        return processActionResponse(result.actionResult);
     };
     return fetchCallServer;
 };

package/dist/runtime/client/navigation.js

@@ -122,6 +122,13 @@ export function initClientNavigation(opts = {}) {
         await globalThis.__rsc_callServer(null, null, "navigation");
     });
     function handleResponse(response) {
+        if (response.status >= 300 && response.status < 400) {
+            const location = response.headers.get("Location");
+            if (location) {
+                window.location.href = location;
+                return false;
+            }
+        }
         if (!response.ok) {
             // Redirect to the current page (window.location) to show the error
             // This means the page that produced the error is called twice.

package/dist/runtime/client/navigation.test.js

@@ -1,8 +1,31 @@
-import { describe, expect, it } from "vitest";
-import { validateClickEvent } from "./navigation";
+import { describe, expect, it, vi, beforeEach } from "vitest";
+import { validateClickEvent, initClientNavigation } from "./navigation";
+// Mocking browser globals
+vi.stubGlobal("window", {
+    location: { href: "http://localhost/" },
+    addEventListener: vi.fn(),
+    history: { scrollRestoration: "auto" },
+});
+vi.stubGlobal("document", {
+    addEventListener: vi.fn(),
+});
+vi.stubGlobal("history", {
+    scrollRestoration: "auto",
+});
+vi.stubGlobal("Headers", class {
+    constructor(init) {
+        this.map = {};
+        if (init && init.Location) {
+            this.map.location = init.Location;
+        }
+    }
+    get(name) {
+        return this.map[name.toLowerCase()] || null;
+    }
+});
 describe("clientNavigation", () => {
     let mockEvent = {
-        button: 0, // right click
+        button: 0,
         metaKey: false,
         altKey: false,
         shiftKey: false,
@@ -52,3 +75,30 @@ describe("clientNavigation", () => {
         })).toBe(true);
     });
 });
+describe("initClientNavigation", () => {
+    beforeEach(() => {
+        window.location.href = "http://localhost/";
+        vi.clearAllMocks();
+    });
+    it("handleResponse should follow redirects", () => {
+        const { handleResponse } = initClientNavigation();
+        const mockResponse = {
+            status: 302,
+            headers: new Headers({ Location: "/new-page" }),
+            ok: false,
+        };
+        const result = handleResponse(mockResponse);
+        expect(result).toBe(false);
+        expect(window.location.href).toBe("/new-page");
+    });
+    it("handleResponse should reload on error", () => {
+        const { handleResponse } = initClientNavigation();
+        const mockResponse = {
+            status: 500,
+            ok: false,
+        };
+        const result = handleResponse(mockResponse);
+        expect(result).toBe(false);
+        expect(window.location.href).toBe("http://localhost/");
+    });
+});

package/dist/runtime/server.d.ts

@@ -1,8 +1,8 @@
-type Interruptor<TArgs extends any[] = any[], TResult = any> = (context: {
+type Interruptor<TArgs extends any[] = any[]> = (context: {
     request: Request;
     ctx: Record<string, any>;
     args: TArgs;
-}) => Promise<Response | void | TResult> | Response | void | TResult;
+}) => Promise<Response | void> | Response | void;
 type ServerFunction<TArgs extends any[] = any[], TResult = any> = (...args: TArgs) => Promise<TResult>;
 type ServerFunctionOptions = {
     method?: "GET" | "POST";
@@ -29,7 +29,7 @@ type WrappedServerFunction<TArgs extends any[] = any[], TResult = any> = {
  * })
  * ```
  */
-export declare function serverQuery<TArgs extends any[] = any[], TResult = any>(fnsOrFn: ServerFunction<TArgs, TResult> | [...Interruptor<TArgs, TResult>[], ServerFunction<TArgs, TResult>], options?: ServerFunctionOptions): WrappedServerFunction<TArgs, TResult>;
+export declare function serverQuery<TArgs extends any[], TResult>(fnsOrFn: ServerFunction<TArgs, TResult> | [...Interruptor<TArgs>[], ServerFunction<TArgs, TResult>], options?: ServerFunctionOptions): WrappedServerFunction<TArgs, TResult>;
 /**
  * Wrap a function to be used as a server action.
  *
@@ -48,5 +48,5 @@ export declare function serverQuery<TArgs extends any[] = any[], TResult = any>(
  * })
  * ```
  */
-export declare function serverAction<TArgs extends any[] = any[], TResult = any>(fnsOrFn: ServerFunction<TArgs, TResult> | [...Interruptor<TArgs, TResult>[], ServerFunction<TArgs, TResult>], options?: ServerFunctionOptions): WrappedServerFunction<TArgs, TResult>;
+export declare function serverAction<TArgs extends any[], TResult>(fnsOrFn: ServerFunction<TArgs, TResult> | [...Interruptor<TArgs>[], ServerFunction<TArgs, TResult>], options?: ServerFunctionOptions): WrappedServerFunction<TArgs, TResult>;
 export {};

package/dist/runtime/server.js

@@ -6,18 +6,14 @@ function createServerFunction(fns, mainFn, options) {
         for (const fn of fns) {
             const result = await fn({ request, ctx, args });
             if (result instanceof Response) {
-                // We can't easily return a Response from a server action function
-                // because the return type is expected to be TResult.
-                // However, if the interruptor returns a Response, it usually means "stop and return this HTTP response".
-                // In the RSC context, throwing a Response is a common pattern to short-circuit.
-                throw result;
+                // We return the Response so it can be handled by the action handler
+                // and serialized into the RSC stream via normalizeActionResult.
+                return result;
             }
         }
         return mainFn(...args);
     };
-    wrapped.method = options?.method ?? "POST"; // Default to POST if not specified, though user said serverQuery defaults to GET?
-    // User said: "export const getProject = serverQuery(...) // Defaults to GET"
-    // So serverQuery defaults to GET, serverAction defaults to POST?
+    wrapped.method = options?.method ?? "POST";
     return wrapped;
 }
 /**

package/dist/vite/transformServerFunctions.test.mjs

@@ -119,6 +119,33 @@ export default serverQuery(async (id) => {
 export default serverAction(async (id) => {
   return { id, name: "Project X" };
 });
+`;
+    let SERVER_QUERY_ARRAY_CODE = `
+"use server";
+export const getProject = serverQuery([
+  auth,
+  async (id) => {
+    return { id, name: "Project X" };
+  }
+]);
+`;
+    let SERVER_ACTION_ARRAY_CODE = `
+"use server";
+export const upvote = serverAction([
+  auth,
+  async (id) => {
+    return { id, count: 1 };
+  }
+]);
+`;
+    let SERVER_QUERY_ARRAY_POST_CODE = `
+"use server";
+export const getProject = serverQuery([
+  auth,
+  async (id) => {
+    return { id, name: "Project X" };
+  }
+], { method: "POST" });
 `;
     const TEST_CASES = {
         COMMENT_CODE,
@@ -137,6 +164,9 @@ export default serverAction(async (id) => {
         SERVER_ACTION_CODE,
         SERVER_QUERY_DEFAULT_CODE,
         SERVER_ACTION_DEFAULT_CODE,
+        SERVER_QUERY_ARRAY_CODE,
+        SERVER_ACTION_ARRAY_CODE,
+        SERVER_QUERY_ARRAY_POST_CODE,
     };
     describe("TRANSFORMS", () => {
         for (const [key, CODE] of Object.entries(TEST_CASES)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rwsdk",
-  "version": "1.0.0-beta.54",
+  "version": "1.0.0-beta.56",
   "description": "Build fast, server-driven webapps on Cloudflare with SSR, RSC, and realtime",
   "type": "module",
   "bin": {
@@ -153,68 +153,68 @@
   "author": "RedwoodSDK <peter@redwoodjs.com>",
   "license": "MIT",
   "dependencies": {
-    "@ast-grep/napi": "~0.39.0",
-    "@cloudflare/workers-types": "~4.20260218.0",
+    "@ast-grep/napi": "~0.41.0",
+    "@cloudflare/workers-types": "~4.20260305.1",
     "@mdx-js/mdx": "~3.1.1",
-    "@puppeteer/browsers": "~2.10.0",
+    "@puppeteer/browsers": "~2.13.0",
     "@types/decompress": "~4.2.7",
     "@types/fs-extra": "~11.0.4",
-    "@types/glob": "^8.1.0",
+    "@types/glob": "^9.0.0",
     "@types/react": "~19.2.14",
     "@types/react-dom": "~19.2.3",
-    "@types/react-is": "~19.0.0",
-    "@vitejs/plugin-react": "~5.0.0",
-    "chokidar": "~4.0.0",
-    "debug": "~4.4.0",
+    "@types/react-is": "~19.2.0",
+    "@vitejs/plugin-react": "~5.1.4",
+    "chokidar": "~5.0.0",
+    "debug": "~4.4.3",
     "decompress": "~4.2.1",
-    "enhanced-resolve": "~5.18.1",
-    "eventsource-parser": "~3.0.0",
-    "execa": "~9.6.0",
+    "enhanced-resolve": "~5.20.0",
+    "eventsource-parser": "~3.0.6",
+    "execa": "~9.6.1",
     "find-up": "~8.0.0",
-    "fs-extra": "~11.3.0",
+    "fs-extra": "~11.3.4",
     "get-port": "^7.1.0",
-    "glob": "~11.1.0",
-    "ignore": "~7.0.4",
+    "glob": "~13.0.6",
+    "ignore": "~7.0.5",
     "jsonc-parser": "~3.3.1",
-    "kysely": "~0.28.2",
+    "kysely": "~0.28.11",
     "kysely-do": "~0.0.1-rc.1",
     "lodash": "~4.17.23",
-    "magic-string": "~0.30.17",
+    "magic-string": "~0.30.21",
     "picocolors": "~1.1.1",
     "proper-lockfile": "~4.1.2",
-    "puppeteer-core": "~24.22.0",
-    "react-is": "~19.1.0",
-    "rsc-html-stream": "~0.0.6",
+    "puppeteer-core": "~24.38.0",
+    "react-is": "~19.2.4",
+    "rsc-html-stream": "~0.0.7",
     "server-only": "^0.0.1",
     "tmp-promise": "~3.0.3",
-    "ts-morph": "~27.0.0",
+    "ts-morph": "~27.0.2",
     "unique-names-generator": "~4.7.1",
-    "vibe-rules": "~0.3.0",
-    "vite-tsconfig-paths": "~5.1.4"
+    "vibe-rules": "~0.3.91",
+    "vite-tsconfig-paths": "~6.1.1"
   },
   "peerDependencies": {
-    "@cloudflare/vite-plugin": "^1.25.1",
-    "capnweb": "~0.2.0",
+    "@cloudflare/vite-plugin": "^1.26.0",
+    "capnweb": "~0.5.0",
     "react": ">=19.2.0-0 <19.3.0 || >=19.3.0-0 <20.0.0",
     "react-dom": ">=19.2.0-0 <19.3.0 || >=19.3.0-0 <20.0.0",
     "react-server-dom-webpack": ">=19.2.0-0 <19.3.0 || >=19.3.0-0 <20.0.0",
     "vite": "^6.2.6 || 7.x",
-    "wrangler": "^4.66.0"
+    "wrangler": "^4.70.0"
   },
   "packageManager": "pnpm@10.28.2",
   "devDependencies": {
-    "@cloudflare/vite-plugin": "1.25.1",
-    "capnweb": "~0.2.0",
+    "@cloudflare/vite-plugin": "1.26.0",
+    "capnweb": "~0.5.0",
     "@types/debug": "~4.1.12",
     "@types/js-beautify": "~1.14.3",
-    "@types/lodash": "~4.17.16",
-    "@types/node": "~24.10.0",
+    "@types/lodash": "~4.17.24",
+    "@types/node": "~25.3.3",
     "@types/proper-lockfile": "~4.1.4",
     "js-beautify": "~1.15.4",
-    "semver": "~7.7.1",
+    "semver": "~7.7.4",
     "tsx": "~4.21.0",
-    "typescript": "~5.9.0",
+    "typescript": "~5.9.3",
     "vite": "~7.3.1",
-    "vitest": "~3.2.0"
+    "vitest": "~4.0.18"
   }
 }