Skip to content

Commit a297a7b

Browse files
committed
chore: make Gemini-review more concise and actionable
This addresses feedback shared by the team on this new workflow.
1 parent 8549912 commit a297a7b

File tree

1 file changed

+15
-8
lines changed

1 file changed

+15
-8
lines changed

.github/workflows/code-review.yml

+15-8
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ jobs:
164164
165165
# Prepare prompt for the LLM
166166
prompt = f"""
167-
Please analyze the following Git diff from a pull request and provide feedback.
167+
You are a code reviewer analyzing a pull request. Provide a concise, actionable review that helps developers quickly understand the key points to consider.
168168
You will be provided with both the diff and the entire codebase for context.
169169
170170
Pull Request: {github_repo}/pull/{pr_number}
@@ -178,14 +178,21 @@ jobs:
178178
CODEBASE:
179179
{codebase_content}
180180
181-
Please provide:
182-
1. A summary of the changes
183-
2. Potential issues or bugs you notice
184-
3. Suggestions for improvements
185-
4. Any security concerns
186-
5. Code quality observations
181+
Please provide a brief review in the following format:
187182
188-
Consider how the changes fit into the existing codebase and if they maintain consistency with the existing code patterns.
183+
## Summary
184+
[2-3 sentences maximum describing the main changes]
185+
186+
## Key Points to Review
187+
- List 2-3 specific areas that need careful attention
188+
- Focus on potential issues that could impact functionality or maintainability, and suggest improvements for each issue identified
189+
- Include any security considerations (only if there are any)
190+
191+
## Style & Consistency
192+
- Only include this section if there are notable deviations from the existing codebase style
193+
- List specific suggestions for maintaining consistency
194+
195+
Keep the review concise and actionable. Focus on the most important points that would benefit from human review.
189196
"""
190197
191198
# Send to LLM and get analysis

0 commit comments

Comments
 (0)