bxo 0.0.5-dev.33 → 0.0.5-dev.34

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 
@@ -606,6 +607,7 @@ export default class BXO {
              ...(ctx.set.headers || {}),
              Location: location
            };
+           ctx.set.redirect = { location, status };
 
            // Prepare headers for immediate Response return (merging any existing headers)
            const responseHeaders: Record<string, string> = { ...ctx.set.headers };
@@ -691,13 +693,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 +726,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.34",
   "description": "A simple and lightweight web framework for Bun",
   "type": "module",
   "devDependencies": {