Skip to content
This repository was archived by the owner on Mar 6, 2024. It is now read-only.

Commit 468f076

Browse files
authored
reset summary conversation for each file (#25)
no description provided <!-- This is an auto-generated comment: release notes by openai --> ### Summary by OpenAI **Release Notes** - **Chore:** Remove `.prettierrc` file and update `.prettierrc.json` to include a new property `proseWrap` with a value of `always`. - **Style:** Formatting changes made to the `src/options.ts` file. <!-- end of auto-generated comment: release notes by openai -->
1 parent 16ff452 commit 468f076

File tree

6 files changed

+328
-309
lines changed

6 files changed

+328
-309
lines changed

.prettierrc

Lines changed: 0 additions & 7 deletions
This file was deleted.

.prettierrc.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,5 +6,6 @@
66
"singleQuote": true,
77
"trailingComma": "none",
88
"bracketSpacing": false,
9-
"arrowParens": "avoid"
9+
"arrowParens": "avoid",
10+
"proseWrap": "always"
1011
}

action.yml

Lines changed: 48 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,22 @@
1-
name: "OpenAI-based PR Reviewer & Summarizer"
2-
description: "OpenAI-based PR Reviewer and Summarizer"
1+
name: 'OpenAI-based PR Reviewer & Summarizer'
2+
description: 'OpenAI-based PR Reviewer and Summarizer'
33
branding:
4-
icon: "aperture"
5-
color: "orange"
6-
author: "FluxNinja, Inc."
4+
icon: 'aperture'
5+
color: 'orange'
6+
author: 'FluxNinja, Inc.'
77
inputs:
88
debug:
99
required: false
10-
description: "Enable debug mode"
11-
default: "false"
10+
description: 'Enable debug mode'
11+
default: 'false'
1212
temperature:
1313
required: false
14-
description: "Temperature for GPT model"
15-
default: "0.0"
14+
description: 'Temperature for GPT model'
15+
default: '0.0'
1616
review_comment_lgtm:
1717
required: false
18-
description: "Leave comments even if the patch is LGTM"
19-
default: "false"
18+
description: 'Leave comments even if the patch is LGTM'
19+
default: 'false'
2020
path_filters:
2121
required: false
2222
description: |
@@ -44,7 +44,7 @@ inputs:
4444
!**/vendor/**
4545
system_message:
4646
required: false
47-
description: "System message to be sent to OpenAI"
47+
description: 'System message to be sent to OpenAI'
4848
default: |
4949
You are a very experienced software engineer. You are able to thoroughly review code changes and uncover
5050
issues (if there are any) such as logic errors, syntax errors, out of bound errors, potential data races,
@@ -53,7 +53,7 @@ inputs:
5353
We will be doing code reviews today.
5454
summarize_beginning:
5555
required: false
56-
description: "The prompt for the whole pull request"
56+
description: 'The prompt for the whole pull request'
5757
default: |
5858
$system_message
5959
@@ -64,7 +64,9 @@ inputs:
6464
6565
I have a pull request with title "$title" and the description is as follows,
6666
67-
> $description.
67+
```
68+
$description
69+
```
6870
6971
Next, I will be providing you diffs for each file for you to summarize. In every response,
7072
you will update the high-level summary of the entire pull request, the table for each file's
@@ -74,11 +76,17 @@ inputs:
7476
Reply "OK" to confirm that you are ready to receive the diffs for summarization.
7577
summarize_file_diff:
7678
required: false
77-
description: "The prompt for each file diff"
79+
description: 'The prompt for each file diff'
7880
default: |
79-
Providing diff for `$filename`.
81+
Summary generated by you for other files, is as follows and you can use
82+
this summary as a basis for your response.
8083
81-
I would like you to summarize the diff and add the summary for this file in the table.
84+
```
85+
$summary
86+
```
87+
88+
I am providing diff for `$filename` below. I would like you to summarize the diff and add the summary
89+
for this file in the table.
8290
8391
In response, please provide the updated high-level summary, the summary table of the files
8492
we have summarized so far and your overall feedback to the developer.
@@ -88,9 +96,16 @@ inputs:
8896
```
8997
summarize:
9098
required: false
91-
description: "The prompt for final summarization response"
99+
description: 'The prompt for final summarization response'
92100
default: |
93-
This is the end of summarization session. Please provide the final response as follows in
101+
This is the end of summarization session. Below is the summary you have generated so far in our previous
102+
conversations. Please use this as a basis for your final response.
103+
104+
```
105+
$summary
106+
```
107+
108+
Please provide the final response as follows in
94109
the `markdown` format with the following content:
95110
- Thank the user for letting you participate in the code review.
96111
- High-level summary (focus on the purpose and intent within 80 words)
@@ -101,7 +116,7 @@ inputs:
101116
request.
102117
summarize_release_notes:
103118
required: false
104-
description: "The prompt for generating release notes"
119+
description: 'The prompt for generating release notes'
105120
default: |
106121
Next, release notes in `markdown` format for this pull request that focuses on the purpose
107122
of this PR. If needed, you can classify the changes as "New Feature", "Bug fix",
@@ -111,7 +126,7 @@ inputs:
111126
used as is in our release notes.
112127
review_beginning:
113128
required: false
114-
description: "The beginning prompt of a code review dialog"
129+
description: 'The beginning prompt of a code review dialog'
115130
default: |
116131
$system_message
117132
@@ -121,16 +136,20 @@ inputs:
121136
122137
I have a pull request with title "$title" and the description is as follows,
123138
124-
> $description.
139+
```
140+
$description
141+
```
125142
126143
OpenAI generated summary is as follows,
127144
128-
> $summary
145+
```
146+
$summary
147+
```
129148
130149
Reply "OK" to confirm that you are ready for further instructions.
131150
review_file:
132151
required: false
133-
description: "The prompt for each file"
152+
description: 'The prompt for each file'
134153
default: |
135154
Providing `$filename` content as context. Please use this context when reviewing patches.
136155
@@ -139,7 +158,7 @@ inputs:
139158
```
140159
review_file_diff:
141160
required: false
142-
description: "The prompt for each file diff"
161+
description: 'The prompt for each file diff'
143162
default: |
144163
Providing entire diff for `$filename` as context. Please use this context when reviewing patches.
145164
@@ -148,7 +167,7 @@ inputs:
148167
```
149168
review_patch_begin:
150169
required: false
151-
description: "The prompt for each file diff"
170+
description: 'The prompt for each file diff'
152171
default: |
153172
Next, I will send you a series of patches, each of them consists of a diff snippet, and you
154173
need to do a brief code review for every patch, and tell me any bug risk or improvement
@@ -159,13 +178,13 @@ inputs:
159178
preferred for your responses. Reply "OK" to confirm.
160179
review_patch:
161180
required: false
162-
description: "The prompt for each chunks/patches"
181+
description: 'The prompt for each chunks/patches'
163182
default: |
164183
$filename
165184
166185
```diff
167186
$patch
168187
```
169188
runs:
170-
using: "node16"
171-
main: "dist/index.js"
189+
using: 'node16'
190+
main: 'dist/index.js'

0 commit comments

Comments
 (0)