@gian-tiaga/eda 3.13.1 → 4.0.0

package/skills/eda-review/SKILL.md

@@ -5,78 +5,81 @@ description: 'Проводит полное read-only ревью diff, ветк
 
 # Скил: Ревьюер (eda-review)
 
-Проводи одно полное read-only ревью в выбранном режиме исполнения. В `subagents` оркестрируй установленных `eda-review-*` агентов и сам не выполняй их специализированные роли. В `main` выполни те же выбранные проверки самостоятельно по общей матрице, не запуская leaf-субагентов. В обоих режимах сохрани один итоговый отчёт с одинаковыми требованиями к доказательствам и результату.
+Проводи одно полное read-only ревью в выбранном режиме исполнения. В `subagents` оркестрируй установленных `eda-review-*` агентов и сам не выполняй их специализированные роли. В `main` выполни те же выбранные проверки самостоятельно по тем же чек-листам, не запуская leaf-субагентов. В обоих режимах сохрани один итоговый отчёт с одинаковыми требованиями к доказательствам и результату.
 
 ## Вход из сообщения пользователя
 
-Текст рядом с вызовом скилла в текущем сообщении — главный вход. Сначала разбери target, план, ссылку, ограничения, режим `subagents` или `main`, отключённые или принудительно включённые проверки и разовые модели.
+Текст рядом с вызовом скилла в текущем сообщении — главный вход. Сначала разбери target, план, ссылку, ограничения, режим `subagents` или `main`, отключённые или принудительно включённые проверки и разовые модели или effort.
 
 Если входа достаточно, продолжай без вопроса. Без аргументов ревьюй незакоммиченный `git diff HEAD`. Ссылка на PR/MR задаёт target однозначно. Если пользователь дал только корень удалённого репозитория, обязательно попроси указать PR/MR, ветку, коммит или файлы; весь репозиторий и последний PR автоматически не выбирай.
 
-Явное слово `subagents` или `main` задаёт режим только для текущего запуска и важнее `docs/settings.yaml`. Разовые указания вида «без performance», «без business», «включи database», «включи business», «architecture на opus» или «tests на gpt-5.6-sol» также важнее настроек.
+Явное слово `subagents` или `main` задаёт режим только для текущего запуска и важнее `docs/settings.yaml`. Разовые указания вида «без performance», «без business», «включи database», «включи business», «architecture на anthropic/claude-opus-5-5#high», «tests на openai/gpt-5.6-sol» или «security на default#xhigh» также важнее настроек и выбора проверок.
 
 Если во входе даны ранее отклонённые находки или путь к предыдущему ревью, прочитай их раздел «Отклонено» и не принимай эти находки повторно без нового доказательства.
 
 ## Настройки
 
-Прочитай `docs/settings.yaml`, если файл есть. Поддерживается только `version: 3`. Используй `review.execution`, `review.agents.<check>.mode` и `review.agents.<check>.model.<platform>`. Если файла нет или версия отличается, используй defaults и добавь предупреждение; старые настройки не применяй.
+Прочитай `docs/settings.yaml`, если файл есть. Поддерживается только `version: 3`. Используй `review.execution` и `review.agents.<role>`. Если файла нет или версия отличается, используй defaults и добавь предупреждение; старые настройки не применяй.
 
 `review.execution` принимает:
 
-- `subagents` — каждая выбранная проверка выполняется отдельным установленным агентом;
-- `main` — все выбранные проверки выполняет текущий агент в одном контексте.
+- `subagents` — выбор проверок и каждая выбранная проверка выполняются отдельными установленными агентами;
+- `main` — выбор и все выбранные проверки выполняет текущий агент в одном контексте.
 
-Default — `subagents`. При неизвестном или отсутствующем значении используй default и добавь предупреждение. Сначала выбери `$EXECUTION` по явному режиму из текущего сообщения, затем по настройке, иначе по default.
+Default — `main`. При неизвестном или отсутствующем значении используй default и добавь предупреждение. Сначала выбери `$EXECUTION` по явному режиму из текущего сообщения, затем по настройке, иначе по default.
 
-Зафиксируй режим и его источник до первой проверки. Выбор из вызова, override оркестратора или `review.execution` обязателен: большой diff, число проверок и нехватка контекста не разрешают переход к субагентам. Сохраняй режим и прогресс после сжатия контекста; если продолжить технически невозможно, верни конкретный блокер без смены режима. Менять режим можно только по новому прямому указанию пользователя. `auto` у отдельной проверки ниже определяет её применимость и не меняет `$EXECUTION`.
+Зафиксируй режим и его источник до первой проверки. Выбор из вызова, override оркестратора или `review.execution` обязателен: большой diff, число проверок и нехватка контекста не разрешают переход к субагентам. Сохраняй режим и прогресс после сжатия контекста; если продолжить технически невозможно, верни конкретный блокер без смены режима. Менять режим можно только по новому прямому указанию пользователя. Выбор проверок не меняет `$EXECUTION`.
 
-Допустимые режимы:
+`review.agents.<role>` — одна строка на роль: `default` или модель в формате `provider/model#variant`, где provider и variant необязательны, а variant задаёт effort. `default` — модель и effort установленного агента: в Claude Code и Codex — из таблицы ниже, в OpenCode — модель агента или сессии. `default#<variant>` меняет только effort.
 
-- `always` — запускать при каждом ревью;
-- `auto` — запускать только когда проверка применима;
-- `off` — не запускать.
+В `subagents` применяй модель роли так:
 
-Допустимые модели по умолчанию для `subagents`: Claude Code — `haiku`, `sonnet`, `opus`; Codex — `gpt-5.6-luna`, `gpt-5.6-terra`, `gpt-5.6-sol`. Неизвестное или отсутствующее значение замени default-значением конкретного агента и добавь предупреждение в покрытие.
+- Claude Code запускает только модели Claude: `anthropic/claude-opus-5-5`, `claude-opus-5-5` или `opus` передай алиасом семейства (`haiku`, `sonnet`, `opus` и т.п.), variant — как effort `low`, `medium`, `high`, `xhigh` или `max`;
+- Codex запускает только модели OpenAI: `openai/<model>` или `<model>` передай как модель, variant — как reasoning effort;
+- OpenCode передай значение как есть в формате `provider/model#variant`; значение без provider, кроме `default`, OpenCode не применит.
 
-В `main` все проверки выполняются текущей моделью агента. Настройки и разовые overrides моделей отдельных проверок не применяются; добавь об этом один warning в покрытие, только если пользователь явно передал такой override. Для completed-проверок укажи фактическую текущую модель, если хост её сообщает, иначе `current`.
+Если текущая среда не может запустить модель или effort роли либо значение не соответствует формату, запусти роль на модели и effort установленного агента и добавь warning в покрытие. Ревью из-за этого не останавливай.
 
