@gian-tiaga/eda 1.0.0 → 1.0.2

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.2`, `eda update` заново спрашивает полный профиль настроек и записывает актуальный `docs/settings.yaml` версии 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` ищет проекты на глубине до двух уровней, один раз спрашивает полный профиль настроек и записывает одинаковый `docs/settings.yaml` версии 2 во все найденные проекты. В non-TTY запуске вопросы не задаются: во все проекты записывается один профиль defaults.
 
 После обновления перезапусти 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 update` ответь на полный набор вопросов: команда соберёт новый профиль и атомарно заменит прежний файл независимо от его версии. `eda update-all` задаст эти вопросы только один раз и применит ответы ко всем проектам.
+
+`eda init` ведёт себя осторожнее: если конфиг уже существует, v2 и неизвестные версии сохраняются, а v1 автоматически мигрируется с переносом поддерживаемых значений. При update-командах прежний файл заменяется выбранным полным профилем, поэтому перед запуском сохрани нестандартные поля отдельно.
 
 Основное соответствие полей:
 
@@ -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` заполняются ответами на вопросы; в non-TTY запуске используются сбалансированные 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` во все найденные проекты. В неинтерактивном запуске во все проекты записывается один профиль по умолчанию.
 
 ---
 
@@ -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` создаёт `docs/settings.yaml`, только если файла ещё нет; существующий v1 автоматически мигрируется в v2, а v2 и неизвестные версии сохраняются. `eda update` всегда заново задаёт все вопросы конфигурации и атомарно перезаписывает файл полным v2-профилем. `eda update-all` задаёт тот же набор вопросов один раз и записывает получившийся общий конфиг во все найденные проекты. Без TTY обе update-команды используют defaults.
 
 ---
 
@@ -422,8 +422,8 @@ discover-automations может запускаться от review, fix-by-revie
 
 ```bash
 eda init      # выбрать Claude Code / Codex / обе среды и установить пакет
-eda update    # обновить скилы и агенты, создать docs/settings.yaml, если его ещё нет
-eda update-all [dir] # обновить пакет во всех найденных проектах внутри dir на глубине 2
+eda update    # заново настроить конфиг и обновить скилы и агенты
+eda update-all [dir] # один раз настроить конфиг и обновить все найденные проекты на глубине 2
 eda --version # показать версию установленного пакета
 eda --help    # справка
 ```
@@ -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` заново собирает полный v2-конфиг, затем идемпотентно синхронизирует обе группы компонентов и отдельно показывает реально изменившиеся скилы и агенты. Manifest хранит только идентификаторы компонентов, которыми владеет `eda`: благодаря этому retired-компоненты удаляются, а чужие скилы и агенты остаются нетронутыми. `eda update-all [dir]` один раз собирает общий конфиг, записывает его во все найденные проекты и синхронизирует компоненты каждого проекта. Старые Codex-файлы `.codex/skills/<skill>.md` также удаляются при миграции.
 
 Установленные копии не являются источником правды. Локальные изменения внутри `.claude/skills/eda-*`, `.codex/skills/eda-*`, `.claude/agents/eda-*` или `.codex/agents/eda-*` будут заменены следующим обновлением.
 

package/bin/cli.js

@@ -10,8 +10,8 @@ const HELP = `eda — установка и обновление скилов и
 
 Использование:
   eda init     — выбрать целевые среды (Claude / Codex / обе) и установить пакет
