@gian-tiaga/eda 3.1.0 → 3.2.0

package/MIGRATION.md

@@ -71,7 +71,7 @@ docs/
 
 Версия npm-пакета и версия схемы настроек независимы: в eda 3.1 актуальным остаётся `docs/settings.yaml` с `version: 3`.
 
-Установщик по-прежнему сохраняет полный валидный v3 байт-в-байт. Старый, неизвестный или неполный YAML нормализуется: известные валидные значения переносятся, а отсутствующие спрашиваются в TTY или получают defaults без TTY.
+Установщик по-прежнему сохраняет полный валидный v3 байт-в-байт — кроме случая, когда в файле остались устаревшие ключи: тогда он перезаписывает файл без вопросов и печатает, что именно убрал. Старый, неизвестный или неполный YAML нормализуется: известные валидные значения переносятся, а отсутствующие спрашиваются в TTY или получают defaults без TTY.
 
 В `eda update-all` режим `configure` один раз спрашивает общий профиль v3 и записывает его во все проекты. Режим `skip` сохраняет полный v3, индивидуально переносит остальные конфиги и создаёт defaults только при отсутствии файла.
 
@@ -80,8 +80,8 @@ docs/
 Что в 3.1 изменится в уже настроенном проекте:
 
 - **Удалены двенадцать агентов `eda-plan-review-*`.** Проверку плана целиком выполняет один субагент по фиксированному чек-листу. `eda update` стирает файлы этих агентов из `.claude/agents/` и `.codex/agents/`.
-- **Секция `plan-review.agents` больше не читается** и при нормализации настроек отбрасывается. Дефолт `plan-review.threshold` — `100`, `plan-polish.limit` — `2`.
-- **У `eda-polish` больше нет порога.** Цикл завершается только при нуле находок и останавливается досрочно, если за круг число открытых находок не сократилось. Существующий валидный `docs/settings.yaml` установщик не переписывает, поэтому аргумент вида `args: "threshold 90 limit 2"` в шаге оркестратора останется в файле и будет игнорироваться — убери из него `threshold N` вручную.
+- **Секция `plan-review.agents` больше не читается.** Начиная с 3.1.1 `eda update` находит её в готовом файле настроек и убирает вместе с другими устаревшими ключами — `review-check`, `review.strict`, `review.include_code_quality`. Дефолт `plan-review.threshold` — `100`, `plan-polish.limit` — `2`.
+- **У `eda-polish` больше нет порога.** Цикл завершается только при нуле находок и останавливается досрочно, если за круг число открытых находок не сократилось. Аргумент вида `args: "threshold 90 limit 2"` в шаге оркестратора `eda update` приводит к виду `args: "limit 2"`.
 - **Формат ревью дополнен.** У каждой находки появилось поле `repro` — чем проблема воспроизводится, — а в отчёт добавлен раздел «Отклонено» с причинами отклонения. `eda-fix-by-review` может отклонить находку, но только фактически неверную и с обоснованием в отчёте. Старые файлы ревью читаются как есть.
 
 ## Проверка миграции
@@ -92,6 +92,7 @@ docs/
 - `docs/business.md`, `docs/business/`, `docs/references.md`, `docs/references/`, `docs/arch.md` и `docs/rules.md` остались на месте;
 - `docs/settings.yaml` содержит `version: 3`;
 - `eda --version` выводит версию `3.1.x`;
+- в `docs/settings.yaml` нет секции `plan-review.agents` и аргумента `threshold N` у шага `eda-polish`;
 - в `.claude/agents/` и `.codex/agents/` не осталось файлов `eda-plan-review-*`;
 - в `.claude/` или `.codex/` установлены обновлённые скилы и агенты.
 

package/README.md

