Compare commits
No commits in common. "a953884826fc7fdad299b6e370c1968d67b2e89e" and "b7a6aedc522c15abe323ed8ac55e0b3db3e20241" have entirely different histories.
a953884826
...
b7a6aedc52
4 changed files with 3 additions and 13 deletions
2
.github/labels.conf
vendored
2
.github/labels.conf
vendored
|
|
@ -1,4 +1,4 @@
|
||||||
panel=codex-reviewer-andresmgsl glm-reviewer-andresmgsl cluade-reviewer-andresmgsl
|
panel=codex-reviewer-andresmgsl kimi-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
|
||||||
|
|
|
||||||
|
|
@ -50,7 +50,7 @@ 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 |
|
| `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
|
||||||
|
|
|
||||||
|
|
@ -1,10 +0,0 @@
|
||||||
### Changed
|
|
||||||
|
|
||||||
- The review panel names `glm-reviewer-andresmgsl` in place of
|
|
||||||
`kimi-reviewer-andresmgsl` while that identity is unavailable. The panel
|
|
||||||
stays three, so a PR still requires two cross-vendor approvals (#222).
|
|
||||||
|
|
||||||
- `test/labels.test.sh`'s table-side roster mutation names an identity the
|
|
||||||
table actually carries. Naming a departed one would mutate nothing and the
|
|
||||||
case would pass while proving nothing — #195's rot class one layer down
|
|
||||||
(#222).
|
|
||||||
|
|
@ -246,7 +246,7 @@ sed 's/^panel=/panel=ghost-bot /' "$CONF" >"$mut_conf"
|
||||||
check "an identity in the conf but not the table goes red" 1 "ghost-bot" \
|
check "an identity in the conf but not the table goes red" 1 "ghost-bot" \
|
||||||
roster_in_sync "$mut_conf" "$CONTRIB"
|
roster_in_sync "$mut_conf" "$CONTRIB"
|
||||||
# shellcheck disable=SC2016 # the backticks are the table's, not a subshell
|
# shellcheck disable=SC2016 # the backticks are the table's, not a subshell
|
||||||
sed 's/^| `glm-reviewer-andresmgsl`/| `ghost-bot`/' "$CONTRIB" >"$mut_contrib"
|
sed 's/^| `kimi-reviewer-andresmgsl`/| `ghost-bot`/' "$CONTRIB" >"$mut_contrib"
|
||||||
check "an identity in the table but not the conf goes red" 1 "ghost-bot" \
|
check "an identity in the table but not the conf goes red" 1 "ghost-bot" \
|
||||||
roster_in_sync "$CONF" "$mut_contrib"
|
roster_in_sync "$CONF" "$mut_contrib"
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue