chore(rules): убрать осиротевшие после PR #3198 ветки bot-pipeline #3203

Merged

lekss361 merged 1 commit from chore/rules-drop-bot-orphans into main

2026-08-29 09:08:34 +00:00

lekss361 commented

2026-08-28 20:27:56 +00:00

Owner

После PR #3198 (снос автономного bot-pipeline) правила остались ссылаться на механику удалённых ботов. Найдено при сверке правил со свежей документацией Anthropic.

Главное — это не косметика

git-pr.md § Polling loop, подсекция «Foreground fallback», шаг 3 без всяких условий предписывал:

3. Новый review/comment: ... Парсь marker <!-- gendesign-review-bot: sha=<sha7> verdict=<approve|changes> -->
   - verdict=approve + SHA match → merge_pull_request
4. Нет новых comments → re-schedule 60s
5. Cap: 30 iter без resolution → stop, ping user

Маркер писал auto-code-reviewer, удалённый в #3198. Никто его больше не пишет. Соло-сессия, идущая по этому правилу, опрашивала PR 30 раз по 60 секунд в ожидании вердикта, которого не будет, и через полчаса пинговала человека.

Что изменено

Файл	Было	Стало
`git-pr.md` Polling loop	парсинг маркера бота, ветки `verdict=approve/changes`	шаг 3 — merge по зелёному CI (с оглядкой на § Auto-merge policy); добавлены ветки «checks красные» и «человеческий review с запросом правок»
`git-pr.md` Commit messages	`Closes #N` для людей, `Refs #N` для бота	всегда `Closes #N`
`git-pr.md` Auto-merge policy	«в bot-pipeline reviewer-окно ставит approve+SHA»	убрано; pre-merge gate = зелёный CI
`delegation.md` split big issues	ветка `bot-pipeline: label status/needs-analysis, снять claim`	снята, осталась интерактивная

Refs #N был обходным путём ровно потому, что issue закрывал qa-бот на status/done. Бота нет — при Refs #N PR мержится, а issue остаётся открытой навсегда.

Итого 2 files changed, 8 insertions(+), 10 deletions(-). Упоминаний bot-pipeline / review-bot / reviewer-окно / qa-pickup в .claude/rules/ и CLAUDE.md больше не осталось (проверено grep'ом).

Что намеренно НЕ менялось

Сам self-extending guard в § Auto-merge policy — только убрано упоминание несуществующего reviewer-окна.
Пороги эвристик сплита (>5 файлов / >500 строк / >2ч).
Cap: 30 iter — рационал изменился (ждём CI, а не бота), но менять число без твоего решения не стал.

⚠️ Merge — только руками

PR правит git-pr.md, файл прямо назван в собственном self-extending guard (§ Запреты). Не мержу.

🤖 Generated with Claude Code

https://claude.ai/code/session_01SWnPegZgvg3NX35yLuVSjH

После PR #3198 (снос автономного bot-pipeline) правила остались ссылаться на механику удалённых ботов. Найдено при сверке правил со свежей документацией Anthropic. ## Главное — это не косметика `git-pr.md` § Polling loop, подсекция «Foreground fallback», шаг 3 **без всяких условий** предписывал: ``` 3. Новый review/comment: ... Парсь marker  - verdict=approve + SHA match → merge_pull_request 4. Нет новых comments → re-schedule 60s 5. Cap: 30 iter без resolution → stop, ping user ``` Маркер писал `auto-code-reviewer`, удалённый в #3198. Никто его больше не пишет. Соло-сессия, идущая по этому правилу, опрашивала PR **30 раз по 60 секунд** в ожидании вердикта, которого не будет, и через полчаса пинговала человека. ## Что изменено | Файл | Было | Стало | |---|---|---| | `git-pr.md` Polling loop | парсинг маркера бота, ветки `verdict=approve/changes` | шаг 3 — merge по зелёному CI (с оглядкой на § Auto-merge policy); добавлены ветки «checks красные» и «человеческий review с запросом правок» | | `git-pr.md` Commit messages | `Closes #N` для людей, `Refs #N` для бота | всегда `Closes #N` | | `git-pr.md` Auto-merge policy | «в bot-pipeline reviewer-окно ставит approve+SHA» | убрано; pre-merge gate = зелёный CI | | `delegation.md` split big issues | ветка `bot-pipeline: label status/needs-analysis, снять claim` | снята, осталась интерактивная | `Refs #N` был обходным путём ровно потому, что issue закрывал qa-бот на `status/done`. Бота нет — при `Refs #N` PR мержится, а issue остаётся открытой навсегда. Итого `2 files changed, 8 insertions(+), 10 deletions(-)`. Упоминаний `bot-pipeline` / `review-bot` / `reviewer-окно` / `qa-pickup` в `.claude/rules/` и `CLAUDE.md` больше не осталось (проверено grep'ом). ## Что намеренно НЕ менялось - Сам self-extending guard в § Auto-merge policy — только убрано упоминание несуществующего reviewer-окна. - Пороги эвристик сплита (>5 файлов / >500 строк / >2ч). - `Cap: 30 iter` — рационал изменился (ждём CI, а не бота), но менять число без твоего решения не стал. ## ⚠️ Merge — только руками PR правит `git-pr.md`, файл прямо назван в собственном self-extending guard (§ Запреты). Не мержу. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01SWnPegZgvg3NX35yLuVSjH

lekss361 added 1 commit 2026-08-28 20:27:57 +00:00

chore(rules): убрать осиротевшие после PR #3198 ветки bot-pipeline

CI Trade-In / changes (pull_request) Successful in 13s

Details

CI Trade-In / backend-tests (pull_request) Has been skipped

Details

CI Trade-In / browser-tests (pull_request) Has been skipped

Details

CI Trade-In / frontend-checks (pull_request) Has been skipped

Details

CI / changes (pull_request) Successful in 18s

Details

CI / backend-tests (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI / openapi-codegen-check (pull_request) Has been skipped

Details

555bbf546c

Правила ссылались на механику удалённых ботов. Главное — не косметика:
секция "Polling loop / Foreground fallback" безусловно предписывала парсить
маркер <!-- gendesign-review-bot: ... -->, который писал auto-code-reviewer.
Агента нет, маркера нет — соло-сессия опрашивала PR до Cap 30 iter × 60s
в ожидании вердикта, которого не будет, и через полчаса пинговала человека.

git-pr.md:
- Polling loop: шаг 3 вместо маркера бота теперь merge по зелёному CI,
  добавлены ветки "checks красные" и "человеческий review с правками"
- Refs #N -> Closes #N: обходной путь был нужен только потому, что issue
  закрывал qa-бот на status/done. Бота нет, иначе issue не закроется никогда
- Auto-merge policy: убрано упоминание reviewer-окна и approve+SHA gate

delegation.md:
- снята ветка "bot-pipeline: label status/needs-analysis, снять claim"

Что НЕ менялось: сам self-extending guard, пороги эвристик, Cap 30 iter.