workflow-ai 1.0.10 → 1.0.12

package/src/scripts/check-conditions.js

@@ -28,7 +28,7 @@
 import fs from 'fs';
 import path from 'path';
 import { findProjectRoot } from '../lib/find-root.mjs';
-import { parseFrontmatter, printResult, normalizePlanId, extractPlanId } from '../lib/utils.mjs';
+import { parseFrontmatter, printResult, normalizePlanId, extractPlanId, serializeFrontmatter } from '../lib/utils.mjs';
 
 const PROJECT_DIR = findProjectRoot();
 const WORKFLOW_DIR = path.join(PROJECT_DIR, '.workflow');
@@ -102,6 +102,34 @@ function readTickets(dir) {
   return tickets;
 }
 
+/**
+ * Перемещает тикет из ready/ в backlog/
+ */
+function demoteToBacklog(ticketId) {
+  const sourcePath = path.join(READY_DIR, `${ticketId}.md`);
+  const targetPath = path.join(BACKLOG_DIR, `${ticketId}.md`);
+
+  if (!fs.existsSync(sourcePath)) {
+    console.error(`[WARN] ${ticketId}: not found in ready/, skipping`);
+    return false;
+  }
+
+  const content = fs.readFileSync(sourcePath, 'utf8');
+  const { frontmatter, body } = parseFrontmatter(content);
+
+  frontmatter.updated_at = new Date().toISOString();
+
+  const newContent = serializeFrontmatter(frontmatter) + body;
+
+  if (!fs.existsSync(BACKLOG_DIR)) {
+    fs.mkdirSync(BACKLOG_DIR, { recursive: true });
+  }
+
+  fs.renameSync(sourcePath, targetPath);
+  fs.writeFileSync(targetPath, newContent, 'utf8');
+  return true;
+}
+
 /**
  * Проверяет все тикеты в backlog/ и возвращает список готовых
  */
@@ -137,6 +165,36 @@ function checkBacklog(planId) {
   return { ready, waiting, total: tickets.length };
 }
 
