@ghyper9023/pi-dev-workflow 0.3.3 → 0.4.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ghyper9023/pi-dev-workflow",
-  "version": "0.3.3",
+  "version": "0.4.1",
   "keywords": [
     "pi-package"
   ],

package/skills/review-html/SKILL.md

@@ -49,8 +49,8 @@ description: review 代码并输出交互式 HTML 报告。适用于代码审查
 1. 根据用户指令运行 `git diff HEAD` 或 `git log -p -n <N>` 获取改动，或遍历代码库。
 2. 合并改动，按文件分组。
 3. 对每个文件进行审查，并生成符合上述要求的 HTML。
-4. 自动确保 `pi-dev-output/pi-review/` 文件夹存在于项目根目录，若不存在则创建；并确保 `pi-dev-output/` 下的 `.gitignore` 包含 `*`（忽略所有内容但保留目录）。
+4. 自动确保 `.pi-dev-output/pi-review/html/` 文件夹存在于项目根目录，若不存在则创建。
 5. HTML 文件命名格式：`年月日时分-任务简述(极简10词以内)-index.html`，若同名文件已存在则编号递增（如 `index1.html`）。
-6. 直接输出 HTML 文件到 `pi-dev-output/pi-review/` 目录，**不要添加任何解释性文字**，仅简短说明工作完成和输出文件路径即可。
+6. 直接输出 HTML 文件到 `.pi-dev-output/pi-review/html/` 目录，**不要添加任何解释性文字**，仅简短说明工作完成和输出文件路径即可。
 
 请严格遵循以上规则。

package/skills/to-prd/SKILL.md

@@ -15,7 +15,7 @@ A deep module (as opposed to a shallow module) is one which encapsulates a lot o
 
 Check with the user that these modules match their expectations. Check with the user which modules they want tests written for.
 
-3. Write the PRD using the template below, then save it to `pi-dev-output/pi-prd/` directory.
+3. Write the PRD using the template below, then save it to `.pi-dev-output/pi-prd/` directory.
 
 <prd-template>
 

package/tests/test-grill-json-fix.mjs

