@gian-tiaga/eda 0.8.3 → 0.8.5

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (5) hide show
  1. package/README.md +6 -5
  2. package/lib/install.js +1 -1
  3. package/package.json +1 -1
  4. package/skills/eda-plan-polish.md +45 -16
  5. package/skills/eda-review.md +7 -9
package/README.md CHANGED
@@ -39,10 +39,10 @@ eda update-all ~/Code
39
39
  | `eda-roadmap` | Создаёт roadmap-файл: список будущих задач с короткими, понятными и недвусмысленными описаниями без деталей реализации, файлов, библиотек, API и пошагового плана | `docs/roadmaps/` |
40
40
  | `eda-explore` | Исследует тему до конкретного результата: описание проблемы, релевантная архитектура, закрытые развилки, риски внутри решений и короткий выбранный вариант решения без плана. В дефолтном режиме ведёт исследование как диалог: факты собирает сам, а значимые развилки выносит тебе с рекомендацией. Если нужны пакеты или ПО — проверяет актуальные стабильные версии через context7 или web search | `docs/researches/` |
41
41
  | `eda-plan` | Пишет план реализации через стандартный Plan Mode хост-агента. `docs/rules.md` и `docs/arch.md` использует как обязательную рамку, а не справочный контекст: план должен строго следовать правилам и архитектуре. В плане явно фиксирует контракты БД/API, если задача их затрагивает. Затем три параллельных субагента/модели (слабая, средняя, мощная) проверяют план на реалистичность, пропущенные шаги и нарушения правил — главный планировщик сам решает, что применить | `docs/plans/` |
42
- | `eda-plan-polish` | Полирует уже готовый план до оценки 95/100 или заданного порога. Каждая итерация сразу запускает три изолированных параллельных агента на быстрой, кодовой и флагманской моделях. Все агенты получают один общий промпт и проверяют весь план целиком. Основной агент после их ответов выставляет оценку, анализирует предложения, правит только файл плана при оценке ниже порога и не запускает проверки проекта | `docs/plans/` |
42
+ | `eda-plan-polish` | Полирует уже готовый план до оценки 95/100 или заданного порога. Каждая итерация сразу запускает три изолированных параллельных агента на быстрой, кодовой и флагманской моделях. В strict дополнительно отдаёт план соседнему CLI (Claude ↔ Codex) на кросс-ревью. Основной агент после ответов выставляет оценку, анализирует предложения, содержательно правит только файл плана при оценке ниже порога и не запускает проверки проекта | `docs/plans/` |
43
43
  | `eda-execute` | Выполняет план. Спрашивает, где работать (текущая ветка, новая ветка, отдельный worktree). Перед правками читает план, `docs/rules.md` и `docs/arch.md`; правила и архитектура обязательны к исполнению. Если шаг плана им противоречит, останавливается и спрашивает. Тесты пишет внутри каждого шага. В конце — полный прогон тестов и линтеров | `docs/executions/` |
44
44
  | `eda-fix` | Делает обычные фиксы по короткому контексту, баг-репорту или файлу плана. Перед правками читает правила и архитектуру, добавляет нужные тесты, прогоняет проверки и сохраняет историю правок | `docs/fixes/` |
45
- | `eda-review` | Делает код-ревью с оценкой 0–100 и сверкой с планом работ, но в файл пишет только проблемы: ошибки, недоделки, неточности, нарушения правил/архитектуры, риски и недостающие тесты. Если запущен без аргументов, ревьюит незакоммиченный `git diff HEAD` без проверки плана. Не перечисляет выполненную работу. В обычном режиме после черновика передаёт файл в `eda-review-check`; с флагом `draft` сохраняет только первичное ревью без субагентов | `docs/reviews/` |
45
+ | `eda-review` | Делает код-ревью с оценкой 0–100 и сверкой с планом работ, но в файл пишет только проблемы: ошибки, недоделки, неточности, нарушения правил/архитектуры, риски и недостающие тесты. Сверяет с планом, только если он указан или однозначно найден; иначе ревьюит без сверки с планом и про план не спрашивает. Без аргументов ревьюит незакоммиченный `git diff HEAD`. Не перечисляет выполненную работу. В обычном режиме после черновика передаёт файл в `eda-review-check`; с флагом `draft` сохраняет только первичное ревью без субагентов | `docs/reviews/` |
46
46
  | `eda-review-check` | Проверяет готовое ревью из `docs/reviews/` специализированными субагентами: архитектуру, правила, выполнение плана при указанном plan-файле и при включённой настройке качество кода. В строгом режиме добавляет кросс-ревью между Claude и Codex. Сам применяет подтверждённые замечания к файлу ревью, но не правит код | `docs/reviews/` |
