workflow-ai 1.0.39 → 1.0.41

package/agent-templates/CLAUDE.md.tpl

@@ -28,6 +28,8 @@
 3. **Выполнение**: Бери задачи из `ready/`, выполняй, перемещай в `done/`
 4. **Отчётность**: Создавай отчёты в `.workflow/reports/`
 
+> **ВАЖНО:** В этой сессии только составляй планы. НЕ выполняй и НЕ декомпозируй их — выполнение и декомпозиция будут происходить позже через workflow автоматически.
+
 ## Шаблоны
 
 - `.workflow/templates/ticket-template.md` — шаблон тикета
@@ -36,7 +38,14 @@
 
 ## Конфигурация
 
-Счётчики ID и настройки в `.workflow/config/config.yaml`
+Настройки в `.workflow/config/config.yaml`
+
+## Правила работы со скилами
+
+> **⛔ ВАЖНО:** Любые изменения в скилах (создание, правка, аудит, улучшение) выполняются ТОЛЬКО через скил коуча (`.workflow/src/skills/coach/SKILL.md`). Не правь скилы напрямую — загрузи коуча и действуй по его воркфлоу.
 
 ## Правила написания кода
 При написании кода использовать методологии TDD, SOLID, DRY
+
+## Общие инструкции
+Отвечай всегда на Русском языке

package/agent-templates/QWEN.md.tpl

@@ -13,6 +13,14 @@
 
 {{SKILLS_TABLE}}
 
+### Скрипты (перемещение и выбор тикетов)
+
+| Действие | Скрипт |
+|----------|--------|
+| Перемещение тикета | `node .workflow/src/scripts/move-ticket.js <id> <target>` |
+| Выбор следующей задачи | `node .workflow/src/scripts/pick-next-task.js` |
+| Перемещение готовых в ready | `node .workflow/src/scripts/move-to-ready.js` |
+
 ## Workflow
 
 1. **Планирование**: Создай план в `.workflow/plans/current/`
@@ -20,6 +28,8 @@
 3. **Выполнение**: Бери задачи из `ready/`, выполняй, перемещай в `done/`
 4. **Отчётность**: Создавай отчёты в `.workflow/reports/`
 
+> **ВАЖНО:** В этой сессии только составляй планы. НЕ выполняй и НЕ декомпозируй их — выполнение и декомпозиция будут происходить позже через workflow автоматически.
+
 ## Шаблоны
 
 - `.workflow/templates/ticket-template.md` — шаблон тикета
@@ -28,7 +38,14 @@
 
 ## Конфигурация
 
-Счётчики ID и настройки в `.workflow/config/config.yaml`
+Настройки в `.workflow/config/config.yaml`
+
+## Правила работы со скилами
+
+> **⛔ ВАЖНО:** Любые изменения в скилах (создание, правка, аудит, улучшение) выполняются ТОЛЬКО через скил коуча (`.workflow/src/skills/coach/SKILL.md`). Не правь скилы напрямую — загрузи коуча и действуй по его воркфлоу.
 
 ## Правила написания кода
 При написании кода использовать методологии TDD, SOLID, DRY
