workflow-ai 1.0.25 → 1.0.26

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "workflow-ai",
-  "version": "1.0.25",
+  "version": "1.0.26",
   "description": "AI Agent Workflow Coordinator — kanban-based pipeline for AI coding agents",
   "type": "module",
   "bin": {

package/src/lib/logger.mjs

@@ -0,0 +1,73 @@
+import fs from 'fs';
+import path from 'path';
+import { findProjectRoot } from './find-root.mjs';
+
+const LEVELS = {
+  DEBUG: -1,
+  INFO: 0,
+  WARN: 1,
+  ERROR: 2
+};
+
+const COLORS = {
+  DEBUG: '\x1b[90m',
+  INFO: '\x1b[36m',
+  WARN: '\x1b[33m',
+  ERROR: '\x1b[31m',
+  RESET: '\x1b[0m'
+};
+
+function formatTimestamp() {
+  const now = new Date();
+  return now.toISOString().replace('T', ' ').substring(0, 19);
+}
+
+function ensureLogDir(logFilePath) {
+  const logDir = path.dirname(logFilePath);
+  if (!fs.existsSync(logDir)) {
+    fs.mkdirSync(logDir, { recursive: true });
+  }
+}
+
+function createLogger(logFilePath = null, consoleLevel = LEVELS.INFO) {
+  const projectRoot = findProjectRoot();
+  const defaultLogPath = path.join(projectRoot, '.workflow/logs/pipeline.log');
+  const logPath = logFilePath || defaultLogPath;
+  
+  ensureLogDir(logPath);
+
+  function writeToFile(formatted) {
+    fs.appendFileSync(logPath, formatted + '\n', 'utf8');
+  }
+
+  function writeToConsole(formatted, level) {
+    if (LEVELS[level] < consoleLevel) return;
+    
+    const color = COLORS[level];
+    const reset = COLORS.RESET;
+    
+    if (level === 'ERROR') {
+      console.error(`${color}${formatted}${reset}`);
+    } else if (level === 'WARN') {
+      console.warn(`${color}${formatted}${reset}`);
+    } else {
+      console.log(`${color}${formatted}${reset}`);
+    }
+  }
+
+  function log(level, message) {
+    const timestamp = formatTimestamp();
+    const formatted = `[${timestamp}] [${level}] ${message}`;
+    writeToFile(formatted);
+    writeToConsole(formatted, level);
+  }
+
+  return {
+    debug: (message) => log('DEBUG', message),
+    info: (message) => log('INFO', message),
+    warn: (message) => log('WARN', message),
+    error: (message) => log('ERROR', message)
+  };
+}
+
+export { createLogger, LEVELS };

package/src/lib/utils.mjs

@@ -1,6 +1,7 @@
 import YAML from 'js-yaml';
 import { fileURLToPath } from 'url';
 import path from 'path';
+import fs from 'fs';
 
 /**
  * Парсит YAML frontmatter из markdown-файла.
@@ -116,17 +117,26 @@ export function getPackageRoot() {
 export function getLastReviewStatus(content) {
   if (!content) return null;
 
-  // Находим секцию "## Ревью" — захватываем всё до следующего заголовка ## или конца файла
-  const headerIdx = content.search(/^##\s*Ревью\s*$/m);
-  if (headerIdx === -1) return null;
+  // Находим последний заголовок H2 "## Ревью" (только строки начинающиеся с "## ")
+  const lines = content.split('\n');
+  let lastHeaderLineIndex = -1;
+  
+  for (let i = 0; i < lines.length; i++) {
+    if (lines[i].startsWith('## ') && lines[i].includes('Ревью')) {
+      lastHeaderLineIndex = i;
+    }
+  }
 
-  const bodyStart = content.indexOf('\n', headerIdx);
-  if (bodyStart === -1) return null;
+  if (lastHeaderLineIndex === -1) return null;
 
-  const nextH2 = content.indexOf('\n## ', bodyStart);
-  const reviewSection = (nextH2 === -1
-    ? content.slice(bodyStart + 1)
-    : content.slice(bodyStart + 1, nextH2)).trim();
+  // Собираем содержимое после заголовка до следующего H2 заголовка
+  const reviewLines = [];
+  for (let i = lastHeaderLineIndex + 1; i < lines.length; i++) {
+    if (lines[i].startsWith('## ')) break; // следующий H2 заголовок
+    reviewLines.push(lines[i]);
+  }
+  
+  const reviewSection = reviewLines.join('\n').trim();
   if (!reviewSection) return null;
 
   // Пробуем распарсить табличный формат
@@ -154,9 +164,96 @@ export function getLastReviewStatus(content) {
   if (listItems.length > 0) {
     // Последний элемент списка = самое свежее ревью (записи ведутся хронологически)
     const latestItem = listItems[listItems.length - 1].trim();
-    const statusMatch = latestItem.match(/:\s*(passed|failed)\b/i);
+    const statusMatch = latestItem.match(/:\s*(passed|failed|skipped)\b/i);
     if (statusMatch) return statusMatch[1].toLowerCase();
   }
 
   return null;
 }
+
+/**
+ * Загружает конфигурацию правил перемещения тикетов.
+ *
+ * @param {string} configPath - Путь к конфигурационному файлу
+ * @returns {object} Объект конфигурации с правилами
+ */
+export function loadTicketMovementRules(configPath) {
+  if (!fs.existsSync(configPath)) {
+    throw new Error(`Config file not found: ${configPath}`);
+  }
+  const content = fs.readFileSync(configPath, 'utf8');
+  return YAML.load(content);
+}
+
+/**
+ * Проверяет все тикеты плана и закрывает его если все выполнены.
+ *
+ * @param {string} workflowDir - Путь к директории .workflow/
+ * @param {string} planId - Нормализованный ID плана (например "PLAN-002")
+ * @returns {{ closed: boolean, reason: string, total: number, done: number }}
+ */
+export function checkAndClosePlan(workflowDir, planId) {
+  if (!workflowDir || !planId) {
+    return { closed: false, reason: 'Missing workflowDir or planId', total: 0, done: 0 };
+  }
+
+  const ticketsDir = path.join(workflowDir, 'tickets');
+  const allDirNames = ['backlog', 'ready', 'in-progress', 'blocked', 'review', 'done'];
+  const allTickets = [];
+
+  for (const dirName of allDirNames) {
+    const dir = path.join(ticketsDir, dirName);
+    if (!fs.existsSync(dir)) continue;
+
+    const files = fs.readdirSync(dir).filter(f => f.endsWith('.md') && f !== '.gitkeep.md');
+    for (const file of files) {
+      try {
+        const content = fs.readFileSync(path.join(dir, file), 'utf8');
+        const { frontmatter } = parseFrontmatter(content);
+        if (normalizePlanId(frontmatter.parent_plan) === planId) {
+          allTickets.push({ id: frontmatter.id || file.replace('.md', ''), dir: dirName });
+        }
+      } catch (_) { /* skip malformed */ }
+    }
+  }
+
+  const total = allTickets.length;
+  const done = allTickets.filter(t => t.dir === 'done').length;
+
+  if (total === 0) {
+    return { closed: false, reason: 'No tickets found for plan', total, done };
+  }
+
+  if (done < total) {
+    return { closed: false, reason: `${done}/${total} tickets done`, total, done };
+  }
+
+  const plansDir = path.join(workflowDir, 'plans', 'current');
+  if (!fs.existsSync(plansDir)) {
+    return { closed: false, reason: 'Plans directory not found', total, done };
+  }
+
+  const planFile = fs.readdirSync(plansDir)
+    .filter(f => f.endsWith('.md'))
+    .find(f => normalizePlanId(f) === planId);
+
+  if (!planFile) {
+    return { closed: false, reason: 'Plan file not found', total, done };
+  }
+
+  const planPath = path.join(plansDir, planFile);
+  const planContent = fs.readFileSync(planPath, 'utf8');
+  const { frontmatter, body } = parseFrontmatter(planContent);
+
+  if (frontmatter.status === 'completed') {
+    return { closed: false, reason: 'Plan already completed', total, done };
+  }
+
+  frontmatter.status = 'completed';
+  frontmatter.completed_at = new Date().toISOString();
+  frontmatter.updated_at = new Date().toISOString();
+
+  fs.writeFileSync(planPath, serializeFrontmatter(frontmatter) + body, 'utf8');
+
+  return { closed: true, reason: 'All tickets done', total, done };
+}

package/src/runner.mjs

@@ -8,16 +8,18 @@ import yaml from 'js-yaml';
 import { findProjectRoot } from './lib/find-root.mjs';
 
 // ============================================================================
-// Logger — система логирования с уровнями INFO/WARN/ERROR
+// Logger — система логирования с уровнями DEBUG/INFO/WARN/ERROR
 // ============================================================================
 class Logger {
   static LEVELS = {
+    DEBUG: -1,
     INFO: 0,
     WARN: 1,
     ERROR: 2
   };
 
   static COLORS = {
+    DEBUG: '\x1b[90m',   // gray
     INFO: '\x1b[36m',    // cyan
     WARN: '\x1b[33m',    // yellow
     ERROR: '\x1b[31m',   // red
@@ -28,6 +30,7 @@ class Logger {
     this.logFilePath = logFilePath;
     this.consoleLevel = consoleLevel;
     this.stats = {
+      debug: 0,
       info: 0,
       warn: 0,
       error: 0,
@@ -53,17 +56,6 @@ class Logger {
     }
   }
 
-  /**
-   * Создаёт директорию для логов если она не существует
-   */
-  _ensureLogDirectory() {
-    const logDir = path.dirname(this.logFilePath);
-    if (!fs.existsSync(logDir)) {
-      fs.mkdirSync(logDir, { recursive: true });
-      console.log(`[Logger] Created log directory: ${logDir}`);
-    }
-  }
-
   /**
    * Открывает файл для записи (append mode)
    */
@@ -142,7 +134,8 @@ class Logger {
     this._writeToConsole(formattedMessage, level);
 
     // Обновляем статистику
-    if (level === 'INFO') this.stats.info++;
+    if (level === 'DEBUG') this.stats.debug++;
+    else if (level === 'INFO') this.stats.info++;
     else if (level === 'WARN') this.stats.warn++;
     else if (level === 'ERROR') this.stats.error++;
   }
@@ -168,6 +161,13 @@ class Logger {
     this._log('ERROR', stage, message);
   }
 
+  /**
+   * Логгирует DEBUG сообщение
+   */
+  debug(message, stage) {
+    this._log('DEBUG', stage, message);
+  }
+
   /**
    * Логгирует старт stage
    */
@@ -242,9 +242,10 @@ class Logger {
       '┌─────────────────────────────────────────────────────────┐',
       '│ LOG STATISTICS                                          │',
       '├─────────────────────────────────────────────────────────┤',
-      `│ INFO messages:    ${String(this.stats.info).padEnd(34)}│`,
-      `│ WARN messages:    ${String(this.stats.warn).padEnd(34)}│`,
-      `│ ERROR messages:   ${String(this.stats.error).padEnd(34)}│`,
+      `│ DEBUG messages:    ${String(this.stats.debug).padEnd(34)}│`,
+      `│ INFO messages:     ${String(this.stats.info).padEnd(34)}│`,
+      `│ WARN messages:     ${String(this.stats.warn).padEnd(34)}│`,
+      `│ ERROR messages:    ${String(this.stats.error).padEnd(34)}│`,
       '├─────────────────────────────────────────────────────────┤',
       '│ STAGE STATISTICS                                        │',
       '├─────────────────────────────────────────────────────────┤',

package/src/scripts/pick-next-task.js

@@ -21,7 +21,10 @@
 import fs from 'fs';
 import path from 'path';
 import { findProjectRoot } from '../lib/find-root.mjs';
-import { parseFrontmatter, printResult, normalizePlanId, extractPlanId, getLastReviewStatus, serializeFrontmatter } from '../lib/utils.mjs';
+import { parseFrontmatter, printResult, normalizePlanId, extractPlanId, getLastReviewStatus, serializeFrontmatter, loadTicketMovementRules, checkAndClosePlan } from '../lib/utils.mjs';
+import { createLogger } from '../lib/logger.mjs';
+
+const logger = createLogger();
 
 // Корень проекта
 const PROJECT_DIR = findProjectRoot();
@@ -72,12 +75,133 @@ function checkCondition(condition) {
       return false;
 
     default:
-      // Неизвестный тип условия - считаем выполненным
-      console.error(`[WARN] Unknown condition type: ${type}`);
+      logger.warn(`Unknown condition type: ${type}`);
       return true;
   }
 }
 
+/**
+ * Парсит секцию "## Ревью" тикета и возвращает все записи ревью.
+ * @param {string} content - Содержимое тикета
+ * @returns {Array<{date: string, status: string, comment: string}>}
+ */
+function parseReviewSection(content) {
+  if (!content) return [];
+
+  const headerIdx = content.search(/^##\s*Ревью\s*$/m);
+  if (headerIdx === -1) return [];
+
+  const bodyStart = content.indexOf('\n', headerIdx);
+  if (bodyStart === -1) return [];
+
+  const nextH2 = content.indexOf('\n## ', bodyStart);
+  const reviewSection = (nextH2 === -1
+    ? content.slice(bodyStart + 1)
+    : content.slice(bodyStart + 1, nextH2)).trim();
+
+  const reviews = [];
+
+  const tableRows = reviewSection.split('\n').filter(line => line.trim().startsWith('|'));
+  if (tableRows.length >= 2) {
+    const dataRows = tableRows.slice(2).filter(row => {
+      const cells = row.split('|').map(c => c.trim()).filter(c => c);
+      return cells.length >= 2;
+    });
+
+    for (const row of dataRows) {
+      const cells = row.split('|').map(c => c.trim()).filter(c => c);
+      const date = cells[0] || '';
+      const statusRaw = cells[1]?.toLowerCase() || '';
+      const comment = cells[2] || '';
+      let status = null;
+      if (statusRaw.includes('passed')) status = 'passed';
+      else if (statusRaw.includes('failed')) status = 'failed';
+      else if (statusRaw.includes('skipped')) status = 'skipped';
+
+      if (status) {
+        reviews.push({ date, status, comment });
+      }
+    }
+  }
+
+  const listItems = reviewSection.split('\n').filter(line => line.trim().match(/^[-*]\s/));
+  for (const item of listItems) {
+    const trimmed = item.trim();
+    const dateMatch = trimmed.match(/^[-*]\s*(\d{4}-\d{2}-\d{2})/);
+    const statusMatch = trimmed.match(/:\s*(passed|failed|skipped)\b/i);
+    if (dateMatch && statusMatch) {
+      reviews.push({
+        date: dateMatch[1],
+        status: statusMatch[1].toLowerCase(),
+        comment: trimmed.replace(/^[-*]\s*\d{4}-\d{2}-\d{2}:\s*(passed|failed|skipped)\b/i, '').trim()
+      });
+    }
+  }
+
+  return reviews;
+}
+
+/**
+ * Вычисляет метрики ревью-итераций для всех тикетов
+ * @returns {object} Метрики: iterations, avgTimeToFirstPassed, failedVsPassed
+ */
+function calculateReviewMetrics() {
+  const allDirs = [BACKLOG_DIR, READY_DIR, IN_PROGRESS_DIR, BLOCKED_DIR, REVIEW_DIR, DONE_DIR];
+  const ticketMetrics = {};
+  let totalFailed = 0;
+  let totalPassed = 0;
+  let firstPassedTimes = [];
+
+  for (const dir of allDirs) {
+    if (!fs.existsSync(dir)) continue;
+
+    const files = fs.readdirSync(dir).filter(f => f.endsWith('.md') && f !== '.gitkeep.md');
+
+    for (const file of files) {
+      const filePath = path.join(dir, file);
+      try {
+        const content = fs.readFileSync(filePath, 'utf8');
+        const { frontmatter } = parseFrontmatter(content);
+        const ticketId = frontmatter.id || file.replace('.md', '');
+
+        const reviews = parseReviewSection(content);
+        if (reviews.length === 0) continue;
+
+        ticketMetrics[ticketId] = reviews.length;
+
+        for (const review of reviews) {
+          if (review.status === 'failed') totalFailed++;
+          else if (review.status === 'passed') totalPassed++;
+        }
+
+        const firstPassed = reviews.find(r => r.status === 'passed');
+        if (firstPassed && firstPassed.date) {
+          const ticketCreated = new Date(frontmatter.created_at || '1970-01-01');
+          const passedDate = new Date(firstPassed.date);
+          const daysToPass = Math.floor((passedDate - ticketCreated) / (1000 * 60 * 60 * 24));
+          if (daysToPass >= 0) {
+            firstPassedTimes.push(daysToPass);
+          }
+        }
+      } catch (e) {
+        // Skip errors
+      }
+    }
+  }
+
+  const avgTimeToFirstPassed = firstPassedTimes.length > 0
+    ? Math.round(firstPassedTimes.reduce((a, b) => a + b, 0) / firstPassedTimes.length)
+    : null;
+
+  return {
+    iterations_per_ticket: ticketMetrics,
+    total_failed: totalFailed,
+    total_passed: totalPassed,
+    avg_time_to_first_passed_days: avgTimeToFirstPassed,
+    tickets_with_reviews: Object.keys(ticketMetrics).length
+  };
+}
+
 /**
  * Проверяет зависимости тикета
  */
@@ -94,19 +218,23 @@ function checkDependencies(dependencies) {
 
 /**
  * Авто-коррекция тикетов на основе статуса ревью.
- * Сканирует все директории и перемещает тикеты по правилам:
- * - blocked → done (если review = passed)
- * - blocked → backlog (если review = failed, НЕ при null)
- * - done → backlog (если review = failed)
- * - review → done (если review = passed)
- * - in-progress → done (если review = passed)
- * - ready → done (если review = passed)
+ * Сканирует все директории и перемещает тикеты по правилам из конфига.
  *
+ * @param {object} config - Конфигурация правил перемещения
  * @returns {object} Результат: { moved: Array<{id, from, to, reason}> }
  */
-function autoCorrectTickets() {
+function autoCorrectTickets(config) {
   const moved = [];
 
+  const dirMap = {
+    backlog: BACKLOG_DIR,
+    ready: READY_DIR,
+    in_progress: IN_PROGRESS_DIR,
+    blocked: BLOCKED_DIR,
+    review: REVIEW_DIR,
+    done: DONE_DIR
+  };
+
   /**
    * Перемещает тикет из одной директории в другую
    */
@@ -119,23 +247,18 @@ function autoCorrectTickets() {
     }
 
     try {
-      // Читаем содержимое
       const content = fs.readFileSync(fromPath, 'utf8');
       const { frontmatter, body } = parseFrontmatter(content);
 
-      // Обновляем updated_at
       frontmatter.updated_at = new Date().toISOString();
 
-      // Если перемещаем в done — ставим completed_at
       if (toDir === DONE_DIR && !frontmatter.completed_at) {
         frontmatter.completed_at = new Date().toISOString();
       }
 
-      // Сериализуем и записываем в новую директорию
       const newContent = serializeFrontmatter(frontmatter) + body;
       fs.writeFileSync(toPath, newContent, 'utf8');
 
-      // Удаляем старый файл
       fs.unlinkSync(fromPath);
 
       console.log(`[AUTO-CORRECT] ${ticketId}: ${path.basename(fromDir)} → ${path.basename(toDir)} (${reason})`);
@@ -149,7 +272,7 @@ function autoCorrectTickets() {
 
       return true;
     } catch (e) {
-      console.error(`[ERROR] Failed to move ticket ${ticketId}: ${e.message}`);
+      logger.error(`Failed to move ticket ${ticketId}: ${e.message}`);
       return false;
     }
   }
@@ -157,7 +280,7 @@ function autoCorrectTickets() {
   /**
    * Обрабатывает тикеты в указанной директории
    */
-  function processDirectory(dir, rules) {
+  function processDirectory(dir, rules, dirName) {
     if (!fs.existsSync(dir)) return;
 
     const files = fs.readdirSync(dir)
@@ -170,112 +293,46 @@ function autoCorrectTickets() {
         const { frontmatter } = parseFrontmatter(content);
         const ticketId = frontmatter.id || file.replace('.md', '');
 
-        // Получаем статус ревью
         const reviewStatus = getLastReviewStatus(content);
 
-        // Применяем правила
         for (const rule of rules) {
-          if (rule.condition(reviewStatus)) {
-            moveTicket(ticketId, dir, rule.toDir, rule.reason);
-            break; // Только одно перемещение на тикет
+          const ruleCondition = rule.condition;
+          let shouldMove = false;
+
+          if (ruleCondition === null) {
+            shouldMove = reviewStatus === null;
+          } else {
+            shouldMove = reviewStatus === ruleCondition;
+          }
+
+          if (shouldMove) {
+            const targetDirName = rule.to_dir;
+            const targetDir = dirMap[targetDirName];
+            if (targetDir) {
+              moveTicket(ticketId, dir, targetDir, rule.reason);
+            }
+            break;
           }
         }
       } catch (e) {
-        console.error(`[WARN] Failed to process ticket ${file}: ${e.message}`);
+        logger.warn(`Failed to process ticket ${file}: ${e.message}`);
       }
     }
   }
 
-  // Правила для backlog/ (защита от ошибочного перемещения завершённых тикетов)
-  processDirectory(BACKLOG_DIR, [
-    {
-      condition: (status) => status === 'passed',
-      toDir: DONE_DIR,
-      reason: 'review passed'
-    },
-    {
-      condition: (status) => status === 'skipped',
-      toDir: DONE_DIR,
-      reason: 'review skipped'
-    }
-  ]);
-
-  // Правила для blocked/
-  processDirectory(BLOCKED_DIR, [
-    {
-      condition: (status) => status === 'passed',
-      toDir: DONE_DIR,
-      reason: 'review passed'
-    },
-    {
-      condition: (status) => status === 'skipped',
-      toDir: DONE_DIR,
-      reason: 'review skipped'
-    },
-    {
-      condition: (status) => status === 'failed',
-      toDir: BACKLOG_DIR,
-      reason: 'review failed'
-    }
-    // null (нет ревью) — не перемещаем
-  ]);
-
-  // Правила для done/
-  processDirectory(DONE_DIR, [
-    {
-      condition: (status) => status === 'failed',
-      toDir: BACKLOG_DIR,
-      reason: 'review failed'
-    },
-    {
-      condition: (status) => status === null,
-      toDir: BACKLOG_DIR,
-      reason: 'no review'
-    }
-    // passed или skipped — не перемещаем
-  ]);
-
-  // Правила для review/
-  processDirectory(REVIEW_DIR, [
-    {
-      condition: (status) => status === 'passed',
-      toDir: DONE_DIR,
-      reason: 'review passed'
-    },
-    {
-      condition: (status) => status === 'skipped',
-      toDir: DONE_DIR,
-      reason: 'review skipped'
-    }
-  ]);
-
-  // Правила для in-progress/
-  processDirectory(IN_PROGRESS_DIR, [
-    {
-      condition: (status) => status === 'passed',
-      toDir: DONE_DIR,
-      reason: 'review passed'
-    },
-    {
-      condition: (status) => status === 'skipped',
-      toDir: DONE_DIR,
-      reason: 'review skipped'
-    }
-  ]);
-
-  // Правила для ready/
-  processDirectory(READY_DIR, [
-    {
-      condition: (status) => status === 'passed',
-      toDir: DONE_DIR,
-      reason: 'review passed'
-    },
-    {
-      condition: (status) => status === 'skipped',
-      toDir: DONE_DIR,
-      reason: 'review skipped'
+  if (!config || !config.rules) {
+    logger.error('Ticket movement rules config not loaded');
+    return { moved };
+  }
+
+  const rulesConfig = config.rules;
+
+  for (const [dirName, rules] of Object.entries(rulesConfig)) {
+    const dir = dirMap[dirName];
+    if (dir) {
+      processDirectory(dir, rules, dirName);
     }
-  ]);
+  }
 
   return { moved };
 }
@@ -438,7 +495,7 @@ function pickNextTicket(planId) {
       const completedInProgress = filterByPlan(findCompletedInProgress(), planId);
       if (completedInProgress.length > 0) {
         const first = completedInProgress[0];
-        console.log(`[INFO] Found completed ticket in in-progress/: ${first.id}`);
+        logger.info(`Found completed ticket in in-progress/: ${first.id}`);
         return {
           status: 'completed_in_progress',
           ticket_id: first.id
@@ -517,32 +574,68 @@ async function main() {
   const planId = extractPlanId();
 
   if (planId) {
-    console.log(`[INFO] Filtering by plan_id: ${planId}`);
+    logger.info(`Filtering by plan_id: ${planId}`);
+  }
+
+  const configPath = path.join(WORKFLOW_DIR, 'config', 'ticket-movement-rules.yaml');
+  let movementConfig = null;
+  try {
+    movementConfig = loadTicketMovementRules(configPath);
+    logger.info('Loaded ticket movement rules from config');
+  } catch (e) {
+    logger.warn(`Failed to load ticket movement config: ${e.message}`);
   }
 
-  // Авто-коррекция тикетов перед выбором задачи
-  console.log('[INFO] Running auto-correction...');
-  const correctionResult = autoCorrectTickets();
+  logger.info('Running auto-correction...');
+  const correctionResult = autoCorrectTickets(movementConfig);
   if (correctionResult.moved.length > 0) {
-    console.log(`[INFO] Auto-corrected ${correctionResult.moved.length} ticket(s)`);
+    logger.info(`Auto-corrected ${correctionResult.moved.length} ticket(s)`);
   }
 
-  console.log(`[INFO] Scanning ready/ directory: ${READY_DIR}`);
+  if (planId) {
+    const closeResult = checkAndClosePlan(WORKFLOW_DIR, planId);
+    if (closeResult.closed) {
+      logger.info(`Plan ${planId} closed: all ${closeResult.total} tickets done`);
+    } else if (closeResult.total > 0) {
+      logger.info(`Plan ${planId} progress: ${closeResult.done}/${closeResult.total} tickets done`);
+    }
+  }
+
+  logger.info(`Scanning ready/ directory: ${READY_DIR}`);
 
   const result = pickNextTicket(planId);
 
   if (result.status === 'found') {
-    console.log(`[INFO] Selected ticket: ${result.ticket_id} (${result.title})`);
-    console.log(`[INFO] Priority: ${result.priority}, Type: ${result.type}`);
+    logger.info(`Selected ticket: ${result.ticket_id} (${result.title})`);
+    logger.info(`Priority: ${result.priority}, Type: ${result.type}`);
   } else {
-    console.log(`[INFO] ${result.reason}`);
+    logger.info(result.reason);
   }
 
-  // Добавляем информацию о авто-коррекции в результат
+  logger.info('Calculating review metrics...');
+  const reviewMetrics = calculateReviewMetrics();
+  logger.info(`Found ${reviewMetrics.tickets_with_reviews} tickets with reviews`);
+  logger.info(`Total failed: ${reviewMetrics.total_failed}, passed: ${reviewMetrics.total_passed}`);
+
+  const metricsDir = path.join(WORKFLOW_DIR, 'metrics');
+  if (!fs.existsSync(metricsDir)) {
+    fs.mkdirSync(metricsDir, { recursive: true });
+  }
+  const metricsFile = path.join(metricsDir, 'review-metrics.json');
+  fs.writeFileSync(metricsFile, JSON.stringify(reviewMetrics, null, 2), 'utf8');
+  logger.info(`Metrics saved to ${metricsFile}`);
+
   const finalResult = {
     ...result,
     auto_corrected: correctionResult.moved.length,
-    moved_tickets: correctionResult.moved.map(m => m.id).join(',')
+    moved_tickets: correctionResult.moved.map(m => m.id).join(','),
+    review_metrics: {
+      tickets_with_reviews: reviewMetrics.tickets_with_reviews,
+      total_failed: reviewMetrics.total_failed,
+      total_passed: reviewMetrics.total_passed,
+      avg_time_to_first_passed_days: reviewMetrics.avg_time_to_first_passed_days,
+      iterations_per_ticket: reviewMetrics.iterations_per_ticket
+    }
   };
 
   printResult(finalResult);
@@ -553,7 +646,7 @@ async function main() {
 }
 
 main().catch(e => {
-  console.error(`[ERROR] ${e.message}`);
+  logger.error(e.message);
   printResult({ status: 'error', error: e.message });
   process.exit(1);
 });

package/src/skills/decompose-gaps/SKILL.md

@@ -63,12 +63,7 @@ description: Декомпозировать недочёты (gaps) из ана
 
 ### 5. Определить тип каждого тикета
 
-| Тип | Префикс | Когда использовать |
-|-----|---------|-------------------|
-| Исправление | FIX | Результат не соответствует ожиданиям |
-| Реализация | IMPL | Задача была пропущена |
-| Ревью | REVIEW | Нужна дополнительная проверка |
-| Документация | DOCS | Не хватает документации |
+Прочитай актуальные типы задач из `.workflow/config/config.yaml` (секция `task_types`). Используй **только** префиксы, определённые в конфиге. Не используй типы, которых нет в конфиге.
 
 ### 6. Назначить приоритеты
 

package/src/skills/decompose-plan/SKILL.md

@@ -39,14 +39,7 @@ description: Декомпозировать план на исполняемые
 
 ### 3. Определить тип каждого тикета
 
-| Тип | Префикс | Описание |
-|-----|---------|----------|
-| Архитектура | ARCH | Создание планов, архитектуры |
-| Реализация | IMPL | Написание кода |
-| Исправление | FIX | Багфиксы |
-| Ревью | REVIEW | Проверка кода/документации |
-| Документация | DOCS | Написание документации |
-| Администрирование | ADMIN | Настройка, конфигурация |
+Прочитай актуальные типы задач из `.workflow/config/config.yaml` (секция `task_types`). Используй **только** префиксы, определённые в конфиге. Не используй типы, которых нет в конфиге.
 
 ### 4. Определить зависимости