@gian-tiaga/eda 0.8.6 → 1.0.1

package/skills/eda-plan-execute/SKILL.md

@@ -0,0 +1,186 @@
+---
+name: eda-plan-execute
+description: 'Управляет выполнением готового плана из docs/plans/: проходит фазы строго последовательно, строит волны задач по зависимостям и запускает безопасно совместимые задачи параллельно в отдельных изолированных субагентах. Разработку, исправления и все проверки выполняют только субагенты; основной агент координирует их, ведёт чекбоксы плана и журнал в docs/executions/. Не коммитит и не делает ревью.'
+---
+
+# Скил: Управляющий выполнением плана (eda-plan-execute)
+
+Оркестрируй выполнение готового плана. Сам не реализуй задачи и не запускай проверки проекта: передавай каждую задачу, исправление и проверку новому изолированному субагенту. Сам выбирай готовые к запуску задачи, связывай результаты, обновляй прогресс плана и журнал выполнения.
+
+## Вход из сообщения пользователя
+
+Текст рядом с вызовом скилла в текущем сообщении пользователя — главный вход. Сначала разбери именно его: тему, задачу, путь к файлу, режим, ограничения и прямые указания.
+
+Если этого входа достаточно, продолжай без вопроса. `AskUserQuestion` задавай только если входа нет, он противоречивый, найдено несколько равных вариантов или есть риск выполнить не тот план. Старое обсуждение в истории не считай входом, если пользователь не связал его с текущим вызовом явно.
+
+## Главные правила
+
+1. **Основной агент только управляет.** Разрешены чтение контекста, построение очереди, запуск и контроль субагентов, вопросы пользователю, обновление `$PLAN_FILE` и `$EXECUTION_FILE`. Код, тесты, миграции, конфиги, зависимости, документацию проекта и git-состояние меняют только назначенные субагенты.
+2. **Фазы выполняй строго последовательно.** Не запускай задачу следующей фазы, пока все задачи текущей фазы не завершены и отдельная проверка фазы не прошла.
+3. **Каждая работа изолирована.** Каждую задачу плана, подготовку ветки/worktree, исправление после падения, проверку фазы и финальную проверку запускай новым субагентом со свежим контекстом без истории диалога.
+4. **Безопасную параллельность используй обязательно.** Внутри текущей фазы одновременно запускай все готовые задачи, которые план взаимно пометил как параллельные и которые не пересекаются по изменяемым файлам, контрактам, данным или общему состоянию. Остальные запускай последовательными волнами по зависимостям.
+5. **План и проектная документация — обязательная рамка исполнения.** Перед запуском прочитай план, `docs/rules.md`, `docs/arch.md` и карточки из `sources.references`; для старого плана выбери через `docs/references.md` только применимые карточки.
+6. **Соблюдай приоритет и блокируй конфликт.** Приоритет: подтверждённое решение пользователя → `docs/rules.md` → `docs/arch.md` → релевантные карточки. При конфликте или необоснованном отклонении от примера применимой карточки не поручай реализацию: спроси, адаптировать план, вернуться к `eda-plan` или отдельно изменить документацию.
+7. **Отмечай только подтверждённое.** Ставь задаче `[x]` только после `completed` от исполнителя и успешных обязательных проверок задачи. Фазу закрывай только после отдельного `passed` от проверяющего фазы. При последующем опровержении верни отметку в незавершённое состояние.
+8. **Проверки нельзя подавлять.** Не поручай отключать линтеры, тесты, typecheck, static analysis и другие проверки, добавлять игноры или ослаблять команды и правила. Причину падения исправляет отдельный субагент, затем новый субагент повторяет проверку.
+9. **Не коммить, не пушь и не делай ревью.** Это другие скилы.
+
+## Интерактивные вопросы
+
+Когда инструкция говорит `AskUserQuestion`, это означает блокирующий интерактивный вопрос.
+
+- Claude Code: используй `AskUserQuestion`.
+- Codex interactive: если доступен `request_user_input`, используй его. Если tool недоступен, задай один короткий вопрос в чат, дай варианты 1–3, напиши «Ответь номером или своим вариантом. Я продолжу только после ответа.» и остановись.
+- Codex exec / неинтерактивный запуск: не задавай вопросы и не пытайся читать stdin. Если без ответа нельзя безопасно продолжать, заверши работу со статусом `blocked: нужен ответ пользователя`, перечисли вопросы и варианты, не выполняй рискованные действия.
+- Не запускай команды, которые ждут интерактивного ввода в терминале.
+
+## Этапы
+
+### 1. Выбрать план
+
+- Если указан путь к плану — используй его как `$PLAN_FILE`.
+- Если указано название или часть названия — найди совпадение в `docs/plans/`.
+- Если написано «последний план» — возьми самый новый файл из `docs/plans/`.
+- Если вставлен текст плана — сохрани или однозначно привяжи его к файлу в `docs/plans/` до инициализации прогресса.
+
+Если найден ровно один план, продолжай без вопроса. При нескольких равных вариантах используй `AskUserQuestion`. Если плана нет, остановись со статусом `blocked: нужен план`.
+
+Не исполняй план со статусом `draft`, открытыми вопросами, отсутствующими ID задач или неразрешёнными зависимостями. Попроси сначала завершить `eda-plan` или `eda-plan-polish`.
+
+### 2. Выбрать место выполнения
+
+Через `AskUserQuestion` предложи текущую ветку, новую ветку от `main`, новую ветку от текущей или worktree от `main`. Если вызывающий оркестратор уже передал подтверждённый выбор, не спрашивай повторно.
+
+Сам git-команды не выполняй. Для выбранной новой ветки или worktree запусти отдельного изолированного субагента подготовки окружения. Передай ему только выбранный вариант, slug плана, рабочую директорию и запрет менять код. Если выбран worktree, после успешного создания остановись и дай готовую команду перехода и нового запуска `eda-plan-execute <PLAN_FILE>`: текущая сессия не переезжает в другую директорию.
+
+### 3. Прочитать рамку и построить граф
+
+Прочитай `$PLAN_FILE`, `docs/rules.md` и `docs/arch.md`, если они есть. Прочитай перечисленные в `sources.references` карточки. Для старого плана без поля прочитай индекс `docs/references.md` и выбери только карточки для затронутых компонентов.
+
+Для каждой фазы извлеки задачи, ID, прямые зависимости, поле `Параллельно`, ожидаемый результат, сценарии тестирования и проверки. До запуска проверь:
+
+- все ID уникальны, а ссылки ведут на существующие задачи;
+- зависимости не образуют цикл и не ведут в будущую фазу;
+- параллельность симметрична и не противоречит зависимостям;
+- параллельные задачи не меняют одни файлы или участки, общий контракт, одну миграционную цепочку, общие генерируемые артефакты или иное изменяемое состояние;
+- шаги не конфликтуют с правилами, архитектурой и применимыми карточками;
+- у каждой фазы есть проверяемый результат и команда либо критерий проверки.
+
+Если граф неоднозначен или небезопасен, не исправляй смысл плана молча. Используй `AskUserQuestion` с предложением вернуть план в `eda-plan` или `eda-plan-polish`.
+
+### 4. Инициализировать прогресс
+
+Создай `$EXECUTION_FILE` по пути `docs/executions/{YYYY-MM-DD}_{HH-MM}_{slug}.md`. В `$PLAN_FILE` добавь или актуализируй один раздел `## Прогресс выполнения` со ссылкой на журнал, фазами и задачами по исходным ID.
+
+Используй в журнале структуру:
+
+```markdown
+---
+plan: <путь>
+started: <YYYY-MM-DD HH:MM>
+status: in_progress
+current_phase: <номер>
+---
+
+# Журнал выполнения: <заголовок>
+
+## Фазы
+| Фаза | Задачи | Проверка | Статус |
+|---|---|---|---|
+
+## Запуски субагентов
+| Время | Роль | Задача | Результат | Файлы | Проверки |
+|---|---|---|---|---|---|
+
+## Решения и блокеры
+## Изменения в документации
+## Финальная проверка
+```
+
+Основной агент пишет только этот журнал и отметки прогресса в плане. Субагентам явно запрети менять `$PLAN_FILE` и `$EXECUTION_FILE`.
+
+### 5. Запускать изолированных субагентов
+
+В Codex используй `spawn_agent` с `fork_turns: "none"` или точный аналог. В Claude Code используй новый `Agent` с самодостаточным task-сообщением. Если чистая изоляция или субагенты недоступны, остановись с `blocked: недоступно изолированное выполнение`; не выполняй работу основным агентом и не добавляй CLI-fallback.
+
+Каждому исполнителю задачи передай только:
+
+- рабочую директорию, `$PLAN_FILE`, ID и полный текст одной задачи;
+- цель и критерии готовности текущей фазы;
+- прямые зависимости и краткие результаты уже завершённых зависимостей;
+- пути к `docs/rules.md`, `docs/arch.md`, применимым карточкам и другим явно нужным источникам;
+- команды или критерии проверки этой задачи;
+- ограничения пользователя, запрет коммитить и запрет менять план и журнал.
+
+Исполнитель сам читает переданные файлы, исследует нужный код, реализует только назначенную задачу и выполняет её точечные проверки. Требуй YAML-результат:
+
+```yaml
+status: completed | needs_input | failed | blocked
+task: <ID>
+summary: <что сделано>
+files_changed: [<пути>]
+checks:
+  - command: <команда или критерий>
+    status: passed | failed | not_run
+    result: <кратко>
+deviations: [<отклонения от плана или пусто>]
+risks: [<остаточные риски или пусто>]
+questions: [<вопросы или пусто>]
+```
+
+Не принимай `completed`, если обязательная проверка задачи не прошла, исполнитель вышел за назначенный scope, молча отклонился от рамки или изменил план/журнал.
+
+### 6. Выполнять фазы волнами
+
+Для каждой фазы по порядку:
+
+1. Вычисли готовые задачи: все их прямые зависимости уже отмечены выполненными.
+2. Из готовых задач собери максимальную безопасную волну по взаимным полям `Параллельно` и отсутствию пересечений. Не включай задачу в параллельную волну только по догадке, если план не разрешает параллельность явно.
+3. Запусти всех исполнителей волны одновременно одним batch до чтения отдельных результатов. Если лимит среды меньше размера волны, используй максимально доступное число слотов и запускай остаток сразу после освобождения слотов.
+4. После завершения каждого исполнителя проверь контракт, фактический scope файлов и результат точечных проверок. Запиши запуск в журнал и поставь `[x]` только успешной задаче.
+5. `needs_input` передай пользователю одним блокирующим вопросом и после ответа продолжи ту же задачу новым изолированным субагентом с ответом и предыдущим результатом.
+6. Для `failed` запусти нового изолированного субагента исправления этой же задачи. Передай исходную задачу, полный неуспешный результат и актуальное состояние файлов. После исправления новый субагент должен повторить точечную проверку. Если две попытки подряд возвращают ту же причину без содержательного изменения результата, остановись с `blocked: выполнение не сходится` и зафиксируй доказательства вместо бесконечного цикла.
+7. Для `blocked` останови зависимые задачи. Если блокер внешний или требует изменения scope, задай вопрос; основной агент не устраняет его содержательно.
+8. Когда волна завершена, пересчитай готовые задачи. Не переходи к следующей фазе, пока текущая не закрыта.
+
+Если параллельные исполнители неожиданно пересеклись по файлам или контракту, не объединяй результаты вслепую. Останови затронутые задачи, запиши конфликт и запусти отдельного субагента для диагностики и безопасного согласования изменений.
+
+### 7. Проверять каждую фазу
+
+После выполнения всех задач фазы запусти нового изолированного субагента-проверяющего. Он не меняет файлы, читает итог фазы, выполняет все проверки из раздела фазы и подтверждает критерии результата. Требуй YAML:
+
+```yaml
+status: passed | failed | blocked
+phase: <номер>
+checks:
+  - command: <команда или критерий>
+    status: passed | failed
+    result: <кратко>
+failures: [<проблемы с доказательствами или пусто>]
+```
+
+При `failed` сопоставь проблемы с задачами и запусти по каждой отдельного изолированного субагента исправления. Затем запусти нового проверяющего всей фазы. Основной агент не чинит код и не переиспользует контекст прежнего проверяющего. При `passed` отметь фазу завершённой и только тогда переходи к следующей.
+
+### 8. Выполнить финальную проверку
+
+После всех фаз запусти отдельного нового субагента для полного набора тестов, линтеров, typecheck, static analysis и других обязательных команд проекта. Он только проверяет и не меняет файлы.
+
+Если финальная проверка упала из-за сделанных изменений, запусти изолированных субагентов исправления по независимым группам ошибок, затем повтори всю финальную проверку новым субагентом. Чужую или несвязанную поломку зафиксируй и используй `AskUserQuestion`, если без решения нельзя завершить план. Запрещено делать проверки зелёными через подавление ошибок, игноры, исключения, ослабление конфигов или изменение команд проверки.
+
+Запиши команды и результаты в `## Финальная проверка`. Только после `passed` поставь в журнале `status: done` и время завершения. Если обязательная проверка объективно недоступна, сохрани `status: blocked`, причину и непройденные критерии; не выдавай план за выполненный.
+
+### 9. Финал
+
+Коротко сообщи пути к плану и журналу, завершённые фазы и задачи, какие волны выполнялись параллельно, результат фазовых и финальной проверок, оставшиеся блокеры, состояние git «не закоммичено» и следующий шаг: `eda-review` или `eda-commit`.
+
+## Чего НЕ делать
+
+- Самому менять код, тесты, миграции, конфиги, зависимости или проектную документацию.
+- Самому запускать тесты, линтеры, typecheck, сборку, миграции, серверы или другие команды разработки и проверки.
+- Передавать одному субагенту несколько независимых задач только ради экономии запусков.
+- Запускать следующую фазу до успешной отдельной проверки текущей.
+- Запускать параллельно задачи без явной безопасной маркировки плана или с пересечением изменяемых ресурсов.
+- Давать субагентам историю диалога или разрешать им менять план и журнал.
+- Исправлять падение основным агентом либо продолжать после него без повторной проверки новым субагентом.
+- Подавлять ошибки проверок, ослаблять правила или менять команды ради зелёного результата.
+- Коммитить, пушить, создавать PR или делать ревью.

