workflow-ai 1.0.35 → 1.0.37

package/package.json

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

package/src/runner.mjs

@@ -314,7 +314,7 @@ class PromptBuilder {
     // Добавляем блок Instructions если поле instructions задано и непустое
     if (stage.instructions && typeof stage.instructions === 'string' && stage.instructions.trim() !== '') {
       parts.push('\n\nInstructions:');
-      parts.push(stage.instructions.trim());
+      parts.push(this.interpolate(stage.instructions.trim()));
     }
 
     return parts.join('\n');
@@ -840,11 +840,14 @@ class StageExecutor {
       // Если args содержит -p с ролью — объединяем роль и промпт в один аргумент
       // Иначе Claude CLI интерпретирует роль как промпт, а реальный промпт игнорирует
       const lastPIdx = args.lastIndexOf('-p');
+      let stdinPrompt = null;
       if (lastPIdx !== -1 && lastPIdx < args.length - 1) {
         const role = args[lastPIdx + 1];
         args[lastPIdx + 1] = `${prompt}\n\nТвоя роль: ${role}`;
       } else {
-        args.push(prompt);
+        // Передаём промпт через stdin вместо аргумента,
+        // т.к. shell: true на Windows обрезает многострочные аргументы
+        stdinPrompt = prompt;
       }
 
       // Логгируем команду перед запуском (вместо промпта — имя skill)
@@ -867,7 +870,10 @@ class StageExecutor {
         shell: process.platform === 'win32' && agent.command !== 'node'
       });
 
-      // Закрываем stdin чтобы агент не ждал дополнительного ввода
+      // Передаём промпт через stdin или закрываем если промпт в аргументах
+      if (stdinPrompt) {
+        child.stdin.write(stdinPrompt);
+      }
       child.stdin.end();
 
       let stdout = '';

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

@@ -131,13 +131,6 @@ async function main() {
 
     console.log(`[INFO] Found plan file: ${planFile}`);
 
-    const planStatus = getPlanStatus(planFile);
-    if (planStatus !== 'approved') {
-      console.log(`[INFO] Plan ${planId} has status "${planStatus}" (not "approved") — skipping decomposition`);
-      printResult({ status: 'decomposed' });
-      return;
-    }
-
     if (hasTicketsForPlan(planId)) {
       console.log(`[INFO] Plan ${planId} already has tickets — decomposed`);
       printResult({ status: 'decomposed' });

package/src/skills/check-relevance/SKILL.md

@@ -30,6 +30,15 @@ reason: already_skipped
 ---RESULT---
 ```
 
+Если последняя запись имеет статус `❌ failed` — тикет был выполнен, но ревью провалилось. Тикету нужна **доработка через execute-task**, а НЕ повторное ревью. Немедленно верни:
+
+```
+---RESULT---
+status: relevant
+reason: review_failed_needs_rework
+---RESULT---
+```
+
 Если последней записи нет или статус другой — переходи к шагу 1.
 
 ### 1. Прочитать тикет
@@ -69,7 +78,9 @@ reason: already_skipped
 - Все критерии уже отмечены как выполненные `[x]`
 - В секции "Результат выполнения" уже есть заполненные данные
 
-**Если DoD полностью выполнен:** тикет `irrelevant` (задача уже сделана)
+**ВАЖНО:** Если DoD выполнен, но последнее ревью `❌ failed` — тикет **НЕ неактуален**, ему нужна доработка. Верни `relevant` (reason: review_failed_needs_rework). Это предотвращает бесконечный цикл повторного ревью без доработки.
+
+**Если DoD полностью выполнен И ревью НЕ failed:** тикет `irrelevant` (задача уже сделана)
 
 ### 4. Проверить зависимости
 
@@ -122,8 +133,9 @@ reason: already_skipped
 
 | Проверка | Результат | Статус |
 |----------|-----------|--------|
+| Последнее ревью failed | Needs rework | `relevant` |
 | План неактивен | Plan inactive | `irrelevant` |
-| DoD выполнен | DoD completed | `irrelevant` |
+| DoD выполнен (ревью НЕ failed) | DoD completed | `irrelevant` |
 | Зависимости неактуальны | Dependencies inactive | `irrelevant` |
 | Все проверки пройдены | All checks passed | `relevant` |
 

package/src/skills/decompose-plan/SKILL.md

@@ -13,19 +13,18 @@ description: Декомпозировать план на исполняемые
 - Нужно разбить крупную задачу на подзадачи
 - Требуется детальная декомпозиция
 
-## ⛔ ПРЕДУСЛОВИЕ: проверка статуса плана
+## ⛔ ЗАПРЕЩЕНО сканировать все планы если указан конкретный план
 
-**ПЕРЕД любой работой** прочитай frontmatter плана и проверь поле `status`.
+Ты получаешь промпт с секциями Context и Instructions.
 
-| Статус плана | Действие |
-|-------------|----------|
-| `approved` | Декомпозировать. Продолжай к шагу 1. |
-| `draft` | **СТОП.** Не декомпозировать. Выведи `status: decomposed` и завершись. Draft-план ещё не одобрен человеком. |
-| `active` | **СТОП.** Не декомпозировать. План уже в работе. Выведи `status: decomposed`. |
-| `completed` | **СТОП.** Не декомпозировать. План завершён. Выведи `status: decomposed`. |
-| Другой/пустой | **СТОП.** Не декомпозировать. Выведи `status: decomposed`. |
+**Если в Context или Instructions есть `plan_id` или `plan_file` или путь к конкретному плану (например `PLAN-005.md`):**
+- Декомпозируй ТОЛЬКО этот план
+- НЕ сканируй `plans/current/` — это ЗАПРЕЩЕНО
+- НЕ проверяй статус плана (`draft`, `active` — неважно) — это ЗАПРЕЩЕНО
+- НЕ читай и не упоминай другие планы — это ЗАПРЕЩЕНО
+- Открой файл плана и сразу переходи к шагу 1
 
-**Почему:** Без этой проверки pipeline автоматически декомпозирует любой план, обнаруженный в `plans/current/`, включая черновики, созданные другими тикетами. Это приводит к неконтролируемому разрастанию backlog (scope creep). Уже дважды происходило.
+**Только если нигде в промпте нет указания на конкретный план** — тогда сканируй `plans/current/` и декомпозируй только планы со статусом `approved`.
 
 ## Шаги выполнения