Skip to content

Commit d0cdaac

Browse files
authored
feat(gastown): add REVIEW.md support to refinery code review prompts (#3608)
1 parent 61d3422 commit d0cdaac

1 file changed

Lines changed: 18 additions & 14 deletions

File tree

services/gastown/src/prompts/refinery-system.prompt.ts

Lines changed: 18 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -73,13 +73,15 @@ ${gateList}
7373
7474
### Step 2: Code Review
7575
If all gates pass (or no gates are configured), review the diff:
76-
1. Run \`git diff ${params.targetBranch}...HEAD\` to see all changes
77-
2. Check for:
78-
- Correctness — does the code do what the bead title/description asked?
79-
- Style — consistent with the existing codebase?
80-
- Test coverage — are new features tested?
81-
- Security — no secrets, no injection vulnerabilities, no unsafe patterns?
82-
- Build artifacts — no compiled files, node_modules, or other generated content?
76+
77+
- First, check for REVIEW.md in the workspace root. If it exists, read and follow its guidance for what to flag, severity calibration, skip rules, and summary format.
78+
- If REVIEW.md is absent, use these default review rules:
79+
- Correctness — does the code do what the bead title/description asked?
80+
- Style — consistent with the existing codebase?
81+
- Test coverage — are new features tested?
82+
- Security — no secrets, no injection vulnerabilities, no unsafe patterns?
83+
- Build artifacts — no compiled files, node_modules, or other generated content?
84+
- Then, run \`git diff ${params.targetBranch}...HEAD\` to see all changes
8385
8486
### Step 3: Decision
8587
@@ -197,13 +199,15 @@ ${params.gateList}
197199
198200
### Step 2: Code Review
199201
Review the diff on the PR:
200-
1. Run \`gh pr diff ${params.prUrl}\` or \`git diff ${params.targetBranch}...HEAD\` to see all changes
201-
2. Check for:
202-
- Correctness — does the code do what the bead title/description asked?
203-
- Style — consistent with the existing codebase?
204-
- Test coverage — are new features tested?
205-
- Security — no secrets, no injection vulnerabilities, no unsafe patterns?
206-
- Build artifacts — no compiled files, node_modules, or other generated content?
202+
203+
- First, check for REVIEW.md in the workspace root. If it exists, read and follow its guidance for what to flag, severity calibration, skip rules, and summary format.
204+
- If REVIEW.md is absent, use these default review rules:
205+
- Correctness — does the code do what the bead title/description asked?
206+
- Style — consistent with the existing codebase?
207+
- Test coverage — are new features tested?
208+
- Security — no secrets, no injection vulnerabilities, no unsafe patterns?
209+
- Build artifacts — no compiled files, node_modules, or other generated content?
210+
- Then, run \`gh pr diff ${params.prUrl}\` or \`git diff ${params.targetBranch}...HEAD\` to see all changes
207211
208212
### Step 3: Submit Your Review
209213

0 commit comments

Comments
 (0)