-Если `version: 3` существует, но раздела `review.agents` нет, используй defaults ниже и добавь предупреждение. Устаревшие поля `review.strict`, `review.include_code_quality` и раздел `review-check` не применяй. Существующий файл настроек не переписывай.
+В `main` выбор и все проверки выполняются текущей моделью агента. Настройки и разовые overrides моделей и effort отдельных ролей не применяются; добавь об этом один warning в покрытие, только если пользователь явно передал такой override. Для completed-проверок укажи фактическую текущую модель, если хост её сообщает, иначе `current`.
 
-| Проверка | Default mode | Claude | Codex |
-|---|---|---|---|
-| `correctness` | `always` | `sonnet` | `gpt-5.6-terra` |
-| `architecture` | `auto` | `opus` | `gpt-5.6-sol` |
-| `rules` | `always` | `haiku` | `gpt-5.6-luna` |
-| `references` | `auto` | `haiku` | `gpt-5.6-luna` |
-| `business` | `auto` | `sonnet` | `gpt-5.6-terra` |
-| `plan_alignment` | `auto` | `sonnet` | `gpt-5.6-terra` |
-| `code_quality` | `always` | `sonnet` | `gpt-5.6-terra` |
-| `tests` | `always` | `sonnet` | `gpt-5.6-terra` |
-| `security` | `auto` | `opus` | `gpt-5.6-sol` |
-| `performance` | `auto` | `sonnet` | `gpt-5.6-terra` |
-| `frontend` | `auto` | `sonnet` | `gpt-5.6-terra` |
-| `api` | `auto` | `sonnet` | `gpt-5.6-terra` |
-| `database` | `auto` | `opus` | `gpt-5.6-sol` |
-| `documentation` | `auto` | `haiku` | `gpt-5.6-luna` |
-| `previous_reviews` | `auto` | `haiku` | `gpt-5.6-luna` |
+Если `version: 3` существует, но раздела `review.agents` нет, используй `default` для всех ролей. Устаревшие поля `review.strict`, `review.include_code_quality` и раздел `review-check` не применяй. Состав проверок определяется по содержимому ревью, а не настройками. Существующий файл настроек не переписывай.
+
+Модели и effort установленных агентов, которые означает `default` в Claude Code и Codex:
+
+| Роль | Claude Code | Codex |
+|---|---|---|
+| `triage` | `sonnet#low` | `gpt-5.6-terra#low` |
+| `correctness` | `sonnet#medium` | `gpt-5.6-terra#medium` |
+| `architecture` | `opus#high` | `gpt-5.6-sol#high` |
+| `rules` | `haiku#low` | `gpt-5.6-luna#low` |
+| `references` | `haiku#low` | `gpt-5.6-luna#low` |
+| `business` | `sonnet#medium` | `gpt-5.6-terra#medium` |
+| `plan_alignment` | `sonnet#medium` | `gpt-5.6-terra#medium` |
+| `code_quality` | `sonnet#medium` | `gpt-5.6-terra#medium` |
+| `tests` | `sonnet#medium` | `gpt-5.6-terra#medium` |
+| `security` | `opus#high` | `gpt-5.6-sol#high` |
+| `performance` | `sonnet#medium` | `gpt-5.6-terra#medium` |
+| `frontend` | `sonnet#medium` | `gpt-5.6-terra#medium` |
+| `api` | `sonnet#medium` | `gpt-5.6-terra#medium` |
+| `database` | `opus#high` | `gpt-5.6-sol#high` |
+| `infra` | `sonnet#medium` | `gpt-5.6-terra#medium` |
+| `seo` | `sonnet#medium` | `gpt-5.6-terra#medium` |
+| `documentation` | `haiku#low` | `gpt-5.6-luna#low` |
+| `previous_reviews` | `haiku#low` | `gpt-5.6-luna#low` |
 
 ## Вопросы
 
 Задавай вопрос только когда target противоречив, дана лишь ссылка на корень репозитория или безопасно выбрать цель невозможно. Вопрос о плане не задавай: используй план только если пользователь указал его или путь определяется однозначно; иначе установи `$PLAN_FILE=none`.
 
-- Claude Code: используй `AskUserQuestion`.
-- Codex interactive: используй `request_user_input`, если доступен; иначе задай короткий вопрос с 2–3 вариантами и остановись до ответа.
-- Codex exec и другой неинтерактивный запуск: не читай stdin; верни `blocked: нужен target` с конкретным вопросом.
+- Интерактивный запуск: `AskUserQuestion`, затем остановись до ответа.
+- Неинтерактивный запуск: не читай stdin; верни `blocked: нужен target` с конкретным вопросом.
 
 ## Подготовить контекст
 
 1. Определи `$TARGET`: явный файл, папка, URL PR/MR, ветка, коммит, указанный diff или `git diff HEAD` без аргументов. Для GitHub/GitLab URL получи metadata, diff, SHA текущего head и статусы CI read-only командами `gh`/`glab`; для другого сервиса используй доступное read-only средство. Если основной diff недоступен, остановись: warning разрешён для недоступных CI-результатов и прошлых обсуждений, но не для самого target.
