workflow-ai 1.0.54 → 1.0.55

package/README.md

@@ -52,6 +52,7 @@ The `workflow init` command creates the `.workflow/` directory structure:
 │   └── ticket-movement-rules.yaml
 ├── plans/
 │   ├── current/                  # Current development plans
+│   ├── templates/                # Plan templates with triggers (recurring plans)
 │   └── archive/                  # Archived plans
 ├── tickets/
 │   ├── backlog/                  # Awaiting conditions
@@ -74,20 +75,21 @@ The `workflow init` command creates the `.workflow/` directory structure:
 The `workflow run` command executes a multi-stage pipeline:
 
 1. **pick-first-task** — select ticket from ready queue
-2. **check-plan-decomposition** — verify plan is decomposed into tickets
-3. **decompose-plan** — break down plan into tickets (if needed)
-4. **check-conditions** — validate ticket readiness conditions
-5. **move-to-ready** — move tickets from backlog to ready
-6. **pick-next-task** — select next ticket for execution
-7. **move-to-in-progress** — start execution
-8. **check-relevance** — verify ticket is still relevant
-9. **execute-task** — perform the work via AI agent
-10. **move-to-review** — submit for review
-11. **review-result** — validate results against Definition of Done
-12. **increment-task-attempts** — track retry attempts
-13. **move-ticket** — move to done/blocked based on review
-14. **create-report** — generate execution report
-15. **analyze-report / decompose-gaps** — analyze results and iterate
+2. **check-plan-templates** — evaluate plan template triggers, create plans if fired
+3. **check-plan-decomposition** — verify plan is decomposed into tickets
+4. **decompose-plan** — break down plan into tickets (if needed)
+5. **check-conditions** — validate ticket readiness conditions
+6. **move-to-ready** — move tickets from backlog to ready
+7. **pick-next-task** — select next ticket for execution
+8. **move-to-in-progress** — start execution
+9. **check-relevance** — verify ticket is still relevant
+10. **execute-task** — perform the work via AI agent
+11. **move-to-review** — submit for review
+12. **review-result** — validate results against Definition of Done
+13. **increment-task-attempts** — track retry attempts
+14. **move-ticket** — move to done/blocked based on review
+15. **create-report** — generate execution report
+16. **analyze-report / decompose-gaps** — analyze results and iterate
 
 ### Supported Agents
 
@@ -121,6 +123,34 @@ Skills are stored globally in `~/.workflow/skills/` and symlinked into projects.
 
 Use `workflow eject <skill>` to copy a skill into the project for customization.
 
+## Plan Templates
+
+Plan templates allow recurring plans to be created automatically. Templates live in `.workflow/plans/templates/` and contain trigger conditions in their frontmatter.
+
+### Template Format
+
+```yaml
+id: "TMPL-001"
+title: "Daily manual testing"
+type: template
+trigger:
+  type: daily          # daily | weekly | date_after | interval_days
+  params: {}           # type-specific params
+last_triggered: ""     # auto-updated on trigger
+enabled: true
+```
+
+### Trigger Types
+
+| Type | Params | Description |
+|------|--------|-------------|
+| `daily` | — | Once per day |
+| `weekly` | `days_of_week: [1,3,5]` (0=Sun) | On specific weekdays |
+| `date_after` | `date: "2026-04-01"` | Once after a specific date |
+| `interval_days` | `days: 3` | Every N days |
+
+When a trigger fires, the pipeline creates a plan in `plans/current/` with status `approved`, then the normal decomposition flow proceeds.
+
 ## Task Types
 
 | Type | Prefix | Description |

package/configs/pipeline.yaml

@@ -543,6 +543,15 @@ pipeline:
     - ".workflow/plans/**"       # Планы — только для чтения
     - ".workflow/config/**"      # Конфигурация
 
+  # ===========================================================================
+  # Доверенные агенты (trusted_agents)
+  # ===========================================================================
+  # Агенты, для которых FileGuard не откатывает изменения в protected_files.
+  # Поддерживает glob-паттерны: "script-*" соответствует "script-move" и т.д.
+  # ===========================================================================
+  trusted_agents:
+    - "script-check-templates"   # Создаёт планы в plans/current/
+
   # ===========================================================================
   # Настройки выполнения (execution)
   # ===========================================================================

package/package.json

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

package/src/runner.mjs

@@ -7,143 +7,6 @@ import crypto from 'crypto';
 import yaml from './lib/js-yaml.mjs';
 import { findProjectRoot } from './lib/find-root.mjs';
 
