@gian-tiaga/eda 0.2.5 → 0.2.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/docs/skills/eda-review.md +14 -3
- package/package.json +1 -1
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: eda-review
|
|
3
|
-
description: 'Ревью изменений (незакоммиченный diff, ветка, PR, папка/файл). Формирует пронумерованные замечания, оценку 0–100, рекомендации «править / на усмотрение / не править». Сохраняет в docs/reviews/. Всегда проверяет своё же ревью тремя параллельными моделями (Sonnet, Haiku, Opus); кросс-CLI (Claude ↔ Codex) включается только флагом `strict`. После каждой проверки сам корректирует ревью. Не правит код — это `eda-fix-by-review`.
|
|
3
|
+
description: 'Ревью изменений (незакоммиченный diff, ветка, PR, папка/файл). Если контекст ревью понятен из запроса или репозитория, не задаёт уточняющий вопрос и сразу выбирает цель. Формирует пронумерованные замечания, оценку 0–100, рекомендации «править / на усмотрение / не править». Сохраняет в docs/reviews/. Всегда проверяет своё же ревью тремя параллельными моделями (Sonnet, Haiku, Opus); кросс-CLI (Claude ↔ Codex) включается только флагом `strict`. После каждой проверки сам корректирует ревью. Не правит код — это `eda-fix-by-review`. Вопросы — только когда без ответа нельзя безопасно продолжать.'
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# Скил: Ревьюер (eda-review)
|
|
@@ -17,7 +17,7 @@ description: 'Ревью изменений (незакоммиченный diff
|
|
|
17
17
|
## Главные правила
|
|
18
18
|
|
|
19
19
|
1. **Перед ревью** прочитай `docs/rules.md` и `docs/arch.md`, если есть. Замечания должны опираться на правила проекта, а не на твои общие представления.
|
|
20
|
-
2. **Интерактивные вопросы** — по разделу ниже.
|
|
20
|
+
2. **Интерактивные вопросы** — по разделу ниже. Если контекст понятен, не спрашивай.
|
|
21
21
|
3. **Не правишь код.** Только ревью. Запись только в `docs/reviews/`.
|
|
22
22
|
4. **Мета-ревью тремя моделями — обязательно**, без флага. Кросс-CLI — только в `strict`.
|
|
23
23
|
5. **Главный ревьюер — ты.** Мета-ревьюеры дают подсказки; решение — добавить, убрать или оставить пункт — за тобой. Не вставляй чужие пункты механически.
|
|
@@ -26,6 +26,11 @@ description: 'Ревью изменений (незакоммиченный diff
|
|
|
26
26
|
|
|
27
27
|
## Интерактивные вопросы
|
|
28
28
|
|
|
29
|
+
Если цель ревью понятна из запроса или текущего состояния репозитория, **не задавай вопрос**. Примеры понятного контекста:
|
|
30
|
+
- пользователь указал файл, папку, PR, ветку или тип diff;
|
|
31
|
+
- в запросе есть «ревью незакоммиченных изменений», «ревью текущей ветки», «проверь PR #N»;
|
|
32
|
+
- в рабочем дереве есть незакоммиченные изменения и пользователь просит «сделай ревью» без других целей.
|
|
33
|
+
|
|
29
34
|
Когда инструкция говорит `AskUserQuestion`, это означает блокирующий интерактивный вопрос.
|
|
30
35
|
- Claude Code: используй `AskUserQuestion`.
|
|
31
36
|
- Codex interactive: если доступен `request_user_input`, используй его. Если tool недоступен, задай один короткий вопрос в чат, дай варианты 1–3, напиши «Ответь номером или своим вариантом. Я продолжу только после ответа.» и остановись.
|
|
@@ -35,7 +40,13 @@ description: 'Ревью изменений (незакоммиченный diff
|
|
|
35
40
|
## Этапы
|
|
36
41
|
|
|
37
42
|
### 1. Выбрать вход и прочитать контекст
|
|
38
|
-
|
|
43
|
+
Сначала попробуй определить цель ревью без вопроса:
|
|
44
|
+
- если пользователь указал файл, папку, PR, ветку или тип diff — используй это;
|
|
45
|
+
- если пользователь просит ревью без уточнений и есть незакоммиченные изменения — используй `git diff HEAD`;
|
|
46
|
+
- если незакоммиченных изменений нет, но текущая ветка не `main`/`master` и отличается от базовой ветки — используй `git diff <base>...HEAD`, где `<base>` выбери из существующих `main`, `master`, `origin/main`, `origin/master`;
|
|
47
|
+
- если цель всё ещё неоднозначна или diff пустой — только тогда через `AskUserQuestion` спроси, что ревьюим: незакоммиченный diff (`git diff HEAD`) / diff ветки от main (`git diff main...HEAD`) / конкретный файл или папка / номер PR.
|
|
48
|
+
|
|
49
|
+
Получи diff/исходник в `$TARGET`. Прочитай `docs/rules.md`, `docs/arch.md`. Если в diff видно отсылки к плану из `docs/plans/` или исследованию из `docs/researches/` — прочитай их тоже.
|
|
39
50
|
|
|
40
51
|
### 2. Сделать первичное ревью и сохранить
|
|
41
52
|
Пройдись по изменениям. Замечания группируй по типу: bug / архитектура / стиль / тесты / безопасность / производительность / документация. Каждое — с локацией (`file:line`) и предложением «что сделать». Поставь оценку 0–100 (100 — идеально, 0 — переделывать).
|
package/package.json
CHANGED