-2. Прочитай `AGENTS.md`, `CLAUDE.md`, `docs/rules.md`, `docs/arch.md` и сам target настолько, чтобы выбрать проверки и сформировать точный контекст. В `subagents` на этом шаге не делай собственное code review; в `main` эти документы и target станут общей основой всех проходов и не должны загружаться заново для каждой проверки.
-3. Определи `$PLAN_FILE`. Если его нет, используй `none` и не пытайся восстановить последний план наугад.
-4. Определи `$REFERENCE_FILES`: если план содержит `sources.references`, используй их; иначе через `docs/references.md` выбери только применимые карточки для компонентов target. Если карточек нет, используй `none`.
-5. Определи `$BUSINESS_FILES`: сначала установи исходную задачу и фактически затронутое пользовательское или доменное поведение по текущему сообщению, плану и target. Если план содержит `sources.business`, используй только относящиеся к этому поведению пути; иначе прочитай `docs/business.md` и по описаниям выбери только применимые карточки. Если индекс отсутствует, подходящих карточек нет или изменения не затрагивают бизнес-поведение, используй `none`. Не загружай все карточки только потому, что индекс существует.
-6. Сформируй краткий `$REVIEW_CONTEXT`: тип target, base, исходную задачу из текущего сообщения, плана, PR/MR title/body или явно связанного issue/артефакта, изменённые области, применимые документы и явные решения пользователя. Если надёжного описания задачи нет, явно напиши `task: unknown`, а не восстанавливай намерение по догадке. Не включай в контекст domain-инструкции агентов.
-7. Для PR/MR сформируй `$CI_CONTEXT`: SHA head, названия, статусы, определимые команды и ссылки на автоматические CI-проверки; для failure добавь только значимый фрагмент лога, не передавай полные логи всех jobs. Учитывай только результаты для текущего head. Завершённую в CI проверку со статусом success или failure не запускай локально повторно: используй её результат и лог как доказательство. Локальный запуск допустим только для отсутствующей, отменённой, пропущенной, устаревшей или не сопоставимой с CI проверки либо по прямому указанию пользователя. Недоступный CI отметь warning и выполняй нужные проверки обычным способом. Для не-PR/MR target используй `CI_CONTEXT: none`.
+2. Определи `$PLAN_FILE`. Если его нет, используй `none` и не пытайся восстановить последний план наугад.
+3. Сформируй краткий `$REVIEW_CONTEXT`: тип target, base, исходную задачу из текущего сообщения, плана, PR/MR title/body или явно связанного issue/артефакта и явные решения пользователя. Если надёжного описания задачи нет, явно напиши `task: unknown`, а не восстанавливай намерение по догадке. Не включай в контекст domain-инструкции агентов.
+4. Для PR/MR сформируй `$CI_CONTEXT`: SHA head, названия, статусы, определимые команды и ссылки на автоматические CI-проверки; для failure добавь только значимый фрагмент лога, не передавай полные логи всех jobs. Учитывай только результаты для текущего head. Завершённую в CI проверку со статусом success или failure не запускай локально повторно: используй её результат и лог как доказательство. Локальный запуск допустим только для отсутствующей, отменённой, пропущенной, устаревшей или не сопоставимой с CI проверки либо по прямому указанию пользователя. Недоступный CI отметь warning и выполняй нужные проверки обычным способом. Для не-PR/MR target используй `CI_CONTEXT: none`.
 
 CI переиспользуется только как результат автоматических команд — тестов, линтеров, typecheck, сборки и аналогичных jobs. Он не заменяет выбранные смысловые проходы `correctness`, `architecture`, `tests` и другие: они по-прежнему анализируют diff и качество покрытия, но не дублируют уже завершённые CI-команды.
 
@@ -84,20 +87,41 @@ CI переиспользуется только как результат ав
 
 ## Выбрать проверки
 
-Сначала примени разовые overrides, затем настройки, затем defaults.
-
-- `always` включай всегда, `off` всегда пропускай.
-- `architecture` в `auto` включай при изменении границ модулей, зависимостей, публичных контрактов или разделения ответственности.
-- `references` включай только при непустом `$REFERENCE_FILES`.
-- `business` включай только при непустом `$BUSINESS_FILES` и изменении пользовательского или доменного поведения.
-- `plan_alignment` включай только при непустом `$PLAN_FILE`.
-- `security` включай для auth, permissions, внешнего ввода, файлов, сети, секретов, криптографии и других границ доверия.
-- `performance` включай для запросов, пакетной обработки, кешей, конкурентности, памяти или горячих путей.
-- `frontend` обязательно включай при изменении компонентов, стилей, клиентских маршрутов, состояния или другой UI-части.
-- `api` включай при изменении routes, handlers, DTO, OpenAPI, GraphQL, RPC или клиент-серверного контракта.
-- `database` включай при изменении миграций, моделей хранения, SQL, схем, индексов или транзакций.
-- `documentation` включай при изменении публичного API, CLI, конфигурации, установки или пользовательского поведения.
-- `previous_reviews` включай для URL PR/MR или другой удалённой цели, у которой могут быть обсуждения ревью.
+Фиксированного набора проверок нет: какие из них нужны, решается по содержимому ревью.
+
+### Выбор в subagents
+
+Первым, до пакета проверок, запусти одного установленного агента `eda-review-triage` с моделью роли `triage` по правилам запуска и технических остановок ниже. Сам target, `docs/references.md` и `docs/business.md` для выбора не читай и проверки сам не выбирай. Передай агенту только:
+
+```text
+Рабочая директория: <cwd>
+TARGET: <target>
+PLAN_FILE: <path | none>
+MODEL: <фактически выбранная модель>
+REVIEW_CONTEXT: <краткий общий контекст>
+CI_CONTEXT: <head SHA, CI-проверки со статусами, командами, ссылками и значимыми failure-логами | none>
+
+Верни результат строго по YAML-контракту установленного агента.
+```
+
+Проверь ответ: `status: completed`, `check: triage`, запрошенная модель, каждая проверка из таблицы настроек, кроме `triage`, ровно один раз в `selected` или `skipped` с причиной, пути карточек существуют. При `blocked`, неверном YAML или нарушении контракта один раз повтори того же агента на той же модели с описанием ошибки; повторный сбой — остановись без итогового файла.
+
+Из ответа возьми `$REFERENCE_FILES` и `$BUSINESS_FILES`, пустой список замени на `none`. `summary` добавь в `$REVIEW_CONTEXT` как изменённые области.
+
+### Выбор в main
+
+Прочитай `AGENTS.md`, `CLAUDE.md`, `docs/rules.md`, `docs/arch.md`, применимые к target карточки из их таблиц и target один раз: они станут общей основой выбора и всех проходов. Затем:
+
+1. Определи `$REFERENCE_FILES`: если план содержит `sources.references`, используй их; иначе через `docs/references.md` выбери только применимые карточки для компонентов target. Если карточек нет, используй `none`.
+2. Определи `$BUSINESS_FILES`: сначала установи фактически затронутое пользовательское или доменное поведение по текущему сообщению, плану и target. Если план содержит `sources.business`, используй только относящиеся к этому поведению пути; иначе прочитай `docs/business.md` и по описаниям выбери только применимые карточки. Если индекс отсутствует, подходящих карточек нет или изменения не затрагивают бизнес-поведение, используй `none`. Не загружай все карточки только потому, что индекс существует.
+3. Выбери проверки по [references/triage.md](references/triage.md) и добавь изменённые области в `$REVIEW_CONTEXT`.
+
+### Поверх выбора
+
+В обоих режимах затем примени:
+
+1. Детерминированные правила: `plan_alignment` выбран только при `$PLAN_FILE` не `none`, `references` — только при непустом `$REFERENCE_FILES`, `business` — только при непустом `$BUSINESS_FILES`, `previous_reviews` — для URL PR/MR или другой удалённой цели с обсуждениями. Расхождение с ответом triage исправь сам, повторно triage не запускай.
+2. Разовые указания пользователя последними: «без X» исключает проверку, «включи X» добавляет её с причиной «указано пользователем».
 
 Для каждой проверки сохрани причину `selected` или `skipped`; она попадёт в покрытие.
 
