workflow-ai 1.1.0 → 1.3.0

package/src/scripts/mark-blocked.js

@@ -0,0 +1,160 @@
+#!/usr/bin/env node
+
+/**
+ * mark-blocked.js - Скрипт для обновления frontmatter тикета и записи в alerts.jsonl
+ *
+ * Использование:
+ *   node mark-blocked.js <ticket_id> --attempts=N --reason=<str>
+ *
+ * Примеры:
+ *   node mark-blocked.js IMPL-59 --attempts=6 --reason=max_review_attempts
+ *   node mark-blocked.js QA-40 --attempts=3 --reason=human_gate_rejected
+ */
+
+import fs from "fs";
+import path from "path";
+import YAML from "workflow-ai/lib/js-yaml.mjs";
+import { findProjectRoot } from "workflow-ai/lib/find-root.mjs";
+import {
+  parseFrontmatter,
+  printResult,
+  serializeFrontmatter,
+} from "workflow-ai/lib/utils.mjs";
+
+// Корень проекта
+const PROJECT_DIR = findProjectRoot();
+// Базовая директория workflow
+const WORKFLOW_DIR = path.join(PROJECT_DIR, ".workflow");
+const TICKETS_DIR = path.join(WORKFLOW_DIR, "tickets");
+// Директория state
+const STATE_DIR = path.join(PROJECT_DIR, ".workflow", "state");
+const ALERTS_FILE = path.join(STATE_DIR, "alerts.jsonl");
+
+// Парсинг аргументов
+const args = process.argv.slice(2);
+if (args.length < 3) {
+  console.error("Ошибка: недостаточно аргументов");
+  console.error("Использование: node mark-blocked.js <ticket_id> --attempts=N --reason=<str>");
+  process.exit(1);
+}
+
+const ticketId = args[0];
+let attempts = null;
+let reason = null;
+
+// Парсинг флагов
+for (let i = 1; i < args.length; i++) {
+  const arg = args[i];
+  if (arg.startsWith("--attempts=")) {
+    attempts = parseInt(arg.substring("--attempts=".length), 10);
+    if (isNaN(attempts)) {
+      console.error("Ошибка: некорректный формат --attempts");
+      process.exit(1);
+    }
+  } else if (arg.startsWith("--reason=")) {
+    reason = arg.substring("--reason=".length);
+  }
+}
+
+// Проверка обязательного параметра reason
+if (!reason) {
+  console.error("Ошибка: параметр --reason обязателен");
+  process.exit(1);
+}
+
+// Поиск файла тикета рекурсивно
+function findTicketFile(ticketId, searchDir) {
+  try {
+    const files = fs.readdirSync(searchDir, { withFileTypes: true });
+    
+    for (const file of files) {
+      const fullPath = path.join(searchDir, file.name);
+      
+      if (file.isDirectory()) {
+        // Рекурсивный поиск в поддиректориях
+        const found = findTicketFile(ticketId, fullPath);
+        if (found) return found;
+      } else if (file.isFile() && file.name.endsWith('.md') && file.name.startsWith(ticketId)) {
+        return fullPath;
+      }
+    }
+  } catch (error) {
+    console.error(`Ошибка при чтении директории ${searchDir}:`, error.message);
+  }
+  
+  return null;
+}
+
+// Основная функция
+function main() {
+  try {
+    // Поиск файла тикета
+    const ticketFile = findTicketFile(ticketId, TICKETS_DIR);
+    if (!ticketFile) {
+      console.error(`Ошибка: тикет ${ticketId} не найден в ${TICKETS_DIR}`);
+      process.exit(1);
+    }
+
+    // Чтение файла тикета
+    const content = fs.readFileSync(ticketFile, 'utf8');
+    const { frontmatter, body } = parseFrontmatter(content);
+
+    // Обновление frontmatter
+    const now = new Date().toISOString();
+    frontmatter.auto_blocked_reason = reason;
+    frontmatter.auto_blocked_attempts = attempts;
+    frontmatter.auto_blocked_at = now;
+
+    // Сериализация и запись обратно в файл
+    const newContent = serializeFrontmatter(frontmatter) + body;
+    fs.writeFileSync(ticketFile, newContent, 'utf8');
+
+    console.log(`✅ Frontmatter тикета ${ticketId} обновлен`);
+
+    // Попытка записи в alerts.jsonl
+    try {
+      // Создание директории state если не существует
+      if (!fs.existsSync(STATE_DIR)) {
+        fs.mkdirSync(STATE_DIR, { recursive: true });
+        console.log(`✅ Директория ${STATE_DIR} создана`);
+      }
+
+      // Формирование JSONL записи
+      const alertEntry = {
+        timestamp: now,
+        severity: "warning",
+        kind: "ticket_auto_blocked",
+        project: path.basename(PROJECT_DIR),
+        ticket_id: ticketId,
+        attempts: attempts,
+        reason: reason,
+        stage: "review-result"
+      };
+
+      // Append-only запись в alerts.jsonl
+      fs.appendFileSync(ALERTS_FILE, JSON.stringify(alertEntry) + '\n', 'utf8');
+      console.log(`✅ Запись добавлена в ${ALERTS_FILE}`);
+
+    } catch (alertError) {
+      console.warn(`⚠️  Предупреждение: не удалось записать в alerts.jsonl: ${alertError.message}`);
+      console.log(`ℹ️  Frontmatter обновлен, запись в alerts пропущена`);
+    }
+
+    // Вывод результата
+    printResult({
+      ticket_id: ticketId,
+      reason: reason,
+      attempts: attempts,
+      blocked_at: now,
+      alerts_file: ALERTS_FILE,
+      status: "completed"
+    });
+
+  } catch (error) {
+    console.error(`Ошибка: ${error.message}`);
+    process.exit(1);
+  }
+}
+
+// Запуск скрипта
+main();

