Compare commits

..

No commits in common. "0371f2cfd6f2335dea78a665bf2c3cbabb9cc6fe" and "a953884826fc7fdad299b6e370c1968d67b2e89e" have entirely different histories.

3 changed files with 4 additions and 10 deletions

2
.github/labels.conf vendored
View file

@ -1,4 +1,4 @@
panel=codex-reviewer-andresmgsl glm-reviewer-andresmgsl cluade-reviewer-andresmgsl kimi-reviewer-andresmgsl panel=codex-reviewer-andresmgsl glm-reviewer-andresmgsl cluade-reviewer-andresmgsl
triage-actors=cluade-reviewer-andresmgsl triage-actors=cluade-reviewer-andresmgsl
scope:release-flow|C5DEF5|The reusable release workflow, decide, the doors scope:release-flow|C5DEF5|The reusable release workflow, decide, the doors
scope:guards|C5DEF5|changelog-armed / changelog-monotonic / drill-recorded scope:guards|C5DEF5|changelog-armed / changelog-monotonic / drill-recorded

View file

@ -41,7 +41,7 @@ only ceremony-specific facts such as the roster and code conventions.
### Roster ### Roster
Four identities share the work (org team `agents`), each living in its own Three identities share the work (org team `agents`), each living in its own
[box](https://github.com/heavy-duty/box) — one box per credential, because [box](https://github.com/heavy-duty/box) — one box per credential, because
the box is the blast-radius boundary; roles are what a session is told, and the box is the blast-radius boundary; roles are what a session is told, and
[AGENTS.md](AGENTS.md) routes from there: [AGENTS.md](AGENTS.md) routes from there:
@ -51,14 +51,13 @@ the box is the blast-radius boundary; roles are what a session is told, and
| `cluade-reviewer-andresmgsl` | `triage` (claude-box) | **triage** — the only door issues come through; this identity mints issues and nothing else writes them (#18's `triage-actors`) — and review. It does not build. | | `cluade-reviewer-andresmgsl` | `triage` (claude-box) | **triage** — the only door issues come through; this identity mints issues and nothing else writes them (#18's `triage-actors`) — and review. It does not build. |
| `codex-reviewer-andresmgsl` | codex-box | build + review | | `codex-reviewer-andresmgsl` | codex-box | build + review |
| `glm-reviewer-andresmgsl` | glm-box | review | | `glm-reviewer-andresmgsl` | glm-box | review |
| `kimi-reviewer-andresmgsl` | kimi-box | review |
**The review panel for any PR is every bench identity except its author** — **The review panel for any PR is every bench identity except its author** —
recusal by construction, enforced by the reconciler (#10): the required recusal by construction, enforced by the reconciler (#10): the required
verdicts are the panel minus the PR's author. On this roster that resolves verdicts are the panel minus the PR's author. On this roster that resolves
to **three** cross-vendor approvals of the current head, because the only to **two** cross-vendor approvals of the current head, because the only
builder is itself on the panel and recuses from its own PRs; the rule is builder is itself on the panel and recuses from its own PRs; the rule is
panel-minus-author, and three is what it currently comes to, not a second panel-minus-author, and two is what it currently comes to, not a second
rule. Builders and triage default to different models so the issue contract rule. Builders and triage default to different models so the issue contract
is honestly exercised — a spec gap should surface as a question on the is honestly exercised — a spec gap should surface as a question on the
issue, not be silently filled by shared priors. Humans (`andres`) decide in issue, not be silently filled by shared priors. Humans (`andres`) decide in

View file

@ -1,5 +0,0 @@
### Changed
- The review panel restores `kimi-reviewer-andresmgsl` alongside GLM. The
four-identity panel now requires three cross-vendor approvals after the PR
author recuses (#224).