@@ -107,24 +131,24 @@ CI переиспользуется только как результат ав
 
 Всю работу выполни сам как основной агент текущего workflow. Не запускай `eda-review-*` агентов, другие субагенты или отдельный CLI. Вызывающий workflow не должен оборачивать `eda-review` в отдельного субагента, если передал режим `main`.
 
-Прочитай [references/review-matrix.md](references/review-matrix.md) целиком. Это обязательный контракт всех специализированных проходов в `main`; не читай packaged prompts из `agents/eda-review-*` и не пытайся вызвать их как скилы.
+Прочитай [references/defects.md](references/defects.md) один раз, а перед проходом каждой выбранной проверки — её чек-лист `references/checks/<проверка>.md`: имя файла — имя агента без префикса `eda-review-`, например `plan_alignment` → `references/checks/plan-alignment.md`. Это те же чек-листы и требования к находке, которые установщик встраивает в агентов для `subagents`. Не читай packaged prompts из `agents/eda-review-*` и не пытайся вызвать их как скилы.
 
-1. Прочитай target и общий код вокруг изменений один раз. Применимые инструкции, `$PLAN_FILE`, `$REFERENCE_FILES` и `$BUSINESS_FILES` также загрузи один раз и только когда соответствующая проверка выбрана. Соблюдай `$CI_CONTEXT`: не перезапускай автоматическую команду, уже завершённую в CI на текущем head.
-2. Последовательно выполни отдельный проход по каждой выбранной проверке из матрицы. Не смешивай области и не пропускай проход из-за того, что соседняя проверка уже изучала тот же код.
+1. Target и общие документы уже прочитаны при выборе; общий код вокруг изменений дочитай один раз. Применимые инструкции, `$PLAN_FILE`, `$REFERENCE_FILES` и `$BUSINESS_FILES` также загрузи один раз и только когда соответствующая проверка выбрана. Соблюдай `$CI_CONTEXT`: не перезапускай автоматическую команду, уже завершённую в CI на текущем head.
+2. Последовательно выполни отдельный проход по каждой выбранной проверке по её чек-листу. Не смешивай области и не пропускай проход из-за того, что соседняя проверка уже изучала тот же код.
 3. Для каждого прохода сохрани внутренний результат: `check`, `status`, `summary` и кандидаты в находки с `severity`, `explanation`, `evidence`, `repro`, `fix` и `tests`. YAML между проходами не нужен.
 4. Если проверка оказалась неприменима, перенеси её в skipped с конкретной причиной. Если обязательную проверку невозможно выполнить из-за недоступного target или контекста, остановись без итогового файла. Недоступные CI-результаты и предыдущие удалённые обсуждения преврати в warning и продолжай.
 5. После всех проходов проверь доказательства, дедуплицируй кандидатов, выставь score и сохрани итог по общим правилам ниже. Не запускай отдельный редакторский или подтверждающий проход.
 
 ### Режим subagents
 
-Имя агента получается из имени проверки:
+Имя агента получается из имени роли:
 
 - `plan_alignment` → `eda-review-plan-alignment`;
 - `code_quality` → `eda-review-code-quality`;
 - `previous_reviews` → `eda-review-previous-reviews`;
 - остальные → `eda-review-<check>`.
 
-В Claude Code запускай установленный custom agent через `Agent` tool. В Codex каждого установленного custom agent запускай через `spawn_agent` с `fork_turns: "none"`; не оставляй `fork_turns` по умолчанию и не передавай историю основного агента. Передай выбранную модель как runtime override. Если хост не умеет применить выбранную модель или нативные субагенты недоступны, остановись: не выполняй роли главным агентом и не создавай отдельные CLI-процессы.
+В Claude Code запускай установленный custom agent через `Agent` tool. В Codex каждого установленного custom agent запускай через `spawn_agent` с `fork_turns: "none"`; не оставляй `fork_turns` по умолчанию и не передавай историю основного агента. Передай модель и effort роли как runtime override по правилам раздела «Настройки»; если среда не может их применить, запусти агента с установленными моделью и effort и добавь warning в покрытие. Если нативные субагенты недоступны, остановись: не выполняй роли главным агентом и не создавай отдельные CLI-процессы.
 
 ### Технические остановки субагентов
 
@@ -139,7 +163,7 @@ CI переиспользуется только как результат ав
 
 Ожидание слота и один технический retry не считаются повторной содержательной проверкой агента.
 
-Сначала сформируй весь набор выбранных проверок, затем отправь все вызовы одним параллельным пакетом в одном ответе ассистента. Не дели проверки заранее на волны, партии или последовательные группы и не жди завершения части агентов перед запуском остальных. Если хост ограничивает concurrency, разреши нативному механизму поставить лишние задачи в очередь; повторный запуск допустим только после фактической ошибки лимита по правилам технических остановок выше.
+После ответа triage сформируй весь набор выбранных проверок, затем отправь все вызовы одним параллельным пакетом в одном ответе ассистента. Не дели проверки заранее на волны, партии или последовательные группы и не жди завершения части агентов перед запуском остальных. Если хост ограничивает concurrency, разреши нативному механизму поставить лишние задачи в очередь; повторный запуск допустим только после фактической ошибки лимита по правилам технических остановок выше.
 
 Ожидай результаты только нативным механизмом субагентов и их уведомлениями о завершении; в Codex используй `wait_agent`. Не используй Bash, shell-циклы, `sleep`, опрос файлов или артефактов и фиксированные таймауты для ожидания агентов. После отправки всего пакета собирай ответы по мере нативного завершения.
 
@@ -178,7 +202,7 @@ CI_CONTEXT: <head SHA, CI-проверки со статусами, команд
 
 1. Проверь находки по указанным evidence и отклони неподтверждённые.
 2. Объедини дубли разных агентов в один пункт, сохранив все полезные доказательства.
-3. Оставь только конкретные ошибки, нарушения контрактов и существенные неточности, которые нельзя безопасно игнорировать. Косметические улучшения, вкусовой рефакторинг, спекулятивное усиление, необязательные тесты, документацию и оптимизации отклони, а не превращай в находку.
+3. Оставь только конкретные ошибки, нарушения контрактов и существенные неточности, которые нельзя безопасно игнорировать. Косметические улучшения, вкусовой рефакторинг, спекулятивное усиление, необязательные тесты, документацию и оптимизации кода, которые изменение не ухудшило, отклони, а не превращай в находку. Рост потребления ресурсов, внесённый изменением, — например, больше запросов к БД или внешних вызовов на ту же операцию — не оптимизация, а дефект.
 4. Отклони находку про будущее: дефект должен существовать в текущем коде. «Сломается, если позже кто-то так сделает» без достижимого на текущем коде сценария — не находка.
 5. Для каждой принятой находки проверь точное место, доказательство, способ воспроизведения, практическое влияние, минимальное исправление и способ проверки. Если хотя бы одного элемента нет, отклони находку.
 6. Каждую отклонённую находку запиши в раздел «Отклонено» с проверкой-источником и причиной. В число находок и в score отклонённые не входят.
