chore(skills): improve pr-review fresh-start instructions
Clarify that every /pr-review invocation must start fresh — fetch, check PRs, and re-read the diff even if the branch was reviewed earlier in the same conversation. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -62,18 +62,23 @@ Pre-existing warnings are not PR blockers but should be tracked for cleanup.
|
||||
|
||||
### 1. Determine the branch to review
|
||||
|
||||
If the user provided a branch name as argument, use it. Otherwise list open
|
||||
PRs and ask the user which branch to review.
|
||||
**Always start fresh.** Even if you reviewed this branch before in this
|
||||
conversation, the branch may have new commits, a new PR, or main may have
|
||||
moved. Do NOT skip steps or reuse earlier results. Every `/pr-review`
|
||||
invocation is a full review cycle.
|
||||
|
||||
**Always fetch and check for PRs first:**
|
||||
|
||||
To list open PRs on Gitea:
|
||||
```bash
|
||||
git fetch --all
|
||||
tea pr list --login schweitz --repo jpmschweitzer/settled-reach --state open --output simple
|
||||
```
|
||||
|
||||
Fetch remote branches first:
|
||||
```bash
|
||||
git fetch --all
|
||||
```
|
||||
Then determine the branch:
|
||||
- If the user provided a branch name as argument, match it to an open PR.
|
||||
If a PR exists for that branch, note the PR number. If no PR exists,
|
||||
proceed with the branch diff but note "no PR found" in the output.
|
||||
- If no argument was given, list open PRs and ask which to review.
|
||||
|
||||
### 2. Determine reviewer team
|
||||
|
||||
|
||||
Reference in New Issue
Block a user