workflow-ai 1.3.0 → 1.5.0

package/CHANGELOG.md

@@ -1,61 +1,73 @@
-## [1.3.0] — 2026-04-30
-
-### Добавлено
-- **Новый тип стейджа: `mark-blocked` и расширение frontmatter** — Добавлены поля `auto_blocked_reason`, `auto_blocked_attempts`, `auto_blocked_at` для автоматического отслеживания причин и попыток блокировки тикетов. Стейдж `mark-blocked` позволяет устанавливать эти поля в зависимости от условий бизнес-логики.
-- **Новый тип стейджа: `manual-gate-human`** — Добавлен статус `human_ready` в `pick-next-task.js`. При достижении этого статуса задача ожидает ручного решения оператора перед продолжением выполнения.
-- **Хук одобрения в `move-ticket.js`** — Добавлен approval-hook, проверяющий наличие ожидающих решения одобрений перед перемещением тикета в терминальные состояния.
-- **Расширение START-лога полем `ticket="X"`** — Добавлено поле с идентификатором тикета в стартовый лог для улучшенной трассировки и связывания логов с конкретными задачами.
-- **Исправление `approval-pending.mjs` (поле `created_at`)** — Исправлено некорректное заполнение поля `created_at` при создании файлов ожидающих решения.
-
-> **Примечание:** версия 1.2.0 не была опубликована в npm. При выполнении `npm publish`
-> сработал prepublishOnly/postversion скрипт, автоматически поднявший версию до 1.2.1.
-> Git-тег `v1.2.0` (коммит 179d52b) соответствует состоянию до publish; `v1.2.1` (коммит 83d1b70) —
-> фактически опубликованной версии.
-
-### New Features
-- **New built-in stage type: `manual-gate`** — Adds support for manual approval steps in pipelines. When a stage with `type: manual-gate` is encountered, the runner creates a pending approval file in `.workflow/approvals/{step_id}.json` and enters a polling loop, waiting for an external decision (`approved`/`rejected`).
-
-  **Key capabilities:**
-  - Deterministic `step_id` generation: `{ticket_id}_{stageId}_{attempt}` (e.g., `QA-12_manual-approve_0`)
-  - Idempotent file creation — pending file is not overwritten on retry/restart
-  - Configurable polling interval (`poll_interval_ms`, default 2000ms) and optional timeout (`timeout_seconds`)
-  - Graceful handling of SIGTERM/runner stop (returns `aborted`)
-  - Immediate return if file already has `approved`/`rejected` status (crash recovery)
-  - JSON approval file format with full audit trail (`created_at`, `updated_at`, `decided_by`, `comment`, `context_snapshot`)
-
-  **Pipeline configuration example:**
-  ```yaml
-  stages:
-    manual-approve-deploy:
-      type: manual-gate
-      poll_interval_ms: 2000
-      timeout_seconds: 86400
-      goto:
-        approved: continue-deploy
-        rejected: rollback
-        timeout: notify-stuck
-        aborted: end
-  ```
-
-  **Two approval methods (both opt-in):**
-  1. **External MCP/client**: tools like `workflow-mcp` can write to approval files programmatically
-  2. **Direct file edit**: users can simply edit `.workflow/approvals/{step_id}.json` and change `status` to `approved` or `rejected`
-
-  **Important:** `manual-gate` is **opt-in** — pipelines without such stages work identically to previous versions. No breaking changes.
-
-### Changed
-- No breaking changes. All existing pipelines without `manual-gate` stages are fully backward compatible.
-
-### Fixes
-- Исправлено сохранение временной метки `created_at` в файлах одобрений (approval-pending.mjs)
-
-### Technical Notes
-- Approval files are stored in `<project_root>/.workflow/approvals/`
-- Runner validates `manual-gate` stages on startup — requires `goto.approved` and `goto.rejected`, validates numeric parameters
-- New methods added to `PipelineRunner`: `computeStepId()`, `writeApprovalPending()`, `readApprovalFile()`, `executeManualGate()`
-
-### References
-- PLAN-009: workflow-ai 1.2 — manual-gate stage and approval files for workflow-mcp Sprint 2 integration
-- IMPL-55, IMPL-56, IMPL-57, IMPL-58: Implementation tickets
-- QA-35, QA-36, QA-37: Test coverage
-- IMPL-51, QA-55 in workflow-mcp: Dependent consumer work
+## [1.5.0] — 2026-05-02
+
+### Added
+- **Audit log агентов в тикетах**: каждая попытка агента (включая fallback) пишет строку в секцию `## История работы` тикета: `| Дата/время | Скил | Агент | Статус |`. 10 классов статуса (ok, error, timeout, empty_response, rate_limit, network_error, auth_error, aborted, blocked, skipped_relevance) детектируются автоматически.
+- **Колонка Агент в `## Ревью`**: миграция 3→4 колонки. Видно кто проставил вердикт.
+- **Agent history aggregation в metrics**: ключ `agent_history` в `metrics/review-metrics.json` с разрезами by_status/by_agent/by_skill/by_skill_by_agent + fallback_stats. Incremental update от runner.
+- New helpers: `lib/agent-history.mjs`, `lib/review-section.mjs`, `lib/metrics-incremental.mjs`.
+
+### Changed
+- `getLastReviewStatus` parsing теперь header-based (whitelist: Статус/Status/Вердикт/Verdict). Поддержка legacy 3-колоночных таблиц через `unknown` placeholder в Agent при first append.
+- `verify-artifacts.js`, `move-ticket.js` fallback, `check-relevance.js` теперь пишут review через `appendReviewEntry` helper.
+
+## [1.3.0] — 2026-04-30
+
+### Добавлено
+- **Новый тип стейджа: `mark-blocked` и расширение frontmatter** — Добавлены поля `auto_blocked_reason`, `auto_blocked_attempts`, `auto_blocked_at` для автоматического отслеживания причин и попыток блокировки тикетов. Стейдж `mark-blocked` позволяет устанавливать эти поля в зависимости от условий бизнес-логики.
+- **Новый тип стейджа: `manual-gate-human`** — Добавлен статус `human_ready` в `pick-next-task.js`. При достижении этого статуса задача ожидает ручного решения оператора перед продолжением выполнения.
+- **Хук одобрения в `move-ticket.js`** — Добавлен approval-hook, проверяющий наличие ожидающих решения одобрений перед перемещением тикета в терминальные состояния.
+- **Расширение START-лога полем `ticket="X"`** — Добавлено поле с идентификатором тикета в стартовый лог для улучшенной трассировки и связывания логов с конкретными задачами.
+- **Исправление `approval-pending.mjs` (поле `created_at`)** — Исправлено некорректное заполнение поля `created_at` при создании файлов ожидающих решения.
+
+> **Примечание:** версия 1.2.0 не была опубликована в npm. При выполнении `npm publish`
+> сработал prepublishOnly/postversion скрипт, автоматически поднявший версию до 1.2.1.
+> Git-тег `v1.2.0` (коммит 179d52b) соответствует состоянию до publish; `v1.2.1` (коммит 83d1b70) —
+> фактически опубликованной версии.
+
+### New Features
+- **New built-in stage type: `manual-gate`** — Adds support for manual approval steps in pipelines. When a stage with `type: manual-gate` is encountered, the runner creates a pending approval file in `.workflow/approvals/{step_id}.json` and enters a polling loop, waiting for an external decision (`approved`/`rejected`).
+
+  **Key capabilities:**
+  - Deterministic `step_id` generation: `{ticket_id}_{stageId}_{attempt}` (e.g., `QA-12_manual-approve_0`)
+  - Idempotent file creation — pending file is not overwritten on retry/restart
+  - Configurable polling interval (`poll_interval_ms`, default 2000ms) and optional timeout (`timeout_seconds`)
+  - Graceful handling of SIGTERM/runner stop (returns `aborted`)
+  - Immediate return if file already has `approved`/`rejected` status (crash recovery)
+  - JSON approval file format with full audit trail (`created_at`, `updated_at`, `decided_by`, `comment`, `context_snapshot`)
+
+  **Pipeline configuration example:**
+  ```yaml
+  stages:
+    manual-approve-deploy:
+      type: manual-gate
+      poll_interval_ms: 2000
+      timeout_seconds: 86400
+      goto:
+        approved: continue-deploy
+        rejected: rollback
+        timeout: notify-stuck
+        aborted: end
+  ```
+
+  **Two approval methods (both opt-in):**
+  1. **External MCP/client**: tools like `workflow-mcp` can write to approval files programmatically
+  2. **Direct file edit**: users can simply edit `.workflow/approvals/{step_id}.json` and change `status` to `approved` or `rejected`
+
+  **Important:** `manual-gate` is **opt-in** — pipelines without such stages work identically to previous versions. No breaking changes.
+
+### Changed
+- No breaking changes. All existing pipelines without `manual-gate` stages are fully backward compatible.
+
+### Fixes
+- Исправлено сохранение временной метки `created_at` в файлах одобрений (approval-pending.mjs)
+
+### Technical Notes
+- Approval files are stored in `<project_root>/.workflow/approvals/`
+- Runner validates `manual-gate` stages on startup — requires `goto.approved` and `goto.rejected`, validates numeric parameters
+- New methods added to `PipelineRunner`: `computeStepId()`, `writeApprovalPending()`, `readApprovalFile()`, `executeManualGate()`
+
+### References
+- PLAN-009: workflow-ai 1.2 — manual-gate stage and approval files for workflow-mcp Sprint 2 integration
+- IMPL-55, IMPL-56, IMPL-57, IMPL-58: Implementation tickets
+- QA-35, QA-36, QA-37: Test coverage
+- IMPL-51, QA-55 in workflow-mcp: Dependent consumer work