@@ -58,7 +58,7 @@ eda update-all ~/Code
 | `eda-manual-test` | Делает ручную smoke/acceptance-проверку после изменений: основной чеклист строит по выполненной задаче и связанному плану/issue/review/fix-отчёту, затем сверяет фактический diff. Применимая business-карточка может добавить только относящийся к задаче инвариант или сценарий; несвязанные карточки не загружаются. Не правит код, не пишет автотесты и не коммитит | `docs/artifacts/manual-tests/` |
 | `eda-polish` | Оркестрирует доводку изменений: запускает полное `eda-review`, затем `eda-fix-by-review` и повторяет цикл до `clean`, отсутствия находок и оценки не ниже 95/100 либо до лимита 5 итераций. Все находки исправляются. Каждый шаг идёт в изолированном нативном субагенте. Не коммитит | `docs/artifacts/reviews/`, `docs/artifacts/review-fixes/` |
 | `eda-send-review` | На `sonnet` / `gpt-5.6-terra` отправляет ревью на GitHub PR через `gh` всегда в едином формате: общий комментарий-сводка плюс комментарии к строкам кода. Без явно указанного типа использует `comment`; `approve` и `request-changes` берёт только из контекста текущего вызова. По настройке может скрыть предыдущие сводки eda-send-review и зарезолвить их inline-треды. По умолчанию шлёт все пункты. Claude-модель закрепляется при установке; Codex-модель хранится как рекомендуемый будущий override | (комментарий в PR) |
-| `eda-commit` | Работает как лёгкий оркестратор одного `haiku` / `gpt-5.6-luna` агента: тот сам собирает git-контекст, выбирает один логический набор файлов, пишет сообщение, точечно коммитит и выполняет явно заданные push/PR/merge-действия. В Claude Code сам оркестратор закреплён за `haiku`; в Codex остаётся на активной модели сессии из-за отсутствия per-skill override. При неоднозначном составе или вероятном секрете агент блокируется до индексации. Если продолжение не задано, предлагает применимые варианты | (git, опционально GitHub PR) |
+| `eda-commit` | Работает как лёгкий оркестратор одного `haiku` / `gpt-5.6-luna` агента: тот сам собирает git-контекст, выбирает один логический набор файлов, пишет сообщение, точечно коммитит и выполняет явно заданные push/PR/merge-действия. В Claude Code сам оркестратор закреплён за `haiku`; в Codex остаётся на активной модели сессии из-за отсутствия per-skill override. При неоднозначном составе или вероятном секрете агент блокируется до индексации. После любого push всегда возвращает ссылку — на существующий PR или на его создание — и спрашивает, создавать ли PR, если его нет. Если продолжение не задано, предлагает применимые варианты | (git, опционально GitHub PR) |
 | `eda-worktree` | На `haiku` / `gpt-5.6-luna` создаёт через пакетного агента git worktree рядом с основным проектом в папке `{name}-work-{n}` и одноимённую ветку. Базу берёт из аргумента или спрашивает | соседняя папка `{name}-work-{n}` |
 | `eda-merge-worktree` | На `haiku` / `gpt-5.6-luna` мержит через пакетного агента ветку из соседнего worktree в текущую ветку. Принимает номер `1`, короткое имя `work-1` или полное имя `{name}-work-1`. Worktree и ветку после merge не удаляет | (git) |
 | `eda-prepare-ai` | Выполняет технический bootstrap нового или существующего проекта: проектирует архитектуру, строгие правила и профиль инструментов, проходит полный каталог автоматических проверок, подбирает применимые AI-скилы, роли и MCP, затем создаёт AI-документацию. Подтверждённые требования, стек и архитектурные драйверы из `eda-new-project` принимает как обязательный вход и не переоткрывает без обнаруженного конфликта. Для пустого проекта не выдумывает reference-карточки и не выдаёт неустановленные инструменты за доступные | `docs/rules.md`, `docs/arch.md`, `docs/references.md`, `docs/references/`, `AGENTS.md`, `CLAUDE.md` |

package/agents/eda-commit-executor/prompt.md

@@ -34,7 +34,7 @@
 2. Прочитай последние 5–10 коммитов и определи наблюдаемый стиль сообщений.
 3. Получи текущую ветку, upstream, remotes и основную ветку из `origin/HEAD`.
 4. Прочитай `AGENTS.md`, `CLAUDE.md`, `docs/rules.md`, `docs/arch.md` и связанные артефакты из `docs/**`, если они есть и применимы к изменениям.
