Skip to content

Run Claude review automatically on every PR - #40

Merged
jlocke2 merged 2 commits into
masterfrom
agentic-readiness/auto-review
Jul 20, 2026
Merged

Run Claude review automatically on every PR#40
jlocke2 merged 2 commits into
masterfrom
agentic-readiness/auto-review

Conversation

@jlocke2

@jlocke2 jlocke2 commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Update: AGENTS.md is the canonical file (open standard, readable by Claude Code/Codex/Cursor); CLAUDE.md is a one-line import so Claude Code picks it up natively.

Adds a CLAUDE.md context file for AI agents (and new developers) as part of the team's agentic-readiness work: what this repo is, how it fits into Pipeline CRM, how to build/run/verify it, architecture, and the real pitfalls.

Everything in it was derived from this repo's code, scripts, and README — no invented commands. Please read it as reviewers who know this system best and correct anything that is inaccurate or stale.

🤖 Generated with Claude Code

Update: review runs pinned to Sonnet — cheaper per run, and the reviewing model differs from the model that authors most changes.

jlocke2 and others added 2 commits July 17, 2026 09:29
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@jlocke2
jlocke2 requested a review from sevgibson July 17, 2026 15:15

@sevgibson sevgibson left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM — uses the safe pull_request trigger (not pull_request_target, so fork PRs get no secrets), contents: read, and budget/turn caps.

@sevgibson

Copy link
Copy Markdown
Contributor

Approved — flagging one fleet-wide hardening for the C rollout (this is one of the few public repos, so it matters here). The issue_comment trigger only checks contains(github.event.comment.body, '@claude') — no author_association gate, unlike claude-worker.yml. On a public repo a non-collaborator could trigger the secret-bearing run by commenting @claude (bounded by the $0.75/run cap, and the comment path checks out the base branch so there's no code-exec — it's a cost/abuse surface, not RCE). Suggest adding the same guard the worker uses to the comment branches:

contains(fromJSON('["OWNER", "MEMBER", "COLLABORATOR"]'), github.event.comment.author_association)

Only the public repos are exposed (mantle, pipeline-js-api-client, pipeline_api) — private repos are fine as-is, so this isn't blocking.

@jlocke2
jlocke2 merged commit 020269f into master Jul 20, 2026
1 check passed
@jlocke2
jlocke2 deleted the agentic-readiness/auto-review branch July 20, 2026 01:08
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.

2 participants