Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 22 additions & 0 deletions .agents/routines/triage-prompt.md
Original file line number Diff line number Diff line change
Expand Up @@ -253,6 +253,28 @@ related fixes, or "items 1-5 after PR #N" — decide:
A single cohesive PR is easier to review than three PRs with
dependencies. The bot reduces maintainer clicks, not multiplies them.

## Pre-PR build + test gate — mandatory before expert review

The expert review is expensive; don't run it on broken code. Before
spawning experts, make sure the diff actually compiles and the
unit tests pass.

1. Run the repo's build + fast test tier (see PR constraints below
for exact commands). If the diff only touches docs/markdown, skip
build and run the relevant doc check instead.
2. **If build or tests fail:** read the errors, fix the code,
re-run. Cap at **2 build→fix iterations.** If still failing,
abandon the PR and Flag for human review with the build log
in the comment.
3. Do **not** skip tests locally because "CI will run them." The
point of this gate is to not ship known-broken code even as a
draft, because (a) review noise, (b) a human reviewer may
admin-merge a draft that looks fine, (c) a green CI on push
is the baseline for the auto-fix loop — a red PR at push time
is indistinguishable from drift after the fact.
4. Only once build + tests pass on the final diff: proceed to
pre-PR expert review.

## Pre-PR expert review — mandatory before `gh pr create`

After the branch is pushed but **before** opening the PR, run a
Expand Down
Loading