fix: do not add file to prompt if only removed lines (#174)

mattzcarey · Aug 14, 2023 · a33f49a · a33f49a
1 parent 80ee5b2
commit a33f49a
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/src/review/prompt/constructPrompt/batchFiles/utils/createPromptFiles.ts b/src/review/prompt/constructPrompt/batchFiles/utils/createPromptFiles.ts
@@ -118,14 +118,18 @@ export const createPromptFiles = (
   maxPromptPayloadLength: number,
   maxSurroundingLines?: number
 ): PromptFile[] => {
-  return files.map((file) => {
+  return files.reduce((result: PromptFile[], file) => {
     const contentLines = file.fileContent.split("\n");
     const changedLinesArray = file.changedLines.split("\n");
 
     // Get the changed indices, total length, and min/max indices
     const { changedIndices, totalChangedLinesLength, minIndex, maxIndex } =
       getChangedIndicesAndLength(contentLines, changedLinesArray);
 
+    if (totalChangedLinesLength == 0) {
+      return result;
+    }
+
     // Calculate remaining space and start/end positions
     let remainingSpace =
       maxPromptPayloadLength - totalChangedLinesLength - file.fileName.length;
@@ -145,9 +149,10 @@ export const createPromptFiles = (
       contentLines
     );
 
-    return {
+    result.push({
       fileName: file.fileName,
       promptContent: promptContent,
-    };
-  });
+    });
+    return result;
+  }, []);
 };