bxo 0.0.5-dev.33 → 0.0.5-dev.35

package/example.ts

@@ -0,0 +1,33 @@
+import BXO, { createCookie, redirect } from 'bxo';
+
+const app = new BXO();
+
+// 1) Explicit return redirect (302 default)
+app.get('/old', ({ redirect }) => {
+  return redirect('/new');
+});
+
+// 2) Implicit redirect (no return needed)
+app.get('/implicit', (ctx) => {
+  ctx.redirect('/new');
+});
+
+// 3) Custom status (303 after POST)
+app.post('/submit', (ctx) => {
+  // ...handle form...
+  return ctx.redirect('/thanks', 303);
+});
+
+// 4) Redirect with cookie
+app.get('/login', (ctx) => {
+  ctx.set.cookies = [
+    createCookie('sid', 'abc123', { httpOnly: true, path: '/' })
+  ];
+  ctx.redirect('/dashboard'); // no return required
+});
+
+// 5) Using top-level helper (outside ctx)
+app.get('/go-external', () => redirect('https://example.com', 307));
+
+await app.start(3000);
+console.log('Server running at http://localhost:3000');

package/index.ts

@@ -67,6 +67,7 @@ export type Context<TConfig extends RouteConfig = {}> = {
     status?: number;
     headers?: Record<string, string>;
     cookies?: Cookie[];
+    redirect?: { location: string; status?: number };
   };
   status: <T extends number>(
     code: TConfig['response'] extends StatusResponseSchema 
@@ -87,6 +88,7 @@ export type Context<TConfig extends RouteConfig = {}> = {
       ? InferZodType<TConfig['response']>
       : any;
   redirect: (location: string, status?: number) => Response;
+  clearRedirect: () => void;
   [key: string]: any;
 };
 
@@ -600,15 +602,14 @@ export default class BXO {
           return data;
         }) as any,
                  redirect: ((location: string, status: number = 302) => {
-           // Persist redirect intent on context so it also works without returning
-           ctx.set.status = status;
-           ctx.set.headers = {
-             ...(ctx.set.headers || {}),
-             Location: location
-           };
+           // Record redirect intent only; avoid mutating generic status/headers so it can be canceled later
+           ctx.set.redirect = { location, status };
 
-           // Prepare headers for immediate Response return (merging any existing headers)
-           const responseHeaders: Record<string, string> = { ...ctx.set.headers };
+           // Prepare headers for immediate Response return without persisting to ctx.set.headers
+           const responseHeaders: Record<string, string> = {
+             Location: location,
+             ...(ctx.set.headers || {})
+           };
 
            // Handle cookies if any are set on context
            if (ctx.set.cookies && ctx.set.cookies.length > 0) {
@@ -632,7 +633,23 @@ export default class BXO {
              status,
              headers: responseHeaders
            });
-         }) as any
+         }) as any,
+        clearRedirect: (() => {
+          // Clear explicit redirect intent
+          delete ctx.set.redirect;
+          // Remove any Location header if present
+          if (ctx.set.headers) {
+            for (const key of Object.keys(ctx.set.headers)) {
+              if (key.toLowerCase() === 'location') {
+                delete ctx.set.headers[key];
+              }
+            }
+          }
+          // Reset status if it is a redirect
+          if (typeof ctx.set.status === 'number' && ctx.set.status >= 300 && ctx.set.status < 400) {
+            delete ctx.set.status;
+          }
+        }) as any
       };
     } catch (validationError) {
       // Validation failed - return error response
@@ -691,13 +708,19 @@ export default class BXO {
 
       // If the handler did not return a response, but a redirect was configured via ctx.set,
       // automatically create a redirect Response so users can call ctx.redirect(...) or set headers without returning.
-      if ((response === undefined || response === null) 
-        && typeof ctx.set.status === 'number' 
-        && ctx.set.status >= 300 
-        && ctx.set.status < 400) {
-        const hasLocationHeader = !!(ctx.set.headers && Object.keys(ctx.set.headers).some(k => k.toLowerCase() === 'location'));
-        if (hasLocationHeader) {
-          let responseHeaders = ctx.set.headers ? { ...ctx.set.headers } : {};
+      const hasImplicitRedirectIntent = !!ctx.set.redirect 
+        || (typeof ctx.set.status === 'number' && ctx.set.status >= 300 && ctx.set.status < 400);
+      if ((response === undefined || response === null) && hasImplicitRedirectIntent) {
+        const locationFromHeaders = ctx.set.headers && Object.entries(ctx.set.headers).find(([k]) => k.toLowerCase() === 'location')?.[1];
+        const location = ctx.set.redirect?.location || locationFromHeaders;
+        if (location) {
+          // Build headers, ensuring Location is present
+          let responseHeaders: Record<string, string> = ctx.set.headers ? { ...ctx.set.headers } : {};
+          if (!Object.keys(responseHeaders).some(k => k.toLowerCase() === 'location')) {
+            responseHeaders['Location'] = location;
+          }
+          // Determine status precedence: redirect.status > set.status > 302
+          const status = ctx.set.redirect?.status ?? ctx.set.status ?? 302;
 
           // Handle cookies if any are set
           if (ctx.set.cookies && ctx.set.cookies.length > 0) {
@@ -718,7 +741,7 @@ export default class BXO {
           }
 
           return new Response(null, {
-            status: ctx.set.status,
+            status,
             headers: responseHeaders
           });
         }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "bxo",
   "module": "index.ts",
-  "version": "0.0.5-dev.33",
+  "version": "0.0.5-dev.35",
   "description": "A simple and lightweight web framework for Bun",
   "type": "module",
   "devDependencies": {