package/skills/{eda-plan-polish.md → eda-plan-polish/SKILL.md}

@@ -1,11 +1,11 @@
 ---
 name: eda-plan-polish
-description: 'Полирует готовый план из `docs/plans/...` до оценки 95/100 или заданного порога. Каждая итерация начинается с трёх изолированных параллельных агентов на быстрой, кодовой и флагманской моделях. В strict дополнительно запускает кросс-CLI ревью плана: Claude отдаёт план Codex, Codex отдаёт план Claude. Основной агент читает результаты, выставляет итоговую оценку, принимает или отклоняет предложения, содержательно правит только файл плана, не меняет код и не запускает проверки.'
+description: 'Полирует готовый план из `docs/plans/...` до оценки 95/100 или заданного порога. Каждая итерация запускает три изолированных параллельных агента, которые читают правила, архитектуру, `sources.references`, research и релевантный код. В strict дополнительно запускает кросс-CLI ревью. Основной агент оценивает ответы, принимает или отклоняет предложения, содержательно правит только файл плана, не меняет код и не запускает проверки.'
 ---
 
 # Скил: Полировка плана (eda-plan-polish)
 
-Берёшь готовый план из `docs/plans/` и доводишь его до состояния, когда его можно передать в `eda-execute`. Порог по умолчанию — `95`, лимит по умолчанию — `5` итераций. В `strict` каждая итерация после трёх моделей дополнительно отдаёт план соседнему CLI на кросс-ревью.
+Берёшь готовый план из `docs/plans/` и доводишь его до состояния, когда его можно передать в `eda-plan-execute`. Порог по умолчанию — `95`, лимит по умолчанию — `5` итераций. В `strict` каждая итерация после трёх моделей дополнительно отдаёт план соседнему CLI на кросс-ревью.
 
 Каждая итерация начинается с трёх независимых моделей с одним и тем же промптом. Основной агент не ставит предварительную оценку сам: он оценивает план только после ответов агентов, принимает или отклоняет предложения, правит план при оценке ниже порога и запускает новую итерацию.
 