@@ -201,9 +225,11 @@ score: <0..100>
 status: <reviewed | reviewed-with-warnings>
 result: <clean | changes-required>
 date: <YYYY-MM-DD HH:MM>
+triage: <модель и effort | main>
 checks:
   - name: <check>
     model: <model>
+    effort: <effort | current>
     status: <completed | skipped | warning>
     reason: <почему запущен, пропущен или неполон>
 ---
@@ -217,7 +243,8 @@ checks:
 ## Покрытие
 
 - **Execution:** <subagents | main>
-- **Completed:** <проверки и модели>
+- **Triage:** <модель и effort разведчика | «main: проверки выбраны основным агентом»>
+- **Completed:** <проверки, модели и effort>
 - **Skipped:** <проверки и причины>
 - **Warnings:** <предупреждения или «нет»>
 - **CI:** <переиспользованные результаты текущего head, локально запущенные отсутствующие проверки и недоступные данные>
@@ -264,7 +291,8 @@ checks:
 
 - Править код, настройки проекта или удалённые обсуждения.
 - В `subagents` выполнять специализированную роль главным агентом или подменять отсутствующих субагентов одним общим ревью.
-- В `main` запускать leaf-субагентов, другой CLI или читать packaged prompts вместо `references/review-matrix.md`.
+- В `subagents` выбирать проверки самому вместо `eda-review-triage` или запускать пакет проверок до его ответа.
+- В `main` запускать leaf-субагентов, другой CLI или читать packaged prompts вместо чек-листов `references/checks/`.
 - Записывать итоговый файл после повторного сбоя обязательного агента в `subagents` или незавершённой обязательной проверки в `main`.
 - Выбирать последний PR или весь репозиторий по ссылке на корень.
 - Загружать все карточки references без отбора.

package/skills/eda-review/references/checks/api.md

@@ -0,0 +1,18 @@
+## Что проверять
+
+Проверяй обе стороны изменённого взаимодействия — того, кто отдаёт, и тех, кто потребляет: HTTP-эндпоинты, GraphQL, RPC, события и сообщения очередей, вебхуки, SDK. Каждое подозрение проверь сценарием: прежний допустимый запрос или ответ → конкретное изменение → путь существующего потребителя → другой результат или побочный эффект. Выводы делай только о потребителях, которых прочитал; непрочитанных назови в `summary` и не считай совместимыми.
+
+- **Ввод и валидация:** прежний допустимый запрос перестал приниматься: новое обязательное поле, сужение типа, enum, длины или диапазона, другое значение по умолчанию, иное поведение для отсутствующего поля и `null`, неизвестные поля теперь отвергаются. Новый необязательный параметр меняет поведение, когда его не передали.
+- **Ответ:** удалено или переименовано поле; изменились тип, nullability, формат даты или числа, набор значений enum; потребитель со строгой схемой или исчерпывающим разбором отвергает новое поле или значение.
+- **Ошибки:** изменились коды, формат тела или смысл ошибки; ошибка отдаётся с кодом успеха; изменилось, какие ошибки можно повторять.
+- **Права:** изменились требуемые роли, scope или проверки владельца; эндпоинт стал доступен шире, чем раньше.
+- **Пагинация:** нет детерминированного порядка с разрывом равенства, и записи дублируются или пропадают между страницами; изменились лимит по умолчанию, максимум, формат курсора или признак следующей страницы.
+- **Асинхронные операции:** путаются «принято» и «выполнено»; изменились статусы, конечные состояния или момент, когда эффект уже зафиксирован.
+- **Повторы:** повтор после таймаута, когда первая попытка уже выполнилась, создаёт второй эффект; ключ идемпотентности перестал учитываться или изменились его область и срок хранения.
+- **Версии и соглашения:** несовместимое изменение без новой версии там, где проект версионирует API; отступление от принятого в проекте формата ответов, ошибок, именования и пагинации.
+- **Описание контракта:** схема OpenAPI, GraphQL или protobuf, сгенерированный клиент или общий тип расходятся с фактическим обработчиком.
+
+## Не находка
+
+- Новое необязательное поле, которое прочитанные потребители допускают.
+- Общая бизнес-логика, безопасность и хранение данных вне контракта — отдельные проверки.

package/skills/eda-review/references/checks/architecture.md

@@ -0,0 +1,16 @@
+## Что проверять
+
+Проверяй изменение по `docs/arch.md`, применимым карточкам `docs/arch/` из его таблицы и разделу `Рамка` плана, если он есть. Находка допустима, только когда решение уже создаёт неправильную связанность, нарушение границы или несовместимый контракт с конкретным практическим риском.
+
+- **Границы и направление зависимостей:** модуль или слой обращается к тому, от чего по архитектуре зависеть не может; появился цикл зависимостей; доменный код тянет инфраструктуру, фреймворк или транспорт.
+- **Ответственность:** логика попала в компонент, который по архитектуре за неё не отвечает (бизнес-правило в контроллере, шаблоне или миграции); одно правило теперь реализовано в двух компонентах.
+- **Чужие внутренности:** код читает внутреннее представление другого модуля — его поля, таблицы, формат хранения — вместо публичной операции, и изменение того модуля заставит менять этот.
+- **Контракты между модулями:** изменены интерфейс, событие или формат обмена без обновления всех потребителей в репозитории.
+- **Новые зависимости и сервисы:** добавлены библиотека, сервис, очередь или хранилище, которых нет в архитектуре и плане.
+- **Рамка плана:** решение противоречит архитектурному пункту раздела `Рамка`.
+
+## Не находка
+
+- Предпочтение другой структуры без нарушения документированной границы и без риска.
+- Существующая зависимость, которую изменение не расширило.
+- Корректность, стиль и тесты — отдельные проверки.

package/skills/eda-review/references/checks/business.md

