@gian-tiaga/eda 3.1.0 → 3.1.1

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/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.1.1",
   "description": "Пакет скилов и специализированных агентов eda-* для Claude Code и Codex CLI",
   "type": "module",
   "bin": {