@ghyper9023/pi-dev-workflow 0.6.3 → 0.8.0

package/tests/test-output-directory-structure.mjs

@@ -1,177 +0,0 @@
-/**
- * test-output-directory-structure.mjs — 验证输出目录结构调整正确
- *
- * Bug: 原先 .pi-dev-output/pi-grill/ 直接存放 answer-*.md 和 questions-*.json，
- * .pi-dev-output/pi-review/ 直接存放 *.html 和 *.md，没有分类子目录。
- *
- * 预期新结构：
- *   .pi-dev-output/pi-grill/questions/  → questions-<id>-<YYYYMMDD-HHmm>.json
- *   .pi-dev-output/pi-grill/answers/    → answer-<id>-<YYYYMMDD-HHmm>.md
- *   .pi-dev-output/pi-review/html/      → *.html
- *   .pi-dev-output/pi-review/md/        → *.md
- *
- * Run: node tests/test-output-directory-structure.mjs
- */
-
-import * as fs from "node:fs";
-import * as path from "node:path";
-import { fileURLToPath } from "node:url";
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const GRILL_ME_PATH = path.resolve(__dirname, "../extensions/grill-me-agent.ts");
-const SUB_AGENTS_PATH = path.resolve(__dirname, "../extensions/sub-agents.ts");
-const WORKFLOW_PATH = path.resolve(__dirname, "../extensions/workflow-engine.ts");
-
-// ── Helpers ──────────────────────────────────────────────────
-
-let pass = 0;
-let fail = 0;
-
-function assert(condition, msg) {
-	if (condition) {
-		pass++;
-		console.log(`  ✅ ${msg}`);
-	} else {
-		fail++;
-		console.error(`  ❌ ${msg}`);
-	}
-}
-
-function assertIncludes(source, substr, msg) {
-	assert(source.includes(substr), msg);
-}
-
-function assertNotIncludes(source, substr, msg) {
-	assert(!source.includes(substr), msg);
-}
-
-// ═══════════════════════════════════════════════════════════════
-//  1. grill-me-agent.ts — 常量与路径
-// ═══════════════════════════════════════════════════════════════
-
-console.log("📋 grill-me-agent.ts — 目录常量与文件路径\n");
-
-const grillMe = fs.readFileSync(GRILL_ME_PATH, "utf-8");
-
-// 1a. 新常量定义
-assertIncludes(grillMe, 'GRILL_ANSWERS_DIRNAME = "answers"', "定义 GRILL_ANSWERS_DIRNAME = answers");
-assertIncludes(grillMe, 'GRILL_QUESTIONS_DIRNAME = "questions"', "定义 GRILL_QUESTIONS_DIRNAME = questions");
-
-// 1b. grillOutputPath 写入 questions 子目录 + 新文件名格式
-assertIncludes(grillMe, 'path.join(GRILL_DIRNAME, GRILL_QUESTIONS_DIRNAME)', "grillOutputPath 使用 questions 子目录");
-assertIncludes(grillMe, "questions-${ts}-${formatTimestamp()}.json", "grillOutputPath 文件名含 formatTimestamp");
-
-// 1c. saveAnswerFile 写入 answers 子目录 + 新文件名格式
-assertIncludes(grillMe, 'path.join(GRILL_DIRNAME, GRILL_ANSWERS_DIRNAME)', "saveAnswerFile 使用 answers 子目录");
-assertIncludes(grillMe, "answer-${ts}-${formatTimestamp()}.md", "saveAnswerFile 文件名含 formatTimestamp");
-assertIncludes(grillMe, "DEV_OUTPUT_DIR, GRILL_DIRNAME, GRILL_ANSWERS_DIRNAME, filename", "saveAnswerFile 返回路径含 answers 子目录");
-
-// 1d. recoverFromBackup 从 answers 子目录读取
-assertIncludes(grillMe, "GRILL_DIRNAME, GRILL_ANSWERS_DIRNAME", "recoverFromBackup 从 answers 子目录读取");
-
-// 1e. formatTimestamp 辅助函数
-assertIncludes(grillMe, "function formatTimestamp", "定义 formatTimestamp 辅助函数");
-
-// 1f. 旧文件名格式不应再出现
-// Note: still allow "questions-" prefix in path.join for grillOutputPath
-const oldSavePattern = "answer-${ts}.md";
-assertNotIncludes(grillMe, oldSavePattern, "saveAnswerFile 不使用旧格式 answer-${ts}.md");
-
-const oldQuestionsPattern = "questions-${ts}.json";
-assertNotIncludes(grillMe, oldQuestionsPattern, "grillOutputPath 不使用旧格式 questions-${ts}.json");
-
-// ═══════════════════════════════════════════════════════════════
-//  2. sub-agents.ts — findNewestReviewHtml 新增优先路径
-// ═══════════════════════════════════════════════════════════════
-
-console.log("\n📋 sub-agents.ts — findNewestReviewHtml 搜索路径\n");
-
-const subAgents = fs.readFileSync(SUB_AGENTS_PATH, "utf-8");
-assertIncludes(subAgents, '".pi-dev-output", "pi-review", "html"', "findNewestReviewHtml 优先搜索 pi-review/html/");
-
-// ═══════════════════════════════════════════════════════════════
-//  3. workflow-engine.ts — reviewer 报告写入路径
-// ═══════════════════════════════════════════════════════════════
-
-console.log("\n📋 workflow-engine.ts — buildReviewTask 输出路径\n");
-
-const workflowEngine = fs.readFileSync(WORKFLOW_PATH, "utf-8");
-assertIncludes(workflowEngine, ".pi-dev-output/pi-review/md/", "buildReviewTask 告诉 reviewer 写入 pi-review/md/");
-
-// ═══════════════════════════════════════════════════════════════
-//  4. agent 定义文件一致性
-// ═══════════════════════════════════════════════════════════════
-
-console.log("\n📋 Agent 定义文件路径一致性\n");
-
-const reviewAgent = fs.readFileSync(
-	path.resolve(__dirname, "../agents/review-agent.md"), "utf-8");
-assertIncludes(reviewAgent, ".pi-dev-output/pi-review/html/",
-	"review-agent 写入 .pi-dev-output/pi-review/html/");
-assertNotIncludes(reviewAgent, ".pi-dev-output/pi-review/ 目录",
-	"review-agent 不再引用旧的 .pi-dev-output/pi-review/ (无子目录)");
-
-const workflowReviewer = fs.readFileSync(
-	path.resolve(__dirname, "../agents/workflow/reviewer-agent.md"), "utf-8");
-assertIncludes(workflowReviewer, ".pi-dev-output/pi-review/md/",
-	"workflow/reviewer-agent 写入 .pi-dev-output/pi-review/md/");
-
-// ═══════════════════════════════════════════════════════════════
-//  5. review-html SKILL 路径一致性
-// ═══════════════════════════════════════════════════════════════
-
-console.log("\n📋 skills/review-html 路径一致性\n");
-
-const reviewSkill = fs.readFileSync(
-	path.resolve(__dirname, "../skills/review-html/SKILL.md"), "utf-8");
-assertIncludes(reviewSkill, ".pi-dev-output/pi-review/html/",
-	"review-html skill 写入 .pi-dev-output/pi-review/html/");
-assertNotIncludes(reviewSkill, ".pi-dev-output/pi-review/ 目录",
-	"review-html skill 不再引用旧的 .pi-dev-output/pi-review/ (无子目录)");
-
-// ═══════════════════════════════════════════════════════════════
-//  6. 文件名格式验证：saveAnswerFile 和 grillOutputPath 生成的名称含时间戳
-// ═══════════════════════════════════════════════════════════════
-
-console.log("\n📋 文件名格式验证\n");
-
-// Simulate the formatTimestamp function
-function formatTimestamp() {
-	const now = new Date();
-	const Y = now.getFullYear().toString();
-	const M = (now.getMonth() + 1).toString().padStart(2, "0");
-	const D = now.getDate().toString().padStart(2, "0");
-	const h = now.getHours().toString().padStart(2, "0");
-	const m = now.getMinutes().toString().padStart(2, "0");
-	return `${Y}${M}${D}-${h}${m}`;
-}
-
-// Verify the format matches YYYYMMDD-HHmm
-const ts = formatTimestamp();
-const formatRegex = /^\d{8}-\d{4}$/;
-assert(formatRegex.test(ts), `formatTimestamp 格式正确 (${ts} 匹配 YYYYMMDD-HHmm)`);
-
-// Simulate filename generation
-const id = Date.now().toString(36);
-const answerFilename = `answer-${id}-${ts}.md`;
-const questionsFilename = `questions-${id}-${ts}.json`;
-
-assert(answerFilename.startsWith("answer-"), "answer 文件名以 answer- 开头");
-assert(answerFilename.endsWith(".md"), "answer 文件名以 .md 结尾");
-assert(questionsFilename.startsWith("questions-"), "questions 文件名以 questions- 开头");
-assert(questionsFilename.endsWith(".json"), "questions 文件名以 .json 结尾");
-
-// ═══════════════════════════════════════════════════════════════
-//  Summary
-// ═══════════════════════════════════════════════════════════════
-
-console.log(`\n━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━`);
-console.log(`结果: ${pass} 通过, ${fail} 失败, 共 ${pass + fail} 个测试`);
-console.log(`━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━`);
-
-if (fail > 0) {
-	console.error("\n⚠️  部分测试未通过");
-	process.exit(1);
-} else {
-	console.log("\n✅ 所有测试通过 — 输出目录结构调整正确");
-}