-5. Проверяй существующий PR текущей ветки только когда `USER_REQUEST` или `REQUESTED_ACTIONS` упоминает PR.
+5. Проверяй существующий PR текущей ветки, когда `USER_REQUEST` или `REQUESTED_ACTIONS` упоминает PR или push.
 6. Определи один логический набор файлов, соответствующий запросу. Отдели несвязанные пользовательские изменения и не включай их.
 7. Проверь выбранные изменения на вероятные секреты, токены, `.env` и локальные артефакты.
 
@@ -57,6 +57,9 @@
 - Для push используй существующий upstream, иначе `git push -u origin <branch>`.
 - Для PR определи base по `origin/HEAD`, сначала проверь текущую ветку через `gh pr view`; существующий PR не дублируй.
 - Если PR отсутствует, создай его через `gh pr create`, затем получи фактические номер и URL через `gh`; не угадывай ссылку.
+- После любого выполненного push всегда возвращай ссылку. Если PR текущей ветки есть — `pr.status: existing` и его фактический URL. Если PR нет — `pr.status: absent` и `pr.create_url`: ссылка на создание PR, собранная из фактического URL репозитория (`gh repo view`), base из `origin/HEAD` и имени ветки.
+- Если PR нет, а его создание не запрошено, сам PR не создавай: верни `completed`, `pr.create_url`, вопрос «Создать PR?» и варианты «создать PR» / «оставить ссылку на создание».
+- Если репозиторий не на GitHub или `gh` недоступен, верни `pr.create_url: null` и назови причину в `error`.
 - Force-push всегда требует отдельного подтверждения. Force-push в `main` или `master` запрещён.
 - Если continuation не задан, ничего после коммита не выполняй и верни применимые варианты для вопроса пользователя.
 
@@ -79,9 +82,10 @@ push:
   remote: <remote | null>
   branch: <branch | null>
 pr:
-  status: skipped | existing | created | failed
+  status: skipped | existing | created | absent | failed
   number: <number | null>
   url: <url | null>
+  create_url: <url | null>
 merge:
   status: skipped | completed | failed
 current_branch: <branch>
@@ -98,6 +102,6 @@ options: []
 
 - `empty` — в рабочем дереве нет изменений для коммита;
 - `committed` — коммит создан, но в `REQUESTED_ACTIONS` не было продолжения; верни применимые варианты;
-- `completed` — вся явно запрошенная цепочка выполнена либо пользователь после коммита выбрал «ничего больше»;
+- `completed` — вся явно запрошенная цепочка выполнена либо пользователь после коммита выбрал «ничего больше»; при `pr.status: absent` после push здесь остаётся вопрос про создание PR;
 - `partial` — часть разрешённой цепочки выполнена, следующая операция завершилась ошибкой без нового вопроса;
 - `blocked` — для безопасного продолжения нужен один ответ пользователя.

package/lib/install.js

@@ -64,6 +64,12 @@ const RETIRED_AGENTS = [
   'eda-plan-review-performance',
   'eda-plan-review-previous'
 ];
