Max files (#84)

<!-- This is an auto-generated comment: release notes by openai -->
### Summary by OpenAI

Chore: Limit the number of files summarized and reviewed based on
configurable options. Default values for these options have been changed
to 40 and 0 respectively.
<!-- end of auto-generated comment: release notes by openai -->

Author: harjotgill

action.yml

@@ -13,11 +13,11 @@ inputs:
     required: false
     description:
       'Max files to summarize. Less than or equal to 0 means no limit.'
-    default: '60'
+    default: '40'
   max_files_to_review:
     required: false
     description: 'Max files to review. Less than or equal to 0 means no limit.'
-    default: '180'
+    default: '0'
   review_comment_lgtm:
     required: false
     description: 'Leave comments even if the patch is LGTM'

dist/index.js

@@ -207,8 +207,8 @@ export class Options {
 
   constructor(
     debug: boolean,
-    max_files_to_summarize = '60',
-    max_files_to_review = '180',
+    max_files_to_summarize = '40',
+    max_files_to_review = '0',
     review_comment_lgtm = false,
     path_filters: string[] | null = null,
     system_message = '',

src/options.ts

@@ -185,7 +185,10 @@ export const codeReview = async (
     const summaryPromises = []
     const skipped_files_to_summarize = []
     for (const [filename, file_content, file_diff] of files_to_review) {
-      if (summaryPromises.length < options.max_files_to_summarize) {
+      if (
+        options.max_files_to_summarize <= 0 ||
+        summaryPromises.length < options.max_files_to_summarize
+      ) {
         summaryPromises.push(
           openai_concurrency_limit(async () =>
             generateSummary(filename, file_content, file_diff)
@@ -403,9 +406,6 @@ ${
     }
 
     // Use Promise.all to run file review processes in parallel
-    // rewrite this to take max_files_to_review limit into account
-    // const reviewPromises = files_to_review.map(
-    //   async ([filename, file_content, file_diff, patches]) =>
     //     openai_concurrency_limit(async () =>
     //       review(filename, file_content, file_diff, patches)
     //     )
@@ -418,7 +418,10 @@ ${
       file_diff,
       patches
     ] of files_to_review) {
-      if (reviewPromises.length < options.max_files_to_review) {
+      if (
+        options.max_files_to_review <= 0 ||
+        reviewPromises.length < options.max_files_to_review
+      ) {
         reviewPromises.push(
           openai_concurrency_limit(async () =>
             review(filename, file_content, file_diff, patches)