@gian-tiaga/eda 0.4.4 → 0.4.6

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.
package/README.md CHANGED
@@ -27,11 +27,11 @@ eda init
27
27
 
28
28
  | Скил | Что делает | Куда складывает |
29
29
  |---|---|---|
30
- | `eda-explore` | Исследует тему до конкретного результата: описание проблемы, релевантная архитектура, закрытые развилки и риски, короткий выбранный вариант решения без плана. Если нужны пакеты или ПО — проверяет актуальные стабильные версии через context7 или web search | `docs/researches/` |
30
+ | `eda-explore` | Исследует тему до конкретного результата: описание проблемы, релевантная архитектура, закрытые развилки, риски внутри решений и короткий выбранный вариант решения без плана. Если нужны пакеты или ПО — проверяет актуальные стабильные версии через context7 или web search | `docs/researches/` |
31
31
  | `eda-plan` | Пишет план реализации через стандартный Plan Mode хост-агента. Подтягивает контекст из `docs/rules.md`, `docs/arch.md`, при желании — релевантное исследование. Затем три параллельных модели (слабая, средняя, мощная) проверяют план на реалистичность, пропущенные шаги и нарушения правил — главный планировщик сам решает, что применить | `docs/plans/` |
32
32
  | `eda-execute` | Выполняет план. Спрашивает, где работать (текущая ветка, новая ветка, отдельный worktree). Тесты пишет внутри каждого шага. В конце — полный прогон тестов и линтеров | `docs/executions/` |
33
33
  | `eda-fix` | Делает обычные фиксы по короткому контексту, баг-репорту или файлу плана. Перед правками читает правила и архитектуру, добавляет нужные тесты, прогоняет проверки и сохраняет историю правок | `docs/fixes/` |
34
- | `eda-review` | Делает код-ревью с оценкой 0–100 и сверкой с планом работ. Замечания пишет блоками: сначала контекст и риск, потом конкретные файлы/строки и шаги исправления. Затем параллельные специализированные агенты проверяют выполнение плана, архитектуру, правила и при включённой настройке качество кода. В строгом режиме — ещё и кросс-ревью между Claude и Codex | `docs/reviews/` |
34
+ | `eda-review` | Делает код-ревью с оценкой 0–100 и сверкой с планом работ, но в файл пишет только проблемы: ошибки, недоделки, неточности, нарушения правил/архитектуры, риски и недостающие тесты. Не перечисляет выполненную работу. Затем параллельные специализированные агенты проверяют выполнение плана, архитектуру, правила и при включённой настройке качество кода. В строгом режиме — ещё и кросс-ревью между Claude и Codex | `docs/reviews/` |
35
35
  | `eda-fix-by-review` | Применяет замечания из ревью. Обязательные — сразу. Спорные — спрашивает. В конце ссылается на отчёт прямо в файле ревью | `docs/review-fixes/` |
36
36
  | `eda-send-review` | Отправляет ревью на GitHub PR через `gh` — как комментарий, review, approve или request-changes. Сам определяет PR по текущей ветке | (комментарий в PR) |
37
37
  | `eda-commit` | Делает один аккуратный коммит в стиле проекта. В конце спрашивает: push, merge в main + удалить ветку, или ничего | (git) |
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@gian-tiaga/eda",
3
- "version": "0.4.4",
3
+ "version": "0.4.6",
4
4
  "description": "Набор скилов eda-* для Claude Code и Codex CLI: explore, plan, execute, fix, review, fix-by-review, send-review, commit, docs, automate",
5
5
  "type": "module",
