From 1d67a3ffe922a6808502f03c057cda5d73d9c425 Mon Sep 17 00:00:00 2001 From: Dmitry Dementev Date: Sun, 5 Jul 2026 23:25:25 +0300 Subject: [PATCH] =?UTF-8?q?fix(skill):=20coordinator-loop=20=E2=80=94=20?= =?UTF-8?q?=D1=83=D1=80=D0=BE=D0=B2=D0=B5=D0=BD=D1=8C=20=D0=BF=D0=BE=20?= =?UTF-8?q?=D1=82=D0=B0=D0=B1=D0=BB=D0=B8=D1=86=D0=B5,=20=D0=B8=D0=BD?= =?UTF-8?q?=D0=B2=D0=B0=D1=80=D0=B8=D0=B0=D0=BD=D1=82=D1=8B=20=D0=B1=D0=B5?= =?UTF-8?q?=D0=B7=20=D0=BF=D1=80=D0=B8=D0=B2=D1=8F=D0=B7=D0=BA=D0=B8=20?= =?UTF-8?q?=D0=BA=20=D0=B4=D0=BE=D0=BC=D0=B5=D0=BD=D1=83?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit По ревью Codex: фраза «дефолт — рабочий уровень» противоречила таблице effort-levels.md (worker обычного issue — экономный) и толкала запускать всех worker-ов на high; заменена на выбор уровня по роли и классу issue. Заодно «форма распределений» расширена до доменных инвариантов данных — скилл общий, а не только для генератора. Co-Authored-By: Claude Fable 5 --- agents/.agents/skills/coordinator-loop/SKILL.md | 12 +++++++----- .../coordinator-loop/references/reviewer-prompt.md | 3 ++- 2 files changed, 9 insertions(+), 6 deletions(-) diff --git a/agents/.agents/skills/coordinator-loop/SKILL.md b/agents/.agents/skills/coordinator-loop/SKILL.md index 854f5e8..b2e049e 100644 --- a/agents/.agents/skills/coordinator-loop/SKILL.md +++ b/agents/.agents/skills/coordinator-loop/SKILL.md @@ -195,7 +195,8 @@ worker-у их индексы **дословно**, без своих интер нельзя), так в дереве нет второго писателя; - **зачем**: те же ссылки на issue/PRD/спеку, что у worker-а, плюс фокус проверки (по задаче или по коду); на гейте — явная цель по двум семействам, слепым для одной - линии: межмодульные инварианты и форма распределений; + линии: межмодульные инварианты и доменные инварианты данных (например, форма + распределений, если она важна проекту); - **формат отчёта — индекс**: находка, `файл:строка`, какой критерий нарушен, минимальный repro; вердикт одной строкой (`APPROVED` / `CHANGES_REQUESTED`); запрет вставлять diff, полный вывод тестов и нарратив рассуждений; @@ -235,10 +236,11 @@ worker-у их индексы **дословно**, без своих интер сеткой страхуют рубежи ниже по течению; task-lens ревью, диагностика, разбор логов), **рабочий** (любое code review — пропуск ревьюера не ловит никто, глубина важнее именно здесь; worker на гейтовом issue), **усиленный** (эскалация споров по политике кругов, -межмодульные инварианты и форма распределений, финальное chain-ревью). Соответствие ролей конкретным моделям и уровням +межмодульные и доменные инварианты, финальное chain-ревью). Соответствие ролей конкретным моделям и уровням хоста — датированная таблица [references/effort-levels.md](references/effort-levels.md); -в логике скилла имён нет, вышла новая модель — правится таблица, не правила. Дефолт — -рабочий уровень; вверх — только по триггеру (шаг 5), не «на всякий случай». Хост без +в логике скилла имён нет, вышла новая модель — правится таблица, не правила. Уровень +выбирай по роли и классу issue из таблицы, дефолтного уровня «для всех» нет; вверх — +только по триггеру (шаг 5), не «на всякий случай». Хост без управляемых уровней: эскалация вырождается в свежего субагента — слабее, но не пусто. ## Классификация находок @@ -302,7 +304,7 @@ Per-finding триаж («реально / ложно / чинить») дела вердикт) — из журнала; - **отклонения от процесса — фактами**: что пробовали, что сделано вместо; причины-предположения («недоступно», «не поддерживается») без проверки фактом — запрещены; - - **открытый риск**: межмодульные инварианты и форма распределения другой родословной + - **открытый риск**: межмодульные и доменные инварианты другой родословной не покрыты → нацеленный бриф для кросс-прохода. Даже если проход не запустят — дыра остаётся **видимым задокументированным риском**, а diff --git a/agents/.agents/skills/coordinator-loop/references/reviewer-prompt.md b/agents/.agents/skills/coordinator-loop/references/reviewer-prompt.md index 949101d..b5cc9b2 100644 --- a/agents/.agents/skills/coordinator-loop/references/reviewer-prompt.md +++ b/agents/.agents/skills/coordinator-loop/references/reviewer-prompt.md @@ -18,7 +18,8 @@ reviewer-субагенту. Адаптирован из `reviewer-prompt.md` с ## Фокус проверки {FOCUS} -# по коду — тонкие баги, межмодульные инварианты, форма распределений +# по коду — тонкие баги, межмодульные инварианты, доменные инварианты данных +# (например, форма распределений, если она важна проекту) # по задаче — сделано ли то, что требует issue/спека (acceptance criteria) # узкий гейт — только спорный вопрос: {SPOR_SUMMARY} # chain — вся реализация цепочки против исходной спеки/PRD (дрейф целого)