@@ -0,0 +1,13 @@
+## Что проверять
+
+Проверяй изменённое пользовательское и доменное поведение только по переданным business-карточкам; требования соседних карточек не добавляй и область задачи не расширяй. Намерение изменений устанавливай по задаче, плану и контексту ревью; если задача неизвестна, не выдумывай его.
+
+- **Нарушение правила:** реализация нарушает применимое подтверждённое правило карточки, а задача или план его не меняют, — дефект бизнес-логики.
+- **Неполное правило:** реализованы не все ветки правила из карточки — исключения, ограничения, состояния и переходы, сроки, лимиты.
+- **Рассинхронизация:** задача или план явно меняют правило, но карточка не обновлена, — рассинхронизация business-документации, а не автоматическое доказательство неверной реализации; исправление — пересогласовать карточку через `eda-business`.
+
+Если задача неизвестна, не объявляй правило намеренно изменённым, но сверку с действующей карточкой продолжай.
+
+## Не находка
+
+- Общая корректность, архитектура, стиль кода, технические references и документация вне business-карточек.

package/skills/eda-review/references/checks/code-quality.md

@@ -0,0 +1,18 @@
+## Что проверять
+
+Проверь, что вносит изменение; эти случаи доказуемы и являются находками:
+
+- **Повтор существующего:** новая реализация того, что в проекте уже есть, — функции, хелпера, компонента, валидации, константы или клиента внешнего сервиса; в evidence укажи путь к существующей реализации. Ищи только для новых функций, хелперов, компонентов и констант из diff — по имени и назначению, без обхода всего проекта.
+- **Копия логики:** скопированный блок внутри diff или из существующего кода вместо общего вызова, когда копии уже расходятся или задают одно и то же правило в двух местах.
+- **Мёртвый код:** неиспользуемые функции, экспорты, параметры, ветки, недостижимые после изменения, закомментированный код.
+- **Отладочный код:** временный вывод (`console.log`, `dd()`, `var_dump`, `print`), заглушки, отключённые проверки, захардкоженные тестовые значения.
+- **Абстракция на будущее:** интерфейс с единственной реализацией, параметр или флаг, который никто не передаёт, точка расширения без второго пользователя, слой, который только передаёт вызов дальше, — если их добавило изменение и их не требует план.
+- **Чужой способ:** расхождение с устоявшимся в модуле способом обработки ошибок, логирования, чтения конфигурации или доступа к данным, когда в соседнем коде есть прямой образец; в evidence укажи образец.
+- **Имя противоречит поведению:** `isValid` что-то меняет, `getUser` создаёт запись, `count` возвращает список — и вызывающий код уже полагается на смысл имени.
+- **Непроверяемая сложность:** сложность или размер, которые уже скрывают ошибку либо делают изменённое поведение непроверяемым тестом.
+
+## Не находка
+
+- Вкусовые предпочтения и мелкие стилистические замечания.
+- Функциональные баги, архитектура, производительность и правила проекта — отдельные проверки.
+- Структура, которая не создаёт конкретного риска ошибок или тестовых пробелов.

package/skills/eda-review/references/checks/correctness.md

@@ -0,0 +1,19 @@
+## Что проверять
+
+Ищи реальные ошибки поведения изменённого кода. Молчание задачи или плана — не разрешение: если в ситуации, о которой план молчит, поведение расходится с ожиданием разумного пользователя, это дефект.
+
+- **Логика и условия:** инвертированное или неполное условие, перепутанные `и`/`или`, ошибка на единицу в границах, циклах, срезах и пагинации, неверный порядок операций.
+- **Граничные значения:** пустая строка или коллекция, `null`, `undefined` или отсутствующий ключ, ноль, отрицательное и максимальное значение, первый и последний элемент, одна запись вместо многих.
+- **Состояния:** недопустимый переход, состояние без выхода, повторная обработка уже обработанного, флаги, которые расходятся между собой.
+- **Тихие сбои:** пустой `catch`; `catch`, который только логирует и продолжает так, будто всё прошло успешно; пустое значение или значение по умолчанию вместо ошибки; слишком широкий `catch`, который глотает чужие ошибки; `?.` или `??`, которые молча пропускают обязательное действие; подмена заглушкой вне тестов; повторы, которые исчерпались без сообщения.
+- **Частичный сбой:** операция из нескольких шагов прерывается посередине и оставляет данные несогласованными, без отката, компенсации или безопасного повтора.
+- **Гонки:** проверка и запись разделены (прочитал остаток — списал) без атомарной операции, блокировки или ограничения уникальности; повторный запрос создаёт дубль.
+- **Повторы:** повтор после таймаута, когда первая попытка уже выполнилась, создаёт второй эффект; ключ идемпотентности генерируется заново для повтора той же операции.
+- **Время, деньги, текст:** часовые пояса и переход на летнее время, сравнение дат в разных зонах, деньги во float, округление, деление на ноль, переполнение, регистр и кодировка при сравнении строк, локаль при разборе чисел и дат.
+- **Ресурсы:** файлы, соединения, подписки и таймеры не освобождаются на пути ошибки.
+- **Кеш:** изменение данных не сбрасывает и не обновляет кеш, который их хранит; ключ кеша не учитывает параметр, от которого зависит результат.
+
+## Не находка
+
+- Сценарий, который невозможен из-за гарантий вызывающего кода, подтверждённых в коде.
+- Архитектура, стиль, документация, безопасность и соответствие плану как отдельные темы: упоминай их, только когда они прямо доказывают ошибку поведения.

package/skills/eda-review/references/checks/database.md

@@ -0,0 +1,17 @@
+## Что проверять
+
+Проверяй схемы, миграции, запросы и транзакции изменения с учётом существующих данных, конкурентных операций и порядка выкладки. Миграции не выполняй и базу не меняй.
+
+- **Существующие данные:** новое `NOT NULL` без значения по умолчанию или заполнения существующих строк; новое ограничение уникальности или внешний ключ, которому противоречат текущие данные; сужение типа или длины обрезает данные; заполнение не учитывает `NULL`, дубли и удалённые записи.
+- **Выкладка без простоя:** старая версия кода, которая работает во время выкладки, читает или пишет удалённую или переименованную колонку; переименование вместо добавления новой колонки и последующего удаления старой; миграция и код требуют определённого порядка выкладки, а он не обеспечен.
+- **Блокировки:** долгая блокировка большой таблицы — индекс без неблокирующего режима (`CONCURRENTLY` в PostgreSQL, online DDL в MySQL), смена типа колонки, заполнение всей таблицы одной транзакцией миграции.
+- **Обратимость:** откат миграции невозможен или теряет данные, хотя проект делает откаты; откат не соответствует прямой миграции.
+- **Схема и модель:** nullable, default, тип и длина в миграции расходятся с моделью, валидацией и планом; деньги во float; время без часового пояса там, где проект хранит его с поясом.
+- **Целостность:** инвариант держится только проверкой в коде (проверил — вставил) без ограничения уникальности, внешнего ключа или блокировки, и параллельные запросы его нарушают; каскадное удаление уносит данные, которые должны остаться, или его нет и остаются сироты; мягкое удаление не учтено в новых запросах и уникальных индексах.
+- **Транзакции:** операции, которые должны быть атомарными, выполняются вне одной транзакции; внешний вызов, письмо или событие отправляются внутри транзакции, которая потом может откатиться; уровень изоляции не защищает от гонки, которую создаёт изменение.
+- **Запросы и индексы:** новый запрос фильтрует, сортирует или соединяет большую таблицу по колонкам без индекса — размер таблицы подтверди данными проекта, миграциями или документацией; функция над колонкой не даёт использовать индекс; новый индекс дублирует существующий.
+
+## Не находка
+
+- Требование индекса без обоснованного запроса и объёма данных.
+- Число запросов на операцию и N+1 — `performance`; логика без последствий для хранения — `correctness`.

