@stacksjs/bun-router 0.0.8 → 0.0.10

package/dist/response/response-factory.d.ts

@@ -39,7 +39,7 @@ export interface ResponseCookieOptions {
     sameSite?: 'strict' | 'lax' | 'none';
 }
 export interface ResponseFactory {
-    json: <T>(data: T, options?: JsonResponseOptions) => Response;
+    json: <T>(data: T, options?: JsonResponseOptions | ResponseStatus) => Response;
     noContent: (headers?: Record<string, string>) => Response;
     download: (filePath: string, filename?: string, headers?: Record<string, string>) => Promise<Response>;
     file: (filePath: string, headers?: Record<string, string>) => Promise<Response>;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@stacksjs/bun-router",
   "type": "module",
-  "version": "0.0.8",
+  "version": "0.0.10",
   "description": "A fast, type-safe router for Bun.",
   "author": "Chris Breuer <chris@stacksjs.org>",
   "license": "MIT",

package/src/development/typescript-utilities.ts

@@ -423,6 +423,10 @@ export type MiddlewareWithConfig<T> = (config: T) => MiddlewareFunction
     const responseType = options.responseType || 'Response'
     const paramsType = options.paramsType || 'Record<string, string>'
 
+    // pickier mis-parses the embedded `req:` token in this template literal
+    // as a real unused function parameter — disable on the next line so a
+    // pre-existing rule false positive doesn't block CI.
+    // eslint-disable-next-line pickier/no-unused-vars
     return `(req: ${requestType} & { params: ${paramsType} }) => Promise<${responseType}>`
   }
 

package/src/response/response-factory.ts

@@ -53,7 +53,7 @@ export interface ResponseCookieOptions {
 // ============================================================================
 
 export interface ResponseFactory {
-  json: <T>(data: T, options?: JsonResponseOptions) => Response
+  json: <T>(data: T, options?: JsonResponseOptions | ResponseStatus) => Response
   noContent: (headers?: Record<string, string>) => Response
   download: (filePath: string, filename?: string, headers?: Record<string, string>) => Promise<Response>
   file: (filePath: string, headers?: Record<string, string>) => Promise<Response>
@@ -89,10 +89,18 @@ export interface ResponseFactory {
  */
 export const response: ResponseFactory = {
   /**
-   * Create a JSON response with proper typing
-   */
-  json: <T>(data: T, options: JsonResponseOptions = {}): Response => {
-    const { status = 200, headers = {}, pretty = false } = options
+   * Create a JSON response with proper typing.
+   *
+   * The second argument may be either a status code (Hono/Express style) or
+   * a `JsonResponseOptions` bag. The number form exists because passing a
+   * status as the second arg is the most common pattern in other web
+   * frameworks, and the silent `response.json(data, 401)` → HTTP 200 trap
+   * (the number was being interpreted as `options` and discarded) caused
+   * real bugs.
+   */
+  json: <T>(data: T, options: JsonResponseOptions | ResponseStatus = {}): Response => {
+    const opts: JsonResponseOptions = typeof options === 'number' ? { status: options } : options
+    const { status = 200, headers = {}, pretty = false } = opts
     const body = pretty ? JSON.stringify(data, null, 2) : JSON.stringify(data)
 
     return new Response(body, {

package/src/router/server.ts

@@ -1,6 +1,7 @@
 import type { Server } from 'bun'
 import type { EnhancedRequest, HTTPMethod, ServerOptions } from '../types'
 import type { Router } from './router'
+import { RequestWithMacros } from '../request/macros'
 import { runWithRequest, setCurrentRequest } from '../request/context'
 
 /**
@@ -333,7 +334,39 @@ export function registerServerHandling(RouterClass: typeof Router): void {
           _cookiesToDelete: [],
         }) as unknown as EnhancedRequest
 
-        // Add cookie methods to the request
+        // Attach registered request macros (input, has, bearerToken, etc.) so
+        // they're available in middleware and handlers without manual setup.
+        // Applied *before* the cookie utility so the cookies macro doesn't
+        // overwrite the get/set/delete object form consumers depend on.
+        RequestWithMacros.applyMacros(enhancedReq)
+
+        // Attach the helpers consumers reach for that aren't shaped as
+        // generic input macros — `getParam`, the function-form `cookie`,
+        // the parsed-query `get` shorthand. These mirror the methods the
+        // bare `Router.enhanceRequest` (router.ts) attaches; without them
+        // here, requests that flow through the registered server.ts
+        // override are missing exactly the helpers that frameworks
+        // layered on bun-router (Stacks, etc.) treat as guaranteed.
+        ;(enhancedReq as any).getParam = <T = string>(name: string, defaultValue?: T): T | undefined => {
+          const value = params?.[name] as T | undefined
+          return value !== undefined ? value : defaultValue
+        }
+        ;(enhancedReq as any).cookie = (name: string, defaultValue?: string): string | null => {
+          const value = getCookies()[name]
+          return value !== undefined ? value : (defaultValue ?? null)
+        }
+        // `get(key)` — read query params (and only query params; the full
+        // input merge with body etc. is provided by the `input` macro).
+        // Skip if a macro already registered it under the same name.
+        if (typeof (enhancedReq as any).get !== 'function') {
+          ;(enhancedReq as any).get = <T = unknown>(key: string, defaultValue?: T): T | undefined => {
+            const url = new URL(req.url)
+            const value = url.searchParams.get(key) as T | null
+            return value !== null && value !== undefined ? value : defaultValue
+          }
+        }
+
+        // Add cookie methods to the request (overrides any macro named `cookies`).
         Object.assign(enhancedReq, { cookies: { ...getCookies(), ...cookies } })
 
         return enhancedReq