package/src/scripts/move-ticket.js

@@ -1,103 +1,325 @@
 #!/usr/bin/env node
 
 /**
- * move-ticket.js - Тонкая прослойка: парсинг argv → вызов operation → печать RESULT
+ * move-ticket.js - Скрипт для перемещения тикетов между директориями канбан-доски
  *
- * Контракт:
- *   1. Прямой режим: node move-ticket.js <id> <target>
- *   2. Runner-режим: node move-ticket.js "<prompt-block>" (с regex ticket_id: и target:)
+ * Использование:
+ *   node move-ticket.js <ticket_id> <target>
  *
- * RESULT format (success):
- *   status: moved
- *   ticket_id: <id>
- *   from: <source-status>
- *   to: <target-status>
- *
- * RESULT format (error):
- *   status: error
- *   ticket_id: <id>  (если удалось распарсить)
- *   from: <source-or-empty>  (если есть)
- *   to: <target>  (если есть)
- *   error: <message>
+ * Пример:
+ *   node move-ticket.js IMPL-001 in-progress
  */
 
+import fs from "fs";
+import path from "path";
+import { fileURLToPath } from "url";
+import YAML from "workflow-ai/lib/js-yaml.mjs";
 import { findProjectRoot } from "workflow-ai/lib/find-root.mjs";
