Open
Conversation
Encodes the recce repo's PR-verification recipe (uv bootstrap, make check, uv run pytest, optional frontend pnpm lint+type:check+test, PR hygiene against origin/main) so colleagues can dispatch the agent instead of re-explaining the toolchain. Smoke-tested twice end-to-end before merge. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Signed-off-by: even-wei <evenwei@infuseai.io>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR checklist
What type of PR is this?
chore: tooling — adds a Claude Code subagent definition.What this PR does / why we need it:
Adds
.claude/agents/recce-pr-verifier.md, a project-level subagent that encodes the recce repo's PR-verification recipe (uv-based bootstrap,make check,uv run pytest tests, optional frontendpnpm lint && type:check && test, PR hygiene againstorigin/main).The team uses many agents to verify PRs. A general-purpose agent burns tokens re-discovering the toolchain (uv vs bare python, which Makefile targets are read-only, where the PR template lives, which gotchas to skip). This subagent is pre-scoped to the repo so a dispatch goes straight to the work.
Encoded gotchas (already in
CLAUDE.md/AGENTS.md, now reachable from a single dispatch):RecceCloudExceptiondoes not inherit fromRecceException— code wrapping cloud-mode DAO ops withexcept RecceExceptionwill leak; flag missing dual-catch (PR fix(mcp): DRC-3307 fix submit_run race condition + run_check auto-approve #1342, DRC-3307)summary.pyrow-countNoneguard forTABLE_NOT_FOUND/PERMISSION_DENIED@datarecce/ui/components, not relative paths into the packagergb(255 173 21), never legacyrgba(...)mcp_server.pyresponse format changes, also updaterecce_instance_launcher/recce_agent/src/recce/agent.tsinrecce-cloud-infraWhich issue(s) this PR fixes:
None — internal tooling addition.
Special notes for your reviewer:
Smoke-tested twice end-to-end before opening:
origin/mainworktree): PASS. Caught + fixed amain..HEADvsorigin/main..HEADbug in the original draft (would have produced 33 false-positive "commits ahead" against stale local main).feature/drc-3383-recce-gitlab-summary-...— 1 commit, 4 files, net −869 LOC): PASS. The "Do NOT Flag" gotcha rules saved 2 real false positives (RecceCloudExceptioncatch pattern +logger.warninglevel), and the cross-repo MCP description sync check correctly confirmed no follow-up needed inrecce-cloud-infra.The agent never auto-fixes or pushes — it produces a pass/fail report and leaves git operations to the dispatcher.
Does this PR introduce a user-facing change?: