workflow-ai 1.3.1 → 1.5.0

package/CHANGELOG.md

@@ -1,3 +1,15 @@
+## [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
 
 ### Добавлено

package/package.json

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

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

@@ -10,6 +10,93 @@ import { loadRules, scanStderrForFatalRule, classify } from './lib/error-classif
 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
@@ -1041,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) {
@@ -1049,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);
@@ -1088,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 из конфигурации
@@ -1197,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 = '';
@@ -1271,7 +1456,7 @@ class StageExecutor {
         reject(err);
       });
 
-      child.on('close', (code) => {
+      child.on('close', (code, signal) => {
         this.currentChild = null;
         clearTimeout(timeoutId);
         // Обрабатываем остаток буфера стриминга
@@ -1340,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()) {

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/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/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 |
 ```
 
 > **Порядок записей:** хронологический сверху вниз. Последняя строка = последнее ревью.