workflow-ai 1.0.34 → 1.0.36

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "workflow-ai",
-  "version": "1.0.34",
+  "version": "1.0.36",
   "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');
@@ -841,8 +841,8 @@ class StageExecutor {
       // Иначе Claude CLI интерпретирует роль как промпт, а реальный промпт игнорирует
       const lastPIdx = args.lastIndexOf('-p');
       if (lastPIdx !== -1 && lastPIdx < args.length - 1) {
-        const rolePrompt = args[lastPIdx + 1];
-        args[lastPIdx + 1] = `${rolePrompt}\n\n${prompt}`;
+        const role = args[lastPIdx + 1];
+        args[lastPIdx + 1] = `${prompt}\n\nТвоя роль: ${role}`;
       } else {
         args.push(prompt);
       }

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,14 @@ description: Декомпозировать план на исполняемые
 - Нужно разбить крупную задачу на подзадачи
 - Требуется детальная декомпозиция
 
-## ⛔ ПРЕДУСЛОВИЕ: проверка статуса плана
+## ⛔ ПЕРВЫЙ ШАГ: определи какой план декомпозировать
 
-**ПЕРЕД любой работой** прочитай frontmatter плана и проверь поле `status`.
+Прочитай свой промпт. Если в нём указан `plan_id` или `plan_file` (например `plan_id: PLAN-005` или `plan_file: plans/current/PLAN-005.md`) — ты ОБЯЗАН декомпозировать именно этот план.
 
-| Статус плана | Действие |
-|-------------|----------|
-| `approved` | Декомпозировать. Продолжай к шагу 1. |
-| `draft` | **СТОП.** Не декомпозировать. Выведи `status: decomposed` и завершись. Draft-план ещё не одобрен человеком. |
-| `active` | **СТОП.** Не декомпозировать. План уже в работе. Выведи `status: decomposed`. |
-| `completed` | **СТОП.** Не декомпозировать. План завершён. Выведи `status: decomposed`. |
-| Другой/пустой | **СТОП.** Не декомпозировать. Выведи `status: decomposed`. |
-
-**Почему:** Без этой проверки pipeline автоматически декомпозирует любой план, обнаруженный в `plans/current/`, включая черновики, созданные другими тикетами. Это приводит к неконтролируемому разрастанию backlog (scope creep). Уже дважды происходило.
+**Правила:**
+1. Если указан `plan_file` — открой `.workflow/{plan_file}` и декомпозируй его. Не читай другие планы. Не проверяй статус. Сразу переходи к шагу 1.
+2. Если указан `plan_id` но нет `plan_file` — найди план `plans/current/{plan_id}.md` и декомпозируй его. Не проверяй статус.
+3. Если ни `plan_id`, ни `plan_file` не указаны — сканируй все планы в `plans/current/` и декомпозируй только те, у которых `status: approved`.
 
 ## Шаги выполнения