allow disabling release notes (#253)

Author: harjotgill

action.yml

@@ -54,10 +54,14 @@ inputs:
       !**/_gen/**
       !**/generated/**
       !**/vendor/**
-  summary_only:
+  disable_review:
     required: false
     description: 'Only provide the summary and skip the code review.'
     default: 'false'
+  disable_release_notes:
+    required: false
+    description: 'Disable release notes'
+    default: 'false'
   openai_light_model:
     required: false
     description:

dist/index.js

@@ -14,7 +14,8 @@ import {handleReviewComment} from './review-comment'
 async function run(): Promise<void> {
   const options: Options = new Options(
     getBooleanInput('debug'),
-    getBooleanInput('summary_only'),
+    getBooleanInput('disable_review'),
+    getBooleanInput('disable_release_notes'),
     getInput('max_files'),
     getBooleanInput('review_simple_changes'),
     getBooleanInput('review_comment_lgtm'),

src/main.ts

@@ -4,7 +4,8 @@ import {TokenLimits} from './limits'
 
 export class Options {
   debug: boolean
-  summaryOnly: boolean
+  disableReview: boolean
+  disableReleaseNotes: boolean
   maxFiles: number
   reviewSimpleChanges: boolean
   reviewCommentLGTM: boolean
@@ -21,7 +22,8 @@ export class Options {
 
   constructor(
     debug: boolean,
-    summaryOnly: boolean,
+    disableReview: boolean,
+    disableReleaseNotes: boolean,
     maxFiles = '0',
     reviewSimpleChanges = false,
     reviewCommentLGTM = false,
@@ -35,7 +37,8 @@ export class Options {
     openaiConcurrencyLimit = '4'
   ) {
     this.debug = debug
-    this.summaryOnly = summaryOnly
+    this.disableReview = disableReview
+    this.disableReleaseNotes = disableReleaseNotes
     this.maxFiles = parseInt(maxFiles)
     this.reviewSimpleChanges = reviewSimpleChanges
     this.reviewCommentLGTM = reviewCommentLGTM
@@ -54,7 +57,8 @@ export class Options {
   // print all options using core.info
   print(): void {
     info(`debug: ${this.debug}`)
-    info(`summary_only: ${this.summaryOnly}`)
+    info(`disable_review: ${this.disableReview}`)
+    info(`disable_release_notes: ${this.disableReleaseNotes}`)
     info(`max_files: ${this.maxFiles}`)
     info(`review_simple_changes: ${this.reviewSimpleChanges}`)
     info(`review_comment_lgtm: ${this.reviewCommentLGTM}`)

src/options.ts

@@ -63,9 +63,9 @@ Description:
 $description
 \`\`\`
 
-OpenAI generated notes:
+OpenAI generated summary:
 \`\`\`
-$release_notes
+$raw_summary
 \`\`\`
 
 Content of \`$filename\` prior to changes:
@@ -200,9 +200,9 @@ Description:
 $description
 \`\`\`
 
-OpenAI generated notes:
+OpenAI generated summary:
 \`\`\`
-$release_notes
+$raw_summary
 \`\`\`
 
 Content of file prior to changes:

src/prompts.ts

@@ -377,8 +377,11 @@ ${filename}: ${summary}
   if (summarizeFinalResponse === '') {
     info('summarize: nothing obtained from openai')
   } else {
-    // final release notes
     nextSummarizeIds = summarizeFinalResponseIds
+  }
+
+  if (options.disableReleaseNotes === false) {
+    // final release notes
     const [releaseNotesResponse, releaseNotesIds] = await heavyBot.chat(
       prompts.renderSummarizeReleaseNotes(inputs),
       nextSummarizeIds
@@ -470,7 +473,7 @@ ${
 }
 `
 
-  if (!options.summaryOnly) {
+  if (!options.disableReview) {
     const filesAndChangesReview = filesAndChanges.filter(([filename]) => {
       const needsReview =
         summaries.find(