package/README.md

@@ -2,6 +2,33 @@
 - Добавлено упоминание `human-gate` в разделе про runner-стадии (пример конфига в pipeline.yaml: human-review-step типа manual-gate).
 - Добавлено упоминание `mark-blocked` как механизм нотификации autoblocked-тикетов (поля auto_blocked_reason/attempts/at).
 
+## Синглтон семантика Pipeline
+
+Система workflow реализует паттерн синглтона для выполнения пайплайна: одновременно разрешена только одна активная инстанция пайплайна для каждого проекта.
+
+### Семантика Синглтона
+При попытке запустить второй пайплайн, когда первый уже работает, система вернёт ошибку `PIPELINE_ALREADY_RUNNING`.
+
+### Структура `.workflow/logs/.pipeline.lock`
+Файл блокировки содержит следующие поля:
+- `pid`: ID процесса работающего пайплайна
+- `started_at`: Временная метка запуска пайплайна
+- `started_by`: Инициатор запуска пайплайна (cli | mcp | extension)
+- `run_id`: Уникальный идентификатор этого запуска
+- `pipeline_log`: Путь к файлу логов пайплайна
+- `project_root`: Корневая директория проекта
+- `pipeline_version`: Версия выполняемого пайплайна
+
+### Команды
+- `workflow run --project <path> [--started-by cli|mcp|extension] [--force]` - Запустить пайплайн для указанного проекта
+- `workflow stop --project <path> [--grace-sec N]` - Мягко остановить пайплайн (SIGTERM → SIGKILL после N секунд)
+- `--force` флаг - Обход блокировки (только для отладки, когда процесс зависает)
+
+### Восстановление
+Если файл блокировки устарел (процесс упал, но блокировка не удалена):
+1. Проверь, что процесс мёртв: `workflow stop --project <path>` (автоматически детектит устаревшие блокировки)
+2. Если стандартная остановка не решает проблему: `workflow run --force --project <path>`
+
 ## Runner-стадии
 
 В системе поддерживаются различные типы стадий для управления процессами разработки:

package/package.json

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

package/src/lib/marker.mjs

