@ghyper9023/pi-dev-workflow 0.5.1 → 0.6.0

package/extensions/dev-prompts.ts

@@ -399,7 +399,7 @@ const FEAT_WORKFLOW_STEPS: WorkflowStepDef[] = [
 		label: "📝 更新文档",
 		type: "confirm",
 		agentName: "docWriter",
-		timeoutMs: 300_000,
+		timeoutMs: 600_000,
 	},
 ];
 
@@ -426,7 +426,7 @@ const FIX_WORKFLOW_STEPS: WorkflowStepDef[] = [
 		label: "📝 更新文档",
 		type: "confirm",
 		agentName: "docWriter",
-		timeoutMs: 300_000,
+		timeoutMs: 600_000,
 	},
 ];
 
@@ -513,7 +513,7 @@ const DOC_WORKFLOW_STEPS: WorkflowStepDef[] = [
 		label: "📝 撰写文档",
 		type: "auto",
 		agentName: "docWriter",
-		timeoutMs: 300_000,
+		timeoutMs: 600_000,
 	},
 ];
 
@@ -536,7 +536,7 @@ const SECURITY_WORKFLOW_STEPS: WorkflowStepDef[] = [
 		label: "🔒 安全审查",
 		type: "auto",
 		agentName: "reviewer",
-		timeoutMs: 300_000,
+		timeoutMs: 900_000,
 	},
 ];
 
