@gian-tiaga/eda 0.3.4 → 0.3.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.
- package/package.json +1 -1
- package/skills/eda-review.md +16 -6
package/package.json
CHANGED
package/skills/eda-review.md
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: eda-review
|
|
3
|
-
description: 'Ревью изменений (незакоммиченный diff, ветка, PR, папка/файл) с обязательной сверкой с планом работ из `docs/plans/`. Если план не указан и его нельзя однозначно определить, спрашивает путь к плану до ревью. Перед ревью читает `docs/rules.md` и `docs/arch.md`. Формирует
|
|
3
|
+
description: 'Ревью изменений (незакоммиченный diff, ветка, PR, папка/файл) с обязательной сверкой с планом работ из `docs/plans/`. Если план не указан и его нельзя однозначно определить, спрашивает путь к плану до ревью. Перед ревью читает `docs/rules.md` и `docs/arch.md`. Формирует ревью по Markdown-шаблону из этапа 2: с блоками «контекст», «риск», «где», «детали», «как исправить» и пустой строкой между каждым блоком. Сохраняет в docs/reviews/. Всегда запускает параллельные специализированные проверки: выполнение плана, следование архитектуре, следование правилам. Кросс-CLI (Claude ↔ Codex) включается только флагом `strict`. Не правит код — это `eda-fix-by-review`.'
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# Скил: Ревьюер (eda-review)
|
|
@@ -27,7 +27,7 @@ description: 'Ревью изменений (незакоммиченный diff
|
|
|
27
27
|
5. **Мета-ревью тремя специализированными агентами — обязательно**, без флага: проверка выполнения плана, проверка архитектуры, проверка правил. Кросс-CLI — только в `strict`.
|
|
28
28
|
6. **Главный ревьюер — ты.** Мета-ревьюеры дают подсказки; решение — добавить, убрать или оставить пункт — за тобой. Не вставляй чужие пункты механически.
|
|
29
29
|
7. **Простой язык** в формулировках замечаний и рекомендаций. Технические термины — только если без них нельзя, и сразу с пояснением.
|
|
30
|
-
8.
|
|
30
|
+
8. **Формат файла ревью — Markdown-блоки по шаблону из этапа 2.** Сверку с планом, каждое замечание и рекомендации пиши отдельными блоками с подзаголовками. Между заголовком блока, статусом/типом, контекстом, риском, локациями, деталями и исправлением всегда оставляй одну пустую строку.
|
|
31
31
|
9. **В каждом замечании сначала объясняй контекст без привязки к коду**, чтобы читатель понял проблему и риск. Только после этого давай конкретные файлы/строки. Исправление оформляй так же: сначала общий подход, затем конкретные шаги по коду.
|
|
32
32
|
|
|
33
33
|
## Интерактивные вопросы
|
|
@@ -77,7 +77,7 @@ description: 'Ревью изменений (незакоммиченный diff
|
|
|
77
77
|
|
|
78
78
|
Поставь оценку 0–100 (100 — идеально, 0 — переделывать).
|
|
79
79
|
|
|
80
|
-
Сохрани в `docs/reviews/{YYYY-MM-DD}_{HH-MM}_{slug}.md
|
|
80
|
+
Сохрани в `docs/reviews/{YYYY-MM-DD}_{HH-MM}_{slug}.md`. Соблюдай этот формат буквально: смысловые части идут отдельными блоками, между соседними блоками всегда есть одна пустая строка.
|
|
81
81
|
|
|
82
82
|
```markdown
|
|
83
83
|
---
|
|
@@ -93,10 +93,13 @@ meta_reviewers: []
|
|
|
93
93
|
# Ревью: <заголовок>
|
|
94
94
|
|
|
95
95
|
## Оценка
|
|
96
|
+
|
|
96
97
|
**<N>/100.** <одно-два предложения почему>
|
|
97
98
|
|
|
98
99
|
## Сверка с планом
|
|
100
|
+
|
|
99
101
|
### 1. <пункт или этап плана>
|
|
102
|
+
|
|
100
103
|
Статус: <выполнено | частично | не выполнено | не проверялось>
|
|
101
104
|
|
|
102
105
|
Контекст:
|
|
@@ -106,10 +109,13 @@ meta_reviewers: []
|
|
|
106
109
|
<что фактически сделано или не сделано>
|
|
107
110
|
|
|
108
111
|
Конкретика:
|
|
112
|
+
|
|
109
113
|
- `<path/file.ts:42>` — <короткое подтверждение>
|
|
110
114
|
|
|
111
115
|
## Замечания
|
|
116
|
+
|
|
112
117
|
### 1. <короткий заголовок проблемы>
|
|
118
|
+
|
|
113
119
|
Тип: `bug`
|
|
114
120
|
|
|
115
121
|
Рекомендация: `править обязательно`
|
|
@@ -121,6 +127,7 @@ meta_reviewers: []
|
|
|
121
127
|
<что сломается или почему это важно>
|
|
122
128
|
|
|
123
129
|
Где:
|
|
130
|
+
|
|
124
131
|
- `<path/file.py:42>` — <что смотреть>
|
|
125
132
|
|
|
126
133
|
Детали:
|
|
@@ -130,15 +137,18 @@ meta_reviewers: []
|
|
|
130
137
|
<общий подход без привязки к конкретному файлу>
|
|
131
138
|
|
|
132
139
|
Конкретные шаги:
|
|
140
|
+
|
|
133
141
|
- <что поменять в коде>
|
|
134
142
|
- <какой тест добавить или обновить>
|
|
135
143
|
|
|
136
144
|
## Рекомендации
|
|
145
|
+
|
|
137
146
|
- **Править обязательно:** <номера>
|
|
138
147
|
- **На усмотрение автора:** <номера>
|
|
139
148
|
- **Не править:** <номера или —>
|
|
140
149
|
|
|
141
150
|
## Изменения после мета-ревью
|
|
151
|
+
|
|
142
152
|
<заполняется на этапах 3 и 4>
|
|
143
153
|
```
|
|
144
154
|
|
|
@@ -185,13 +195,13 @@ meta_reviewers: []
|
|
|
185
195
|
В `strict`: отдай файл соседнему агенту через `Bash`. Если ты в Claude Code — запускай Codex CLI:
|
|
186
196
|
|
|
187
197
|
```bash
|
|
188
|
-
codex exec "Прочитай $REVIEW_FILE, $TARGET, $PLAN_FILE, docs/rules.md, docs/arch.md, связанный research при наличии. Затем глубоко проверь ревью по плану, правилам, архитектуре и релевантному коду: открой файлы, модули, API и тесты, которые следуют из diff и замечаний, и оцени фактическую корректность. Мета-ревью на русском: что добавить, убрать, переформулировать; какие баги, риски для смежного кода и недостающие тесты пропущены. Формат: '+', '−', '~', '?'
|
|
198
|
+
codex exec "Прочитай $REVIEW_FILE, $TARGET, $PLAN_FILE, docs/rules.md, docs/arch.md, связанный research при наличии. Затем глубоко проверь ревью по плану, правилам, архитектуре и релевантному коду: открой файлы, модули, API и тесты, которые следуют из diff и замечаний, и оцени фактическую корректность. Мета-ревью на русском: что добавить, убрать, переформулировать; какие баги, риски для смежного кода и недостающие тесты пропущены. Формат: '+', '−', '~', '?'." > "${REVIEW_FILE%.md}_cli_meta.md"
|
|
189
199
|
```
|
|
190
200
|
|
|
191
201
|
Если ты в Codex — запускай Claude CLI с теми же требованиями глубины:
|
|
192
202
|
|
|
193
203
|
```bash
|
|
194
|
-
claude -p "Прочитай $REVIEW_FILE, $TARGET, $PLAN_FILE, docs/rules.md, docs/arch.md, связанный research при наличии. Затем глубоко проверь ревью по плану, правилам, архитектуре и релевантному коду: открой файлы, модули, API и тесты, которые следуют из diff и замечаний, и оцени фактическую корректность. Мета-ревью на русском: что добавить, убрать, переформулировать; какие баги, риски для смежного кода и недостающие тесты пропущены. Формат: '+', '−', '~', '?'
|
|
204
|
+
claude -p "Прочитай $REVIEW_FILE, $TARGET, $PLAN_FILE, docs/rules.md, docs/arch.md, связанный research при наличии. Затем глубоко проверь ревью по плану, правилам, архитектуре и релевантному коду: открой файлы, модули, API и тесты, которые следуют из diff и замечаний, и оцени фактическую корректность. Мета-ревью на русском: что добавить, убрать, переформулировать; какие баги, риски для смежного кода и недостающие тесты пропущены. Формат: '+', '−', '~', '?'." > "${REVIEW_FILE%.md}_cli_meta.md"
|
|
195
205
|
```
|
|
196
206
|
|
|
197
207
|
Соседней CLI нет — `AskUserQuestion`: завершить как есть или прервать.
|
|
@@ -218,4 +228,4 @@ claude -p "Прочитай $REVIEW_FILE, $TARGET, $PLAN_FILE, docs/rules.md, do
|
|
|
218
228
|
- Вставлять чужие предложения механически — каждое решение принимай ты.
|
|
219
229
|
- Задавать вопросы без блокировки и продолжать работу до ответа.
|
|
220
230
|
- Сохранять ревью куда-либо кроме `docs/reviews/`.
|
|
221
|
-
- Оформлять замечания
|
|
231
|
+
- Оформлять замечания без контекста перед конкретикой.
|