@dudousxd/nestjs-codegen 0.13.0 → 0.13.2

package/CHANGELOG.md

@@ -1,5 +1,31 @@
 # @dudousxd/nestjs-codegen
 
+## 0.13.2
+
+### Patch Changes
+
+- 889af1f: Resolve the members of an inline object-literal type (`{ a: Foo; b: Bar }`) in response/stream types instead of emitting the node's raw text. A named type nested in an object literal — most commonly an SSE payload's `Observable<{ data: SomeType }>`, where `SomeType` is imported from another package — was previously copied verbatim, leaving a bare, unimported identifier that is undefined in the generated file. Each member's type is now resolved (expanded inline, or reduced to `unknown` when unresolvable) like any other named reference.
+
+## 0.13.1
+
+### Patch Changes
+
+- 6b51c7b: fix(multipart): intersect the uploaded-file field at emit time so it survives a named `bodyRef`, and leave deliberately-loose bodies untouched.
+
+  Two fixes to the multipart upload routes shipped in 0.13.0:
+
+  - **Named body refs now include the file field.** Discovery carries the uploaded-file
+    field(s) in a new `multipartBody` (kept off `body`), and the emitter intersects it onto
+    whichever body expression it picks — a named `bodyRef` (`BaseFileUploadDto`) or the inline
+    text. Previously the merge lived on the inline `body` string, so a route whose `@Body`
+    resolved to an imported DTO emitted the plain `BaseFileUploadDto` and dropped the file
+    field (`api.X({ body: { ...fields, file } })` failed to type-check).
+
+  - **Deliberately-loose bodies are left alone.** A `@Body() x: SomeDto | any` handler resolves
+    to a top-level `unknown`/`any` union arm; intersecting `(Dto | unknown) & { file }` collapses
+    it and wrongly tightens the type. The emitter now detects a permissive body and skips the
+    intersection, keeping the author's loose `@Body()` (the route is still flagged `multipart`).
+
 ## 0.13.0
 
 ### Minor Changes

package/dist/cli/main.cjs

@@ -785,7 +785,15 @@ function emitRouterTypeBlock(tree, indent, outDir, serialization) {
       const isFilterQuery = c.contractSource.filterSource === "query" && !!c.contractSource.filterFields?.length;
       const query = queryRef ? queryRef.isArray ? `Array<${queryRef.name}>` : queryRef.name : isFilterQuery ? emitFilterQueryType(c) : c.contractSource.query ?? "never";
       const bodyRef = c.contractSource.bodyRef;
-      const body = method === "GET" ? "never" : bodyRef ? bodyRef.isArray ? `Array<${bodyRef.name}>` : bodyRef.name : c.contractSource.body ?? "never";
+      let body = method === "GET" ? "never" : bodyRef ? bodyRef.isArray ? `Array<${bodyRef.name}>` : bodyRef.name : c.contractSource.body ?? "never";
+      const multipartBody = c.contractSource.multipartBody;
+      if (c.contractSource.multipart && multipartBody) {
+        if (body === "never") {
+          body = multipartBody;
+        } else if (!bodyAcceptsAnything(body)) {
+          body = `(${body}) & ${multipartBody}`;
+        }
+      }
       const response = buildResponseType(c, outDir, serialization);
       const error = buildErrorType(c);
       const params = buildParamsType(c.params);
@@ -804,6 +812,25 @@ function emitRouterTypeBlock(tree, indent, outDir, serialization) {
   }
   return lines;
 }
+function topLevelUnionArms(type) {
+  const arms = [];
+  let depth = 0;
+  let start = 0;
+  for (let i = 0; i < type.length; i++) {
+    const ch = type[i];
+    if (ch === "{" || ch === "[" || ch === "<" || ch === "(") depth++;
+    else if (ch === "}" || ch === "]" || ch === ">" || ch === ")") depth--;
+    else if (ch === "|" && depth === 0) {
+      arms.push(type.slice(start, i).trim());
+      start = i + 1;
+    }
+  }
+  arms.push(type.slice(start).trim());
+  return arms;
+}
+function bodyAcceptsAnything(body) {
+  return topLevelUnionArms(body).some((arm) => arm === "unknown" || arm === "any");
+}
 function buildRequestModel(c) {
   const m = c.method.toLowerCase();
   const flat = JSON.stringify(c.name);
@@ -3579,6 +3606,19 @@ function resolveTypeNodeToString(typeNode, sourceFile, project, depth, subst = /
     dbg("unresolvable type:", name, "in", sourceFile.getFilePath());
     return "unknown";
   }
+  if (import_ts_morph7.Node.isTypeLiteral(typeNode)) {
+    const members = [];
+    for (const member of typeNode.getMembers()) {
+      if (import_ts_morph7.Node.isPropertySignature(member)) {
+        const memberTypeNode = member.getTypeNode();
+        const memberType = memberTypeNode ? resolveTypeNodeToString(memberTypeNode, sourceFile, project, depth, subst) : "unknown";
+        members.push(`${member.getName()}${member.hasQuestionToken() ? "?" : ""}: ${memberType}`);
+      } else {
+        members.push(member.getText());
+      }
+    }
+    return members.length > 0 ? `{ ${members.join("; ")} }` : "{}";
+  }
   const kind = typeNode.getKind();
   if (kind === import_ts_morph7.SyntaxKind.StringKeyword) return "string";
   if (kind === import_ts_morph7.SyntaxKind.NumberKeyword) return "number";
@@ -3885,10 +3925,7 @@ function extractDtoContract(method, sourceFile, project) {
   const filterInfo = extractApplyFilterInfo(method, sourceFile, project);
   const query = extractQueryType(method, sourceFile, project);
   const uploads = extractUploadedFiles(method);
-  if (uploads.fields) {
-    const fileObject = `{ ${uploads.fields} }`;
-    body = body ? `(${body}) & ${fileObject}` : fileObject;
-  }
+  const multipartBody = uploads.fields ? `{ ${uploads.fields} }` : null;
   const streamElement = detectStreamElement(method);
   const isStream = streamElement !== null;
   if (filterInfo && filterInfo.source === "body") {
@@ -3972,7 +4009,8 @@ function extractDtoContract(method, sourceFile, project) {
     bodySchema,
     querySchema,
     stream: isStream,
-    multipart: uploads.multipart
+    multipart: uploads.multipart,
+    multipartBody
   };
 }
 function resolveParamClass(method, decoratorName, sourceFile, project) {
@@ -4458,7 +4496,8 @@ function extractDtoRoute(args) {
       bodySchema: dtoContract?.bodySchema ?? null,
       querySchema: dtoContract?.querySchema ?? null,
       stream: dtoContract?.stream ?? false,
-      multipart: dtoContract?.multipart ?? false
+      multipart: dtoContract?.multipart ?? false,
+      multipartBody: dtoContract?.multipartBody ?? null
     }
   });
 }
@@ -4688,7 +4727,7 @@ async function watch(config, onChange, options = {}) {
 }
 
 // src/index.ts
-var VERSION = "0.13.0";
+var VERSION = "0.13.2";
 
 // src/cli/codegen.ts
 async function runCodegen(opts = {}) {