workflow-ai 1.1.0 → 1.2.1

package/src/scripts/move-ticket.js

@@ -1,29 +1,222 @@
 #!/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 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";
+import {
+  parseFrontmatter,
+  printResult,
+  serializeFrontmatter,
+  getLastReviewStatus,
+} 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");
+
+// Доступные статусы
+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;
+    }
+  }
+  return null;
+}
+
+/**
+ * Проверяет допустимость перехода
+ */
+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 };
+}
+
+/**
+ * Основная функция перемещения тикета
+ */
+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}`,
+    };
+  }
+
+  return {
+    status: "moved",
+    ticket_id: ticketId,
+    from: currentStatus,
+    to: target,
+  };
+}
 
 // Main entry point
 const rawArgs = process.argv.slice(2);
@@ -34,70 +227,34 @@ if (rawArgs.length >= 2) {
   ticketId = rawArgs[0];
   target = rawArgs[1];
 } else if (rawArgs.length === 1) {
-  // Runner-режим: один аргумент — многострочный текст с ticket_id: и target:
+  // Вызов через 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 || "",
-      from: "",
-      to: target || "",
-      error: "Missing ticket_id or target",
+      error: "Missing ticket_id or target in pipeline context",
     });
     process.exit(1);
   }
 } else {
-  printResult({
-    status: "error",
-    error: "Missing arguments",
-  });
+  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);
 }
 
-async function main() {
-  try {
-    const projectRoot = findProjectRoot();
-    const result = await moveTicket(projectRoot, ticketId, target);
-    printResult({
-      status: "moved",
-      ticket_id: ticketId,
-      from: result.from,
-      to: result.to,
-    });
-  } 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 {
-      printResult({
-        status: "error",
-        ticket_id: ticketId,
-        error: err.toString(),
-      });
-    }
+moveTicket(ticketId, target).then((result) => {
+  printResult(result);
+  if (result.status === "error") {
     process.exit(1);
   }
-}
-
-main();
+});