package/skills/eda-review/references/checks/documentation.md

@@ -0,0 +1,13 @@
+## Что проверять
+
+Ищи только документацию, которую обязательно обновить из-за изменения. Сопоставь поведение после изменения с документами, которые его описывают: README, руководства, справка CLI, примеры конфигурации, описание API, `docs/`, CHANGELOG и заметки о миграции, если проект их ведёт.
+
+- **Устаревшая инструкция:** документ описывает команду, флаг, параметр, endpoint, настройку или шаг установки, которые изменение удалило, переименовало или изменило, и пользователь, следуя ему, получит ошибку.
+- **Новое без описания:** новая обязательная переменная окружения, настройка, команда или шаг развёртывания не описаны там, где проект описывает аналогичные.
+- **Несовместимость без заметки:** изменение ломает существующих пользователей, а проект ведёт CHANGELOG или руководство по миграции, и записи в нём нет.
+- **Сломанный пример:** пример кода или конфигурации в документации перестал работать.
+
+## Не находка
+
+- Комментарии и документация для внутренних очевидных изменений; редакторская правка текста.
+- Правила и эталоны — `rules` и `references`.

package/skills/eda-review/references/checks/frontend.md

@@ -0,0 +1,19 @@
+## Что проверять
+
+Проверяй по исходникам изменённые компоненты, состояние, формы, действия, клиентские данные и стили. Рассмотри основной и ошибочные пользовательские сценарии. Не утверждай, что видел интерфейс: браузерная проверка относится к `eda-manual-test`.
+
+- **Состояния:** у асинхронных данных нет состояния загрузки, пустых данных, ошибки или успеха; при ошибке запроса интерфейс остаётся в загрузке или ничего не сообщает.
+- **Действия:** повторное нажатие отправляет действие дважды; кнопка доступна во время отправки; после ошибки форма теряет введённые данные; оптимистичное обновление не откатывается при ошибке.
+- **Асинхронность:** ответ на устаревший запрос перетирает более новый (быстрый ввод в поиске, смена фильтра, переход между страницами); запрос продолжает менять состояние после ухода со страницы.
+- **Формы:** ошибки серверной валидации не привязаны к полям; клиентская проверка расходится с серверной; у поля нет подписи или `name`, и не работают автозаполнение и экранный диктор.
+- **Утечки:** подписки, таймеры, обработчики событий и наблюдатели не снимаются при размонтировании.
+- **Доступность:** интерактивный элемент сделан не кнопкой или ссылкой и недоступен с клавиатуры; нет видимого фокуса; модальное окно не удерживает и не возвращает фокус; у содержательного изображения нет `alt`; у кнопки-иконки нет доступного имени.
+- **Вёрстка:** фиксированные размеры ломают узкий экран; изображение или блок без размеров сдвигает вёрстку при загрузке; длинный текст или перевод не помещается.
+- **Маршруты и роли:** прямой переход по URL открывает экран без нужных данных или с ошибкой; элемент скрыт по роли только визуально, а действие на клиенте выполняется.
+- **Ошибки наружу:** пользователю показывается сырой текст исключения, stack trace или ответ сервера; отладочный вывод в консоль остаётся в production-сборке.
+- **SSR и гидратация:** разметка сервера и клиента расходится (текущая дата, случайное значение, обращение к `window` при серверном рендере).
+
+## Не находка
+
+- Общая проблема корректности, API или тестов, не специфичная для интерфейса.
+- Вкусовые замечания к дизайну без нарушения сценария, доступности или вёрстки.

package/skills/eda-review/references/checks/infra.md

@@ -0,0 +1,16 @@
+## Что проверять
+
+Проверяй только затронутые изменением инфраструктурные файлы и связанные с ними места приложения: порт, переменные окружения, healthcheck, команду запуска и пути статики. Полный аудит инфраструктуры проекта не проводи. Проектные решения — канонический хост, политику редиректов, базовые образы, обязательные переменные — бери из `docs/rules.md` и `docs/arch.md`; универсальные требования ниже их не заменяют. Если в проекте настроены hadolint, actionlint, gixy, `nginx -t` или валидаторы compose, Kubernetes и Terraform, используй их результат из CI или безопасный локальный запуск.
+
+- **Образ:** базовый образ закреплён версией, а не `latest`; зависимости ставятся по lock-файлу; инструменты сборки и исходники не попадают в итоговый образ, если есть отдельная стадия сборки; процесс работает не от root; секреты не попадают в слои через `ARG`, `ENV`, `COPY` или историю; `.dockerignore` исключает `.env`, `.git` и локальные артефакты; `CMD` и `ENTRYPOINT` в exec-форме, чтобы сигнал остановки доходил до процесса.
+- **Оркестрация:** порт приложения совпадает в коде, `EXPOSE`, compose, сервисе и upstream прокси; данные БД и загрузок лежат в постоянном томе; порты БД, кеша, очередей, отладки и админок не опубликованы наружу без необходимости, а нужные на хосте привязаны к `127.0.0.1`; зависимый сервис ждёт готовности, а не только старта; healthcheck проверяет готовность и обращается к существующему пути; логи идут в stdout и stderr, а не в файлы внутри контейнера, и их размер ограничен ротацией.
+- **Прокси и TLS:** HTTP отдаёт 301 на HTTPS; все варианты хоста отдают 301 на канонический из `docs/rules.md` с сохранением пути и query; если канонический хост не зафиксирован, оба варианта не должны отвечать 200 одним содержимым; HSTS только на HTTPS; приложение получает `Host`, `X-Forwarded-For` и `X-Forwarded-Proto`; `add_header` во вложенном блоке не теряет заголовки родителя, в том числе заголовки безопасности; `alias` и `proxy_pass` не допускают обход пути; лимит размера тела и таймауты прокси не меньше, чем требует изменённый сценарий; WebSocket получает заголовки upgrade.
+- **Отдача:** текстовые ответы (HTML, CSS, JS, JSON, SVG) сжимаются gzip или brotli; статика с хешем в имени отдаётся с долгим `Cache-Control` и `immutable`, а HTML — без долгого кеша; статику отдаёт веб-сервер или CDN, а не процесс приложения; изменённый конфиг не отключает keep-alive и HTTP/2; sourcemap-файлы не публикуются, если проект их скрывает.
+- **Production-настройки:** debug выключен в production-шаблоне окружения (`APP_DEBUG`, `DEBUG`, `NODE_ENV`); если изменены образ или конфиг runtime, opcache или его аналог включён и ему хватает памяти, а лимиты процессов PHP-FPM, воркеров и пула соединений соответствуют памяти сервера; деплой собирает кеши фреймворка (`config:cache`, `route:cache`, `view:cache` и аналоги) и ставит зависимости без dev-пакетов с оптимизированным автозагрузчиком, если проект так делает.
+- **CI/CD:** секреты не печатаются в лог и не передаются недоверенному коду из форка; сторонние actions и образы закреплены версией; права токена не шире нужных; деплой идёт после обязательных проверок; миграции выполняются в порядке, совместимом с выкладкой; после выкладки проверяется работоспособность.
+- **Окружение:** новая или переименованная переменная окружения, которую читает код, задана во всех шаблонах и манифестах окружений проекта или имеет безопасный default; секреты не закоммичены в конфиги.
+
+## Не находка
+
+- Уязвимости кода приложения — `security`.
+- Инструмент, процесс или усиление, которых нет в проекте (лимиты ресурсов, отдельные сети, скрипты отката), без конкретного дефекта изменения.