@@ -19,9 +19,9 @@ description: 'Полирует готовый план из `docs/plans/...` д
 
 Прочитай `docs/settings.yaml`, если файл есть. Прямое указание пользователя в текущем сообщении важнее настроек.
 
-Дефолты: `defaults.strict: false`.
+Поддерживается только `version: 2`. При другой версии, отсутствующем разделе или неизвестном значении используй дефолт; настройки старого формата не применяй. Дефолт: `plan-polish.strict: false`.
 
-- `strict` включает кросс-CLI ревью плана после трёх агентов в каждой итерации; выключение: `normal`, `без strict`, `без строгого режима`.
+- `plan-polish.strict` включает кросс-CLI ревью плана после трёх агентов в каждой итерации; выключение: `normal`, `без strict`, `без строгого режима`.
 - Порог качества по умолчанию — `95`, можно переопределить словами `до 90`, `threshold 90`, `цель 90`.
 - Лимит по умолчанию — `5` итераций, можно переопределить словами `лимит 3`, `max iterations 3`, `не больше 3 кругов`.
 - Оценка `95–100` означает готовность к исполнению; `80–94` — нужно уточнить; `60–79` — рискованный план; ниже `60` — нужен серьёзный переплан или вопросы пользователю.
@@ -57,9 +57,9 @@ description: 'Полирует готовый план из `docs/plans/...` д
 - входа нет — спроси, показав несколько последних планов;
 - `docs/plans/` нет или она пуста — остановись: `blocked: нужен готовый план`.
 
