workflow-ai 1.0.7 → 1.0.9

package/configs/pipeline.yaml

@@ -97,6 +97,9 @@ pipeline:
       workdir: "."
       description: "Скрипт для перемещения завершённых тикетов из in-progress/ в review/"
 
+      
+  default_agent: qwen-code
+
   # ===========================================================================
   # Этапы (stages)
   # ===========================================================================
@@ -233,6 +236,12 @@ pipeline:
       agent: qwen-code
       fallback_agent: kilo-deepseek
       skill: execute-task
+      counter: task_attempts
+      agent_by_attempt:
+        1: qwen-code
+        2: claude-sonnet
+        3: claude-opus
+        4: kilo-deepseek
       goto:
         default:
           stage: move-to-review
@@ -275,6 +284,7 @@ pipeline:
       agent: claude-sonnet
       fallback_agent: qwen-code
       skill: review-result
+      counter: task_attempts
       goto:
         passed:
           stage: move-ticket
@@ -300,7 +310,7 @@ pipeline:
       description: "Обновить счётчик попыток выполнения тикета"
       type: update-counter
       counter: task_attempts
-      max: 3
+      max: 6
       goto:
         default:
           stage: move-ticket
@@ -366,7 +376,7 @@ pipeline:
       description: "Обновить счётчик итераций анализа плана"
       type: update-counter
       counter: plan_iterations