@@ -0,0 +1,108 @@
+import fs from 'fs';
+import path from 'path';
+
+const MARKER_FILE = '.pipeline.lock';
+const LOGS_DIR = '.workflow/logs';
+
+/**
+ * Ensures the logs directory exists
+ */
+function ensureLogsDir(projectRoot) {
+  const logsPath = path.join(projectRoot, LOGS_DIR);
+  if (!fs.existsSync(logsPath)) {
+    fs.mkdirSync(logsPath, { recursive: true });
+  }
+}
+
+/**
+ * Atomic write via temp file + rename.
+ * Fallback to O_EXCL (wx flag) on EXDEV/ENOTSUP.
+ * Throws Error if marker file already exists (to prevent race conditions).
+ */
+export function writeMarker(projectRoot, payload) {
+  ensureLogsDir(projectRoot);
+  const markerPath = path.join(projectRoot, LOGS_DIR, MARKER_FILE);
+  const content = JSON.stringify(payload, null, 2);
+
+  // First, try exclusive create (O_EXCL) - this guarantees atomicity
+  // and prevents race conditions where two processes both think they created the marker
+  try {
+    const fd = fs.openSync(markerPath, 'wx');
+    fs.writeFileSync(fd, content, 'utf-8');
+    fs.closeSync(fd);
+    return;
+  } catch (openErr) {
+    if (openErr.code === 'EEXIST') {
+      // Marker already exists - another process created it
+      throw new Error(`Marker file already exists at ${markerPath}`);
+    }
+    // If EXDEV/ENOTSUP (cross-device link), fall back to temp+rename
+    if (openErr.code !== 'EXDEV' && openErr.code !== 'ENOTSUP') {
+      throw openErr;
+    }
+  }
+
+  // Fallback: temp file + rename (less atomic but works across devices)
+  const tempPath = markerPath + '.tmp.' + process.pid + '.' + Date.now();
+  try {
+    fs.writeFileSync(tempPath, content, 'utf-8');
+    fs.renameSync(tempPath, markerPath);
+    return;
+  } catch (renameErr) {
+    // Cleanup temp file if it still exists
+    try {
+      fs.unlinkSync(tempPath);
+    } catch {
+      // ignore
+    }
+
+    // If rename failed because marker was created by another process in the meantime
+    if (renameErr.code === 'EEXIST' || !fs.existsSync(tempPath)) {
+      throw new Error(`Marker file already exists at ${markerPath}`);
+    }
+
+    throw renameErr;
+  }
+}
+
+/**
+ * Reads and parses marker file. Returns null if file doesn't exist or is invalid.
+ */
+export function readMarker(projectRoot) {
+  const markerPath = path.join(projectRoot, LOGS_DIR, MARKER_FILE);
+  try {
+    const data = fs.readFileSync(markerPath, 'utf-8');
+    return JSON.parse(data);
+  } catch (err) {
+    // ENOENT (file missing) or SyntaxError (invalid JSON) → return null
+    return null;
+  }
+}
+
+/**
+ * Silently removes marker file. Ignores ENOENT.
+ */
+export function removeMarker(projectRoot) {
+  const markerPath = path.join(projectRoot, LOGS_DIR, MARKER_FILE);
+  try {
+    fs.unlinkSync(markerPath);
+  } catch (err) {
+    // Ignore ENOENT — file already gone
+    if (err.code !== 'ENOENT') {
+      // Log warning but don't throw — silent unlink
+      console.warn(`[marker] failed to remove ${markerPath}: ${err.message}`);
+    }
+  }
+}
+
+/**
+ * Validates that marker exists and its pid matches expectedPid.
+ * Returns true only if both conditions hold, false otherwise.
+ */
+export function validateMarker(projectRoot, expectedPid) {
+  const marker = readMarker(projectRoot);
+  if (!marker) {
+    return false;
+  }
+  return marker.pid === expectedPid;
+}

package/src/lib/process-alive.mjs

@@ -0,0 +1,11 @@
+export function processAlive(pid) {
+  if (!Number.isInteger(pid) || pid <= 0) return false;
+  try {
+    process.kill(pid, 0); // signal 0: existence check, не убивает процесс
+    return true;
+  } catch (err) {
+    // ESRCH = no process; EPERM = process exists but no permission
+    if (err.code === 'EPERM') return true;
+    return false;
+  }
+}

package/src/lib/stop-command.mjs

@@ -0,0 +1,82 @@
+import { readMarker, removeMarker } from './marker.mjs';
+import { processAlive } from './process-alive.mjs';
+import { execSync } from 'node:child_process';
+import { existsSync } from 'node:fs';
+
+/**
+ * Sleep helper
+ */
+function sleep(ms) {
+  return new Promise(resolve => setTimeout(resolve, ms));
+}
+
+/**
+ * Gracefully stops a running pipeline.
+ *
+ * @param {string} projectRoot - Absolute path to project root
+ * @param {object} options - Options
+ * @param {number} [options.graceSec=10] - Grace period in seconds before SIGKILL
+ * @returns {object} Result object
+ *   - { ok: true, pid: number, escalated: boolean, duration_ms: number } on success
+ *   - { ok: true, was_stale: true } if marker existed but process was dead
+ *   - { ok: false, code: 'NOT_RUNNING' } if no marker found
+ */
+export async function stopPipeline(projectRoot, options = {}) {
+  const graceSec = options.graceSec ?? 10;
+  const marker = readMarker(projectRoot);
+
+  if (!marker) {
+    return { ok: false, code: 'NOT_RUNNING' };
+  }
+
+  if (!processAlive(marker.pid)) {
+    removeMarker(projectRoot);
+    return { ok: true, was_stale: true };
+  }
+
+  const startMs = Date.now();
+  const isWindows = process.platform === 'win32';
+
+  if (isWindows) {
+    // taskkill /T /F /PID <pid> — kills process tree immediately (forceful)
+    // No graceful period on Windows — taskkill /T does tree kill in one shot
+    try {
+      execSync(`taskkill /T /F /PID ${marker.pid}`, { stdio: 'ignore' });
+    } catch (err) {
+      // Ignore errors — process may have exited already
+    }
+  } else {
+    // POSIX: send SIGTERM, wait, then escalate to SIGKILL if needed
+    try {
+      process.kill(marker.pid, 'SIGTERM');
+    } catch (err) {
+      // Process may have exited between check and kill
+    }
+
+    let escalated = false;
+    const deadline = Date.now() + graceSec * 1000;
+
+    while (processAlive(marker.pid)) {
+      if (Date.now() >= deadline) {
+        try {
+          process.kill(marker.pid, 'SIGKILL');
+          escalated = true;
+        } catch (err) {
+          // Process may have already exited
+        }
+        break;
+      }
+      await sleep(200);
+    }
+  }
+
+  // Ensure marker is removed
+  removeMarker(projectRoot);
+
+  return {
+    ok: true,
+    pid: marker.pid,
+    escalated: isWindows ? false : (escalated || false),
+    duration_ms: Date.now() - startMs
+  };
+}

package/src/lib/utils.mjs