@@ -857,7 +857,7 @@ const COMPARE_QUESTIONS = [
 export default function (pi: ExtensionAPI) {
 	// ── /dev-feat ──────────────────────────────────────────────
 	pi.registerCommand("dev-feat", {
-		description: "(prompt wizard) 新功能/创意生成 — 支持设计评审 (Grill) + 自动化工作流",
+		description: "(prompt wizard) 新功能/创意生成 — 支持设计方案追问完善 (Grill) + 自动化工作流",
 		handler: async (_args, ctx) => {
 			const answers: Record<string, string> = {};
 			let featIdx = 0;
@@ -911,17 +911,17 @@ export default function (pi: ExtensionAPI) {
 
 	// ── /dev-fix ───────────────────────────────────────────────
 	pi.registerCommand("dev-fix", {
-		description: "(prompt wizard) 问题排查/错误修正 — 支持根因分析评审 (Grill)",
+		description: "(prompt wizard) 问题排查/错误修正 — 支持 Bug 根因追问 (Grill)",
 		handler: async (_args, ctx) => {
 			await runWizardWithGrill(
 				ctx, pi, "fix", "问题排查/错误修正",
 				FIX_QUESTIONS, assembleFixPrompt,
 				{
 					agentDef: _fixGrillAgent,
-					title: "🐛 Bug 根因分析评审",
-					description: "AI 会从复现条件、根因推理、修复方案、回归风险等维度挑战你的理解。",
+					title: "🐛 Bug 根因追问",
+					description: "AI 会通过一步步追问帮你精准定位根因：从复现条件到根本原因推理，再到修复方案验证和回归风险评估。",
 					questionTitle: "Bug 根因分析",
-					loaderLabel: "🧠 AI 正在分析代码并生成根因评审问题...",
+					loaderLabel: "🧠 AI 正在分析代码并生成根因追问问题...",
 				},
 				{ steps: FIX_WORKFLOW_STEPS },
 			);
@@ -930,17 +930,17 @@ export default function (pi: ExtensionAPI) {
 
 	// ── /dev-doc ───────────────────────────────────────────────
 	pi.registerCommand("dev-doc", {
-		description: "(prompt wizard) 文档生成/总结 — 支持大纲评审 (Grill)",
+		description: "(prompt wizard) 文档生成/总结 — 支持文档大纲追问完善 (Grill)",
 		handler: async (_args, ctx) => {
 			await runWizardWithGrill(
 				ctx, pi, "doc", "文档生成/总结",
 				DOC_QUESTIONS, assembleDocPrompt,
 				{
 					agentDef: _docGrillAgent,
-					title: "📄 文档大纲评审",
-					description: "AI 会从受众定位、结构安排、示例选择等维度审视你的文档计划。",
-					questionTitle: "文档大纲评审",
-					loaderLabel: "🧠 AI 正在分析并生成文档大纲评审问题...",
+					title: "📄 文档大纲追问完善",
+					description: "AI 会通过追问帮你完善文档大纲：从受众定位到结构安排，确认术语一致性和示例覆盖范围。",
+					questionTitle: "文档大纲追问完善",
+					loaderLabel: "🧠 AI 正在分析并生成文档大纲追问问题...",
 				},
 				{ steps: DOC_WORKFLOW_STEPS },
 			);
@@ -949,17 +949,17 @@ export default function (pi: ExtensionAPI) {
 
 	// ── /dev-refactor ──────────────────────────────────────────
 	pi.registerCommand("dev-refactor", {
-		description: "(prompt wizard) 重构/优化现有结构 — 支持重构计划评审 (Grill)",
+		description: "(prompt wizard) 重构/优化现有结构 — 支持重构方案追问 (Grill)",
 		handler: async (_args, ctx) => {
 			await runWizardWithGrill(
 				ctx, pi, "refactor", "重构/优化",
 				REFACTOR_QUESTIONS, assembleRefactorPrompt,
 				{
 					agentDef: _refactorGrillAgent,
-					title: "🔧 重构方案评审",
-					description: "AI 会从模块边界、API 兼容性、测试策略、迁移风险等维度审视你的重构计划。",
-					questionTitle: "重构方案评审",
-					loaderLabel: "🧠 AI 正在分析代码并生成重构评审问题...",
+					title: "🔧 重构方案追问",
+					description: "AI 会通过追问帮你识别隐藏耦合风险：从模块边界到 API 兼容性，验证行为保持和迁移路径安全性。",
+					questionTitle: "重构方案追问",
+					loaderLabel: "🧠 AI 正在分析代码并生成重构追问问题...",
 				},
 				{ steps: REFACTOR_WORKFLOW_STEPS },
 			);
@@ -968,17 +968,17 @@ export default function (pi: ExtensionAPI) {
 
 	// ── /dev-test ──────────────────────────────────────────────
 	pi.registerCommand("dev-test", {
-		description: "(prompt wizard) 测试用例生成 — 支持测试计划评审 (Grill)",
+		description: "(prompt wizard) 测试用例生成 — 支持测试策略追问 (Grill)",
 		handler: async (_args, ctx) => {
 			await runWizardWithGrill(
 				ctx, pi, "test", "测试用例/评估",
 				TEST_QUESTIONS, assembleTestPrompt,
 				{
 					agentDef: _testGrillAgent,
-					title: "🧪 测试计划评审",
-					description: "AI 会从覆盖维度、边界条件、模拟策略等角度审视你的测试方案。",
-					questionTitle: "测试计划评审",
-					loaderLabel: "🧠 AI 正在分析并生成测试评审问题...",
+					title: "🧪 测试策略追问",
+					description: "AI 会通过追问帮你发现测试缺口：从覆盖维度到边界条件，验证模拟策略和测试隔离是否到位。",
+					questionTitle: "测试策略追问",
+					loaderLabel: "🧠 AI 正在分析并生成测试追问问题...",
 				},
 				{ steps: TEST_WORKFLOW_STEPS },
 			);
@@ -995,17 +995,17 @@ export default function (pi: ExtensionAPI) {
 
 	// ── /dev-perf ──────────────────────────────────────────────
 	pi.registerCommand("dev-perf", {
-		description: "(prompt wizard) 性能优化 — 支持优化方案评审 (Grill)",
+		description: "(prompt wizard) 性能优化 — 支持性能优化方案追问 (Grill)",
 		handler: async (_args, ctx) => {
 			await runWizardWithGrill(
 				ctx, pi, "perf", "性能优化",
 				PERF_QUESTIONS, assemblePerfPrompt,
 				{
 					agentDef: _perfGrillAgent,
-					title: "⚡ 性能优化方案评审",
-					description: "AI 会从基准测试方法、优化方向、回归风险等维度审视你的方案。",
-					questionTitle: "性能优化方案评审",
-					loaderLabel: "🧠 AI 正在分析并生成性能优化评审问题...",
+					title: "⚡ 性能优化方案追问",
+					description: "AI 会通过追问帮你验证瓶颈判断和优化方向：从基准测试方法到潜在回归风险，确保方案合理性。",
+					questionTitle: "性能优化方案追问",
+					loaderLabel: "🧠 AI 正在分析并生成性能优化追问问题...",
 				},
 				{ steps: PERF_WORKFLOW_STEPS },
 			);

package/extensions/grill-me-agent.ts

@@ -1,5 +1,5 @@
 /**
- * grill-me-agent.ts — 设计评审 (Grill) 和 PRD 生成的独立管理器
+ * grill-me-agent.ts — 设计 (Grill) 和 PRD 生成的独立管理器
  *
  * 职责：
  *   1. runGrillPhase()  — 启动 sub-agent 生成评审问题，TUI 逐题呈现（选项 + 自定义输入）
@@ -372,10 +372,10 @@ export async function runGrillPhase(
 	};
 
 	const agentDef = options?.agentDef ?? _defaultGrillAgent;
-	const confirmTitle = options?.title ?? "🔍 设计方案评审";
-	const confirmDesc = options?.description ?? "AI 会从架构、数据流、边界条件、安全等多个维度挑战你的设计。";
-	const qTitlePrefix = options?.questionTitle ?? "设计方案评审";
-	const loaderLabel = options?.loaderLabel ?? "🧠 AI 子代理正在分析代码并生成评审问题...";
+	const confirmTitle = options?.title ?? "🔍 设计方案追问完善";
+	const confirmDesc = options?.description ?? "AI 会通过系统性追问帮你打磨方案：从术语精确化到边界条件验证，确保架构决策的每个分支都经过推敲。";
+	const qTitlePrefix = options?.questionTitle ?? "设计方案追问完善";
+	const loaderLabel = options?.loaderLabel ?? "🧠 AI 子代理正在分析代码并生成追问问题...";
 
 	// ── Step 1: Confirm entering grill mode ──────────────────
 	const enterGrill = await uiConfirm(ctx, confirmTitle, confirmDesc);
@@ -438,16 +438,16 @@ export async function runGrillPhase(
 
 		const choice = await uiSelect(
 			ctx,
-			"⚠️ AI 未能成功生成评审问题",
+			"⚠️ AI 未能成功生成追问问题",
 			[
-				"🔄 重新尝试生成评审问题",
+				"🔄 重新尝试生成追问问题",
 				"⏭️ 跳过 Grill，直接发送 Prompt",
 				"❌ 取消 (Esc)",
 			],
 		);
 
 		switch (choice) {
-			case "🔄 重新尝试生成评审问题": {
+			case "🔄 重新尝试生成追问问题": {
 				const retryPath = grillOutputPath(ctx.cwd);
 				const errorFeedback = parseErrorMsg
 					? [
@@ -513,13 +513,13 @@ export async function runGrillPhase(
 					rIdx++;
 				}
 				const qaBlock = pairs
-					.map((p, i) => `[评审问题 ${i + 1}]\n问题: ${p.question}\n回答: ${p.answer}`)
+					.map((p, i) => `[追问问题 ${i + 1}]\n问题: ${p.question}\n回答: ${p.answer}`)
 					.join("\n\n");
 				const finalEnhancedPrompt = [
 					assembledPrompt,
 					"",
 					"---",
-					"## 设计评审记录",
+					"## 设计追问记录",
 					"",
 					"以下是在开发前进行的设计评审问答，所有决策已确认：",
 					"",
@@ -572,16 +572,16 @@ export async function runGrillPhase(
 
 	// ── Step 6: Assemble enhanced prompt ─────────────────────
 	const qaBlock = pairs
-		.map((p, i) => `[评审问题 ${i + 1}]\n问题: ${p.question}\n回答: ${p.answer}`)
+		.map((p, i) => `[追问问题 ${i + 1}]\n问题: ${p.question}\n回答: ${p.answer}`)
 		.join("\n\n");
 
 	const finalEnhancedPrompt = [
 		assembledPrompt,
 		"",
 		"---",
-		"## 设计评审记录",
+		"## 方案追问记录",
 		"",
-		"以下是在开发前进行的设计评审问答，所有决策已确认：",
+		"以下是在开发前进行的追问式方案打磨记录，所有决策已确认：",
 		"",
 		qaBlock,
 	].join("\n");
@@ -615,7 +615,7 @@ async function showQuestionTUI(
 	q: GrillQuestion,
 	currentIndex: number,
 	totalCount: number,
-	titlePrefix = "设计方案评审",
+	titlePrefix = "设计方案追问完善",
 	backable = false,
 	previousAnswer?: string,
 ): Promise<string | null> {
@@ -707,8 +707,8 @@ async function showQuestionTUI(
 
 		container.addChild(new Spacer(1));
 		const hint = backable && currentIndex > 1
-			? "  ↑↓ 导航 • Enter 选择 • Ctrl+Shift+← 返回上一题 • Esc 取消全部评审"
-			: "  ↑↓ 导航 • Enter 选择 • Esc 取消全部评审";
+			? "  ↑↓ 导航 • Enter 选择 • Ctrl+Shift+← 返回上一题 • Esc 取消全部追问"
+			: "  ↑↓ 导航 • Enter 选择 • Esc 取消全部追问";
 		container.addChild(
 			new Text(theme.fg("dim", hint), 0, 0),
 		);

package/extensions/workflow-engine.ts

@@ -447,13 +447,19 @@ function captureBaseline(cwd: string): void {
  *
  * Uses git-format status codes (M, A, D) for display consistency.
  * Deduplicates against existing _workflowFileChanges.
+ *
+ * The dedup key includes stepIndex (filePath:stepIndex) so the same file
+ * can appear across different workflow steps (e.g., created by worker in
+ * loop-group, then reviewed by reviewer) without being collapsed into one.
+ * Without :stepIndex, the Set dedup would wrongly skip a file modified in
+ * a later step just because it was already tracked in an earlier step.
  */
 function updateToolsFromGit(cwd: string, stepIndex: number, agentName: string): void {
 	const currentChanges = getGitDiffChanges(cwd);
-	const seen = new Set(_workflowFileChanges.map(c => c.filePath));
-
+	const seen = new Set(_workflowFileChanges.map(c => `${c.filePath}:${c.stepIndex}`));
+	
 	for (const change of currentChanges) {
-		if (seen.has(change.path)) continue;
+		if (seen.has(`${change.path}:${stepIndex}`)) continue;
 
 		// ── Baseline filtering ──────────────────────────────
 		// Skip files that were already dirty at workflow start and haven't been touched.
@@ -470,7 +476,13 @@ function updateToolsFromGit(cwd: string, stepIndex: number, agentName: string):
 			}
 		}
 
-		seen.add(change.path);
+		// Skip files in .pi-dev-output/ — these are workflow-generated artifacts (plans, reviews)
+		// and should NOT be reported as user repo changes. Real git clients (VSCode, Zed) also
+		// ignore this directory (it's in .gitignore). This filter is defense-in-depth.
+		// Note: git always uses forward slashes in paths, even on Windows.
+		if (change.path.startsWith(".pi-dev-output/")) continue;
+
+		seen.add(`${change.path}:${stepIndex}`);
 		const type: FileChangeEntry["type"] =
 			change.status === "A" ? "new" :
 			change.status === "D" ? "delete" :
@@ -883,47 +895,6 @@ function addWidgetSubStepTool(stepIndex: number, agentName: string, tool: string
 		if (!sub.tools) sub.tools = [];
 		sub.tools.push(tool);
 		if (sub.tools.length > 20) sub.tools = sub.tools.slice(-20); // keep last 20
-
-		// Also track as file change for checkpoint
-		// Support both old format ("edit: path") and new git-format ("M   path", "A   path", "D   path")
-		const oldMatch = tool.match(/^(edit|new|delete|read):\s*(.+)/i);
-		const gitMatch = !oldMatch ? tool.match(/^([MAD])\s{2,}(.+)$/) : null;
-		if (oldMatch) {
-			const changeType = oldMatch[1]!.toLowerCase() as FileChangeEntry["type"];
-			const filePath = oldMatch[2]!.trim();
-			const exists = _workflowFileChanges.some(
-				c => c.filePath === filePath && c.type === changeType && c.stepIndex === stepIndex && c.agent === agentName,
-			);
-			if (!exists && filePath.length > 3) {
-				_workflowFileChanges.push({
-					agent: agentName,
-					stepIndex,
-					type: changeType,
-					filePath,
-					timestamp: new Date().toISOString(),
-				});
-			}
-		} else if (gitMatch) {
-			const gitStatus = gitMatch[1]!;
-			const changeType: FileChangeEntry["type"] =
-				gitStatus === "A" ? "new" :
-				gitStatus === "D" ? "delete" :
-				"edit";
-			const filePath = gitMatch[2]!.trim();
-			const exists = _workflowFileChanges.some(
-				c => c.filePath === filePath && c.type === changeType && c.stepIndex === stepIndex && c.agent === agentName,
-			);
-			if (!exists && filePath.length > 3) {
-				_workflowFileChanges.push({
-					agent: agentName,
-					stepIndex,
-					type: changeType,
-					filePath,
-					timestamp: new Date().toISOString(),
-				});
-			}
-		}
-
 		refreshWidget();
 	}
 }
@@ -1130,7 +1101,7 @@ async function runAgentWithProgress(
 	const result = await spawnSubagent(agent, task, _workflowCwd, signal, timeoutMs, (progress) => {
 		// Try to parse tool calls from progress messages
 		// Only match if it looks like a file path (contains a dot or path separator)
-		const toolMatch = progress.match(/(edit|read|write|new|bash|grep|find|ls|delete|remove)\s*[:：]\s*(\S+)/i);
+		const toolMatch = progress.match(/(edit|read|write|new|bash|grep|find|ls|delete|remove)\s*[:：]\s*([\w./\\-]+)/i);
 		if (toolMatch) {
 			const toolType = toolMatch[1]!.toLowerCase();
 			const target = toolMatch[2]!;
@@ -1141,12 +1112,14 @@ async function runAgentWithProgress(
 				_widgetExtraToolCount++;
 			}
 		}
-		// Detect output file paths — ONLY match explicit .pi-dev-output paths, never free-form text
-		const outputMatch = progress.match(/\.pi-dev-output\/[^\s,;)\]}]{5,}\.\w+/i);
-		if (outputMatch) {
-			const pathCandidate = outputMatch[0]!.trim();
-			if (pathCandidate.length > 15 && pathCandidate.length < 300) {
-				addWidgetSubStepOutput(stepIndex, agentName, pathCandidate);
+		// Detect output file paths — only match .pi-dev-output paths belonging to current workflow
+		if (_workflowId) {
+			const outputMatch = progress.match(/\.pi-dev-output\/[^\s,;)\]}'"`]{5,}\.\w+/i);
+			if (outputMatch) {
+				const pathCandidate = outputMatch[0]!.trim().replace(/["'`\\]+$/g, '');
+				if (pathCandidate.length > 15 && pathCandidate.length < 300 && pathCandidate.includes(_workflowId)) {
+					addWidgetSubStepOutput(stepIndex, agentName, pathCandidate);
+				}
 			}
 		}
 	}, _workflowId ? { workflowId: _workflowId } : undefined);
@@ -1163,127 +1136,47 @@ async function runAgentWithProgress(
 		toolCount: _widgetExtraToolCount,
 	});
 
-	// ── Post-completion: parse subagent output for tool calls and file paths ──
-	// Progress messages from spawnSubagent rarely contain tool info,
-	// so we scan the full output after completion.
+	// ── Post-completion: file change detection via git diff ONLY ──
+	// We do NOT parse agent output text for file paths — that approach is fragile and
+	// produces false positives (e.g., matching JS property names like `.push`, `.length`
+	// from code snippets). Instead, we rely entirely on git diff --name-status, which is
+	// the same approach used by VSCode, Zed, and every other professional git client.
+	// It's deterministic, noise-free, and always produces correct relative paths.
 	const allOutput = (result.output || "") + "\n" + (result.stderr || "");
 	const finalOutput = extractFinalOutput(result.output) || result.output;
 	const searchText = allOutput + "\n" + finalOutput;
 
-	// Detect file creation/modification patterns from agent's final output text
-	// The agent's response typically lists files using markdown backticks or bullet points
-	const filePatterns = [
-		// Markdown code blocks with file paths: `src/main.rs`, `path/to/file.ts`
-		/`([^`]+\.[a-zA-Z0-9_]+)`/g,
-		// Bullet points with file operation verbs: - Modify `src/main.rs`, * Created `file.ts`
-		/(?:^|\n)\s*[-*]\s*(?:modified|created|updated|edited|added|deleted|removed|changed|wrote|writes?)\s*[`"']?([^`"'\n,]+\.[a-zA-Z0-9_]+)[`"']?/gim,
-		// Descriptive: "I've modified src/main.rs", "reading config.json"
-		/(?:modified|created|updated|edited|added|deleted|removed|changed|wrote|write|writes|read|reads?)\s+(?:the\s+)?[`"']?([^`"'\n,]+\.[a-zA-Z0-9_]+)[`"']?/gi,
-		// Chinese patterns
-		/(?:编写|创建|修改|删除|读取|写入|更新)\s*(?:了|文件)?\s*[:：]?\s*[`"']?([^`"'\s,，]+\.[a-zA-Z0-9_]+)[`"']?/gi,
-		// File path with action prefix: "edit: src/file.ts", "new: src/file.ts"
-		/(?:^|\n)\s*(?:edit|new|delete|read|modify|create|update|add|remove)\s*[:：]\s*([^\n]+\.[a-zA-Z0-9_]+)/gim,
-	];
-	const seenTools = new Set<string>();
-	for (const pattern of filePatterns) {
+	// Find output file paths (plans, reviews) for display in the "output:" section
+	// These are workflow artifacts (.pi-dev-output/) — only show files belonging to
+	// the CURRENT workflow to avoid cross-contamination from previous runs.
+	// Note: file change detection itself relies exclusively on git diff below, not text parsing.
+	const seenOutputs = new Set<string>();
+	if (_workflowId) {
+		// Only match .pi-dev-output paths containing the current workflow UUID
+		const escapedId = _workflowId.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+		const outputPattern = new RegExp(`\\.pi-dev-output\\/[^\\s,;)\\]}'"\\`]*${escapedId}[^\\s,;)\\]}'"\\`]*`, 'g');
 		let m;
-		while ((m = pattern.exec(searchText)) !== null) {
-			const filePath = m[1]!.trim()
-				.replace(/[`'"\)\(\]]+$/, "")
-				.replace(/^[`'"\)\(\[]+/, "")
-				.split(/[\s,;]/)[0]!;
-			// Validate it's a real file path
-			if (filePath.length > 3 && filePath.length < 300 && !seenTools.has(filePath)) {
-				// Skip common non-file matches
-				if (filePath.match(/^(the|a|an|this|that|it|its|my|your|our|their|some|any|all|each|every|both|few|many|several|most|other|another|such|what|which|whose|whom|when|where|why|how|who|being|having|doing|making|taking|giving|getting|setting|using|running|going|coming|looking|finding|keeping|putting)/i)) continue;
-				if (filePath.startsWith("http")) continue;
-				if (filePath.length < 6 && !filePath.includes("/")) continue;
-
-				// 额外过滤器：排除明显不是文件路径的脏数据
-				if (filePath.includes("${") || filePath.includes("\\n") || filePath.includes("\\t")) continue;  // 排除模板字符串和转义字符
-				if (filePath.includes("[]") || filePath.includes("{}")) continue;  // 排除数组/对象字面量
-				if (filePath.match(/^[\s,;)\]}]+$/)) continue;  // 排除纯符号
-				seenTools.add(filePath);
-				const fullMatch = m[0]!.toLowerCase();
-				// Determine operation type and convert to git status
-				let toolType = "edit";
-				if (fullMatch.includes("write") || fullMatch.includes("创建") || fullMatch.includes("new") || fullMatch.includes("add") || fullMatch.includes("created") || fullMatch.includes("added")) {
-					toolType = "new";
-				} else if (fullMatch.includes("delete") || fullMatch.includes("删除") || fullMatch.includes("remove") || fullMatch.includes("deleted") || fullMatch.includes("removed")) {
-					toolType = "delete";
-				} else if (fullMatch.includes("read") || fullMatch.includes("读取")) {
-					toolType = "read";
-				}
-				if (toolType !== "read") {
-					const gitStatus = toGitStatus(toolType);
-					addWidgetSubStepTool(stepIndex, agentName, `${gitStatus}   ${filePath}`);
-					_widgetExtraToolCount++;
-				}
+		while ((m = outputPattern.exec(searchText)) !== null) {
+			const path_ = m[0]!.trim();
+			if (path_.length > 15 && path_.length < 300 && !seenOutputs.has(path_)) {
+				seenOutputs.add(path_);
+				addWidgetSubStepOutput(stepIndex, agentName, path_);
 			}
 		}
-	}
-
-	// If we found no file tools from text patterns, try alternative approaches
-	// Look for explicit tool call patterns in the raw JSON output
-	if (seenTools.size === 0) {
-		const jsonLines = (result.output || "").split("\n");
-		for (const line of jsonLines) {
-			try {
-				const event = JSON.parse(line);
-				// Look for tool_use events in the JSON stream
-				if (event.type === "message_update" && event.assistantMessageEvent?.type === "tool_use") {
-					const toolName = event.assistantMessageEvent.name;
-					const args = event.assistantMessageEvent.args || {};
-					// write tool: args contains file_path
-					if (toolName === "write" && args.file_path) {
-						const fp = args.file_path.trim();
-						if (!seenTools.has(fp)) {
-							seenTools.add(fp);
-							addWidgetSubStepTool(stepIndex, agentName, `A   ${fp}`);
-							_widgetExtraToolCount++;
-						}
-					}
-					// edit tool: args contains file_path
-					if (toolName === "edit" && args.file_path) {
-						const fp = args.file_path.trim();
-						if (!seenTools.has(fp)) {
-							seenTools.add(fp);
-							addWidgetSubStepTool(stepIndex, agentName, `M   ${fp}`);
-							_widgetExtraToolCount++;
-						}
-					}
-				}
-			} catch { /* not JSON, skip */ }
-		}
-	}
-
-	// Find output file paths (.pi-dev-output, review reports, plan files)
-	// NOTE: Only match explicit paths in .pi-dev-output/ or known review/plan file patterns.
-	// The old pattern matching loose "output:" text was the root cause of "output::0.00004508" noise.
-	// File change detection now relies on git diff --name-status (updateToolsFromGit below),
-	// which is deterministic and noise-free.
-	const outputPathPatterns = [
-		// Direct reference to .pi-dev-output paths: ".pi-dev-output/pi-plans/xxx.md"
-		/\.pi-dev-output\/[a-zA-Z0-9_\/-]+\.[a-zA-Z0-9]+/g,
-		// Review file patterns: "review-20260520-162800.md"
-		/review-\d{8}-\d{6}\.md/g,
-		// Plan file patterns: "20260520-1628-*.md"
-		/\d{8}-\d{4,6}-[a-zA-Z0-9_-]+\.md/g,
-	];
-	const seenOutputs = new Set<string>();
-	for (const pattern of outputPathPatterns) {
+	} else {
+		// No workflow ID (unlikely): match .pi-dev-output paths conservatively
+		const outputPattern = /\.pi-dev-output\/[a-zA-Z0-9_\/-]+\.[a-zA-Z0-9]+/g;
 		let m;
-		while ((m = pattern.exec(searchText)) !== null) {
+		while ((m = outputPattern.exec(searchText)) !== null) {
 			const path_ = m[0]!.trim();
-			if (path_.length > 10 && path_.length < 300 && !seenOutputs.has(path_)) {
+			if (path_.length > 15 && path_.length < 300 && !seenOutputs.has(path_)) {
 				seenOutputs.add(path_);
 				addWidgetSubStepOutput(stepIndex, agentName, path_);
 			}
 		}
 	}
 
-
-	// ── Update file changes from git diff (more accurate than text scraping) ──
+	// ── Update file changes from git diff (more accurate than text scraping) ──	
 	updateToolsFromGit(_workflowCwd, stepIndex, agentName);
 	// Update sub-step status based on result
 	const subStatus: WorkflowSubStepWidgetState["status"] =
@@ -1350,41 +1243,19 @@ async function executeSingleStep(
 		throw new Error(`Agent 错误 (exit ${result.exitCode}): ${result.stderr.slice(0, 500)}`);
 	}
 
-	// ── Capture chain context for single-step agents ──
-	const agentChanges = _workflowFileChanges
-		.filter(c => c.stepIndex === stepIndex && c.agent === agentName)
-		.map(c => `${c.type === "new" ? "A" : c.type === "delete" ? "D" : "M"}   ${c.filePath}`);
-
-	let chainKey: string;
-	if (agentName === "planner") chainKey = "计划制定摘要";
-	else if (agentName === "docWriter") chainKey = "文档更新摘要";
-	else chainKey = `${agentName} 执行摘要`;
-
-	if (agentChanges.length > 0) {
-		const editCount = agentChanges.filter(c => c.startsWith("M")).length;
-		const newCount = agentChanges.filter(c => c.startsWith("A")).length;
-		const delCount = agentChanges.filter(c => c.startsWith("D")).length;
-		const statsParts: string[] = [];
-		if (editCount > 0) statsParts.push(`修改 ${editCount} 个`);
-		if (newCount > 0) statsParts.push(`新增 ${newCount} 个`);
-		if (delCount > 0) statsParts.push(`删除 ${delCount} 个`);
-		const statsLine = statsParts.length > 0 ? `改动统计: ${statsParts.join("，")}\n\n` : "";
-
-		updateChainContext(chainKey,
-			`${agentName} 已完成。\n` +
-			statsLine +
-			`变更文件列表:\n${agentChanges.join("\n")}`
-		);
-	} else {
-		updateChainContext(chainKey, `${agentName} 已完成执行，未检测到文件变更。`);
-	}
-
 	// ── Capture AI work summary as supplementary chain context ──
+	// We intentionally do NOT create chain contexts from _workflowFileChanges here.
+	// File-change-based chain contexts ("计划制定摘要"/"文档更新摘要") were removed in v0.6.0
+	// because their M/A/D counts provided negligible value for downstream agents compared
+	// to the agent's own free-text work summary. The file-change approach also created
+	// tight coupling between executeSingleStep and the _workflowFileChanges data structure.
 	const workSummary = extractFinalOutput(result.output);
 	if (workSummary) {
-		updateChainContext(`${agentName} 工作总结`,
-			`${agentName} 已完成工作，以下是其工作总结：\n\n${workSummary}`
-		);
+		updateChainContext(`${agentName} 工作总结`, `${agentName} 已完成工作，以下是其工作总结：\n\n${workSummary}`);
+	} else if (result.output) {
+		// 保底：使用输出前 200 字符作为简略摘要
+		const fallback = result.output.slice(0, 200).trim();
+		if (fallback) updateChainContext(`${agentName} 工作总结`, fallback);
 	}
 }
 
@@ -1448,37 +1319,6 @@ async function executeLoopGroup(
             }
         }
 
-		// ── After loop agent completes: capture changes + stats for chain context ──
-		const loopAgentChanges = _workflowFileChanges
-			.filter(c => c.stepIndex === stepIndex && c.agent === step.loopAgentName)
-			.map(c => `${c.type === "new" ? "A" : c.type === "delete" ? "D" : "M"}   ${c.filePath}`);
-
-		// Use agent-specific keys so worker and trimmer changes coexist without overwriting
-		const chainKey = step.loopAgentName === "worker" ? "代码实施摘要" : "代码精简摘要";
-
-		if (loopAgentChanges.length > 0) {
-			const editCount = loopAgentChanges.filter(c => c.startsWith("M")).length;
-			const newCount = loopAgentChanges.filter(c => c.startsWith("A")).length;
-			const delCount = loopAgentChanges.filter(c => c.startsWith("D")).length;
-			const statsParts: string[] = [];
-			if (editCount > 0) statsParts.push(`修改 ${editCount} 个`);
-			if (newCount > 0) statsParts.push(`新增 ${newCount} 个`);
-			if (delCount > 0) statsParts.push(`删除 ${delCount} 个`);
-			const statsLine = statsParts.length > 0 ? `改动统计: ${statsParts.join("，")}\n\n` : "";
-
-			updateChainContext(chainKey,
-				`${step.loopAgentName === "worker" ? "代码实施" : "代码精简"}已完成。\n` +
-				statsLine +
-				`变更文件列表:\n${loopAgentChanges.join("\n")}` +
-				(planFileRelPath ? `\n\n实施计划: ${planFileRelPath}\n(可在 .pi-dev-output/pi-plans/ 中 grep UUID ${_workflowId} 找到)` : "")
-			);
-		} else {
-			updateChainContext(chainKey,
-				`${step.loopAgentName} 已完成执行，未检测到文件变更。\n` +
-				(planFileRelPath ? `实施计划: ${planFileRelPath}\n(可在 .pi-dev-output/pi-plans/ 中 grep UUID ${_workflowId} 找到)` : "")
-			);
-		}
-
 		// ── Capture loop agent work summary as supplementary chain context ──
 		const loopFinalText = extractFinalOutput(agentResult.output);
 		if (loopFinalText) {
@@ -1525,25 +1365,11 @@ async function executeLoopGroup(
 			}
 		}
 
-		// ── After reviewer: capture review context for next loop ──
-		// Use agent-specific key so worker-reviewer and trimmer-reviewer feedback don't interfere
-		const reviewChainKey = step.loopAgentName === "worker" ? "代码审查反馈" : "精简审查反馈";
-		if (reviewSummary) {
-			const reviewCountParts: string[] = [];
-			if (reviewSummary.critical > 0) reviewCountParts.push(`严重 ${reviewSummary.critical} 个`);
-			if (reviewSummary.medium > 0) reviewCountParts.push(`中等 ${reviewSummary.medium} 个`);
-			if (reviewSummary.low > 0) reviewCountParts.push(`低 ${reviewSummary.low} 个`);
-			const reviewStats = reviewCountParts.length > 0 ? `发现 ${reviewCountParts.join("，")} 问题。` : "未发现问题。";
-
-			updateChainContext(reviewChainKey,
-				`${reviewStats}\n` +
-				`完整审查报告在 .pi-dev-output/pi-review/md/ 目录中，\n` +
-				`请在工作流输出目录中 grep UUID "${_workflowId}" 查找最新报告。`
-			);
-		}
-
 		// ── Capture reviewer work summary as supplementary chain context ──
-		// Reuse already-parsed extractedOutput to avoid double parsing
+		// The old review-chain-context entries ("代码审查反馈"/"精简审查反馈") based on
+		// _workflowFileChanges were removed in v0.6.0 because they duplicated information
+		// from the review report file and polluted chain context with file-path-heavy data.
+		// Reuse already-parsed extractedOutput to avoid double parsing.
 		if (extractedOutput) {
 			updateChainContext(`审查工作总结`,
 				`审查者已完成审查，以下是其审查总结：\n\n${extractedOutput}`

package/package.json

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