@nextrush/router 3.0.7 → 4.0.0-beta.0

package/dist/index.d.ts

@@ -1,230 +1,186 @@
-import { RouterOptions, RouteHandler, HttpMethod, Middleware, RouteMatch, Context } from '@nextrush/types';
+import { RouteHandler, Middleware, MetadataContribution, RouteMetaMarker, RouterOptions, RouteEntry, HttpMethod, RouteDefinition, RouteMatch, Context } from '@nextrush/types';
 export { HttpMethod, Middleware, Route, RouteHandler, RouteMatch, Router as RouterInterface, RouterOptions } from '@nextrush/types';
 
 /**
- * @nextrush/router - Router Implementation
+ * @nextrush/router - Redirect Handler Compilation
  *
- * High-performance router using a segment trie for route matching.
- * Routes are keyed by full path segments (e.g. "users", ":id"), not by
- * individual characters — this is a segment-based trie, not a compressed
- * radix tree. Supports parameters, wildcards, and method-based routing.
+ * Single, shared redirect implementation used by both `Router.redirect()` and
+ * route groups (audit RT-4). Previously the group variant used a naive
+ * `replaceAll(':key', value)` that could mis-substitute overlapping param names
+ * and re-scanned the string per request; this precompiles the target template
+ * once and only substitutes route-style `:param` slots (a `:` at position 0 or
+ * preceded by `/`), so `https://` and other literal colons are never touched.
  *
  * @packageDocumentation
  */
 
