Skip to content

ci: AI code review on PRs (self-hosted, claude -p, advisory)#1037

Closed
EmersonBraun wants to merge 1 commit into
mainfrom
ci/code-review-pr-gate
Closed

ci: AI code review on PRs (self-hosted, claude -p, advisory)#1037
EmersonBraun wants to merge 1 commit into
mainfrom
ci/code-review-pr-gate

Conversation

@EmersonBraun

Copy link
Copy Markdown
Member

Runs the AgentsKit code-review agent on every PR — locally on a self-hosted runner via claude -p (subscription, no per-call API cost). Advisory: posts an inline + summary review, never fails the check.

Safety

  • if: github.event.pull_request.head.repo.full_name == github.repositoryonly non-fork PRs (branches in this repo). Fork PRs are skipped so untrusted fork code never executes on the self-hosted runner.
  • Only run: step is a static PATH echo; no untrusted input is interpolated into a shell.

Requires

  • A self-hosted runner online (label self-hosted) — registered.
  • Secret CLAUDE_CODE_OAUTH_TOKEN (from claude setup-token) at repo or org level.

Tune max-files / block; flip fail-on-block: 'true' later to gate merges. This PR itself (a same-repo branch) will be reviewed by the agent once merged + on its next push.

Runs the AgentsKit code-review agent locally on a self-hosted runner via
claude -p (subscription, no per-call API cost). Advisory — posts a review,
never blocks. Guarded to non-fork PRs only (head.repo == repo) so untrusted
fork code never runs on the self-hosted runner.
@vercel

vercel Bot commented Jun 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agentskit-doc Building Building Preview, Comment Jun 18, 2026 6:02pm
agentskit-registry Ready Ready Preview, Comment Jun 18, 2026 6:02pm

Request Review

@EmersonBraun

Copy link
Copy Markdown
Member Author

Moving the PR-gate to the private agentskit-os repo (no fork-RCE surface there → covers all PRs).

@EmersonBraun
EmersonBraun deleted the ci/code-review-pr-gate branch June 18, 2026 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant