provide option to review simple changes

README.md

@@ -58,6 +58,7 @@ jobs:
           OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
         with:
           debug: false
+          review_simple_changes: false
           review_comment_lgtm: false
 ```
 
@@ -101,7 +102,9 @@ To ignore a PR, add the following keyword in the PR description:
 - `OPENAI_API_KEY`: use this to authenticate with OpenAI API. You can get one
   [here](https://platform.openai.com/account/api-keys). Please add this key to
   your GitHub Action secrets.
-- `OPENAI_API_ORG`: (optional) use this to use the specified organisation with OpenAI API if you have multiple. Please add this key to your GitHub Action secrets.
+- `OPENAI_API_ORG`: (optional) use this to use the specified organisation with
+  OpenAI API if you have multiple. Please add this key to your GitHub Action
+  secrets.
 
 ### Models: `gpt-4` and `gpt-3.5-turbo`
 
@@ -217,6 +220,7 @@ jobs:
           OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
         with:
           debug: false
+          review_simple_changes: false
           review_comment_lgtm: false
 ```
 

action.yml

@@ -15,6 +15,10 @@ inputs:
       'Max files to summarize and review. Less than or equal to 0 means no
       limit.'
     default: '150'
+  review_simple_changes:
+    required: false
+    description: 'Review even when the changes are simple'
+    default: 'false'
   review_comment_lgtm:
     required: false
     description: 'Leave comments even if the patch is LGTM'

src/main.ts

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

src/options.ts

@@ -6,6 +6,7 @@ export class Options {
   debug: boolean
   summaryOnly: boolean
   maxFiles: number
+  reviewSimpleChanges: boolean
   reviewCommentLGTM: boolean
   pathFilters: PathFilter
   systemMessage: string
@@ -22,6 +23,7 @@ export class Options {
     debug: boolean,
     summaryOnly: boolean,
     maxFiles = '0',
+    reviewSimpleChanges = false,
     reviewCommentLGTM = false,
     pathFilters: string[] | null = null,
     systemMessage = '',
@@ -35,6 +37,7 @@ export class Options {
     this.debug = debug
     this.summaryOnly = summaryOnly
     this.maxFiles = parseInt(maxFiles)
+    this.reviewSimpleChanges = reviewSimpleChanges
     this.reviewCommentLGTM = reviewCommentLGTM
     this.pathFilters = new PathFilter(pathFilters)
     this.systemMessage = systemMessage
@@ -53,6 +56,7 @@ export class Options {
     info(`debug: ${this.debug}`)
     info(`summary_only: ${this.summaryOnly}`)
     info(`max_files: ${this.maxFiles}`)
+    info(`review_simple_changes: ${this.reviewSimpleChanges}`)
     info(`review_comment_lgtm: ${this.reviewCommentLGTM}`)
     info(`path_filters: ${this.pathFilters}`)
     info(`system_message: ${this.systemMessage}`)

src/prompts.ts

@@ -23,8 +23,8 @@ $file_diff
 \`\`\`
 
 I would like you to summarize the diff within 50 words.
-
-Below the summary, I would also like you to triage the diff as \`NEEDS_REVIEW\` or 
+`
+  triageFileDiff = `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 modifications to the logic or functionality, even if they 
@@ -248,8 +248,15 @@ $patches
     this.summarizeReleaseNotes = summarizeReleaseNotes
   }
 
-  renderSummarizeFileDiff(inputs: Inputs): string {
-    return inputs.render(this.summarizeFileDiff)
+  renderSummarizeFileDiff(
+    inputs: Inputs,
+    reviewSimpleChanges: boolean
+  ): string {
+    let prompt = this.summarizeFileDiff
+    if (reviewSimpleChanges === false) {
+      prompt += this.triageFileDiff
+    }
+    return inputs.render(prompt)
   }
 
   renderSummarizeChangesets(inputs: Inputs): string {

src/review.ts

@@ -261,7 +261,9 @@ ${hunks.oldHunk}
     ins.filename = filename
 
     // render prompt based on inputs so far
-    let tokens = getTokenCount(prompts.renderSummarizeFileDiff(ins))
+    let tokens = getTokenCount(
+      prompts.renderSummarizeFileDiff(ins, options.reviewSimpleChanges)
+    )
 
     const diffTokens = getTokenCount(fileDiff)
     if (tokens + diffTokens > options.lightTokenLimits.requestTokens) {
@@ -291,7 +293,7 @@ ${hunks.oldHunk}
     // summarize content
     try {
       const [summarizeResp] = await lightBot.chat(
-        prompts.renderSummarizeFileDiff(ins),
+        prompts.renderSummarizeFileDiff(ins, options.reviewSimpleChanges),
         {}
       )
 
@@ -300,23 +302,24 @@ ${hunks.oldHunk}
         summariesFailed.push(`${filename} (nothing obtained from openai)`)
         return null
       } else {
-        // parse the comment to look for triage classification
-        // Format is : [TRIAGE]: <NEEDS_REVIEW or APPROVED>
-        // if the change needs review return true, else false
-        const triageRegex = /\[TRIAGE\]:\s*(NEEDS_REVIEW|APPROVED)/
-        const triageMatch = summarizeResp.match(triageRegex)
-
-        if (triageMatch != null) {
-          const triage = triageMatch[1]
-          const needsReview = triage === 'NEEDS_REVIEW'
-
-          // remove this line from the comment
-          const summary = summarizeResp.replace(triageRegex, '').trim()
-          info(`filename: ${filename}, triage: ${triage}`)
-          return [filename, summary, needsReview]
-        } else {
-          return [filename, summarizeResp, true]
+        if (options.reviewSimpleChanges === false) {
+          // parse the comment to look for triage classification
+          // Format is : [TRIAGE]: <NEEDS_REVIEW or APPROVED>
+          // if the change needs review return true, else false
+          const triageRegex = /\[TRIAGE\]:\s*(NEEDS_REVIEW|APPROVED)/
+          const triageMatch = summarizeResp.match(triageRegex)
+
+          if (triageMatch != null) {
+            const triage = triageMatch[1]
+            const needsReview = triage === 'NEEDS_REVIEW'
+
+            // remove this line from the comment
+            const summary = summarizeResp.replace(triageRegex, '').trim()
+            info(`filename: ${filename}, triage: ${triage}`)
+            return [filename, summary, needsReview]
+          }
         }
+        return [filename, summarizeResp, true]
       }
     } catch (e: any) {
       warning(`summarize: error from openai: ${e as string}`)