+/** HTTP redirect status codes supported by `redirect()`. */
+type RedirectStatus = 301 | 302 | 303 | 307 | 308;
+
+/**
+ * @nextrush/router - Route Groups
+ *
+ * A route group applies a shared path prefix and middleware to a set of routes
+ * registered inside a callback. Extracted from `router.ts` (audit RT-3) and
+ * given a real public type (audit RT-6): `router.group()` callbacks receive a
+ * {@link RouteGroup}, not a mis-cast `Router`.
+ *
+ * @packageDocumentation
+ */
+
+/**
+ * The object passed to a `router.group(prefix, callback)` callback.
+ *
+ * @remarks
+ * Exposes only the route-registration surface that is valid inside a group —
+ * intentionally NOT the full {@link Router} (no `mount`/`use`/`reset`), which is
+ * why the previous `as unknown as Router` cast was a lie (audit RT-6).
+ */
+interface RouteGroup {
+    get(path: string, ...handlers: RouteHandler[]): this;
+    post(path: string, ...handlers: RouteHandler[]): this;
+    put(path: string, ...handlers: RouteHandler[]): this;
+    delete(path: string, ...handlers: RouteHandler[]): this;
+    patch(path: string, ...handlers: RouteHandler[]): this;
+    head(path: string, ...handlers: RouteHandler[]): this;
+    options(path: string, ...handlers: RouteHandler[]): this;
+    all(path: string, ...handlers: RouteHandler[]): this;
+    redirect(from: string, to: string, status?: RedirectStatus): this;
+    group(prefix: string, middlewareOrCallback: Middleware[] | ((router: RouteGroup) => void), callback?: (router: RouteGroup) => void): this;
+}
+
 /**
- * Router class — high-performance segment trie router
+ * @nextrush/router - Route Metadata Contributions
  *
- * Routes are indexed by path segment, giving O(d) lookup where d is the
- * number of segments. Static routes are additionally stored in a hash map
- * for O(1) fast-path lookup.
+ * Inline route metadata (`endpoint()`) and the registration-time merge of
+ * contributions from `validate()`, `endpoint()`, etc. Extracted from `router.ts`
+ * (audit RT-3). None of this is on the request hot path — it is read only at
+ * registration and by `getRoutes()` at doc-generation time.
+ *
+ * @packageDocumentation
+ */
+
+/**
+ * Declare route metadata inline in a route's argument list. Returns a pure data
+ * marker (not a middleware) — the router reads its metadata at registration and
+ * never executes it per request.
  *
  * @example
  * ```typescript
- * const router = createRouter();
+ * router.post('/users',
+ *   validate(User),
+ *   endpoint({ summary: 'Create a user', responses: { 201: UserResponse } }),
+ *   handler,
+ * );
+ * ```
+ */
+declare function endpoint(metadata: MetadataContribution): RouteMetaMarker;
+
+/**
+ * @nextrush/router - Router Implementation
  *
- * router.get('/users', listUsers);
- * router.get('/users/:id', getUser);
- * router.post('/users', createUser);
+ * The public `Router` shell: a thin, chainable facade delegating registration,
+ * matching, dispatch, composition, and grouping to focused sibling modules
+ * (design.md D1/D2). Segment trie keyed by whole path segments, not a radix tree.
  *
- * app.use(router.routes());
- * ```
+ * @packageDocumentation
+ */
+
+/**
+ * High-performance segment-trie router: O(d) lookup by path segment, with a
+ * static-route hash map for an O(1) fast path.
+ * @see {@link https://github.com/0xTanzim/nextRush/blob/main/packages/router/README.md | @nextrush/router README}
  */
 declare class Router {
     private readonly root;
     private readonly opts;
     private readonly routerMiddleware;
+    /** Static-route fast path: method-nested map for O(1) lookup with no per-request key string (HP-9). */
+    private readonly staticRoutes;
     /**
-     * Static route hash map for O(1) lookup.
-     * Key: "METHOD path" (e.g. "GET /users"), Value: HandlerEntry
+     * Introspection registry, kept SEPARATE from the hot-path trie/staticRoutes
+     * so request dispatch never reads metadata — only getRoutes() touches it.
      */
-    private readonly staticRoutes;
+    private readonly routeDefinitions;
     /** Whether any routes have params or wildcards (disables static-only fast path) */
     private hasParamRoutes;
+    /** Whether router-level middleware has already been sealed into executors (audit RT-7) */
+    private _sealed;
+    /** Memoized state the extracted registration/matching functions read (see {@link createRouterState}). */
+    private readonly state;
     constructor(options?: RouterOptions);
     /**
-     * Normalize path based on router options
-     */
-    private normalizePath;
-    /**
-     * Add a route to the radix tree
+     * Validate + normalize a raw path, then delegate trie insertion to the
+     * extracted `addRoute` (design.md D2); flips `hasParamRoutes` from its return.
      */
     private addRoute;
-    get(path: string, ...handlers: RouteHandler[]): this;
-    post(path: string, ...handlers: RouteHandler[]): this;
-    put(path: string, ...handlers: RouteHandler[]): this;
-    delete(path: string, ...handlers: RouteHandler[]): this;
-    patch(path: string, ...handlers: RouteHandler[]): this;
-    head(path: string, ...handlers: RouteHandler[]): this;
-    options(path: string, ...handlers: RouteHandler[]): this;
-    all(path: string, ...handlers: RouteHandler[]): this;
-    route(method: HttpMethod, path: string, ...handlers: RouteHandler[]): this;
+    get(path: string, ...entries: RouteEntry[]): this;
+    post(path: string, ...entries: RouteEntry[]): this;
+    put(path: string, ...entries: RouteEntry[]): this;
+    delete(path: string, ...entries: RouteEntry[]): this;
+    patch(path: string, ...entries: RouteEntry[]): this;
+    head(path: string, ...entries: RouteEntry[]): this;
+    options(path: string, ...entries: RouteEntry[]): this;
     /**
-     * Register a redirect route from one path to another
-     *
-     * @param from - Source path to redirect from
-     * @param to - Target path or URL to redirect to
-     * @param status - HTTP status code (default: 301 permanent redirect)
-     * @returns this for chaining
-     *
-     * @example
-     * ```typescript
-     * // Permanent redirect (301)
-     * router.redirect('/old-page', '/new-page');
-     *
-     * // Temporary redirect (302)
-     * router.redirect('/temp', '/destination', 302);
-     *
-     * // Redirect to external URL
-     * router.redirect('/docs', 'https://docs.example.com');
-     *
-     * // With parameters - redirects /users/:id to /profiles/:id
-     * router.redirect('/users/:id', '/profiles/:id');
-     * ```
+     * Register a route for every HTTP method under one consolidated `isAnyMethod`
+     * introspection row (T016) — matching is unchanged (see {@link pushAnyMethodDefinition}).
      */
-    redirect(from: string, to: string, status?: 301 | 302 | 303 | 307 | 308): this;
-    use(pathOrMiddleware: string | Middleware | Router, routerOrUndefined?: Router): this;
+    all(path: string, ...entries: RouteEntry[]): this;
+    route(method: HttpMethod, path: string, ...entries: RouteEntry[]): this;
     /**
-     * Mount a sub-router at a path prefix (Hono-style)
-     *
-     * This is the explicit API for mounting sub-routers.
-     * Equivalent to `router.use(path, subRouter)` but more semantic.
-     *
-     * @param path - Path prefix for the sub-router
-     * @param router - Router instance to mount
-     * @returns this for chaining
-     *
-     * @example
-     * ```typescript
-     * // Create modular routers
-     * const users = createRouter();
-     * users.get('/', listUsers);
-     * users.get('/:id', getUser);
-     *
-     * const posts = createRouter();
-     * posts.get('/', listPosts);
-     *
-     * // Mount sub-routers
-     * const api = createRouter();
-     * api.mount('/users', users);
-     * api.mount('/posts', posts);
-     *
-     * // Or use on main router
-     * const router = createRouter();
-     * router.mount('/api', api);
-     *
-     * app.use(router.routes());
-     * ```
+     * Every registered route as a read-only list, for renderers (`@nextrush/openapi`,
+     * SDK/RPC generators). Doc-generation-time only — never on the request path.
      */
-    mount(path: string, router: Router): this;
+    getRoutes(): readonly RouteDefinition[];
     /**
-     * Mount a sub-router (internal)
-     *
-     * Carries the sub-router's own `routerMiddleware` forward so that
-     * `subrouter.use(mw)` middleware applies to every copied route.
+     * Register a redirect from one path to another (301 by default). 307/308
+     * additionally register POST/PUT/PATCH/DELETE to preserve the method.
+     * @see {@link https://github.com/0xTanzim/nextRush/blob/main/packages/router/README.md#redirects | README: Redirects}
      */
-    private mountRouter;
-    /**
-     * Recursively copy routes from another router
-     */
-    private copyRoutes;
+    redirect(from: string, to: string, status?: RedirectStatus): this;
+    use(pathOrMiddleware: string | Middleware | Router, routerOrUndefined?: Router): this;
     /**
-     * Match a route and return handler + params
+     * Mount a sub-router at a path prefix (Hono-style) — the explicit, more
+     * semantic equivalent of `router.use(path, subRouter)`.
+     * @see {@link https://github.com/0xTanzim/nextRush/blob/main/packages/router/README.md#sub-router-mounting | README: Sub-Router Mounting}
      */
+    mount(path: string, router: Router): this;
+    /** Mount a sub-router, carrying its own `routerMiddleware` onto every copied route. */
+    private mountRouter;
+    /** Match a request to a route — delegates to {@link resolveMatch} (design.md D1). */
     match(method: HttpMethod, path: string): RouteMatch | null;
     /**
-     * Extract the next segment from path at given position without allocating arrays.
-     * Returns [segment, nextIndex] where nextIndex is position after the trailing '/'.
-     */
-    private extractSegment;
-    /**
-     * Index-based recursive node matching (avoids array allocation)
-     */
-    private matchNodeIndexed;
-    /**
-     * Get routes middleware function
-     * Mount this on the application
-     *
-     * @example
-     * ```typescript
-     * app.use(router.routes());
-     * ```
+     * Return the router's dispatch middleware — mount this on the application.
+     * @see {@link https://github.com/0xTanzim/nextRush/blob/main/packages/router/README.md#routerroutes | README: router.routes()}
      */
     routes(): Middleware;
     /**
-     * Re-compile all route executors to include router-level middleware.
-     * Called once when routes() is invoked, not per-request.
-     */
-    private sealRouterMiddleware;
-    /**
-     * Generate allowed methods middleware
-     * Responds to OPTIONS and sets Allow header
+     * Generate allowed-methods middleware. Responds to OPTIONS with an `Allow`
+     * header and returns 405 for a known path hit with an unregistered method.
      */
     allowedMethods(): Middleware;
     /**
-     * Find all HTTP methods registered for a given path via single tree walk
-     * @internal
+     * Create a route group with a shared prefix and middleware. The callback
+     * receives a {@link RouteGroup} to register routes against.
+     * @see {@link https://github.com/0xTanzim/nextRush/blob/main/packages/router/README.md#route-groups | README: Route Groups}
      */
-    private findAllowedMethods;
+    group(prefix: string, middlewareOrCallback: Middleware[] | ((router: RouteGroup) => void), callback?: (router: RouteGroup) => void): this;
     /**
-     * Walk the tree to find the node matching a path (ignoring HTTP method)
-     * @internal
-     */
-    private findNode;
-    /**
-     * Create a route group with shared prefix and middleware
-     *
-     * @param prefix - Path prefix for all routes in the group
-     * @param middlewareOrCallback - Middleware array or callback function
-     * @param callback - Callback function if middleware is provided
-     * @returns this for chaining
-     *
-     * @example
-     * ```typescript
-     * // Simple group with prefix
-     * router.group('/api', (r) => {
-     *   r.get('/users', listUsers);
-     *   r.get('/posts', listPosts);
-     * });
-     *
-     * // Group with middleware
-     * router.group('/admin', [authMiddleware], (r) => {
-     *   r.get('/dashboard', dashboard);
-     *   r.post('/settings', updateSettings);
-     * });
-     *
-     * // Nested groups
-     * router.group('/api/v1', (r) => {
-     *   r.group('/users', [rateLimit], (ur) => {
-     *     ur.get('/', listUsers);
-     *     ur.get('/:id', getUser);
-     *   });
-     * });
-     * ```
-     */
-    group(prefix: string, middlewareOrCallback: Middleware[] | ((router: Router) => void), callback?: (router: Router) => void): this;
-    /**
-     * Remove all registered routes and middleware, resetting the router to its initial state.
-     * Useful for plugin `destroy()` to cleanly un-register routes.
+     * Remove all routes and middleware, resetting the router to its initial
+     * state — for test isolation or hot-reload re-registration.
      */
     reset(): void;
+    /** Register a route on behalf of a {@link RouteGroup} (group context). @internal */
+    _addGroupRoute(method: HttpMethod, path: string, handlers: RouteHandler[], groupMiddleware: Middleware[], recordIntrospection?: boolean): void;
     /**
-     * Internal method to add route with group context
-     * @internal
+     * Group-facing entry point to {@link pushAnyMethodDefinition} — a group's `.all()`
+     * records its consolidated row here since group routes live on the parent. @internal
      */
-    _addGroupRoute(method: HttpMethod, path: string, handlers: RouteHandler[], groupMiddleware: Middleware[]): void;
+    _pushAnyMethodRouteDefinition(path: string): void;
 }
 /**
- * Create a new Router instance
- *
- * @param options - Router options
- * @returns New Router instance
- *
- * @example
- * ```typescript
- * const router = createRouter();
- * const apiRouter = createRouter({ prefix: '/api/v1' });
- * ```
+ * Create a new {@link Router} instance.
+ * @see {@link https://github.com/0xTanzim/nextRush/blob/main/packages/router/README.md#quick-start | README: Quick Start}
  */
 declare function createRouter(options?: RouterOptions): Router;
 