-// ============================================================================
-// ProcessTracker — файловый трекинг дочерних процессов.
-//
-// Проблема: runner spawn → claude CLI → npx → node (MCP-сервер).
-// При жёстком завершении runner'а (kill, закрытие терминала) process.on('exit')
-// не срабатывает, MCP-серверы остаются сиротами навсегда.
-//
-// Решение: PID-файл (.workflow/logs/.runner-pids).
-// - Перед spawn: записываем снапшот существующих PID'ов в файл
-// - При старте runner'а: читаем файл, вычисляем сирот (текущие − снапшот), убиваем
-// - Файл переживает крэш/kill — cleanup гарантирован при следующем запуске
-// ============================================================================
-const PIDFILE_NAME = '.runner-pids';
-
-function getPidFilePath(projectRoot) {
-  return path.resolve(projectRoot, '.workflow/logs', PIDFILE_NAME);
-}
-
-function getProcessSnapshot() {
-  if (process.platform !== 'win32') return new Set();
-  try {
-    const output = execSync(
-      'wmic process where "name=\'node.exe\' or name=\'python.exe\' or name=\'python3.exe\' or name=\'cmd.exe\'" get ProcessId /FORMAT:LIST',
-      { stdio: ['pipe', 'pipe', 'pipe'], encoding: 'utf-8', timeout: 5000 }
-    );
-    const pids = new Set();
-    for (const line of output.split('\n')) {
-      const match = line.match(/ProcessId=(\d+)/);
-      if (match) pids.add(parseInt(match[1], 10));
-    }
-    return pids;
-  } catch {
-    return new Set();
-  }
-}
-
-/**
- * Записывает снапшот + runner PID в файл.
- * Формат: JSON { runnerPid, snapshot: [...], timestamp }
- */
-function writePidFile(pidFilePath, snapshot) {
-  try {
-    const dir = path.dirname(pidFilePath);
-    if (!fs.existsSync(dir)) fs.mkdirSync(dir, { recursive: true });
-    fs.writeFileSync(pidFilePath, JSON.stringify({
-      runnerPid: process.pid,
-      snapshot: [...snapshot],
-      timestamp: new Date().toISOString()
-    }));
-  } catch { /* non-critical */ }
-}
-
-/**
- * Читает PID-файл. Возвращает { runnerPid, snapshot } или null.
- */
-function readPidFile(pidFilePath) {
-  try {
-    if (!fs.existsSync(pidFilePath)) return null;
-    const data = JSON.parse(fs.readFileSync(pidFilePath, 'utf-8'));
-    return {
-      runnerPid: data.runnerPid,
-      snapshot: new Set(data.snapshot || [])
-    };
-  } catch {
-    return null;
-  }
-}
-
-function removePidFile(pidFilePath) {
-  try { fs.unlinkSync(pidFilePath); } catch { /* ok */ }
-}
-
-/**
- * Cleanup сирот от предыдущего запуска.
- * Читает PID-файл, берёт текущие процессы, убивает разницу (текущие − снапшот).
- * Пропускает собственный PID runner'а.
- */
-function cleanupOrphansFromFile(pidFilePath, logger = null) {
-  const saved = readPidFile(pidFilePath);
-  if (!saved) return;
-
-  // Проверяем что предыдущий runner мёртв (иначе не трогаем — он ещё работает)
-  if (saved.runnerPid && isProcessAlive(saved.runnerPid)) {
-    return;
-  }
-
-  const currentPids = getProcessSnapshot();
-  const orphans = [];
-  for (const pid of currentPids) {
-    if (!saved.snapshot.has(pid) && pid !== process.pid) {
-      orphans.push(pid);
-    }
-  }
-
-  if (orphans.length > 0) {
-    killPids(orphans, logger);
-  }
-
-  removePidFile(pidFilePath);
-}
-
-function isProcessAlive(pid) {
-  try {
-    process.kill(pid, 0);
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-function killPids(pids, logger = null) {
-  if (pids.length === 0) return;
-  if (process.platform === 'win32') {
-    for (const pid of pids) {
-      try {
-        execSync(`taskkill /pid ${pid} /F`, { stdio: 'pipe' });
-      } catch { /* already dead */ }
-    }
-  } else {
-    for (const pid of pids) {
-      try { process.kill(pid, 'SIGKILL'); } catch { /* already dead */ }
-    }
-  }
-  if (logger) {
-    logger.info(`Cleaned up ${pids.length} orphan process(es): ${pids.join(', ')}`, 'ProcessCleanup');
-  }
-}
-
-function killChildProcess(child) {
-  if (!child.pid) return;
-  if (process.platform === 'win32') {
-    try { execSync(`taskkill /pid ${child.pid} /T /F`, { stdio: 'pipe' }); } catch {}
-  } else {
-    try { child.kill('SIGTERM'); } catch {}
-  }
-}
-
 // ============================================================================
 // Logger — система логирования с уровнями DEBUG/INFO/WARN/ERROR
 // ============================================================================
@@ -899,36 +762,21 @@ class StageExecutor {
     this.promptBuilder = new PromptBuilder(context, counters, previousResults);
     this.resultParser = new ResultParser();
 
-    // Трекинг активных дочерних процессов для cleanup
-    this.activeChildren = new Set();
-    // Снапшот PID'ов перед spawn — для вычисления порождённых процессов
-    this.preSpawnSnapshot = new Set();
+    // Текущий дочерний процесс агента (для kill при shutdown)
+    this.currentChild = null;
   }
 
   /**
-   * Убивает все активные дочерние процессы и порождённых ими сирот.
-   * Snapshot-based: сравнивает текущие PID'ы с снапшотом до spawn,
-   * новые процессы = порождённые нашим агентом (включая внуков/MCP-серверов).
+   * Убивает текущий дочерний процесс агента
    */
-  killAllChildren() {
-    // Убиваем прямых детей через taskkill /T (best effort)
-    for (const child of this.activeChildren) {
-      killChildProcess(child);
-    }
-    this.activeChildren.clear();
-
-    // Snapshot-based: убиваем все процессы, появившиеся после spawn
-    if (this.preSpawnSnapshot.size > 0) {
-      const currentPids = getProcessSnapshot();
-      const orphans = [];
-      for (const pid of currentPids) {
-        if (!this.preSpawnSnapshot.has(pid) && pid !== process.pid) {
-          orphans.push(pid);
-        }
-      }
-      killPids(orphans, this.logger);
+  killCurrentChild() {
+    const child = this.currentChild;
+    if (!child || !child.pid) return;
+    if (process.platform === 'win32') {
+      try { execSync(`taskkill /pid ${child.pid} /T /F`, { stdio: 'pipe' }); } catch {}
+    } else {
+      try { child.kill('SIGTERM'); } catch {}
     }
-    this.preSpawnSnapshot = new Set();
   }
 
   /**
@@ -1066,19 +914,12 @@ class StageExecutor {
         }
       }
 
-      // Снапшот PID'ов до spawn — для snapshot-based cleanup сирот
-      if (this.preSpawnSnapshot.size === 0) {
-        this.preSpawnSnapshot = getProcessSnapshot();
-      }
-
       const child = spawn(agent.command, args, {
         cwd: path.resolve(this.projectRoot, agent.workdir || '.'),
         stdio: ['pipe', 'pipe', 'pipe'],
         shell: useShell
       });
-
-      // Регистрируем дочерний процесс для cleanup
-      this.activeChildren.add(child);
+      this.currentChild = child;
 
       // Передаём промпт через stdin или закрываем если не нужно
       if (useStdin) {
@@ -1092,10 +933,15 @@ class StageExecutor {
       let stderr = '';
       let timedOut = false;
 
-      // Таймаут — убиваем child + snapshot-based cleanup через killAllChildren
+      // Таймаут
       const timeoutId = setTimeout(() => {
         timedOut = true;
-        this.killAllChildren();  // убьёт child + сирот по snapshot
+        // На Windows SIGTERM игнорируется — используем taskkill /T /F для убийства дерева
+        if (process.platform === 'win32' && child.pid) {
+          try { execSync(`taskkill /pid ${child.pid} /T /F`, { stdio: 'pipe' }); } catch {}
+        } else {
+          child.kill('SIGTERM');
+        }
         if (this.logger) {
           this.logger.timeout(stageId, timeout);
         }
@@ -1144,8 +990,8 @@ class StageExecutor {
       });
 
       child.on('close', (code) => {
+        this.currentChild = null;
         clearTimeout(timeoutId);
-        this.activeChildren.delete(child);
         // Обрабатываем остаток буфера стриминга
         if (stdoutBuffer.trim()) {
           try {
@@ -1217,7 +1063,6 @@ class StageExecutor {
 
       child.on('error', (err) => {
         clearTimeout(timeoutId);
-        this.activeChildren.delete(child);
         if (!timedOut) {
           if (this.logger) {
             this.logger.error(`CLI error: ${err.message}`, stageId);
@@ -1333,13 +1178,7 @@ class PipelineRunner {
     const trustedAgents = this.pipeline.trusted_agents || [];
     this.fileGuard = new FileGuard(protectedPatterns, projectRoot, trustedAgents);
     this.projectRoot = projectRoot;
-
-    // Текущий executor для доступа к activeChildren при shutdown
     this.currentExecutor = null;
-    // Путь к PID-файлу для cleanup сирот между запусками
-    this.pidFilePath = getPidFilePath(projectRoot);
-    // Снапшот PID'ов перед стартом пайплайна — для snapshot-based cleanup
-    this.preRunSnapshot = new Set();
 
     // Настройка graceful shutdown
     this.setupGracefulShutdown();
@@ -1406,13 +1245,6 @@ class PipelineRunner {
     const maxSteps = this.pipeline.execution?.max_steps || 100;
     const delayBetweenStages = this.pipeline.execution?.delay_between_stages || 5;
 
-    // Cleanup сирот от предыдущего запуска (если runner был убит жёстко)
-    cleanupOrphansFromFile(this.pidFilePath, this.logger);
-
-    // Снапшот процессов до старта + запись PID-файла
-    this.preRunSnapshot = getProcessSnapshot();
-    writePidFile(this.pidFilePath, this.preRunSnapshot);
-
     this.logger.info('=== Pipeline Runner Started ===', 'PipelineRunner');
     this.logger.info(`Entry stage: ${this.pipeline.entry}`, 'PipelineRunner');
     this.logger.info(`Max steps: ${maxSteps}`, 'PipelineRunner');
@@ -1499,9 +1331,6 @@ class PipelineRunner {
     // Записываем итоговый summary
     this.logger.writeSummary();
 
-    // Cleanup: убиваем процессы-сироты, порождённые агентами во время пайплайна
-    this._cleanupOrphans();
-
     return {
       steps: this.stepCount,
       tasksExecuted: this.tasksExecuted,
@@ -1628,61 +1457,20 @@ class PipelineRunner {
   /**
    * Настройка graceful shutdown
    */
-  /**
-   * Snapshot-based cleanup: убивает процессы, появившиеся после старта пайплайна.
-   * Сравнивает текущие PID'ы node/python/cmd с снапшотом до run() —
-   * разница = процессы порождённые агентами (включая MCP-серверы-сироты).
-   */
-  _cleanupOrphans() {
-    if (this.preRunSnapshot.size === 0) return;
-    const currentPids = getProcessSnapshot();
-    const orphans = [];
-    for (const pid of currentPids) {
-      if (!this.preRunSnapshot.has(pid) && pid !== process.pid) {
-        orphans.push(pid);
-      }
-    }
-    if (orphans.length > 0) {
-      killPids(orphans, this.logger);
-    }
-    // PID-файл больше не нужен — cleanup выполнен
-    removePidFile(this.pidFilePath);
-  }
-
   setupGracefulShutdown() {
     const shutdown = (signal) => {
       if (this.logger) {
-        this.logger.info(`Received ${signal}. Killing child processes and shutting down...`, 'PipelineRunner');
+        this.logger.info(`Received ${signal}. Shutting down gracefully...`, 'PipelineRunner');
       }
       this.running = false;
-      // Убиваем активные дочерние процессы текущего stage
+      // Убиваем текущего агента
       if (this.currentExecutor) {
-        this.currentExecutor.killAllChildren();
+        this.currentExecutor.killCurrentChild();
       }
-      // Snapshot-based: убиваем всех сирот от всех stage'ей
-      this._cleanupOrphans();
     };
 
     process.on('SIGINT', () => shutdown('SIGINT'));
     process.on('SIGTERM', () => shutdown('SIGTERM'));
-
-    // Финальный синхронный cleanup при выходе — последний шанс убить сирот
-    process.on('exit', () => {
-      // Убиваем активные child objects (если exit во время работы stage)
-      if (this.currentExecutor) {
-        for (const child of this.currentExecutor.activeChildren) {
-          if (child.pid) {
-            if (process.platform === 'win32') {
-              try { execSync(`taskkill /pid ${child.pid} /T /F`, { stdio: 'pipe' }); } catch {}
-            } else {
-              try { child.kill('SIGKILL'); } catch {}
-            }
-          }
-        }
-      }
-      // Snapshot-based cleanup
-      this._cleanupOrphans();
-    });
   }
 }