workflow-ai 1.0.68 → 1.2.1

package/src/runner.mjs

@@ -1406,6 +1406,38 @@ class StageExecutor {
 class PipelineRunner {
   constructor(config, args) {
     this.config = config;
+    
+    // Validate manual-gate stages in pipeline.yaml at startup
+    if (config.pipeline && config.pipeline.stages) {
+      for (const [stageId, stage] of Object.entries(config.pipeline.stages)) {
+        if (stage.type === 'manual-gate') {
+          // Validate goto.approved
+          if (!stage.goto || !stage.goto.approved) {
+            throw new Error(`pipeline.yaml validation error in stage '${stageId}' (type: manual-gate): missing required 'goto.approved'`);
+          }
+          
+          // Validate goto.rejected
+          if (!stage.goto || !stage.goto.rejected) {
+            throw new Error(`pipeline.yaml validation error in stage '${stageId}' (type: manual-gate): missing required 'goto.rejected'`);
+          }
+          
+          // Validate poll_interval_ms if present
+          if (stage.poll_interval_ms !== undefined) {
+            if (typeof stage.poll_interval_ms !== 'number' || stage.poll_interval_ms < 100) {
+              throw new Error(`pipeline.yaml validation error in stage '${stageId}' (type: manual-gate): poll_interval_ms must be a number >= 100`);
+            }
+          }
+          
+          // Validate timeout_seconds if present
+          if (stage.timeout_seconds !== undefined) {
+            if (typeof stage.timeout_seconds !== 'number' || stage.timeout_seconds <= 0) {
+              throw new Error(`pipeline.yaml validation error in stage '${stageId}' (type: manual-gate): timeout_seconds must be a number > 0`);
+            }
+          }
+        }
+      }
+    }
+    
     this.args = args;
     this.pipeline = config.pipeline;
     this.context = { ...this.pipeline.context };
@@ -1464,36 +1496,266 @@ class PipelineRunner {
     }
   }
 
-  /**
-   * Выполняет встроенный стейдж типа update-counter:
-   * инкрементирует счётчик и возвращает статус для goto-перехода.
-   *
-   * Конфигурация стейджа:
-   *   type: update-counter
-   *   counter: <name>   — имя счётчика
-   *   max: <number>     — максимальное значение (опционально)
-   *   goto:
-   *     default: <stage>           — следующий стейдж
-   *     max_reached: <stage>       — стейдж при достижении max
-   */
-  executeUpdateCounter(stageId, stage) {
-    const counterName = stage.counter;
-    if (!counterName) {
-      throw new Error(`Stage "${stageId}" has type update-counter but no counter specified`);
-    }
+   /**
+    * Вычисляет детерминированный идентификатор шага для approval-файла.
+    * Формат: {ticket_id}_{stageId}_{attempt}
+    *
+    * @param {Object} context - Контекст выполнения пайплайна
+    * @param {string} [context.ticket_id] - ID тикета (может быть undefined)
+    * @param {string} stageId - Идентификатор стадии
+    * @param {Object} counters - Счётчики выполнения
+    * @param {number} [counters.task_attempts] - Номер попытки выполнения (0 если не задано)
+    * @returns {string} Детерминированный step_id
+    */
+   computeStepId(context, stageId, counters) {
+     const ticketId = context.ticket_id || 'no-ticket';
+     const attempt = counters.task_attempts ?? 0;
+     return `${ticketId}_${stageId}_${attempt}`;
+   }
+
+    /**
+     * Создаёт approval-файл со статусом "pending" (идемпотентно).
+     * Если файл уже существует — не перезаписывает, возвращает его содержимое.
+     *
+     * @param {string} filePath - Путь к approval-файлу (абсолютный)
+     * @param {Object} payload - Данные для approval-файла
+     * @param {string} payload.step_id - Идентификатор шага (должен совпадать с именем файла)
+     * @param {string} payload.ticket_id - ID тикета
+     * @param {string} payload.stage_id - ID стадии
+     * @param {number} payload.attempt - Номер попытки
+     * @param {Object} [payload.context_snapshot] - Снапшот контекста (опционально)
+     * @returns {Promise<Object>} Объект approval-файла (прочитанный или созданный)
+     * @throws {Error} При ошибке создания директории или записи файла
+     */
+    async writeApprovalPending(filePath, payload) {
+      // Создаём директорию заранее (recursive, безопасно)
+      const dir = path.dirname(filePath);
+      await fs.promises.mkdir(dir, { recursive: true });
+
+      // Формируем данные approval-файла
+      const now = new Date().toISOString();
+      const approvalData = {
+        step_id: payload.step_id,
+        ticket_id: payload.ticket_id,
+        stage_id: payload.stage_id,
+        attempt: payload.attempt,
+        status: 'pending',
+        created_at: now,
+        updated_at: now,
+        decided_by: null,
+        comment: null,
+        context_snapshot: payload.context_snapshot || {}
+      };
 
-    this.counters[counterName] = (this.counters[counterName] || 0) + 1;
-    const value = this.counters[counterName];
+      const content = JSON.stringify(approvalData, null, 2);
 
-    if (this.logger) {
-      this.logger.info(`Counter "${counterName}" incremented to ${value}`, stageId);
+      // Атомарно создаём файл с флагом 'wx' (fail if exists)
+      try {
+        const handle = await fs.promises.open(filePath, 'wx');
+        await handle.write(content);
+        await handle.close();
+        return approvalData;
+      } catch (err) {
+        if (err.code === 'EEXIST') {
+          // Файл уже был создан (например, при recovery) — читаем и возвращаем
+          const existing = await fs.promises.readFile(filePath, 'utf8');
+          return JSON.parse(existing);
+        }
+        throw err;
+      }
+    }
+
+    /**
+     * Читает approval-файл и парсит его как JSON.
+     *
+     * @param {string} filePath - Путь к approval-файлу (абсолютный)
+     * @returns {Promise<Object|null>} Объект approval-файла или null если файл не существует
+     * @throws {Error} При невалидном JSON — с сообщением "corrupt approval file at {path}: {parse error}"
+     */
+    async readApprovalFile(filePath) {
+      try {
+        const content = await fs.promises.readFile(filePath, 'utf8');
+        return JSON.parse(content);
+      } catch (err) {
+        if (err.code === 'ENOENT') {
+          return null;
+        }
+        if (err instanceof SyntaxError) {
+          throw new Error(`corrupt approval file at ${filePath}: ${err.message}`);
+        }
+        // Перебрасываем другие ошибки (например, fs-ошибки) без изменений
+        throw err;
+      }
+    }
+
+    /**
+     * Выполняет встроенный стейдж типа update-counter:
+     * инкрементирует счётчик и возвращает статус для goto-перехода.
+     *
+     * Конфигурация стейджа:
+     *   type: update-counter
+     *   counter: <name>   — имя счётчика
+     *   max: <number>     — максимальное значение (опционально)
+     *   goto:
+     *     default: <stage>           — следующий стейдж
+     *     max_reached: <stage>       — стейдж при достижении max
+     */
+    executeUpdateCounter(stageId, stage) {
+      const counterName = stage.counter;
+      if (!counterName) {
+        throw new Error(`Stage "${stageId}" has type update-counter but no counter specified`);
+      }
+
+      this.counters[counterName] = (this.counters[counterName] || 0) + 1;
+      const value = this.counters[counterName];
+
+      if (this.logger) {
+        this.logger.info(`Counter "${counterName}" incremented to ${value}`, stageId);
+      }
+
+      const max = stage.max;
+      const status = (max && value >= max) ? 'max_reached' : 'default';
+
+      return { status, result: { counter: counterName, value } };
     }
 
-    const max = stage.max;
-    const status = (max && value >= max) ? 'max_reached' : 'default';
+    /**
+     * Выполняет встроенный стейдж типа manual-gate: создаёт approval-файл
+     * и ждёт решения (approved/rejected) через polling файловой системы.
+     *
+     * Конфигурация стейджа:
+     *   type: manual-gate
+     *   poll_interval_ms: <number>  — интервал опроса в мс (опц., default 2000)
+     *   timeout_seconds: <number>   — таймаут в секундах (опц., default null)
+     *   goto:
+     *     approved: <stage>         — обязательный, при одобрении
+     *     rejected: <stage>         — обязательный, при отклонении
+     *     timeout: <stage>          — опц., при истечении таймаута
+     *     aborted: <stage>          — опц., при остановке runner'а
+     *
+     * @param {string} stageId - ID стадии
+     * @param {object} stage - конфигурация стадии
+     * @returns {Promise<{status: 'approved'|'rejected'|'timeout'|'aborted', result: object}>}
+     */
+    async executeManualGate(stageId, stage) {
+      const stepId = this.computeStepId(this.context, stageId, this.counters);
+      const filePath = path.join(this.projectRoot, '.workflow', 'approvals', `${stepId}.json`);
+
+      const payload = {
+        step_id: stepId,
+        ticket_id: this.context.ticket_id || 'no-ticket',
+        stage_id: stageId,
+        attempt: this.counters.task_attempts || 0,
+        context_snapshot: { ...this.context }
+      };
 
-    return { status, result: { counter: counterName, value } };
-  }
+      const existing = await this.writeApprovalPending(filePath, payload);
+
+      // Если файл уже был и статус уже resolved — сразу возвращаем результат (recovery)
+      if (existing.status === 'approved' || existing.status === 'rejected') {
+        if (this.logger) {
+          this.logger.info(
+            `[${stageId}] manual-gate: already ${existing.status} by ${existing.decided_by || 'unknown'}${existing.comment ? `, comment="${existing.comment}"` : ''}`,
+            stageId
+          );
+        }
+        return {
+          status: existing.status,
+          result: {
+            step_id: stepId,
+            decided_by: existing.decided_by,
+            comment: existing.comment
+          }
+        };
+      }
+
+      // Создан новый pending-файл
+      if (this.logger) {
+        this.logger.info(`[${stageId}] manual-gate: created pending approval at ${filePath}`, stageId);
+      }
+
+      const pollIntervalMs = stage.poll_interval_ms || 2000;
+      const timeoutSeconds = stage.timeout_seconds || null;
+      const startTime = Date.now();
+
+      // Polling-цикл
+      while (this.running) {
+        await new Promise(resolve => setTimeout(resolve, pollIntervalMs));
+
+        // Проверка остановки runner'а
+        if (!this.running) {
+          if (this.logger) {
+            this.logger.warn(`[${stageId}] manual-gate: aborted (runner stopped)`, stageId);
+          }
+          return { status: 'aborted', result: { step_id: stepId } };
+        }
+
+        // Проверка таймаута
+        if (timeoutSeconds !== null) {
+          const elapsed = (Date.now() - startTime) / 1000;
+          if (elapsed >= timeoutSeconds) {
+            if (this.logger) {
+              this.logger.warn(`[${stageId}] manual-gate: timeout after ${timeoutSeconds}s, no decision`, stageId);
+            }
+            return { status: 'timeout', result: { step_id: stepId } };
+          }
+        }
+
+        // Читаем текущее состояние файла
+        let data;
+        try {
+          data = await this.readApprovalFile(filePath);
+        } catch (err) {
+          // Corrupt JSON — пробрасываем ошибку для goto.error
+          throw err;
+        }
+
+        if (data && data.status === 'approved') {
+          if (this.logger) {
+            this.logger.info(
+              `[${stageId}] manual-gate: approved by ${data.decided_by || 'unknown'}${data.comment ? `, comment="${data.comment}"` : ''}`,
+              stageId
+            );
+          }
+          return {
+            status: 'approved',
+            result: {
+              step_id: stepId,
+              decided_by: data.decided_by,
+              comment: data.comment
+            }
+          };
+        }
+
+        if (data && data.status === 'rejected') {
+          if (this.logger) {
+            this.logger.info(
+              `[${stageId}] manual-gate: rejected by ${data.decided_by || 'unknown'}${data.comment ? `, comment="${data.comment}"` : ''}`,
+              stageId
+            );
+          }
+          return {
+            status: 'rejected',
+            result: {
+              step_id: stepId,
+              decided_by: data.decided_by,
+              comment: data.comment
+            }
+          };
+        }
+
+        // DEBUG-лог polling (оставляем только если нужен)
+        if (this.logger && this.logger.level === 'debug') {
+          this.logger.debug(`[${stageId}] manual-gate: polling, current status=pending`, stageId);
+        }
+      }
+
+      // Выход по this.running = false
+      if (this.logger) {
+        this.logger.warn(`[${stageId}] manual-gate: aborted (runner stopped)`, stageId);
+      }
+      return { status: 'aborted', result: { step_id: stepId } };
+    }
 
   /**
    * Запускает основной цикл выполнения
@@ -1528,16 +1790,18 @@ class PipelineRunner {
           throw new Error(`Stage not found: ${this.currentStage}`);
         }
 
-        let result;
+         let result;
 
-        // Встроенный тип стейджа: update-counter — инкрементирует счётчик без вызова агента
-        if (stage.type === 'update-counter') {
-          result = this.executeUpdateCounter(this.currentStage, stage);
-        } else {
-          this.currentExecutor = new StageExecutor(this.config, this.context, this.counters, {}, this.fileGuard, this.logger, this.projectRoot);
-          result = await this.currentExecutor.execute(this.currentStage);
-          this.currentExecutor = null;
-        }
+         // Встроенные типы стейджа — выполняются без вызова внешних агентов
+         if (stage.type === 'update-counter') {
+           result = this.executeUpdateCounter(this.currentStage, stage);
+         } else if (stage.type === 'manual-gate') {
+           result = await this.executeManualGate(this.currentStage, stage);
+         } else {
+           this.currentExecutor = new StageExecutor(this.config, this.context, this.counters, {}, this.fileGuard, this.logger, this.projectRoot);
+           result = await this.currentExecutor.execute(this.currentStage);
+           this.currentExecutor = null;
+         }
 
         this.logger.info(`Stage ${this.currentStage} completed with status: ${result.status}`, 'PipelineRunner');
 
@@ -1843,6 +2107,22 @@ function validateConfig(config) {
         errors.push(`Stage "${stageId}" references non-existent agent: ${resolvedAgent}`);
       }
 
+      // Валидация для manual-gate стадии
+      if (stage.type === 'manual-gate') {
+        if (!stage.goto || !stage.goto.approved) {
+          errors.push(`Stage "${stageId}" has type manual-gate but missing required goto.approved`);
+        }
+        if (!stage.goto || !stage.goto.rejected) {
+          errors.push(`Stage "${stageId}" has type manual-gate but missing required goto.rejected`);
+        }
+        if (stage.poll_interval_ms !== undefined && (typeof stage.poll_interval_ms !== 'number' || stage.poll_interval_ms < 100)) {
+          errors.push(`Stage "${stageId}" has invalid poll_interval_ms: must be a number >= 100`);
+        }
+        if (stage.timeout_seconds !== undefined && (typeof stage.timeout_seconds !== 'number' || stage.timeout_seconds <= 0)) {
+          errors.push(`Stage "${stageId}" has invalid timeout_seconds: must be a number > 0`);
+        }
+      }
+
       if (stage.goto) {
         for (const [status, transition] of Object.entries(stage.goto)) {
           if (status === 'default') continue;

package/src/scripts/check-conditions.js

@@ -46,10 +46,10 @@ function checkCondition(condition) {
 
   switch (type) {
     case 'file_exists':
-      return fs.existsSync(path.join(PROJECT_DIR, value));
+      return fs.existsSync(path.isAbsolute(value) ? value : path.join(PROJECT_DIR, value));
 
     case 'file_not_exists':
-      return !fs.existsSync(path.join(PROJECT_DIR, value));
+      return !fs.existsSync(path.isAbsolute(value) ? value : path.join(PROJECT_DIR, value));
 
     case 'tasks_completed': {
       if (!value || (Array.isArray(value) && value.length === 0)) return true;

package/src/scripts/get-next-id.js

@@ -21,6 +21,7 @@ import fs from "fs";
 import path from "path";
 import { findProjectRoot } from "workflow-ai/lib/find-root.mjs";
 import { printResult } from "workflow-ai/lib/utils.mjs";
+import { getNextId } from "workflow-ai/lib/operations/tickets.mjs";
 
 const PROJECT_DIR = findProjectRoot();
 
@@ -77,15 +78,6 @@ function findMaxNumber(targetDir, prefix) {
   return maxNum;
 }
 
-function formatNumber(num) {
-  return num.toString().padStart(3, "0");
-}
-
-/**
- * Извлекает значения prefix из .workflow/config/config.yaml → task_types.*.prefix.
- * Минимальный YAML-парсер: не тянем зависимость, читаем только нужную секцию.
- * Возвращает массив уникальных префиксов в порядке появления.
- */
 function readPrefixesFromConfig() {
   const configPath = path.join(PROJECT_DIR, ".workflow", "config", "config.yaml");
 
@@ -93,7 +85,7 @@ function readPrefixesFromConfig() {
     throw new Error(`config.yaml not found: ${configPath}`);
   }
 
-  const text = fs.readFileSync(configPath, "utf8");
+  const text = fs.readFileSync(configPath, "utf8");
   const lines = text.split(/\r?\n/);
 
   const prefixes = [];
@@ -136,30 +128,15 @@ function readPrefixesFromConfig() {
   return prefixes;
 }
 
-function resolveTargetDir(dir) {
-  if (dir === "tickets") {
-    return path.join(PROJECT_DIR, ".workflow", "tickets");
-  } else if (dir === "plans") {
-    return path.join(PROJECT_DIR, ".workflow", "plans");
-  } else {
-    return path.join(PROJECT_DIR, dir);
-  }
-}
-
 async function runSinglePrefix(prefix, dir) {
-  const targetDir = resolveTargetDir(dir);
-
-  if (!fs.existsSync(targetDir)) {
-    const nextId = `${prefix}-001`;
+  try {
+    const nextId = await getNextId(PROJECT_DIR, prefix, { dir });
     printResult({ status: "success", id: nextId });
-    return;
+  } catch (err) {
+    console.error(err.message);
+    printResult({ status: "error", error: err.message });
+    process.exit(1);
   }
-
-  const maxNum = findMaxNumber(targetDir, prefix);
-  const nextNum = maxNum + 1;
-  const nextId = `${prefix}-${formatNumber(nextNum)}`;
-
-  printResult({ status: "success", id: nextId });
 }
 
 async function runAllFromConfig() {

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

@@ -48,11 +48,11 @@ function checkCondition(condition) {
 
   switch (type) {
     case 'file_exists':
-      const filePath = path.join(PROJECT_DIR, value);
+      const filePath = path.isAbsolute(value) ? value : path.join(PROJECT_DIR, value);
       return fs.existsSync(filePath);
 
     case 'file_not_exists':
-      const filePath2 = path.join(PROJECT_DIR, value);
+      const filePath2 = path.isAbsolute(value) ? value : path.join(PROJECT_DIR, value);
       return !fs.existsSync(filePath2);
 
     case 'tasks_completed':
@@ -768,13 +768,7 @@ async function main() {
     ...result,
     auto_corrected: correctionResult.moved.length,
     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
-    }
+    review_metrics: JSON.stringify(reviewMetrics)
   };
 
   printResult(finalResult);

package/src/skills/coach/SKILL.md

@@ -148,7 +148,7 @@ ticket_prefix: COACH
 Если на (1), (2) или (4) ответ «да» — замени на общую формулировку или критерий, либо перенеси в shared knowledge проекта. Правка должна быть применима к любому проекту, использующему этот скил.
 
 **Дополнительная проверка на копирование старого текста:** если правка **заменяет** существующий блок, не воспроизводи автоматически терминологию исходника. Pre-existing нарушения в заменяемом тексте — твоя ответственность; молча скопировав их в новый текст, ты унаследуешь нарушения и они станут частью твоей правки. Прогон чеклиста по 4 вопросам обязателен **по новому тексту целиком**, а не только по дельте «новое - старое».
-10. **Self-Correct** — после каждой правки в скил **обязательно** перечитай принципы коуча (1-12) и проверь, не нарушает ли внесённая правка какой-либо из них. Не жди указания стейкхолдера — проверяй проактивно. **Процедура:** после каждого Edit/Write в файл скила — прежде чем отвечать пользователю — (1) перечитай `Read` записанный файл, (2) пройди по каждому принципу 1-12, (3) если нарушение найдено — исправь сразу, не дожидаясь фидбека. **(4) Проверка доставки:** если правка добавляет знание в shared или knowledge — открой SKILL.md целевого скила и убедись, что файл загружается по триггеру текущей задачи. Если триггер загрузки отсутствует или слишком слабый (последняя строка таблицы, без ⛔, без «обязательно») — **усиль триггер в том же CHG**, иначе правка не дойдёт до агента-исполнителя. Когда стейкхолдер указывает на ошибку — это сигнал, что проактивная проверка не сработала: исправь не только текущую работу и целевой скил, но и усиль собственные инструкции коуча, чтобы ошибка не повторялась.
+10. **Self-Correct** — после каждой правки в скил **обязательно** перечитай принципы коуча (1-12) и проверь, не нарушает ли внесённая правка какой-либо из них. Не жди указания стейкхолдера — проверяй проактивно. **Процедура:** после каждого Edit/Write в файл скила — прежде чем отвечать пользователю — (1) перечитай `Read` записанный файл, (2) пройди по каждому принципу 1-12, (3) если нарушение найдено — исправь сразу, не дожидаясь фидбека. **(4) Проверка доставки:** если правка добавляет знание в shared или knowledge — открой SKILL.md целевого скила и убедись, что файл загружается по триггеру текущей задачи. Если триггер загрузки отсутствует или слишком слабый (последняя строка таблицы, без ⛔, без «обязательно») — **усиль триггер в том же CHG**, иначе правка не дойдёт до агента-исполнителя. **(5) Тест:** после проверки принципов 1-12 **обязательно** выполни секцию «Обязательный шаг: Тест» (→ `workflows/test.md`). Self-check без теста — незавершённый self-check: принципы проверяют корректность текста, тест проверяет корректность поведения агента при чтении этого текста. Когда стейкхолдер указывает на ошибку — это сигнал, что проактивная проверка не сработала: исправь не только текущую работу и целевой скил, но и усиль собственные инструкции коуча, чтобы ошибка не повторялась.
 11. **Context Budget** — при анализе и аудите скила обязательно оценивай его **суммарный размер** (SKILL.md + все файлы из knowledge/ + algorithms/ + workflows/). Агент загружает эти файлы в контекст перед работой. **Порог:** если суммарный размер > 800 строк — это finding уровня HIGH. Каждая правка, добавляющая текст, должна оцениваться: «Не приведёт ли это к context overflow у агента-исполнителя?» **При аудите:** измерь `wc -l` всех файлов скила, укажи суммарный размер и сравни с порогом. Если превышен — рекомендуй консолидацию: объединение дублирующих секций, вынос редко используемых блоков в отдельные файлы с ленивой загрузкой (загружать только по триггеру, а не всегда), сжатие примеров.
 12. **Consistency** — скил не должен содержать взаимоисключающих или противоречащих друг другу инструкций. При каждой правке и аудите проверяй: не конфликтует ли новая инструкция с существующими. **Процедура:** после внесения правки grep'ни файл на ключевые термины правки и прочитай все совпадения — убедись, что нигде не сказано противоположное. **Типичные противоречия:** «всегда делай X» в одном месте и «никогда не делай X» в другом; разные значения по умолчанию для одного параметра; разные приоритеты действий в SKILL.md и в workflow. При обнаружении противоречия — устрани его сразу, выбрав одну версию и обновив все места.
 

package/src/skills/manual-testing/SKILL.md

@@ -119,6 +119,8 @@ ticket_prefix: QA
 
 Перед сохранением убедись, что директория `reports/` существует. Если нет — создай её.
 
+**⚠️ Переиспользование существующего теста (discovery вариант C):** если тест уже существует и сохраняет артефакты с hardcoded prefix другого тикета — **обнови prefix на ID текущего тикета в тест-файле перед запуском**. Ручное копирование/переименование файлов после запуска — антипаттерн: создаёт гонку mtime между артефактами и `updated_at` тикета, ломает artifact-snapshot diff верификатора.
+
 ### Cleanup evidence
 
 После завершения тестирования в `reports/` должны остаться **только файлы, на которые ссылается тикет**. Удали промежуточные и отладочные файлы (пробные скриншоты, дубликаты, скриншоты с неверным содержимым). Имя файла evidence должно соответствовать TC, для которого он используется — если evidence для TC-001 фактически сохранён как `TC-002-*.png`, переименуй или пересохрани с корректным именем.

package/src/skills/review-result/SKILL.md

@@ -36,6 +36,7 @@ description: >
 |--------|----------------|
 | `knowledge/dod-patterns.md` | При определении типа проверки для конкретного пункта DoD |
 | `knowledge/test-hygiene.md` | Когда DoD требует создать/изменить автотест — красные и зелёные флаги изоляции, что считать нарушением и как оформить issue |
+| `knowledge/baseline-snapshot-validation.md` | **⛔ Обязательно**, когда DoD/план тикета содержит формулировки уровня эталона: baseline, snapshot, inline-snapshot, побайтное/точное совпадение, регрессионный эталон, «output не должен меняться». Проверяет способ реализации теста (точное сравнение с литералом-эталоном vs регекс/includes), а не только зелёный прогон |
 | `../shared/*` | **ВСЕГДА** перед началом работы — общие знания проекта |
 
 ## Загрузка шаблонов

package/src/scripts/tests/timeout-cascade.test.js

@@ -1,28 +0,0 @@
-function parseTimeout(testCase, index) {
-  return testCase.execution?.timeout_s || index.execution?.default_timeout_s || 300;
-}
-
-function testTimeoutCascade() {
-  const index = { execution: { default_timeout_s: 300 } };
-  
-  const testCases = [
-    { name: 'TC-001', execution: { timeout_s: 600 } },
-    { name: 'TC-002', execution: {} },
-    { name: 'TC-003' }
-  ];
-  
-  const results = testCases.map(tc => ({
-    case: tc.name,
-    timeout: parseTimeout(tc, index)
-  }));
-  
-  console.log('Timeout cascade results:', results);
-  
-  const expected = [600, 300, 300];
-  const passed = results.every((r, i) => r.timeout === expected[i]);
-  
-  console.log(`Test ${passed ? 'PASSED' : 'FAILED'}`);
-  process.exit(passed ? 0 : 1);
-}
-
-testTimeoutCascade();

package/src/skills/analyze-report/README.md

@@ -1,44 +0,0 @@
-# Analyze Report — Agent Skill
-
-Аналитик отчётов для workflow-ai. Анализирует отчёты о выполненных задачах, оценивает прогресс по планам, проводит ретроспективы и формирует actionable рекомендации.
-
-## Структура
-
-```
-analyze-report/
-├── SKILL.md                              # Ядро: роль, маршрутизация, принципы
-├── README.md                             # Документация
-├── workflows/
-│   ├── progress.md                       # Оценка прогресса по плану
-│   └── retrospective.md                  # Ретроспектива завершённого плана
-├── knowledge/
-│   ├── analysis-frameworks.md            # Методологии анализа, метрики, паттерны
-│   └── report-structure.md               # Структура отчётов, правила извлечения данных
-├── algorithms/
-│   └── progress-assessment.md            # Алгоритм оценки прогресса (формулы, пороги)
-└── templates/
-    └── analysis-report.md                # Шаблон результата анализа
-```
-
-## Как это работает
-
-1. Любой скил проекта создаёт тикет `ANL-*` с запросом на анализ
-2. Analyze Report определяет тип (PROGRESS / RETROSPECTIVE)
-3. Загружает соответствующий workflow
-4. Собирает данные из отчётов, тикетов, планов
-5. Рассчитывает метрики, выявляет проблемы, формулирует рекомендации
-6. Формирует структурированный результат анализа
-
-## Как расширять
-
-### Новый тип анализа
-1. Создай файл в `workflows/{type}.md`
-2. Добавь запись в таблицу маршрутизации в `SKILL.md`
-
-### Новый knowledge-модуль
-1. Создай файл в `knowledge/{module}.md`
-2. Добавь запись в таблицу загрузки знаний в `SKILL.md`
-
-### Новый шаблон вывода
-1. Создай файл в `templates/{template}.md`
-2. Добавь запись в таблицу шаблонов в `SKILL.md`