claude-code-workflow 6.3.43 → 6.3.44

package/.claude/skills/ccw-coordinator/tools/chain-validate.cjs

@@ -0,0 +1,320 @@
+#!/usr/bin/env node
+
+/**
+ * Chain Validation Tool
+ * 
+ * Validates workflow command chains against defined rules.
+ * 
+ * Usage:
+ *   node chain-validate.js plan execute test-cycle-execute
+ *   node chain-validate.js --json "plan,execute,test-cycle-execute"
+ *   node chain-validate.js --file custom-chain.json
+ */
+
+const fs = require('fs');
+const path = require('path');
+
+// Optional registry loading - gracefully degrade if not found
+let registry = null;
+try {
+  const registryPath = path.join(__dirname, '..', 'specs', 'chain-registry.json');
+  if (fs.existsSync(registryPath)) {
+    registry = JSON.parse(fs.readFileSync(registryPath, 'utf8'));
+  }
+} catch (error) {
+  // Registry not available - dependency validation will be skipped
+}
+
+class ChainValidator {
+  constructor(registry) {
+    this.registry = registry;
+    this.errors = [];
+    this.warnings = [];
+  }
+
+  validate(chain) {
+    this.errors = [];
+    this.warnings = [];
+
+    this.validateSinglePlanning(chain);
+    this.validateCompatiblePairs(chain);
+    this.validateTestingPosition(chain);
+    this.validateReviewPosition(chain);
+    this.validateBugfixStandalone(chain);
+    this.validateDependencies(chain);
+    this.validateNoRedundancy(chain);
+    this.validateCommandExistence(chain);
+
+    return {
+      valid: this.errors.length === 0,
+      errors: this.errors,
+      warnings: this.warnings
+    };
+  }
+
+  validateSinglePlanning(chain) {
+    const planningCommands = chain.filter(cmd => 
+      ['plan', 'lite-plan', 'multi-cli-plan', 'tdd-plan'].includes(cmd)
+    );
+
+    if (planningCommands.length > 1) {
+      this.errors.push({
+        rule: 'Single Planning Command',
+        message: `Too many planning commands: ${planningCommands.join(', ')}`,
+        severity: 'error'
+      });
+    }
+  }
+
+  validateCompatiblePairs(chain) {
+    const compatibility = {
+      'lite-plan': ['lite-execute'],
+      'multi-cli-plan': ['lite-execute', 'execute'],
+      'plan': ['execute'],
+      'tdd-plan': ['execute']
+    };
+
+    const planningCmd = chain.find(cmd => 
+      ['plan', 'lite-plan', 'multi-cli-plan', 'tdd-plan'].includes(cmd)
+    );
+
+    const executionCmd = chain.find(cmd => 
+      ['execute', 'lite-execute'].includes(cmd)
+    );
+
+    if (planningCmd && executionCmd) {
+      const compatible = compatibility[planningCmd] || [];
+      if (!compatible.includes(executionCmd)) {
+        this.errors.push({
+          rule: 'Compatible Pairs',
+          message: `${planningCmd} incompatible with ${executionCmd}`,
+          fix: `Use ${planningCmd} with ${compatible.join(' or ')}`,
+          severity: 'error'
+        });
+      }
+    }
+  }
+
+  validateTestingPosition(chain) {
+    const executionIdx = chain.findIndex(cmd => 
+      ['execute', 'lite-execute', 'develop-with-file'].includes(cmd)
+    );
+
+    const testingIdx = chain.findIndex(cmd => 
+      ['test-cycle-execute', 'tdd-verify', 'test-gen', 'test-fix-gen'].includes(cmd)
+    );
+
+    if (testingIdx !== -1 && executionIdx !== -1 && executionIdx > testingIdx) {
+      this.errors.push({
+        rule: 'Testing After Execution',
+        message: 'Testing commands must come after execution',
+        severity: 'error'
+      });
+    }
+
+    if (testingIdx !== -1 && executionIdx === -1) {
+      const hasTestGen = chain.some(cmd => ['test-gen', 'test-fix-gen'].includes(cmd));
+      if (!hasTestGen) {
+        this.warnings.push({
+          rule: 'Testing After Execution',
+          message: 'test-cycle-execute without execution context - needs test-gen or execute first',
+          severity: 'warning'
+        });
+      }
+    }
+  }
+
+  validateReviewPosition(chain) {
+    const executionIdx = chain.findIndex(cmd => 
+      ['execute', 'lite-execute'].includes(cmd)
+    );
+
+    const reviewIdx = chain.findIndex(cmd => 
+      cmd.includes('review')
+    );
+
+    if (reviewIdx !== -1 && executionIdx !== -1 && executionIdx > reviewIdx) {
+      this.errors.push({
+        rule: 'Review After Changes',
+        message: 'Review commands must come after execution',
+        severity: 'error'
+      });
+    }
+
+    if (reviewIdx !== -1 && executionIdx === -1) {
+      const isModuleReview = chain[reviewIdx] === 'review-module-cycle';
+      if (!isModuleReview) {
+        this.warnings.push({
+          rule: 'Review After Changes',
+          message: 'Review without execution - needs git changes to review',
+          severity: 'warning'
+        });
+      }
+    }
+  }
+
+  validateBugfixStandalone(chain) {
+    if (chain.includes('lite-fix')) {
+      const others = chain.filter(cmd => cmd !== 'lite-fix');
+      if (others.length > 0) {
+        this.errors.push({
+          rule: 'BugFix Standalone',
+          message: 'lite-fix must be standalone, cannot combine with other commands',
+          fix: 'Use lite-fix alone OR use plan + execute for larger changes',
+          severity: 'error'
+        });
+      }
+    }
+  }
+
+  validateDependencies(chain) {
+    // Skip if registry not available
+    if (!this.registry || !this.registry.commands) {
+      return;
+    }
+
+    for (let i = 0; i < chain.length; i++) {
+      const cmd = chain[i];
+      const cmdMeta = this.registry.commands[cmd];
+
+      if (!cmdMeta) continue;
+
+      const deps = cmdMeta.dependencies || [];
+      const depsOptional = cmdMeta.dependencies_optional || false;
+
+      if (deps.length > 0 && !depsOptional) {
+        const hasDependency = deps.some(dep => {
+          const depIdx = chain.indexOf(dep);
+          return depIdx !== -1 && depIdx < i;
+        });
+
+        if (!hasDependency) {
+          this.errors.push({
+            rule: 'Dependency Satisfaction',
+            message: `${cmd} requires ${deps.join(' or ')} before it`,
+            severity: 'error'
+          });
+        }
+      }
+    }
+  }
+
+  validateNoRedundancy(chain) {
+    const seen = new Set();
+    const duplicates = [];
+
+    for (const cmd of chain) {
+      if (seen.has(cmd)) {
+        duplicates.push(cmd);
+      }
+      seen.add(cmd);
+    }
+
+    if (duplicates.length > 0) {
+      this.errors.push({
+        rule: 'No Redundant Commands',
+        message: `Duplicate commands: ${duplicates.join(', ')}`,
+        severity: 'error'
+      });
+    }
+  }
+
+  validateCommandExistence(chain) {
+    // Skip if registry not available
+    if (!this.registry || !this.registry.commands) {
+      return;
+    }
+
+    for (const cmd of chain) {
+      if (!this.registry.commands[cmd]) {
+        this.errors.push({
+          rule: 'Command Existence',
+          message: `Unknown command: ${cmd}`,
+          severity: 'error'
+        });
+      }
+    }
+  }
+}
+
+function main() {
+  const args = process.argv.slice(2);
+
+  if (args.length === 0) {
+    console.log('Usage:');
+    console.log('  chain-validate.js <command1> <command2> ...');
+    console.log('  chain-validate.js --json "cmd1,cmd2,cmd3"');
+    console.log('  chain-validate.js --file chain.json');
+    process.exit(1);
+  }
+
+  let chain;
+
+  if (args[0] === '--json') {
+    chain = args[1].split(',').map(s => s.trim());
+  } else if (args[0] === '--file') {
+    const filePath = args[1];
+
+    // SEC-001: 路径遍历验证 - 只允许访问工作目录下的文件
+    const resolvedPath = path.resolve(filePath);
+    const workDir = path.resolve('.');
+    if (!resolvedPath.startsWith(workDir)) {
+      console.error('Error: File path must be within current working directory');
+      process.exit(1);
+    }
+
+    // CORR-001: JSON 解析错误处理
+    let fileContent;
+    try {
+      fileContent = JSON.parse(fs.readFileSync(resolvedPath, 'utf8'));
+    } catch (error) {
+      console.error(`Error: Failed to parse JSON file ${filePath}: ${error.message}`);
+      process.exit(1);
+    }
+
+    // CORR-002: 嵌套属性 null 检查
+    chain = fileContent.chain || fileContent.steps?.map(s => s.command) || [];
+    if (chain.length === 0) {
+      console.error('Error: No valid chain found in file (expected "chain" array or "steps" with "command" fields)');
+      process.exit(1);
+    }
+  } else {
+    chain = args;
+  }
+
+  const validator = new ChainValidator(registry);
+  const result = validator.validate(chain);
+
+  console.log('\n=== Chain Validation Report ===\n');
+  console.log('Chain:', chain.join(' → '));
+  console.log('');
+
+  if (result.valid) {
+    console.log('✓ Chain is valid!\n');
+  } else {
+    console.log('✗ Chain has errors:\n');
+    result.errors.forEach(err => {
+      console.log(`  [${err.rule}] ${err.message}`);
+      if (err.fix) {
+        console.log(`    Fix: ${err.fix}`);
+      }
+    });
+    console.log('');
+  }
+
+  if (result.warnings.length > 0) {
+    console.log('⚠ Warnings:\n');
+    result.warnings.forEach(warn => {
+      console.log(`  [${warn.rule}] ${warn.message}`);
+    });
+    console.log('');
+  }
+
+  process.exit(result.valid ? 0 : 1);
+}
+
+if (require.main === module) {
+  main();
+}
+
+module.exports = { ChainValidator };

