@dudousxd/nestjs-codegen 0.27.0 → 0.27.1

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @dudousxd/nestjs-codegen
 
+## 0.27.1
+
+### Patch Changes
+
+- [#103](https://github.com/DavideCarvalho/nestjs-codegen/pull/103) [`9489714`](https://github.com/DavideCarvalho/nestjs-codegen/commit/94897149bb7133c61de43d66ab6c1bdc5de98cd9) Thanks [@DavideCarvalho](https://github.com/DavideCarvalho)! - Type discovery no longer overflows the stack on bundled declaration files. A `.d.ts` chunk that imports a type under an alias and re-publishes it bare (`import { p as ToolKind } from './chunk'; export { ToolKind }` — what tsup/rollup emit) used to be matched by the import's source name, miss, and re-enter the same file with a fresh cycle guard until `Maximum call stack size exceeded` aborted the whole `codegen` run (seen with `@dudousxd/nestjs-agent-core` 0.29 through `@dudousxd/nestjs-agent-codegen`). Imports are now matched by their local name and followed under their source name, and the cycle guard is carried through the walk (keyed by file and name).
+
 ## 0.27.0
 
 ### Minor Changes

package/dist/cli/main.cjs

@@ -743,10 +743,11 @@ function resolveModuleSpecifier(moduleSpecifier, sourceFile, project) {
   }
   return [];
 }
-function resolveImportedType(name, sourceFile, project) {
+function resolveImportedType(name, sourceFile, project, seen = /* @__PURE__ */ new Set()) {
   for (const importDecl of sourceFile.getImportDeclarations()) {
-    const namedImport = importDecl.getNamedImports().find((n) => n.getName() === name);
+    const namedImport = importDecl.getNamedImports().find((n) => (n.getAliasNode()?.getText() ?? n.getName()) === name);
     if (!namedImport) continue;
+    const sourceName = namedImport.getName();
     const moduleSpecifier = importDecl.getModuleSpecifierValue();
     const candidates = resolveModuleSpecifier(moduleSpecifier, sourceFile, project);
     for (const candidate of candidates) {
@@ -758,17 +759,17 @@ function resolveImportedType(name, sourceFile, project) {
           continue;
         }
       }
-      const result = findTypeInFile(name, importedFile);
+      const result = findTypeInFile(sourceName, importedFile);
       if (result) return result;
-      const viaReExport = resolveReExportedType(name, importedFile, project, /* @__PURE__ */ new Set());
+      const viaReExport = resolveReExportedType(sourceName, importedFile, project, /* @__PURE__ */ new Set());
       if (viaReExport) return viaReExport;
     }
     if (candidates.length === 0) {
-      const viaCompiler = resolveBareSpecifierType(name, importDecl, project);
+      const viaCompiler = resolveBareSpecifierType(sourceName, importDecl, project);
       if (viaCompiler) return viaCompiler;
     }
   }
-  return resolveReExportedType(name, sourceFile, project, /* @__PURE__ */ new Set());
+  return resolveReExportedType(name, sourceFile, project, seen);
 }
 function resolveBareSpecifierType(name, importDecl, project) {
   let target;
@@ -783,9 +784,9 @@ function resolveBareSpecifierType(name, importDecl, project) {
   return resolveReExportedType(name, target, project, /* @__PURE__ */ new Set());
 }
 function resolveReExportedType(name, file, project, seen) {
-  const filePath = file.getFilePath();
-  if (seen.has(filePath)) return null;
-  seen.add(filePath);
+  const visit = `${file.getFilePath()}#${name}`;
+  if (seen.has(visit)) return null;
+  seen.add(visit);
   for (const exportDecl of file.getExportDeclarations()) {
     const moduleSpecifier = exportDecl.getModuleSpecifierValue();
     const namedExports = exportDecl.getNamedExports();
@@ -807,7 +808,7 @@ function resolveReExportedType(name, file, project, seen) {
     const sourceName = namedExports.find((n) => (n.getAliasNode()?.getText() ?? n.getName()) === name)?.getName() ?? name;
     const local = findTypeInFile(sourceName, file);
     if (local) return local;
-    const imported = resolveImportedType(sourceName, file, project);
+    const imported = resolveImportedType(sourceName, file, project, seen);
     if (imported) return imported;
   }
   return null;
@@ -6020,7 +6021,7 @@ async function watch(config, onChange, options = {}) {
 }
 
 // src/index.ts
-var VERSION = "0.27.0";
+var VERSION = "0.27.1";
 
 // src/cli/codegen.ts
 async function runCodegen(opts = {}) {