-Прочитай только выбранный план целиком. По front matter извлеки `status`, `mode`, `plan_size`, `test_strategy`, `logging_strategy`, `sources.rules`, `sources.arch`, `sources.research`, `meta_reviewers`.
+Прочитай только выбранный план целиком. По front matter извлеки `status`, `mode`, `plan_size`, `test_strategy`, `logging_strategy`, `sources.rules`, `sources.arch`, `sources.references`, `sources.research`, `meta_reviewers`.
 
-Сам не читай `docs/rules.md`, `docs/arch.md`, `sources.research` и релевантный код на этом шаге. Передай эти пути агентам в общем промпте: проверка правил, архитектуры, research и кода — их работа. Основной агент может открыть внешний файл только позже и точечно, если нужно проверить конкретное предложение агента перед правкой плана. Не запускай проверки.
+Сам не читай `docs/rules.md`, `docs/arch.md`, `sources.references`, `sources.research` и релевантный код на этом шаге. Передай эти пути агентам в общем промпте: проверка правил, архитектуры, эталонов, research и кода — их работа. Если старый план не содержит `sources.references`, но существует `docs/references.md`, передай агентам индекс: они должны выбрать только применимые карточки. Если индекса нет, продолжай без references. Основной агент может открыть внешний файл только позже и точечно, если нужно проверить конкретное предложение агента перед правкой плана. Не запускай проверки.
 
 ### 2. Запустить три модели
 