47
47
  | `eda-fix-by-review` | Применяет замечания из ревью. Обязательные — все сразу. Optional-пункты в обычном режиме спрашивает, а в `apply-optional` сам решает: применяет полезные, отклонённые обосновывает в отчёте. В конце ссылается на отчёт прямо в файле ревью | `docs/review-fixes/` |
48
48
  | `eda-manual-test` | Делает ручную smoke/acceptance-проверку после изменений: API прогоняет через `curl`/HTTP-запросы, фронт — через Playwright или доступную browser automation. Сам строит чеклист из входа, плана или diff, запускает очевидные dev/API-серверы и сохраняет QA-отчёт. Не правит код, не пишет автотесты и не коммитит | `docs/manual-tests/` |
@@ -66,6 +66,7 @@ eda update-all ~/Code
66
66
  |---|---|---|---|
67
67
  | `eda-explore` | `strict` | После сохранения отчёта отдаёт его соседнему CLI (Claude → Codex или наоборот) на критическое ревью конкретности, фактов, развилок, рисков и версий; затем доправляет отчёт по замечаниям | Серьёзные исследования, которые лягут в основу больших решений; когда хочется второго мнения от другой модели/среды |
68
68
  | `eda-plan` | `strict` | После обычного мета-ревью трёх субагентов/моделей отдаёт план соседнему CLI на дополнительный круг проверки; доправляет план | Большие или ответственные планы, особенно затрагивающие смежные системы |
69
+ | `eda-plan-polish` | `strict` | В каждой итерации после трёх полных ревью плана отдаёт план соседнему CLI (Claude → Codex или Codex → Claude) на кросс-ревью; затем учитывает его в итоговой оценке и правках | Максимальная полировка ответственных планов перед `eda-execute` |
69
70
  | `eda-review` | `draft` | Сохраняет первичное ревью со статусом `draft` и не запускает `eda-review-check`, субагентов и кросс-CLI | Когда ревью нужно как черновик для внешней оркестрации, например `eda-polish` |
70
71
  | `eda-review` | `strict` | После draft-ревью запускает `eda-review-check strict`: специализированные субагенты плюс соседний CLI; доправляет ревью по подтверждённым замечаниям | Ревью важных PR; когда нужен максимальный охват |
71
72
  | `eda-review-check` | `strict` | Проверяет уже готовое ревью специализированными субагентами и затем отдаёт его соседнему CLI на дополнительный круг проверки | Когда черновик ревью уже есть и нужно усилить только проверку ревью |
@@ -83,7 +84,7 @@ eda update-all ~/Code
83
84
  version: 1
84
85
 
85
86
  defaults:
86
- # Включает strict-режим по умолчанию для eda-explore, eda-plan, eda-review и eda-review-check.
87
+ # Включает strict-режим по умолчанию для eda-explore, eda-plan, eda-plan-polish, eda-review и eda-review-check.
87
88
  # true | false
88
89
  strict: false
89
90
  # Задаёт размер плана по умолчанию для eda-plan.
