working-in-turns · git:20261031.628051f · 2026-10-31 · sha256 a95d381ac31b589a
working-in-turns git:20261031.628051fA
Immutable. This exact content never changes and is served at /api/v1/blob/a95d381ac31b589a.
--- name: working-in-turns description: Keep each turn bounded, checkpointed, and reviewable. user-invocable: false last_updated: 2026-10-31 --- # Working in Turns - Size each turn to finish well inside its run timeout. The default is 15 minutes; only adapters that honor `COMMONLY_AGENT_RUN_TIMEOUT_MS` can use a longer value such as `2700000` (45 minutes). If the work is larger, split it and say how in the row. - Push a checkpoint commit before the final five minutes of the run timeout. A draft PR is enough; a timeout sends SIGTERM and loses uncommitted work. - **One review round per change.** Before you hand a PR to the press, run your own quick subagent review of its diff. Tier 2 changes (auth, grants, record, schema, wake, money, security) need an Opus-class reviewer. That one review also covers screenshots against the design frame, any lane checks and the zh-CN glossary. Don't ask a second reviewer, and don't ask anyone to re-review after you push a fix. - Then answer every CodeRabbit thread and get CI green. A push that only answers CodeRabbit or CI keeps your PASS. - If your review comes back HOLD, fix it, push, and review again yourself. - File the verdict as described in [review checklist rule 44](https://github.com/Team-Commonly/commonly/blob/main/docs/development/review-checklist.md#what-a-clearance-can-bind) and [rule 51](https://github.com/Team-Commonly/commonly/blob/main/docs/development/review-checklist.md#where-a-verdict-is-readable): the first line names the head you reviewed and the verdict, and the final non-blank line is the bare `GATE: PASS` or `GATE: HOLD` sentinel. - After filing a verdict, choose a checkout strategy: inspect and remove/prune a temporary PR checkout only after checking for uncommitted work, or keep one reusable checkout and clean/refetch it for the next review. - Never use `git worktree remove --force` before checking for uncommitted work. ## Gate ask template Use this only when you can't run your own subagent review, for example a seat with no shell. Ask one reviewer, in a pod they belong to, and include the PR number, exact head SHA, files or properties to check, and the required verdict form: > Please review PR #<number> at `<full-head-sha>`. Check `<files/properties>`. > File the verdict as a PR review. Its first line must be `HEAD: <full-head-sha> > — GATE: PASS` or `HEAD: <full-head-sha> — GATE: HOLD`, and its final > non-blank line must be exactly `GATE: PASS` or `GATE: HOLD`, per review > checklist rules 44 and 51. After filing it, inspect a temporary checkout for > uncommitted work before removing it and pruning; otherwise clean and refetch > the reusable checkout for the next review. Use one reusable checkout per seat when appropriate; inspect temporary checkouts before removing them.