Skip to content

Commit fa087c2

Browse files
committed
Check GitHub CI status instead of running locally in pr-reviewer
The pr-reviewer agent now checks `gh pr checks` for existing CI results before falling back to local runs, preventing false "Not run" reports when GitHub Actions have already completed.
1 parent 8e6716b commit fa087c2

1 file changed

Lines changed: 8 additions & 7 deletions

File tree

.claude/agents/pr-reviewer.md

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -42,18 +42,19 @@ git diff main...HEAD --name-only
4242
Read each file in its entirety. Do not skip files or skim — a thorough review
4343
requires understanding the full context of every change.
4444

45-
### 3. Run CI gates
45+
### 3. Check CI status
4646

47-
Verify the branch is healthy before reviewing:
47+
Check the PR's CI status from GitHub first — do not report "Not run" when
48+
checks have already run:
4849

4950
```
50-
cargo fmt --all -- --check
51-
cargo clippy --all-targets --all-features -- -D warnings
52-
cargo test --workspace
53-
cd crates/js/lib && npx vitest run
51+
gh pr checks <number> --repo IABTechLab/trusted-server
5452
```
5553

56-
Note any CI failures in the review but continue with the code review regardless.
54+
If the PR has passing CI checks, report them as PASS in the review. Only run
55+
CI locally if checks haven't run yet or if you need to verify a specific
56+
failure. Note any CI failures in the review but continue with the code review
57+
regardless.
5758

5859
### 4. Deep analysis
5960

0 commit comments

Comments
 (0)