6
6
  "bin": {
@@ -1,11 +1,11 @@
1
1
  ---
2
2
  name: eda-explore
3
- description: 'Исследует тему или задачу до конкретного результата: проясняет проблему, архитектуру, развилки, риски и короткий выбранный вариант решения без плана реализации. Работает read-only, задаёт блокирующие вопросы, когда без ответа нельзя выбрать решение, а существенные решения подтверждает по `defaults.decision_mode` из `docs/settings.yaml`. Если исследование касается пакетов, библиотек, CLI, сервисов или другого ПО, проверяет актуальные стабильные версии через context7, если он доступен, или через web search по официальным источникам. Итог сохраняет в docs/researches/. Кросс-ревью соседним агентом (Claude ↔ Codex) включается флагом `strict` или настройкой `defaults.strict: true` в `docs/settings.yaml`.'
3
+ description: 'Исследует тему или задачу до конкретного результата: проясняет проблему, архитектуру, развилки, связанные риски и короткий выбранный вариант решения без плана реализации. Работает read-only, задаёт блокирующие вопросы, когда без ответа нельзя выбрать решение, а существенные решения подтверждает по `defaults.decision_mode` из `docs/settings.yaml`. Если исследование касается пакетов, библиотек, CLI, сервисов или другого ПО, проверяет актуальные стабильные версии через context7, если он доступен, или через web search по официальным источникам. Итог сохраняет в docs/researches/. Кросс-ревью соседним агентом (Claude ↔ Codex) включается флагом `strict` или настройкой `defaults.strict: true` в `docs/settings.yaml`.'
4
4
  ---
5
5
 
6
6
  # Скил: Исследователь решений (eda-explore)
7
7
 
8
- Проводишь исследование так, чтобы на выходе была не подборка рассуждений, а конкретная рамка для следующего шага: что за проблема, как устроена релевантная архитектура, какие решения приняты, какие риски закрыты и какой вариант решения выбран.
8
+ Проводишь исследование так, чтобы на выходе была не подборка рассуждений, а конкретная рамка для следующего шага: что за проблема, как устроена релевантная архитектура, какие решения приняты, какие риски учтены в этих решениях и какой вариант решения выбран.
9
9
 
10
10
  ## Режимы вызова
11
11
 
@@ -55,12 +55,12 @@ description: 'Исследует тему или задачу до конкре
55
55
  1. **Никаких правок кода.** Запись только в `docs/researches/`.
56
56
  2. **Итог — конкретика.** Нельзя заканчивать абстрактными описаниями, списком возможностей или «можно выбрать один из вариантов».
57
57
  3. **Развилки закрывай по `decision_mode`.** Обычные локальные развилки можно выбрать по коду, правилам, архитектуре, источникам и рискам. Существенные решения в `recommend_and_ask` и `ask_each_time` подтверждай у пользователя до финализации. Если выбор зависит только от пользователя — задай блокирующий вопрос в любом режиме.
58
- 4. **Риски закрывай.** Для каждого риска зафиксируй решение: mitigation, acceptance или out of scope. Нельзя оставлять риск просто перечисленным.
58
+ 4. **Риски вплетай в исследование.** Не создавай отдельный обязательный раздел или этап рисков. Если риск влияет на выбор, источник, ограничение или будущую проверку, фиксируй его рядом с соответствующим решением: что может пойти не так и как это учитывается — снимаем, принимаем или выносим за рамки.
59
59
  5. **Версии проверяй.** Если речь о пакетах, библиотеках, CLI, сервисах или другом ПО, а версия не задана контекстом, проверь последнюю стабильную версию через context7, если он доступен, или через web search по официальным источникам.
60
60
  6. **Прочитай `docs/rules.md` и `docs/arch.md`**, если есть, и используй их как рамку.
61
61
  7. **Каждое фактическое утверждение — со ссылкой** (`file.ext:42`, URL, коммит). Без ссылки — помечай как «гипотеза» и не делай её основанием выбранного решения.
62
62
  8. **Простой язык.** Термины — только когда без них нельзя, сразу с пояснением.
63
- 9. **Таблицы > сплошной текст** для фактов, развилок, рисков и решений.
63
+ 9. **Таблицы > сплошной текст** для фактов, развилок, сравнений, версий и источников.
64
64
 
65
65
  ## Интерактивные вопросы
66
66
 
@@ -100,6 +100,7 @@ description: 'Исследует тему или задачу до конкре
100
100
  - перечисли варианты;
101
101
  - выбери один вариант или подготовь рекомендацию;
102
102
  - запиши источник и причину выбора;
103
+ - если у варианта есть важный риск, опиши его здесь же и сразу зафиксируй, как он влияет на выбранное решение;
103
104
  - если развилка существенная и `decision_mode: recommend_and_ask` — задай `AskUserQuestion`: 2–3 варианта, первым рекомендованный, короткая причина рекомендации, вариант «свой вариант», фраза «Я продолжу только после ответа.»;
104
105
  - если развилка значимая и `decision_mode: ask_each_time` — задай `AskUserQuestion` даже при сильной рекомендации;
105
106
  - если `decision_mode: autonomous` и выбор можно обосновать кодом, правилами, архитектурой, источниками и рисками — выбери сам и запиши почему;
@@ -107,21 +108,13 @@ description: 'Исследует тему или задачу до конкре
107
108
 
108
109
  Не прячь существенный выбор внутри текста. Например, если нужен пакет для базы данных, сначала сравни 2–3 подходящих варианта, затем в `recommend_and_ask` спроси подтверждение выбора пакета/БД до записи финального решения.
109
110
 
110
- ### 4. Закрыть риски
111
+ Если по ходу исследования находится высокий риск без понятного решения, это блокер: задай `AskUserQuestion` или заверши со статусом `blocked`. Не выноси риски в отдельную секцию ради формы; они должны естественно лежать в `Решении`, причинах выбора, ограничениях, проверках версий или итоговом выводе.
111
112
 
112
- Для каждого риска зафиксируй:
113
- - что может пойти не так;
114
- - почему это важно;
115
- - выбранное решение: как снимаем, принимаем или выносим за рамки;
116
- - что это значит для будущего плана.
117
-
118
- Высокий риск без решения — блокер. Задай вопрос или заверши со статусом `blocked`.
119
-
120
- ### 5. Собрать отчёт
113
+ ### 4. Собрать отчёт
121
114
 
122
115
  Сохрани отчёт в `docs/researches/{YYYY-MM-DD}_{HH-MM}_{slug}.md`.
123
116
 
124
- Формат отчёта должен быть компактным и универсальным: три основных раздела, без обязательного раздувания под каждый тип исследования. Таблицы, ASCII-диаграммы и дополнительные подпункты добавляй внутри этих разделов только когда они действительно помогают ясности.
117
+ Формат отчёта должен быть компактным и универсальным: четыре основных раздела, без обязательного раздувания под каждый тип исследования. Таблицы, ASCII-диаграммы и дополнительные подпункты добавляй внутри этих разделов только когда они действительно помогают ясности.
125
118
 
126
119
  ```markdown
127
120
  ---
@@ -145,7 +138,8 @@ sources:
145
138
  Описание выбранного решения. Внутри этого раздела, если нужно, добавь:
146
139
  - описание релевантной архитектуры;
147
140
  - ASCII-диаграммы;
148
- - таблицы фактов, развилок, рисков, версий пакетов/ПО и источников;
141
+ - таблицы фактов, развилок, версий пакетов/ПО и источников;
142
+ - риски рядом с теми решениями, ограничениями или проверками, на которые они влияют;
149
143
  - объяснение, почему выбран этот вариант, а не альтернативы.
150
144
 
151
145
  ## Ответы на вопросы
@@ -163,13 +157,14 @@ Quality gate перед сохранением:
163
157
  - в `Ответы на вопросы` записаны все вопросы, заданные пользователю в процессе исследования, и ответы на них, чтобы `eda-plan` не задавал их повторно;
164
158
  - в `Итог` есть короткая конкретика, что делать дальше на уровне подхода, без пошагового плана;
165
159
  - нет незакрытых развилок;
166
- - нет рисков без решения;
160
+ - нет рисков, которые перечислены отдельно от решения и никак не влияют на выбор, ограничения или будущие проверки;
161
+ - нет высокого риска без понятного решения;
167
162
  - версии пакетов/ПО проверены или объяснено, почему это не требуется;
168
163
  - ключевые утверждения имеют источники.
169
164
 
170
- ### 6. Кросс-ревью — только в `strict`
165
+ ### 5. Кросс-ревью — только в `strict`
171
166
 
172
- Обычный режим — пропусти этапы 6–7, иди на финал. `reviewer: none`, `status: draft`.
167
+ Обычный режим — пропусти этапы 5–6, иди на финал. `reviewer: none`, `status: draft`.
173
168
 
174
169
  В `strict`: отдай файл соседнему агенту через `Bash` и сформулируй задание как глубокую проверку конкретности, фактов, развилок и рисков.
175
170
 
@@ -187,11 +182,11 @@ claude -p "Прочитай $RESEARCH_FILE, docs/rules.md, docs/arch.md при
187
182
 
188
183
  CLI нет — `AskUserQuestion`: продолжать без ревью или прервать.
189
184
 
190
- ### 7. Допилить по ревью — только в `strict`
185
+ ### 6. Допилить по ревью — только в `strict`
191
186
 
192
187
  Бесспорные замечания → правь. Спорные → `AskUserQuestion`. Отклонённые → оставь с причиной. В конец отчёта добавь раздел `## Реакция на ревью` (Принято / Отклонено / Спорное). `status: reviewed`. Покажи дифф.
193
188
 
194
- ### 8. Финал
189
+ ### 7. Финал
195
190
 
196
191
  Короткое сообщение: режим, путь к отчёту, путь к ревью (если `strict`), 3–5 конкретных выводов, выбранный вариант решения одной строкой. Не предлагай «исследовать дальше», если нет блокера.
197
192
 
@@ -302,21 +302,17 @@ sources:
302
302
 
303
303
  ## Документация и эксплуатация
304
304
  Что обновить в env/docs/runbook и что важно для релиза.
305
-
306
- ## Риски
307
- Риски и конкретное снижение каждого риска.
308
-
309
- ## Порядок выполнения
310
- Краткий линейный список действий для исполнителя.
311
305
  ```
312
306
 
313
307
  Правила структуры:
314
308
  - В `Контекст` попадают исследовательские наблюдения и ограничения.
315
309
  - В `Принятые решения` попадают только выбранные решения, без альтернатив.
310
+ - Риски не выносятся в отдельный обязательный раздел. Закрытые риски из research, контекста и мета-ревью должны быть встроены туда, где они исполняются: в `Принятые решения`, фазы, сценарии тестирования, проверки, логирование или эксплуатационные шаги.
316
311
  - Существенные решения в `Принятых решениях` имеют источник подтверждения или явное обоснование автономного выбора.
317
312
  - В `Целевой алгоритм` попадает сквозная картина процесса.
318
313
  - В `Фазы выполнения` попадают только исполнимые шаги.
319
314
  - Каждая фаза обязана иметь `Цель`, `Что сделать`, `Результат`, `Сценарии тестирования`, `Проверка`.
315
+ - Отдельный раздел порядка выполнения не нужен: линейный порядок задаётся номерами фаз и шагами внутри фаз.
320
316
  - `test_strategy` обязан менять структуру фаз: TDD — тесты в начале фаз, after_each_phase — тесты после каждой фазы, end_of_plan — отдельная финальная фаза тестов.
321
317
  - `logging_strategy` обязан быть отражён в целевом алгоритме, фазах и отдельном разделе `Логирование`.
322
318
  - Для `plan_size: short` фаз должно быть не больше 2, а ожидаемый объём должен быть явно указан в `Принятых решениях`.
@@ -332,7 +328,7 @@ sources:
332
328
  - мета-уровень не смешан с шагами исполнения;
333
329
  - стратегия тестов зафиксирована в разделе `Тесты` и последовательно применена в фазах, сценариях тестирования и проверках;
334
330
  - стратегия логирования зафиксирована и последовательно применена в алгоритме, фазах и разделе `Логирование`;
335
- - тесты связаны с ключевыми сценариями и рисками;
331
+ - тесты связаны с ключевыми сценариями и закрытыми рисками из research/контекста, если такие риски есть;
336
332
  - если `plan_size: short`, фаз не больше 2, ожидаемый объём укладывается в ориентиры short или исключение явно объяснено;
337
333
  - все пакеты/ПО, которые нужно установить, имеют конкретные версии или явное объяснение, почему версия берётся из существующего lock/config-файла;
338
334
  - нет неподтверждённых существенных решений при `decision_mode: recommend_and_ask` или `decision_mode: ask_each_time`;
@@ -362,7 +358,7 @@ sources:
362
358
 
363
359
  Базовый формат ответа для всех: `+ N | − N | ~ N: причина`, без воды. Для мощной модели добавь явную инструкцию: «Если план ссылается на модуль, API, тесты или сценарий, открой соответствующие файлы и проверь фактическую структуру проекта.»
364
360
 
365
- Когда все три ответили — прочитай ответы, **сам реши**, что применять (не вставляй чужие пункты механически). Принятые замечания сначала интегрируй в основные разделы плана: решения, алгоритм, фазы, тесты, риски. Не оставляй важные требования только в changelog.
361
+ Когда все три ответили — прочитай ответы, **сам реши**, что применять (не вставляй чужие пункты механически). Принятые замечания сначала интегрируй в основные разделы плана: решения, алгоритм, фазы, тесты, логирование, документацию и эксплуатацию. Не оставляй важные требования только в changelog.
366
362
 
367
363
  После интеграции снова прогони quality gate из этапа 5. Затем допиши в конец короткий раздел:
368
364
 
@@ -1,11 +1,11 @@
1
1
  ---
2
2
  name: eda-review
3
- description: 'Ревью изменений (незакоммиченный diff, ветка, PR, папка/файл) с обязательной сверкой с планом работ из `docs/plans/`. Если план не указан и его нельзя однозначно определить, спрашивает путь к плану до ревью. Перед ревью читает `docs/rules.md` и `docs/arch.md`. Формирует ревью по Markdown-шаблону из этапа 2: с блоками «контекст», «риск», «где», «детали», «как исправить» и пустой строкой между каждым блоком. Сохраняет в docs/reviews/. Всегда запускает параллельные специализированные проверки: выполнение плана, следование архитектуре, следование правилам; проверка качества кода включается настройкой `review.include_code_quality: true` в `docs/settings.yaml`. Кросс-CLI (Claude ↔ Codex) включается флагом `strict` или настройкой `defaults.strict: true`. Не правит код — это `eda-fix-by-review`.'
3
+ description: 'Ревью изменений (незакоммиченный diff, ветка, PR, папка/файл) с обязательной сверкой с планом работ из `docs/plans/`. Если план не указан и его нельзя однозначно определить, спрашивает путь к плану до ревью. Перед ревью читает `docs/rules.md` и `docs/arch.md`. Формирует ревью только по проблемам: ошибки, недоделки, неточности, нарушения правил/архитектуры, риски и недостающие тесты. Не перечисляет выполненную работу. Сохраняет в docs/reviews/. Всегда запускает параллельные специализированные проверки: выполнение плана, следование архитектуре, следование правилам; проверка качества кода включается настройкой `review.include_code_quality: true` в `docs/settings.yaml`. Кросс-CLI (Claude ↔ Codex) включается флагом `strict` или настройкой `defaults.strict: true`. Не правит код — это `eda-fix-by-review`.'
4
4
  ---
5
5
 
6
6
  # Скил: Ревьюер (eda-review)
7
7
 
8
- Делаешь ревью изменений: сверяешь diff с планом работ, правилами и архитектурой, пишешь замечания, ставишь оценку, сохраняешь файл. Затем проверяешь ревью параллельными специализированными агентами; в `strict` — ещё и через соседний CLI. После каждой проверки **сам корректируешь** своё ревью: что-то добавляешь, что-то убираешь, что-то переформулируешь.
8
+ Делаешь ревью изменений: сверяешь diff с планом работ, правилами и архитектурой, пишешь только найденные проблемы, ставишь оценку, сохраняешь файл. Не перечисляешь, что выполнено хорошо или без замечаний. Затем проверяешь ревью параллельными специализированными агентами; в `strict` — ещё и через соседний CLI. После каждой проверки **сам корректируешь** своё ревью: что-то добавляешь, что-то убираешь, что-то переформулируешь.
9
9
 
10
10
  ## Режимы вызова
11
11
 
@@ -39,8 +39,9 @@ description: 'Ревью изменений (незакоммиченный diff
39
39
  5. **Мета-ревью специализированными агентами — обязательно**, без флага: проверка выполнения плана, проверка архитектуры, проверка правил. Если включена проверка качества кода — добавь `quality-check` тем же batch. Кросс-CLI — только в `strict`.
40
40
  6. **Главный ревьюер — ты.** Мета-ревьюеры дают подсказки; решение — добавить, убрать или оставить пункт — за тобой. Не вставляй чужие пункты механически.
41
41
  7. **Простой язык** в формулировках замечаний и рекомендаций. Технические термины — только если без них нельзя, и сразу с пояснением.
42
- 8. **Формат файла ревью Markdown-блоки по шаблону из этапа 2.** Сверку с планом, каждое замечание и рекомендации пиши отдельными блоками с подзаголовками. Между заголовком блока, статусом/типом, контекстом, риском, локациями, деталями и исправлением всегда оставляй одну пустую строку.
43
- 9. **В каждом замечании сначала объясняй контекст без привязки к коду**, чтобы читатель понял проблему и риск. Только после этого давай конкретные файлы/строки. Исправление оформляй так же: сначала общий подход, затем конкретные шаги по коду.
42
+ 8. **Ревью содержит только проблемы.** Не пиши отчёт о выполненной работе, список успешно сделанных пунктов, похвалу или нейтральное перечисление изменений. В файл попадают только ошибки, недоделки, неточности, спорные решения, нарушения правил/архитектуры, риски и недостающие проверки. Если проблем нет, напиши коротко: «Явных проблем не найдено», без пересказа diff.
43
+ 9. **Формат файла ревью Markdown-блоки по шаблону из этапа 2.** Проблемы сверки с планом, каждое замечание и рекомендации пиши отдельными блоками с подзаголовками. Между заголовком блока, статусом/типом, контекстом, риском, локациями, деталями и исправлением всегда оставляй одну пустую строку.
44
+ 10. **В каждом замечании сначала объясняй контекст без привязки к коду**, чтобы читатель понял проблему и риск. Только после этого давай конкретные файлы/строки. Исправление оформляй так же: сначала общий подход, затем конкретные шаги по коду.
44
45
 
45
46
  ## Интерактивные вопросы
46
47
 
@@ -75,11 +76,11 @@ description: 'Ревью изменений (незакоммиченный diff
75
76
  Если в diff видно отсылки к исследованию из `docs/researches/` — прочитай его тоже.
76
77
 
77
78
  ### 2. Сделать первичное ревью и сохранить
78
- Пройдись по изменениям. Сначала проверь, выполнен ли `$PLAN_FILE`: что сделано, что сделано частично, что не сделано, какие отклонения создают риск. Затем проверь нарушения правил, архитектуры, баги, тестовые пробелы, безопасность, производительность и документацию. Если `review.include_code_quality: true` или пользователь явно попросил качество кода, отдельно проверь качество реализации: читаемость, сложность, дублирование, имена, границы ответственности, связность модулей, поддерживаемость и соответствие стилю уже существующего кода.
79
+ Пройдись по изменениям. Сначала проверь, выполнен ли `$PLAN_FILE`, но в ревью выноси только проблемы: что сделано частично, что не сделано, какие отклонения создают риск, какие проверки отсутствуют. Не перечисляй выполненные пункты плана. Затем проверь нарушения правил, архитектуры, баги, тестовые пробелы, безопасность, производительность и документацию. Если `review.include_code_quality: true` или пользователь явно попросил качество кода, отдельно проверь качество реализации: читаемость, сложность, дублирование, имена, границы ответственности, связность модулей, поддерживаемость и соответствие стилю уже существующего кода.
79
80
 
80
81
  Каждое замечание должно быть самостоятельным блоком:
81
82
  - заголовок с номером и коротким смыслом;
82
- - тип и рекомендация: `править обязательно`, `на усмотрение автора` или `не править`;
83
+ - тип и рекомендация: `править обязательно` или `на усмотрение автора`;
83
84
  - контекст: объяснение проблемы без привязки к конкретному коду;
84
85
  - риск: почему это важно для поведения, сопровождения, денег, безопасности или эксплуатации;
85
86
  - где: конкретные файлы/строки;
@@ -108,24 +109,31 @@ meta_reviewers: []
108
109
 
109
110
  **<N>/100.** <одно-два предложения почему>
110
111
 
111
- ## Сверка с планом
112
+ ## Проблемы сверки с планом
112
113
 
113
- ### 1. <пункт или этап плана>
114
+ <Если проблем по плану нет: «Явных проблем по плану не найдено.»>
114
115
 
115
- Статус: <выполнено | частично | не выполнено | не проверялось>
116
+ ### 1. <недоделка, отклонение или непроверенный пункт>
117
+
118
+ Статус: <частично | не выполнено | не проверялось | отклонение>
116
119
 
117
120
  Контекст:
118
121
  <что требовал план простыми словами, без привязки к коду>
119
122
 
120
- Итог:
121
- <что фактически сделано или не сделано>
123
+ Проблема:
124
+ <что не сделано, сделано неточно или не подтверждено>
122
125
 
123
- Конкретика:
126
+ Риск:
127
+ <почему это важно>
124
128
 
125
- - `<path/file.ts:42>` — <короткое подтверждение>
129
+ Где:
130
+
131
+ - `<path/file.ts:42>` — <что смотреть>
126
132
 
127
133
  ## Замечания
128
134
 
135
+ <Если замечаний нет: «Явных проблем в коде не найдено.»>
136
+
129
137
  ### 1. <короткий заголовок проблемы>
130
138
 
131
139
  Тип: `bug`
@@ -157,7 +165,6 @@ meta_reviewers: []
157
165
 
158
166
  - **Править обязательно:** <номера>
159
167
  - **На усмотрение автора:** <номера>
160
- - **Не править:** <номера или —>
161
168
 
162
169
  ## Изменения после мета-ревью
163
170
 
@@ -183,14 +190,14 @@ meta_reviewers: []
183
190
  Каждому дай путь к файлу ревью, `$TARGET`, `$PLAN_FILE`, `docs/rules.md`, `docs/arch.md`, `docs/settings.yaml`, связанный research при наличии. Каждый агент должен читать не только ревью, но и фактический diff/код, достаточный для проверки своей роли.
184
191
 
185
192
  Формат ответа агентов:
186
- - `+` что добавить в ревью;
193
+ - `+` какую проблему добавить в ревью;
187
194
  - `−` что убрать как неверное или недоказанное;
188
195
  - `~` что переформулировать;
189
196
  - `?` что невозможно проверить и почему.
190
197
 
191
198
  Для `plan-check` добавь явную инструкцию: «Если `$PLAN_FILE` не передан или файл не найден, верни только `? план не проверен: нужен путь к файлу`; не пытайся угадывать план.»
192
199
 
193
- Когда все агенты ответили: прочитай ответы, **сам реши**, что применять. Внеси правки в файл ревью. В разделе `## Изменения после мета-ревью` запиши:
200
+ Когда все агенты ответили: прочитай ответы, **сам реши**, что применять. Добавляй только подтверждённые проблемы; предложения вида «добавить, что пункт выполнен» отклоняй как неформат ревью. Внеси правки в файл ревью. В разделе `## Изменения после мета-ревью` запиши:
194
201
 
195
202
  ```markdown
196
203
  ### После plan-check / architecture-check / rules-check / quality-check
@@ -208,13 +215,13 @@ meta_reviewers: []
208
215
  В `strict`: отдай файл соседнему агенту через `Bash`. Если ты в Claude Code — запускай Codex CLI:
209
216
 
210
217
  ```bash
211
- codex exec "Прочитай $REVIEW_FILE, $TARGET, $PLAN_FILE, docs/rules.md, docs/arch.md, связанный research при наличии. Затем глубоко проверь ревью по плану, правилам, архитектуре и релевантному коду: открой файлы, модули, API и тесты, которые следуют из diff и замечаний, и оцени фактическую корректность. Мета-ревью на русском: что добавить, убрать, переформулировать; какие баги, риски для смежного кода и недостающие тесты пропущены. Формат: '+', '−', '~', '?'." > "${REVIEW_FILE%.md}_cli_meta.md"
218
+ codex exec "Прочитай $REVIEW_FILE, $TARGET, $PLAN_FILE, docs/rules.md, docs/arch.md, связанный research при наличии. Затем глубоко проверь ревью по плану, правилам, архитектуре и релевантному коду: открой файлы, модули, API и тесты, которые следуют из diff и замечаний, и оцени фактическую корректность. Мета-ревью на русском: какие проблемы добавить, убрать, переформулировать; какие баги, риски для смежного кода и недостающие тесты пропущены. Не предлагай перечислять выполненную работу. Формат: '+', '−', '~', '?'." > "${REVIEW_FILE%.md}_cli_meta.md"
212
219
  ```
213
220
 
214
221
  Если ты в Codex — запускай Claude CLI с теми же требованиями глубины:
215
222
 
216
223
  ```bash
217
- claude -p "Прочитай $REVIEW_FILE, $TARGET, $PLAN_FILE, docs/rules.md, docs/arch.md, связанный research при наличии. Затем глубоко проверь ревью по плану, правилам, архитектуре и релевантному коду: открой файлы, модули, API и тесты, которые следуют из diff и замечаний, и оцени фактическую корректность. Мета-ревью на русском: что добавить, убрать, переформулировать; какие баги, риски для смежного кода и недостающие тесты пропущены. Формат: '+', '−', '~', '?'." > "${REVIEW_FILE%.md}_cli_meta.md"
224
+ claude -p "Прочитай $REVIEW_FILE, $TARGET, $PLAN_FILE, docs/rules.md, docs/arch.md, связанный research при наличии. Затем глубоко проверь ревью по плану, правилам, архитектуре и релевантному коду: открой файлы, модули, API и тесты, которые следуют из diff и замечаний, и оцени фактическую корректность. Мета-ревью на русском: какие проблемы добавить, убрать, переформулировать; какие баги, риски для смежного кода и недостающие тесты пропущены. Не предлагай перечислять выполненную работу. Формат: '+', '−', '~', '?'." > "${REVIEW_FILE%.md}_cli_meta.md"
218
225
  ```
219
226
 
220
227
  Соседней CLI нет — `AskUserQuestion`: завершить как есть или прервать.
@@ -231,7 +238,7 @@ claude -p "Прочитай $REVIEW_FILE, $TARGET, $PLAN_FILE, docs/rules.md, do
231
238
  Добавь в `meta_reviewers` имя CLI. `status: meta-reviewed` → `final`.
232
239
 
233
240
  ### 5. Финал
234
- Короткое сообщение: режим (`normal` или `strict`), путь к файлу ревью, итоговая оценка, сколько пунктов «править обязательно» / «на усмотрение» / «не править». Если режим был обычный — добавь строку «кросс-CLI не запускался; для проверки соседним агентом — `eda-review strict`». Что предлагаешь дальше — `eda-fix-by-review` для применения или `eda-commit` если оценка приемлема.
241
+ Короткое сообщение: режим (`normal` или `strict`), путь к файлу ревью, итоговая оценка, сколько пунктов «править обязательно» / «на усмотрение». Не пересказывай выполненную работу. Если режим был обычный — добавь строку «кросс-CLI не запускался; для проверки соседним агентом — `eda-review strict`». Что предлагаешь дальше — `eda-fix-by-review` для применения или `eda-commit` если оценка приемлема.
235
242
 
236
243
  ## Чего НЕ делать
237
244
  - Править код. Это `eda-fix-by-review`.
@@ -242,3 +249,4 @@ claude -p "Прочитай $REVIEW_FILE, $TARGET, $PLAN_FILE, docs/rules.md, do
242
249
  - Задавать вопросы без блокировки и продолжать работу до ответа.
243
250
  - Сохранять ревью куда-либо кроме `docs/reviews/`.
244
251
  - Оформлять замечания без контекста перед конкретикой.
252
+ - Перечислять выполненные пункты плана или давать общий отчёт о проделанной работе.