@dudousxd/nestjs-codegen 0.7.1 → 0.9.0

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # @dudousxd/nestjs-codegen
 
+## 0.9.0
+
+### Minor Changes
+
+- ff9e27b: feat(core): gate schema-translation advisories behind a new `debug` config flag (default off).
+
+  On every codegen pass the discovery layer logged a `[nestjs-codegen]` line to the
+  terminal for each schema-translation advisory — `@X is not translatable to a client
+validation schema and was skipped`, `T is a recursive type; ... lazy self-reference`,
+  over-deep nesting, and unresolvable `@IsEnum`. On a real project these fire dozens of
+  times per run and are pure noise.
+
+  These advisories are already preserved where they matter: in the returned
+  `SchemaModule.warnings` array and as `// warning:` comments in the generated output.
+  The terminal copy is now opt-in: add `debug: true` to `nestjs-codegen.config.ts`
+  (or `NestjsCodegenModule.forRoot({ debug: true })`) to print them again. Default is
+  `false`, so a normal run is quiet. No effect on generated artifacts.
+
+## 0.8.0
+
+### Minor Changes
+
+- 685583d: feat(core): synthesize the route `query` type from individual `@Query('name')` params. Handlers using named query params (e.g. `@Query('years') years?: number[]`, `@Query('q') q?: string | string[]`) now emit a typed `query` object — one property per param, keyed by the string-literal name, typed by the parameter annotation, optional when the param has `?` / a default / a `| undefined` type — instead of `query: never`. The existing whole-object `@Query() dto` form is unchanged and still wins when both forms appear on the same handler.
+
 ## 0.7.1
 
 ### Patch Changes

package/dist/cli/main.cjs

@@ -142,6 +142,7 @@ function applyDefaults(userConfig, cwd) {
     };
   }
   return {
+    debug: userConfig.debug ?? false,
     extensions: userConfig.extensions ?? [],
     // Non-null: validateUserConfig() above throws when `validation` is absent.
     validation: resolveAdapter(userConfig.validation),
@@ -2108,8 +2109,18 @@ function buildEmpty() {
   ].join("\n");
 }
 
+// src/util/debug-log.ts
+var debugEnabled = false;
+function setCodegenDebug(enabled) {
+  debugEnabled = enabled;
+}
+function debugWarn(message) {
+  if (debugEnabled) console.warn(`[nestjs-codegen] ${message}`);
+}
+
 // src/generate.ts
 async function generate(config, inputRoutes = []) {
+  setCodegenDebug(config.debug);
   const extensions = config.extensions ?? [];
   let routes = inputRoutes;
   const ctx = createExtensionContext(config, () => routes);
@@ -2705,7 +2716,7 @@ function buildProperty(prop, classFile, ctx) {
         ctx.warnedDecorators.add(name);
         const msg = `@${name} is not translatable to a client validation schema and was skipped (server-only validation).`;
         ctx.warnings.push(msg);
-        console.warn(`[nestjs-codegen] ${msg}`);
+        debugWarn(msg);
       }
     }
   }
@@ -2756,7 +2767,7 @@ function buildNestedReference(className, fromFile, ctx, typeArgs = []) {
       ctx.warnedDecorators.add(`recursive:${reserved}`);
       const msg = `${className} is a recursive type; the generated schema validates it via a lazy self-reference.`;
       ctx.warnings.push(msg);
-      console.warn(`[nestjs-codegen] ${msg}`);
+      debugWarn(msg);
     }
     return { kind: "lazyRef", name: reserved };
   }
@@ -2765,7 +2776,7 @@ function buildNestedReference(className, fromFile, ctx, typeArgs = []) {
       ctx.warnedDecorators.add(`deep:${className}`);
       const msg = `${className} nesting is too deep to expand; the generated schema uses unknown for it.`;
       ctx.warnings.push(msg);
-      console.warn(`[nestjs-codegen] ${msg}`);
+      debugWarn(msg);
     }
     return { kind: "unknown", note: "nesting too deep \u2014 not expanded" };
   }
@@ -2906,7 +2917,7 @@ function enumSchemaFromDecorator(decorator, classFile, ctx) {
     if (!ctx.warnedDecorators.has(`IsEnum:${name}`)) {
       ctx.warnedDecorators.add(`IsEnum:${name}`);
       ctx.warnings.push(msg);
-      console.warn(`[nestjs-codegen] ${msg}`);
+      debugWarn(msg);
     }
     return { kind: "unknown", note: `@IsEnum(${name}): enum not resolvable to literals` };
   }
@@ -3536,14 +3547,33 @@ function extractQueryType(method, sourceFile, project) {
   for (const param of method.getParameters()) {
     const queryDecorator = param.getDecorators().find((d) => d.getName() === "Query");
     if (!queryDecorator) continue;
-    const queryArgs = queryDecorator.getArguments();
-    if (queryArgs.length > 0) continue;
+    if (queryDecorator.getArguments().length > 0) continue;
     const typeNode = param.getTypeNode();
     if (typeNode) {
       return resolveTypeNodeToString(typeNode, sourceFile, project, 3);
     }
   }
-  return null;
+  const entries = [];
+  for (const param of method.getParameters()) {
+    const queryDecorator = param.getDecorators().find((d) => d.getName() === "Query");
+    if (!queryDecorator) continue;
+    const queryArgs = queryDecorator.getArguments();
+    const nameArg = queryArgs[0];
+    if (!nameArg || !import_ts_morph7.Node.isStringLiteral(nameArg)) continue;
+    const queryName = nameArg.getLiteralValue();
+    const typeNode = param.getTypeNode();
+    const queryType = typeNode ? resolveTypeNodeToString(typeNode, sourceFile, project, 3) : "string";
+    entries.push(`${queryName}${isParamOptional(param) ? "?" : ""}: ${queryType}`);
+  }
+  return entries.length > 0 ? `{ ${entries.join("; ")} }` : null;
+}
+function isParamOptional(param) {
+  if (param.hasQuestionToken() || param.hasInitializer()) return true;
+  const typeNode = param.getTypeNode();
+  if (typeNode && import_ts_morph7.Node.isUnionTypeNode(typeNode)) {
+    return typeNode.getTypeNodes().some((t) => t.getKind() === import_ts_morph7.SyntaxKind.UndefinedKeyword);
+  }
+  return false;
 }
 function extractParamsType(method, sourceFile, project) {
   const entries = [];
@@ -4481,7 +4511,7 @@ async function watch(config, onChange) {
 }
 
 // src/index.ts
-var VERSION = "0.7.1";
+var VERSION = "0.9.0";
 
 // src/cli/codegen.ts
 async function runCodegen(opts = {}) {