fix(skill): coordinator-loop — уровень по таблице, инварианты без привязки к домену
По ревью Codex: фраза «дефолт — рабочий уровень» противоречила таблице effort-levels.md (worker обычного issue — экономный) и толкала запускать всех worker-ов на high; заменена на выбор уровня по роли и классу issue. Заодно «форма распределений» расширена до доменных инвариантов данных — скилл общий, а не только для генератора. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
1 parent
4504410569
commit
1d67a3ffe9
2 files changed
+9
-6
No files matched your search
@@ -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 триаж («реально / ложно / чинить») дела
|
||||
вердикт) — из журнала;
|
||||
- **отклонения от процесса — фактами**: что пробовали, что сделано вместо;
|
||||
причины-предположения («недоступно», «не поддерживается») без проверки фактом — запрещены;
|
||||
- **открытый риск**: межмодульные инварианты и форма распределения другой родословной
|
||||
- **открытый риск**: межмодульные и доменные инварианты другой родословной
|
||||
не покрыты → нацеленный бриф для кросс-прохода.
|
||||
|
||||
Даже если проход не запустят — дыра остаётся **видимым задокументированным риском**, а
|
||||
|
||||
@@ -18,7 +18,8 @@ reviewer-субагенту. Адаптирован из `reviewer-prompt.md` с
|
||||
## Фокус проверки
|
||||
|
||||
{FOCUS}
|
||||
# по коду — тонкие баги, межмодульные инварианты, форма распределений
|
||||
# по коду — тонкие баги, межмодульные инварианты, доменные инварианты данных
|
||||
# (например, форма распределений, если она важна проекту)
|
||||
# по задаче — сделано ли то, что требует issue/спека (acceptance criteria)
|
||||
# узкий гейт — только спорный вопрос: {SPOR_SUMMARY}
|
||||
# chain — вся реализация цепочки против исходной спеки/PRD (дрейф целого)
|
||||
|
||||
Reference in new issue
Block a user