@gian-tiaga/eda 1.0.0 → 1.0.1

package/MIGRATION.md

@@ -1,6 +1,6 @@
 # Миграция на eda 1.0
 
-Версия 1.0 переводит `eda` с набора файлов-скилов на пакет с каноническими скилами и специализированными агентами для Claude Code и Codex CLI. Установщик обновляет управляемые компоненты автоматически, но существующий `docs/settings.yaml` версии 1 нужно перенести вручную.
+Версия 1.0 переводит `eda` с набора файлов-скилов на пакет с каноническими скилами и специализированными агентами для Claude Code и Codex CLI. Начиная с `1.0.1`, установщик обновляет управляемые компоненты и автоматически переводит существующий `docs/settings.yaml` версии 1 в актуальный формат версии 2.
 
 ## Короткий путь
 
@@ -11,7 +11,7 @@ npm install -g @gian-tiaga/eda@1
 eda --version
 ```
 
-Команда должна вывести `1.0.0`.
+Команда должна вывести актуальную версию ветки `1.0.x`.
 
 В одном проекте выполни:
 
@@ -26,7 +26,7 @@ eda update
 eda update-all /path/to/projects
 ```
 
-`update-all` ищет проекты на глубине до двух уровней, обновляет только уже найденные среды Claude/Codex и не создаёт и не меняет `docs/settings.yaml`.
+`update-all` ищет проекты на глубине до двух уровней, обновляет только уже найденные среды Claude/Codex и не создаёт отсутствующий `docs/settings.yaml`. Если конфиг существует с `version: 1`, команда автоматически мигрирует его на v2.
 
 После обновления перезапусти Claude Code или Codex CLI, чтобы среда перечитала установленные скилы и агентов.
 
@@ -57,7 +57,9 @@ eda update-all /path/to/projects
 
 ## Миграция `docs/settings.yaml`
 
-Версия 1.0 поддерживает только формат настроек `version: 2`. Установщик намеренно сохраняет существующий файл без изменений, поэтому файл с `version: 1` нужно отредактировать вручную.
+Версия 1.0 поддерживает только формат настроек `version: 2`. Команды `eda init`, `eda update` и `eda update-all` распознают `version: 1`, переносят поддерживаемые значения, добавляют новые настройки с defaults и атомарно заменяют файл. Повторных интерактивных вопросов при миграции нет.
+
+Существующий v2 сохраняется без изменений. Файл без версии или с неизвестной версией установщик не переписывает, чтобы не повредить неподдерживаемый формат. Поля вне документированной схемы v1 в новый файл не переносятся.
 
 Основное соответствие полей:
 
@@ -74,7 +76,7 @@ eda update-all /path/to/projects
 
 У старого `review.strict` нет прямого аналога. В 1.0 каждая проверка в `review.agents` отдельно получает режим `always`, `auto` или `off` и модели для Claude/Codex. Раздел `review-check` больше не используется.
 
-Минимальный перенос старых общих настроек выглядит так:
+Результат переноса старых общих настроек выглядит так:
 
 ```yaml
 version: 2
@@ -97,7 +99,7 @@ discover-automations:
   include_plans: false
 ```
 
-Новые разделы `orhestra`, `aim`, `review.agents` и `send-review` можно взять из полного актуального примера в [README](README.md). Если раздел или значение отсутствует, соответствующий скил использует документированный default.
+Новые разделы `orhestra`, `aim`, `review.agents` и `send-review` установщик добавляет автоматически из сбалансированных defaults. Полный актуальный пример приведён в [README](README.md).
 
 ## Проверка после миграции
 

package/README.md