@@ -111,7 +112,7 @@ review:
111
112
  ```
112
113
 
113
114
  Что означают настройки:
114
- - `defaults.strict` — включает strict-режим по умолчанию для `eda-explore`, `eda-plan`, `eda-review` и `eda-review-check`.
115
+ - `defaults.strict` — включает strict-режим по умолчанию для `eda-explore`, `eda-plan`, `eda-plan-polish`, `eda-review` и `eda-review-check`.
115
116
  - `defaults.plan_size` — размер плана для `eda-plan`: `normal`, `short` или `ask_each_time`.
116
117
  - `defaults.decision_mode` — как `eda-explore` ведёт исследовательские развилки, а `eda-plan` принимает существенные решения: `autonomous` выбирает сам, `recommend_and_ask` рекомендует и спрашивает по значимым развилкам, `ask_each_time` спрашивает по каждой развилке, которая влияет на ход работы или итоговый выбор.
117
118
  - `defaults.test_strategy` — стратегия тестов для `eda-plan`: `after_each_phase`, `tdd_each_phase`, `end_of_plan` или `ask_each_time`.
@@ -147,7 +148,7 @@ roadmap ────────> explore ─────> plan ───> plan-
147
148
  fix ─────────────> manual-test
148
149
 
149
150
  polish = review draft ─> review-check ─> fix-by-review apply-optional ─> повтор до нужной оценки
150
- plan-polish = 3 полных ревью плана разными моделями ─> оценка ─> правка плана ─> повтор до 95/100
151
+ plan-polish = 3 полных ревью плана разными моделями ─> strict: кросс-CLI ─> оценка ─> правка плана ─> повтор до 95/100
151
152
 
152
153
  automate может запускаться от review, fix-by-review или fix.
153
154
  ```