-      max: 3
+      max: 2
       goto:
         default:
           stage: decompose-gaps

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "workflow-ai",
-  "version": "1.0.7",
+  "version": "1.0.9",
   "description": "AI Agent Workflow Coordinator — kanban-based pipeline for AI coding agents",
   "type": "module",
   "bin": {
@@ -23,7 +23,8 @@
     "node": ">=18.0.0"
   },
   "scripts": {
-    "test": "node --test src/tests/*.test.mjs"
+    "test": "node --test src/tests/*.test.mjs",
+    "release": "npm version patch && npm publish"
   },
   "dependencies": {
     "js-yaml": "^4.1.0"

package/src/runner.mjs

@@ -378,6 +378,41 @@ class PromptBuilder {
 // ResultParser — парсит вывод агентов и извлекает структурированные данные
 // ============================================================================
 class ResultParser {
+  // Карта нормализации статусов: синонимы → каноническое значение
+  static STATUS_ALIASES = {
+    pass:        'passed',
+    approved:    'passed',
+    success:     'passed',
+    succeeded:   'passed',
+    ok:          'passed',
+    accepted:    'passed',
+    lgtm:        'passed',
+    fixed:       'passed',
+    resolved:    'passed',
+    fail:        'failed',
+    rejected:    'failed',
+    denied:      'failed',
+    not_passed:  'failed',
+    err:         'error',
+    crash:       'error',
+    timeout:     'error',
+  };
+
+  /**
+   * Нормализует статус: приводит синонимы к каноническому значению
+   * @param {string} status
+   * @returns {string}
+   */
+  normalizeStatus(status) {
+    const lower = status.toLowerCase();
+    const canonical = ResultParser.STATUS_ALIASES[lower];
+    if (canonical) {
+      console.log(`[ResultParser] Normalized status: "${status}" → "${canonical}"`);
+      return canonical;
+    }
+    return status;
+  }
+
   /**
    * Парсит вывод агента и извлекает результат между маркерами
    * @param {string} output - stdout агента
@@ -396,10 +431,11 @@ class ResultParser {
       const resultBlock = output.substring(startIdx + marker.length, endIdx).trim();
       const data = this.parseResultBlock(resultBlock);
 
-      console.log(`[ResultParser] Parsed structured result for ${stageId}: status=${data.status}`);
+      const normalizedStatus = this.normalizeStatus(data.status || 'default');
+      console.log(`[ResultParser] Parsed structured result for ${stageId}: status=${normalizedStatus}`);
 
       return {
-        status: data.status || 'default',
+        status: normalizedStatus,
         data: data.data || {},
         raw: output,
         parsed: true
@@ -484,10 +520,11 @@ class ResultParser {
       }
     }
 
-    console.log(`[ResultParser] Fallback parsing for ${stageId}: status=${status}`);
+    const normalizedStatus = this.normalizeStatus(status);
+    console.log(`[ResultParser] Fallback parsing for ${stageId}: status=${normalizedStatus}`);
 
     return {
-      status,
+      status: normalizedStatus,
       data: extractedData,
       raw: output,
       parsed: false
@@ -681,7 +718,17 @@ class StageExecutor {
       throw new Error(`Stage not found: ${stageId}`);
     }
 
-    const agentId = stage.agent;
+    // Выбираем агента: если есть agent_by_attempt и счётчик — ротация по попыткам
+    let agentId = stage.agent || this.pipeline.default_agent;
+    if (stage.agent_by_attempt && stage.counter) {
+      const attempt = this.counters[stage.counter] || 0;
+      if (stage.agent_by_attempt[attempt]) {
+        agentId = stage.agent_by_attempt[attempt];
+        if (this.logger) {
+          this.logger.info(`Agent rotation: attempt ${attempt} → ${agentId}`, stageId);
+        }
+      }
+    }
     const agent = this.pipeline.agents[agentId];
     if (!agent) {
       throw new Error(`Agent not found: ${agentId}`);
@@ -953,10 +1000,12 @@ class PipelineRunner {
     // Базовая директория проекта вычисляется динамически
     const projectRoot = args.project ? path.resolve(args.project) : findProjectRoot();
 
-    // Инициализация Logger
-    const logFilePath = this.pipeline.execution?.log_file
-      ? path.resolve(projectRoot, this.pipeline.execution.log_file)
-      : path.resolve(projectRoot, '.workflow/logs/pipeline.log');
+    // Инициализация Logger — каждый запуск пишется в отдельный файл
+    const logDir = this.pipeline.execution?.log_file
+      ? path.dirname(path.resolve(projectRoot, this.pipeline.execution.log_file))
+      : path.resolve(projectRoot, '.workflow/logs');
+    const timestamp = new Date().toISOString().replace(/[:.]/g, '-').replace('T', '_').substring(0, 19);
+    const logFilePath = path.resolve(logDir, `pipeline_${timestamp}.log`);
     this.logger = new Logger(logFilePath);
     this.loggerInitialized = false;
 
@@ -1157,41 +1206,6 @@ class PipelineRunner {
         this.updateContext(transition.params, result.result);
       }
 
-      // Проверяем retry-логику с agent_by_attempt
-      if (transition.agent_by_attempt && stage.counter) {
-        const attempt = this.counters[stage.counter] || 1;
-        const maxAttempts = transition.max || 3;
-
-        if (attempt < maxAttempts) {
-          // Ещё есть попытки — переходим на указанный stage с переопределением агента
-          const nextStage = transition.stage || stageId;
-          const overrideAgent = transition.agent_by_attempt[attempt];
-
-          if (overrideAgent) {
-            // Переопределяем агента для следующей попытки
-            stage.agent = overrideAgent;
-            this.logger.info(`Retry attempt ${attempt}: overriding agent to ${overrideAgent}`, stageId);
-          }
-
-          this.logger.retry(stageId, attempt, maxAttempts);
-          this.logger.gotoTransition(stageId, nextStage, status, transition.params);
-          return nextStage;
-        } else {
-          // Попытки исчерпаны — переходим на on_max
-          this.logger.info(`Max attempts (${maxAttempts}) reached for ${stageId}`, stageId);
-          this.logger.retry(stageId, attempt, maxAttempts);
-
-          if (transition.on_max) {
-            if (transition.on_max.params) {
-              this.updateContext(transition.on_max.params, result.result);
-            }
-            const nextStage = transition.on_max.stage || 'end';
-            this.logger.gotoTransition(stageId, nextStage, status, transition.on_max.params);
-            return nextStage;
-          }
-        }
-      }
-
       const nextStage = transition.stage || 'end';
       this.logger.gotoTransition(stageId, nextStage, status, transition.params);
       return nextStage;
@@ -1398,8 +1412,9 @@ function validateConfig(config) {
     const stageIds = Object.keys(pipeline.stages);
 
     for (const [stageId, stage] of Object.entries(pipeline.stages)) {
-      if (stage.agent && !agentIds.includes(stage.agent)) {
-        errors.push(`Stage "${stageId}" references non-existent agent: ${stage.agent}`);
+      const resolvedAgent = stage.agent || pipeline.default_agent;
+      if (resolvedAgent && !agentIds.includes(resolvedAgent)) {
+        errors.push(`Stage "${stageId}" references non-existent agent: ${resolvedAgent}`);
       }
 
       if (stage.goto) {

package/src/skills/execute-task/SKILL.md

@@ -17,13 +17,15 @@ description: Выполнить задачу из тикета. Использу
 
 ### 1. Прочитать тикет
 
-ID тикета передаётся в промпте как `ticket_id` в секции Context.
+**ОБЯЗАТЕЛЬНО:** Используй ТОЛЬКО `ticket_id` из секции Context промпта. Запрещено выбирать другой тикет.
 
 ```
-Путь: .workflow/tickets/ready/{TICKET-ID}.md
+Путь: .workflow/tickets/in-progress/{TICKET-ID}.md
 ```
 
-**Важно:** НЕ перемещай тикет — он остаётся в `ready/` на время выполнения. Перемещение выполняется отдельным stage пайплайна.
+Если тикет не найден в `in-progress/`, проверь `review/` (при повторном выполнении тикет может быть там).
+
+**Важно:** НЕ перемещай тикет. Перемещение выполняется отдельным stage пайплайна.
 
 Извлечь:
 - Описание задачи
@@ -109,7 +111,7 @@ status: default
 
 - Выполненная работа (код, документация и т.д.)
 - Обновлённый тикет с секцией Result
-- Тикет остаётся в `ready/` (перемещение выполняется отдельным stage)
+- Тикет остаётся на месте (перемещение выполняется отдельным stage)
 
 ## Связанные skills