create-nexwire 4.2.0 → 4.2.1

package/express/src/framework/http/router.ts

@@ -127,8 +127,16 @@ export function createRouter(): NexwireRouter {
   const nativeRoute = router.route.bind(router);
   states.set(router, { routes: [], children: [] });
 
+  // Group middlewares are kept here rather than registered with router.use(),
+  // because a catch-all use() also matches requests whose route lives in a
+  // sibling router or in the parent. They are merged into every .api() call
+  // instead, so they travel with the routes they were declared on.
+  const groupMiddlewares: RequestHandler[] = [];
+
   router.group = function (...middlewares: RequestHandler[]) {
-    if (middlewares.length) router.use(...middlewares);
+    for (const middleware of middlewares) {
+      groupMiddlewares.push(middleware);
+    }
     return this;
   };
 
@@ -140,7 +148,13 @@ export function createRouter(): NexwireRouter {
       throw new Error("api(route, middlewares, handler) requires a handler function");
     }
 
-    (router as any)[route.method](expressPath(route.path), validationMiddleware(route), ...middlewares, finalHandler);
+    (router as any)[route.method](
+      expressPath(route.path),
+      validationMiddleware(route),
+      ...groupMiddlewares,
+      ...middlewares,
+      finalHandler
+    );
     stateOf(router).routes.push(route);
     return this;
   };

package/express/src/framework/maker-cli/stubs/seeder/name.ts.stub

@@ -13,6 +13,6 @@ export default async function {{ClassName}}Seeder() {
 
   for (const row of rows) await db.insert({{tableVariable}}).values(row);
 
-  console.log("{{ClassName}}: " + rows.length + " row(s) prepared");
+  console.log(`{{ClassName}}: ${rows.length} row(s) prepared`);
 }
 

package/express/src/framework/maker-cli/stubs/seeder/standalone.ts.stub

@@ -18,5 +18,5 @@ export default async function {{ClassName}}Seeder() {
 
   // for (const row of rows) await db.insert({{tableVariable}}).values(row);
 
-  console.log("{{ClassName}}: " + rows.length + " row(s) ready");
+  console.log(`{{ClassName}}: ${rows.length} row(s) ready`);
 }

package/express/vitest.config.ts

@@ -25,14 +25,14 @@ export default defineConfig({
       // framework's own tests under `src/framework/**/__tests__/` are not
       // published, because they test the framework rather than the application.
       // That measures 8.49% statements, so a floor calibrated for this repository
-      // (14%) reports a failure for a developer who has done nothing wrong and
-      // whose 56 tests all pass.
+      // reports a failure for a developer who has done nothing wrong and whose
+      // 56 tests all pass.
       //
-      // The floor is a guard on this repository, so it is applied by CI, which
-      // passes --coverage.thresholds.* on the command line. That keeps this file
-      // byte-identical across both engines, keeps it free of a number that would
-      // be meaningless in someone else's project, and still fails the build when
-      // untested code is added here.
+      // The floor is a guard on this repository, not on yours, so it is passed on
+      // the command line when the framework's own suite is run - it is never read
+      // from this file. That keeps this file byte-identical across both engines,
+      // keeps it free of a number that would be meaningless in someone else's
+      // project, and still fails when untested code is added here.
     }
   }
 });

package/hono/src/framework/http/router.ts

