Skip to content

chore: avoid stale gh diffs in pr review skill @W-21873441 #386

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
shetzel wants to merge 1 commit into
base: main
Choose a base branch
from
+12 −2
Open

chore: avoid stale gh diffs in pr review skill @W-21873441 #386

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 12 additions & 2 deletions .cursor/skills/agent-pr-review/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,16 +7,26 @@ description: Review pull requests in plugin-agent using GitHub CLI. Use for "rev

## Required

- `gh` CLI required (no git-only fallback)
- `gh` CLI
- `git` (PR branch must be checked out locally for authoritative diff)

## Working directory

- repo root (parent of this `.cursor`)

## Commands

Run all three together:

- `gh pr view <PR_NUMBER> --json title,number,body,files,commits,additions,deletions,changedFiles,baseRefName,headRefName,author,labels`
- `gh pr diff <PR_NUMBER>`
- `gh pr diff <PR_NUMBER>` — GitHub API diff (may be stale)
- `git diff $(gh pr view <PR_NUMBER> --json baseRefName --jq '.baseRefName') -- $(gh pr view <PR_NUMBER> --json files --jq '[.files[].path] | join(" ")')` — authoritative local diff

## Diff precedence

- `gh pr diff` can be stale after a recent push
- cross-check with local `git diff <base>` and `Read` tool on changed files
- local git diff + file contents win any disagreement

## Review checklist

Expand Down
Loading