package/tests/test-save-answer-file-workflow.mjs

@@ -1,187 +0,0 @@
-/**
- * test-save-answer-file-workflow.mjs — 验证 saveAnswerFile 在 Workflow 路径中未被跳过
- *
- * 重构后所有 workflow 入口统一经过 promptWorkflowDecision() 函数，
- * 该函数在调用 runWorkflow 前一定会调用 saveAnswerFile。
- *
- * 本测试验证：
- *   1. promptWorkflowDecision 函数体：两个路径（默认/自定义）都在 runWorkflow 前调用了 saveAnswerFile
- *   2. 所有 caller（runWizardWithGrill / runWizard / dev-feat handler）都调用 promptWorkflowDecision
- *   3. 所有非 workflow 路径的 saveAnswerFile 未被移除
- *
- * Run: node tests/test-save-answer-file-workflow.mjs
- */
-
-import * as fs from "node:fs";
-import * as path from "node:path";
-import { fileURLToPath } from "node:url";
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const SOURCE_PATH = path.resolve(__dirname, "../extensions/dev-prompts.ts");
-
-// ── Helpers ──────────────────────────────────────────────────
-
-let pass = 0;
-let fail = 0;
-
-function assert(condition, msg) {
-	if (condition) {
-		pass++;
-		console.log(`  ✅ ${msg}`);
-	} else {
-		fail++;
-		console.error(`  ❌ ${msg}`);
-	}
-}
-
-function assertEq(actual, expected, msg) {
-	if (actual === expected) {
-		pass++;
-		console.log(`  ✅ ${msg}`);
-	} else {
-		fail++;
-		console.error(`  ❌ ${msg} — 期望 ${JSON.stringify(expected)}, 得到 ${JSON.stringify(actual)}`);
-	}
-}
-
-// ═══════════════════════════════════════════════════════════════
-//  Read source
-// ═══════════════════════════════════════════════════════════════
-
-let source;
-try {
-	source = fs.readFileSync(SOURCE_PATH, "utf-8");
-} catch (e) {
-	console.error(`Failed to read source file: ${e.message}`);
-	process.exit(1);
-}
-
-console.log(`📄 源文件: ${SOURCE_PATH}`);
-console.log(`📏 文件大小: ${source.length} 字节\n`);
-
-// ═══════════════════════════════════════════════════════════════
-//  Test 1: promptWorkflowDecision 在 runWorkflow 前调用 saveAnswerFile
-// ═══════════════════════════════════════════════════════════════
-
-console.log("📋 Test 1: promptWorkflowDecision 的两个路径都在 runWorkflow 前调用 saveAnswerFile\n");
-
-// Find the promptWorkflowDecision function body
-const pwdMatch = source.match(/async function promptWorkflowDecision\([\s\S]*?^\}/m);
-assert(pwdMatch !== null, "应找到 promptWorkflowDecision 函数");
-
-const pwdFunc = pwdMatch[0];
-assert(
-	pwdFunc.includes("saveAnswerFile(ctx.cwd, finalPrompt)"),
-	"promptWorkflowDecision 应包含 saveAnswerFile(finalPrompt) 调用",
-);
-
-// Verify the default path (choice "1"): saveAnswerFile before runWorkflow
-const defaultPath = pwdFunc.match(/if \(choice\.startsWith\("1"\)\) \{[\s\S]*?return true;\s*\}/);
-assert(defaultPath !== null, "默认模式路径应存在");
-if (defaultPath) {
-	const saveIdx = defaultPath[0].indexOf("saveAnswerFile");
-	const runIdx = defaultPath[0].indexOf("runWorkflow");
-	assert(saveIdx >= 0 && saveIdx < runIdx, "默认路径应在 runWorkflow 前调用 saveAnswerFile");
-}
-
-// Verify the custom path: saveAnswerFile before runWorkflow
-const customPath = pwdFunc.match(/if \(customSteps\.length === 0\) \{[\s\S]*?return true;\s*\}/);
-assert(customPath !== null, "自定义模式路径应存在");
-if (customPath) {
-	const saveIdx = customPath[0].indexOf("saveAnswerFile");
-	const runIdx = customPath[0].indexOf("runWorkflow");
-	assert(saveIdx >= 0 && runIdx >= 0, "自定义路径应包含 saveAnswerFile 和 runWorkflow");
-	assert(saveIdx < runIdx, "自定义路径应在 runWorkflow 前调用 saveAnswerFile");
-}
-
-// ═══════════════════════════════════════════════════════════════
-//  Test 2: 所有 caller 正确调用 promptWorkflowDecision
-// ═══════════════════════════════════════════════════════════════
-
-console.log("\n📋 Test 2: 所有 caller 正确调用 promptWorkflowDecision\n");
-
-// Count occurrences of promptWorkflowDecision calls in handlers (not definition)
-// We need at least: runWizardWithGrill + runWizard + dev-feat = 3 call sites
-const workflowDecisionCalls = source.match(/await promptWorkflowDecision\(/g);
-assert(workflowDecisionCalls !== null && workflowDecisionCalls.length >= 3,
-	`应找到至少 3 个 promptWorkflowDecision 调用，实际 ${workflowDecisionCalls?.length ?? 0}`);
-console.log(`   共 ${workflowDecisionCalls.length} 个 promptWorkflowDecision 调用`);
-
-// ═══════════════════════════════════════════════════════════════
-//  Test 3: runWizard 的非 workflow 路径调用 saveAnswerFile
-// ═══════════════════════════════════════════════════════════════
-
-console.log("\n📋 Test 3: runWizard 非 workflow 路径调用 saveAnswerFile\n");
-
-const runWizardMatch = source.match(/async function runWizard\([\s\S]*?\n\}/);
-assert(runWizardMatch !== null, "应找到 runWizard 函数");
-
-const runWizardFunc = runWizardMatch[0];
-assert(
-	runWizardFunc.includes("saveAnswerFile(ctx.cwd, prompt);"),
-	"runWizard 非 workflow 路径应调用 saveAnswerFile(ctx.cwd, prompt)",
-);
-
-// ═══════════════════════════════════════════════════════════════
-//  Test 4: runWizardWithGrill 的非 workflow 路径保留 saveAnswerFile
-// ═══════════════════════════════════════════════════════════════
-
-console.log("\n📋 Test 4: runWizardWithGrill 非 workflow 路径保留 saveAnswerFile\n");
-
-const runWizardWithGrillMatch = source.match(/async function runWizardWithGrill\([\s\S]*?\n\}/);
-assert(runWizardWithGrillMatch !== null, "应找到 runWizardWithGrill 函数");
-
-const grillFunc = runWizardWithGrillMatch[0];
-assert(
-	grillFunc.includes("const answerPath = saveAnswerFile(ctx.cwd, finalPrompt);"),
-	"runWizardWithGrill 的非 workflow 路径应包含 answerPath = saveAnswerFile(...)",
-);
-
-// ═══════════════════════════════════════════════════════════════
-//  Test 5: dev-feat handler 调用 promptWorkflowDecision
-// ═══════════════════════════════════════════════════════════════
-
-console.log("\n📋 Test 5: dev-feat handler 调用 promptWorkflowDecision\n");
-
-// Find the dev-feat registerCommand block
-const featMatch = source.match(/pi\.registerCommand\("dev-feat"[\s\S]*?\n\t\}\);/);
-assert(featMatch !== null, "应找到 /dev-feat handler");
-
-const featHandler = featMatch[0];
-assert(
-	featHandler.includes("promptWorkflowDecision(ctx, pi, finalPrompt, FEAT_WORKFLOW_STEPS)"),
-	"dev-feat handler 应调用 promptWorkflowDecision 并传递 FEAT_WORKFLOW_STEPS",
-);
-
-// ═══════════════════════════════════════════════════════════════
-//  Test 6: 所有 saveAnswerFile 调用之前都有 finalPrompt/prompt 已赋值（空安全）
-// ═══════════════════════════════════════════════════════════════
-
-console.log("\n📋 Test 6: saveAnswerFile 调用时的参数非空\n");
-
-const saveCalls = source.match(/saveAnswerFile\(ctx\.cwd, \w+\)/g);
-assert(saveCalls !== null && saveCalls.length >= 5,
-	`应找到至少 5 个 saveAnswerFile 调用，实际 ${saveCalls?.length ?? 0}`);
-
-console.log(`   共 ${saveCalls.length} 个 saveAnswerFile 调用`);
-for (const call of saveCalls) {
-	assert(
-		call.includes("finalPrompt") || call.includes("prompt") || call.includes("recovered"),
-		`saveAnswerFile 参数应为 finalPrompt/prompt/recovered，实际: ${call}`,
-	);
-}
-
-// ═══════════════════════════════════════════════════════════════
-//  Summary
-// ═══════════════════════════════════════════════════════════════
-
-console.log(`\n━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━`);
-console.log(`结果: ${pass} 通过, ${fail} 失败, 共 ${pass + fail} 个测试`);
-console.log(`━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━`);
-
-if (fail > 0) {
-	console.error("\n⚠️  部分测试未通过");
-	process.exit(1);
-} else {
-	console.log("\n✅ 所有测试通过 — saveAnswerFile 在所有 workflow 路径和非 workflow 路径中均被正确调用");
-}

