@gian-tiaga/eda 2.1.0 → 3.1.1

package/lib/install.js

@@ -19,6 +19,22 @@ const SKILLS_SRC = path.join(PACKAGE_ROOT, 'skills');
 const AGENTS_SRC = path.join(PACKAGE_ROOT, 'agents');
 const PACKAGE_JSON_PATH = path.join(PACKAGE_ROOT, 'package.json');
 const SETTINGS_RELATIVE_PATH = 'docs/settings.yaml';
+const ARTIFACTS_RELATIVE_PATH = 'docs/artifacts';
+const ARTIFACT_DIRECTORY_NAMES = [
+  'aims',
+  'automations',
+  'executions',
+  'fixes',
+  'manual-tests',
+  'plan-review-fixes',
+  'plan-reviews',
+  'plans',
+  'project-starts',
+  'researches',
+  'review-fixes',
+  'reviews',
+  'roadmaps'
+];
 const TARGET_CHOICES = [
   { value: 'claude', label: 'Claude Code', dir: '.claude/skills/' },
   { value: 'codex', label: 'Codex CLI', dir: '.codex/skills/' }
@@ -34,7 +50,25 @@ const RETIRED_SKILLS = [
   'eda-start'
 ];
 const RETIRED_AGENTS = [
-  'eda-commit-context'
+  'eda-commit-context',
+  'eda-plan-review-requirements',
+  'eda-plan-review-rules',
+  'eda-plan-review-architecture',
+  'eda-plan-review-feasibility',
+  'eda-plan-review-execution',
+  'eda-plan-review-verification',
+  'eda-plan-review-api',
+  'eda-plan-review-database',
+  'eda-plan-review-security',
+  'eda-plan-review-frontend',
+  '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;
@@ -170,114 +204,6 @@ const DEFAULT_REVIEW_AGENTS = Object.fromEntries(REVIEW_AGENT_DEFINITIONS.map(ag
     }
   }
 ]));
