codex-bot-andresmgsl/ceremony
1
0
Fork 0
forked from heavy-duty/ceremony
Code Pull requests Activity

chore: restore kimi to review panel (#224)

Browse source
This commit is contained in:
Codex Review 2026-08-09 14:57:09 +00:00
parent a953884826
commit 5487f71ef8
3 changed files with 10 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
.github/labels.conf vendored
View file

@ -1,4 +1,4 @@
panel=codex-reviewer-andresmgsl glm-reviewer-andresmgsl cluade-reviewer-andresmgsl panel=codex-reviewer-andresmgsl glm-reviewer-andresmgsl cluade-reviewer-andresmgsl kimi-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

7
CONTRIBUTING.md
View file

@ -41,7 +41,7 @@ only ceremony-specific facts such as the roster and code conventions.
### Roster ### Roster
Three identities share the work (org team `agents`), each living in its own Four 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,13 +51,14 @@ 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 **two** cross-vendor approvals of the current head, because the only to **three** 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 two is what it currently comes to, not a second panel-minus-author, and three 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

5
changelog.d/224.md Normal file
View file

@ -0,0 +1,5 @@
### 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).