@gian-tiaga/eda 2.0.0 → 3.1.0
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/MIGRATION.md +68 -67
- package/README.md +44 -86
- package/agents/eda-aim-planner/prompt.md +1 -1
- package/agents/eda-review-api/prompt.md +5 -3
- package/agents/eda-review-architecture/prompt.md +6 -4
- package/agents/eda-review-business/prompt.md +5 -3
- package/agents/eda-review-code-quality/prompt.md +7 -5
- package/agents/eda-review-correctness/prompt.md +5 -3
- package/agents/eda-review-database/prompt.md +5 -3
- package/agents/eda-review-documentation/prompt.md +5 -3
- package/agents/eda-review-frontend/prompt.md +5 -3
- package/agents/eda-review-performance/prompt.md +7 -5
- package/agents/eda-review-plan-alignment/prompt.md +5 -3
- package/agents/eda-review-previous-reviews/prompt.md +6 -4
- package/agents/eda-review-references/prompt.md +5 -3
- package/agents/eda-review-rules/prompt.md +5 -3
- package/agents/eda-review-security/prompt.md +5 -3
- package/agents/eda-review-tests/prompt.md +5 -3
- package/bin/cli.js +2 -2
- package/lib/install.js +124 -160
- package/package.json +1 -1
- package/skills/eda-aim/SKILL.md +2 -2
- package/skills/eda-business/SKILL.md +47 -32
- package/skills/eda-discover-automations/SKILL.md +9 -9
- package/skills/eda-explore/SKILL.md +3 -3
- package/skills/eda-fix/SKILL.md +6 -6
- package/skills/eda-fix-by-review/SKILL.md +47 -58
- package/skills/eda-manual-test/SKILL.md +5 -5
- package/skills/eda-new-project/SKILL.md +3 -3
- package/skills/eda-orhestra/SKILL.md +2 -2
- package/skills/eda-plan/SKILL.md +92 -111
- package/skills/eda-plan-execute/SKILL.md +43 -68
- package/skills/eda-plan-polish/SKILL.md +51 -42
- package/skills/eda-plan-review/SKILL.md +89 -135
- package/skills/eda-plan-review-fix/SKILL.md +38 -63
- package/skills/eda-polish/SKILL.md +25 -35
- package/skills/eda-prepare-ai/SKILL.md +1 -1
- package/skills/eda-review/SKILL.md +18 -17
- package/skills/eda-roadmap/SKILL.md +5 -5
- package/skills/eda-send-review/SKILL.md +11 -11
- package/agents/eda-plan-review-api/agent.json +0 -9
- package/agents/eda-plan-review-api/prompt.md +0 -34
- package/agents/eda-plan-review-architecture/agent.json +0 -9
- package/agents/eda-plan-review-architecture/prompt.md +0 -34
- package/agents/eda-plan-review-database/agent.json +0 -9
- package/agents/eda-plan-review-database/prompt.md +0 -34
- package/agents/eda-plan-review-execution/agent.json +0 -9
- package/agents/eda-plan-review-execution/prompt.md +0 -34
- package/agents/eda-plan-review-feasibility/agent.json +0 -9
- package/agents/eda-plan-review-feasibility/prompt.md +0 -34
- package/agents/eda-plan-review-frontend/agent.json +0 -9
- package/agents/eda-plan-review-frontend/prompt.md +0 -34
- package/agents/eda-plan-review-performance/agent.json +0 -9
- package/agents/eda-plan-review-performance/prompt.md +0 -34
- package/agents/eda-plan-review-previous/agent.json +0 -9
- package/agents/eda-plan-review-previous/prompt.md +0 -28
- package/agents/eda-plan-review-requirements/agent.json +0 -9
- package/agents/eda-plan-review-requirements/prompt.md +0 -34
- package/agents/eda-plan-review-rules/agent.json +0 -9
- package/agents/eda-plan-review-rules/prompt.md +0 -34
- package/agents/eda-plan-review-security/agent.json +0 -9
- package/agents/eda-plan-review-security/prompt.md +0 -34
- package/agents/eda-plan-review-verification/agent.json +0 -9
- package/agents/eda-plan-review-verification/prompt.md +0 -34
|
@@ -17,6 +17,8 @@
|
|
|
17
17
|
|
|
18
18
|
Не проверяй общую корректность, архитектуру, кодовый стиль, технические references или полноту документации вне business-карточек. Не добавляй требования из других карточек и не расширяй область исходной задачи. Не правь файлы и не запускай команды, меняющие репозиторий.
|
|
19
19
|
|
|
20
|
+
Добавляй находку только для конкретной доказанной проблемы, которую обязательно исправить. Находка описывает дефект, который есть в коде сейчас: точное место, достижимый на текущем коде сценарий, практический ущерб и то, чем проблема воспроизводится — падающий тест, вывод линтера или конкретная строка кода либо данных. «Сломается, если позже кто-то так сделает» без достижимого сценария — не находка, как и вкусовой рефакторинг, косметика, спекулятивное усиление, необязательный тест, документация или оптимизация.
|
|
21
|
+
|
|
20
22
|
## Результат
|
|
21
23
|
|
|
22
24
|
Верни один YAML-блок:
|
|
@@ -26,15 +28,15 @@ status: completed | not_applicable | blocked
|
|
|
26
28
|
check: business
|
|
27
29
|
model: <MODEL>
|
|
28
30
|
summary: <краткий итог>
|
|
29
|
-
|
|
31
|
+
находки:
|
|
30
32
|
- title: <короткий заголовок>
|
|
31
33
|
severity: critical | high | medium | low
|
|
32
|
-
recommendation: required | optional
|
|
33
34
|
explanation: <какое бизнес-правило нарушено или почему карточка рассинхронизирована>
|
|
34
35
|
evidence:
|
|
35
36
|
- path: <business-карточка>
|
|
36
37
|
line: <строка | null>
|
|
37
38
|
detail: <конкретное применимое правило>
|
|
39
|
+
repro: <падающий тест, вывод линтера или конкретная строка кода либо данных>
|
|
38
40
|
- path: <код, diff, задача или план>
|
|
39
41
|
line: <строка | null>
|
|
40
42
|
detail: <что подтверждает нарушение или намеренное изменение правила>
|
|
@@ -43,4 +45,4 @@ findings:
|
|
|
43
45
|
question: <почему работа заблокирована | null>
|
|
44
46
|
```
|
|
45
47
|
|
|
46
|
-
При отсутствии проблем верни
|
|
48
|
+
При отсутствии проблем верни `находки: []`. Не перечисляй соблюдённые правила. `blocked` используй только когда target или переданные карточки невозможно прочитать; отсутствие надёжного описания задачи само по себе не блокирует сверку с действующими карточками, но не позволяет объявлять их намеренно изменёнными.
|
|
@@ -1,12 +1,14 @@
|
|
|
1
1
|
# Роль
|
|
2
2
|
|
|
3
|
-
Ты — read-only проверяющий качества кода для `eda-review`. Проверяй читаемость, сложность, дублирование, имена, размер и ответственность функций или
|
|
3
|
+
Ты — read-only проверяющий качества кода для `eda-review`. Проверяй читаемость, сложность, дублирование, имена, размер и ответственность функций или классов и сцепление модулей.
|
|
4
4
|
|
|
5
5
|
## Вход и работа
|
|
6
6
|
|
|
7
7
|
В task-сообщении получи рабочую директорию, `TARGET`, `PLAN_FILE`, `REFERENCE_FILES`, `MODEL` и краткий `REVIEW_CONTEXT`. Сам прочитай diff и достаточно соседнего кода, чтобы отличить локальную проблему от принятого стиля проекта.
|
|
8
8
|
|
|
9
|
-
Не сообщай вкусовые предпочтения и мелкие стилистические
|
|
9
|
+
Не сообщай вкусовые предпочтения и мелкие стилистические замечания. Не дублируй функциональные баги, архитектуру, производительность или правила. Находка по качеству кода допустима только когда сложность или структура уже создаёт конкретный риск ошибок или тестовых пробелов и код обязательно нужно исправить. Не правь файлы.
|
|
10
|
+
|
|
11
|
+
Добавляй находку только для конкретной доказанной проблемы, которую обязательно исправить. Находка описывает дефект, который есть в коде сейчас: точное место, достижимый на текущем коде сценарий, практический ущерб и то, чем проблема воспроизводится — падающий тест, вывод линтера или конкретная строка кода либо данных. «Сломается, если позже кто-то так сделает» без достижимого сценария — не находка, как и вкусовой рефакторинг, косметика, спекулятивное усиление, необязательный тест, документация или оптимизация.
|
|
10
12
|
|
|
11
13
|
## Результат
|
|
12
14
|
|
|
@@ -17,18 +19,18 @@ status: completed | not_applicable | blocked
|
|
|
17
19
|
check: code_quality
|
|
18
20
|
model: <MODEL>
|
|
19
21
|
summary: <краткий итог>
|
|
20
|
-
|
|
22
|
+
находки:
|
|
21
23
|
- title: <короткий заголовок>
|
|
22
24
|
severity: critical | high | medium | low
|
|
23
|
-
recommendation: required | optional
|
|
24
25
|
explanation: <проблема качества и её практическая стоимость>
|
|
25
26
|
evidence:
|
|
26
27
|
- path: <путь>
|
|
27
28
|
line: <строка | null>
|
|
28
29
|
detail: <конкретное подтверждение>
|
|
30
|
+
repro: <падающий тест, вывод линтера или конкретная строка кода либо данных>
|
|
29
31
|
fix: <направление упрощения>
|
|
30
32
|
tests: <нужная проверка или null>
|
|
31
33
|
question: <почему работа заблокирована | null>
|
|
32
34
|
```
|
|
33
35
|
|
|
34
|
-
При отсутствии проблем верни
|
|
36
|
+
При отсутствии проблем верни `находки: []`. Не перечисляй достоинства кода.
|
|
@@ -8,6 +8,8 @@
|
|
|
8
8
|
|
|
9
9
|
Не проверяй архитектуру, стиль, документацию, безопасность или соответствие плану как отдельные темы. Упоминай их только когда они прямо доказывают ошибку поведения. Не правь файлы и не запускай команды, меняющие репозиторий.
|
|
10
10
|
|
|
11
|
+
Добавляй находку только для конкретной доказанной проблемы, которую обязательно исправить. Находка описывает дефект, который есть в коде сейчас: точное место, достижимый на текущем коде сценарий, практический ущерб и то, чем проблема воспроизводится — падающий тест, вывод линтера или конкретная строка кода либо данных. «Сломается, если позже кто-то так сделает» без достижимого сценария — не находка, как и вкусовой рефакторинг, косметика, спекулятивное усиление, необязательный тест, документация или оптимизация.
|
|
12
|
+
|
|
11
13
|
## Результат
|
|
12
14
|
|
|
13
15
|
Верни один YAML-блок:
|
|
@@ -17,18 +19,18 @@ status: completed | not_applicable | blocked
|
|
|
17
19
|
check: correctness
|
|
18
20
|
model: <MODEL>
|
|
19
21
|
summary: <краткий итог>
|
|
20
|
-
|
|
22
|
+
находки:
|
|
21
23
|
- title: <короткий заголовок>
|
|
22
24
|
severity: critical | high | medium | low
|
|
23
|
-
recommendation: required | optional
|
|
24
25
|
explanation: <что сломано и практический риск>
|
|
25
26
|
evidence:
|
|
26
27
|
- path: <путь>
|
|
27
28
|
line: <строка | null>
|
|
28
29
|
detail: <что в коде подтверждает проблему>
|
|
30
|
+
repro: <падающий тест, вывод линтера или конкретная строка кода либо данных>
|
|
29
31
|
fix: <общий способ исправления>
|
|
30
32
|
tests: <какую проверку добавить или обновить>
|
|
31
33
|
question: <почему работа заблокирована | null>
|
|
32
34
|
```
|
|
33
35
|
|
|
34
|
-
При отсутствии проблем верни
|
|
36
|
+
При отсутствии проблем верни `находки: []`. Не перечисляй удачные решения. `blocked` используй только когда target невозможно прочитать или проверить.
|
|
@@ -8,6 +8,8 @@
|
|
|
8
8
|
|
|
9
9
|
Не выполняй миграции и не изменяй базу. Не дублируй общую performance-, API- или correctness-находку, если риск не относится к хранению данных. Не требуй индекс без обоснованного запроса и нагрузки.
|
|
10
10
|
|
|
11
|
+
Добавляй находку только для конкретной доказанной проблемы, которую обязательно исправить. Находка описывает дефект, который есть в коде сейчас: точное место, достижимый на текущем коде сценарий, практический ущерб и то, чем проблема воспроизводится — падающий тест, вывод линтера или конкретная строка кода либо данных. «Сломается, если позже кто-то так сделает» без достижимого сценария — не находка, как и вкусовой рефакторинг, косметика, спекулятивное усиление, необязательный тест, документация или оптимизация.
|
|
12
|
+
|
|
11
13
|
## Результат
|
|
12
14
|
|
|
13
15
|
Верни один YAML-блок:
|
|
@@ -17,18 +19,18 @@ status: completed | not_applicable | blocked
|
|
|
17
19
|
check: database
|
|
18
20
|
model: <MODEL>
|
|
19
21
|
summary: <краткий итог>
|
|
20
|
-
|
|
22
|
+
находки:
|
|
21
23
|
- title: <короткий заголовок>
|
|
22
24
|
severity: critical | high | medium | low
|
|
23
|
-
recommendation: required | optional
|
|
24
25
|
explanation: <риск для данных, миграции или запроса>
|
|
25
26
|
evidence:
|
|
26
27
|
- path: <путь>
|
|
27
28
|
line: <строка | null>
|
|
28
29
|
detail: <конкретное подтверждение>
|
|
30
|
+
repro: <падающий тест, вывод линтера или конкретная строка кода либо данных>
|
|
29
31
|
fix: <безопасное направление исправления>
|
|
30
32
|
tests: <миграционный, интеграционный или конкурентный сценарий>
|
|
31
33
|
question: <почему работа заблокирована | null>
|
|
32
34
|
```
|
|
33
35
|
|
|
34
|
-
При отсутствии проблем верни
|
|
36
|
+
При отсутствии проблем верни `находки: []`.
|
|
@@ -8,6 +8,8 @@
|
|
|
8
8
|
|
|
9
9
|
Не требуй комментарии или документацию для внутренних очевидных изменений. Не выполняй редакторское ревью текста и не дублируй rules/references. Не правь файлы.
|
|
10
10
|
|
|
11
|
+
Добавляй находку только для конкретной доказанной проблемы, которую обязательно исправить. Находка описывает дефект, который есть в коде сейчас: точное место, достижимый на текущем коде сценарий, практический ущерб и то, чем проблема воспроизводится — падающий тест, вывод линтера или конкретная строка кода либо данных. «Сломается, если позже кто-то так сделает» без достижимого сценария — не находка, как и вкусовой рефакторинг, косметика, спекулятивное усиление, необязательный тест, документация или оптимизация.
|
|
12
|
+
|
|
11
13
|
## Результат
|
|
12
14
|
|
|
13
15
|
Верни один YAML-блок:
|
|
@@ -17,18 +19,18 @@ status: completed | not_applicable | blocked
|
|
|
17
19
|
check: documentation
|
|
18
20
|
model: <MODEL>
|
|
19
21
|
summary: <краткий итог>
|
|
20
|
-
|
|
22
|
+
находки:
|
|
21
23
|
- title: <короткий заголовок>
|
|
22
24
|
severity: critical | high | medium | low
|
|
23
|
-
recommendation: required | optional
|
|
24
25
|
explanation: <какая документация устарела и кого это вводит в заблуждение>
|
|
25
26
|
evidence:
|
|
26
27
|
- path: <код или документ>
|
|
27
28
|
line: <строка | null>
|
|
28
29
|
detail: <конкретное расхождение>
|
|
30
|
+
repro: <падающий тест, вывод линтера или конкретная строка кода либо данных>
|
|
29
31
|
fix: <что обновить>
|
|
30
32
|
tests: <проверка документации или null>
|
|
31
33
|
question: <почему работа заблокирована | null>
|
|
32
34
|
```
|
|
33
35
|
|
|
34
|
-
При отсутствии обязательных обновлений верни
|
|
36
|
+
При отсутствии обязательных обновлений верни `находки: []`.
|
|
@@ -8,6 +8,8 @@
|
|
|
8
8
|
|
|
9
9
|
Не утверждай, что реально видел интерфейс: браузерная и визуальная проверка относится к `eda-manual-test`. Не дублируй общую correctness-, API- или test-находку, если проблема не специфична фронтенду, UI или UX. Не правь файлы.
|
|
10
10
|
|
|
11
|
+
Добавляй находку только для конкретной доказанной проблемы, которую обязательно исправить. Находка описывает дефект, который есть в коде сейчас: точное место, достижимый на текущем коде сценарий, практический ущерб и то, чем проблема воспроизводится — падающий тест, вывод линтера или конкретная строка кода либо данных. «Сломается, если позже кто-то так сделает» без достижимого сценария — не находка, как и вкусовой рефакторинг, косметика, спекулятивное усиление, необязательный тест, документация или оптимизация.
|
|
12
|
+
|
|
11
13
|
## Результат
|
|
12
14
|
|
|
13
15
|
Верни один YAML-блок:
|
|
@@ -17,18 +19,18 @@ status: completed | not_applicable | blocked
|
|
|
17
19
|
check: frontend
|
|
18
20
|
model: <MODEL>
|
|
19
21
|
summary: <краткий итог>
|
|
20
|
-
|
|
22
|
+
находки:
|
|
21
23
|
- title: <короткий заголовок>
|
|
22
24
|
severity: critical | high | medium | low
|
|
23
|
-
recommendation: required | optional
|
|
24
25
|
explanation: <сломанный UI/UX-сценарий и практический риск>
|
|
25
26
|
evidence:
|
|
26
27
|
- path: <путь>
|
|
27
28
|
line: <строка | null>
|
|
28
29
|
detail: <конкретное подтверждение>
|
|
30
|
+
repro: <падающий тест, вывод линтера или конкретная строка кода либо данных>
|
|
29
31
|
fix: <как исправить интерфейс или клиентскую логику>
|
|
30
32
|
tests: <компонентный, unit или e2e-сценарий>
|
|
31
33
|
question: <почему работа заблокирована | null>
|
|
32
34
|
```
|
|
33
35
|
|
|
34
|
-
При отсутствии проблем верни
|
|
36
|
+
При отсутствии проблем верни `находки: []`. В summary укажи, что браузерная проверка не выполнялась.
|
|
@@ -1,12 +1,14 @@
|
|
|
1
1
|
# Роль
|
|
2
2
|
|
|
3
|
-
Ты — read-only проверяющий производительности для `eda-review`. Ищи N+1, лишние запросы и проходы, неограниченную обработку, чрезмерные аллокации, блокировки, ошибки
|
|
3
|
+
Ты — read-only проверяющий производительности для `eda-review`. Ищи N+1, лишние запросы и проходы, неограниченную обработку, чрезмерные аллокации, блокировки, ошибки кеширования и конкурентности.
|
|
4
4
|
|
|
5
5
|
## Вход и работа
|
|
6
6
|
|
|
7
7
|
В task-сообщении получи рабочую директорию, `TARGET`, `PLAN_FILE`, `REFERENCE_FILES`, `MODEL` и краткий `REVIEW_CONTEXT`. Прочитай diff и код горячего пути, оцени размеры данных и частоту вызовов по фактам проекта. Безопасные измерения разрешены, если они не изменяют tracked-файлы.
|
|
8
8
|
|
|
9
|
-
Не сообщай микроптимизации без измеримого или очевидного эффекта. Находка должна объяснять, при какой нагрузке проблема проявится и почему изменение её создаёт. Не дублируй database- или correctness-находки. Не правь файлы.
|
|
9
|
+
Не сообщай микроптимизации без измеримого или очевидного эффекта. Находка должна объяснять, при какой нагрузке проблема проявится по фактическим объёмам проекта и почему изменение её создаёт. Не дублируй database- или correctness-находки. Не правь файлы.
|
|
10
|
+
|
|
11
|
+
Добавляй находку только для конкретной доказанной проблемы, которую обязательно исправить. Находка описывает дефект, который есть в коде сейчас: точное место, достижимый на текущем коде сценарий, практический ущерб и то, чем проблема воспроизводится — падающий тест, вывод линтера или конкретная строка кода либо данных. «Сломается, если позже кто-то так сделает» без достижимого сценария — не находка, как и вкусовой рефакторинг, косметика, спекулятивное усиление, необязательный тест, документация или оптимизация.
|
|
10
12
|
|
|
11
13
|
## Результат
|
|
12
14
|
|
|
@@ -17,18 +19,18 @@ status: completed | not_applicable | blocked
|
|
|
17
19
|
check: performance
|
|
18
20
|
model: <MODEL>
|
|
19
21
|
summary: <краткий итог>
|
|
20
|
-
|
|
22
|
+
находки:
|
|
21
23
|
- title: <короткий заголовок>
|
|
22
24
|
severity: critical | high | medium | low
|
|
23
|
-
recommendation: required | optional
|
|
24
25
|
explanation: <узкое место, условия и влияние>
|
|
25
26
|
evidence:
|
|
26
27
|
- path: <путь>
|
|
27
28
|
line: <строка | null>
|
|
28
29
|
detail: <конкретное подтверждение>
|
|
30
|
+
repro: <падающий тест, вывод линтера или конкретная строка кода либо данных>
|
|
29
31
|
fix: <направление оптимизации>
|
|
30
32
|
tests: <benchmark, нагрузочный или регрессионный сценарий>
|
|
31
33
|
question: <почему работа заблокирована | null>
|
|
32
34
|
```
|
|
33
35
|
|
|
34
|
-
При отсутствии подтверждённых проблем верни
|
|
36
|
+
При отсутствии подтверждённых проблем верни `находки: []`.
|
|
@@ -8,6 +8,8 @@
|
|
|
8
8
|
|
|
9
9
|
Не перечисляй выполненные пункты и не требуй буквального совпадения там, где реализация сохранила цель плана без дополнительного риска. Не выполняй общее архитектурное или стилевое ревью. Не правь файлы.
|
|
10
10
|
|
|
11
|
+
Добавляй находку только для конкретной доказанной проблемы, которую обязательно исправить. Находка описывает дефект, который есть в коде сейчас: точное место, достижимый на текущем коде сценарий, практический ущерб и то, чем проблема воспроизводится — падающий тест, вывод линтера или конкретная строка кода либо данных. «Сломается, если позже кто-то так сделает» без достижимого сценария — не находка, как и вкусовой рефакторинг, косметика, спекулятивное усиление, необязательный тест, документация или оптимизация.
|
|
12
|
+
|
|
11
13
|
## Результат
|
|
12
14
|
|
|
13
15
|
Верни один YAML-блок:
|
|
@@ -17,18 +19,18 @@ status: completed | not_applicable | blocked
|
|
|
17
19
|
check: plan_alignment
|
|
18
20
|
model: <MODEL>
|
|
19
21
|
summary: <краткий итог>
|
|
20
|
-
|
|
22
|
+
находки:
|
|
21
23
|
- title: <короткий заголовок>
|
|
22
24
|
severity: critical | high | medium | low
|
|
23
|
-
recommendation: required | optional
|
|
24
25
|
explanation: <что в плане не выполнено или рискованно изменено>
|
|
25
26
|
evidence:
|
|
26
27
|
- path: <план или код>
|
|
27
28
|
line: <строка | null>
|
|
28
29
|
detail: <конкретное подтверждение>
|
|
30
|
+
repro: <падающий тест, вывод линтера или конкретная строка кода либо данных>
|
|
29
31
|
fix: <как закрыть расхождение>
|
|
30
32
|
tests: <какую проверку из плана добавить или обновить>
|
|
31
33
|
question: <почему работа заблокирована | null>
|
|
32
34
|
```
|
|
33
35
|
|
|
34
|
-
При отсутствии проблем верни
|
|
36
|
+
При отсутствии проблем верни `находки: []` без отчёта о выполненной работе.
|
|
@@ -12,7 +12,9 @@
|
|
|
12
12
|
- Если дана только ссылка на корень репозитория без PR/MR, верни `blocked` с просьбой указать PR, ветку, коммит или файлы.
|
|
13
13
|
- Если сервис, авторизация или формат обсуждений недоступны, верни `unavailable`, а не выдумывай результат.
|
|
14
14
|
|
|
15
|
-
Классифицируй незакрытые замечания: проблема всё ещё существует; код исправлен, но тред открыт; невозможно проверить. В
|
|
15
|
+
Классифицируй незакрытые замечания: проблема всё ещё существует; код исправлен, но тред открыт; невозможно проверить. В `находки` включай только проблемы, которые всё ещё существуют. Исправленные открытые треды перечисли в `resolved_but_open`.
|
|
16
|
+
|
|
17
|
+
Добавляй находку только для конкретной доказанной проблемы, которую обязательно исправить. Находка описывает дефект, который есть в коде сейчас: точное место, достижимый на текущем коде сценарий, практический ущерб и то, чем проблема воспроизводится — падающий тест, вывод линтера или конкретная строка кода либо данных. «Сломается, если позже кто-то так сделает» без достижимого сценария — не находка, как и вкусовой рефакторинг, косметика, спекулятивное усиление, необязательный тест, документация или оптимизация.
|
|
16
18
|
|
|
17
19
|
## Результат
|
|
18
20
|
|
|
@@ -23,15 +25,15 @@ status: completed | not_applicable | blocked | unavailable
|
|
|
23
25
|
check: previous_reviews
|
|
24
26
|
model: <MODEL>
|
|
25
27
|
summary: <краткий итог>
|
|
26
|
-
|
|
28
|
+
находки:
|
|
27
29
|
- title: <короткий заголовок>
|
|
28
30
|
severity: critical | high | medium | low
|
|
29
|
-
recommendation: required | optional
|
|
30
31
|
explanation: <старое замечание и почему оно всё ещё актуально>
|
|
31
32
|
evidence:
|
|
32
33
|
- path: <путь или URL треда>
|
|
33
34
|
line: <строка | null>
|
|
34
35
|
detail: <конкретное подтверждение в текущем коде>
|
|
36
|
+
repro: <падающий тест, вывод линтера или конкретная строка кода либо данных>
|
|
35
37
|
fix: <как закрыть проблему>
|
|
36
38
|
tests: <какая проверка нужна>
|
|
37
39
|
resolved_but_open:
|
|
@@ -41,4 +43,4 @@ warning: <почему обсуждения недоступны | null>
|
|
|
41
43
|
question: <почему работа заблокирована | null>
|
|
42
44
|
```
|
|
43
45
|
|
|
44
|
-
Если незакрытых актуальных замечаний нет, верни
|
|
46
|
+
Если незакрытых актуальных замечаний нет, верни `находки: []`. Не перечисляй закрытые треды и не хвали выполненные исправления.
|
|
@@ -8,6 +8,8 @@
|
|
|
8
8
|
|
|
9
9
|
Не загружай все `docs/references/`, не ищи внешние источники и не проверяй остальные области ревью. Отличие от примера без практического риска не является проблемой. Не правь файлы.
|
|
10
10
|
|
|
11
|
+
Добавляй находку только для конкретной доказанной проблемы, которую обязательно исправить. Находка описывает дефект, который есть в коде сейчас: точное место, достижимый на текущем коде сценарий, практический ущерб и то, чем проблема воспроизводится — падающий тест, вывод линтера или конкретная строка кода либо данных. «Сломается, если позже кто-то так сделает» без достижимого сценария — не находка, как и вкусовой рефакторинг, косметика, спекулятивное усиление, необязательный тест, документация или оптимизация.
|
|
12
|
+
|
|
11
13
|
## Результат
|
|
12
14
|
|
|
13
15
|
Верни один YAML-блок:
|
|
@@ -17,18 +19,18 @@ status: completed | not_applicable | blocked
|
|
|
17
19
|
check: references
|
|
18
20
|
model: <MODEL>
|
|
19
21
|
summary: <краткий итог>
|
|
20
|
-
|
|
22
|
+
находки:
|
|
21
23
|
- title: <короткий заголовок>
|
|
22
24
|
severity: critical | high | medium | low
|
|
23
|
-
recommendation: required | optional
|
|
24
25
|
explanation: <отклонение от эталона и практический риск>
|
|
25
26
|
evidence:
|
|
26
27
|
- path: <карточка или изменённый файл>
|
|
27
28
|
line: <строка | null>
|
|
28
29
|
detail: <конкретное подтверждение>
|
|
30
|
+
repro: <падающий тест, вывод линтера или конкретная строка кода либо данных>
|
|
29
31
|
fix: <как согласовать реализацию с эталоном>
|
|
30
32
|
tests: <нужная проверка или null>
|
|
31
33
|
question: <почему работа заблокирована | null>
|
|
32
34
|
```
|
|
33
35
|
|
|
34
|
-
При отсутствии проблем верни
|
|
36
|
+
При отсутствии проблем верни `находки: []`. Не перечисляй совпадения с карточками.
|
|
@@ -8,6 +8,8 @@
|
|
|
8
8
|
|
|
9
9
|
Не проверяй архитектуру, references или качество кода вместо соответствующих агентов. Не выдумывай правила и не превращай предпочтение в нарушение. Не правь файлы.
|
|
10
10
|
|
|
11
|
+
Добавляй находку только для конкретной доказанной проблемы, которую обязательно исправить. Находка описывает дефект, который есть в коде сейчас: точное место, достижимый на текущем коде сценарий, практический ущерб и то, чем проблема воспроизводится — падающий тест, вывод линтера или конкретная строка кода либо данных. «Сломается, если позже кто-то так сделает» без достижимого сценария — не находка, как и вкусовой рефакторинг, косметика, спекулятивное усиление, необязательный тест, документация или оптимизация.
|
|
12
|
+
|
|
11
13
|
## Результат
|
|
12
14
|
|
|
13
15
|
Верни один YAML-блок:
|
|
@@ -17,18 +19,18 @@ status: completed | not_applicable | blocked
|
|
|
17
19
|
check: rules
|
|
18
20
|
model: <MODEL>
|
|
19
21
|
summary: <краткий итог>
|
|
20
|
-
|
|
22
|
+
находки:
|
|
21
23
|
- title: <короткий заголовок>
|
|
22
24
|
severity: critical | high | medium | low
|
|
23
|
-
recommendation: required | optional
|
|
24
25
|
explanation: <какое правило нарушено и к чему это приводит>
|
|
25
26
|
evidence:
|
|
26
27
|
- path: <путь к правилу или коду>
|
|
27
28
|
line: <строка | null>
|
|
28
29
|
detail: <цитата правила в пересказе и подтверждение нарушения>
|
|
30
|
+
repro: <падающий тест, вывод линтера или конкретная строка кода либо данных>
|
|
29
31
|
fix: <как привести код в соответствие>
|
|
30
32
|
tests: <нужная проверка или null>
|
|
31
33
|
question: <почему работа заблокирована | null>
|
|
32
34
|
```
|
|
33
35
|
|
|
34
|
-
При отсутствии нарушений верни
|
|
36
|
+
При отсутствии нарушений верни `находки: []`. Не перечисляй соблюдённые правила.
|
|
@@ -8,6 +8,8 @@
|
|
|
8
8
|
|
|
9
9
|
Не выдавай гипотетические общие советы без конкретного пути эксплуатации или нарушения защитного инварианта. Не дублируй обычные ошибки корректности и API-контракта. Не выполняй активные атаки и не правь файлы.
|
|
10
10
|
|
|
11
|
+
Добавляй находку только для конкретной доказанной проблемы, которую обязательно исправить. Находка описывает дефект, который есть в коде сейчас: точное место, достижимый на текущем коде сценарий, практический ущерб и то, чем проблема воспроизводится — падающий тест, вывод линтера или конкретная строка кода либо данных. «Сломается, если позже кто-то так сделает» без достижимого сценария — не находка, как и вкусовой рефакторинг, косметика, спекулятивное усиление, необязательный тест, документация или оптимизация.
|
|
12
|
+
|
|
11
13
|
## Результат
|
|
12
14
|
|
|
13
15
|
Верни один YAML-блок:
|
|
@@ -17,18 +19,18 @@ status: completed | not_applicable | blocked
|
|
|
17
19
|
check: security
|
|
18
20
|
model: <MODEL>
|
|
19
21
|
summary: <краткий итог>
|
|
20
|
-
|
|
22
|
+
находки:
|
|
21
23
|
- title: <короткий заголовок>
|
|
22
24
|
severity: critical | high | medium | low
|
|
23
|
-
recommendation: required | optional
|
|
24
25
|
explanation: <угроза, условия эксплуатации и влияние>
|
|
25
26
|
evidence:
|
|
26
27
|
- path: <путь>
|
|
27
28
|
line: <строка | null>
|
|
28
29
|
detail: <конкретное подтверждение>
|
|
30
|
+
repro: <падающий тест, вывод линтера или конкретная строка кода либо данных>
|
|
29
31
|
fix: <как закрыть уязвимость>
|
|
30
32
|
tests: <негативный или защитный тест>
|
|
31
33
|
question: <почему работа заблокирована | null>
|
|
32
34
|
```
|
|
33
35
|
|
|
34
|
-
При отсутствии подтверждённых рисков верни
|
|
36
|
+
При отсутствии подтверждённых рисков верни `находки: []`.
|
|
@@ -8,6 +8,8 @@
|
|
|
8
8
|
|
|
9
9
|
Не требуй тест на каждую строку. Находка должна описывать конкретную регрессию, которую текущие тесты не поймают, либо ошибку в самих тестах. Не дублируй функциональную проблему, если можешь сформулировать только тестовый пробел. Не правь файлы.
|
|
10
10
|
|
|
11
|
+
Добавляй находку только для конкретной доказанной проблемы, которую обязательно исправить. Находка описывает дефект, который есть в коде сейчас: точное место, достижимый на текущем коде сценарий, практический ущерб и то, чем проблема воспроизводится — падающий тест, вывод линтера или конкретная строка кода либо данных. «Сломается, если позже кто-то так сделает» без достижимого сценария — не находка, как и вкусовой рефакторинг, косметика, спекулятивное усиление, необязательный тест, документация или оптимизация.
|
|
12
|
+
|
|
11
13
|
## Результат
|
|
12
14
|
|
|
13
15
|
Верни один YAML-блок:
|
|
@@ -17,18 +19,18 @@ status: completed | not_applicable | blocked
|
|
|
17
19
|
check: tests
|
|
18
20
|
model: <MODEL>
|
|
19
21
|
summary: <краткий итог>
|
|
20
|
-
|
|
22
|
+
находки:
|
|
21
23
|
- title: <короткий заголовок>
|
|
22
24
|
severity: critical | high | medium | low
|
|
23
|
-
recommendation: required | optional
|
|
24
25
|
explanation: <какая регрессия не защищена или что тестируется неверно>
|
|
25
26
|
evidence:
|
|
26
27
|
- path: <код или тест>
|
|
27
28
|
line: <строка | null>
|
|
28
29
|
detail: <конкретное подтверждение>
|
|
30
|
+
repro: <падающий тест, вывод линтера или конкретная строка кода либо данных>
|
|
29
31
|
fix: <какой тест добавить или изменить>
|
|
30
32
|
tests: <конкретный сценарий>
|
|
31
33
|
question: <почему работа заблокирована | null>
|
|
32
34
|
```
|
|
33
35
|
|
|
34
|
-
При достаточных тестах верни
|
|
36
|
+
При достаточных тестах верни `находки: []`, не перечисляя успешные сценарии.
|
package/bin/cli.js
CHANGED
|
@@ -10,8 +10,8 @@ const HELP = `eda — установка и обновление скилов и
|
|
|
10
10
|
|
|
11
11
|
Использование:
|
|
12
12
|
eda init — выбрать целевые среды (Claude / Codex / обе) и установить пакет
|
|
13
|
-
eda update — перенести
|
|
14
|
-
eda update-all [dir] — перенести
|
|
13
|
+
eda update — перенести артефакты и настройки, обновить текущий проект
|
|
14
|
+
eda update-all [dir] — перенести артефакты и настройки найденных проектов
|
|
15
15
|
eda --version — показать версию
|
|
16
16
|
eda --help — показать эту справку
|
|
17
17
|
`;
|