workflow-ai 1.0.22 → 1.0.24

package/configs/pipeline.yaml

@@ -97,6 +97,12 @@ pipeline:
       workdir: "."
       description: "Скрипт для проверки условий тикетов в backlog/"
 
+    script-check-plan-decomposed:
+      command: "node"
+      args: [".workflow/src/scripts/check-plan-decomposed.js"]
+      workdir: "."
+      description: "Скрипт для проверки, декомпозирован ли текущий план"
+
     script-move-to-review:
       command: "node"
       args: [".workflow/src/scripts/move-to-review.js"]
@@ -167,9 +173,40 @@ pipeline:
           stage: move-to-review
           params:
             ticket_id: "$result.ticket_id"
-        empty: check-conditions
+        empty: check-plan-decomposition
         error: create-report
 
+    # -------------------------------------------------------------------------
+    # 0b. check-plan-decomposition
+    # Если нет тикетов — проверяет, есть ли недекомпозированный план.
+    # Если план не декомпозирован — декомпозирует перед check-conditions.
+    # -------------------------------------------------------------------------
+    check-plan-decomposition:
+      description: "Проверить, декомпозирован ли текущий план"
+      agent: script-check-plan-decomposed
+      goto:
+        needs_decomposition:
+          stage: decompose-plan
+          params:
+            plan_file: "$result.plan_file"
+        decomposed: check-conditions
+        no_plan: check-conditions
+        default: check-conditions
+        error: check-conditions
+
+    # -------------------------------------------------------------------------
+    # 0c. decompose-plan
+    # Декомпозирует план на тикеты в backlog/
+    # -------------------------------------------------------------------------
+    decompose-plan:
+      description: "Декомпозировать план на тикеты"
+      agent: claude-sonnet
+      fallback_agent: qwen-code
+      skill: decompose-plan
+      instructions: "Декомпозируй план .workflow/$context.plan_file на тикеты."
+      goto:
+        default: check-conditions
+
     # -------------------------------------------------------------------------
     # 1. check-conditions
     # Проверяет условия тикетов в backlog/, выводит список готовых
@@ -482,6 +519,6 @@ pipeline:
   # ===========================================================================
   execution:
     max_steps: 1500
-    delay_between_stages: 1
+    delay_between_stages: 5
     timeout_per_stage: 1800
     log_file: ".workflow/logs/pipeline.log"

package/package.json

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

package/src/lib/utils.mjs