@@ -29,7 +29,7 @@ eda init
 eda update-all ~/Code
 ```
 
-`eda update-all` ищет проекты в указанной директории, её прямых подпапках и подпапках второго уровня. Проект считается найденным, если в нём уже есть установленный скил, агент или manifest `eda` для Claude/Codex. Команда обновляет только найденные среды и не создаёт `docs/settings.yaml`, чтобы массовый запуск не добавлял новые файлы в проекты.
+`eda update-all` ищет проекты в указанной директории, её прямых подпапках и подпапках второго уровня. Проект считается найденным, если в нём уже есть установленный скил, агент или manifest `eda` для Claude/Codex. Команда обновляет только найденные среды, не создаёт отсутствующий `docs/settings.yaml`, но автоматически переводит найденный конфиг с `version: 1` на `version: 2`.
 
 ---
 
@@ -347,7 +347,7 @@ discover-automations:
 
 Поддерживается только `version: 2`. Для старого v2 без `review.agents` новый review использует сбалансированные defaults и предупреждает, что устаревшие `review.strict`, `review.include_code_quality` и `review-check` игнорируются. Установщик существующий файл не переписывает.
 
-`eda init` и `eda update` создают `docs/settings.yaml`, только если файла ещё нет. Существующий файл, включая v1, не перезаписывается; его нужно перевести вручную. `eda update-all` настройки не создаёт и не меняет.
+`eda init` и `eda update` создают `docs/settings.yaml`, только если файла ещё нет. Существующий v2 сохраняется без изменений, а v1 автоматически и атомарно переписывается в v2 с переносом поддерживаемых значений. Файл неизвестной версии установщик не меняет. `eda update-all` не создаёт отсутствующие настройки, но также мигрирует найденный v1.
 
 ---
 
@@ -456,7 +456,7 @@ eda update-all ~/Code
 | Агент | `.claude/agents/<agent>.md` | `.codex/agents/<agent>.toml` |
 | Manifest владения | `.claude/eda-manifest.json` | `.codex/eda-manifest.json` |
 
-`eda update` идемпотентно синхронизирует обе группы компонентов и отдельно показывает реально изменившиеся скилы и агенты. Manifest хранит только идентификаторы компонентов, которыми владеет `eda`: благодаря этому retired-компоненты удаляются, а чужие скилы и агенты остаются нетронутыми. `eda update-all [dir]` делает то же самое для каждого найденного проекта, но не создаёт `docs/settings.yaml`. Старые Codex-файлы `.codex/skills/<skill>.md` также удаляются при миграции.
+`eda update` идемпотентно синхронизирует обе группы компонентов и отдельно показывает реально изменившиеся скилы и агенты. Manifest хранит только идентификаторы компонентов, которыми владеет `eda`: благодаря этому retired-компоненты удаляются, а чужие скилы и агенты остаются нетронутыми. `eda update-all [dir]` делает то же самое для каждого найденного проекта, не создаёт отсутствующий `docs/settings.yaml`, но переводит найденный v1 в v2. Старые Codex-файлы `.codex/skills/<skill>.md` также удаляются при миграции.
 
 Установленные копии не являются источником правды. Локальные изменения внутри `.claude/skills/eda-*`, `.codex/skills/eda-*`, `.claude/agents/eda-*` или `.codex/agents/eda-*` будут заменены следующим обновлением.
 

package/lib/install.js

@@ -1,5 +1,6 @@
 import checkbox from '@inquirer/checkbox';
 import select from '@inquirer/select';
+import { randomUUID } from 'node:crypto';
 import fs from 'node:fs/promises';
 import path from 'node:path';
 import { fileURLToPath } from 'node:url';
@@ -423,6 +424,7 @@ export async function updateAll({
       }
 
       output.write(`Найдены установленные среды: ${targets.join(', ')}\n`);
+      await migrateSettingsIfNeeded(projectDir, { output, reportExisting: false });
       const result = await syncPackage(projectDir, targets, output, {
         action: 'update',
         writeDone: false
@@ -785,12 +787,8 @@ async function findInstalledProjects(rootDir, maxDepth) {
 
 async function ensureSettings(cwd, { input = process.stdin, output = process.stdout } = {}) {
   const settingsPath = path.join(cwd, SETTINGS_RELATIVE_PATH);
-  if (await fileExists(settingsPath)) {
-    output.write(`Настройки уже есть: ${SETTINGS_RELATIVE_PATH}\n`);
-    output.write('Существующий файл не перезаписываю. Актуальный формат:\n\n');
-    output.write(formatSettings(DEFAULT_SETTINGS));
-    return;
-  }
+  const existing = await migrateSettingsIfNeeded(cwd, { output });
+  if (existing.exists) return;
 
   const settings = await askSettings({ input, output });
   await fs.mkdir(path.dirname(settingsPath), { recursive: true });
@@ -798,6 +796,160 @@ async function ensureSettings(cwd, { input = process.stdin, output = process.std
   output.write(`Создан файл настроек: ${SETTINGS_RELATIVE_PATH}\n`);
 }
 
+async function migrateSettingsIfNeeded(cwd, {
+  output = process.stdout,
+  reportExisting = true
+} = {}) {
+  const settingsPath = path.join(cwd, SETTINGS_RELATIVE_PATH);
+  const content = await readFileIfExists(settingsPath);
+  if (content === null) return { exists: false, migrated: false };
+
+  const version = readTopLevelYamlScalar(content, 'version');
+  if (version !== '1') {
+    if (reportExisting) {
+      output.write(`Настройки уже есть: ${SETTINGS_RELATIVE_PATH}\n`);
+      if (version !== '2') {
+        output.write('Автомиграция не выполнена: безопасно поддерживается только переход version: 1 → version: 2.\n');
+      }
+    }
+    return { exists: true, migrated: false, version };
+  }
+
+  const migratedSettings = migrateSettingsV1(content);
+  await writeFileAtomically(settingsPath, formatSettings(migratedSettings));
+  output.write(`Мигрированы настройки ${SETTINGS_RELATIVE_PATH}: version: 1 → version: 2.\n`);
+  return { exists: true, migrated: true, version: '2' };
+}
+
+function migrateSettingsV1(content) {
+  const defaultsStrict = readLegacyBoolean(content, 'defaults', 'strict', false);
+  const decisionMode = readLegacyEnum(
+    content,
+    'defaults',
+    'decision_mode',
+    ['autonomous', 'recommend_and_ask', 'ask_each_time'],
+    DEFAULT_SETTINGS.explore.decisionMode
+  );
+  const reviewAgents = structuredClone(DEFAULT_REVIEW_AGENTS);
+  reviewAgents.code_quality.mode = readLegacyBoolean(
+    content,
+    'review',
+    'include_code_quality',
+    true
+  ) ? 'always' : 'off';
+
+  return {
+    ...structuredClone(DEFAULT_SETTINGS),
+    explore: {
+      strict: defaultsStrict,
+      decisionMode
+    },
+    plan: {
+      strict: defaultsStrict,
+      size: readLegacyEnum(
+        content,
+        'defaults',
+        'plan_size',
+        ['normal', 'short', 'ask_each_time'],
+        DEFAULT_SETTINGS.plan.size
+      ),
+      decisionMode,
+      testStrategy: readLegacyEnum(
+        content,
+        'defaults',
+        'test_strategy',
+        ['after_each_phase', 'tdd_each_phase', 'end_of_plan', 'ask_each_time'],
+        DEFAULT_SETTINGS.plan.testStrategy
+      ),
+      loggingStrategy: readLegacyEnum(
+        content,
+        'defaults',
+        'logging_strategy',
+        ['debug_precise', 'standard', 'ask_each_time'],
+        DEFAULT_SETTINGS.plan.loggingStrategy
+      )
+    },
+    planPolish: {
+      strict: defaultsStrict
+    },
+    review: {
+      agents: reviewAgents
+    },
+    discoverAutomations: {
+      includePlans: readLegacyBoolean(content, 'automate', 'include_plans', false)
+    }
+  };
+}
+
+function readLegacyBoolean(content, section, key, fallback) {
+  const value = readNestedYamlScalar(content, section, key);
+  if (value === 'true') return true;
+  if (value === 'false') return false;
+  return fallback;
+}
+
+function readLegacyEnum(content, section, key, allowedValues, fallback) {
+  const value = readNestedYamlScalar(content, section, key);
+  return allowedValues.includes(value) ? value : fallback;
+}
+
+function readTopLevelYamlScalar(content, key) {
+  for (const line of content.replace(/^\uFEFF/, '').split(/\r?\n/)) {
+    const match = line.match(/^([A-Za-z0-9_-]+):\s*(.*?)\s*$/);
+    if (match?.[1] === key) return normalizeYamlScalar(match[2]);
+  }
+  return null;
+}
+
+function readNestedYamlScalar(content, section, key) {
+  let currentSection = null;
+  for (const line of content.replace(/^\uFEFF/, '').split(/\r?\n/)) {
+    if (/^\s*(?:#.*)?$/.test(line)) continue;
+
+    const sectionMatch = line.match(/^([A-Za-z0-9_-]+):\s*(?:#.*)?$/);
+    if (sectionMatch) {
+      currentSection = sectionMatch[1];
+      continue;
+    }
+
+    if (/^\S/.test(line)) {
+      currentSection = null;
+      continue;
+    }
+
+    if (currentSection !== section) continue;
+    const valueMatch = line.match(/^ {2}([A-Za-z0-9_-]+):\s*(.*?)\s*$/);
+    if (valueMatch?.[1] === key) return normalizeYamlScalar(valueMatch[2]);
+  }
+  return null;
+}
+
+function normalizeYamlScalar(value) {
+  const withoutComment = value.replace(/\s+#.*$/, '').trim();
+  if (
+    withoutComment.length >= 2
+    && ((withoutComment.startsWith('"') && withoutComment.endsWith('"'))
+      || (withoutComment.startsWith("'") && withoutComment.endsWith("'")))
+  ) {
+    return withoutComment.slice(1, -1);
+  }
+  return withoutComment;
+}
+
+async function writeFileAtomically(filePath, content) {
+  const currentStat = await fs.stat(filePath);
+  const tempPath = path.join(
+    path.dirname(filePath),
+    `.${path.basename(filePath)}.${process.pid}.${randomUUID()}.tmp`
+  );
+  try {
+    await fs.writeFile(tempPath, content, { mode: currentStat.mode & 0o777 });
+    await fs.rename(tempPath, filePath);
+  } finally {
+    await fs.rm(tempPath, { force: true });
+  }
+}
+
 function formatSettings(settings) {
   const orhestra = settings.orhestra ?? DEFAULT_SETTINGS.orhestra;
   const aim = settings.aim ?? DEFAULT_SETTINGS.aim;

package/package.json

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