@@ -0,0 +1,243 @@
+/**
+ * test-grill-json-fix.mjs — Verify the JSON stability fixes for grill-me-agent
+ *
+ * Tests:
+ * 1. writeToolPromptSuffix 包含 JSON Schema
+ * 2. writeToolPromptSuffix 包含转义规则
+ * 3. writeToolPromptSuffix 包含自我校验指令
+ * 4. readQuestionsFromFile 正确处理合法 JSON
+ * 5. readQuestionsFromFile 正确处理无效 JSON（返回空数组 + 不报错）
+ * 6. readQuestionsFromFile 正确处理含特殊字符的 JSON
+ * 7. 输出文件不再被删除
+ * 8. 重试 prompt 包含错误上下文
+ * 9. ensureOutputDir 的 .gitignore 让文件不被 git 跟踪（保留在本地）
+ *
+ * Run: node tests/test-grill-json-fix.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 EXT_PATH = path.resolve(__dirname, "../extensions/grill-me-agent.ts");
+
+// ── Helpers ──────────────────────────────────────────────────
+
+let pass = 0;
+let fail = 0;
+
+function assert(condition, msg) {
+	if (condition) {
+		pass++;
+		console.log(`  ✅ ${msg}`);
+	} else {
+		fail++;
+		console.error(`  ❌ ${msg}`);
+	}
+}
+
+// ── Read the source file ─────────────────────────────────────
+
+let source;
+try {
+	source = fs.readFileSync(EXT_PATH, "utf-8");
+} catch (e) {
+	console.error(`Failed to read source file: ${e.message}`);
+	process.exit(1);
+}
+
+console.log(`📄 源文件: ${EXT_PATH}`);
+console.log(`📏 文件大小: ${source.length} 字节\n`);
+
+// ── Test 1: Contains JSON Schema ─────────────────────────────
+
+assert(
+	source.includes('"$schema": "http://json-schema.org/draft-07/schema#"'),
+	"writeToolPromptSuffix 应包含 Draft-07 JSON Schema",
+);
+
+assert(
+	source.includes('"required": ["id", "question", "options"]'),
+	"JSON Schema 要求 id, question, options 为必填字段",
+);
+
+assert(
+	source.includes('"minItems": 1'),
+	"JSON Schema 要求 options 至少 1 项",
+);
+
+// ── Test 2: Contains escaping rules ──────────────────────────
+
+assert(
+	source.includes("String escaping rules") ||
+	source.includes("JSON-escaped"),
+	"writeToolPromptSuffix 应包含字符串转义规则",
+);
+
+assert(
+	source.includes("Double quotes inside text"),
+	"转义规则应提及双引号",
+);
+
+assert(
+	source.includes("Newlines"),
+	"转义规则应提及换行符",
+);
+
+assert(
+	source.includes("Backslashes"),
+	"转义规则应提及反斜杠",
+);
+
+// ── Test 3: Contains self-review instruction ─────────────────
+
+assert(
+	source.includes("Self-review") ||
+	source.includes("mentally validate") ||
+	source.includes("mentally validate your JSON"),
+	"writeToolPromptSuffix 应包含自我校验指令",
+);
+
+// ── Test 4: Example uses unicode escapes ─────────────────────
+// Note: The source file has \\uXXXX (double backslash in raw text)
+// because TypeScript single-quoted string \\ → single backslash in value.
+// search for "\\\\u" (two backslashes in raw file = "\\u" in JS regex)
+assert(
+	source.includes("\\\\u9879\\\\u76ee") ||
+	(source.includes("\\\\u9879") && source.includes("u9879")),
+	"示例应包含 JSON unicode 转义序列（合法 JSON 示例）",
+);
+
+// ── Test 5: readQuestionsFromFile has error logging ──────────
+
+assert(
+	source.includes("[grill-me-agent] JSON parse error in"),
+	"readQuestionsFromFile 应在 JSON 解析失败时输出错误日志",
+);
+
+// ── Test 6: Output files are no longer deleted ───────────────
+
+assert(
+	!source.includes("fs.unlinkSync(outputFilePath)") &&
+	!source.includes("fs.unlinkSync(retryPath)"),
+	"输出文件不应再被 fs.unlinkSync 删除",
+);
+
+// ── Test 7: Retry prompt includes error context ──────────────
+
+assert(
+	source.includes("Previous attempt had JSON errors") ||
+	source.includes("parseErrorMsg"),
+	"重试 prompt 应包含前次错误的上下文",
+);
+
+// ── Test 8: Parse valid JSON ─────────────────────────────────
+
+function simulateReadQuestions(raw) {
+	try {
+		const parsed = JSON.parse(raw);
+		const items = Array.isArray(parsed) ? parsed : parsed.questions;
+		if (!Array.isArray(items)) return 0;
+		return items.filter(
+			(q) => q && typeof q.question === "string" && Array.isArray(q.options) && q.options.length > 0,
+		).length;
+	} catch {
+		return 0;
+	}
+}
+
+const validJson = JSON.stringify({
+	questions: [
+		{ id: 1, question: "测试问题", options: ["选项A", "选项B"] },
+		{ id: 2, question: "第二个问题", options: ["选项X", "选项Y", "选项Z"] },
+	],
+});
+
+assert(
+	simulateReadQuestions(validJson) === 2,
+	"readQuestionsFromFile 应解析合法 JSON 并返回正确数量的问题",
+);
+
+// ── Test 9: Invalid JSON (unescaped double quotes) ───────────
+
+const invalidJson = `{
+  "questions": [
+    {
+      "id": 1,
+      "question": "他说"这个不行"，如何处理？",
+      "options": ["忽略", "修复"]
+    }
+  ]
+}`;
+
+assert(
+	simulateReadQuestions(invalidJson) === 0,
+	"readQuestionsFromFile 遇到非法 JSON 应返回空数组，而非崩溃",
+);
+
+// ── Test 10: JSON with properly escaped special chars ────────
+
+const escapedJson = JSON.stringify({
+	questions: [
+		{
+			id: 1,
+			question: '他说"这个不行"，如何处理？',
+			options: ["忽略", "修复"],
+		},
+	],
+});
+
+assert(
+	simulateReadQuestions(escapedJson) === 1,
+	"readQuestionsFromFile 应解析含转义双引号的合法 JSON",
+);
+
+// ── Test 11: Bare array format backward compat ───────────────
+
+const bareArray = JSON.stringify([
+	{ id: 1, question: "测试问题", options: ["A", "B"] },
+]);
+
+assert(
+	simulateReadQuestions(bareArray) === 1,
+	"readQuestionsFromFile 应后向兼容裸数组格式 [...]",
+);
+
+// ── Test 12: Missing vs empty fields ─────────────────────────
+// Empty string "" passes filter (typeof "" === "string" is true).
+// Only truly missing (undefined) fields are filtered.
+
+const withIncomplete = JSON.stringify({
+	questions: [
+		{ id: 1, question: "完整问题", options: ["A"] },
+		{ id: 2, question: "", options: ["A"] },          // empty string, passes filter
+		{ id: 3, options: ["A"] },                         // missing question, filtered
+		{ id: 4, question: "无选项" },                      // missing options, filtered
+	],
+});
+
+assert(
+	simulateReadQuestions(withIncomplete) === 2,
+	"readQuestionsFromFile 应过滤缺失字段的问题，空字符串字段应保留",
+);
+
+// ── Test 13: gitignore for output dir exists ─────────────────
+
+assert(
+	source.includes("pi-grill"),
+	"代码中应引用 pi-grill 输出目录",
+);
+
+// ── 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-output-directory-structure.mjs

@@ -0,0 +1,177 @@
+/**
+ * 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

@@ -0,0 +1,187 @@
+/**
+ * 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 路径中均被正确调用");
+}