package/tests/test-workflow-config.mjs

@@ -1,244 +0,0 @@
-/**
- * test-workflow-config.mjs — 验证所有 dev-* 命令正确激活 Workflow 选项
- *
- * Bug: /dev-fix 的 handler 调用 runWizardWithGrill 时未传入 workflowConfig，
- * 导致 FIX_WORKFLOW_STEPS（已定义却未使用）永远不会被触发。工作流确认对话框不出现，
- * 用户无法进入自动化工作流。
- *
- * 本测试通过静态分析源码，验证：
- *   1. 每个定义了 *_WORKFLOW_STEPS 常量的 dev-* 命令，其 handler 必须将
- *      该常量作为 workflowConfig 传给 runWizardWithGrill / runWizard，
- *      或在内联 handler 中直接调用 runWorkflow。
- *   2. 没有遗漏或「定义但未使用」的 WORKFLOW_STEPS。
- *
- * Run: node tests/test-workflow-config.mjs
- */
-
-import * as fs from "node:fs";
-import * as path from "node:path";
-import { fileURLToPath } from "node:url";
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const SOURCE_PATH = path.resolve(__dirname, "../extensions/dev-prompts.ts");
-
-// ── Helpers ──────────────────────────────────────────────────
-
-let pass = 0;
-let fail = 0;
-
-function assert(condition, msg) {
-	if (condition) {
-		pass++;
-		console.log(`  ✅ ${msg}`);
-	} else {
-		fail++;
-		console.error(`  ❌ ${msg}`);
-	}
-}
-
-function assertEq(actual, expected, msg) {
-	if (actual === expected) {
-		pass++;
-		console.log(`  ✅ ${msg}`);
-	} else {
-		fail++;
-		console.error(`  ❌ ${msg} — 期望 ${JSON.stringify(expected)}, 得到 ${JSON.stringify(actual)}`);
-	}
-}
-
-// ═══════════════════════════════════════════════════════════════
-//  Read source
-// ═══════════════════════════════════════════════════════════════
-
-let source;
-try {
-	source = fs.readFileSync(SOURCE_PATH, "utf-8");
-} catch (e) {
-	console.error(`Failed to read source file: ${e.message}`);
-	process.exit(1);
-}
-
-console.log(`📄 源文件: ${SOURCE_PATH}`);
-console.log(`📏 文件大小: ${source.length} 字节\n`);
-
-// ═══════════════════════════════════════════════════════════════
-//  Test 1: Every *_WORKFLOW_STEPS constant is used by its handler
-// ═══════════════════════════════════════════════════════════════
-
-console.log("📋 Test 1: 所有 *_WORKFLOW_STEPS 常量被其 handler 使用\n");
-
-// Each entry: [constantName, expectedCommandName, expectedHandlerType]
-// handlerType: "runWizardWithGrill" | "runWizard" | "inline"
-const workflowSteps = [
-	{ const: "FEAT_WORKFLOW_STEPS",   command: "dev-feat",   type: "inline" },              // inline handler
-	{ const: "FIX_WORKFLOW_STEPS",    command: "dev-fix",    type: "runWizardWithGrill" },   // was BROKEN — missing arg
-	{ const: "DOC_WORKFLOW_STEPS",    command: "dev-doc",    type: "runWizardWithGrill" },   // has it
-	{ const: "REFACTOR_WORKFLOW_STEPS", command: "dev-refactor", type: "runWizardWithGrill" }, // has it
-	{ const: "TEST_WORKFLOW_STEPS",   command: "dev-test",   type: "runWizardWithGrill" },   // has it
-	{ const: "PERF_WORKFLOW_STEPS",   command: "dev-perf",   type: "runWizardWithGrill" },   // has it
-	{ const: "STYLE_WORKFLOW_STEPS",  command: "dev-style",  type: "runWizard" },            // has it
-	{ const: "SECURITY_WORKFLOW_STEPS", command: "dev-security", type: "runWizard" },         // has it
-];
-
-for (const ws of workflowSteps) {
-	// 1. Verify constant is defined in source
-	const constDefined = source.includes(`const ${ws.const}: WorkflowStepDef[] = [`);
-	assert(constDefined, `${ws.const} 应在源文件中定义`);
-
-	// 2. Usage check: the constant name must appear in the source more than just its definition.
-	//    Count occurrences of the bare constant name (e.g. "FIX_WORKFLOW_STEPS").
-	//    Definition has 1 occurrence (const XXX: ...), usage adds at least 1 more.
-	const bareName = ws.const;
-	const allOccurrences = source.match(new RegExp(bareName, 'g'));
-	const occurrenceCount = allOccurrences ? allOccurrences.length : 0;
-	// Minimum: 1 for definition + 1 for usage = 2
-	assert(occurrenceCount >= 2,
-		`${bareName} 应至少出现 2 次 (定义 + 使用), 实际 ${occurrenceCount} 次`);
-}
-
-// ═══════════════════════════════════════════════════════════════
-//  Test 2: runWizardWithGrill calls pass both grillOptions and workflowConfig
-// ═════════════════──────────────────────────────────────────────
-//  All runWizardWithGrill calls that have a workflowConfig should
-//  have it as the last argument before the closing parenthesis.
-
-console.log("\n📋 Test 2: runWizardWithGrill 调用完整性\n");
-
-// Find all runWizardWithGrill(...) calls
-const grillCalls = source.match(/await runWizardWithGrill\([\s\S]*?\);/g);
-assert(grillCalls !== null && grillCalls.length >= 4,
-	`应找到至少 4 个 runWizardWithGrill 调用，实际 ${grillCalls?.length ?? 0}`);
-
-console.log(`   共 ${grillCalls.length} 个 runWizardWithGrill 调用`);
-
-for (let i = 0; i < grillCalls.length; i++) {
-	const call = grillCalls[i];
-
-	// Extract the command type from the call
-	const typeMatch = call.match(/await runWizardWithGrill\(\s*ctx,\s*pi,\s*"([^"]+)"/);
-	const type = typeMatch ? typeMatch[1] : `#${i}`;
-
-	// Count arguments: split by top-level commas (ignoring those inside braces/brackets)
-	let depth = 0;
-	let argCount = 0;
-	for (const ch of call) {
-		if (ch === '(' || ch === '{' || ch === '[') depth++;
-		else if (ch === ')' || ch === '}' || ch === ']') depth--;
-		else if (ch === ',' && depth === 1) argCount++;
-	}
-	// Number of arguments = commas + 1 (inside top-level parens)
-	// But the last comma before `)` doesn't count, so total args = number of top-level commas + 1
-	// Actually, let me recalculate: top-level commas separate args.
-	// For `runWizardWithGrill(a, b, c, ...)` — the paren depth at commas inside the function call is 1.
-	// The last `)` decrements depth and we shouldn't count commas after that.
-	// Let me just count the number of top-level commas before the last closing paren.
-	
-	let topLevelCommas = 0;
-	depth = 0;
-	for (const ch of call) {
-		if (ch === '(') depth++;
-		else if (ch === ')') { depth--; if (depth === 0) break; }
-		else if (ch === ',' && depth === 1) topLevelCommas++;
-	}
-	const totalArgs = topLevelCommas + 1;
-
-	// runWizardWithGrill has 8 parameters (ctx, pi, type, label, questions, assembler, grillOptions?, workflowConfig?)
-	// If we see 7 args → missing workflowConfig; 8 args → has workflowConfig
-	const hasWorkflowConfig = totalArgs >= 8;
-
-	assert(hasWorkflowConfig,
-		`runWizardWithGrill("${type}") 应有 8 个参数 (当前 ${totalArgs}) — 缺少 workflowConfig 参数`);
-
-	if (hasWorkflowConfig) {
-		console.log(`    ✅ "${type}": ${totalArgs} 参数, workflowConfig 已传递`);
-	} else {
-		console.error(`    ❌ "${type}": ${totalArgs} 参数, 缺少 workflowConfig`);
-	}
-}
-
-// ═══════════════════════════════════════════════════════════════
-//  Test 3: Inline handlers (dev-feat) call promptWorkflowDecision
-// ═══════════════════════════════════════════════════════════════
-
-console.log("\n📋 Test 3: 内联 handler 调用 promptWorkflowDecision\n");
-
-// dev-feat: inline handler that calls runWorkflow directly
-const featHandler = source.match(/pi\.registerCommand\("dev-feat"[\s\S]*?FEAT_WORKFLOW_STEPS \}\);/);
-assert(featHandler !== null, "应找到 /dev-feat handler");
-
-const featCallsPromptDecision = featHandler && featHandler[0].includes("promptWorkflowDecision(ctx, pi, finalPrompt, FEAT_WORKFLOW_STEPS)");
-assert(featCallsPromptDecision, "/dev-feat 内联 handler 应调用 promptWorkflowDecision 并传递 FEAT_WORKFLOW_STEPS");
-
-// dev-workflow-continue: also uses FEAT_WORKFLOW_STEPS
-const continueHandler = source.match(/pi\.registerCommand\("dev-workflow-continue"[\s\S]*?FEAT_WORKFLOW_STEPS \}\);/);
-assert(continueHandler !== null, "应找到 /dev-workflow-continue handler");
-if (continueHandler) {
-	assert(
-		continueHandler[0].includes("{ steps: FEAT_WORKFLOW_STEPS }"),
-		"/dev-workflow-continue handler 应引用 FEAT_WORKFLOW_STEPS",
-	);
-}
-
-// ═══════════════════════════════════════════════════════════════
-//  Test 4: runWizard calls that have workflow steps
-// ═══════════════════════════════════════════════════════════════
-
-console.log("\n📋 Test 4: runWizard 调用完整性\n");
-
-// Find all runWizard(...) calls
-const wizardCalls = source.match(/await runWizard\([\s\S]*?\);/g);
-assert(wizardCalls !== null && wizardCalls.length >= 5,
-	`应找到至少 5 个 runWizard 调用，实际 ${wizardCalls?.length ?? 0}`);
-
-console.log(`   共 ${wizardCalls.length} 个 runWizard 调用`);
-
-// runWizard has 7 parameters (ctx, pi, type, label, questions, assembler, workflowConfig?)
-// If the command has _WORKFLOW_STEPS defined nearby, it should pass it.
-const nonWorkflowCommands = ["chore", "explain", "compare"]; // intentionally no workflow
-
-for (let i = 0; i < wizardCalls.length; i++) {
-	const call = wizardCalls[i];
-	const typeMatch = call.match(/await runWizard\(\s*ctx,\s*pi,\s*"([^"]+)"/);
-	const type = typeMatch ? typeMatch[1] : `#${i}`;
-
-	// Count top-level arguments
-	let topLevelCommas = 0;
-	let depth = 0;
-	for (const ch of call) {
-		if (ch === '(') depth++;
-		else if (ch === ')') { depth--; if (depth === 0) break; }
-		else if (ch === ',' && depth === 1) topLevelCommas++;
-	}
-	const totalArgs = topLevelCommas + 1;
-
-	const expectsWorkflow = !nonWorkflowCommands.includes(type);
-
-	if (expectsWorkflow) {
-		const hasWorkflowConfig = totalArgs >= 7;
-		assert(hasWorkflowConfig,
-			`runWizard("${type}") 应有 7 个参数 (当前 ${totalArgs}) — 缺少 workflowConfig`);
-		if (hasWorkflowConfig) {
-			console.log(`    ✅ "${type}": ${totalArgs} 参数, workflowConfig 已传递`);
-		}
-	} else {
-		// These commands intentionally don't pass workflowConfig
-		const noWorkflowConfig = totalArgs <= 6;
-		assert(noWorkflowConfig,
-			`runWizard("${type}") 应有 6 个参数 (当前 ${totalArgs}) — 此类命令不含 workflow`);
-	}
-}
-
-// ═══════════════════════════════════════════════════════════════
-//  Summary
-// ═══════════════════════════════════════════════════════════════
-
-console.log(`\n━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━`);
-console.log(`结果: ${pass} 通过, ${fail} 失败, 共 ${pass + fail} 个测试`);
-console.log(`━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━`);
-
-if (fail > 0) {
-	console.error("\n⚠️  部分测试未通过");
-	process.exit(1);
-} else {
-	console.log("\n✅ 所有测试通过 — 所有 dev-* 命令的 Workflow 配置均完整可用");
-}