fix: Only check file dependencies once/file. (#901)

Fixes #900
stephenh · Aug 13, 2023 · 8d61980 · 8d61980
1 parent 5fb444e
commit 8d61980
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/plugin.ts b/src/plugin.ts
@@ -22,17 +22,18 @@ async function main() {
 
   if (options.emitImportedFiles) {
     const fileSet = new Set();
-    const addFilesUnlessAliased = (filenames: string[]) => {
+    function addFilesUnlessAliased(filenames: string[]) {
       filenames
         .filter((name) => !options.M[name])
         .forEach((name) => {
+          if (fileSet.has(name)) return;
           fileSet.add(name);
           const file = request.protoFile.find((file) => file.name === name);
           if (file && file.dependency.length > 0) {
             addFilesUnlessAliased(file.dependency);
           }
         });
-    };
+    }
     addFilesUnlessAliased(request.fileToGenerate);
     filesToGenerate = request.protoFile.filter((file) => fileSet.has(file.name));
   } else {