@gian-tiaga/eda 3.14.0 → 4.0.0

package/lib/install.js

@@ -1,12 +1,11 @@
-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';
 import { parse as parseYaml } from 'yaml';
 
-import { listAgents, listSkills } from './catalog.js';
+import { expandIncludes, listAgents, listSkills } from './catalog.js';
+import { checkboxPrompt as checkbox, finishPrompts, selectPrompt as select, startPrompts } from './prompts.js';
 import { renderClaudeAgent } from './renderers/claude-agent.js';
 import { renderClaudeSkill } from './renderers/claude-skill.js';
 import { renderCodexAgent } from './renderers/codex-agent.js';
@@ -42,6 +41,8 @@ const TARGET_CHOICES = [
   { value: 'codex', label: 'Codex CLI', dir: '.agents/skills/' },
   { value: 'opencode', label: 'OpenCode', dir: '.opencode/skills/' }
 ];
+// В вопросах платформы показываются в этом порядке; установка идёт в порядке TARGET_CHOICES.
+const TARGET_PROMPT_ORDER = ['claude', 'opencode', 'codex'];
 const UPDATE_ALL_MAX_DEPTH = 2;
 const UPDATE_ALL_SKIP_DIRS = new Set(['.git', '.agents', '.claude', '.codex', '.opencode', 'node_modules']);
 const RETIRED_SKILLS = [
@@ -68,6 +69,8 @@ const RETIRED_AGENTS = [
   'eda-plan-review-previous'
 ];
 const RETIRED_SETTINGS_PATHS = [
+  'plan.review',
+  'plan.meta_review',
   'plan-review.agents',
   'review-check',
   'review.strict',
@@ -75,138 +78,30 @@ const RETIRED_SETTINGS_PATHS = [
 ];
 const MANIFEST_FILE = 'eda-manifest.json';
 const SETTINGS_VERSION = 3;
-const REVIEW_AGENT_DEFINITIONS = [
-  {
-    key: 'correctness',
-    description: 'Проверяет ошибки в логике, крайние случаи и обработку ошибок.',
-    auto: 'Эта базовая проверка по умолчанию запускается всегда.',
-    mode: 'always',
-    claude: 'sonnet',
-    codex: 'gpt-5.6-terra'
-  },
-  {
-    key: 'architecture',
-    description: 'Проверяет архитектурные границы, зависимости и разделение ответственности.',
-    auto: 'В auto запускается при изменении модулей, зависимостей, публичных контрактов или границ системы.',
-    mode: 'auto',
-    claude: 'opus',
-    codex: 'gpt-5.6-sol'
-  },
-  {
-    key: 'rules',
-    description: 'Проверяет соблюдение AGENTS.md, CLAUDE.md и docs/rules.md.',
-    auto: 'Эта базовая проверка по умолчанию запускается всегда.',
-    mode: 'always',
-    claude: 'haiku',
-    codex: 'gpt-5.6-luna'
-  },
-  {
-    key: 'references',
-    description: 'Проверяет соответствие применимым карточкам из docs/references/.',
-    auto: 'В auto запускается только когда для изменений выбраны применимые карточки.',
-    mode: 'auto',
-    claude: 'haiku',
-    codex: 'gpt-5.6-luna'
-  },
-  {
-    key: 'business',
-    description: 'Проверяет изменённое поведение по применимым карточкам из docs/business/.',
-    auto: 'В auto запускается только когда изменения затрагивают поведение с применимой business-карточкой.',
-    mode: 'auto',
-    claude: 'sonnet',
-    codex: 'gpt-5.6-terra'
-  },
-  {
-    key: 'plan_alignment',
-    description: 'Проверяет, что реализация соответствует указанному или однозначно найденному плану.',
-    auto: 'В auto запускается только когда у ревью есть конкретный план.',
-    mode: 'auto',
-    claude: 'sonnet',
-    codex: 'gpt-5.6-terra'
-  },
-  {
-    key: 'code_quality',
-    description: 'Проверяет читаемость, сложность, дублирование и поддерживаемость кода.',
-    auto: 'Эта базовая проверка по умолчанию запускается всегда.',
-    mode: 'always',
-    claude: 'sonnet',
-    codex: 'gpt-5.6-terra'
-  },
-  {
-    key: 'tests',
-    description: 'Проверяет достаточность, корректность и устойчивость тестов.',
-    auto: 'Эта базовая проверка по умолчанию запускается всегда.',
-    mode: 'always',
-    claude: 'sonnet',
-    codex: 'gpt-5.6-terra'
-  },
-  {
-    key: 'security',
-    description: 'Проверяет уязвимости, права доступа, валидацию и работу с чувствительными данными.',
-    auto: 'В auto запускается для auth, permissions, внешнего ввода, файлов, сети, секретов и криптографии.',
-    mode: 'auto',
-    claude: 'opus',
-    codex: 'gpt-5.6-sol'
-  },
-  {
-    key: 'performance',
-    description: 'Проверяет запросы, циклы, память, конкурентность и другие риски производительности.',
-    auto: 'В auto запускается при изменении запросов, пакетной обработки, кешей, конкурентности или горячих путей.',
-    mode: 'auto',
-    claude: 'sonnet',
-    codex: 'gpt-5.6-terra'
-  },
-  {
-    key: 'frontend',
-    description: 'Проверяет фронтенд-код, UI, UX, адаптивность и состояния интерфейса.',
-    auto: 'В auto обязательно запускается при изменении компонентов, стилей, клиентских маршрутов или состояния UI.',
-    mode: 'auto',
-    claude: 'sonnet',
-    codex: 'gpt-5.6-terra'
-  },
-  {
-    key: 'api',
-    description: 'Проверяет API-контракты, совместимость, валидацию и обработку ошибок.',
-    auto: 'В auto запускается при изменении маршрутов, handlers, DTO, схем API, GraphQL или RPC.',
-    mode: 'auto',
-    claude: 'sonnet',
-    codex: 'gpt-5.6-terra'
-  },
-  {
-    key: 'database',
-    description: 'Проверяет схемы, миграции, запросы, индексы и целостность данных.',
-    auto: 'В auto запускается при изменении миграций, моделей хранения, SQL, схем или транзакций.',
-    mode: 'auto',
-    claude: 'opus',
-    codex: 'gpt-5.6-sol'
-  },
-  {
-    key: 'documentation',
-    description: 'Проверяет, обновлена ли документация для изменившегося пользовательского поведения.',
-    auto: 'В auto запускается при изменении публичного API, CLI, настроек или документируемого поведения.',
-    mode: 'auto',
-    claude: 'haiku',
-    codex: 'gpt-5.6-luna'
-  },
-  {
-    key: 'previous_reviews',
-    description: 'Перепроверяет незакрытые замечания предыдущих ревью для удалённого PR или MR.',
-    auto: 'В auto запускается только для ссылки на PR/MR или другой цели с доступными удалёнными обсуждениями.',
-    mode: 'auto',
-    claude: 'haiku',
-    codex: 'gpt-5.6-luna'
-  }
+// Состав проверок выбирает eda-review-triage по содержимому ревью. В настройках у роли одна строка:
+// default — модель установленного агента — или provider/model#variant, где provider и variant необязательны.
+const REVIEW_AGENT_KEYS = [
+  'triage',
+  'correctness',
+  'architecture',
+  'rules',
+  'references',
+  'business',
+  'plan_alignment',
+  'code_quality',
+  'tests',
+  'security',
+  'performance',
+  'frontend',
+  'api',
+  'database',
+  'infra',
+  'seo',
+  'documentation',
+  'previous_reviews'
 ];
-const DEFAULT_REVIEW_AGENTS = Object.fromEntries(REVIEW_AGENT_DEFINITIONS.map(agent => [
-  agent.key,
-  {
-    mode: agent.mode,
-    model: {
-      claude: agent.claude,
-      codex: agent.codex
-    }
-  }
-]));
+const DEFAULT_REVIEW_AGENTS = Object.fromEntries(REVIEW_AGENT_KEYS.map(key => [key, 'default']));
+const REVIEW_AGENT_MODEL_PATTERN = /^[\w.:-]+(?:\/[\w.:-]+)*(?:#[\w.-]+)?$/;
 const ORHESTRA_STEP_DEFINITIONS = [
   {
     id: 'plan',
@@ -266,20 +161,19 @@ const DEFAULT_SETTINGS = {
   },
   plan: {
     strict: false,
-    review: true,
     size: 'normal',
     decisionMode: 'recommend_and_ask',
-    testStrategy: 'ask_each_time',
-    loggingStrategy: 'ask_each_time'
+    testStrategy: 'phase_tests_final_checks',
+    loggingStrategy: 'standard'
   },
   planReview: {
     threshold: 100
   },
   planPolish: {
-    limit: 1
+    limit: 5
   },
   planExecute: {
-    mode: 'auto'
+    mode: 'main'
   },
   polish: {
     execution: 'subagents'
@@ -288,249 +182,83 @@ const DEFAULT_SETTINGS = {
     depth: 'full'
   },
   review: {
-    execution: 'subagents',
+    execution: 'main',
     agents: DEFAULT_REVIEW_AGENTS
   },
   sendReview: {
-    closePreviousReviews: false
+    closePreviousReviews: true
   },
   discoverAutomations: {
     includePlans: false
   }
 };
+// Разделы идут по ходу работы: исследование, план, его проверка, выполнение, ревью,
+// ручные тесты; оркестраторы, которые собирают остальные скилы, — в конце.
 const SETTINGS_SECTION_NAMES = [
-  'orhestra',
-  'aim',
   'explore',
   'plan',
   'plan-review',
   'plan-polish',
   'plan-execute',
+  'review',
   'polish',
   'manual-test',
-  'review',
   'send-review',
-  'discover-automations'
-];
-const SETTINGS_CHOICES = [
-  {
-    section: 'explore',
-    key: 'explore.strict',
-    value: 'exploreStrict',
-    name: 'Strict по умолчанию для eda-explore',
-    checked: DEFAULT_SETTINGS.explore.strict
-  },
-  {
-    section: 'plan',
-    key: 'plan.strict',
-    value: 'planStrict',
-    name: 'Strict по умолчанию для eda-plan',
-    checked: DEFAULT_SETTINGS.plan.strict
-  },
-  {
-    section: 'plan',
-    key: 'plan.review',
-    value: 'planReview',
-    name: 'Запускать eda-plan-review внутри eda-plan',
-    checked: DEFAULT_SETTINGS.plan.review
-  },
-  {
-    section: 'send-review',
-    key: 'sendReview.closePreviousReviews',
-    value: 'sendReviewClosePreviousReviews',
-    name: 'Закрывать предыдущие ревью eda-send-review',
-    checked: DEFAULT_SETTINGS.sendReview.closePreviousReviews
-  },
-  {
-    section: 'discover-automations',
-    key: 'discoverAutomations.includePlans',
-    value: 'discoverAutomationsIncludePlans',
-    name: 'Анализировать планы в eda-discover-automations по умолчанию',
-    checked: DEFAULT_SETTINGS.discoverAutomations.includePlans
-  }
-];
-const ORHESTRA_MODE_CHOICES = [
-  {
-    value: 'automatic',
-    name: 'Автоматически пройти план, выполнение, ревью и ручные тесты'
-  },
-  {
-    value: 'manual',
-    name: 'Писать план вместе с человеком и передавать ему вопросы'
-  }
-];
-const ORHESTRA_EXECUTION_CHOICES = [
-  {
-    value: 'subagents',
-    name: 'Каждый этап выполнять отдельным изолированным субагентом'
-  },
-  {
-    value: 'main',
-    name: 'Все этапы выполнять основным агентом без передачи режима внутрь'
-  },
-  {
-    value: 'main_propagated',
-    name: 'Все этапы выполнять основным агентом и передавать main совместимым скилам'
-  }
-];
-const AIM_MODE_CHOICES = [
-  {
-    value: 'automatic',
-    name: 'Автоматически отвечать на безопасные рабочие вопросы'
-  },
-  {
-    value: 'manual',
-    name: 'Передавать рабочие вопросы человеку'
-  }
-];
-const ORHESTRA_POLISH_LIMIT_CHOICES = [1, 2, 3, 5, 10].map(value => ({
-  value,
-  name: `${value} ${value === 1 ? 'итерация' : value < 5 ? 'итерации' : 'итераций'}`
-}));
-const PLAN_REVIEW_THRESHOLD_CHOICES = [80, 90, 100].map(value => ({
-  value,
-  name: `${value}/100`
-}));
-const PLAN_POLISH_LIMIT_CHOICES = [1, 2, 3].map(value => ({
-  value,
-  name: `${value} ${value === 1 ? 'итерация' : value < 5 ? 'итерации' : 'итераций'}`
-}));
-const PLAN_EXECUTE_MODE_CHOICES = [
-  {
-    value: 'auto',
-    name: 'Решать по плану: один контекст для простого плана, субагенты для остальных'
-  },
-  {
-    value: 'subagents',
-    name: 'Каждую фазу выполнять отдельным изолированным субагентом'
-  },
-  {
-    value: 'main',
-    name: 'Выполнять весь план основным агентом без субагентов'
-  }
-];
-const POLISH_EXECUTION_CHOICES = [
-  {
-    value: 'subagents',
-    name: 'Каждое review и исправление выполнять отдельным изолированным субагентом'
-  },
-  {
-    value: 'main',
-    name: 'Review и исправления выполнять основным агентом без субагентов'
-  }
+  'discover-automations',
+  'orhestra',
+  'aim'
 ];
+// Варианты ответа: первым — значение по умолчанию, дальше — от ближайшего к дальнему,
+// «Спрашивать каждый раз» всегда последним.
 const PLAN_SIZE_CHOICES = [
-  {
-    value: 'normal',
-    name: 'Обычный план по умолчанию'
-  },
-  {
-    value: 'short',
-    name: 'Короткий plan short по умолчанию'
-  },
-  {
-    value: 'ask_each_time',
-    name: 'Спрашивать размер плана каждый раз'
-  }
-];
-const MANUAL_TEST_DEPTH_CHOICES = [
-  {
-    value: 'full',
-    name: 'Полная проверка всех областей: API, база, консоль, интерфейс, алгоритмы'
-  },
-  {
-    value: 'smoke',
-    name: 'Только smoke: запуск и основной сценарий изменённой области'
-  },
-  {
-    value: 'ask_each_time',
-    name: 'Спрашивать глубину проверки каждый раз'
-  }
+  { value: 'normal', name: 'normal', description: 'обычный план до 40 КБ' },
+  { value: 'short', name: 'short', description: 'до 12 КБ и 2 фаз' },
+  { value: 'ask_each_time', name: 'Спрашивать каждый раз' }
 ];
 const DECISION_MODE_CHOICES = [
-  {
-    value: 'recommend_and_ask',
-    name: 'Рекомендовать вариант и спрашивать по важным развилкам'
-  },
-  {
-    value: 'autonomous',
-    name: 'Самостоятельно выбирать по коду, правилам и рискам'
-  },
-  {
-    value: 'ask_each_time',
-    name: 'Спрашивать по каждой значимой развилке'
-  }
+  { value: 'recommend_and_ask', name: 'Рекомендовать и спрашивать', description: 'вопрос только по существенным развилкам' },
+  { value: 'autonomous', name: 'Самостоятельно', description: 'агент выбирает сам и записывает причину' },
+  { value: 'ask_each_time', name: 'Спрашивать по каждой развилке' }
 ];
 const TEST_STRATEGY_CHOICES = [
-  {
-    value: 'after_each_phase',
-    name: 'Писать тесты и запускать все проверки после каждой фазы'
-  },
-  {
-    value: 'phase_tests_final_checks',
-    name: 'Писать тесты в фазах, запускать только их, остальные проверки — в конце'
-  },
-  {
-    value: 'tdd_each_phase',
-    name: 'В каждой фазе сначала тесты, затем код и все проверки фазы'
-  },
-  {
-    value: 'end_of_plan',
-    name: 'Писать тесты и запускать все проверки в конце плана'
-  },
-  {
-    value: 'ask_each_time',
-    name: 'Спрашивать стратегию тестов и проверок каждый раз'
-  }
+  { value: 'phase_tests_final_checks', name: 'Тесты в фазах, проверки в конце', description: 'в фазе — только её тесты, остальные проверки в конце плана' },
+  { value: 'tdd_each_phase', name: 'TDD', description: 'в каждой фазе сначала тесты, затем код и все проверки фазы' },
+  { value: 'after_each_phase', name: 'После каждой фазы', description: 'тесты и все проверки после каждой фазы' },
+  { value: 'end_of_plan', name: 'В конце плана', description: 'тесты и все проверки один раз в конце' },
+  { value: 'ask_each_time', name: 'Спрашивать каждый раз' }
 ];
-const LOGGING_STRATEGY_CHOICES = [
-  {
-    value: 'standard',
-    name: 'Стандартные info / warning / error по необходимости'
-  },
-  {
-    value: 'debug_precise',
-    name: 'Подробные debug-логи на важных шагах'
-  },
-  {
-    value: 'ask_each_time',
-    name: 'Спрашивать стратегию логирования каждый раз'
-  }
-];
-const REVIEW_MODE_CHOICES = [
-  { value: 'always', name: 'always — запускать при каждом ревью' },
-  { value: 'auto', name: 'auto — запускать только когда проверка применима' },
-  { value: 'off', name: 'off — отключить проверку' }
+const PLAN_EXECUTE_MODE_CHOICES = [
+  { value: 'main', name: 'Основной агент', description: 'весь план одним контекстом' },
+  { value: 'auto', name: 'auto', description: 'по плану: небольшой — одним контекстом, остальные — субагентами' },
+  { value: 'subagents', name: 'Субагенты', description: 'каждая фаза — отдельный субагент с независимой проверкой' }
 ];
 const REVIEW_EXECUTION_CHOICES = [
-  {
-    value: 'subagents',
-    name: 'Каждую review-проверку выполнять отдельным специализированным субагентом'
-  },
-  {
-    value: 'main',
-    name: 'Все review-проверки выполнять основным агентом без субагентов'
-  }
+  { value: 'main', name: 'Основной агент', description: 'все проверки в одном контексте' },
+  { value: 'subagents', name: 'Субагенты', description: 'разведчик и отдельный агент на каждую проверку' }
+];
+const POLISH_EXECUTION_CHOICES = [
+  { value: 'subagents', name: 'Субагенты', description: 'каждое ревью и исправление — отдельный субагент' },
+  { value: 'main', name: 'Основной агент', description: 'ревью и исправления в одном контексте' }
 ];
-const CLAUDE_REVIEW_MODEL_CHOICES = [
-  { value: 'haiku', name: 'haiku — быстрая проверка' },
-  { value: 'sonnet', name: 'sonnet — сильная проверка кода' },
-  { value: 'opus', name: 'opus — максимальная глубина' }
+const MANUAL_TEST_DEPTH_CHOICES = [
+  { value: 'full', name: 'Полная', description: 'API, БД, консоль, интерфейс, алгоритмы, фон, права, логи, регрессии' },
+  { value: 'smoke', name: 'Smoke', description: 'запуск и основной сценарий изменённой области' },
+  { value: 'ask_each_time', name: 'Спрашивать каждый раз' }
+];
+const ORHESTRA_MODE_CHOICES = [
+  { value: 'automatic', name: 'automatic', description: 'сам отвечает на безопасные рабочие вопросы' },
+  { value: 'manual', name: 'manual', description: 'план пишется с человеком, вопросы передаются ему' }
 ];
-const CODEX_REVIEW_MODEL_CHOICES = [
-  { value: 'gpt-5.6-luna', name: 'gpt-5.6-luna — быстрая проверка' },
-  { value: 'gpt-5.6-terra', name: 'gpt-5.6-terra — сильная проверка кода' },
-  { value: 'gpt-5.6-sol', name: 'gpt-5.6-sol — максимальная глубина' }
+const ORHESTRA_EXECUTION_CHOICES = [
+  { value: 'subagents', name: 'Субагенты', description: 'каждый этап — отдельный субагент' },
+  { value: 'main', name: 'Основной агент', description: 'цепочку ведёт текущий агент, вложенные скилы — по своим настройкам' },
+  { value: 'main_propagated', name: 'Основной агент везде', description: 'plan-execute, polish и review тоже без субагентов' }
 ];
+// План и выполнение в оркестраторе включены всегда; спрашиваются только необязательные этапы.
+const ORHESTRA_OPTIONAL_STEPS = ['eda-plan-polish', 'eda-polish', 'eda-manual-test'];
 const UPDATE_ALL_SETTINGS_MODE_CHOICES = [
-  {
-    value: 'configure',
-    name: 'Настроить один общий профиль и записать его во все проекты'
-  },
-  {
-    value: 'skip',
-    name: 'Skip — сохранить полный v3, остальные конфиги перенести с defaults'
-  }
+  { value: 'configure', name: 'Настроить общий профиль', description: 'ответить один раз и записать во все проекты' },
+  { value: 'skip', name: 'Skip', description: 'сохранить актуальные конфиги, старые перенести с defaults' }
 ];
 
 export async function init({ cwd, input = process.stdin, output = process.stdout }) {
@@ -630,6 +358,12 @@ export async function updateAll({
           output.write(`Полный файл настроек сохранён: ${SETTINGS_RELATIVE_PATH}\n`);
         } else {
           await writeSettingsContent(projectDir, rendered);
+          if (migrated.legacyReviewAgents) {
+            output.write(`review.agents переведён на формат 4.0, все роли — default: ${SETTINGS_RELATIVE_PATH}\n`);
+          }
+          if (migrated.resetReviewAgents.length > 0) {
+            output.write(`Недопустимые модели review.agents заменены на default (${migrated.resetReviewAgents.join(', ')}): ${SETTINGS_RELATIVE_PATH}\n`);
+          }
           output.write(migrated.retiredPaths.length > 0
             ? `Из файла настроек убраны устаревшие ключи (${migrated.retiredPaths.join(', ')}): ${SETTINGS_RELATIVE_PATH}\n`
             : migrated.normalizedPaths.length > 0
@@ -685,18 +419,20 @@ export async function askTargets({ input = process.stdin, output = process.stdou
     return TARGET_CHOICES.map(choice => choice.value);
   }
 
-  return checkbox({
-    message: 'Куда устанавливать пакет eda?',
-    instructions: 'Стрелки — выбрать, Space — отметить, Enter — продолжить',
-    choices: TARGET_CHOICES.map(choice => ({
-      name: `${choice.label} (${choice.dir})`,
+  const selected = await checkbox({
+    message: 'Куда установить eda?',
+    choices: targetPromptChoices().map(choice => ({
+      name: choice.label,
       value: choice.value,
-      checked: true
-    }))
+      description: choice.dir,
+      checked: false
+    })),
+    required: true
   }, {
     input,
     output
   });
+  return TARGET_CHOICES.map(choice => choice.value).filter(target => selected.includes(target));
 }
 
 export async function askUpdateTargets({
@@ -707,22 +443,25 @@ export async function askUpdateTargets({
 }) {
   if (!input.isTTY || !output.isTTY) return [...installedTargets];
 
+  // Установленные платформы обновляются всегда: снятая с них отметка ничего не удаляет.
   const selected = await checkboxPrompt({
-    message: 'Какие платформы добавить? Установленные уже отмечены и будут обновлены.',
-    instructions: 'Space — добавить платформу, Enter — обновить выбранные; установленные снимать нельзя',
-    choices: TARGET_CHOICES.map(choice => ({
-      name: `${choice.label} (${choice.dir})${installedTargets.includes(choice.value) ? ' — установлена' : ''}`,
+    message: 'Какие платформы обновить? Установленные уже отмечены.',
+    choices: targetPromptChoices().map(choice => ({
+      name: choice.label,
       value: choice.value,
+      description: installedTargets.includes(choice.value) ? 'установлена — обновится в любом случае' : choice.dir,
       checked: installedTargets.includes(choice.value)
-    })),
-    validate: choices => installedTargets.every(target => choices.some(choice => choice.value === target))
-      || 'Установленные платформы нужно оставить отмеченными: update их не удаляет.'
+    }))
   }, { input, output });
 
   return TARGET_CHOICES.map(choice => choice.value)
     .filter(target => installedTargets.includes(target) || selected.includes(target));
 }
 
+function targetPromptChoices() {
+  return TARGET_PROMPT_ORDER.map(value => TARGET_CHOICES.find(choice => choice.value === value));
+}
+
 export async function askUpdateAllSettingsMode({
   input = process.stdin,
   output = process.stdout,
@@ -743,6 +482,8 @@ export async function askUpdateAllSettingsMode({
   });
 }
 
+// Спрашиваются только основные настройки в порядке работы. Редкие получают defaults и
+// меняются вручную в docs/settings.yaml, как и модели review.agents.
 export async function askSettings({
   input = process.stdin,
   output = process.stdout,
@@ -759,157 +500,52 @@ export async function askSettings({
   }
 
   const requestedSections = new Set(sections);
-  const shouldAsk = key => missingKeys === null || missingKeys.has(key);
-  const booleanChoices = SETTINGS_CHOICES.filter(choice => (
-    requestedSections.has(choice.section) && shouldAsk(choice.key)
-  ));
-  const selected = booleanChoices.length > 0
-    ? await checkboxPrompt({
-        message: 'Какие настройки включить?',
-        instructions: 'Стрелки — выбрать, Space — отметить, Enter — продолжить',
-        choices: booleanChoices.map(({ section, key, ...choice }) => choice)
-      }, { input, output })
-    : [];
-
-  if (requestedSections.has('explore') && shouldAsk('explore.strict')) {
-    settings.explore.strict = selected.includes('exploreStrict');
-  }
-  if (requestedSections.has('plan') && shouldAsk('plan.strict')) {
-    settings.plan.strict = selected.includes('planStrict');
-  }
-  if (requestedSections.has('plan') && shouldAsk('plan.review')) {
-    settings.plan.review = selected.includes('planReview');
-  }
-  if (requestedSections.has('send-review') && shouldAsk('sendReview.closePreviousReviews')) {
-    settings.sendReview.closePreviousReviews = selected.includes('sendReviewClosePreviousReviews');
-  }
-  if (requestedSections.has('discover-automations') && shouldAsk('discoverAutomations.includePlans')) {
-    settings.discoverAutomations.includePlans = selected.includes('discoverAutomationsIncludePlans');
-  }
-
-  if (requestedSections.has('orhestra') && shouldAsk('orhestra.mode')) {
-    settings.orhestra.mode = await selectPrompt({
-      message: 'Как eda-orhestra должен вести полный цикл по умолчанию?',
-      choices: ORHESTRA_MODE_CHOICES,
-      default: settings.orhestra.mode
-    }, { input, output });
+  const shouldAsk = (section, key) => requestedSections.has(section) && (missingKeys === null || missingKeys.has(key));
+  const scalarPrompts = [
+    ['plan', 'plan.size', settings.plan, 'size', 'eda-plan · размер плана', PLAN_SIZE_CHOICES],
+    ['plan', 'plan.decisionMode', settings.plan, 'decisionMode', 'eda-plan · как принимать существенные решения', DECISION_MODE_CHOICES],
+    ['plan', 'plan.testStrategy', settings.plan, 'testStrategy', 'eda-plan · когда писать тесты и запускать проверки', TEST_STRATEGY_CHOICES],
+    ['plan-execute', 'planExecute.mode', settings.planExecute, 'mode', 'eda-plan-execute · где выполнять фазы плана', PLAN_EXECUTE_MODE_CHOICES],
+    ['review', 'review.execution', settings.review, 'execution', 'eda-review · где выполнять проверки', REVIEW_EXECUTION_CHOICES],
+    ['polish', 'polish.execution', settings.polish, 'execution', 'eda-polish · где выполнять ревью и исправления', POLISH_EXECUTION_CHOICES],
+    ['manual-test', 'manualTest.depth', settings.manualTest, 'depth', 'eda-manual-test · глубина ручной проверки', MANUAL_TEST_DEPTH_CHOICES],
+    ['orhestra', 'orhestra.mode', settings.orhestra, 'mode', 'eda-orhestra · как вести полный цикл', ORHESTRA_MODE_CHOICES]
+  ].filter(([section, key]) => shouldAsk(section, key));
+  const askSteps = shouldAsk('orhestra', 'orhestra.steps');
+  const askOrhestraExecution = shouldAsk('orhestra', 'orhestra.execution');
+  if (scalarPrompts.length === 0 && !askSteps && !askOrhestraExecution) return settings;
+
+  startPrompts(`Настройки ${SETTINGS_RELATIVE_PATH}`, { output });
+  for (const [, , target, property, message, choices] of scalarPrompts) {
+    target[property] = await selectPrompt({ message, choices, default: target[property] }, { input, output });
   }
-
-  if (requestedSections.has('orhestra') && shouldAsk('orhestra.steps')) {
+  if (askSteps) {
     const enabledSteps = await checkboxPrompt({
-      message: 'Какие этапы eda-orhestra включить по умолчанию?',
-      instructions: 'Порядок можно изменить вручную в docs/settings.yaml',
-      choices: ORHESTRA_STEP_DEFINITIONS.map(step => ({
-        name: step.name,
-        value: step.skill,
-        checked: settings.orhestra.steps.find(item => item.skill === step.skill)?.enabled !== false
-      }))
+      message: 'eda-orhestra · какие этапы включить, кроме плана и выполнения',
+      choices: ORHESTRA_STEP_DEFINITIONS
+        .filter(step => ORHESTRA_OPTIONAL_STEPS.includes(step.skill))
+        .map(step => ({
+          name: step.name,
+          value: step.skill,
+          checked: settings.orhestra.steps.find(item => item.skill === step.skill)?.enabled !== false
+        }))
     }, { input, output });
-    const codePolishLimit = enabledSteps.includes('eda-polish')
-      ? await selectPrompt({
-        message: 'Сколько итераций разрешить eda-polish внутри eda-orhestra?',
-        choices: ORHESTRA_POLISH_LIMIT_CHOICES,
-        default: 5
-      }, { input, output })
-      : 5;
     settings.orhestra.steps = DEFAULT_ORHESTRA_STEPS.map(step => ({
       ...step,
-      enabled: enabledSteps.includes(step.skill),
-      ...(step.skill === 'eda-polish' ? { args: `limit ${codePolishLimit}` } : {})
+      enabled: !ORHESTRA_OPTIONAL_STEPS.includes(step.skill) || enabledSteps.includes(step.skill)
     }));
   }
-
-  const scalarPrompts = [
-    ['orhestra', 'orhestra.execution', settings.orhestra, 'execution', 'Как eda-orhestra должен выполнять этапы по умолчанию?', ORHESTRA_EXECUTION_CHOICES],
-    ['aim', 'aim.mode', settings.aim, 'mode', 'Как eda-aim должен отвечать на рабочие вопросы по умолчанию?', AIM_MODE_CHOICES],
-    ['explore', 'explore.decisionMode', settings.explore, 'decisionMode', 'Как принимать важные решения в eda-explore?', DECISION_MODE_CHOICES],
-    ['plan', 'plan.size', settings.plan, 'size', 'Какой размер плана eda-plan использовать по умолчанию?', PLAN_SIZE_CHOICES],
-    ['plan', 'plan.decisionMode', settings.plan, 'decisionMode', 'Как принимать важные решения в eda-plan?', DECISION_MODE_CHOICES],
-    ['plan', 'plan.testStrategy', settings.plan, 'testStrategy', 'Какую стратегию тестов и проверок eda-plan использовать по умолчанию?', TEST_STRATEGY_CHOICES],
-    ['plan', 'plan.loggingStrategy', settings.plan, 'loggingStrategy', 'Какую стратегию логирования eda-plan использовать по умолчанию?', LOGGING_STRATEGY_CHOICES],
-    ['plan-review', 'planReview.threshold', settings.planReview, 'threshold', 'Какой порог готовности использовать в eda-plan-review?', PLAN_REVIEW_THRESHOLD_CHOICES],
-    ['plan-polish', 'planPolish.limit', settings.planPolish, 'limit', 'Сколько итераций «проверка + исправление» разрешить eda-plan-polish?', PLAN_POLISH_LIMIT_CHOICES],
-    ['plan-execute', 'planExecute.mode', settings.planExecute, 'mode', 'Как eda-plan-execute должен выполнять фазы по умолчанию?', PLAN_EXECUTE_MODE_CHOICES],
-    ['polish', 'polish.execution', settings.polish, 'execution', 'Как eda-polish должен выполнять review и исправления по умолчанию?', POLISH_EXECUTION_CHOICES],
-    ['manual-test', 'manualTest.depth', settings.manualTest, 'depth', 'Какую глубину ручной проверки использовать в eda-manual-test по умолчанию?', MANUAL_TEST_DEPTH_CHOICES],
-    ['review', 'review.execution', settings.review, 'execution', 'Как eda-review должен выполнять проверки по умолчанию?', REVIEW_EXECUTION_CHOICES]
-  ];
-  for (const [section, key, target, property, message, choices] of scalarPrompts) {
-    if (!requestedSections.has(section) || !shouldAsk(key)) continue;
-    target[property] = await selectPrompt({ message, choices, default: target[property] }, { input, output });
-  }
-
-  if (requestedSections.has('review')) {
-    settings.review.agents = await askReviewAgentSettings({
-      input,
-      output,
-      selectPrompt,
-      seedAgents: settings.review.agents,
-      missingKeys,
-      skipModels: settings.review.execution === 'main'
-    });
+  if (askOrhestraExecution) {
+    settings.orhestra.execution = await selectPrompt({
+      message: 'eda-orhestra · где выполнять этапы',
+      choices: ORHESTRA_EXECUTION_CHOICES,
+      default: settings.orhestra.execution
+    }, { input, output });
   }
+  finishPrompts(`Остальные настройки — в ${SETTINGS_RELATIVE_PATH}`, { output });
   return settings;
 }
 
-export async function askReviewAgentSettings({
-  input = process.stdin,
-  output = process.stdout,
-  selectPrompt = select,
-  seedAgents = DEFAULT_REVIEW_AGENTS,
-  missingKeys = null,
-  skipModels = false
-} = {}) {
-  return askAgentSettings({
-    input,
-    output,
-    selectPrompt,
-    definitions: REVIEW_AGENT_DEFINITIONS,
-    seedAgents,
-    missingKeys,
-    keyPrefix: 'review.agents',
-    label: 'review',
-    skipModels
-  });
-}
-
-async function askAgentSettings({ input, output, selectPrompt, definitions, seedAgents, missingKeys, keyPrefix, label, skipModels = false }) {
-  const configuredAgents = structuredClone(seedAgents);
-  const shouldAsk = key => missingKeys === null || missingKeys.has(key);
-  for (const agent of definitions) {
-    const modeKey = `${keyPrefix}.${agent.key}.mode`;
-    const claudeKey = `${keyPrefix}.${agent.key}.model.claude`;
-    const codexKey = `${keyPrefix}.${agent.key}.model.codex`;
-    let mode = configuredAgents[agent.key].mode;
-    if (shouldAsk(modeKey)) {
-      mode = await selectPrompt({
-        message: `Когда запускать ${label}-проверку ${agent.key}?`,
-        choices: REVIEW_MODE_CHOICES,
-        default: mode
-      }, { input, output });
-    }
-
-    let claude = configuredAgents[agent.key].model.claude;
-    let codex = configuredAgents[agent.key].model.codex;
-    if (!skipModels && mode !== 'off' && shouldAsk(claudeKey)) {
-      claude = await selectPrompt({
-        message: `Какой моделью Claude проверять ${agent.key}?`,
-        choices: CLAUDE_REVIEW_MODEL_CHOICES,
-        default: claude
-      }, { input, output });
-    }
-    if (!skipModels && mode !== 'off' && shouldAsk(codexKey)) {
-      codex = await selectPrompt({
-        message: `Какой моделью Codex проверять ${agent.key}?`,
-        choices: CODEX_REVIEW_MODEL_CHOICES,
-        default: codex
-      }, { input, output });
-    }
-    configuredAgents[agent.key] = { mode, model: { claude, codex } };
-  }
-  return configuredAgents;
-}
-
 async function detectTargets(cwd) {
   const targets = [];
   for (const { value } of TARGET_CHOICES) {
@@ -1141,6 +777,12 @@ async function ensureSettings(cwd, { input = process.stdin, output = process.std
   if (migrated.retiredPaths.length > 0) {
     output.write(`Убираю устаревшие настройки: ${migrated.retiredPaths.join(', ')}.\n`);
   }
+  if (migrated.legacyReviewAgents) {
+    output.write('Перевожу review.agents на формат 4.0: все роли — default, нужные модели впишите вручную.\n');
+  }
+  if (migrated.resetReviewAgents.length > 0) {
+    output.write(`Заменяю недопустимые модели review.agents на default: ${migrated.resetReviewAgents.join(', ')}.\n`);
+  }
   if (needsMigration) {
     output.write(`Переношу ${SETTINGS_RELATIVE_PATH} на version: ${SETTINGS_VERSION}; спрашиваются только отсутствующие значения.\n`);
   }
@@ -1179,6 +821,12 @@ async function configureSettings(cwd, { input = process.stdin, output = process.
   if (migrated.retiredPaths.length > 0) {
     output.write(`Убираю устаревшие настройки: ${migrated.retiredPaths.join(', ')}.\n`);
   }
+  if (migrated.legacyReviewAgents) {
+    output.write('Перевожу review.agents на формат 4.0: все роли — default, нужные модели впишите вручную.\n');
+  }
+  if (migrated.resetReviewAgents.length > 0) {
+    output.write(`Заменяю недопустимые модели review.agents на default: ${migrated.resetReviewAgents.join(', ')}.\n`);
+  }
   if (needsMigration) {
     output.write(`Переношу ${SETTINGS_RELATIVE_PATH} на version: ${SETTINGS_VERSION}; спрашиваются только отсутствующие значения.\n`);
   }
@@ -1198,7 +846,9 @@ function resolveSettingsUpdate(existingContent) {
   const migrated = migrateSettingsContent(existingContent);
   const needsMigration = migrated.version !== SETTINGS_VERSION
     || migrated.missingKeys.size > 0
-    || migrated.normalizedPaths.length > 0;
+    || migrated.normalizedPaths.length > 0
+    || migrated.legacyReviewAgents
+    || migrated.resetReviewAgents.length > 0;
   const rendered = formatSettings(migrated.settings);
   return { migrated, needsMigration, rendered, formatChanged: rendered !== existingContent };
 }
@@ -1269,7 +919,6 @@ function migrateSettingsContent(content) {
 
   const plan = objectValue(raw.plan);
   transfer('plan.strict', plan.strict ?? legacyStrict, bool, value => { settings.plan.strict = value; });
-  transfer('plan.review', plan.review ?? plan.meta_review, bool, value => { settings.plan.review = value; });
   transfer(
     'plan.size',
     plan.size ?? legacyDefaults.plan_size,
@@ -1332,17 +981,7 @@ function migrateSettingsContent(content) {
     oneOf(['subagents', 'main']),
     value => { settings.review.execution = value; }
   );
-  transferAgentSettings({
-    rawAgents: objectValue(review.agents),
-    settingsAgents: settings.review.agents,
-    definitions: REVIEW_AGENT_DEFINITIONS,
-    prefix: 'review.agents',
-    missingKeys
-  });
-  if (typeof review.include_code_quality === 'boolean' && !objectValue(review.agents).code_quality) {
-    settings.review.agents.code_quality.mode = review.include_code_quality ? 'always' : 'off';
-    missingKeys.delete('review.agents.code_quality.mode');
-  }
+  const { legacyReviewAgents, resetReviewAgents } = transferReviewAgents(objectValue(review.agents), settings.review.agents);
 
   const sendReview = objectValue(raw['send-review']);
   transfer(
@@ -1364,25 +1003,27 @@ function migrateSettingsContent(content) {
     settings,
     missingKeys,
     normalizedPaths,
+    legacyReviewAgents,
+    resetReviewAgents,
     retiredPaths: findRetiredSettingsPaths(raw),
     version: Number.isInteger(raw.version) ? raw.version : null
   };
 }
 
-function transferAgentSettings({ rawAgents, settingsAgents, definitions, prefix, missingKeys }) {
-  const allowedModes = ['always', 'auto', 'off'];
-  const allowedClaudeModels = CLAUDE_REVIEW_MODEL_CHOICES.map(choice => choice.value);
-  const allowedCodexModels = CODEX_REVIEW_MODEL_CHOICES.map(choice => choice.value);
-  for (const definition of definitions) {
-    const configured = objectValue(rawAgents[definition.key]);
-    const model = objectValue(configured.model);
-    if (allowedModes.includes(configured.mode)) settingsAgents[definition.key].mode = configured.mode;
-    else missingKeys.add(`${prefix}.${definition.key}.mode`);
-    if (allowedClaudeModels.includes(model.claude)) settingsAgents[definition.key].model.claude = model.claude;
-    else missingKeys.add(`${prefix}.${definition.key}.model.claude`);
-    if (allowedCodexModels.includes(model.codex)) settingsAgents[definition.key].model.codex = model.codex;
-    else missingKeys.add(`${prefix}.${definition.key}.model.codex`);
-  }
+// Формат до 4.0 ({ mode, model } по платформам) не переносится: при мажорном обновлении
+// все роли получают default. Иное значение не в формате default | provider/model#variant тоже
+// заменяется на default.
+function transferReviewAgents(rawAgents, settingsAgents) {
+  const legacyReviewAgents = Object.values(rawAgents).some(value => value && typeof value === 'object');
+  const resetReviewAgents = [];
+  if (legacyReviewAgents) return { legacyReviewAgents, resetReviewAgents };
+  for (const key of REVIEW_AGENT_KEYS) {
+    if (!Object.hasOwn(rawAgents, key)) continue;
+    const value = typeof rawAgents[key] === 'string' ? rawAgents[key].trim() : '';
+    if (REVIEW_AGENT_MODEL_PATTERN.test(value)) settingsAgents[key] = value;
+    else resetReviewAgents.push(key);
+  }
+  return { legacyReviewAgents, resetReviewAgents };
 }
 
 // У eda-polish больше нет порога: цикл завершается по нулю находок.
@@ -1483,66 +1124,36 @@ ${onFailureThen}
       # Строка аргументов передаётся скиллу как часть текущего вызова.
       args: ${JSON.stringify(step.args ?? '')}${onFailure}`;
   }).join('\n');
-  const formatAgents = (definitions, configuredAgents, defaults) => definitions.map(agent => {
-    const configured = configuredAgents[agent.key] ?? defaults[agent.key];
-    return `    # ${agent.description}
-    # ${agent.auto}
-    ${agent.key}:
-      # always — всегда, auto — когда применимо, off — отключено.
-      mode: ${configured.mode}
-      model:
-        # Модель для Claude Code.
-        claude: ${configured.model.claude}
-        # Модель для Codex.
-        codex: ${configured.model.codex}`;
-  }).join('\n\n');
-  const reviewAgents = formatAgents(REVIEW_AGENT_DEFINITIONS, settings.review.agents, DEFAULT_REVIEW_AGENTS);
+  const reviewAgents = REVIEW_AGENT_KEYS
+    .map(key => `    ${key}: ${settings.review.agents[key] ?? DEFAULT_REVIEW_AGENTS[key]}`)
+    .join('\n');
 
   return `version: ${SETTINGS_VERSION}
 
-orhestra:
-  # Режим полного цикла eda-orhestra.
-  # automatic | manual
-  mode: ${orhestra.mode ?? 'automatic'}
-  # Где выполняются этапы и передаётся ли main совместимым вложенным скилам.
-  # subagents | main | main_propagated
-  execution: ${orhestra.execution ?? 'subagents'}
-  # Упорядоченная цепочка. Шаги можно отключать, удалять и переставлять.
-  steps:
-${orhestraSteps}
-
-aim:
-  # Режим ответов на рабочие вопросы eda-aim.
-  # automatic | manual
-  mode: ${aim.mode ?? 'automatic'}
-
 explore:
   # Включает кросс-CLI ревью в eda-explore.
-  # true | false
+  # false | true
   strict: ${settings.explore.strict ? 'true' : 'false'}
-  # Определяет, как eda-explore ведёт исследовательские развилки.
-  # autonomous | recommend_and_ask | ask_each_time
+  # Как eda-explore ведёт исследовательские развилки.
+  # recommend_and_ask | autonomous | ask_each_time
   decision_mode: ${settings.explore.decisionMode}
 
 plan:
-  # Включает кросс-CLI ревью в eda-plan.
-  # true | false
-  strict: ${settings.plan.strict ? 'true' : 'false'}
-  # Запускает изолированный цикл eda-plan-polish.
-  # true | false
-  review: ${settings.plan.review !== false ? 'true' : 'false'}
-  # Задаёт размер плана.
+  # Размер плана.
   # normal | short | ask_each_time
   size: ${settings.plan.size}
-  # Определяет, как eda-plan принимает существенные решения.
-  # autonomous | recommend_and_ask | ask_each_time
+  # Как eda-plan принимает существенные решения.
+  # recommend_and_ask | autonomous | ask_each_time
   decision_mode: ${settings.plan.decisionMode}
-  # Задаёт стратегию тестов и автоматических проверок.
-  # after_each_phase | phase_tests_final_checks | tdd_each_phase | end_of_plan | ask_each_time
+  # Когда писать тесты и запускать автоматические проверки.
+  # phase_tests_final_checks | tdd_each_phase | after_each_phase | end_of_plan | ask_each_time
   test_strategy: ${settings.plan.testStrategy}
-  # Задаёт стратегию логирования.
-  # debug_precise | standard | ask_each_time
+  # Стратегия логирования.
+  # standard | debug_precise | ask_each_time
   logging_strategy: ${settings.plan.loggingStrategy}
+  # Включает кросс-CLI ревью в eda-plan.
+  # false | true
+  strict: ${settings.plan.strict ? 'true' : 'false'}
 
 plan-review:
   # Доля закрытых пунктов чек-листа, при которой план считается готовым.
@@ -1556,9 +1167,21 @@ plan-polish:
 
 plan-execute:
   # Где eda-plan-execute выполняет фазы плана.
-  # subagents — каждая фаза в отдельном субагенте, main — весь план одним контекстом, auto — решает по плану.
+  # main — весь план одним контекстом, auto — решает по плану, subagents — каждая фаза в отдельном субагенте.
   mode: ${settings.planExecute.mode}
 
+review:
+  # Где eda-review выполняет выбранные проверки.
+  # main — все проверки основным агентом без субагентов, subagents — разведчик и отдельный агент на проверку.
+  execution: ${settings.review.execution ?? 'main'}
+  # Какие проверки нужны, решает eda-review-triage по содержимому ревью.
+  # Модель роли в subagents: default — модель установленного агента, иначе provider/model#variant,
+  # например anthropic/claude-opus-5-5#high или deepseek/deepseek-flash-4.1; variant задаёт effort.
+  # Если среда не может запустить модель, роль идёт на модели агента с предупреждением в ревью.
+  # В main все проверки выполняет текущая модель.
+  agents:
+${reviewAgents}
+
 polish:
   # Где eda-polish выполняет review и исправления цикла полировки.
   # subagents — каждое review и каждое исправление в отдельном изолированном субагенте, main — оба контракта основным агентом.
@@ -1571,14 +1194,6 @@ manual-test:
   # full | smoke | ask_each_time
   depth: ${settings.manualTest.depth}
 
-review:
-  # Где eda-review выполняет выбранные проверки.
-  # subagents — отдельный специализированный агент на проверку, main — все проверки основным агентом без субагентов.
-  execution: ${settings.review.execution ?? 'subagents'}
-  # Каждая проверка имеет собственный режим запуска; модели применяются только в subagents.
-  agents:
-${reviewAgents}
-
 send-review:
   # После успешной отправки скрывает предыдущие сводки eda-send-review и резолвит их inline-треды.
   # true | false
@@ -1586,8 +1201,24 @@ send-review:
 
 discover-automations:
   # Добавляет docs/artifacts/plans/ в обычный запуск eda-discover-automations.
-  # true | false
+  # false | true
   include_plans: ${settings.discoverAutomations.includePlans ? 'true' : 'false'}
+
+orhestra:
+  # Режим полного цикла eda-orhestra.
+  # automatic | manual
+  mode: ${orhestra.mode ?? 'automatic'}
+  # Где выполняются этапы и передаётся ли main совместимым вложенным скилам.
+  # subagents | main | main_propagated
+  execution: ${orhestra.execution ?? 'subagents'}
+  # Упорядоченная цепочка. Шаги можно отключать, удалять и переставлять.
+  steps:
+${orhestraSteps}
+
+aim:
+  # Режим ответов на рабочие вопросы eda-aim.
+  # automatic | manual
+  mode: ${aim.mode ?? 'automatic'}
 `;
 }
 
@@ -1662,7 +1293,7 @@ async function buildSkillFiles(skill, target) {
   const entrypoint = files.get('SKILL.md');
   if (!entrypoint) throw new Error(`У скила ${skill.name} нет SKILL.md.`);
 
-  const content = entrypoint.toString('utf8');
+  const content = await expandIncludes(`Скил ${skill.name}`, entrypoint.toString('utf8'), PACKAGE_ROOT);
   const renderSkill = { claude: renderClaudeSkill, codex: renderCodexSkill, opencode: renderOpenCodeSkill }[target];
   const rendered = renderSkill(content, skill.config);
   files.set('SKILL.md', Buffer.from(rendered));