+const RETIRED_SETTINGS_PATHS = [
+  'plan-review.agents',
+  'review-check',
+  'review.strict',
+  'review.include_code_quality'
+];
 const MANIFEST_FILE = 'eda-manifest.json';
 const SETTINGS_VERSION = 3;
 const REVIEW_AGENT_DEFINITIONS = [
@@ -536,11 +542,13 @@ export async function updateAll({
       if (resolvedSettingsMode === 'skip' && await fileExists(projectSettingsPath)) {
         const existingContent = await fs.readFile(projectSettingsPath, 'utf8');
         const migrated = migrateSettingsContent(existingContent);
-        if (migrated.version === SETTINGS_VERSION && migrated.missingKeys.size === 0) {
+        if (migrated.version === SETTINGS_VERSION && migrated.missingKeys.size === 0 && migrated.retiredPaths.length === 0) {
           output.write(`Полный файл настроек сохранён: ${SETTINGS_RELATIVE_PATH}\n`);
         } else {
           await writeSettingsContent(projectDir, formatSettings(migrated.settings));
-          output.write(`Старый или неполный файл настроек перенесён на version: ${SETTINGS_VERSION}: ${SETTINGS_RELATIVE_PATH}\n`);
+          output.write(migrated.retiredPaths.length > 0
+            ? `Из файла настроек убраны устаревшие ключи (${migrated.retiredPaths.join(', ')}): ${SETTINGS_RELATIVE_PATH}\n`
+            : `Старый или неполный файл настроек перенесён на version: ${SETTINGS_VERSION}: ${SETTINGS_RELATIVE_PATH}\n`);
         }
       } else {
         await writeSettingsContent(projectDir, sharedSettingsContent);
@@ -1000,12 +1008,18 @@ async function ensureSettings(cwd, { input = process.stdin, output = process.std
   }
 
   const migrated = migrateSettingsContent(existingContent);
-  if (migrated.version === SETTINGS_VERSION && migrated.missingKeys.size === 0) {
+  const needsMigration = migrated.version !== SETTINGS_VERSION || migrated.missingKeys.size > 0;
+  if (!needsMigration && migrated.retiredPaths.length === 0) {
     output.write(`Настройки уже есть: ${SETTINGS_RELATIVE_PATH}\n`);
     return;
   }
 
-  output.write(`Переношу ${SETTINGS_RELATIVE_PATH} на version: ${SETTINGS_VERSION}; спрашиваются только отсутствующие значения.\n`);
+  if (migrated.retiredPaths.length > 0) {
+    output.write(`Убираю устаревшие настройки: ${migrated.retiredPaths.join(', ')}.\n`);
+  }
+  if (needsMigration) {
+    output.write(`Переношу ${SETTINGS_RELATIVE_PATH} на version: ${SETTINGS_VERSION}; спрашиваются только отсутствующие значения.\n`);
+  }
   const settings = await askSettings({
     input,
     output,
@@ -1028,12 +1042,18 @@ async function configureSettings(cwd, { input = process.stdin, output = process.
   }
 
   const migrated = migrateSettingsContent(existingContent);
-  if (migrated.version === SETTINGS_VERSION && migrated.missingKeys.size === 0) {
+  const needsMigration = migrated.version !== SETTINGS_VERSION || migrated.missingKeys.size > 0;
+  if (!needsMigration && migrated.retiredPaths.length === 0) {
     output.write(`Настройки ${SETTINGS_RELATIVE_PATH} версии ${SETTINGS_VERSION} полные — вопросы не требуются.\n`);
     return;
   }
 
-  output.write(`Переношу ${SETTINGS_RELATIVE_PATH} на version: ${SETTINGS_VERSION}; спрашиваются только отсутствующие значения.\n`);
+  if (migrated.retiredPaths.length > 0) {
+    output.write(`Убираю устаревшие настройки: ${migrated.retiredPaths.join(', ')}.\n`);
+  }
+  if (needsMigration) {
+    output.write(`Переношу ${SETTINGS_RELATIVE_PATH} на version: ${SETTINGS_VERSION}; спрашиваются только отсутствующие значения.\n`);
+  }
   const settings = await askSettings({
     input,
     output,
@@ -1159,6 +1179,7 @@ function migrateSettingsContent(content) {
   return {
     settings,
     missingKeys,
+    retiredPaths: findRetiredSettingsPaths(raw),
     version: Number.isInteger(raw.version) ? raw.version : null
   };
 }
@@ -1179,6 +1200,12 @@ function transferAgentSettings({ rawAgents, settingsAgents, definitions, prefix,
   }
 }
 
+// У eda-polish больше нет порога: цикл завершается по нулю находок.
+function stripRetiredStepArgs(skill, args) {
+  if (skill !== 'eda-polish') return args;
+  return args.replace(/\bthreshold\s+\d+\s*/gi, '').replace(/\s+/g, ' ').trim();
+}
+
 function normalizeStoredOrhestraSteps(value) {
   if (!Array.isArray(value) || value.length === 0) return null;
   const steps = [];
@@ -1192,7 +1219,7 @@ function normalizeStoredOrhestraSteps(value) {
       id: step.id,
       skill: step.skill,
       enabled: step.enabled !== false,
-      args: step.args ?? ''
+      args: stripRetiredStepArgs(step.skill, step.args ?? '')
     };
     const onFailure = objectValue(step.on_failure);
     if (Object.keys(onFailure).length > 0) {
@@ -1216,6 +1243,23 @@ function objectValue(value) {
   return value && typeof value === 'object' && !Array.isArray(value) ? value : {};
 }
 
+function findRetiredSettingsPaths(raw) {
+  const found = [];
+  for (const settingPath of RETIRED_SETTINGS_PATHS) {
+    let current = raw;
+    let present = true;
+    for (const segment of settingPath.split('.')) {
+      if (!current || typeof current !== 'object' || Array.isArray(current) || !(segment in current)) {
+        present = false;
+        break;
+      }
+      current = current[segment];
+    }
+    if (present && current !== undefined) found.push(settingPath);
+  }
+  return found;
+}
+
 async function writeFileAtomically(filePath, content) {
   const currentStat = await fs.stat(filePath);
   const tempPath = path.join(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gian-tiaga/eda",
-  "version": "3.1.0",
+  "version": "3.2.0",
   "description": "Пакет скилов и специализированных агентов eda-* для Claude Code и Codex CLI",
   "type": "module",
   "bin": {

package/skills/eda-commit/SKILL.md

@@ -66,14 +66,13 @@ REQUESTED_ACTIONS:
 - `empty` — сообщи, что коммитить нечего, и закончи;
 - `blocked` — задай только конкретный вопрос агента и продолжи его после ответа;
 - `committed` — если продолжение не было задано, задай вопрос из следующего этапа;
-- `completed` или `partial` — переходи к финалу.
+- `completed` — если push выполнен, а PR нет, задай вопрос из следующего этапа, иначе переходи к финалу;
+- `partial` — переходи к финалу.
 
 При продолжении передай тому же агенту исходный `$USER_REQUEST`, актуальные `$REQUESTED_ACTIONS`, полный предыдущий результат и новый ответ пользователя. Не теряй commit hash и уже выполненные действия.
 
 ### 2. Решить, нужен ли вопрос после коммита
 
-Если `$REQUESTED_ACTIONS` не пуст и агент выполнил всю цепочку, сразу переходи к финалу.
-
 Если `$REQUESTED_ACTIONS` пуст и статус `committed`, спроси пользователя, используя только применимые варианты, которые вернул executor:
 
 - push;
@@ -81,6 +80,10 @@ REQUESTED_ACTIONS:
 - merge в основную ветку, переключение и удаление текущей ветки;
 - ничего больше.
 
+Если push выполнен, а PR текущей ветки нет (`pr.status: absent`), спроси один раз, создавать ли PR: «создать PR» или «оставить ссылку на создание». Не спрашивай, если PR уже есть или пользователь его запретил.
+
+Если других вопросов нет и вся запрошенная цепочка выполнена, сразу переходи к финалу.
+
 Передай выбор тому же `eda-commit-executor` вместе с полным предыдущим результатом. Основной агент команды не выполняет.
 
 ### 3. Финал
@@ -93,6 +96,8 @@ REQUESTED_ACTIONS:
 - текущую ветку;
 - оставшиеся незакоммиченные файлы.
 
+Если push выполнен, ссылка обязательна всегда: URL существующего или созданного PR, иначе `pr.create_url` — ссылку на создание PR. Если executor вернул обе ссылки пустыми, прямо скажи, что ссылку получить не удалось, и назови причину.
+
 Для `partial` сначала назови выполненные действия, затем ошибку и следующий безопасный шаг.
 
 ## Чего НЕ делать
@@ -102,5 +107,6 @@ REQUESTED_ACTIONS:
 - Пересказывать предыдущий результат вместо передачи полного YAML при продолжении.
 - Терять начало `$USER_REQUEST` из-за последующего вопроса или ответа.
 - Спрашивать про push или PR, если пользователь уже явно приказал выполнить их.
+- Заканчивать выполненный push без ссылки на PR или на его создание.
 - Отключать hooks, использовать `--amend`, делать force-push в `main`/`master` или создавать дубликат PR.
 - Делать ревью или править логику кода — это другие скилы.