add option for less verbose reviews

README.md

@@ -28,6 +28,10 @@ features of this action are:
   changes (e.g. typo fixes) and when changes look good for the most part. It can
   be disabled by setting `review_simple_changes` and `review_comment_lgtm` to
   `true`.
+- **Less verbose reviews**: For more experienced users the Red Rover can err on 
+  side of ignoring a change unless it is a major one. This is disabled by default,
+  to enable set `review_simple_changes` to `false` and `less_verbose_review` to
+  `true`.
 - **Customizable prompts**: Tailor the `system_message`, `summarize`, and
   `summarize_release_notes` prompts to focus on specific aspects of the review
   process or even change the review objective.

src/main.ts

@@ -18,6 +18,7 @@ async function run(): Promise<void> {
     getBooleanInput('disable_release_notes'),
     getInput('max_files'),
     getBooleanInput('review_simple_changes'),
+    getBooleanInput('less_verbose_review'),
     getBooleanInput('review_comment_lgtm'),
     getMultilineInput('path_filters'),
     getInput('system_message'),

src/options.ts

@@ -8,6 +8,7 @@ export class Options {
   disableReleaseNotes: boolean
   maxFiles: number
   reviewSimpleChanges: boolean
+  lessVerboseReview: boolean
   reviewCommentLGTM: boolean
   pathFilters: PathFilter
   systemMessage: string
@@ -27,6 +28,7 @@ export class Options {
     disableReleaseNotes: boolean,
     maxFiles = '0',
     reviewSimpleChanges = false,
+    lessVerboseReview = false,
     reviewCommentLGTM = false,
     pathFilters: string[] | null = null,
     systemMessage = '',
@@ -43,6 +45,7 @@ export class Options {
     this.disableReleaseNotes = disableReleaseNotes
     this.maxFiles = parseInt(maxFiles)
     this.reviewSimpleChanges = reviewSimpleChanges
+    this.lessVerboseReview = lessVerboseReview
     this.reviewCommentLGTM = reviewCommentLGTM
     this.pathFilters = new PathFilter(pathFilters)
     this.systemMessage = systemMessage
@@ -64,6 +67,7 @@ export class Options {
     info(`disable_release_notes: ${this.disableReleaseNotes}`)
     info(`max_files: ${this.maxFiles}`)
     info(`review_simple_changes: ${this.reviewSimpleChanges}`)
+    info(`less_verbose_review: ${this.lessVerboseReview}`)
     info(`review_comment_lgtm: ${this.reviewCommentLGTM}`)
     info(`path_filters: ${this.pathFilters}`)
     info(`system_message: ${this.systemMessage}`)

src/prompts.ts

@@ -38,6 +38,24 @@ lines changed, the potential impact on the overall system, and the likelihood of
 introducing new bugs or security vulnerabilities. 
 When in doubt, always err on the side of caution and triage the diff as \`NEEDS_REVIEW\`.
 
+You must follow the format below strictly for triaging the diff and 
+do not add any additional text in your response:
+[TRIAGE]: <NEEDS_REVIEW or APPROVED>
+`
+  lessVerboseTriageFileDiff = `Below the summary, I would also like you to triage the diff as \`NEEDS_REVIEW\` or 
+\`APPROVED\` based on the following criteria:
+
+- If the diff involves any major modifications to the logic or functionality triage 
+  it as \`NEEDS_REVIEW\`. This includes changes to control structures, 
+  function calls, or variable assignments that might impact the behavior of the code.
+- If the diff contains minor changes that don't affect the code logic, such as 
+  fixing typos, formatting, or renaming variables for clarity, triage it as \`APPROVED\`.
+
+Please evaluate the diff thoroughly and take into account factors such as the number of 
+lines changed, the potential impact on the overall system, and the likelihood of 
+introducing new bugs or security vulnerabilities. 
+When in doubt, always err on the side of approving and triage the diff as \`APPROVED\`.
+
 You must follow the format below strictly for triaging the diff and 
 do not add any additional text in your response:
 [TRIAGE]: <NEEDS_REVIEW or APPROVED>
@@ -275,12 +293,16 @@ $comment
 
   renderSummarizeFileDiff(
     inputs: Inputs,
-    reviewSimpleChanges: boolean
+    reviewSimpleChanges: boolean,
+    lessVerboseReview: boolean
   ): string {
     let prompt = this.summarizeFileDiff
-    if (reviewSimpleChanges === false) {
+    if (reviewSimpleChanges === false && lessVerboseReview === false) {
       prompt += this.triageFileDiff
     }
+    if (reviewSimpleChanges === false && lessVerboseReview === true) {
+      prompt += this.lessVerboseTriageFileDiff
+    }
     return inputs.render(prompt)
   }
 

src/review.ts

@@ -266,7 +266,11 @@ ${hunks.oldHunk}
 
     // render prompt based on inputs so far
     let tokens = getTokenCount(
-      prompts.renderSummarizeFileDiff(ins, options.reviewSimpleChanges)
+      prompts.renderSummarizeFileDiff(
+        ins,
+        options.reviewSimpleChanges,
+        options.lessVerboseReview
+      )
     )
 
     const diffTokens = getTokenCount(fileDiff)
@@ -282,7 +286,11 @@ ${hunks.oldHunk}
     // summarize content
     try {
       const [summarizeResp] = await lightBot.chat(
-        prompts.renderSummarizeFileDiff(ins, options.reviewSimpleChanges),
+        prompts.renderSummarizeFileDiff(
+          ins,
+          options.reviewSimpleChanges,
+          options.lessVerboseReview
+        ),
         {}
       )
 
@@ -312,7 +320,9 @@ ${hunks.oldHunk}
       }
     } catch (e: any) {
       warning(`summarize: error from RedRover: ${e as string}`)
-      summariesFailed.push(`${filename} (error from RedRover: ${e as string})})`)
+      summariesFailed.push(
+        `${filename} (error from RedRover: ${e as string})})`
+      )
       return null
     }
   }