workflow-ai 1.0.68 → 1.1.0

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

@@ -1,110 +1,36 @@
 #!/usr/bin/env node
 
-/**
- * get-next-id.js - Универсальный генератор ID для тикетов, планов и других артефактов
- *
- * Режимы:
- *
- *   1) Одиночный префикс (legacy, для create-plan / create-report):
- *      node get-next-id.js --prefix TASK --dir tickets
- *      node get-next-id.js --prefix PLAN --dir plans
- *      Вывод: { status: "success", id: "TASK-007" }
- *
- *   2) Карта по всем префиксам из config.yaml (для decompose-plan):
- *      node get-next-id.js --all-from-config
- *      Читает .workflow/config/config.yaml → task_types.*.prefix,
- *      для каждого префикса возвращает следующий свободный номер по .workflow/tickets/.
- *      Вывод: { status: "success", id_ranges: { TASK: 8, QA: 26, HUMAN: 2, ... } }
- */
-
 import fs from "fs";
 import path from "path";
-import { findProjectRoot } from "workflow-ai/lib/find-root.mjs";
-import { printResult } from "workflow-ai/lib/utils.mjs";
+import { findProjectRoot } from "../lib/find-root.mjs";
+import { printResult } from "../lib/utils.mjs";
+import { getNextId } from "../lib/operations/tickets.mjs";
 
 const PROJECT_DIR = findProjectRoot();
 
 function parseArgs() {
   const args = process.argv.slice(2);
-  let prefix = null;
-  let dir = null;
-  let allFromConfig = false;
-
+  const result = { prefix: null, dir: null, allFromConfig: false };
   for (let i = 0; i < args.length; i++) {
-    if (args[i] === "--prefix" && i + 1 < args.length) {
-      prefix = args[i + 1];
-      i++;
-    } else if (args[i] === "--dir" && i + 1 < args.length) {
-      dir = args[i + 1];
-      i++;
-    } else if (args[i] === "--all-from-config") {
-      allFromConfig = true;
-    }
+    if (args[i] === "--prefix" && i + 1 < args.length) result.prefix = args[++i];
+    else if (args[i] === "--dir" && i + 1 < args.length) result.dir = args[++i];
+    else if (args[i] === "--all-from-config") result.allFromConfig = true;
   }
-
-  return { prefix, dir, allFromConfig };
+  return result;
 }
 
