chore(claude): явный CI-gate перед merge (false-green trap) + анти-rubber-stamp формат verdict'а #2302

Merged

lekss361 merged 1 commit from chore/claude-review-ci-gate into main

2026-07-03 20:09:55 +00:00

lekss361 commented

2026-07-03 20:07:02 +00:00

Owner

Зачем

Ресёрч community/официальных практик Claude Code (vault: research/CC_Community_Quality_Cost_Trends_0703.md) подтвердил первоисточником Anthropic два тезиса, которые у нас были только в памяти как урок, но не закодированы в реальный agent-файл:

Детерминированная верификация должна быть явной, иначе reviewer/verifier "rubber-stamps" вывод.
У нас уже был задокументирован инцидент "CI watcher false-green trap" (2026-07-03): опрос статуса сразу после push может не видеть ещё-не-заспавненные job'ы → ложный зелёный свет → merge до реального прогона тестов.

Что сделано

.claude/agents/auto-code-reviewer.md: явный CI-gate первым действием в ветке ✅ APPROVE — list_workflow_runs(head_sha=...) должен вернуть непустой список с status=="success" для относящихся к PR workflow. Пусто/waiting/running → не merge в этот тик.
Строка reviewer-bias caution после severity rubric — 🟠 FIX только при конкретном failure scenario (input/state → неверный output), не за абстрактные придирки.
.claude/agents/code-reviewer.md: формат verdict'а (Critical/Minor issues) теперь явно требует failure scenario, иначе — не критикал.

Note

auto-code-reviewer.md в self-extending guard списке (git-pr.md § Auto-merge policy) — автономный reviewer-бот не должен мержить этот PR сам, только human/main-сессия. Изменение продиктовано пользователем явно в этой сессии (ресёрч → согласованный план внедрения).

## Зачем Ресёрч community/официальных практик Claude Code (vault: `research/CC_Community_Quality_Cost_Trends_0703.md`) подтвердил первоисточником Anthropic два тезиса, которые у нас были только в памяти как урок, но не закодированы в реальный agent-файл: 1. **Детерминированная верификация должна быть явной**, иначе reviewer/verifier "rubber-stamps" вывод. 2. У нас уже был задокументирован инцидент "CI watcher false-green trap" (2026-07-03): опрос статуса сразу после push может не видеть ещё-не-заспавненные job'ы → ложный зелёный свет → merge до реального прогона тестов. ## Что сделано - **`.claude/agents/auto-code-reviewer.md`**: явный CI-gate первым действием в ветке `✅ APPROVE` — `list_workflow_runs(head_sha=...)` должен вернуть непустой список с `status=="success"` для относящихся к PR workflow. Пусто/waiting/running → не merge в этот тик. - Строка **reviewer-bias caution** после severity rubric — 🟠 FIX только при конкретном failure scenario (input/state → неверный output), не за абстрактные придирки. - **`.claude/agents/code-reviewer.md`**: формат verdict'а (Critical/Minor issues) теперь явно требует failure scenario, иначе — не критикал. ## Note `auto-code-reviewer.md` в self-extending guard списке (`git-pr.md` § Auto-merge policy) — автономный reviewer-бот не должен мержить этот PR сам, только human/main-сессия. Изменение продиктовано пользователем явно в этой сессии (ресёрч → согласованный план внедрения).

lekss361 added 1 commit 2026-07-03 20:07:03 +00:00