@@ -117,10 +117,16 @@ export function getLastReviewStatus(content) {
   if (!content) return null;
 
   // Находим секцию "## Ревью" — захватываем всё до следующего заголовка ## или конца файла
-  const reviewSectionMatch = content.match(/^##\s*Ревью\s*\n([\s\S]*)(?=\n^##\s|$)/m);
-  if (!reviewSectionMatch) return null;
+  const headerIdx = content.search(/^##\s*Ревью\s*$/m);
+  if (headerIdx === -1) return null;
 
-  const reviewSection = reviewSectionMatch[1].trim();
+  const bodyStart = content.indexOf('\n', headerIdx);
+  if (bodyStart === -1) return null;
+
+  const nextH2 = content.indexOf('\n## ', bodyStart);
+  const reviewSection = (nextH2 === -1
+    ? content.slice(bodyStart + 1)
+    : content.slice(bodyStart + 1, nextH2)).trim();
   if (!reviewSection) return null;
 
   // Пробуем распарсить табличный формат

package/src/scripts/check-plan-decomposed.js

@@ -0,0 +1,156 @@
+#!/usr/bin/env node
+
+/**
+ * check-plan-decomposed.js — Проверяет, есть ли недекомпозированные планы.
+ *
+ * Логика:
+ *   A) Если plan_id задан — проверяет только этот план
+ *   B) Если plan_id НЕ задан — сканирует все планы в plans/current/
+ *
+ *   Для каждого плана:
+ *   1. Если есть тикеты (backlog/, ready/, in-progress/, review/) с parent_plan == planId — decomposed
+ *   2. Иначе — needs_decomposition
+ *
+ *   Результат:
+ *   - needs_decomposition + plan_file — найден первый недекомпозированный план
+ *   - decomposed — все планы декомпозированы
+ *   - no_plan — нет планов в plans/current/
+ *
+ * Использование:
+ *   node check-plan-decomposed.js "plan_id: PLAN-007"
+ *   node check-plan-decomposed.js
+ */
+
+import fs from 'fs';
+import path from 'path';
+import { findProjectRoot } from '../lib/find-root.mjs';
+import { parseFrontmatter, printResult, normalizePlanId, extractPlanId } from '../lib/utils.mjs';
+
+const PROJECT_DIR = findProjectRoot();
+const WORKFLOW_DIR = path.join(PROJECT_DIR, '.workflow');
+const TICKETS_DIR = path.join(WORKFLOW_DIR, 'tickets');
+const PLANS_DIR = path.join(WORKFLOW_DIR, 'plans', 'current');
+
+const TICKET_DIRS = ['backlog', 'ready', 'in-progress', 'review'];
+
+/**
+ * Проверяет, есть ли тикеты, привязанные к данному плану
+ */
+function hasTicketsForPlan(planId) {
+  for (const dir of TICKET_DIRS) {
+    const dirPath = path.join(TICKETS_DIR, dir);
+    if (!fs.existsSync(dirPath)) continue;
+
+    const files = fs.readdirSync(dirPath)
+      .filter(f => f.endsWith('.md') && f !== '.gitkeep.md');
+
+    for (const file of files) {
+      try {
+        const content = fs.readFileSync(path.join(dirPath, file), 'utf8');
+        const { frontmatter } = parseFrontmatter(content);
+        if (normalizePlanId(frontmatter.parent_plan) === planId) {
+          return true;
+        }
+      } catch (e) {
+        console.error(`[WARN] Failed to read ${dir}/${file}: ${e.message}`);
+      }
+    }
+  }
+  return false;
+}
+
+/**
+ * Находит файл плана в plans/current/
+ */
+function findPlanFile(planId) {
+  if (!fs.existsSync(PLANS_DIR)) return null;
+
+  const expectedName = `${planId}.md`;
+  const filePath = path.join(PLANS_DIR, expectedName);
+  if (fs.existsSync(filePath)) {
+    return `plans/current/${expectedName}`;
+  }
+
+  // Поиск по всем файлам на случай другого именования
+  const files = fs.readdirSync(PLANS_DIR).filter(f => f.endsWith('.md'));
+  for (const file of files) {
+    if (normalizePlanId(file) === planId) {
+      return `plans/current/${file}`;
+    }
+  }
+
+  return null;
+}
+
+/**
+ * Возвращает все файлы планов из plans/current/
+ */
+function getAllPlanFiles() {
+  if (!fs.existsSync(PLANS_DIR)) return [];
+
+  return fs.readdirSync(PLANS_DIR)
+    .filter(f => f.endsWith('.md'))
+    .map(f => ({
+      planId: normalizePlanId(f),
+      planFile: `plans/current/${f}`
+    }))
+    .filter(p => p.planId !== null);
+}
+
+async function main() {
+  const planId = extractPlanId();
+
+  if (planId) {
+    // Режим A: конкретный план
+    console.log(`[INFO] Checking decomposition for plan: ${planId}`);
+
+    const planFile = findPlanFile(planId);
+    if (!planFile) {
+      console.log(`[INFO] Plan ${planId} not found in plans/current/`);
+      printResult({ status: 'no_plan' });
+      return;
+    }
+
+    console.log(`[INFO] Found plan file: ${planFile}`);
+
+    if (hasTicketsForPlan(planId)) {
+      console.log(`[INFO] Plan ${planId} already has tickets — decomposed`);
+      printResult({ status: 'decomposed' });
+      return;
+    }
+
+    console.log(`[INFO] Plan ${planId} has no tickets — needs decomposition`);
+    printResult({ status: 'needs_decomposition', plan_file: planFile });
+    return;
+  }
+
+  // Режим B: сканируем все планы в plans/current/
+  console.log('[INFO] No plan_id specified, scanning all plans in plans/current/');
+
+  const allPlans = getAllPlanFiles();
+  if (allPlans.length === 0) {
+    console.log('[INFO] No plans found in plans/current/');
+    printResult({ status: 'no_plan' });
+    return;
+  }
+
+  console.log(`[INFO] Found ${allPlans.length} plan(s) in plans/current/`);
+
+  for (const { planId: pid, planFile } of allPlans) {
+    if (!hasTicketsForPlan(pid)) {
+      console.log(`[INFO] Plan ${pid} has no tickets — needs decomposition`);
+      printResult({ status: 'needs_decomposition', plan_file: planFile });
+      return;
+    }
+    console.log(`[INFO] Plan ${pid} already decomposed`);
+  }
+
+  console.log('[INFO] All plans are decomposed');
+  printResult({ status: 'decomposed' });
+}
+
+main().catch(e => {
+  console.error(`[ERROR] ${e.message}`);
+  printResult({ status: 'error', error: e.message });
+  process.exit(1);
+});

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

@@ -83,6 +83,8 @@ ID тикета передаётся в промпте как `ticket_id` в с
 
 ### 4. Сформировать вердикт
 
+> **ВАЖНО:** Review-result НИКОГДА не пишет статус `skipped`. Допустимы только `passed` или `failed`. Статус `skipped` — прерогатива check-relevance.
+
 Возвращать структурированный результат строго в одном из двух форматов:
 
 > **КРИТИЧНО**: `status` принимает ТОЛЬКО два значения: `passed` или `failed`.