@@ -232,7 +188,8 @@ declare function createRouter(options?: RouterOptions): Router;
  * @nextrush/router - Segment Trie Node
  *
  * Internal segment trie implementation for high-performance route matching.
- * Uses a compressed trie structure for O(k) lookups where k is path length.
+ * Segments are split by `/` and matched one trie level per segment for O(k)
+ * lookups where k is path length.
  *
  * @packageDocumentation
  * @internal
@@ -250,23 +207,23 @@ declare const enum NodeType {
     WILDCARD = 2
 }
 /**
- * Radix tree node
+ * Segment trie node
  */
-interface RadixNode {
+interface TrieNode {
     /** Path segment for this node */
     segment: string;
     /** Node type */
     type: NodeType;
-    /** Children nodes keyed by first character */
-    children: Map<string, RadixNode>;
+    /** Static child nodes keyed by whole path segment (e.g. `users`), not the first character */
+    children: Map<string, TrieNode>;
     /** Parameter name if this is a param node */
     paramName?: string;
     /** Handlers keyed by HTTP method */
     handlers: Map<HttpMethod, HandlerEntry>;
     /** Wildcard child if any */
-    wildcardChild?: RadixNode;
+    wildcardChild?: TrieNode;
     /** Parameter child if any */
-    paramChild?: RadixNode;
+    paramChild?: TrieNode;
 }
 /**
  * Handler entry with middleware and pre-compiled executor
@@ -278,9 +235,9 @@ interface HandlerEntry {
     executor?: (ctx: Context) => Promise<void>;
 }
 /**
- * Create a new radix node
+ * Create a new segment trie node
  */
-declare function createNode(segment: string, type?: NodeType): RadixNode;
+declare function createNode(segment: string, type?: NodeType): TrieNode;
 /**
  * Parse path segments
  * Splits path into segments and identifies param/wildcard types
@@ -298,4 +255,4 @@ interface ParsedSegment {
     paramName?: string;
 }
 
-export { type HandlerEntry, NodeType, type ParsedSegment, type RadixNode, Router, createNode, createRouter, parseSegments };
+export { type HandlerEntry, NodeType, type ParsedSegment, type RouteGroup, Router, type TrieNode, createNode, createRouter, endpoint, parseSegments };