@@ -11,47 +11,71 @@ export type NexwireRouter = OpenAPIHono & {
  * When: Building root app and module route files.
  * Where: HTTP app bootstrap and route stubs.
  * How: Wraps OpenAPIHono and adds group/api convenience APIs.
+ *
+ * group() records its middlewares here instead of calling this.use("*", mw).
+ * A use("*") is a route entry: mounting the router copies it and "*" rewrites
+ * to the mount prefix, so a limiter declared on one sub-router also ran for its
+ * siblings and for every route added after the mount. Recorded middlewares are
+ * merged into each .api() call instead, so they travel with the routes they
+ * were declared on and reach nothing else.
  */
 export function createRouter(): NexwireRouter {
   const router = new OpenAPIHono({ strict: false, defaultHook }) as NexwireRouter;
   const baseOpenapi = router.openapi.bind(router);
 
+  // Store group middlewares on the instance
+  const groupMiddlewares: any[] = [];
+
   router.group = function (...middlewares) {
+    // Accumulate group middlewares (supports multiple .group() calls)
     for (const middleware of middlewares) {
-      this.use("*", middleware);
+      groupMiddlewares.push(middleware);
     }
-
     return this;
   };
 
   router.api = function (route: any, handlerOrMiddlewares: any, handler?: any) {
+    // Normalize arguments:
+    //   .api(route, handler)                      → [groupMiddlewares], handler
+    //   .api(route, [mw1, mw2], handler)          → [groupMiddlewares, mw1, mw2], handler
+    let routeMiddlewares: any[] = [];
+    let finalHandler: any;
+
     if (!Array.isArray(handlerOrMiddlewares)) {
-      baseOpenapi(route, handlerOrMiddlewares);
-      return this;
+      finalHandler = handlerOrMiddlewares;
+    } else {
+      routeMiddlewares = handlerOrMiddlewares;
+      finalHandler = handler;
     }
 
-    const middlewares = handlerOrMiddlewares;
-    const finalHandler = handler;
+    // Combine: group middleware runs first, then route middleware
+    const allMiddlewares = [...groupMiddlewares, ...routeMiddlewares];
 
     if (typeof finalHandler !== "function") {
       throw new Error("api(route, middlewares, handler) requires a handler function");
     }
 
+    // No middleware at all → simple pass-through
+    if (allMiddlewares.length === 0) {
+      baseOpenapi(route, finalHandler);
+      return this;
+    }
+
+    // Wrap middleware chain into the handler — travels with the route,
+    // no matter where the router is mounted.
     const wrappedHandler = async (c: any) => {
       let index = 0;
 
       const next = async (): Promise<any> => {
-        if (index < middlewares.length) {
-          const middleware = middlewares[index++];
+        if (index < allMiddlewares.length) {
+          const middleware = allMiddlewares[index++];
           let downstream: any;
           const result = await middleware(c, async () => {
             downstream = await next();
             return downstream;
           });
-
           return result === undefined ? downstream : result;
         }
-
         return await finalHandler(c);
       };
 
@@ -66,10 +90,7 @@ export function createRouter(): NexwireRouter {
 }
 
 /**
- * Why: Shorthand for creating grouped router middleware chain.
- * When: Route files need shared middleware composition.
- * Where: Module route declarations.
- * How: Creates a router instance and applies all provided middlewares.
+ * Shorthand for creating a router with group middleware pre-applied.
  */
 export function group(...middlewares: any[]) {
   return createRouter().group(...middlewares);

package/hono/src/framework/maker-cli/stubs/seeder/name.ts.stub

@@ -13,6 +13,6 @@ export default async function {{ClassName}}Seeder() {
 
   for (const row of rows) await db.insert({{tableVariable}}).values(row);
 
-  console.log("{{ClassName}}: " + rows.length + " row(s) prepared");
+  console.log(`{{ClassName}}: ${rows.length} row(s) prepared`);
 }
 

package/hono/src/framework/maker-cli/stubs/seeder/standalone.ts.stub

@@ -18,5 +18,5 @@ export default async function {{ClassName}}Seeder() {
 
   // for (const row of rows) await db.insert({{tableVariable}}).values(row);
 
-  console.log("{{ClassName}}: " + rows.length + " row(s) ready");
+  console.log(`{{ClassName}}: ${rows.length} row(s) ready`);
 }

package/hono/vitest.config.ts

@@ -25,14 +25,14 @@ export default defineConfig({
       // framework's own tests under `src/framework/**/__tests__/` are not
       // published, because they test the framework rather than the application.
       // That measures 8.49% statements, so a floor calibrated for this repository
-      // (14%) reports a failure for a developer who has done nothing wrong and
-      // whose 56 tests all pass.
+      // reports a failure for a developer who has done nothing wrong and whose
+      // 56 tests all pass.
       //
-      // The floor is a guard on this repository, so it is applied by CI, which
-      // passes --coverage.thresholds.* on the command line. That keeps this file
-      // byte-identical across both engines, keeps it free of a number that would
-      // be meaningless in someone else's project, and still fails the build when
-      // untested code is added here.
+      // The floor is a guard on this repository, not on yours, so it is passed on
+      // the command line when the framework's own suite is run - it is never read
+      // from this file. That keeps this file byte-identical across both engines,
+      // keeps it free of a number that would be meaningless in someone else's
+      // project, and still fails when untested code is added here.
     }
   }
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-nexwire",
-  "version": "4.2.0",
+  "version": "4.2.1",
   "type": "module",
   "description": "Scaffold a new nexwire project",
   "bin": {