@@ -79,9 +79,9 @@ description: 'Полирует готовый план из `docs/plans/...` д
 Общий промпт каждому агенту:
 
 ```text
-Прочитай $PLAN_FILE, docs/rules.md, docs/arch.md, sources.research при наличии и релевантные файлы, на которые ссылается план.
+Прочитай $PLAN_FILE, docs/rules.md, docs/arch.md, карточки из sources.references, sources.research при наличии и релевантные файлы, на которые ссылается план. Если sources.references отсутствует, прочитай docs/references.md и выбери только применимые карточки.
 
-Проверь весь план целиком: исполнимость, соответствие правилам и архитектуре, полноту целевого алгоритма, БД/API/внешние контракты, фазы, тестовые сценарии, логирование, неподтверждённые решения, скрытые риски и места, где eda-execute не сможет продолжить без нового исследования.
+Проверь весь план целиком: исполнимость, соответствие правилам, архитектуре и применимым эталонам, обоснованность отклонений от примеров, полноту целевого алгоритма, БД/API/внешние контракты, фазы, тестовые сценарии, логирование, неподтверждённые решения, скрытые риски и места, где eda-plan-execute не сможет продолжить без нового исследования.
 
 Не правь файлы и не запускай проверки, тесты, линтеры, билды, typecheck, dev-серверы, миграции, браузерные проверки или API-запросы. Можно только читать файлы.
 
@@ -100,13 +100,13 @@ score: <0–100>
 Если ты в Claude Code — запускай Codex CLI:
 
 ```bash