+/**
+ * Проверяет тикеты в ready/ и возвращает тикеты в backlog при невыполненных условиях
+ */
+function checkReady(planId) {
+  const allTickets = readTickets(READY_DIR);
+  const tickets = planId
+    ? allTickets.filter(t => normalizePlanId(t.frontmatter.parent_plan) === planId)
+    : allTickets;
+
+  const demoted = [];
+
+  for (const ticket of tickets) {
+    const { frontmatter, id } = ticket;
+    const conditions = frontmatter.conditions || [];
+    const dependencies = frontmatter.dependencies || [];
+
+    const depsMet = checkDependencies(dependencies);
+    const conditionsMet = conditions.every(checkCondition);
+
+    if (!depsMet || !conditionsMet) {
+      if (demoteToBacklog(id)) {
+        console.log(`[INFO] ${id}: ready/ → backlog/ (условия не выполнены)`);
+        demoted.push(id);
+      }
+    }
+  }
+
+  return { demoted, total: tickets.length };
+}
+
 async function main() {
   const planId = extractPlanId();
 
@@ -144,6 +202,13 @@ async function main() {
     console.log(`[INFO] Filtering by plan_id: ${planId}`);
   }
 
+  // Сначала демотирование невалидных тикетов из ready/
+  console.log(`[INFO] Checking ready/ for invalid tickets: ${READY_DIR}`);
+  const { demoted, total: readyTotal } = checkReady(planId);
+  console.log(`[INFO] Total in ready/${planId ? ` (plan ${planId})` : ''}: ${readyTotal}`);
+  console.log(`[INFO] Demoted to backlog: ${demoted.length}`);
+
+  // Затем проверка backlog — демотированные тикеты сразу переоцениваются
   console.log(`[INFO] Scanning backlog/: ${BACKLOG_DIR}`);
 
   const { ready, waiting, total } = checkBacklog(planId);
@@ -160,7 +225,7 @@ async function main() {
   }
 
   if (ready.length > 0) {
-    printResult({ status: 'has_ready', ready_tickets: ready.join(', ') });
+    printResult({ status: 'has_ready', ready_tickets: ready.join(', '), demoted_tickets: demoted.join(', ') });
     return;
   }
 
@@ -168,10 +233,10 @@ async function main() {
   const readyDirTickets = readTickets(READY_DIR);
   if (readyDirTickets.length > 0) {
     console.log(`[INFO] No new ready tickets, but ready/ has ${readyDirTickets.length} ticket(s)`);
-    printResult({ status: 'default', ready_tickets: '' });
+    printResult({ status: 'default', ready_tickets: '', demoted_tickets: demoted.join(', ') });
   } else {
     console.log('[INFO] No ready tickets and ready/ is empty');
-    printResult({ status: 'empty', ready_tickets: '' });
+    printResult({ status: 'empty', ready_tickets: '', demoted_tickets: demoted.join(', ') });
   }
 }
 

package/src/scripts/move-ticket.js

@@ -27,12 +27,12 @@ const VALID_STATUSES = ['backlog', 'ready', 'in-progress', 'blocked', 'review',
 
 // Таблица допустимых переходов
 const VALID_TRANSITIONS = {
-  'backlog': ['ready'],
-  'ready': ['in-progress', 'review'],
+  'backlog': ['ready', 'blocked', 'done'],
+  'ready': ['in-progress', 'review', 'backlog'],
   'in-progress': ['done', 'blocked', 'review'],
   'blocked': ['ready'],
   'review': ['done', 'ready', 'in-progress', 'blocked'],
-  'done': []
+  'done': ['ready', 'blocked']
 };
 
 /**

package/src/scripts/pick-next-task.js

@@ -21,7 +21,7 @@
 import fs from 'fs';
 import path from 'path';
 import { findProjectRoot } from '../lib/find-root.mjs';
-import { parseFrontmatter, printResult, normalizePlanId, extractPlanId } from '../lib/utils.mjs';
+import { parseFrontmatter, printResult, normalizePlanId, extractPlanId, getLastReviewStatus, serializeFrontmatter } from '../lib/utils.mjs';
 
 // Корень проекта
 const PROJECT_DIR = findProjectRoot();
@@ -31,6 +31,9 @@ const TICKETS_DIR = path.join(WORKFLOW_DIR, 'tickets');
 const READY_DIR = path.join(TICKETS_DIR, 'ready');
 const DONE_DIR = path.join(TICKETS_DIR, 'done');
 const IN_PROGRESS_DIR = path.join(TICKETS_DIR, 'in-progress');
+const BLOCKED_DIR = path.join(TICKETS_DIR, 'blocked');
+const REVIEW_DIR = path.join(TICKETS_DIR, 'review');
+const BACKLOG_DIR = path.join(TICKETS_DIR, 'backlog');
 
 
 /**
@@ -89,6 +92,164 @@ function checkDependencies(dependencies) {
   });
 }
 
+/**
+ * Авто-коррекция тикетов на основе статуса ревью.
+ * Сканирует все директории и перемещает тикеты по правилам:
+ * - blocked → done (если review = passed)
+ * - blocked → backlog (если review = failed, НЕ при null)
+ * - done → backlog (если review = failed)
+ * - review → done (если review = passed)
+ * - in-progress → done (если review = passed)
+ * - ready → done (если review = passed)
+ *
+ * @returns {object} Результат: { moved: Array<{id, from, to, reason}> }
+ */
+function autoCorrectTickets() {
+  const moved = [];
+
+  /**
+   * Перемещает тикет из одной директории в другую
+   */
+  function moveTicket(ticketId, fromDir, toDir, reason) {
+    const fromPath = path.join(fromDir, `${ticketId}.md`);
+    const toPath = path.join(toDir, `${ticketId}.md`);
+
+    if (!fs.existsSync(fromPath)) {
+      return false;
+    }
+
+    try {
+      // Читаем содержимое
+      const content = fs.readFileSync(fromPath, 'utf8');
+      const { frontmatter, body } = parseFrontmatter(content);
+
+      // Обновляем updated_at
+      frontmatter.updated_at = new Date().toISOString();
+
+      // Если перемещаем в done — ставим completed_at
+      if (toDir === DONE_DIR && !frontmatter.completed_at) {
+        frontmatter.completed_at = new Date().toISOString();
+      }
+
+      // Сериализуем и записываем в новую директорию
+      const newContent = serializeFrontmatter(frontmatter) + body;
+      fs.writeFileSync(toPath, newContent, 'utf8');
+
+      // Удаляем старый файл
+      fs.unlinkSync(fromPath);
+
+      console.log(`[AUTO-CORRECT] ${ticketId}: ${path.basename(fromDir)} → ${path.basename(toDir)} (${reason})`);
+
+      moved.push({
+        id: ticketId,
+        from: path.basename(fromDir),
+        to: path.basename(toDir),
+        reason
+      });
+
+      return true;
+    } catch (e) {
+      console.error(`[ERROR] Failed to move ticket ${ticketId}: ${e.message}`);
+      return false;
+    }
+  }
+
+  /**
+   * Обрабатывает тикеты в указанной директории
+   */
+  function processDirectory(dir, rules) {
+    if (!fs.existsSync(dir)) return;
+
+    const files = fs.readdirSync(dir)
+      .filter(f => f.endsWith('.md') && f !== '.gitkeep.md');
+
+    for (const file of files) {
+      const filePath = path.join(dir, file);
+      try {
+        const content = fs.readFileSync(filePath, 'utf8');
+        const { frontmatter } = parseFrontmatter(content);
+        const ticketId = frontmatter.id || file.replace('.md', '');
+
+        // Получаем статус ревью
+        const reviewStatus = getLastReviewStatus(content);
+
+        // Применяем правила
+        for (const rule of rules) {
+          if (rule.condition(reviewStatus)) {
+            moveTicket(ticketId, dir, rule.toDir, rule.reason);
+            break; // Только одно перемещение на тикет
+          }
+        }
+      } catch (e) {
+        console.error(`[WARN] Failed to process ticket ${file}: ${e.message}`);
+      }
+    }
+  }
+
+  // Правила для backlog/ (защита от ошибочного перемещения завершённых тикетов)
+  processDirectory(BACKLOG_DIR, [
+    {
+      condition: (status) => status === 'passed',
+      toDir: DONE_DIR,
+      reason: 'review passed'
+    }
+  ]);
+
+  // Правила для blocked/
+  processDirectory(BLOCKED_DIR, [
+    {
+      condition: (status) => status === 'passed',
+      toDir: DONE_DIR,
+      reason: 'review passed'
+    },
+    {
+      condition: (status) => status === 'failed',
+      toDir: BACKLOG_DIR,
+      reason: 'review failed'
+    }
+    // null (нет ревью) — не перемещаем
+  ]);
+
+  // Правила для done/
+  processDirectory(DONE_DIR, [
+    {
+      condition: (status) => status === 'failed',
+      toDir: BACKLOG_DIR,
+      reason: 'review failed'
+    }
+    // passed или null — не перемещаем (важно для legacy-тикетов без ревью)
+  ]);
+
+  // Правила для review/
+  processDirectory(REVIEW_DIR, [
+    {
+      condition: (status) => status === 'passed',
+      toDir: DONE_DIR,
+      reason: 'review passed'
+    }
+  ]);
+
+  // Правила для in-progress/
+  processDirectory(IN_PROGRESS_DIR, [
+    {
+      condition: (status) => status === 'passed',
+      toDir: DONE_DIR,
+      reason: 'review passed'
+    }
+  ]);
+
+  // Правила для ready/
+  processDirectory(READY_DIR, [
+    {
+      condition: (status) => status === 'passed',
+      toDir: DONE_DIR,
+      reason: 'review passed'
+    }
+  ]);
+
+  return { moved };
+}
+
 /**
  * Считывает все тикеты из директории ready/
  */
@@ -329,6 +490,13 @@ async function main() {
     console.log(`[INFO] Filtering by plan_id: ${planId}`);
   }
 
+  // Авто-коррекция тикетов перед выбором задачи
+  console.log('[INFO] Running auto-correction...');
+  const correctionResult = autoCorrectTickets();
+  if (correctionResult.moved.length > 0) {
+    console.log(`[INFO] Auto-corrected ${correctionResult.moved.length} ticket(s)`);
+  }
+
   console.log(`[INFO] Scanning ready/ directory: ${READY_DIR}`);
 
   const result = pickNextTicket(planId);
@@ -340,7 +508,14 @@ async function main() {
     console.log(`[INFO] ${result.reason}`);
   }
 
-  printResult(result);
+  // Добавляем информацию о авто-коррекции в результат
+  const finalResult = {
+    ...result,
+    auto_corrected: correctionResult.moved.length,
+    moved_tickets: correctionResult.moved.map(m => m.id).join(',')
+  };
+
+  printResult(finalResult);
 
   if (result.status === 'empty') {
     process.exit(0);