@gian-tiaga/eda 3.3.0 → 3.3.1
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
CHANGED
package/skills/eda-plan/SKILL.md
CHANGED
|
@@ -57,7 +57,7 @@ description: 'Создаёт понятный исполнимый план ре
|
|
|
57
57
|
7. План не отклоняется от `docs/rules.md`, `docs/arch.md` и карточек. Если нужное решение им противоречит, план сначала меняет сам документ: добавь отдельный пункт `Обновить правила` или `Обновить архитектуру: <тема и изменение>` до зависимой реализации. Отклонения «с причиной в докблоке» не бывает — она разводит код и документацию.
|
|
58
58
|
8. Не переформулируй запрос пользователя перед Plan Mode: передавай оригинал, ответы на вопросы и найденную рамку.
|
|
59
59
|
9. План сохраняется до любой включённой проверки.
|
|
60
|
-
10. План фиксирует решения, границы, контракты и критерии приёмки.
|
|
60
|
+
10. План фиксирует решения, границы, контракты и критерии приёмки. Формулы и компактный псевдокод допустимы, когда они точнее прозы задают бизнес-правило, контракт, инвариант или критерий приёмки, например `retryAt = now + 5 minutes`. Сам формульный синтаксис не превращает контракт в деталь реализации. Не фиксируй сигнатуры чужих методов, приведения типов, порядок строк внутри метода, форму классов и внутренние вычисления, которые не задают наблюдаемое поведение. Если утверждение можно проверить только по конкретному коду, оно относится к code review, а не к плану.
|
|
61
61
|
11. Не пересказывай прозой то, что уже лежит рядом: правила, архитектуру и кодовые примеры карточек. Ссылайся на них.
|
|
62
62
|
12. Один факт — одно место. Поведение описано в `Целевом алгоритме`, контракты — в `Контрактах реализации`, фаза на них опирается и не повторяет их.
|
|
63
63
|
13. План доводится до первой точки, где результат можно запустить и проверить целиком. Дальше планируй следующей вехой, на фактах.
|
|
@@ -186,7 +186,7 @@ Quality gate перед сохранением и после каждой вкл
|
|
|
186
186
|
- каждый отсутствующий, неполный или расширяемый пункт целевого поведения покрыт отдельной задачей `Дополнить бизнесовые правила` до зависимой реализации; для чисто технических изменений такая задача не добавлена;
|
|
187
187
|
- план строго следует `docs/rules.md` и `docs/arch.md`, а также применимым reference-карточкам; там, где нужно другое поведение, есть отдельный пункт на изменение самого документа до зависимой реализации;
|
|
188
188
|
- каждая фаза имеет цель, что сделать, результат и проверку и выполнима одним агентом за один проход;
|
|
189
|
-
- в
|
|
189
|
+
- в плане нет сигнатур, приведений типов, формы классов, внутренних вычислений без контрактного смысла и пересказа карточек; формулы, задающие обязательное поведение, не удалены только из-за формы записи;
|
|
190
190
|
- ни один факт не описан в двух разделах;
|
|
191
191
|
- `test_strategy` и `logging_strategy` применены;
|
|
192
192
|
- пакеты имеют конкретные версии или ссылку на существующий lock;
|
|
@@ -233,7 +233,7 @@ codex exec "Прочитай $PLAN_FILE, docs/rules.md, docs/arch.md и прим
|
|
|
233
233
|
|
|
234
234
|
- Править код, миграции, конфиги или зависимости.
|
|
235
235
|
- Считать `docs/rules.md` и `docs/arch.md` просто контекстом для ознакомления.
|
|
236
|
-
- Писать в план сигнатуры чужих методов,
|
|
236
|
+
- Писать в план сигнатуры чужих методов, приведения типов, форму классов и формулы внутренних вычислений, которые не задают бизнес-правило, контракт, инвариант или критерий приёмки.
|
|
237
237
|
- Пересказывать прозой правила, архитектуру и кодовые примеры карточек.
|
|
238
238
|
- Отклоняться от правил и архитектуры «с причиной» вместо того, чтобы менять сам документ.
|
|
239
239
|
- Разбивать фазы на задачи с идентификаторами и графом зависимостей.
|
|
@@ -44,7 +44,7 @@ Read-only проверка одного плана по списку ниже.
|
|
|
44
44
|
| 7 | Пакеты, библиотеки и сервисы имеют конкретные версии или ссылку на существующий lock |
|
|
45
45
|
| 8 | Есть `Целевой алгоритм` и `Контракты реализации` с подразделами `Данные и БД` и `API и внешние контракты` |
|
|
46
46
|
| 9 | Ни один факт не описан в двух разделах |
|
|
47
|
-
| 10 | В плане нет сигнатур чужих методов,
|
|
47
|
+
| 10 | В плане нет сигнатур чужих методов, приведений типов, формы классов, внутренних вычислений без контрактного смысла и пересказа кодовых примеров карточек; формулы, задающие обязательное поведение, допустимы |
|
|
48
48
|
| 11 | У каждой фазы есть цель, что сделать, результат и проверка |
|
|
49
49
|
| 12 | Каждая фаза выполнима одним агентом за один проход вместе с её проверками |
|
|
50
50
|
| 13 | Команды в `Проверке` фазы существуют к моменту её выполнения или создаются в этой же фазе |
|
|
@@ -69,6 +69,10 @@ Read-only проверка одного плана по списку ниже.
|
|
|
69
69
|
|
|
70
70
|
Пункт не касается: условных веток («при ошибке — повтор» это описание ветки, а не модальность) и умолчаний, заданных `rules.md` или карточкой — их повторять в плане не нужно.
|
|
71
71
|
|
|
72
|
+
### Как проверять пункт 10
|
|
73
|
+
|
|
74
|
+
Сам по себе формульный синтаксис не является основанием для `fail`. Формула или компактный псевдокод допустимы, если они точнее прозы задают бизнес-правило, контракт, инвариант или критерий приёмки. Например, `retryAt = now + 5 minutes` фиксирует обязательный момент повторной обработки и остаётся в плане. Ставь `fail` только за внутренние вычисления и кодовые детали, которые не меняют наблюдаемое поведение и не нужны исполнителю для однозначного результата.
|
|
75
|
+
|
|
72
76
|
## Правила проверки
|
|
73
77
|
|
|
74
78
|
1. Не правь план, код и документацию. Не запускай тесты, линтеры, сборку, миграции и внешние запросы.
|
|
@@ -21,7 +21,7 @@ description: 'Закрывает проваленные пункты чек-ли
|
|
|
21
21
|
4. Закрой каждый пункт со статусом `fail`. Пункты `pass` и `n/a` не трогай. Своих замечаний не добавляй: чего нет в отчёте, того нет в работе.
|
|
22
22
|
5. **План не растёт.** Сначала ищи, что удалить или заменить, и только потом — что дописать. Прирост объёма за один fix не больше 3%; если пункт иначе не закрывается, остановись и верни `blocked: пункт требует роста плана` с объяснением.
|
|
23
23
|
6. Не повторяй один факт в двух разделах: поведение живёт в `Целевом алгоритме`, контракты — в `Контрактах реализации`, фаза на них ссылается.
|
|
24
|
-
7. Не добавляй в план сигнатуры чужих методов,
|
|
24
|
+
7. Не добавляй в план сигнатуры чужих методов, приведения типов, форму классов, внутренние вычисления без контрактного смысла и пересказ карточек — это те же пункты 9 и 10 чек-листа, только с другой стороны. Не удаляй и не переписывай только из-за формы записи формулу, которая задаёт бизнес-правило, контракт, инвариант или критерий приёмки.
|
|
25
25
|
8. Продуктовый выбор, конфликт с business, rules или architecture и расширение scope не решай сам: собери все такие случаи и задай один блокирующий вопрос до любых правок.
|
|
26
26
|
9. Не правь код, конфиги, зависимости, миграции и другие документы. Не запускай проверки проекта.
|
|
27
27
|
|