-import { moveTicket } from "../lib/operations/tickets.mjs";
-import { printResult } from "workflow-ai/lib/utils.mjs";
-
-// Main entry point
-const rawArgs = process.argv.slice(2);
-let ticketId, target;
-
-if (rawArgs.length >= 2) {
-  // Прямой вызов: node move-ticket.js IMPL-001 in-progress
-  ticketId = rawArgs[0];
-  target = rawArgs[1];
-} else if (rawArgs.length === 1) {
-  // Runner-режим: один аргумент — многострочный текст с ticket_id: и target:
-  const prompt = rawArgs[0];
-  const ticketMatch = prompt.match(/ticket_id:\s*(\S+)/);
-  const targetMatch = prompt.match(/target:\s*(\S+)/);
-  ticketId = ticketMatch?.[1];
-  target = targetMatch?.[1];
-  if (!ticketId || !target) {
-    printResult({
-      status: "error",
-      ticket_id: ticketId || "",
-      from: "",
-      to: target || "",
-      error: "Missing ticket_id or target",
-    });
-    process.exit(1);
+import {
+  parseFrontmatter,
+  printResult,
+  serializeFrontmatter,
+  getLastReviewStatus,
+} from "workflow-ai/lib/utils.mjs";
+
+const logger = {
+  info: (msg) => console.error(`[INFO] ${msg}`),
+  warn: (msg) => console.error(`[WARN] ${msg}`),
+};
+
+// Корень проекта
+const PROJECT_DIR = findProjectRoot();
+// Базовая директория workflow
+const WORKFLOW_DIR = path.join(PROJECT_DIR, ".workflow");
+const TICKETS_DIR = path.join(WORKFLOW_DIR, "tickets");
+
+// Доступные статусы
+const VALID_STATUSES = [
+  "backlog",
+  "ready",
+  "in-progress",
+  "blocked",
+  "review",
+  "done",
+  "archive",
+];
+
+// Таблица допустимых переходов
+const VALID_TRANSITIONS = {
+  backlog: ["ready", "blocked", "done"],
+  ready: ["in-progress", "review", "backlog"],
+  "in-progress": ["done", "blocked", "review"],
+  blocked: ["ready"],
+  review: ["done", "ready", "in-progress", "blocked"],
+  done: ["ready", "blocked", "archive"],
+  archive: ["backlog"],
+};
+
+/**
+ * Определяет текущий статус тикета по расположению файла
+ */
+function getStatusFromPath(filePath) {
+  const fileName = path.basename(filePath);
+  for (const status of VALID_STATUSES) {
+    const statusDir = path.join(TICKETS_DIR, status);
+    const expectedPath = path.join(statusDir, fileName);
+    if (filePath === expectedPath) {
+      return status;
+    }
   }
-} else {
-  printResult({
-    status: "error",
-    error: "Missing arguments",
-  });
-  process.exit(1);
+  return null;
 }
 
-async function main() {
+/**
+ * Проверяет допустимость перехода
+ */
+function isValidTransition(from, to) {
+  if (!VALID_STATUSES.includes(from)) {
+    return { valid: false, error: `Неверный исходный статус: ${from}` };
+  }
+  if (!VALID_STATUSES.includes(to)) {
+    return {
+      valid: false,
+      error: `Неверный целевой статус: ${to}. Доступные: ${VALID_STATUSES.join(", ")}`,
+    };
+  }
+
+  const allowedTransitions = VALID_TRANSITIONS[from] || [];
+  if (!allowedTransitions.includes(to)) {
+    return {
+      valid: false,
+      error: `Переход из ${from} в ${to} недопустим. Доступные переходы: ${allowedTransitions.join(", ") || "нет"}`,
+    };
+  }
+
+  return { valid: true };
+}
+
+/**
+ * Hook для обновления approval-файлов при перемещении тикета
+ * @param {string} ticketId - ID тикета
+ * @param {string} target - целевой статус
+ * @param {object} fsModule - модуль fs (для mock в тестах)
+ * @param {string} workflowDir - директория .workflow
+ */
+function updateApprovalFilesHook(ticketId, target, fsModule = fs, workflowDir = WORKFLOW_DIR) {
   try {
-    const projectRoot = findProjectRoot();
-    const result = await moveTicket(projectRoot, ticketId, target);
-    printResult({
-      status: "moved",
-      ticket_id: ticketId,
-      from: result.from,
-      to: result.to,
-    });
+    const approvalsDir = path.join(workflowDir, "approvals");
+    if (fsModule.existsSync(approvalsDir)) {
+      const files = fsModule.readdirSync(approvalsDir);
+      const escapedTicketId = ticketId.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+      const pattern = new RegExp(`^${escapedTicketId}_manual-gate-.*_\\d+\\.json$`);
+      for (const file of files) {
+        if (!pattern.test(file)) continue;
+        const filePath = path.join(approvalsDir, file);
+        try {
+          const data = JSON.parse(fsModule.readFileSync(filePath, "utf8"));
+          if (data.status === "pending") {
+            data.status = "approved";
+            data.decided_by = "move-ticket";
+            data.comment = `auto-approved on move to ${target}`;
+            data.updated_at = new Date().toISOString();
+            fsModule.writeFileSync(filePath, JSON.stringify(data, null, 2), "utf8");
+            logger.info(`Approval file ${file} auto-approved on move to ${target}`);
+          }
+        } catch (err) {
+          logger.warn(`Corrupt approval file ${file}: ${err.message}`);
+          // продолжаем, не падаем
+        }
+      }
+    }
   } catch (err) {
-    // moveTicket throws { code: 'INVALID_TRANSITION', from, to, id } для ошибок валидации
-    if (err.code === "INVALID_TRANSITION") {
-      const fromDisplay = err.from || "";
-      const errorMsg =
-        !err.from || err.from === null
-          ? `Тикет ${ticketId} не найден ни в одной из директорий`
-          : `Недопустимый переход: ${err.from} → ${err.to}`;
-      printResult({
-        status: "error",
-        ticket_id: ticketId,
-        from: fromDisplay,
-        to: err.to,
-        error: errorMsg,
-      });
-    } else if (err.message) {
-      printResult({
-        status: "error",
-        ticket_id: ticketId,
-        error: err.message,
-      });
-    } else {
+    // Ошибка hook'а не должна фейлить само перемещение
+    logger.warn(`Approval hook error: ${err.message}`);
+  }
+}
+
+/**
+ * Основная функция перемещения тикета
+ */
+async function moveTicket(ticketId, target) {
+  // Поиск файла тикета во всех директориях
+  let sourceDir = null;
+  let currentStatus = null;
+
+  for (const status of VALID_STATUSES) {
+    const statusDir = path.join(TICKETS_DIR, status);
+    const ticketPath = path.join(statusDir, `${ticketId}.md`);
+    if (fs.existsSync(ticketPath)) {
+      sourceDir = statusDir;
+      currentStatus = status;
+      break;
+    }
+  }
+
+  if (!sourceDir) {
+    return {
+      status: "error",
+      ticket_id: ticketId,
+      error: `Тикет ${ticketId} не найден ни в одной из директорий`,
+    };
+  }
+
+  // Проверка допустимости перехода
+  const transitionCheck = isValidTransition(currentStatus, target);
+  if (!transitionCheck.valid) {
+    return {
+      status: "error",
+      ticket_id: ticketId,
+      from: currentStatus,
+      to: target,
+      error: transitionCheck.error,
+    };
+  }
+
+  const sourcePath = path.join(sourceDir, `${ticketId}.md`);
+  const targetDir = path.join(TICKETS_DIR, target);
+  const targetPath = path.join(targetDir, `${ticketId}.md`);
+
+  // Чтение файла тикета
+  let content;
+  try {
+    content = fs.readFileSync(sourcePath, "utf8");
+  } catch (e) {
+    return {
+      status: "error",
+      ticket_id: ticketId,
+      error: `Не удалось прочитать файл: ${e.message}`,
+    };
+  }
+
+  // Парсинг frontmatter
+  let frontmatter, body;
+  try {
+    ({ frontmatter, body } = parseFrontmatter(content));
+  } catch (e) {
+    return {
+      status: "error",
+      ticket_id: ticketId,
+      error: e.message,
+    };
+  }
+
+  // Обновление frontmatter
+  const now = new Date().toISOString();
+  frontmatter.updated_at = now;
+
+  // Если переход в done, добавляем completed_at
+  if (target === "done" && currentStatus !== "done") {
+    frontmatter.completed_at = now;
+  }
+
+  // Fallback: если тикет идёт в done из review, но агент не записал секцию "## Ревью" — дописываем
+  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`;
+    body = body.trimEnd() + "\n" + reviewSection;
+  }
+
+  // Если переход из blocked, удаляем blocked_reason
+  if (currentStatus === "blocked" && frontmatter.blocked_reason) {
+    delete frontmatter.blocked_reason;
+  }
+
+  // Сериализация нового контента
+  const newContent = serializeFrontmatter(frontmatter) + body;
+
+  // Создание целевой директории если не существует
+  if (!fs.existsSync(targetDir)) {
+    fs.mkdirSync(targetDir, { recursive: true });
+  }
+
+  // Перемещение файла
+  try {
+    fs.renameSync(sourcePath, targetPath);
+  } catch (e) {
+    return {
+      status: "error",
+      ticket_id: ticketId,
+      error: `Не удалось переместить файл: ${e.message}`,
+    };
+  }
+
+  // Запись обновлённого контента
+  try {
+    fs.writeFileSync(targetPath, newContent, "utf8");
+  } catch (e) {
+    return {
+      status: "error",
+      ticket_id: ticketId,
+      error: `Не удалось записать файл: ${e.message}`,
+    };
+  }
+
+  // Hook: обновление approval-файлов (если есть) — срабатывает на любой move
+  updateApprovalFilesHook(ticketId, target);
+
+  return {
+    status: "moved",
+    ticket_id: ticketId,
+    from: currentStatus,
+    to: target,
+  };
+}
+
+// Export for testing
+export { moveTicket, updateApprovalFilesHook };
+
+// Main entry point — guard prevents execution when imported as module in tests.
+// Используем fs.realpathSync чтобы корректно сравнивать пути на Windows когда .workflow/src/scripts/ — junction.
+// Без realpathSync argv[1] = путь через junction, а import.meta.url = разрешённый target — строки не совпадают.
+function __isMainModule() {
+  try {
+    const argvPath = fs.realpathSync(path.resolve(process.argv[1] || ""));
+    const metaPath = fileURLToPath(import.meta.url);
+    return argvPath === metaPath;
+  } catch {
+    return false;
+  }
+}
+
+if (__isMainModule()) {
+  const rawArgs = process.argv.slice(2);
+  let ticketId, target;
+
+  if (rawArgs.length >= 2) {
+    // Прямой вызов: node move-ticket.js IMPL-001 in-progress
+    ticketId = rawArgs[0];
+    target = rawArgs[1];
+  } else if (rawArgs.length === 1) {
+    // Вызов через pipeline runner: один аргумент — промпт с контекстом
+    // Формат: "skill-name\n\nContext:\n  ticket_id: X\n  target: Y\n..."
+    const prompt = rawArgs[0];
+    const ticketMatch = prompt.match(/ticket_id:\s*(\S+)/);
+    const targetMatch = prompt.match(/target:\s*(\S+)/);
+    ticketId = ticketMatch?.[1];
+    target = targetMatch?.[1];
+    if (!ticketId || !target) {
+      console.error(
+        "[ERROR] Cannot parse ticket_id or target from pipeline context",
+      );
       printResult({
         status: "error",
-        ticket_id: ticketId,
-        error: err.toString(),
+        error: "Missing ticket_id or target in pipeline context",
       });
+      process.exit(1);
     }
+  } else {
+    console.error("Usage: node move-ticket.js <ticket_id> <target>");
+    console.error("Example: node move-ticket.js IMPL-001 in-progress");
+    console.error("Available targets:", VALID_STATUSES.join(", "));
+    printResult({ status: "error", error: "Missing arguments" });
     process.exit(1);
   }
-}
 
-main();
+  moveTicket(ticketId, target).then((result) => {
+    printResult(result);
+    if (result.status === "error") {
+      process.exit(1);
+    }
+  });
+}