-  eda update   — обновить уже установленные скилы и агенты в текущем проекте
-  eda update-all [dir] — обновить пакет во всех найденных проектах внутри dir
+  eda update   — заново настроить конфиг и обновить текущий проект
+  eda update-all [dir] — один раз настроить конфиг и обновить все проекты внутри dir
   eda --version — показать версию
   eda --help   — показать эту справку
 `;

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';
@@ -370,12 +371,13 @@ export async function update({ cwd, input = process.stdin, output = process.stdo
     return;
   }
   output.write(`Найдены установленные среды: ${targets.join(', ')}\n`);
-  await ensureSettings(cwd, { input, output });
+  await configureSettings(cwd, { input, output });
   await syncPackage(cwd, targets, output, { action: 'update' });
 }
 
 export async function updateAll({
   root = process.cwd(),
+  input = process.stdin,
   output = process.stdout,
   maxDepth = UPDATE_ALL_MAX_DEPTH
 } = {}) {
@@ -405,6 +407,9 @@ export async function updateAll({
   }
 
   output.write(`Найдено ${formatProjectCount(projects.length)}: ${projects.map(project => formatProjectPath(rootDir, project)).join(', ')}\n`);
+  output.write('Настройки будут запрошены один раз и записаны одинаково во все найденные проекты.\n');
+  const sharedSettings = await askSettings({ input, output });
+  const sharedSettingsContent = formatSettings(sharedSettings);
 
   const updatedProjects = [];
   const skippedProjects = [];
@@ -423,6 +428,8 @@ export async function updateAll({
       }
 
       output.write(`Найдены установленные среды: ${targets.join(', ')}\n`);
+      await writeSettingsContent(projectDir, sharedSettingsContent);
+      output.write(`Записан общий файл настроек: ${SETTINGS_RELATIVE_PATH}\n`);
       const result = await syncPackage(projectDir, targets, output, {
         action: 'update',
         writeDone: false
@@ -479,7 +486,7 @@ export async function askTargets({ input = process.stdin, output = process.stdou
 
 export async function askSettings({ input = process.stdin, output = process.stdout } = {}) {
   if (!input.isTTY || !output.isTTY) {
-    output.write(`Нет интерактивного терминала — создаю ${SETTINGS_RELATIVE_PATH} с настройками по умолчанию.\n`);
+    output.write(`Нет интерактивного терминала — использую настройки ${SETTINGS_RELATIVE_PATH} по умолчанию.\n`);
     return structuredClone(DEFAULT_SETTINGS);
   }
 
@@ -784,18 +791,183 @@ async function findInstalledProjects(rootDir, maxDepth) {
 }
 
 async function ensureSettings(cwd, { input = process.stdin, output = process.stdout } = {}) {
+  const existing = await migrateSettingsIfNeeded(cwd, { output });
+  if (existing.exists) return;
+
+  const settings = await askSettings({ input, output });
+  await writeSettingsContent(cwd, formatSettings(settings));
+  output.write(`Создан файл настроек: ${SETTINGS_RELATIVE_PATH}\n`);
+}
+
+async function configureSettings(cwd, { input = process.stdin, output = process.stdout } = {}) {
+  output.write(`Настраиваю ${SETTINGS_RELATIVE_PATH} заново.\n`);
+  const settings = await askSettings({ input, output });
+  await writeSettingsContent(cwd, formatSettings(settings));
+  output.write(`Записан файл настроек: ${SETTINGS_RELATIVE_PATH}\n`);
+}
+
+async function writeSettingsContent(cwd, content) {
   const settingsPath = path.join(cwd, SETTINGS_RELATIVE_PATH);
+  await fs.mkdir(path.dirname(settingsPath), { recursive: true });
   if (await fileExists(settingsPath)) {
-    output.write(`Настройки уже есть: ${SETTINGS_RELATIVE_PATH}\n`);
-    output.write('Существующий файл не перезаписываю. Актуальный формат:\n\n');
-    output.write(formatSettings(DEFAULT_SETTINGS));
+    await writeFileAtomically(settingsPath, content);
     return;
   }
+  await fs.writeFile(settingsPath, content);
+}
 
-  const settings = await askSettings({ input, output });
-  await fs.mkdir(path.dirname(settingsPath), { recursive: true });
-  await fs.writeFile(settingsPath, formatSettings(settings));
-  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) {

package/package.json

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