package/lib/install.js CHANGED
@@ -378,7 +378,7 @@ function formatSettings(settings) {
378
378
  return `version: 1
379
379
 
380
380
  defaults:
381
- # Включает strict-режим по умолчанию для eda-explore, eda-plan, eda-review и eda-review-check.
381
+ # Включает strict-режим по умолчанию для eda-explore, eda-plan, eda-plan-polish, eda-review и eda-review-check.
382
382
  # true | false
383
383
  strict: ${settings.strict ? 'true' : 'false'}
384
384
  # Задаёт размер плана по умолчанию для eda-plan.
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@gian-tiaga/eda",
3
- "version": "0.8.3",
3
+ "version": "0.8.5",
4
4
  "description": "Набор скилов eda-* для Claude Code и Codex CLI: start, roadmap, explore, plan, plan-polish, execute, fix, review, review-check, fix-by-review, manual-test, polish, send-review, commit, worktree, merge-worktree, docs, automate",
5
5
  "type": "module",
6
6
  "bin": {
package/skills/eda-plan-polish.md CHANGED
@@ -1,11 +1,11 @@
1
1
  ---
2
2
  name: eda-plan-polish
3
- description: 'Полирует готовый план из `docs/plans/...` до оценки 95/100 или заданного порога. Каждая итерация начинается с трёх изолированных параллельных агентов на быстрой, кодовой и флагманской моделях. Все агенты получают один общий промпт и проверяют весь план целиком. Основной агент читает их результаты, выставляет итоговую оценку, принимает или отклоняет предложения, правит только файл плана, не меняет код и не запускает проверки.'
3
+ description: 'Полирует готовый план из `docs/plans/...` до оценки 95/100 или заданного порога. Каждая итерация начинается с трёх изолированных параллельных агентов на быстрой, кодовой и флагманской моделях. В strict дополнительно запускает кросс-CLI ревью плана: Claude отдаёт план Codex, Codex отдаёт план Claude. Основной агент читает результаты, выставляет итоговую оценку, принимает или отклоняет предложения, содержательно правит только файл плана, не меняет код и не запускает проверки.'
4
4
  ---
5
5
 
6
6
  # Скил: Полировка плана (eda-plan-polish)
7
7
 
8
- Берёшь готовый план из `docs/plans/` и доводишь его до состояния, когда его можно передать в `eda-execute`. Порог по умолчанию — `95`, лимит по умолчанию — `5` итераций.
8
+ Берёшь готовый план из `docs/plans/` и доводишь его до состояния, когда его можно передать в `eda-execute`. Порог по умолчанию — `95`, лимит по умолчанию — `5` итераций. В `strict` каждая итерация после трёх моделей дополнительно отдаёт план соседнему CLI на кросс-ревью.
9
9
 
10
10
  Каждая итерация начинается с трёх независимых моделей с одним и тем же промптом. Основной агент не ставит предварительную оценку сам: он оценивает план только после ответов агентов, принимает или отклоняет предложения, правит план при оценке ниже порога и запускает новую итерацию.
11
11
 
@@ -17,6 +17,11 @@ description: 'Полирует готовый план из `docs/plans/...` д
17
17
 
18
18
  ## Настройки
19
19
 
20
+ Прочитай `docs/settings.yaml`, если файл есть. Прямое указание пользователя в текущем сообщении важнее настроек.
21
+
22
+ Дефолты: `defaults.strict: false`.
23
+
24
+ - `strict` включает кросс-CLI ревью плана после трёх агентов в каждой итерации; выключение: `normal`, `без strict`, `без строгого режима`.
20
25
  - Порог качества по умолчанию — `95`, можно переопределить словами `до 90`, `threshold 90`, `цель 90`.
21
26
  - Лимит по умолчанию — `5` итераций, можно переопределить словами `лимит 3`, `max iterations 3`, `не больше 3 кругов`.
22
27
  - Оценка `95–100` означает готовность к исполнению; `80–94` — нужно уточнить; `60–79` — рискованный план; ниже `60` — нужен серьёзный переплан или вопросы пользователю.
@@ -24,14 +29,15 @@ description: 'Полирует готовый план из `docs/plans/...` д
24
29
  ## Жёсткие правила
25
30
 
26
31
  1. Работай только с готовым файлом плана в `docs/plans/`.
27
- 2. Не правишь код. Менять можно только выбранный файл плана; миграции, конфиги, зависимости и lock-файлы не трогай.
32
+ 2. Не правишь код. Содержательно менять можно только выбранный файл плана; в `strict` можно создать или обновить только вспомогательный файл `${PLAN_FILE%.md}_strict_cli_review.md`. Миграции, конфиги, зависимости и lock-файлы не трогай.
28
33
  3. Не запускай проверки и исполняющие команды: `test`, `lint`, `build`, `typecheck`, `check`, `dev`, `start`, миграции, seed, smoke-тесты, браузерные проверки и API-запросы. При полировке плана можно читать файлы и искать по коду, но нельзя проверять проект выполнением команд.
29
34
  4. Не ставь предварительную оценку без агентов. Сразу запускай три изолированных агента параллельно, одним batch, до чтения любого результата.
30
- 5. Итоговую оценку 0–100 выставляй только после чтения результатов всех трёх агентов.
35
+ 5. Итоговую оценку 0–100 выставляй только после чтения результатов всех трёх агентов, а в `strict` — ещё и результата соседнего CLI.
31
36
  6. У всех трёх агентов один общий промпт: каждый проверяет весь план целиком, а не отдельную роль.
32
37
  7. Не добавляй неподтверждённые существенные решения: БД/API-контракты, зависимости, архитектурные развилки, auth/permissions/payments/secrets/персональные данные и миграции.
33
38
  8. Сохраняй структуру `eda-plan`: `Целевой алгоритм`, `Контракты реализации`, `Фазы выполнения`, `Тесты`, `Логирование`, если эти разделы есть или должны быть по формату.
34
- 9. Если качество не растёт два круга подряд, достигнут лимит или нужен ответ пользователя, остановись и честно назови причину.
39
+ 9. Не пропускай кросс-CLI молча в `strict`: если соседней CLI нет, спроси через `AskUserQuestion`, завершить как normal или прервать.
40
+ 10. Если качество не растёт два круга подряд, достигнут лимит или нужен ответ пользователя, остановись и честно назови причину.
35
41
 
36
42
  ## Интерактивные вопросы
37
43
 
@@ -51,9 +57,9 @@ description: 'Полирует готовый план из `docs/plans/...` д
51
57
  - входа нет — спроси, показав несколько последних планов;
52
58
  - `docs/plans/` нет или она пуста — остановись: `blocked: нужен готовый план`.
53
59
 
54
- Прочитай план целиком. По front matter извлеки `status`, `mode`, `plan_size`, `test_strategy`, `logging_strategy`, `sources.rules`, `sources.arch`, `sources.research`, `meta_reviewers`.
60
+ Прочитай только выбранный план целиком. По front matter извлеки `status`, `mode`, `plan_size`, `test_strategy`, `logging_strategy`, `sources.rules`, `sources.arch`, `sources.research`, `meta_reviewers`.
55
61
 
56
- Прочитай `docs/rules.md`, `docs/arch.md` и `sources.research`, если они есть. Если план ссылается на конкретные файлы, модули, API, миграции или тесты, открой только нужные файлы для понимания. Не запускай проверки.
62
+ Сам не читай `docs/rules.md`, `docs/arch.md`, `sources.research` и релевантный код на этом шаге. Передай эти пути агентам в общем промпте: проверка правил, архитектуры, research и кода — их работа. Основной агент может открыть внешний файл только позже и точечно, если нужно проверить конкретное предложение агента перед правкой плана. Не запускай проверки.
57
63
 
58
64
  ### 2. Запустить три модели
59
65
 
@@ -87,9 +93,29 @@ score: <0–100>
87
93
  ? Вопрос: <что нельзя безопасно решить без пользователя>
88
94
  ```
89
95
 
90
- ### 3. Оценить результаты
96
+ ### 3. Strict: кросс-CLI ревью плана
97
+
98
+ Обычный режим пропускает этот этап. В `strict` после ответов трёх агентов, но до итоговой оценки, отдай план соседнему CLI через `Bash`.
99
+
100
+ Если ты в Claude Code — запускай Codex CLI:
101
+
102
+ ```bash
103
+ 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"
104
+ ```
105
+
106
+ Если ты в Codex — запускай Claude CLI:
107
+
108
+ ```bash
109
+ 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"
110
+ ```
111
+
112
+ Если соседней CLI нет — `AskUserQuestion`: продолжить как `normal` без кросс-CLI или прервать полировку. Не заменяй кросс-CLI четвёртым локальным субагентом.
113
+
114
+ Прочитай ответ соседнего CLI и дальше оцени его вместе с тремя агентами. Файл `${PLAN_FILE%.md}_strict_cli_review.md` можно создать и обновлять в `strict`; это единственный вспомогательный файл, кроме самого плана, который разрешён в strict-режиме.
115
+
116
+ ### 4. Оценить результаты
91
117
 
92
- Когда все три агента ответили, прочитай их замечания и оценки. После этого основной агент выставляет итоговый `$SCORE` по шкале 0–100.
118
+ Когда все три агента ответили, а в `strict` ещё и соседний CLI, прочитай их замечания и оценки. После этого основной агент выставляет итоговый `$SCORE` по шкале 0–100.
93
119
 
94
120
  Оцени:
95
121
  - план следует `docs/rules.md` и `docs/arch.md`;
@@ -108,13 +134,14 @@ score: <0–100>
108
134
  - **Итоговая оценка:** <score>/100
109
135
  - **Порог:** <threshold>/100
110
136
  - **Итерации:** <N>
111
- - **Модели:** <фактически запущенные модели>
137
+ - **Модели:** <фактически запущенные модели и соседний CLI в strict>
138
+ - **Режим:** <normal | strict>
112
139
  - **Решение:** план готов к исполнению.
113
140
  ```
114
141
 
115
142
  На этом остановись.
116
143
 
117
- ### 4. Отредактировать план
144
+ ### 5. Отредактировать план
118
145
 
119
146
  Если `$SCORE < threshold`, сгруппируй замечания:
120
147
  - **Принять** — подтверждено планом, правилами, архитектурой, research или кодом и улучшает исполнимость без новой развилки.
@@ -130,7 +157,7 @@ score: <0–100>
130
157
 
131
158
  ### Итерация <N>
132
159
  - **Оценка после проверки:** <score>/100
133
- - **Модели:** <фактически запущенные модели>
160
+ - **Модели:** <фактически запущенные модели и соседний CLI в strict>
134
161
  - **+ Добавлено:** <3–7 главных изменений>
135
162
  - **~ Уточнено:** <1–5 важных уточнений>
136
163
  - **− Убрано:** <если удалено>
@@ -138,9 +165,9 @@ score: <0–100>
138
165
  - **Вопросы пользователя:** <если были, кратко решение>
139
166
  ```
140
167
 
141
- В `meta_reviewers`, если поле есть, добавь фактически запущенные модели. Если `status: draft` и открытых вопросов не осталось, поменяй на `status: meta-reviewed`; `meta-reviewed` и `reviewed` не понижай.
168
+ В `meta_reviewers`, если поле есть, добавь фактически запущенные модели и имя соседнего CLI в `strict`. Если `status: draft` и открытых вопросов не осталось, поменяй на `status: meta-reviewed`; `meta-reviewed` и `reviewed` не понижай.
142
169
 
143
- ### 5. Продолжить или остановиться
170
+ ### 6. Продолжить или остановиться
144
171
 
145
172
  После правок обнови:
146
173
 
@@ -149,7 +176,8 @@ score: <0–100>
149
176
  - **Итоговая оценка:** <score>/100
150
177
  - **Порог:** <threshold>/100
151
178
  - **Итерации:** <N>
152
- - **Модели:** <фактически запущенные модели>
179
+ - **Модели:** <фактически запущенные модели и соседний CLI в strict>
180
+ - **Режим:** <normal | strict>
153
181
  - **Решение:** <готов к исполнению | нужен следующий круг | blocked | достигнут лимит>
154
182
  ```
155
183
 
@@ -157,7 +185,7 @@ score: <0–100>
157
185
 
158
186
  ## Финал
159
187
 
160
- Сообщи кратко: путь к плану, итоговую оценку и порог, число итераций, какие модели запускались, главные принятые изменения, существенные отклонения, причину остановки и следующий шаг `eda-execute <PLAN_FILE>` при успехе.
188
+ Сообщи кратко: режим (`normal` или `strict`), путь к плану, путь к strict-review-файлу при наличии, итоговую оценку и порог, число итераций, какие модели и CLI запускались, главные принятые изменения, существенные отклонения, причину остановки и следующий шаг `eda-execute <PLAN_FILE>` при успехе. Если режим был обычный — добавь строку «кросс-CLI не запускался; для проверки соседним агентом — `eda-plan-polish strict <PLAN_FILE>`».
161
189
 
162
190
  ## Чего НЕ делать
163
191
 
@@ -167,6 +195,7 @@ score: <0–100>
167
195
  - Ставить предварительную оценку без трёх агентных проверок.
168
196
  - Запускать агентов последовательно или давать им разные промпты.
169
197
  - Запускать все три прохода на одной модели.
198
+ - Пропускать кросс-CLI молча в `strict`.
170
199
  - Вставлять предложения агентов без анализа.
171
200
  - Добавлять существенные решения без подтверждения пользователя.
172
201
  - Превращать план в длинный отчёт о ревью.
package/skills/eda-review.md CHANGED
@@ -1,11 +1,11 @@
1
1
  ---
2
2
  name: eda-review
3
- description: 'Ревью изменений (незакоммиченный diff, ветка, PR, папка/файл) с оценкой и сверкой с планом работ из `docs/plans/`, когда план указан или однозначно найден. Если запущен без аргументов, ревьюит незакоммиченный `git diff HEAD` без проверки плана. Перед ревью читает `docs/rules.md` и `docs/arch.md`. Формирует ревью только по проблемам, сохраняет в `docs/reviews/` и в обычном режиме передаёт файл в `eda-review-check` для мета-проверки субагентами. Флаг `draft` сохраняет только черновое ревью без субагентов. Кросс-CLI включается флагом `strict` или настройкой `defaults.strict: true`. Не правит код — это `eda-fix-by-review`.'
3
+ description: 'Ревью изменений (незакоммиченный diff, ветка, PR, папка/файл) с оценкой и сверкой с планом работ из `docs/plans/`, когда план указан или однозначно найден. Если план не указан и не найден однозначно, ревьюит без сверки с планом и не спрашивает про него. Без аргументов ревьюит незакоммиченный `git diff HEAD`. Перед ревью читает `docs/rules.md` и `docs/arch.md`. Формирует ревью только по проблемам, сохраняет в `docs/reviews/` и в обычном режиме передаёт файл в `eda-review-check` для мета-проверки субагентами. Флаг `draft` сохраняет только черновое ревью без субагентов. Кросс-CLI включается флагом `strict` или настройкой `defaults.strict: true`. Не правит код — это `eda-fix-by-review`.'
4
4
  ---
5
5
 
6
6
  # Скил: Ревьюер (eda-review)
7
7
 
8
- Делаешь ревью изменений: сверяешь diff с планом работ, правилами и архитектурой, пишешь только найденные проблемы, ставишь оценку, сохраняешь файл. Если скилл вызван без аргументов, ревьюишь незакоммиченный `git diff HEAD` и не проверяешь план. Каждый пункт начинаешь с короткого человеческого объяснения риска, без технической перегрузки; файлы, строки, детали кода и способ исправления даёшь ниже в блоке «Технические детали». Не перечисляешь, что выполнено хорошо или без замечаний. Затем в обычном режиме запускаешь `eda-review-check`, который проверяет ревью параллельными специализированными агентами; в `strict` он добавляет соседний CLI. В `draft` останавливаешься сразу после сохранения черновика.
8
+ Делаешь ревью изменений: сверяешь diff с планом работ, правилами и архитектурой, пишешь только найденные проблемы, ставишь оценку, сохраняешь файл. Если скилл вызван без аргументов, ревьюишь незакоммиченный `git diff HEAD`. План сверяешь, только если он указан или однозначно найден; иначе ревьюишь без сверки с планом и про план не спрашиваешь. Каждый пункт начинаешь с короткого человеческого объяснения риска, без технической перегрузки; файлы, строки, детали кода и способ исправления даёшь ниже в блоке «Технические детали». Не перечисляешь, что выполнено хорошо или без замечаний. Затем в обычном режиме запускаешь `eda-review-check`, который проверяет ревью параллельными специализированными агентами; в `strict` он добавляет соседний CLI. В `draft` останавливаешься сразу после сохранения черновика.
9
9
 
10
10
  ## Режим запуска
11
11
 
@@ -33,7 +33,7 @@ description: 'Ревью изменений (незакоммиченный diff
33
33
  1. **Перед ревью** прочитай `docs/rules.md` и `docs/arch.md`, если есть, и строго следуй им. Замечания должны опираться на правила и архитектуру проекта, а не на твои общие представления.
34
34
  2. **Интерактивные вопросы** — по разделу ниже. Если контекст понятен, не спрашивай.
35
35
  3. **Не правишь код.** Только ревью. Запись только в `docs/reviews/`.
36
- 4. **План работ обязателен для явно заданного ревью выполненных изменений.** Если пользователь указал файл, папку, PR, ветку, тип diff или другую цель ревью, а путь к плану не указан и его нельзя однозначно определить из diff/ссылок, спроси путь к файлу плана и остановись до ответа. Исключение: запуск без аргументов — ревью незакоммиченного `git diff HEAD` без проверки плана. Не подставляй «последний план» наугад.
36
+ 4. **Сверка с планом — только если план есть.** Используй план, если пользователь указал путь к нему или его можно однозначно определить из diff/ссылок/существующего review-файла. Если плана нет или вариантов несколько — не спрашивай: делай ревью без сверки с планом и отметь это в ревью. Не подставляй «последний план» наугад.
37
37
  5. **Мета-ревью специализированными агентами — обязательно в normal/strict**, но выполняется отдельным скиллом `eda-review-check`. В `draft` мета-ревью не запускается.
38
38
  6. **Главный ревьюер — ты.** Мета-ревьюеры дают подсказки; решение — добавить, убрать или оставить пункт — за тобой. Не вставляй чужие пункты механически.
39
39
  7. **Простой язык** в формулировках замечаний и рекомендаций. Технические термины — только если без них нельзя, и сразу с пояснением.
@@ -48,7 +48,7 @@ description: 'Ревью изменений (незакоммиченный diff
48
48
 
49
49
  ## Интерактивные вопросы
50
50
 
51
- Если цель ревью понятна из запроса или текущего состояния репозитория, **не задавай вопрос о цели ревью**. Это не отменяет обязательный вопрос о `$PLAN_FILE` для явно заданной цели, если план не указан и не найден однозначно. Для запуска без аргументов вопрос о плане не задавай: проверка плана пропускается. Примеры понятного контекста:
51
+ Если цель ревью понятна из запроса или текущего состояния репозитория, **не задавай вопрос о цели ревью**. Вопрос о плане не задавай никогда: если план не указан и не найден однозначно, ревьюй без сверки с планом. Примеры понятного контекста:
52
52
  - пользователь указал файл, папку, PR, ветку или тип diff;
53
53
  - в запросе есть «ревью незакоммиченных изменений», «ревью текущей ветки», «проверь PR #N»;
54
54
  - в рабочем дереве есть незакоммиченные изменения и пользователь просит «сделай ревью» без других целей.
@@ -70,12 +70,10 @@ description: 'Ревью изменений (незакоммиченный diff
70
70
 
71
71
  Получи diff/исходник в `$TARGET`. Прочитай `docs/rules.md`, `docs/arch.md` и строго следуй им.
72
72
 
73
- Определи `$PLAN_FILE` и `$PLAN_MODE`:
74
- - если пользователь вызвал скилл без аргументов, а целью выбран незакоммиченный `git diff HEAD` — установи `$PLAN_FILE=none`, `$PLAN_MODE=skipped_no_args` и не спрашивай план;
73
+ Определи `$PLAN_FILE`, не задавая вопросов о плане:
75
74
  - если пользователь указал путь к плану — используй его;
76
75
  - если в diff, описании задачи или уже существующем review-файле есть однозначная ссылка на `docs/plans/...` — используй её;
77
- - если цель ревью явно задана пользователем и есть несколько возможных планов или нет ни одного — спроси: «Какой файл плана использовать для проверки выполнения работ?» и остановись до ответа;
78
- - если пользователь явно просит ревью без сверки с планом — продолжай, но в ревью укажи, что проверка выполнения плана пропущена по явному указанию пользователя.
76
+ - иначе (план не указан, не найден или вариантов несколько) — установи `$PLAN_FILE=none` и ревьюй без сверки с планом. Не подставляй «последний план» наугад.
79
77
 
80
78
  Если в diff видно отсылки к исследованию из `docs/researches/` — прочитай его тоже.
81
79
 
@@ -111,7 +109,7 @@ meta_reviewers: []
111
109
 
112
110
  ## Проблемы сверки с планом
113
111
 
114
- <Если `$PLAN_FILE=none`: «Проверка плана пропущена: запуск без аргументов использует незакоммиченный `git diff HEAD` без сверки с планом.» Если проблем по плану нет: «Явных проблем по плану не найдено.»>
112
+ <Если `$PLAN_FILE=none`: «Проверка плана пропущена: план не указан и не найден.» Если проблем по плану нет: «Явных проблем по плану не найдено.»>
115
113
 
116
114
  ### 1. <недоделка, отклонение или непроверенный пункт>
117
115