From 345caa01ba294a95a84b096f31399f3d004083bc Mon Sep 17 00:00:00 2001 From: Dmitry Dementiev Date: Tue, 4 Aug 2026 00:22:43 +0300 Subject: [PATCH] =?UTF-8?q?docs(skills):=20handoff=20=D1=81=D0=B5=D1=81?= =?UTF-8?q?=D1=81=D0=B8=D0=B8=20=D1=85=D0=BE=D0=BB=D0=BE=D0=B4=D0=BD=D0=BE?= =?UTF-8?q?=D0=B3=D0=BE=20=D1=80=D0=B5=D0=B2=D1=8C=D1=8E=20=D0=BF=D0=BB?= =?UTF-8?q?=D0=B5=D0=B9=D0=B1=D1=83=D0=BA=D0=B0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Зачем: - индексы находок и промпты обеих слепых линий жили в скретчпаде сессии и её не переживут; открытые вопросы и неотработанные боем механики надо было вынести в долговечное место рядом с прежними handoff'ами скилла. - Что: - добавлен handoffs/20260803-kholodnoe-revyu-skilla-i-pravki.md: устройство холодного ревью, три вопроса к владельцу, находки оставленные сознательно (с доводами), пять пунктов «не обкатано боем» и список подсказанных скиллов. - содержимое коммитов не дублируется — диапазон назван ссылкой c18472e..HEAD. - Проверка: - строки ≤80 символов, объём сопоставим с соседними handoff'ами. Co-Authored-By: Claude Opus 5 --- ...0260803-kholodnoe-revyu-skilla-i-pravki.md | 79 +++++++++++++++++++ 1 file changed, 79 insertions(+) create mode 100644 agents/.agents/skills/claude-subagent-playbook/handoffs/20260803-kholodnoe-revyu-skilla-i-pravki.md diff --git a/agents/.agents/skills/claude-subagent-playbook/handoffs/20260803-kholodnoe-revyu-skilla-i-pravki.md b/agents/.agents/skills/claude-subagent-playbook/handoffs/20260803-kholodnoe-revyu-skilla-i-pravki.md new file mode 100644 index 0000000..68ce216 --- /dev/null +++ b/agents/.agents/skills/claude-subagent-playbook/handoffs/20260803-kholodnoe-revyu-skilla-i-pravki.md @@ -0,0 +1,79 @@ +# Handoff разработки скилла: холодное ревью правок и лечение находок + +Дата: 2026-08-03/04. Контекст: не боевой прогон, а заход на сам скилл. +Начали под `/writing-great-skills` (разбор `references/lessons.md`), +закончили под `/brainstorm-with-docs` (холодное ревью и триаж находок). +Диапазон коммитов — `c18472e..HEAD`, девять штук, `git log --oneline` +даёт список; здесь только то, чего в них нет. + +## Холодное ревью: как было устроено и что от него осталось + +Две слепые линии по доктрине самого плейбука: Codex `gpt-5.6-sol` (high, +свежий тред) — исполнимость механик; свежий `craftsman` — уместность +инструкций. Обе получили один параметрический блок с порядком риска, +мандат из `prompts/review-boilerplate.md`, и не знали ни друг о друге, ни +о рассуждениях автора правок. Итог: 19 находок, пересечение — 3 темы; +обе линии CHANGES_REQUIRED. Codex — 90 выполненных команд, дерево после +ревью чистое. + +Промпты и индексы находок лежали в скретчпаде сессии и не переживут её. +Всё, что не закрыто, перенесено сюда. + +## Открытые вопросы к владельцу (не правки) + +1. **Две копии определений субагентов.** `references/claude-agents.md` + (заготовки) и `.claude/agents/*.md` (созданы коммитом `f650451`). + Сегодня совпадают, главная не названа. Расхождение будет тихим: + ступень оркестратор читает из первого, харнесс — из второго. +2. **Вкусовую линию нечем эскалировать.** `craftsman` пришит к `medium`, + единственный более сильный Опус (`deep-reasoner`) описан только как + временный фолбэк при отсутствии `craftsman`. При этом разрыв рангов + «Опус medium против Кодекса high» назван в `lessons.md` причиной, ради + которой правились шапки триажа. Вопрос: заводить ли `craftsman` + ступенью выше. +3. **Записывать ли в уроки случай с песочницей.** Правило «не режь + ревьюеру read-only» стояло в двух местах и было нарушено при сборке + команды для этого же холодного ревью — не по незнанию, а рассуждением + («ревью документа рук не требует»). Вылечено значением, а не третьим + запретом (коммит `f481831`). Владелец решает сам, идёт ли это в + журнал. + +## Находки, оставленные сознательно + +- **Отклонена с доводом:** гонка между проверкой отчёта и `wc` в стороже + (микросекунды, а `STALLED` по тексту — повод посмотреть, не вердикт). +- **Оставлены как решение владельца:** формулировки в + `prompts/triage-header.md` про «прогон без отклонений — повод + перечитать» и про NEEDS_DECISION при собственной правке. Обе линия + ревью считает переусердствованием, но текст согласован с владельцем + 2026-08-03 и линия этого не знала. +- **Оставлены после спора:** обоснование переименования рядом с правилом + в `prompt-templates.md` (именно оно удерживает следующего от + «упрощения» имён обратно); стойка «follow it exactly» в начале + определения `craftsman` — территория пользователя. + +## Не обкатано боем — проверить первым же прогоном + +1. **Новые имена обменных файлов**: `review-spec-*` (этап 2), + `review-impl-*` (этап 4), логи ревьюера тоже несут этап. Осторожно: + `task`/`code` — это **линии** этапа 4, а не этапы; на этом уже + ошиблись один раз, предупреждение стоит в `prompt-templates.md`. +2. **Второй вердикт перепроверки** `PROPORTIONATE | OVERREACH` и его + вынос в финальную строку `RECHECK:`. +3. **Сторож зависшего треда**: пара `REPORT`/`LOG` подставляется от + страхуемой ветки. Проверен только симуляцией четырёх исходов. +4. **Рецепт ветки ревьюера** целиком (появился 2026-08-03) с пином + `high` и `--strict-config` на `resume`. +5. **Шапки триажа и перепроверки** в целом: правились ради того, чтобы + исполнитель не брал ревью на веру. Мерка простая — за три прогона до + правки было ноль отклонений находок. + +## Suggested skills + +- `claude-subagent-playbook` — если завтра боевой прогон по тикету. +- `writing-great-skills` — если продолжаем править сам скилл: оптика + этой сессии (дублирование, осадок, разрастание, запрет против + значения) вся оттуда. +- `conventional-commits` — обязателен перед любым коммитом. +- `brainstorm-with-docs` — для открытых вопросов выше: они решения, а не + правки.