+
+## Общие инструкции
+Отвечай всегда на Русском языке

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "workflow-ai",
-  "version": "1.0.39",
+  "version": "1.0.41",
   "description": "AI Agent Workflow Coordinator — kanban-based pipeline for AI coding agents",
   "type": "module",
   "bin": {
@@ -12,6 +12,8 @@
     "src/init.mjs",
     "src/runner.mjs",
     "src/wf-loader.mjs",
+    "src/global-dir.mjs",
+    "src/junction-manager.mjs",
     "src/lib/",
     "src/scripts/",
     "src/skills/",

package/src/cli.mjs

@@ -3,8 +3,10 @@
 import { initProject } from './init.mjs';
 import { runPipeline } from './runner.mjs';
 import { readFileSync } from 'node:fs';
-import { join, dirname } from 'node:path';
+import { join, dirname, resolve } from 'node:path';
 import { fileURLToPath } from 'node:url';
+import { getGlobalDir, refreshGlobalDir, ensureGlobalDir } from './global-dir.mjs';
+import { createSkillJunctions, createScriptHardlinks, ejectSkill, listSkillsWithStatus } from './junction-manager.mjs';
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 const pkgPath = join(__dirname, '..', 'package.json');
@@ -14,6 +16,9 @@ const HELP_TEXT = `workflow-ai v1.0.0
 Usage:
   workflow init [path] [--force]   Initialize .workflow/ in target directory
   workflow run [options]           Run the AI pipeline
+  workflow update [path]           Update global dir and recreate junctions/hardlinks
+  workflow eject <skill> [path]    Eject a skill (copy from global to project)
+  workflow list [path]             List skills with status (shared/ejected/project-only)
   workflow help                    Show this help
   workflow version                 Show version
 
@@ -76,6 +81,70 @@ async function runInit(args) {
   }
 }
 
+function getPackageRoot() {
+  return resolve(__dirname, '..');
+}
+
+function getWorkflowRoot(projectRoot) {
+  return join(projectRoot, '.workflow');
+}
+
+function runUpdate(args) {
+  const projectRoot = resolve(args._[0] || process.cwd());
+  const workflowRoot = getWorkflowRoot(projectRoot);
+  const packageRoot = getPackageRoot();
+  const globalDir = getGlobalDir();
+
+  refreshGlobalDir(packageRoot);
+  console.log('✅ Global dir updated (~/.workflow/)');
+
+  const skillsDir = join(workflowRoot, 'src', 'skills');
+  createSkillJunctions(globalDir, skillsDir);
+  console.log('✅ Skill junctions recreated');
+
+  const scriptsDir = join(workflowRoot, 'src', 'scripts');
+  createScriptHardlinks(globalDir, scriptsDir);
+  console.log('✅ Script hardlinks recreated');
+}
+
+function runEject(args) {
+  const skillName = args._[0];
+  if (!skillName) {
+    console.error('Error: skill name is required. Usage: workflow eject <skill>');
+    process.exit(1);
+  }
+  const projectRoot = resolve(args._[1] || process.cwd());
+  const workflowRoot = getWorkflowRoot(projectRoot);
+  const globalDir = getGlobalDir();
+  const skillsDir = join(workflowRoot, 'src', 'skills');
+
+  ejectSkill(skillName, globalDir, skillsDir);
+  console.log(`✅ Skill "${skillName}" ejected (copied to project)`);
+}
+
+function runList(args) {
+  const projectRoot = resolve(args._[0] || process.cwd());
+  const workflowRoot = getWorkflowRoot(projectRoot);
+  const globalDir = getGlobalDir();
+  const skillsDir = join(workflowRoot, 'src', 'skills');
+
+  const skills = listSkillsWithStatus(globalDir, skillsDir);
+
+  if (skills.length === 0) {
+    console.log('No skills found.');
+    return;
+  }
+
+  const maxName = Math.max(...skills.map(s => s.name.length), 4);
+  const maxStatus = Math.max(...skills.map(s => s.status.length), 6);
+
+  console.log(`${'Skill'.padEnd(maxName)}  ${'Status'.padEnd(maxStatus)}`);
+  console.log(`${'─'.repeat(maxName)}  ${'─'.repeat(maxStatus)}`);
+  for (const skill of skills) {
+    console.log(`${skill.name.padEnd(maxName)}  ${skill.status.padEnd(maxStatus)}`);
+  }
+}
+
 async function runRun(args) {
   // Expose wf's node_modules to child ESM scripts via a custom loader
   const loaderPath = join(__dirname, 'wf-loader.mjs');
@@ -109,6 +178,15 @@ export function run(argv) {
     case 'run':
       runRun(args);
       break;
+    case 'update':
+      runUpdate(args);
+      break;
+    case 'eject':
+      runEject(args);
+      break;
+    case 'list':
+      runList(args);
+      break;
     case 'help':
       showHelp();
       break;

package/src/global-dir.mjs

@@ -0,0 +1,90 @@
+import { homedir } from 'node:os';
+import { join } from 'node:path';
+import { existsSync, mkdirSync, readFileSync, writeFileSync, cpSync, rmSync } from 'node:fs';
+
+export function getGlobalDir() {
+  if (process.env.WORKFLOW_HOME) {
+    return process.env.WORKFLOW_HOME;
+  }
+  return join(homedir(), '.workflow');
+}
+
+function getPackageVersion(packageRoot) {
+  const packageJsonPath = join(packageRoot, 'package.json');
+  if (!existsSync(packageJsonPath)) {
+    throw new Error(`package.json not found in ${packageRoot}`);
+  }
+  const pkg = JSON.parse(readFileSync(packageJsonPath, 'utf-8'));
+  return pkg.version;
+}
+
+function getGlobalVersion() {
+  const globalDir = getGlobalDir();
+  const versionFile = join(globalDir, '.version');
+  if (!existsSync(versionFile)) {
+    return null;
+  }
+  return readFileSync(versionFile, 'utf-8').trim();
+}
+
+function copyDirectory(src, dest) {
+  if (!existsSync(src)) {
+    return;
+  }
+  rmSync(dest, { recursive: true, force: true });
+  cpSync(src, dest, { recursive: true });
+}
+
+function copySkillsAndScripts(packageRoot) {
+  const globalDir = getGlobalDir();
+  const srcSkills = join(packageRoot, 'src', 'skills');
+  const srcScripts = join(packageRoot, 'src', 'scripts');
+  const destSkills = join(globalDir, 'skills');
+  const destScripts = join(globalDir, 'scripts');
+
+  if (existsSync(srcSkills)) {
+    copyDirectory(srcSkills, destSkills);
+  }
+  if (existsSync(srcScripts)) {
+    copyDirectory(srcScripts, destScripts);
+  }
+}
+
+export function isGlobalDirStale(packageRoot) {
+  const globalDir = getGlobalDir();
+  if (!existsSync(globalDir)) {
+    return true;
+  }
+  const globalVersion = getGlobalVersion();
+  if (globalVersion === null) {
+    return true;
+  }
+  const packageVersion = getPackageVersion(packageRoot);
+  return packageVersion !== globalVersion;
+}
+
+export function ensureGlobalDir(packageRoot) {
+  const globalDir = getGlobalDir();
+  if (!existsSync(globalDir)) {
+    mkdirSync(globalDir, { recursive: true });
+    copySkillsAndScripts(packageRoot);
+    const version = getPackageVersion(packageRoot);
+    writeFileSync(join(globalDir, '.version'), version);
+    return;
+  }
+  if (isGlobalDirStale(packageRoot)) {
+    const version = getPackageVersion(packageRoot);
+    writeFileSync(join(globalDir, '.version'), version);
+    copySkillsAndScripts(packageRoot);
+  }
+}
+
+export function refreshGlobalDir(packageRoot) {
+  const globalDir = getGlobalDir();
+  if (!existsSync(globalDir)) {
+    mkdirSync(globalDir, { recursive: true });
+  }
+  const version = getPackageVersion(packageRoot);
+  writeFileSync(join(globalDir, '.version'), version);
+  copySkillsAndScripts(packageRoot);
+}

package/src/init.mjs

@@ -2,6 +2,8 @@ import { existsSync, mkdirSync, copyFileSync, readFileSync, writeFileSync, appen
 import { join, resolve, dirname, basename } from 'node:path';
 import { execSync } from 'node:child_process';
 import { fileURLToPath } from 'node:url';
+import { getGlobalDir, ensureGlobalDir } from './global-dir.mjs';
+import { createSkillJunctions, createScriptHardlinks } from './junction-manager.mjs';
 
 /**
  * Возвращает абсолютный путь к корню npm-пакета через import.meta.url.
@@ -325,7 +327,7 @@ export function initProject(targetPath = process.cwd(), options = {}) {
     errors: []
   };
   
-  // Step 1: Create .workflow/ structure (17 directories)
+  // Step 1: Create .workflow/ structure (15 directories)
   const directories = [
     'tickets/backlog',
     'tickets/ready',
@@ -339,43 +341,26 @@ export function initProject(targetPath = process.cwd(), options = {}) {
     'logs',
     'templates',
     'config',
-    'src/skills',
-    'src/scripts',
-    'src/lib'
+    'src/skills'
   ];
   
   for (const dir of directories) {
     ensureDir(join(workflowRoot, dir));
   }
-  result.steps.push('Created .workflow/ directory structure (17 directories)');
-  
-  // Step 2: Copy skills recursively
-  const srcSkillsSrc = join(packageRoot, 'src', 'skills');
+  result.steps.push('Created .workflow/ directory structure (15 directories)');
+
+  // Step 2: Ensure global dir and create skill junctions
+  const globalDir = getGlobalDir();
+  ensureGlobalDir(packageRoot);
   const srcSkillsDest = join(workflowRoot, 'src', 'skills');
-  copyDirRecursive(srcSkillsSrc, srcSkillsDest);
-  result.steps.push('Copied skills from getPackageRoot()/src/skills/ → .workflow/src/skills/');
-  
-  // Step 3: Copy scripts
-  const srcScriptsSrc = join(packageRoot, 'src', 'scripts');
+  createSkillJunctions(globalDir, srcSkillsDest);
+  result.steps.push('Created skill junctions from global dir → .workflow/src/skills/');
+
+  // Step 3: Create script hardlinks
   const srcScriptsDest = join(workflowRoot, 'src', 'scripts');
-  copyDirRecursive(srcScriptsSrc, srcScriptsDest);
-  result.steps.push('Copied scripts from getPackageRoot()/src/scripts/ → .workflow/src/scripts/');
-  
-  // Step 4: Copy lib (utils.mjs + find-root.mjs)
-  const libSrc = join(packageRoot, 'src', 'lib');
-  const libDest = join(workflowRoot, 'src', 'lib');
-  ensureDir(libDest);
-  
-  const libFiles = ['utils.mjs', 'find-root.mjs', 'js-yaml.mjs', 'logger.mjs'];
+  createScriptHardlinks(globalDir, srcScriptsDest);
+  result.steps.push('Created script hardlinks from global dir → .workflow/src/scripts/');
 
-  for (const file of libFiles) {
-    const fileSrc = join(libSrc, file);
-    if (existsSync(fileSrc)) {
-      copyFile(fileSrc, join(libDest, file));
-    }
-  }
-  result.steps.push('Copied lib files (utils.mjs, find-root.mjs, js-yaml.mjs) → .workflow/src/lib/');
-  
   // Step 5: Copy templates (3 templates)
   const templatesSrc = join(packageRoot, 'templates');
   const templatesDest = join(workflowRoot, 'templates');

package/src/junction-manager.mjs

@@ -0,0 +1,184 @@
+import { execSync } from 'node:child_process';
+import {
+  existsSync,
+  mkdirSync,
+  rmSync,
+  readdirSync,
+  lstatSync,
+  symlinkSync,
+  linkSync,
+  cpSync,
+  readFileSync,
+  writeFileSync,
+  unlinkSync
+} from 'node:fs';
+import { join, basename } from 'node:path';
+
+const isWindows = process.platform === 'win32';
+
+export function createJunction(target, linkPath) {
+  if (!existsSync(target)) {
+    throw new Error(`Target does not exist: ${target}`);
+  }
+  const linkDir = join(linkPath, '..');
+  if (!existsSync(linkDir)) {
+    mkdirSync(linkDir, { recursive: true });
+  }
+  rmSync(linkPath, { recursive: true, force: true });
+
+  if (isWindows) {
+    execSync(`mklink /J "${linkPath}" "${target}"`, { stdio: 'pipe' });
+  } else {
+    symlinkSync(target, linkPath, 'dir');
+  }
+}
+
+export function removeJunction(linkPath) {
+  if (existsSync(linkPath)) {
+    rmSync(linkPath, { recursive: true, force: true });
+  }
+}
+
+export function isJunction(path) {
+  if (!existsSync(path)) {
+    return false;
+  }
+  try {
+    const stats = lstatSync(path);
+    return stats.isSymbolicLink();
+  } catch {
+    if (isWindows) {
+      try {
+        const output = execSync(`fsutil reparsepoint query "${path}"`, { encoding: 'utf-8', stdio: 'pipe' });
+        return output.includes('Symbolic Link') || output.includes('Mount Point');
+      } catch {
+        return false;
+      }
+    }
+    return false;
+  }
+}
+
+export function createHardlink(target, linkPath) {
+  if (!existsSync(target)) {
+    throw new Error(`Target does not exist: ${target}`);
+  }
+  const linkDir = join(linkPath, '..');
+  if (!existsSync(linkDir)) {
+    mkdirSync(linkDir, { recursive: true });
+  }
+  rmSync(linkPath, { force: true });
+
+  if (isWindows) {
+    execSync(`mklink /H "${linkPath}" "${target}"`, { stdio: 'pipe' });
+  } else {
+    linkSync(target, linkPath);
+  }
+}
+
+export function removeHardlink(linkPath) {
+  if (existsSync(linkPath)) {
+    unlinkSync(linkPath);
+  }
+}
+
+export function createSkillJunctions(globalDir, projectSkillsDir) {
+  const globalSkillsDir = join(globalDir, 'skills');
+  if (!existsSync(globalSkillsDir)) {
+    return;
+  }
+  if (!existsSync(projectSkillsDir)) {
+    mkdirSync(projectSkillsDir, { recursive: true });
+  }
+
+  const skills = readdirSync(globalSkillsDir, { withFileTypes: true });
+  for (const skill of skills) {
+    if (skill.isDirectory()) {
+      const skillName = skill.name;
+      const targetPath = join(globalSkillsDir, skillName);
+      const linkPath = join(projectSkillsDir, skillName);
+      createJunction(targetPath, linkPath);
+    }
+  }
+}
+
+export function createScriptHardlinks(globalDir, projectScriptsDir) {
+  const globalScriptsDir = join(globalDir, 'scripts');
+  if (!existsSync(globalScriptsDir)) {
+    return;
+  }
+  if (!existsSync(projectScriptsDir)) {
+    mkdirSync(projectScriptsDir, { recursive: true });
+  }
+
+  const files = readdirSync(globalScriptsDir);
+  for (const file of files) {
+    const targetPath = join(globalScriptsDir, file);
+    const stats = lstatSync(targetPath);
+    if (stats.isFile()) {
+      const linkPath = join(projectScriptsDir, file);
+      createHardlink(targetPath, linkPath);
+    }
+  }
+}
+
+export function ejectSkill(skillName, globalDir, projectSkillsDir) {
+  const globalSkillPath = join(globalDir, 'skills', skillName);
+  const projectSkillPath = join(projectSkillsDir, skillName);
+
+  if (!existsSync(globalSkillPath)) {
+    throw new Error(`Skill does not exist in global dir: ${skillName}`);
+  }
+
+  removeJunction(projectSkillPath);
+  cpSync(globalSkillPath, projectSkillPath, { recursive: true });
+}
+
+export function listSkillsWithStatus(globalDir, projectSkillsDir) {
+  const result = [];
+  const globalSkillsDir = join(globalDir, 'skills');
+  const projectSkillsDirFull = projectSkillsDir;
+
+  if (!existsSync(globalSkillsDir)) {
+    if (existsSync(projectSkillsDirFull)) {
+      const projectSkills = readdirSync(projectSkillsDirFull, { withFileTypes: true });
+      for (const skill of projectSkills) {
+        if (skill.isDirectory()) {
+          result.push({ name: skill.name, status: 'project-only' });
+        }
+      }
+    }
+    return result;
+  }
+
+  const globalSkills = readdirSync(globalSkillsDir, { withFileTypes: true });
+  const projectSkills = existsSync(projectSkillsDirFull)
+    ? readdirSync(projectSkillsDirFull, { withFileTypes: true })
+    : [];
+
+  const projectSkillNames = new Set(projectSkills.map(s => s.name));
+
+  for (const skill of globalSkills) {
+    if (skill.isDirectory()) {
+      const skillName = skill.name;
+      const projectSkillPath = join(projectSkillsDirFull, skillName);
+      let status = 'shared';
+
+      if (projectSkillNames.has(skillName)) {
+        if (!isJunction(projectSkillPath)) {
+          status = 'ejected';
+        }
+      }
+
+      result.push({ name: skillName, status });
+    }
+  }
+
+  for (const skill of projectSkills) {
+    if (skill.isDirectory() && !globalSkills.some(s => s.name === skill.name)) {
+      result.push({ name: skill.name, status: 'project-only' });
+    }
+  }
+
+  return result;
+}

package/src/runner.mjs

@@ -543,13 +543,30 @@ class ResultParser {
 // FileGuard — защита файлов от несанкционированного изменения агентами
 // ============================================================================
 class FileGuard {
-  constructor(patterns, projectRoot = process.cwd()) {
+  constructor(patterns, projectRoot = process.cwd(), trustedAgents = []) {
     // Паттерны указаны относительно корня проекта
     this.patterns = (patterns || []).map(p => p.replace(/\\/g, '/'));
     this.enabled = this.patterns.length > 0;
     this.snapshots = new Map();
     // projectRoot — корневая директория проекта, относительно которой указаны паттерны
     this.projectRoot = projectRoot;
+    // Доверенные агенты — для них FileGuard не откатывает изменения
+    this.trustedAgents = trustedAgents;
+  }
+
+  /**
+   * Проверяет, является ли агент доверенным (пропускает FileGuard)
+   * Поддерживает glob-паттерны: "script-*" соответствует "script-move", "script-pick" и т.д.
+   * @param {string} agentId - ID агента
+   * @returns {boolean}
+   */
+  isTrusted(agentId) {
+    return this.trustedAgents.some(pattern => {
+      if (pattern.endsWith('*')) {
+        return agentId.startsWith(pattern.slice(0, -1));
+      }
+      return agentId === pattern;
+    });
   }
 
   /**
@@ -805,8 +822,9 @@ class StageExecutor {
       console.log(`  Skill: ${stage.skill}`);
     }
 
-    // Снимаем snapshot защищённых файлов перед выполнением
-    if (this.fileGuard) {
+    // Снимаем snapshot защищённых файлов перед выполнением (кроме trusted agents)
+    const skipGuard = this.fileGuard && this.fileGuard.isTrusted(agentId);
+    if (this.fileGuard && !skipGuard) {
       this.fileGuard.takeSnapshot();
     }
 
@@ -818,8 +836,8 @@ class StageExecutor {
       this.logger.stageComplete(stageId, result.status, result.exitCode);
     }
 
-    // Проверяем и откатываем несанкционированные изменения
-    if (this.fileGuard) {
+    // Проверяем и откатываем несанкционированные изменения (кроме trusted agents)
+    if (this.fileGuard && !skipGuard) {
       const violations = this.fileGuard.checkAndRollback();
       if (violations.length > 0) {
         result.violations = violations;
@@ -1100,7 +1118,8 @@ class PipelineRunner {
 
     // Инициализация FileGuard для защиты файлов от изменений агентами
     const protectedPatterns = this.pipeline.protected_files || [];
-    this.fileGuard = new FileGuard(protectedPatterns, projectRoot);
+    const trustedAgents = this.pipeline.trusted_agents || [];
+    this.fileGuard = new FileGuard(protectedPatterns, projectRoot, trustedAgents);
     this.projectRoot = projectRoot;
 
     // Настройка graceful shutdown
@@ -1573,4 +1592,4 @@ async function runPipeline(argv = process.argv.slice(2)) {
 }
 
 // Export for use as ES module
-export { runPipeline, parseArgs, PipelineRunner };
+export { runPipeline, parseArgs, PipelineRunner, FileGuard };

package/src/scripts/check-conditions.js

@@ -144,6 +144,13 @@ function checkBacklog(planId) {
 
   for (const ticket of tickets) {
     const { frontmatter, id } = ticket;
+
+    // Пропускаем тикеты, требующие ручного выполнения
+    if (frontmatter.type === 'human') {
+      console.log(`[INFO] ${id}: type is 'human', skipping (requires manual execution)`);
+      continue;
+    }
+
     const conditions = frontmatter.conditions || [];
     const dependencies = frontmatter.dependencies || [];
 

package/src/scripts/move-to-ready.js

@@ -51,6 +51,12 @@ function moveToReady(ticketId) {
   const content = fs.readFileSync(sourcePath, 'utf8');
   const { frontmatter, body } = parseFrontmatter(content);
 
+  // Пропускаем тикеты, требующие ручного выполнения
+  if (frontmatter.type === 'human') {
+    console.log(`[INFO] ${ticketId}: type is 'human', skipping (requires manual execution)`);
+    return false;
+  }
+
   frontmatter.updated_at = new Date().toISOString();
 
   const newContent = serializeFrontmatter(frontmatter) + body;

package/src/scripts/move-to-review.js

@@ -39,7 +39,11 @@ function moveToReview(ticketId) {
   const targetPath = path.join(REVIEW_DIR, `${ticketId}.md`);
 
   if (!fs.existsSync(sourcePath)) {
-    // Ticket may have been moved directly to done/ by the agent — treat as already done
+    // Ticket may have been moved by the agent — check other locations
+    const reviewPath = path.join(REVIEW_DIR, `${ticketId}.md`);
+    if (fs.existsSync(reviewPath)) {
+      return { status: 'skipped', ticket_id: ticketId, reason: `${ticketId} already in review/` };
+    }
     const donePath = path.join(TICKETS_DIR, 'done', `${ticketId}.md`);
     if (fs.existsSync(donePath)) {
       return { status: 'skipped', ticket_id: ticketId, reason: `${ticketId} already in done/` };