-function findMaxNumber(targetDir, prefix) {
-  let maxNum = 0;
-  const regex = new RegExp(`^${prefix}-(\\d+)\\.md$`, "i");
-
-  function scanDirectory(dir) {
-    if (!fs.existsSync(dir)) {
-      return;
-    }
-
-    const entries = fs.readdirSync(dir, { withFileTypes: true });
-
-    for (const entry of entries) {
-      const fullPath = path.join(dir, entry.name);
-
-      if (entry.isDirectory()) {
-        scanDirectory(fullPath);
-      } else if (entry.isFile()) {
-        const match = entry.name.match(regex);
-        if (match) {
-          const num = parseInt(match[1], 10);
-          if (num > maxNum) {
-            maxNum = num;
-          }
-        }
-      }
-    }
-  }
-
-  scanDirectory(targetDir);
-  return maxNum;
-}
-
-function formatNumber(num) {
-  return num.toString().padStart(3, "0");
-}
-
-/**
- * Извлекает значения prefix из .workflow/config/config.yaml → task_types.*.prefix.
- * Минимальный YAML-парсер: не тянем зависимость, читаем только нужную секцию.
- * Возвращает массив уникальных префиксов в порядке появления.
- */
 function readPrefixesFromConfig() {
   const configPath = path.join(PROJECT_DIR, ".workflow", "config", "config.yaml");
-
-  if (!fs.existsSync(configPath)) {
-    throw new Error(`config.yaml not found: ${configPath}`);
-  }
+  if (!fs.existsSync(configPath)) throw new Error("config.yaml not found");
 
   const text = fs.readFileSync(configPath, "utf8");
-  const lines = text.split(/\r?\n/);
-
   const prefixes = [];
-  let inTaskTypes = false;
-  let taskTypesIndent = -1;
+  const lines = text.split(/\r?\n/);
+  let inTaskTypes = false, taskTypesIndent = -1;
 
   for (const rawLine of lines) {
-    // Убрать комментарии
     const line = rawLine.replace(/\s+#.*$/, "").replace(/^#.*$/, "");
-    if (line.trim() === "") continue;
-
+    if (!line.trim()) continue;
     const indent = line.length - line.trimStart().length;
 
     if (!inTaskTypes) {
@@ -115,100 +41,48 @@ function readPrefixesFromConfig() {
       continue;
     }
 
-    // Вышли из секции task_types (вернулись на тот же или меньший отступ с новым ключом)
-    if (indent <= taskTypesIndent && line.trim() !== "") {
-      break;
-    }
-
+    if (indent <= taskTypesIndent && line.trim()) break;
     const m = line.match(/^\s*prefix\s*:\s*["']?([A-Z][A-Z0-9_]*)["']?\s*$/);
-    if (m) {
-      const p = m[1];
-      if (!prefixes.includes(p)) {
-        prefixes.push(p);
-      }
-    }
-  }
-
-  if (prefixes.length === 0) {
-    throw new Error("No prefixes found in config.yaml → task_types.*.prefix");
+    if (m && !prefixes.includes(m[1])) prefixes.push(m[1]);
   }
 
   return prefixes;
 }
 
 function resolveTargetDir(dir) {
-  if (dir === "tickets") {
-    return path.join(PROJECT_DIR, ".workflow", "tickets");
-  } else if (dir === "plans") {
-    return path.join(PROJECT_DIR, ".workflow", "plans");
-  } else {
-    return path.join(PROJECT_DIR, dir);
-  }
-}
-
-async function runSinglePrefix(prefix, dir) {
-  const targetDir = resolveTargetDir(dir);
-
-  if (!fs.existsSync(targetDir)) {
-    const nextId = `${prefix}-001`;
-    printResult({ status: "success", id: nextId });
-    return;
-  }
-
-  const maxNum = findMaxNumber(targetDir, prefix);
-  const nextNum = maxNum + 1;
-  const nextId = `${prefix}-${formatNumber(nextNum)}`;
-
-  printResult({ status: "success", id: nextId });
-}
-
-async function runAllFromConfig() {
-  const prefixes = readPrefixesFromConfig();
-  const ticketsDir = path.join(PROJECT_DIR, ".workflow", "tickets");
-
-  const idRanges = {};
-  for (const prefix of prefixes) {
-    const maxNum = fs.existsSync(ticketsDir) ? findMaxNumber(ticketsDir, prefix) : 0;
-    idRanges[prefix] = maxNum + 1;
-  }
-
-  // Параллельно возвращаем JSON-строку, потому что runner workflow-ai
-  // при подстановке $context.<key> в строку instructions применяет неявный
-  // toString() — объекты превращаются в "[object Object]". Скалярная
-  // JSON-строка подставляется корректно, декомпозитор распарсит её сам.
-  printResult({
-    status: "success",
-    id_ranges: idRanges,
-    id_ranges_json: JSON.stringify(idRanges),
-  });
+  if (dir === "tickets") return path.join(PROJECT_DIR, ".workflow", "tickets");
+  if (dir === "plans") return path.join(PROJECT_DIR, ".workflow", "plans");
+  return path.join(PROJECT_DIR, dir);
 }
 
 async function main() {
   const { prefix, dir, allFromConfig } = parseArgs();
 
-  if (allFromConfig) {
-    try {
-      await runAllFromConfig();
-    } catch (err) {
-      console.error(err.message);
-      printResult({ status: "error", error: err.message });
-      process.exit(1);
+  try {
+    if (allFromConfig) {
+      const prefixes = readPrefixesFromConfig();
+      const idRanges = {};
+      for (const p of prefixes) {
+        const id = await getNextId(PROJECT_DIR, p);
+        const num = parseInt(id.split("-")[1], 10);
+        idRanges[p] = num;
+      }
+      const bulkResult = { status: "success", id_ranges: idRanges, id_ranges_json: JSON.stringify(idRanges) };
+      console.log("---RESULT---");
+      console.log(JSON.stringify(bulkResult, null, 2));
+      console.log("---RESULT---");
+    } else {
+      if (!prefix || !dir) {
+        throw new Error("Either --all-from-config, or both --prefix and --dir required");
+      }
+      const id = await getNextId(PROJECT_DIR, prefix);
+      printResult({ status: "success", id });
     }
-    return;
-  }
-
-  if (!prefix || !dir) {
-    console.error("Usage:");
-    console.error("  node get-next-id.js --prefix <PREFIX> --dir <DIRECTORY>");
-    console.error("  node get-next-id.js --all-from-config");
-    printResult({
-      status: "error",
-      error: "Missing required arguments: either --all-from-config, or both --prefix and --dir",
-    });
+  } catch (err) {
+    console.error(err.message);
+    printResult({ status: "error", error: err.message });
     process.exit(1);
   }
-
-  await runSinglePrefix(prefix, dir);
 }
 
 main();

package/src/scripts/move-ticket.js

@@ -1,222 +1,29 @@
 #!/usr/bin/env node
 
 /**
- * move-ticket.js - Скрипт для перемещения тикетов между директориями канбан-доски
+ * move-ticket.js - Тонкая прослойка: парсинг argv → вызов operation → печать RESULT
  *
- * Использование:
- *   node move-ticket.js <ticket_id> <target>
+ * Контракт:
+ *   1. Прямой режим: node move-ticket.js <id> <target>
+ *   2. Runner-режим: node move-ticket.js "<prompt-block>" (с regex ticket_id: и target:)
  *
- * Пример:
- *   node move-ticket.js IMPL-001 in-progress
+ * 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>
  */
 
-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,
-  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,
-  };
-}
+import { moveTicket } from "../lib/operations/tickets.mjs";
+import { printResult } from "workflow-ai/lib/utils.mjs";
 
 // Main entry point
 const rawArgs = process.argv.slice(2);
@@ -227,34 +34,70 @@ if (rawArgs.length >= 2) {
   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..."
+  // 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) {
-    console.error(
-      "[ERROR] Cannot parse ticket_id or target from pipeline context",
-    );
     printResult({
       status: "error",
-      error: "Missing ticket_id or target in pipeline context",
+      ticket_id: ticketId || "",
+      from: "",
+      to: target || "",
+      error: "Missing ticket_id or target",
     });
     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" });
+  printResult({
+    status: "error",
+    error: "Missing arguments",
+  });
   process.exit(1);
 }
 
-moveTicket(ticketId, target).then((result) => {
-  printResult(result);
-  if (result.status === "error") {
+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(),
+      });
+    }
     process.exit(1);
   }
-});
+}
+
+main();