package/.claude/skills/ccw-coordinator/tools/command-registry.cjs

@@ -0,0 +1,255 @@
+#!/usr/bin/env node
+
+/**
+ * Command Registry Tool
+ *
+ * 功能:
+ * 1. 根据命令名称查找并提取 YAML 头
+ * 2. 从全局 .claude/commands/workflow 目录读取
+ * 3. 支持按需提取（不是全量扫描）
+ */
+
+const fs = require('fs');
+const path = require('path');
+const os = require('os');
+
+class CommandRegistry {
+  constructor(commandDir = null) {
+    // 优先使用传入的目录
+    if (commandDir) {
+      this.commandDir = commandDir;
+    } else {
+      // 自动查找 .claude/commands/workflow
+      this.commandDir = this.findCommandDir();
+    }
+    this.cache = {};
+  }
+
+  /**
+   * 自动查找 .claude/commands/workflow 目录
+   * 支持: 项目相对路径、用户 home 目录
+   */
+  findCommandDir() {
+    // 1. 尝试相对于当前工作目录
+    const relativePath = path.join('.claude', 'commands', 'workflow');
+    if (fs.existsSync(relativePath)) {
+      return path.resolve(relativePath);
+    }
+
+    // 2. 尝试用户 home 目录
+    const homeDir = os.homedir();
+    const homeCommandDir = path.join(homeDir, '.claude', 'commands', 'workflow');
+    if (fs.existsSync(homeCommandDir)) {
+      return homeCommandDir;
+    }
+
+    // 未找到时返回 null，后续操作会失败并提示
+    return null;
+  }
+
+  /**
+   * 解析 YAML 头 (简化版本)
+   *
+   * 限制:
+   * - 只支持简单的 key: value 对 (单行值)
+   * - 不支持多行值、嵌套对象、复杂列表
+   * - allowed-tools 字段支持逗号分隔的字符串，自动转为数组
+   *
+   * 示例:
+   * ---
+   * name: lite-plan
+   * description: "Lightweight planning workflow"
+   * allowed-tools: Read, Write, Bash
+   * ---
+   */
+  parseYamlHeader(content) {
+    // 处理 Windows 行结尾 (\r\n)
+    const match = content.match(/^---[\r\n]+([\s\S]*?)[\r\n]+---/);
+    if (!match) return null;
+
+    const yamlContent = match[1];
+    const result = {};
+
+    try {
+      const lines = yamlContent.split(/[\r\n]+/);
+      for (const line of lines) {
+        const trimmed = line.trim();
+        if (!trimmed || trimmed.startsWith('#')) continue; // 跳过空行和注释
+
+        const colonIndex = trimmed.indexOf(':');
+        if (colonIndex === -1) continue;
+
+        const key = trimmed.substring(0, colonIndex).trim();
+        const value = trimmed.substring(colonIndex + 1).trim();
+
+        if (!key) continue; // 跳过无效行
+
+        // 去除引号 (单引号或双引号)
+        let cleanValue = value.replace(/^["']|["']$/g, '');
+
+        // allowed-tools 字段特殊处理：转为数组
+        // 支持格式: "Read, Write, Bash" 或 "Read,Write,Bash"
+        if (key === 'allowed-tools') {
+          cleanValue = Array.isArray(cleanValue)
+            ? cleanValue
+            : cleanValue.split(',').map(t => t.trim()).filter(t => t);
+        }
+
+        result[key] = cleanValue;
+      }
+    } catch (error) {
+      console.error('YAML parsing error:', error.message);
+      return null;
+    }
+
+    return result;
+  }
+
+  /**
+   * 获取单个命令的元数据
+   * @param {string} commandName 命令名称 (e.g., "lite-plan" 或 "/workflow:lite-plan")
+   * @returns {object|null} 命令信息或 null
+   */
+  getCommand(commandName) {
+    if (!this.commandDir) {
+      console.error('ERROR: .claude/commands/workflow 目录未找到');
+      return null;
+    }
+
+    // 标准化命令名称
+    const normalized = commandName.startsWith('/workflow:')
+      ? commandName.substring('/workflow:'.length)
+      : commandName;
+
+    // 检查缓存
+    if (this.cache[normalized]) {
+      return this.cache[normalized];
+    }
+
+    // 读取命令文件
+    const filePath = path.join(this.commandDir, `${normalized}.md`);
+    if (!fs.existsSync(filePath)) {
+      return null;
+    }
+
+    try {
+      const content = fs.readFileSync(filePath, 'utf-8');
+      const header = this.parseYamlHeader(content);
+
+      if (header && header.name) {
+        const result = {
+          name: header.name,
+          command: `/workflow:${header.name}`,
+          description: header.description || '',
+          argumentHint: header['argument-hint'] || '',
+          allowedTools: Array.isArray(header['allowed-tools'])
+            ? header['allowed-tools']
+            : (header['allowed-tools'] ? [header['allowed-tools']] : []),
+          filePath: filePath
+        };
+
+        // 缓存结果
+        this.cache[normalized] = result;
+        return result;
+      }
+    } catch (error) {
+      console.error(`读取命令失败 ${filePath}:`, error.message);
+    }
+
+    return null;
+  }
+
+  /**
+   * 批量获取多个命令的元数据
+   * @param {array} commandNames 命令名称数组
+   * @returns {object} 命令信息映射
+   */
+  getCommands(commandNames) {
+    const result = {};
+
+    for (const name of commandNames) {
+      const cmd = this.getCommand(name);
+      if (cmd) {
+        result[cmd.command] = cmd;
+      }
+    }
+
+    return result;
+  }
+
+  /**
+   * 获取所有命令的名称和描述
+   * @returns {object} 命令名称和描述的映射
+   */
+  getAllCommandsSummary() {
+    const result = {};
+    const commandDir = this.commandDir;
+
+    if (!commandDir) {
+      return result;
+    }
+
+    try {
+      const files = fs.readdirSync(commandDir);
+
+      for (const file of files) {
+        if (!file.endsWith('.md')) continue;
+
+        const filePath = path.join(commandDir, file);
+        const stat = fs.statSync(filePath);
+
+        if (stat.isDirectory()) continue;
+
+        try {
+          const content = fs.readFileSync(filePath, 'utf-8');
+          const header = this.parseYamlHeader(content);
+
+          if (header && header.name) {
+            const commandName = `/workflow:${header.name}`;
+            result[commandName] = {
+              name: header.name,
+              description: header.description || ''
+            };
+          }
+        } catch (error) {
+          // 跳过读取失败的文件
+          continue;
+        }
+      }
+    } catch (error) {
+      // 目录读取失败
+      return result;
+    }
+
+    return result;
+  }
+
+  /**
+   * 生成注册表 JSON
+   */
+  toJSON(commands = null) {
+    const data = commands || this.cache;
+    return JSON.stringify(data, null, 2);
+  }
+}
+
+// CLI 模式
+if (require.main === module) {
+  const args = process.argv.slice(2);
+
+  if (args.length === 0 || args[0] === '--all') {
+    // 获取所有命令的名称和描述
+    const registry = new CommandRegistry();
+    const commands = registry.getAllCommandsSummary();
+    console.log(JSON.stringify(commands, null, 2));
+    process.exit(0);
+  }
+
+  const registry = new CommandRegistry();
+  const commands = registry.getCommands(args);
+
+  console.log(JSON.stringify(commands, null, 2));
+}
+
+module.exports = CommandRegistry;
+

package/.claude/skills/ccw-help/command.json

@@ -15,7 +15,7 @@
     "/workflow:review-session-cycle",
     "/memory:docs",
     "/workflow:brainstorm:artifacts",
-    "/workflow:action-plan-verify",
+    "/workflow:plan-verify",
     "/version"
   ],
 
@@ -69,7 +69,7 @@
       "difficulty": "Intermediate",
       "essential": true,
       "flow": {
-        "next_steps": ["/workflow:action-plan-verify", "/workflow:execute"],
+        "next_steps": ["/workflow:plan-verify", "/workflow:execute"],
         "alternatives": ["/workflow:tdd-plan"]
       },
       "source": "../../../commands/workflow/plan.md"
@@ -89,8 +89,8 @@
       "source": "../../../commands/workflow/execute.md"
     },
     {
-      "name": "action-plan-verify",
-      "command": "/workflow:action-plan-verify",
+      "name": "plan-verify",
+      "command": "/workflow:plan-verify",
       "description": "Cross-artifact consistency analysis",
       "arguments": "[--session session-id]",
       "category": "workflow",
@@ -100,7 +100,7 @@
         "prerequisites": ["/workflow:plan"],
         "next_steps": ["/workflow:execute"]
       },
-      "source": "../../../commands/workflow/action-plan-verify.md"
+      "source": "../../../commands/workflow/plan-verify.md"
     },
     {
       "name": "init",