Skip to content

Commit 4e4b3a4

Browse files
feat: simplify PR review prompt, expand tool access, add local skill support
- Remove rigid 4-agent orchestration in favor of letting Claude decide how to parallelize review work - Add Read/Glob/Grep/Task/Skill to allowed tools so Claude can actually explore the codebase and invoke local skills - Add .claude/skills/ci-review.md mechanism for repo-specific review rules - Add previous comment deduplication to avoid repeating findings - Add severity system and gh pr review verdict step - Optimize fetch depth (PR commits + 1 instead of full clone) - Add concurrency group to cancel stale review runs - Incorporate coderabbit learnings: R4/R11/C4 exceptions, known safe SDK patterns, provisioning idempotency nuance
1 parent b9652b2 commit 4e4b3a4

2 files changed

Lines changed: 128 additions & 198 deletions

File tree

.github/workflows/pr-review.yaml

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,28 @@
11
on:
22
pull_request_target:
3+
concurrency:
4+
group: pr-review-${{ github.event.pull_request.number }}
5+
cancel-in-progress: true
36
jobs:
47
pr-review:
58
if: github.repository != 'ConductorOne/github-workflows'
69
runs-on: ubuntu-latest
710
permissions:
11+
actions: read
812
contents: read
913
pull-requests: write
1014
issues: write
1115
steps:
16+
- name: PR Commit Count
17+
run: echo "PR_FETCH_DEPTH=$(( ${{ github.event.pull_request.commits }} + 1 ))" >> "${GITHUB_ENV}"
1218
- name: Checkout PR head
13-
uses: actions/checkout@v5
19+
uses: actions/checkout@v6
1420
with:
1521
repository: ${{ github.event.pull_request.head.repo.full_name }}
1622
ref: ${{ github.event.pull_request.head.sha }}
17-
fetch-depth: 0
23+
fetch-depth: ${{ env.PR_FETCH_DEPTH }}
1824
- name: Checkout workflows repo
19-
uses: actions/checkout@v5
25+
uses: actions/checkout@v6
2026
with:
2127
repository: ConductorOne/github-workflows
2228
path: _workflows
@@ -34,6 +40,7 @@ jobs:
3440
use_sticky_comment: true
3541
track_progress: true
3642
include_fix_links: true
37-
claude_args: --model claude-opus-4-6 --max-turns 100 --allowedTools "mcp__github_inline_comment__create_inline_comment,Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*)"
43+
allowed_bots: "*"
44+
claude_args: --model claude-opus-4-6 --max-turns 100 --allowedTools "Read,Glob,Grep,Task,Skill,mcp__github_inline_comment__create_inline_comment,Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr review:*)"
3845
prompt: |
3946
Use the /pr-review skill to review this PR.

0 commit comments

Comments
 (0)