@@ -98,78 +98,7 @@ export function getPackageRoot() {
   return path.resolve(__dirname, '../../');
 }
 
-/**
- * Парсит секцию "## Ревью" тикета и возвращает статус последней записи.
- * Поддерживает табличный и текстовый форматы.
- *
- * Табличный формат:
- * | Дата | Статус | Комментарий |
- * |------|--------|-------------|
- * | 2026-03-08 | passed | Всё ок |
- *
- * Текстовый формат:
- * - 2026-03-08: passed - Всё ок
- * - 2026-03-08: failed - Есть замечания
- *
- * @param {string} content - Содержимое тикета (markdown)
- * @returns {string|null} "passed", "failed" или null (если нет ревью)
- */
-export function getLastReviewStatus(content) {
-  if (!content) 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;
-    }
-  }
-
-  if (lastHeaderLineIndex === -1) return null;
-
-  // Собираем содержимое после заголовка до следующего 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;
-
-  // Пробуем распарсить табличный формат
-  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;
-    });
-
-    if (dataRows.length > 0) {
-      // Последняя строка таблицы = самое свежее ревью (записи ведутся хронологически сверху вниз)
-      const latestRow = dataRows[dataRows.length - 1];
-      const cells = latestRow.split('|').map(c => c.trim()).filter(c => c);
-      const statusRaw = cells[1]?.toLowerCase() || '';
-      if (statusRaw.includes('passed')) return 'passed';
-      if (statusRaw.includes('failed')) return 'failed';
-      if (statusRaw.includes('skipped')) return 'skipped';
-    }
-  }
-
-  // Пробуем распарсить текстовый формат (список)
-  const listItems = reviewSection.split('\n').filter(line => line.trim().match(/^[-*]\s/));
-  if (listItems.length > 0) {
-    // Последний элемент списка = самое свежее ревью (записи ведутся хронологически)
-    const latestItem = listItems[listItems.length - 1].trim();
-    const statusMatch = latestItem.match(/:\s*(passed|failed|skipped)\b/i);
-    if (statusMatch) return statusMatch[1].toLowerCase();
-  }
-
-  return null;
-}
+export { getLastReviewStatus, appendReviewEntry } from '../../workflow-ai/src/lib/review-section.mjs';
 
 /**
  * Загружает конфигурацию правил перемещения тикетов.

package/src/runner.mjs

@@ -9,6 +9,94 @@ import { findProjectRoot } from './lib/find-root.mjs';
 import { loadRules, scanStderrForFatalRule, classify } from './lib/error-classifier.mjs';
 import { snapshot, diff, isEmpty } from './lib/artifact-snapshot.mjs';
 import { markUnhealthy, isHealthy } from './lib/agent-health-registry.mjs';
+import { writeMarker, removeMarker } from './lib/marker.mjs';
+import { appendAgentRun, classifyAgentResult } from '../workflow-ai/src/lib/agent-history.mjs';
+import { incrementMetrics } from '../workflow-ai/src/lib/metrics-incremental.mjs';
+
+// ============================================================================
+// Audit-log helpers (used by executeWithFallback hook — IMPL-83)
+// ============================================================================
+
+function formatLocalDateTime(d) {
+  const pad = n => String(n).padStart(2, '0');
+  return `${d.getFullYear()}-${pad(d.getMonth()+1)}-${pad(d.getDate())} ${pad(d.getHours())}:${pad(d.getMinutes())}:${pad(d.getSeconds())}`;
+}
+
+function findTicketPathForId(ticketId, projectRoot) {
+  if (!ticketId) return null;
+  const dirs = ['ready', 'in-progress', 'review', 'done', 'backlog', 'blocked'];
+  for (const d of dirs) {
+    const p = path.join(projectRoot, '.workflow', 'tickets', d, `${ticketId}.md`);
+    try { if (fs.existsSync(p)) return p; } catch {}
+  }
+  return null;
+}
+
+// IMPL-86: normalize agent_id in last row of ## Ревью section
+// Re-writes the agent column if it differs from expectedAgent. Other columns untouched.
+// Returns { ok: true, changed: boolean } or { ok: false, code, error }.
+function normalizeReviewAgentId(ticketPath, expectedAgent) {
+  if (!ticketPath || !expectedAgent) return { ok: false, code: 'INVALID_INPUT' };
+  let content;
+  try {
+    content = fs.readFileSync(ticketPath, 'utf8');
+  } catch (err) {
+    return { ok: false, code: 'READ_ERROR', error: err.message };
+  }
+
+  const sectionRegex = /(##\s+Ревью\s*\r?\n)([\s\S]*?)(?=\r?\n##\s+|$)/i;
+  const match = content.match(sectionRegex);
+  if (!match) return { ok: false, code: 'NO_SECTION' };
+
+  const sectionBody = match[2];
+  const lines = sectionBody.split('\n');
+  let headerIdx = -1;
+  for (let i = 0; i < lines.length; i++) {
+    const t = lines[i].trim();
+    if (t.startsWith('|') && t.endsWith('|') && !/^\s*\|[-:|\s]+\|\s*$/.test(t)) {
+      headerIdx = i;
+      break;
+    }
+  }
+  if (headerIdx === -1) return { ok: false, code: 'NO_HEADER' };
+
+  const headerCells = lines[headerIdx].trim().slice(1, -1).split(/(?<!\\)\|/).map(c => c.trim());
+  const agentColIdx = headerCells.findIndex(c => /^(агент|agent)$/i.test(c));
+  if (agentColIdx === -1) return { ok: false, code: 'NO_AGENT_COLUMN' };
+
+  let lastDataIdx = -1;
+  for (let i = lines.length - 1; i > headerIdx; i--) {
+    const t = lines[i].trim();
+    if (t.startsWith('|') && t.endsWith('|') && !/^\s*\|[-:|\s]+\|\s*$/.test(t)) {
+      lastDataIdx = i;
+      break;
+    }
+  }
+  if (lastDataIdx === -1) return { ok: false, code: 'NO_DATA_ROW' };
+
+  const cells = lines[lastDataIdx].trim().slice(1, -1).split(/(?<!\\)\|/).map(c => c.trim());
+  if (agentColIdx >= cells.length) return { ok: false, code: 'CELL_MISSING' };
+
+  const currentAgent = cells[agentColIdx];
+  if (currentAgent === expectedAgent) return { ok: true, changed: false };
+
+  cells[agentColIdx] = expectedAgent;
+  lines[lastDataIdx] = `| ${cells.join(' | ')} |`;
+
+  const newSection = match[1] + lines.join('\n');
+  const newContent = content.replace(sectionRegex, newSection);
+
+  const dir = path.dirname(ticketPath);
+  const tmp = path.join(dir, `.${path.basename(ticketPath)}.normagent.${process.pid}.${Date.now()}`);
+  try {
+    fs.writeFileSync(tmp, newContent, 'utf8');
+    fs.renameSync(tmp, ticketPath);
+    return { ok: true, changed: true };
+  } catch (err) {
+    try { fs.unlinkSync(tmp); } catch {}
+    return { ok: false, code: 'WRITE_ERROR', error: err.message };
+  }
+}
 
 // ============================================================================
 // Logger — система логирования с уровнями DEBUG/INFO/WARN/ERROR
@@ -1040,6 +1128,29 @@ class StageExecutor {
 
         const result = await this.callAgent(agent, prompt, stageId, effectiveStage.skill, agentId);
 
+        // IMPL-83: audit-log hook (success path)
+        await this._auditAgentRun(stageId, effectiveStage, agentId, {
+          exitCode: result.exitCode ?? 0,
+          stderr: '',
+          stdout: result.output || '',
+          parsedResult: result.result || null,
+        });
+
+        // IMPL-86: normalize agent_id in ## Ревью after review-result stage
+        if ((effectiveStage.skill === 'review-result' || stageId === 'review-result') && this.context?.ticket_id) {
+          try {
+            const tp = findTicketPathForId(this.context.ticket_id, this.projectRoot);
+            if (tp) {
+              const r = normalizeReviewAgentId(tp, agentId);
+              if (!r.ok && r.code !== 'NO_SECTION' && r.code !== 'NO_DATA_ROW' && this.logger) {
+                this.logger.warn(`review agent normalize: ${r.code} ${r.error || ''}`, stageId);
+              }
+            }
+          } catch (err) {
+            if (this.logger) this.logger.warn(`review agent normalize threw: ${err.message}`, stageId);
+          }
+        }
+
         if (this.logger) this.logger.stageComplete(stageId, result.status, result.exitCode);
         return result;
       } catch (err) {
@@ -1048,6 +1159,16 @@ class StageExecutor {
         const exitCode = err.exitCode ?? err.code;
         const stderr = err.stderr || '';
 
+        // IMPL-83: audit-log hook (failure path)
+        await this._auditAgentRun(stageId, effectiveStage, agentId, {
+          exitCode,
+          stderr,
+          stdout: err.stdout || '',
+          parsedResult: err.parsedResult || null,
+          timedOut: err.timedOut === true,
+          signal: err.signal,
+        });
+
         const after = snapshotEnabled ? await snapshot(this.projectRoot, snapshotOpts) : null;
         const diffResult = snapshotEnabled ? diff(before, after) : null;
         const diffEmpty = snapshotEnabled && isEmpty(diffResult);
@@ -1087,6 +1208,68 @@ class StageExecutor {
     }
   }
 
+  /**
+   * IMPL-83: Audit-log hook — write entry to ticket history + bump metrics.
+   * Non-blocking: errors are logged via logger.warn, never thrown.
+   */
+  async _auditAgentRun(stageId, effectiveStage, agentId, callResult) {
+    try {
+      const ticketId = this.context?.ticket_id;
+      if (!ticketId) return;
+
+      const agentType = (agentId || '').startsWith('script-') ? 'script' : 'ai';
+      const status = classifyAgentResult({
+        exitCode: callResult.exitCode ?? 0,
+        stderr: callResult.stderr || '',
+        stdout: callResult.stdout || '',
+        timedOut: callResult.timedOut === true,
+        signal: callResult.signal,
+        parsedResult: callResult.parsedResult || null,
+        agentType,
+      });
+
+      // For move-* stages prefer destination from parsedResult.to
+      let ticketPath = null;
+      if (stageId.startsWith('move-') && callResult.parsedResult?.to) {
+        ticketPath = path.join(this.projectRoot, '.workflow/tickets', callResult.parsedResult.to, `${ticketId}.md`);
+        try { if (!fs.existsSync(ticketPath)) ticketPath = null; } catch { ticketPath = null; }
+      }
+      if (!ticketPath) {
+        ticketPath = findTicketPathForId(ticketId, this.projectRoot);
+      }
+      if (!ticketPath) return;
+
+      const skillName = effectiveStage.skill || stageId;
+      const entry = {
+        timestamp: formatLocalDateTime(new Date()),
+        skill: skillName,
+        agent: agentId || 'unknown',
+        status,
+      };
+
+      try {
+        const r = appendAgentRun(ticketPath, entry);
+        if (!r?.ok && this.logger) {
+          this.logger.warn(`audit-log appendAgentRun failed: ${r?.code || 'unknown'} ${r?.error || ''}`, stageId);
+        }
+      } catch (err) {
+        if (this.logger) this.logger.warn(`audit-log appendAgentRun threw: ${err.message}`, stageId);
+      }
+
+      try {
+        const r = incrementMetrics(this.projectRoot, entry, ticketId);
+        if (!r?.ok && this.logger) {
+          this.logger.warn(`metrics update failed: ${r?.code || 'unknown'} ${r?.error || ''}`, stageId);
+        }
+      } catch (err) {
+        if (this.logger) this.logger.warn(`metrics update threw: ${err.message}`, stageId);
+      }
+    } catch (outer) {
+      // Final safety net — never let audit-log break the pipeline
+      if (this.logger) this.logger.warn(`audit-log hook crashed: ${outer.message}`, stageId);
+    }
+  }
+
   /**
    * Выполняет stage через выбранного CLI-агента (новая модель выбора).
    * @param {string} stageId - ID stage из конфигурации
@@ -1196,7 +1379,10 @@ class StageExecutor {
         if (this.logger) {
           this.logger.timeout(stageId, timeout);
         }
-        reject(new Error(`Stage "${stageId}" timed out after ${timeout}s`));
+        const err = new Error(`Stage "${stageId}" timed out after ${timeout}s`);
+        err.timedOut = true;
+        err.exitCode = -1;
+        reject(err);
       }, timeout * 1000);
 
       let stdoutBuffer = '';
@@ -1270,7 +1456,7 @@ class StageExecutor {
         reject(err);
       });
 
-      child.on('close', (code) => {
+      child.on('close', (code, signal) => {
         this.currentChild = null;
         clearTimeout(timeoutId);
         // Обрабатываем остаток буфера стриминга
@@ -1339,6 +1525,7 @@ class StageExecutor {
           err.code = 'NON_ZERO_EXIT';
           err.exitCode = code;
           err.stderr = stderr;
+          err.signal = signal;
           if (this.logger) {
             this.logger.error(`Agent exited with code ${code}`, stageId);
             if (stderr.trim()) {
@@ -1564,7 +1751,7 @@ class PipelineRunner {
         }
         throw err;
       }
-    }
+    }
 
     /**
      * Читает approval-файл и парсит его как JSON.
@@ -1587,7 +1774,7 @@ class PipelineRunner {
         // Перебрасываем другие ошибки (например, fs-ошибки) без изменений
         throw err;
       }
-    }
+    }
 
     /**
      * Выполняет встроенный стейдж типа update-counter:
@@ -1756,7 +1943,7 @@ class PipelineRunner {
         this.logger.warn(`[${stageId}] manual-gate: aborted (runner stopped)`, stageId);
       }
       return { status: 'aborted', result: { step_id: stepId } };
-    }
+    }
 
   /**
    * Запускает основной цикл выполнения
@@ -1802,7 +1989,7 @@ class PipelineRunner {
            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');
 
@@ -2146,18 +2333,40 @@ async function runPipeline(argv = process.argv.slice(2)) {
     return { exitCode: 0, help: true };
   }
 
-  // Resolve config path
+  // Resolve config path and projectRoot
   if (!args.config) {
     const projectRoot = args.project ? path.resolve(args.project) : findProjectRoot();
     args.config = path.resolve(projectRoot, '.workflow/config/pipeline.yaml');
+    args.projectRoot = projectRoot;
   }
 
+  const projectRoot = args.projectRoot || (args.project ? path.resolve(args.project) : findProjectRoot());
+
   console.log('=== Workflow Runner ===');
   console.log(`Config: ${args.config}`);
   if (args.plan) console.log(`Plan: ${args.plan}`);
   if (args.project) console.log(`Project: ${args.project}`);
   console.log('');
 
+  // Write marker to protect against stale processes
+  try {
+    writeMarker(projectRoot, {
+      pid: process.pid,
+      timestamp: new Date().toISOString()
+    });
+  } catch (err) {
+    console.error(`[runner] failed to write marker: ${err.message}`);
+    return { exitCode: 1, error: 'Failed to acquire pipeline lock', details: err.message };
+  }
+
+  // Register signal handlers for cleanup
+  const cleanup = () => {
+    removeMarker(projectRoot);
+    process.exit(130); // 128 + SIGINT(2) — standard exit code for signal-terminated
+  };
+  process.once('SIGINT', cleanup);
+  process.once('SIGTERM', cleanup);
+
   try {
     const config = loadConfig(args.config);
     const errors = validateConfig(config);
@@ -2192,6 +2401,16 @@ async function runPipeline(argv = process.argv.slice(2)) {
     await new Promise(resolve => setTimeout(resolve, 100));
 
     return { exitCode: 1, error: err.message, stack: err.stack };
+  } finally {
+    // Ensure marker is cleaned up on normal exit
+    try {
+      removeMarker(projectRoot);
+    } catch (err) {
+      // ENOENT is expected, log warnings for other errors
+      if (err.code && err.code !== 'ENOENT') {
+        console.warn(`[runner] cleanup warning: ${err.message}`);
+      }
+    }
   }
 }
 

package/src/scripts/check-relevance.js

@@ -21,6 +21,7 @@ import {
   parseFrontmatter,
   serializeFrontmatter,
   getLastReviewStatus,
+  appendReviewEntry,
 } from "workflow-ai/lib/utils.mjs";
 
 const PROJECT_DIR = findProjectRoot();
@@ -212,43 +213,18 @@ async function checkRelevance(ticketPath) {
   return { verdict: "relevant", reason: "all_checks_passed" };
 }
 
+// IMPL-89: Replace manual markdown-write with appendReviewEntry from review-section.mjs.
 function addSkippedReview(ticketPath, reason) {
-  const now = new Date();
-  const date = now.toISOString().slice(0, 10);
-
-  let content;
-  try {
-    content = fs.readFileSync(ticketPath, "utf8");
-  } catch (e) {
-    throw new Error(`Failed to read ticket: ${e.message}`);
-  }
-
-  let { frontmatter, body } = parseFrontmatter(content);
-
-  const reviewSectionMatch = body.match(/##\s*Ревью\s*\n([\s\S]*)/i);
-  let newBody;
-
-  if (reviewSectionMatch) {
-    const reviewContent = reviewSectionMatch[1];
-    const lines = reviewContent.split("\n");
-    let insertIndex = 0;
-    for (let i = 0; i < lines.length; i++) {
-      if (lines[i].trim().startsWith("|") && lines[i].includes("---")) {
-        insertIndex = i + 1;
-        break;
-      }
-    }
-
-    const newRow = `| ${date} | ⏭️ skipped | ${reason} |`;
-    lines.splice(insertIndex, 0, newRow);
-    newBody = body.slice(0, reviewSectionMatch.index) + lines.join("\n");
-  } else {
-    const reviewTable = `\n## Ревью\n\n| Дата | Статус | Самари |\n|------|--------|--------|\n| ${date} | ⏭️ skipped | ${reason} |\n`;
-    newBody = body.trimEnd() + reviewTable;
+  const date = new Date().toISOString().slice(0, 10);
+  const r = appendReviewEntry(ticketPath, {
+    date,
+    agent: 'script-check-relevance',
+    status: 'skipped',
+    summary: reason,
+  });
+  if (!r?.ok) {
+    throw new Error(`addSkippedReview failed: ${r?.code || 'unknown'} ${r?.error || ''}`);
   }
-
-  const newContent = serializeFrontmatter(frontmatter) + newBody;
-  fs.writeFileSync(ticketPath, newContent, "utf8");
 }
 
 async function main() {

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

@@ -85,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 = [];

package/src/scripts/move-ticket.js

@@ -20,6 +20,7 @@ import {
   printResult,
   serializeFrontmatter,
   getLastReviewStatus,
+  appendReviewEntry,
 } from "workflow-ai/lib/utils.mjs";
 
 const logger = {
@@ -210,13 +211,19 @@ async function moveTicket(ticketId, target) {
   }
 
   // Fallback: если тикет идёт в done из review, но агент не записал секцию "## Ревью" — дописываем
+  // IMPL-88: используем appendReviewEntry вместо ручного markdown-write.
+  // ВАЖНО: пишем напрямую через body manipulation (а не file rewrite через appendReviewEntry),
+  // потому что move-ticket в этой же транзакции serializeFrontmatter(...) + renameSync — иначе
+  // запись в исходный файл потеряется при rename. Используем те же поля что и appendReviewEntry.
   if (
     target === "done" &&
     currentStatus === "review" &&
     getLastReviewStatus(content) === null
   ) {
     const date = now.slice(0, 16).replace("T", " ");
-    const reviewSection = `\n## Ревью\n\n| Дата | Статус | Самари |\n|------|--------|--------|\n| ${date} | ✅ passed | Pipeline fallback: агент не записал секцию ревью |\n`;
+    const summary = "Pipeline fallback: агент не записал секцию ревью";
+    const agent = "script-move-fallback";
+    const reviewSection = `\n## Ревью\n\n| Дата | Статус | Самари | Агент |\n|------|--------|--------|-------|\n| ${date} | ✅ passed | ${summary} | ${agent} |\n`;
     body = body.trimEnd() + "\n" + reviewSection;
   }
 

package/src/skills/coach/SKILL.md

@@ -128,6 +128,7 @@ ticket_prefix: COACH
 **⚠️ Антипаттерн «уход в формулировки вместо root cause»:** стейкхолдер задаёт вопрос о наблюдаемом дефекте («почему не поймали?»), а коуч анализирует текст формулировок, семантику переносов, чеклисты — вместо того чтобы ответить на прямой вопрос: какой конкретный шаг в какой конкретной стадии не выполнил конкретное физическое действие (открыть файл, посмотреть на картинку, запустить команду). Формулировки — это причина второго порядка; причина первого порядка — «агент X не сделал действие Y». Всегда начинай с причины первого порядка, потом объясняй, почему инструкции это допустили.
 **⚠️ Антипаттерн «оценка по результату вместо сверки с инструкцией»:** при анализе действия агента — **не оценивай** его «разумность» или «допустимость» по своему суждению. Вместо этого открой скил агента и **дословно сверь** действие с инструкцией. Если инструкция говорит «разбей тикет», а агент объединил шаги — это нарушение, даже если результат выглядит «приемлемо». Коуч не имеет права смягчать finding на основании того, что дефект «небольшой» или «единичный» — скил либо нарушен, либо нет.
 **⚠️ Антипаттерн «пересказ вместо цитаты» при утверждениях о коде:** перед утверждением вида «скрипт/функция X использует/читает/пишет Y» обязан открыть файл и **дословно процитировать** строку, на которой это поведение происходит. Пересказ по памяти (даже свежей) теряет операторы-fallback (`a || b`, `a ?? b`), условные ветви, ранние return'ы — те детали, которые как раз и задают реальное поведение. Источник ошибки: агент видит ключевое слово в строке, строит «достаточное» утверждение о поведении и идёт дальше. Правило: если утверждение про код войдёт в финдинг, CHG, черновик правки или ответ стейкхолдеру — строка должна быть в отчёте целиком (либо скопированной в цитату, либо явной ссылкой `file:line`, открытой и перечитанной непосредственно перед утверждением).
+**⚠️ Антипаттерн «отрицательное утверждение о capability инструмента без проверки» (расширение предыдущего):** правило «дословная цитата» применяется не только к **позитивным** утверждениям («X делает Y»), но и к **отрицательным** («инструмент/фреймворк/тест X **не умеет / не покрывает / не подходит для** Y»). Отрицательное утверждение о capability — такое же утверждение о коде, как и позитивное, и требует тех же доказательств: либо (1) Grep по тестам/коду проекта, использующим инструмент, на ключевой паттерн использования (например, для гипотезы «Playwright не покрывает extension popup» — `Grep "loadExtension|launchPersistentContext|chrome-extension://"` по `tests/`), либо (2) дословная цитата из официальной документации. Источник ошибки: агент экстраполирует «общеизвестное» назначение инструмента (Playwright = веб-сайты, jest = unit-тесты и т.п.) без проверки фактической практики проекта или актуальных возможностей. Триггер срабатывания: если в черновике финдинга/ответа есть конструкция «{инструмент} не {глагол} {объект}» и от неё зависит root cause или CHG — **обязательна** проверка перед показом стейкхолдеру. Без проверки финдинг невалиден, даже если интуиция кажется правильной.
 3. **Итеративность** — улучшай скилы инкрементально. Маленькие точечные улучшения > масштабные переписывания.
 4. **Обратная совместимость** — улучшения не должны ломать существующие воркфлоу и интеграции.
 5. **Актуальность знаний** — активно ищи в интернете лучшие практики, фреймворки и подходы для обогащения скилов.

package/src/skills/execute-task/SKILL.md

@@ -70,6 +70,14 @@ description: >
 |----------|----------------|
 | `algorithms/execution-strategy.md` | **ВСЕГДА** — стратегия анализа, выполнения и верификации задачи |
 
+## Загрузка шаблонов
+
+Подгружай из `templates/` при необходимости:
+
+| Шаблон | Когда загружать |
+|--------|----------------|
+| `templates/result-template.md` | При создании секции `## Result` в тикете — структура и правила заполнения |
+
 ## Шаги выполнения
 
 ### 1. Прочитать тикет
@@ -122,6 +130,8 @@ description: >
 3. Прочитать `context.notes` — дополнительный контекст от создателя тикета
 4. Если тикет ссылается на план (`parent_plan`) — прочитать план для понимания общей картины
 
+**⛔ Валидация путей перед Edit:** если DoD требует изменить конкретный файл, указанный в `context.files` или описании, используй **ровно тот путь**, который указан. Перед каждым Edit сверь целевой путь с путём из контекста тикета. Одноимённый файл в другой директории — не целевой файл. Пример нарушения: тикет указывает `workflow-ai/README.md`, агент редактирует `./README.md`.
+
 ### 5. Выполнить работу и фиксировать результат инкрементально
 
 Действовать по описанию и DoD тикета. Подход определяется **содержимым тикета**, а не типом:
@@ -131,6 +141,8 @@ description: >
 - Если тикет требует тестирования — выполнить чеклист проверок из DoD, зафиксировать pass/fail по каждому пункту
 - Если тикет требует исследования — использовать доступные инструменты для сбора данных, подкреплять источниками
 
+**⛔ Перед вставкой кода в существующий файл — прочитай целевой участок и пойми его структуру.** Определи границы функции/класса/блока, в который вставляешь код. Вставляй код в семантически правильное место, а не в произвольную точку файла. Если не уверен в месте вставки — прочитай окружающий контекст (строки до и после) и убедись, что вставка не разрывает существующую структуру.
+
 **⚠️ ИНКРЕМЕНТАЛЬНАЯ ЗАПИСЬ (ОБЯЗАТЕЛЬНО):**
 
 После выполнения **каждого пункта** — **сразу** запиши результат в тикет:
@@ -156,7 +168,7 @@ description: >
 
 К этому моменту секция Result уже содержит результаты по каждому пункту (записаны инкрементально на шаге 5). Осталось:
 
-- Обновить/добавить **Summary** — краткое резюме всей работы
+- Обновить/добавить **Что сделано** — краткое резюме всей работы
 - Дополнить **Изменённые файлы** и **Заметки** если нужно
 - **НЕ удалять и не переписывать** уже записанные результаты
 
@@ -191,6 +203,14 @@ description: >
 
 ### 9. Вывести структурированный результат
 
+**⛔ Перед выводом RESULT пройди все три GATE из `workflows/execute.md` (шаг 6).** GATE-1 (Edit-проверка), GATE-2 (механическая Read-проверка), GATE-3 (self-check). При любом нарушении — вернись к шагам 5–7.
+
+⛔ **GATE-1 — EDIT-ПРОВЕРКА (выполни ПЕРЕД Read-проверкой):**
+
+За текущую сессию ты должен был вызвать инструмент **`Edit`** на файл тикета как минимум дважды: один раз для обновления DoD-чекбоксов (`[ ]` → `[x]`), один раз для записи секции Result. Если ни одного вызова `Edit` на файл тикета `.workflow/tickets/in-progress/{TICKET-ID}.md` не было — секция Result физически пустая и DoD не отмечен, независимо от написанного в stdout. Это призрачное выполнение (ограничение #9). Немедленно вернись к шагам 5–7.
+
+**Ключевой принцип:** stdout (текст ответа) ≠ файл тикета. Результат существует только в том, что записано через инструмент `Edit`.
+
 **⛔ ОБЯЗАТЕЛЬНАЯ МЕХАНИЧЕСКАЯ ПРОВЕРКА — перечитай файл тикета перед RESULT:**
 
 Перед выводом `---RESULT---` выполни `Read` на файл тикета (`.workflow/tickets/in-progress/{TICKET-ID}.md`) и глазами убедись:
@@ -219,6 +239,7 @@ description: >
 - Нет побочных эффектов — не созданы тикеты/планы, не перемещены файлы
 - Поля `status` и `completed_at` не записаны в файл тикета ни в каком виде
 - Секция `## Ревью` не создавалась и не редактировалась тобой
+- Все временные/промежуточные файлы и пустые директории, созданные в ходе работы и не являющиеся deliverable, удалены; файлы, путь которых зависит от конфига инструмента (mock/test/fixture/snapshot/output), лежат в директории, указанной в конфиге, а не в произвольном месте
 
 **⛔ ФОРМАТ STDOUT — СТРОГО:**
 
@@ -273,6 +294,12 @@ status: default
 
    При визуальных/семантических/поведенческих критериях в Result **явно обоснуй**, почему структурной проверки недостаточно (одна строка). Полная таблица соответствий — `algorithms/execution-strategy.md` раздел «Соразмерность проверки критерию».
 
+9. **Configured paths и cleanup** — файлы, создаваемые во время работы тикета, размещай в местах, явно указанных в конфиге соответствующего инструмента. Перед созданием файла, путь которого может зависеть от конфигурации (mock, тест, фикстура, стаб, snapshot, сгенерированный artifact, временный файл сборки и подобное) — **прочитай соответствующий конфиг проекта**, найди в нём поле target-директории (`roots`, `testDir`, `paths`, `output`, `outDir`, `srcDir` или аналог в конфиге твоего инструмента) и помести файл туда. Создавать файл в корне репозитория без проверки конфига запрещено.
+
+   **После удаления временных/ошибочных файлов — проверь и удали пустые родительские директории**, созданные в ходе тикета. Каждая созданная директория — твоя ответственность вплоть до закрытия тикета. Пустая папка или артефакт вне scope, оставленные в репозитории, — побочный эффект (см. принцип 4 «No Side Effects»).
+
+   ⛔ **Антипаттерн:** создал файл в корне (mock/snapshot/test) → инструмент не подхватил из-за неправильного пути → удалил файл, но оставил пустую папку. Лечение: до создания — проверка конфига; после rm — удаление пустых директорий.
+
 ## Формат вывода
 
 - Русский язык

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

@@ -139,13 +139,17 @@ issues:
 
 ## Формат секции ревью в тикете
 
+При записи результата ревью используй 4-колоночный формат таблицы.
+Колонки в порядке: **Дата → Статус → Самари → Агент** (Агент идёт последним, чтобы основная информация о результате ревью читалась слева направо без шумовой колонки в начале).
+В колонку Агент запиши имя своей модели (claude-sonnet, claude-haiku и т.д.). Если не знаешь точного имени — пиши `unknown`.
+
 ```markdown
 ## Ревью
 
-| Дата | Статус | Самари |
-|------|--------|--------|
-| 2026-03-25 14:30 | ❌ failed | Не пройдены тесты, отсутствует файл X |
-| 2026-03-25 15:45 | ✅ passed | Все критерии DoD выполнены |
+| Дата | Статус | Самари | Агент |
+|------|--------|--------|-------|
+| 2026-03-25 14:30 | ❌ failed | Не пройдены тесты, отсутствует файл X | claude-sonnet |
+| 2026-03-25 15:45 | ✅ passed | Все критерии DoD выполнены | claude-sonnet |
 ```
 
 > **Порядок записей:** хронологический сверху вниз. Последняя строка = последнее ревью.