-const PLAN_REVIEW_AGENT_DEFINITIONS = [
-  {
-    key: 'requirements',
-    description: 'Проверяет соответствие плана задаче, scope, критериям готовности и применимым business-требованиям.',
-    auto: 'Эта базовая проверка по умолчанию запускается всегда.',
-    mode: 'always',
-    claude: 'sonnet',
-    codex: 'gpt-5.6-terra'
-  },
-  {
-    key: 'rules',
-    description: 'Проверяет соблюдение AGENTS.md, CLAUDE.md и docs/rules.md.',
-    auto: 'Эта базовая проверка по умолчанию запускается всегда.',
-    mode: 'always',
-    claude: 'haiku',
-    codex: 'gpt-5.6-luna'
-  },
-  {
-    key: 'architecture',
-    description: 'Проверяет архитектурные границы, зависимости и соответствие docs/arch.md.',
-    auto: 'Эта базовая проверка по умолчанию запускается всегда.',
-    mode: 'always',
-    claude: 'opus',
-    codex: 'gpt-5.6-sol'
-  },
-  {
-    key: 'feasibility',
-    description: 'Проверяет реализуемость плана относительно фактического кода и отсутствие скрытого исследования.',
-    auto: 'Эта базовая проверка по умолчанию запускается всегда.',
-    mode: 'always',
-    claude: 'opus',
-    codex: 'gpt-5.6-sol'
-  },
-  {
-    key: 'execution',
-    description: 'Проверяет связность фаз, ID задач, зависимости и размер работы для одного исполнителя.',
-    auto: 'Эта базовая проверка по умолчанию запускается всегда.',
-    mode: 'always',
-    claude: 'sonnet',
-    codex: 'gpt-5.6-terra'
-  },
-  {
-    key: 'verification',
-    description: 'Проверяет тесты, команды проверок, ошибки, логирование, документацию и эксплуатационную готовность.',
-    auto: 'Эта базовая проверка по умолчанию запускается всегда.',
-    mode: 'always',
-    claude: 'sonnet',
-    codex: 'gpt-5.6-terra'
-  },
-  {
-    key: 'api',
-    description: 'Проверяет API, DTO, события, очереди, внешние интеграции и совместимость контрактов.',
-    auto: 'В auto запускается при изменении API, событий, очередей или внешних интеграций.',
-    mode: 'auto',
-    claude: 'sonnet',
-    codex: 'gpt-5.6-terra'
-  },
-  {
-    key: 'database',
-    description: 'Проверяет схему данных, миграции, backfill, индексы, транзакции и rollback.',
-    auto: 'В auto запускается при изменении хранения, моделей, SQL, схем или миграций.',
-    mode: 'auto',
-    claude: 'opus',
-    codex: 'gpt-5.6-sol'
-  },
-  {
-    key: 'security',
-    description: 'Проверяет auth, permissions, внешний ввод, секреты, персональные данные и опасные операции.',
-    auto: 'В auto запускается для auth, permissions, внешнего ввода, секретов и чувствительных данных.',
-    mode: 'auto',
-    claude: 'opus',
-    codex: 'gpt-5.6-sol'
-  },
-  {
-    key: 'frontend',
-    description: 'Проверяет пользовательский сценарий, состояния UI, ошибки, accessibility и адаптивность.',
-    auto: 'В auto запускается для компонентов, стилей, клиентских маршрутов и состояния UI.',
-    mode: 'auto',
-    claude: 'sonnet',
-    codex: 'gpt-5.6-terra'
-  },
-  {
-    key: 'performance',
-    description: 'Проверяет нагрузку, запросы, конкурентность, кеши и горячие пути.',
-    auto: 'В auto запускается для запросов, пакетной обработки, кешей, конкурентности и горячих путей.',
-    mode: 'auto',
-    claude: 'opus',
-    codex: 'gpt-5.6-sol'
-  },
-  {
-    key: 'previous',
-    description: 'Проверяет закрытие находок предыдущей связанной итерации plan-review.',
-    auto: 'В auto запускается только при наличии связанной цепочки review/fix для текущего SHA-256 плана.',
-    mode: 'auto',
-    claude: 'haiku',
-    codex: 'gpt-5.6-luna'
-  }
-];
-const DEFAULT_PLAN_REVIEW_AGENTS = Object.fromEntries(PLAN_REVIEW_AGENT_DEFINITIONS.map(agent => [
-  agent.key,
-  {
-    mode: agent.mode,
-    model: {
-      claude: agent.claude,
-      codex: agent.codex
-    }
-  }
-]));
 const ORHESTRA_STEP_DEFINITIONS = [
   {
     id: 'plan',
@@ -343,11 +269,10 @@ const DEFAULT_SETTINGS = {
     loggingStrategy: 'ask_each_time'
   },
   planReview: {
-    threshold: 95,
-    agents: DEFAULT_PLAN_REVIEW_AGENTS
+    threshold: 100
   },
   planPolish: {
-    limit: 3
+    limit: 2
   },
   review: {
     agents: DEFAULT_REVIEW_AGENTS
@@ -431,11 +356,11 @@ const ORHESTRA_POLISH_LIMIT_CHOICES = [1, 2, 3, 5, 10].map(value => ({
   value,
   name: `${value} ${value === 1 ? 'итерация' : value < 5 ? 'итерации' : 'итераций'}`
 }));
-const PLAN_REVIEW_THRESHOLD_CHOICES = [90, 95, 100].map(value => ({
+const PLAN_REVIEW_THRESHOLD_CHOICES = [80, 90, 100].map(value => ({
   value,
   name: `${value}/100`
 }));
-const PLAN_POLISH_LIMIT_CHOICES = [1, 2, 3, 5, 10].map(value => ({
+const PLAN_POLISH_LIMIT_CHOICES = [1, 2, 3].map(value => ({
   value,
   name: `${value} ${value === 1 ? 'итерация' : value < 5 ? 'итерации' : 'итераций'}`
 }));
@@ -542,6 +467,7 @@ export async function update({ cwd, input = process.stdin, output = process.stdo
     return;
   }
   output.write(`Найдены установленные среды: ${targets.join(', ')}\n`);
+  await migrateArtifactDirectories(cwd, output);
   await configureSettings(cwd, { input, output });
   await syncPackage(cwd, targets, output, { action: 'update' });
 }
@@ -611,15 +537,18 @@ export async function updateAll({
       }
 
       output.write(`Найдены установленные среды: ${targets.join(', ')}\n`);
+      await migrateArtifactDirectories(projectDir, output);
       const projectSettingsPath = path.join(projectDir, SETTINGS_RELATIVE_PATH);
       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);
@@ -793,15 +722,6 @@ export async function askSettings({
     target[property] = await selectPrompt({ message, choices, default: target[property] }, { input, output });
   }
 
-  if (requestedSections.has('plan-review')) {
-    settings.planReview.agents = await askPlanReviewAgentSettings({
-      input,
-      output,
-      selectPrompt,
-      seedAgents: settings.planReview.agents,
-      missingKeys
-    });
-  }
   if (requestedSections.has('review')) {
     settings.review.agents = await askReviewAgentSettings({
       input,
@@ -833,25 +753,6 @@ export async function askReviewAgentSettings({
   });
 }
 
-export async function askPlanReviewAgentSettings({
-  input = process.stdin,
-  output = process.stdout,
-  selectPrompt = select,
-  seedAgents = DEFAULT_PLAN_REVIEW_AGENTS,
-  missingKeys = null
-} = {}) {
-  return askAgentSettings({
-    input,
-    output,
-    selectPrompt,
-    definitions: PLAN_REVIEW_AGENT_DEFINITIONS,
-    seedAgents,
-    missingKeys,
-    keyPrefix: 'planReview.agents',
-    label: 'plan-review'
-  });
-}
-
 async function askAgentSettings({ input, output, selectPrompt, definitions, seedAgents, missingKeys, keyPrefix, label }) {
   const configuredAgents = structuredClone(seedAgents);
   const shouldAsk = key => missingKeys === null || missingKeys.has(key);
@@ -1010,6 +911,92 @@ async function findInstalledProjects(rootDir, maxDepth) {
   return projects;
 }
 
+async function migrateArtifactDirectories(cwd, output = process.stdout) {
+  const docsDir = path.join(cwd, 'docs');
+  const artifactsDir = path.join(cwd, ARTIFACTS_RELATIVE_PATH);
+  const legacyDirectories = [];
+
+  for (const directoryName of ARTIFACT_DIRECTORY_NAMES) {
+    const legacyPath = path.join(docsDir, directoryName);
+    const legacyStat = await lstatIfExists(legacyPath);
+    if (!legacyStat) continue;
+    if (!legacyStat.isDirectory() || legacyStat.isSymbolicLink()) {
+      throw new Error(`Ожидался каталог артефактов: ${legacyPath}`);
+    }
+    legacyDirectories.push({ directoryName, legacyPath });
+  }
+
+  if (legacyDirectories.length === 0) {
+    return { migratedDirectories: 0, movedFiles: 0, discardedFiles: 0 };
+  }
+
+  const artifactsStat = await lstatIfExists(artifactsDir);
+  if (artifactsStat && (!artifactsStat.isDirectory() || artifactsStat.isSymbolicLink())) {
+    throw new Error(`Ожидался каталог артефактов: ${artifactsDir}`);
+  }
+  await fs.mkdir(artifactsDir, { recursive: true });
+
+  const result = {
+    migratedDirectories: legacyDirectories.length,
+    movedFiles: 0,
+    discardedFiles: 0
+  };
+  for (const { directoryName, legacyPath } of legacyDirectories) {
+    await mergeArtifactEntry(legacyPath, path.join(artifactsDir, directoryName), result);
+  }
+
+  output.write(
+    `Артефакты перенесены в ${ARTIFACTS_RELATIVE_PATH}/: каталогов ${result.migratedDirectories}, `
+    + `файлов перемещено ${result.movedFiles}, конфликтующих старых файлов удалено ${result.discardedFiles}.\n`
+  );
+  return result;
+}
+
+async function mergeArtifactEntry(sourcePath, destinationPath, result) {
+  const sourceStat = await fs.lstat(sourcePath);
+  const destinationStat = await lstatIfExists(destinationPath);
+
+  if (!destinationStat) {
+    result.movedFiles += await countArtifactFiles(sourcePath, sourceStat);
+    await fs.mkdir(path.dirname(destinationPath), { recursive: true });
+    await fs.rename(sourcePath, destinationPath);
+    return;
+  }
+
+  const mergeDirectories = sourceStat.isDirectory()
+    && !sourceStat.isSymbolicLink()
+    && destinationStat.isDirectory()
+    && !destinationStat.isSymbolicLink();
+  if (!mergeDirectories) {
+    result.discardedFiles += await countArtifactFiles(sourcePath, sourceStat);
+    await fs.rm(sourcePath, { recursive: true, force: true });
+    return;
+  }
+
+  const entries = await fs.readdir(sourcePath, { withFileTypes: true });
+  entries.sort((left, right) => left.name.localeCompare(right.name));
+  for (const entry of entries) {
+    await mergeArtifactEntry(
+      path.join(sourcePath, entry.name),
+      path.join(destinationPath, entry.name),
+      result
+    );
+  }
+  await fs.rmdir(sourcePath);
+}
+
+async function countArtifactFiles(entryPath, entryStat = null) {
+  const stat = entryStat ?? await fs.lstat(entryPath);
+  if (!stat.isDirectory() || stat.isSymbolicLink()) return 1;
+
+  let count = 0;
+  const entries = await fs.readdir(entryPath, { withFileTypes: true });
+  for (const entry of entries) {
+    count += await countArtifactFiles(path.join(entryPath, entry.name));
+  }
+  return count;
+}
+
 async function ensureSettings(cwd, { input = process.stdin, output = process.stdout } = {}) {
   const settingsPath = path.join(cwd, SETTINGS_RELATIVE_PATH);
   const existingContent = await readFileIfExists(settingsPath);
@@ -1021,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,
@@ -1049,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,
@@ -1144,13 +1143,6 @@ function migrateSettingsContent(content) {
 
   const planReview = objectValue(raw['plan-review']);
   transfer('planReview.threshold', planReview.threshold, score, value => { settings.planReview.threshold = value; });
-  transferAgentSettings({
-    rawAgents: objectValue(planReview.agents),
-    settingsAgents: settings.planReview.agents,
-    definitions: PLAN_REVIEW_AGENT_DEFINITIONS,
-    prefix: 'planReview.agents',
-    missingKeys
-  });
 
   const planPolish = objectValue(raw['plan-polish']);
   transfer('planPolish.limit', planPolish.limit, positiveInt, value => { settings.planPolish.limit = value; });
@@ -1187,6 +1179,7 @@ function migrateSettingsContent(content) {
   return {
     settings,
     missingKeys,
+    retiredPaths: findRetiredSettingsPaths(raw),
     version: Number.isInteger(raw.version) ? raw.version : null
   };
 }
@@ -1207,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 = [];
@@ -1220,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) {
@@ -1244,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(
@@ -1295,11 +1311,6 @@ ${onFailureThen}
         # Модель для Codex.
         codex: ${configured.model.codex}`;
   }).join('\n\n');
-  const planReviewAgents = formatAgents(
-    PLAN_REVIEW_AGENT_DEFINITIONS,
-    settings.planReview.agents,
-    DEFAULT_PLAN_REVIEW_AGENTS
-  );
   const reviewAgents = formatAgents(REVIEW_AGENT_DEFINITIONS, settings.review.agents, DEFAULT_REVIEW_AGENTS);
 
   return `version: ${SETTINGS_VERSION}
@@ -1346,12 +1357,9 @@ plan:
   logging_strategy: ${settings.plan.loggingStrategy}
 
 plan-review:
-  # План готов только без открытых находок и при score не ниже этого порога.
+  # Доля закрытых пунктов чек-листа, при которой план считается готовым.
   # 1..100
   threshold: ${settings.planReview.threshold}
-  # Каждая проверка имеет собственный режим запуска и модели для обеих сред.
-  agents:
-${planReviewAgents}
 
 plan-polish:
   # Максимальное число подтверждающих review-итераций.
@@ -1369,7 +1377,7 @@ send-review:
   close_previous_reviews: ${settings.sendReview.closePreviousReviews ? 'true' : 'false'}
 
 discover-automations:
-  # Добавляет docs/plans/ в обычный запуск eda-discover-automations.
+  # Добавляет docs/artifacts/plans/ в обычный запуск eda-discover-automations.
   # true | false
   include_plans: ${settings.discoverAutomations.includePlans ? 'true' : 'false'}
 `;

package/package.json

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

package/skills/eda-aim/SKILL.md

@@ -28,7 +28,7 @@ aim:
 
 В автоматическом режиме основной агент сам выбирает ответ на рабочие вопросы по рекомендации субагента и фактам проекта. Это не отменяет одно обязательное подтверждение человеком точного списка целей до создания нового файла.
 
-Если пользователь передал существующий файл из `docs/aims/`, продолжай его с текущего статуса. Не создавай второй файл для той же цели.
+Если пользователь передал существующий файл из `docs/artifacts/aims/`, продолжай его с текущего статуса. Не создавай второй файл для той же цели.
 
 ## Пакетные агенты
 
@@ -59,7 +59,7 @@ aim:
 
 ## Файл цели
 
-После подтверждения `eda-aim-planner` создаёт `docs/aims/{YYYY-MM-DD}_{HH-MM}_{slug}.md`. Файл должен оставаться коротким и содержать только цель и фактический журнал:
+После подтверждения `eda-aim-planner` создаёт `docs/artifacts/aims/{YYYY-MM-DD}_{HH-MM}_{slug}.md`. Файл должен оставаться коротким и содержать только цель и фактический журнал:
 
 ```markdown
 # <Короткое название>

package/skills/eda-discover-automations/SKILL.md

@@ -19,7 +19,7 @@ description: 'Аудирует автоматические проверки п
 |---|---|---|
 | `discover-automations.include_plans` | `true`, `false` | `plans`, «с планами»; выключение: `normal`, «без планов» |
 
-Обычный режим анализирует текущий проект и, если они есть, `docs/reviews/`, `docs/review-fixes/`, `docs/fixes/`. Режим `discover-automations.include_plans: true` добавляет `docs/plans/`. Отсутствие истории не отменяет полный аудит текущих проверок.
+Обычный режим анализирует текущий проект и, если они есть, `docs/artifacts/reviews/`, `docs/artifacts/review-fixes/`, `docs/artifacts/fixes/`. Режим `discover-automations.include_plans: true` добавляет `docs/artifacts/plans/`. Отсутствие истории не отменяет полный аудит текущих проверок.
 
 ## Вход из сообщения пользователя
 
@@ -49,11 +49,11 @@ description: 'Аудирует автоматические проверки п
 
 Если вход неоднозначен — `AskUserQuestion`: все ревью и фиксы / все ревью, фиксы и планы / последние N / период / конкретные файлы.
 
-Получи список файлов из `docs/reviews/`, `docs/review-fixes/` и `docs/fixes/`. В режиме `plans` дополнительно получи список файлов из `docs/plans/`. Отсутствующие или пустые папки отметь в отчёте, но не блокируйся: проведи baseline gap-аудит по коду, manifest-файлам, конфигам, scripts и CI текущего проекта.
+Получи список файлов из `docs/artifacts/reviews/`, `docs/artifacts/review-fixes/` и `docs/artifacts/fixes/`. В режиме `plans` дополнительно получи список файлов из `docs/artifacts/plans/`. Отсутствующие или пустые папки отметь в отчёте, но не блокируйся: проведи baseline gap-аудит по коду, manifest-файлам, конфигам, scripts и CI текущего проекта.
 
 ### 2. Прочитать контекст
 - Все выбранные ревью, отчёты о фиксах по ревью и обычные фиксы.
-- В режиме `plans`: все выбранные планы из `docs/plans/`.
+- В режиме `plans`: все выбранные планы из `docs/artifacts/plans/`.
 - `docs/rules.md`, `docs/arch.md` (если есть) — прочитай и строго следуй им.
 - Сначала определи затронутые историей и текущим аудитом доменные области. Затем через `docs/business.md` выбери только карточки этих областей; если подходящих нет, продолжай без business-контекста. Используй карточки лишь для поиска автоматизируемых инвариантов и acceptance-сценариев, не расширяй аудит на несвязанные темы.
 - Какие языки и фреймворки в проекте — определи сам.
@@ -96,19 +96,19 @@ description: 'Аудирует автоматические проверки п
 4. `agent` — MCP-сервер, новая/обновлённая eda-инструкция, специализированная агентная проверка или обязательный источник контекста. Предлагай только когда это закрывает значимую ошибку AI-процесса или даёт агенту данные, которые нельзя надёжно получить из кода.
 5. `rules` / `architecture` — текстовые правила и архитектурные уточнения, если проблема связана с договорённостью, границей модулей или ownership.
 
-Предпочитай уже установленный инструмент, если он ловит этот класс проблем. Новый инструмент предлагай только если он не дублирует инструмент того же класса. Для проблем из планов предлагай проверку `docs/plans/*.md`, а не runtime-линтер кода. Если правило или архитектура уже зафиксированы, не дублируй текст; предложи только уточнение, которое закрывает найденный случай.
+Предпочитай уже установленный инструмент, если он ловит этот класс проблем. Новый инструмент предлагай только если он не дублирует инструмент того же класса. Для проблем из планов предлагай проверку `docs/artifacts/plans/*.md`, а не runtime-линтер кода. Если правило или архитектура уже зафиксированы, не дублируй текст; предложи только уточнение, которое закрывает найденный случай.
 
 Подменять линтер, статанализатор или тест MCP-сервером/скилом нельзя, если ошибку можно надёжно поймать программной проверкой.
 
 Для каждой программной проверки укажи точный инструмент, команду/сценарий и контур: `pre-commit`, `pre-push`, `PR CI`, `nightly`, `release`, `post-deploy`. Быстрые проверки отдели от тяжёлых. Для MCP укажи сервер/класс инструмента, проверяемый сценарий, минимальные права, тестовые данные и контур `MCP/manual`; MCP не называй автоматической CI-проверкой, если он реально не запускается без участия агента.
 
 ### 5. Сохранить отчёт
-`docs/automations/{YYYY-MM-DD}_{HH-MM}_{slug}.md`. Slug — короткое описание набора (например, `php-strict-rules`).
+`docs/artifacts/automations/{YYYY-MM-DD}_{HH-MM}_{slug}.md`. Slug — короткое описание набора (например, `php-strict-rules`).
 
 ```markdown
 ---
 date: <YYYY-MM-DD HH:MM>
-sources: [docs/reviews/..., docs/review-fixes/..., docs/fixes/..., docs/business/...]
+sources: [docs/artifacts/reviews/..., docs/artifacts/review-fixes/..., docs/artifacts/fixes/..., docs/business/...]
 mode: <normal | plans>
 status: draft
 ---
@@ -132,7 +132,7 @@ status: draft
 ### 1. <название паттерна>
 - **Тип:** automation / tests / tooling
 - **Что ловит:** ...
-- **Встречалось в:** `docs/reviews/...` (×1), `docs/reviews/...` (×2)
+- **Встречалось в:** `docs/artifacts/reviews/...` (×1), `docs/artifacts/reviews/...` (×2)
 - **Почему автоматизировать:** <повторяемость либо высокий ущерб и вероятность повторения>
 - **Инструмент:** <название или текущий test runner> (если новый для проекта — пометка «требует установки»)
 - **Команда / сценарий:** <точная команда, glob, правило или тестовый сценарий>
@@ -150,7 +150,7 @@ status: draft
 ### 1. <название паттерна>
 - **Тип:** agent
 - **Что закрывает:** <какую ошибку процесса или нехватку контекста>
-- **Встречалось в:** `docs/reviews/...` (×1), `docs/fixes/...` (×1)
+- **Встречалось в:** `docs/artifacts/reviews/...` (×1), `docs/artifacts/fixes/...` (×1)
 - **Почему автоматизировать:** <повторяемость либо высокий ущерб и вероятность повторения>
 - **Предложение:** MCP-сервер / новый eda-скил / правка существующего скила / специализированная агентная проверка
 - **Сценарий и доступ:** <что проверяет, минимальные права, тестовые данные, почему CLI/CI недостаточно>
@@ -166,7 +166,7 @@ status: draft
 - **Тип:** rules / architecture
 - **Куда:** `docs/rules.md` / `docs/arch.md`, раздел <название или «новый раздел»>
 - **Почему:** <какой паттерн закрывает>
-- **Встречалось в:** `docs/fixes/...` (×1), `docs/reviews/...` (×1)
+- **Встречалось в:** `docs/artifacts/fixes/...` (×1), `docs/artifacts/reviews/...` (×1)
 - **Черновик текста:**
   ```markdown
   <готовый фрагмент для docs/rules.md или docs/arch.md>

package/skills/eda-explore/SKILL.md

@@ -38,7 +38,7 @@ description: 'Исследует техническую тему или зада
 
 ## Главные правила
 
-1. **Никаких правок кода.** Запись только в `docs/researches/`.
+1. **Никаких правок кода.** Запись только в `docs/artifacts/researches/`.
 2. **Итог — конкретика.** Нельзя заканчивать абстрактными описаниями, списком возможностей или «можно выбрать один из вариантов».
 3. **Развилки закрывай по `decision_mode`.** Значимые развилки подтверждай до финализации, мелкие локальные решения выбирай сам и не создавай шум.
 4. **Риски вплетай в исследование.** Не создавай отдельный обязательный раздел или этап рисков. Если риск влияет на выбор, источник, ограничение или будущую проверку, фиксируй его рядом с соответствующим решением: что может пойти не так и как это учитывается — снимаем, принимаем или выносим за рамки.
@@ -87,7 +87,7 @@ description: 'Исследует техническую тему или зада
 
 ### 4. Собрать отчёт
 
-Сохрани отчёт в `docs/researches/{YYYY-MM-DD}_{HH-MM}_{slug}.md`.
+Сохрани отчёт в `docs/artifacts/researches/{YYYY-MM-DD}_{HH-MM}_{slug}.md`.
 
 Формат отчёта должен быть компактным и универсальным: четыре основных раздела, без обязательного раздувания под каждый тип исследования. Таблицы, ASCII-диаграммы и дополнительные подпункты добавляй внутри этих разделов только когда они действительно помогают ясности.
 
@@ -176,4 +176,4 @@ CLI нет — `AskUserQuestion`: продолжать без ревью или
 - Перечислять риски без выбранного закрытия.
 - Планировать реализацию по шагам — это работа `eda-plan`.
 - Указывать `latest` без конкретной проверенной версии и источника.
-- Сохранять отчёт куда-либо кроме `docs/researches/`.
+- Сохранять отчёт куда-либо кроме `docs/artifacts/researches/`.