package/skills/eda-review/references/checks/performance.md

@@ -0,0 +1,21 @@
+## Что проверять
+
+Ищи рост потребления ресурсов, который вносит изменение. Для каждой изменённой точки входа — обработчика запроса, страницы, job, консольной команды или обработчика события — сравни base и head по одному и тому же сценарию.
+
+- **Число операций:** запросы к БД, HTTP- и RPC-вызовы, сообщения в очереди, обращения к кешу, файловые операции и подпроцессы на одну операцию. Считай по самому diff: в hunk видны удалённые и добавленные строки. Версию base читай только когда из hunk не видно, оказался ли вызов внутри цикла или на другом пути.
+- **Рост от данных:** ввод-вывод внутри цикла, ленивая загрузка связей, повторная загрузка уже полученных данных.
+- **Без ограничения:** выборка или обработка без limit или пагинации, загрузка всей коллекции или файла в память вместо потока.
+- **Синхронный путь:** новые последовательные внешние вызовы на пути запроса; внешний вызов без таймаута; письма, отчёты и обработка файлов в запросе вместо очереди, когда очередь в проекте есть.
+- **Кеш:** результат пересчитывается на каждом запросе, хотя аналогичный в модуле кешируется; кеш без срока жизни или с ключами, число которых растёт без предела; одновременный пересчёт истёкшего ключа на горячем пути.
+- **Отдача фронтенда:** заметно растёт вес CSS, JS или шрифтов, которые грузятся на каждой странице (библиотека целиком ради одной функции, стили без удаления неиспользуемого, импорт всего пакета вместо модуля); изображение основного экрана без сжатия, размеров и подходящего формата; новый скрипт блокирует рендер в `<head>`; новая статика без хеша в имени, и её нельзя кешировать надолго.
+- **Конкурентность и память:** блокировка удерживается на время внешнего вызова; глобальная блокировка на горячем пути; память растёт с числом запросов (неограниченный внутренний кеш, накопление в статическом поле).
+
+Рост потребления ресурсов, внесённый изменением, — дефект, а не оптимизация: было 2 запроса — стало 20, было N запросов на список — стало N+1, появился внешний вызов на каждом запросе, CSS вырос с 80 до 680 КБ. Находка называет точку входа, счёт или размер до и после и условие роста.
+
+Время выполнения подтверждай только измерением: результатом бенчмарка или профиля из CI, безопасным локальным замером или существующим тестом на число запросов. Без измерения называй рост числа операций, размера или сложности, а не «станет медленнее». Вес фронтенда подтверждай размером собранного файла из CI или безопасной локальной сборки либо размером добавленной зависимости.
+
+## Не находка
+
+- Неоптимальность, которая уже была в base и не ухудшилась.
+- Микрооптимизация без измеримого или очевидного эффекта на фактических объёмах проекта.
+- Индексы, блокировки таблиц и транзакции — `database`.

package/skills/eda-review/references/checks/plan-alignment.md

@@ -0,0 +1,16 @@
+## Что проверять
+
+Проверяй реализацию только по конкретному плану. Прочитай его целиком: `Рамку`, `Целевой алгоритм`, контракты, фазы, тесты, документацию и `Принятые решения`.
+
+- **Не сделано:** отсутствует или сделан частично пункт плана, критерий готовности, шаг целевого алгоритма, таблица или поле, endpoint, тест, миграция, документация или эксплуатационный шаг; файл, который план называет изменяемым, не изменён.
+- **Сделано не так:** поведение расходится с планом — другой порядок шагов алгоритма, пропущенная ветка «Если …» или ошибка, другие поля, типы и коды ответов.
+- **Сделано лишнее:** поведение, механизм, зависимость или изменение вне плана и подтверждённых решений.
+- **Решение нарушено:** реализация противоречит пункту `Принятых решений`.
+- **Отчёт — не доказательство:** заявления исполнителя, комментарии и сообщения коммитов проверяй по коду; обоснование вроде «оставил так для простоты» не снижает серьёзность.
+- **Дефект по плану:** если план сам предписал дефект — тест без проверок, дублирование логики, небезопасное решение, — это находка с пометкой «по плану»: решение за человеком.
+
+## Не находка
+
+- Отличие от буквы плана, если цель сохранена без дополнительного риска.
+- Выполненные пункты: их не перечисляй.
+- Общее архитектурное или стилевое ревью — отдельные проверки.

package/skills/eda-review/references/checks/previous-reviews.md

@@ -0,0 +1,9 @@
+## Что проверять
+
+Найди read-only средствами незакрытые комментарии и треды ревью выбранного PR/MR и сверь каждый с текущим кодом.
+
+- **Всё ещё актуально:** проблема из замечания существует в текущем коде — находка со ссылкой на тред и подтверждением в коде.
+- **Исправлено, но тред открыт:** код уже исправлен — не находка; перечисли такие треды отдельно от находок.
+- **Невозможно проверить:** замечание не привязано к коду или требует недоступного контекста — не находка; упомяни в `summary`.
+
+Не отправляй комментарии, не скрывай сводки, не resolve/close треды и не меняй удалённое состояние. Если сервис, авторизация или формат обсуждений недоступны, верни warning и не утверждай, что история проверена.