-codex exec "Прочитай $PLAN_FILE, docs/rules.md, docs/arch.md, sources.research при наличии и релевантные файлы, на которые ссылается план. Глубоко проверь весь план целиком: исполнимость, соответствие правилам и архитектуре, полноту целевого алгоритма, БД/API/внешние контракты, фазы, тесты, логирование, неподтверждённые решения, скрытые риски и места, где eda-execute не сможет продолжить без нового исследования. Не правь файлы и не запускай проверки, тесты, линтеры, билды, typecheck, dev-серверы, миграции, браузерные проверки или API-запросы. Можно только читать файлы. Ответь кратко: score: <0–100>; + Принять; ~ Уточнить; -- Убрать; ? Вопрос." > "${PLAN_FILE%.md}_strict_cli_review.md"
+codex exec "Прочитай $PLAN_FILE, docs/rules.md, docs/arch.md, карточки из sources.references, sources.research при наличии и релевантные файлы, на которые ссылается план. Если sources.references отсутствует, выбери применимые карточки через docs/references.md. Глубоко проверь весь план целиком: исполнимость, соответствие правилам, архитектуре и эталонам, обоснованность отклонений, полноту целевого алгоритма, БД/API/внешние контракты, фазы, тесты, логирование, неподтверждённые решения и скрытые риски. Не правь файлы и не запускай проверки, тесты, линтеры, билды, typecheck, dev-серверы, миграции, браузерные проверки или API-запросы. Можно только читать файлы. Ответь кратко: score: <0–100>; + Принять; ~ Уточнить; -- Убрать; ? Вопрос." > "${PLAN_FILE%.md}_strict_cli_review.md"
 ```
 
 Если ты в Codex — запускай Claude CLI:
 
 ```bash
-claude -p "Прочитай $PLAN_FILE, docs/rules.md, docs/arch.md, sources.research при наличии и релевантные файлы, на которые ссылается план. Глубоко проверь весь план целиком: исполнимость, соответствие правилам и архитектуре, полноту целевого алгоритма, БД/API/внешние контракты, фазы, тесты, логирование, неподтверждённые решения, скрытые риски и места, где eda-execute не сможет продолжить без нового исследования. Не правь файлы и не запускай проверки, тесты, линтеры, билды, typecheck, dev-серверы, миграции, браузерные проверки или API-запросы. Можно только читать файлы. Ответь кратко: score: <0–100>; + Принять; ~ Уточнить; -- Убрать; ? Вопрос." > "${PLAN_FILE%.md}_strict_cli_review.md"
+claude -p "Прочитай $PLAN_FILE, docs/rules.md, docs/arch.md, карточки из sources.references, sources.research при наличии и релевантные файлы, на которые ссылается план. Если sources.references отсутствует, выбери применимые карточки через docs/references.md. Глубоко проверь весь план целиком: исполнимость, соответствие правилам, архитектуре и эталонам, обоснованность отклонений, полноту целевого алгоритма, БД/API/внешние контракты, фазы, тесты, логирование, неподтверждённые решения и скрытые риски. Не правь файлы и не запускай проверки, тесты, линтеры, билды, typecheck, dev-серверы, миграции, браузерные проверки или API-запросы. Можно только читать файлы. Ответь кратко: score: <0–100>; + Принять; ~ Уточнить; -- Убрать; ? Вопрос." > "${PLAN_FILE%.md}_strict_cli_review.md"
 ```
 
 Если соседней CLI нет — `AskUserQuestion`: продолжить как `normal` без кросс-CLI или прервать полировку. Не заменяй кросс-CLI четвёртым локальным субагентом.
@@ -119,13 +119,14 @@ claude -p "Прочитай $PLAN_FILE, docs/rules.md, docs/arch.md, sources.res
 
 Оцени:
 - план следует `docs/rules.md` и `docs/arch.md`;
+- план следует применимым карточкам или содержит подтверждённую причину отклонения;
 - нет неподтверждённых существенных решений;
 - `Целевой алгоритм` описывает сквозное поведение;
 - `Контракты реализации` конкретны, особенно БД/API/внешние контракты;
 - фазы имеют цель, шаги, результат, сценарии тестирования и проверку;
 - `test_strategy` и `logging_strategy` отражены в фазах и разделах;
 - нет альтернатив вида «выбрать», «решить», «уточнить позже»;
-- план можно передать в `eda-execute` без нового исследования.
+- план можно передать в `eda-plan-execute` без нового исследования.
 
 Если `$SCORE >= threshold`, не вноси содержательные правки ради формы. Только добавь или обнови:
 
@@ -185,7 +186,7 @@ claude -p "Прочитай $PLAN_FILE, docs/rules.md, docs/arch.md, sources.res
 
 ## Финал
 
-Сообщи кратко: режим (`normal` или `strict`), путь к плану, путь к strict-review-файлу при наличии, итоговую оценку и порог, число итераций, какие модели и CLI запускались, главные принятые изменения, существенные отклонения, причину остановки и следующий шаг `eda-execute <PLAN_FILE>` при успехе. Если режим был обычный — добавь строку «кросс-CLI не запускался; для проверки соседним агентом — `eda-plan-polish strict <PLAN_FILE>`».
+Сообщи кратко: режим (`normal` или `strict`), путь к плану, путь к strict-review-файлу при наличии, итоговую оценку и порог, число итераций, какие модели и CLI запускались, главные принятые изменения, существенные отклонения, причину остановки и следующий шаг `eda-plan-execute <PLAN_FILE>` при успехе. Если режим был обычный — добавь строку «кросс-CLI не запускался; для проверки соседним агентом — `eda-plan-polish strict <PLAN_FILE>`».
 
 ## Чего НЕ делать
 
@@ -199,4 +200,4 @@ claude -p "Прочитай $PLAN_FILE, docs/rules.md, docs/arch.md, sources.res
 - Вставлять предложения агентов без анализа.
 - Добавлять существенные решения без подтверждения пользователя.
 - Превращать план в длинный отчёт о ревью.
-- Коммитить, пушить или запускать `eda-execute`.
+- Коммитить, пушить или запускать `eda-plan-execute`.

package/skills/eda-polish/SKILL.md

@@ -0,0 +1,80 @@
+---
+name: eda-polish
+description: 'Автономно доводит изменения до заданной оценки: в каждой итерации запускает полное `eda-review`, затем `eda-fix-by-review apply-optional` и повторяет цикл до порога или лимита. Все обязательные замечания исправляются, optional-пункты fix-агент применяет или отклоняет с причиной. По умолчанию цель — 95/100, лимит — 5 итераций. Каждый шаг идёт в изолированном субагенте. Не делает ревью или правки основным агентом и не коммитит.'
+---
+
+# Скил: Полировка изменений (eda-polish)
+
+Оркестрируй цикл `eda-review` → `eda-fix-by-review apply-optional` → повторное `eda-review`. Основной агент только управляет итерациями, читает сохранённые артефакты и сообщает результат.
+
+## Вход из сообщения пользователя
+
+Текст рядом с вызовом скилла в текущем сообщении — главный вход. Разбери target, путь к плану, PR/ветку/diff/файл, желаемый порог, лимит итераций и ограничения. Передавай в каждый `eda-review` разовые overrides проверок и моделей из запроса пользователя.
+
+Порог по умолчанию — `95`, лимит — `5` итераций. Пользователь может задать `до 90`, `threshold 90`, `цель 90` и `limit N`; допустимый score — 1–100, лимит — положительное целое число.
+
+## Правила
+
+1. Каждый вызов `eda-review` и `eda-fix-by-review apply-optional` запускай отдельным изолированным субагентом. Основной агент не делает их работу.
+2. Если нативные субагенты недоступны, остановись; не заменяй шаги CLI-процессами.
+3. Сравнивай порог только со `score` полного ревью со статусом `reviewed` или `reviewed-with-warnings`.
+4. Все пункты «править обязательно» исправь. Для каждого optional-пункта fix-агент сам решает, применять ли его, и сохраняет причины отклонения, чтобы следующие ревьюеры не повторяли решение без новых аргументов.
+5. Не коммить и не отправляй ревью.
+6. Останавливайся при повторном сбое обязательной проверки review, блокере fix-агента, отсутствии применимых замечаний ниже порога или если score не растёт два круга подряд.
+
+## Вопросы
+
+Если target неоднозначен, передай управление первому `eda-review`: он задаст один конкретный вопрос по правилам своей среды. Если шаг вернул `blocked`, основной агент повторяет только этот вопрос и после ответа продолжает тот же шаг.
+
+## Цикл
+
+Сформируй `$TARGET_CONTEXT`, `$THRESHOLD`, `$LIMIT` и внутренний журнал: номер итерации, review file, score, fix report и причина продолжения.
+
+Для каждой итерации:
+
+1. Запусти отдельного субагента с задачей:
+
+   ```text
+   Используй eda-review для <TARGET_CONTEXT>.
+   План: <PLAN_FILE | none>.
+   Overrides проверок и моделей: <из запроса | none>.
+   Верни путь к файлу, status, score и coverage.
+   ```
+
+2. Если review остановился из-за обязательного агента, останови весь цикл. `reviewed-with-warnings` из-за недоступных previous reviews разрешён и не блокирует цикл.
+3. Если `score >= threshold`, заверши успешно.
+4. Если score ниже порога, прочитай рекомендации. Если обязательных и optional-пунктов нет, остановись `blocked: нет применимых замечаний для повышения оценки`.
+5. Запусти отдельного субагента:
+
+   ```text
+   Используй eda-fix-by-review apply-optional для <REVIEW_FILE>.
+   Примени все обязательные замечания. По каждому optional сам реши, применять ли его; отклонённые решения сохрани с причиной.
+   Верни путь к fix-отчёту и результат проверок.
+   ```
+
+6. При успешных исправлениях начни следующую итерацию с новым полным `eda-review`.
+
+Если score не растёт два проверенных круга подряд, остановись раньше лимита. Если достигнут лимит, не скрывай незавершённую сходимость.
+
+## Финал
+
+Сообщи:
+
+- результат: достигнут порог, достигнут лимит, blocked, нет применимых замечаний или score не растёт;
+- итоговую оценку и порог;
+- количество итераций;
+- последний review file и fix report;
+- предупреждения coverage, если они были;
+- что изменения не закоммичены.
+
+При успехе предложи `eda-commit`. При неуспехе назови конкретный блокер или оставшиеся замечания.
+
+## Чего НЕ делать
+
+- Делать ревью или исправления основным агентом.
+- Запускать отдельный этап проверки готового ревью.
+- Сравнивать порог с неполным или заблокированным ревью.
+- Механически применять все optional-пункты.
+- Терять причины отклонённых optional-решений.
+- Переписывать `docs/settings.yaml`, правила или архитектуру ради score.
+- Коммитить, пушить или отправлять ревью.