mcp-probe-kit 3.6.10 → 3.6.11

package/build/tools/__tests__/fix_bug.unit.test.js

@@ -1,31 +1,68 @@
 import { describe, expect, test } from "vitest";
+import * as fs from "node:fs";
+import * as os from "node:os";
+import * as path from "node:path";
 import { fixBug } from "../fix_bug.js";
 describe("fix_bug 单元测试", () => {
-    test("返回 TBP8 结构化输出", async () => {
+    test("返回 SRC-8 delegated plan、真因工作表与门禁", async () => {
         const result = await fixBug({
             error_message: "登录提交后页面白屏",
-            stack_trace: "TypeError: Cannot read property 'token' of undefined at src/auth/login.ts:45:12",
-            expected_behavior: "提示错误并保持页面可交互",
-            actual_behavior: "页面白屏，无法继续操作",
+            stack_trace: "TypeError at src/auth/login.ts:45:12",
+            expected_behavior: "提示错误",
+            actual_behavior: "白屏",
+            success_sample: "错误密码时表单正常提示",
+            verification_target: "原复现步骤通过且 auth 单测绿",
         });
         expect(result.isError).toBe(false);
-        expect("structuredContent" in result).toBe(true);
         const structured = result.structuredContent;
-        expect(structured.analysisMode).toBe("tbp8");
-        expect(structured.tbp.phenomenon).toMatch(/页面白屏|登录提交/);
-        expect(Array.isArray(structured.tbp.timeline)).toBe(true);
-        expect(structured.tbp.timeline.length).toBeGreaterThan(0);
-        expect(Array.isArray(structured.tbp.repair)).toBe(true);
-        expect(structured.affectedFiles).toContain("src/auth/login.ts");
+        const text = String(result.content[0].text);
+        expect(structured.mode).toBe("delegated");
+        expect(structured.methodology).toBe("src8");
+        expect(structured.metadata.plan.mode).toBe("delegated");
+        expect(structured.metadata.plan.steps.map((s) => s.id)).toEqual([
+            "src8-1",
+            "src8-2",
+            "src8-3",
+            "src8-4",
+            "src8-5",
+            "src8-6",
+            "src8-7",
+            "src8-8",
+        ]);
+        expect(structured.src8Checklist).toHaveLength(8);
+        expect(structured.rootCauseWorksheet.substeps).toHaveLength(5);
+        expect(structured.src8Gate.blockCodeChangeUntilStep).toBe(4);
+        expect(structured.attributionLayers.some((l) => l.id === "agent_behavior")).toBe(true);
+        expect(text).toMatch(/SRC-8/);
+        expect(text).not.toMatch(/方法论渊源/);
+        expect(text).not.toMatch(/docs\/src8-methodology/);
+        expect(text).toMatch(/rootCauseAnalysis|真因工作表/);
+        expect(text).toMatch(/metadata\.plan/);
+        expect(structured.bugfixInput.hasVerificationTarget).toBe(true);
     });
-    test("支持 code_context 并进入证据链", async () => {
+    test("支持 code_context 进入证据链", async () => {
         const result = await fixBug({
-            error_message: "接口返回成功但前端没生效",
-            code_context: "图谱摘要: auth flow\n图谱线索: query: src/modules/record/index.js",
+            error_message: "接口成功但前端未生效",
+            code_context: "auth flow summary",
         });
-        expect(result.isError).toBe(false);
         const structured = result.structuredContent;
-        expect(structured.tbp.evidence.some((item) => item.source === "code_context")).toBe(true);
-        expect(structured.fixPlan.steps[0]).toMatch(/TBP-1/);
+        expect(structured.bugfixInput.evidence.some((item) => item.source === "code_context")).toBe(true);
+    });
+    test("file_path 可读时注入代码上下文", async () => {
+        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "fix-bug-"));
+        const filePath = path.join(dir, "handler.ts");
+        fs.writeFileSync(filePath, "throw new Error('boom');\n", "utf-8");
+        try {
+            const result = await fixBug({
+                error_message: "handler 抛错",
+                file_path: filePath,
+                project_root: dir,
+            });
+            const structured = result.structuredContent;
+            expect(structured.bugfixInput.code_context).toContain("boom");
+        }
+        finally {
+            fs.rmSync(dir, { recursive: true, force: true });
+        }
     });
 });

package/build/tools/__tests__/gentest.unit.test.d.ts

@@ -0,0 +1 @@
+export {};

package/build/tools/__tests__/gentest.unit.test.js

@@ -0,0 +1,40 @@
+import { describe, expect, test } from "vitest";
+import * as fs from "node:fs";
+import * as os from "node:os";
+import * as path from "node:path";
+import { gentest } from "../gentest.js";
+describe("gentest 单元测试", () => {
+    test("传入 code 时注入待测内容并标明 guidance 模式", async () => {
+        const sample = "export function add(a: number, b: number) { return a + b; }\n";
+        const result = await gentest({ code: sample, framework: "vitest" });
+        expect(result.isError).toBeFalsy();
+        const text = String(result.content[0].text);
+        expect(text).toContain("return a + b");
+        expect(text).toMatch(/指南型|guidance/i);
+        const structured = result.structuredContent;
+        expect(structured.mode).toBe("guidance");
+        expect(structured.gentestInput.received).toBe(true);
+        expect(structured.gentestInput.framework).toBe("vitest");
+        expect(structured.gentestInput.code).toContain("add");
+    });
+    test("未传 code/file_path 时明确提示 Agent 需先获取代码", async () => {
+        const result = await gentest({ framework: "jest" });
+        const text = String(result.content[0].text);
+        expect(text).toMatch(/未提供 code/);
+        expect(result.structuredContent.gentestInput.received).toBe(false);
+    });
+    test("file_path 可读时注入文件内容", async () => {
+        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "gentest-"));
+        const filePath = path.join(dir, "math.ts");
+        fs.writeFileSync(filePath, "export const mul = (a: number, b: number) => a * b;\n", "utf-8");
+        try {
+            const result = await gentest({ file_path: filePath, project_root: dir });
+            const text = String(result.content[0].text);
+            expect(text).toContain("a * b");
+            expect(result.structuredContent.gentestInput.file).toBe(filePath);
+        }
+        finally {
+            fs.rmSync(dir, { recursive: true, force: true });
+        }
+    });
+});

package/build/tools/__tests__/refactor.unit.test.d.ts

@@ -0,0 +1 @@
+export {};

package/build/tools/__tests__/refactor.unit.test.js

@@ -0,0 +1,40 @@
+import { describe, expect, test } from "vitest";
+import * as fs from "node:fs";
+import * as os from "node:os";
+import * as path from "node:path";
+import { refactor } from "../refactor.js";
+describe("refactor 单元测试", () => {
+    test("传入 code 时注入待重构内容并标明 guidance 模式", async () => {
+        const sample = "function f(x){if(x){if(x>0){return x*2}}return 0}\n";
+        const result = await refactor({ code: sample, goal: "reduce_complexity" });
+        expect(result.isError).toBeFalsy();
+        const text = String(result.content[0].text);
+        expect(text).toContain("return x*2");
+        expect(text).toMatch(/指南型|guidance/i);
+        const structured = result.structuredContent;
+        expect(structured.mode).toBe("guidance");
+        expect(structured.refactorInput.received).toBe(true);
+        expect(structured.refactorInput.goal).toBe("reduce_complexity");
+        expect(structured.refactorInput.code).toContain("return x*2");
+    });
+    test("未传 code/file_path 时明确提示 Agent 需先获取代码", async () => {
+        const result = await refactor({ goal: "improve_readability" });
+        const text = String(result.content[0].text);
+        expect(text).toMatch(/未提供 code/);
+        expect(result.structuredContent.refactorInput.received).toBe(false);
+    });
+    test("file_path 可读时注入文件内容", async () => {
+        const dir = fs.mkdtempSync(path.join(os.tmpdir(), "refactor-"));
+        const filePath = path.join(dir, "legacy.js");
+        fs.writeFileSync(filePath, "const MAGIC = 42;\n", "utf-8");
+        try {
+            const result = await refactor({ file_path: filePath, project_root: dir });
+            const text = String(result.content[0].text);
+            expect(text).toContain("MAGIC = 42");
+            expect(result.structuredContent.refactorInput.file).toBe(filePath);
+        }
+        finally {
+            fs.rmSync(dir, { recursive: true, force: true });
+        }
+    });
+});

package/build/tools/__tests__/start_bugfix.unit.test.js

@@ -21,6 +21,18 @@ describe('start_bugfix 单元测试', () => {
         expect(result.isError).toBe(true);
         expect(result.content[0].text).toMatch(/缺少必填参数|错误信息/i);
     });
+    test('description 可作为 error_message 别名', async () => {
+        const result = await startBugfix({
+            description: 'memory-config 读取 items 时未做空值检查导致 map 报错',
+            analysis_mode: 'src8',
+        });
+        expect(result.isError).toBe(false);
+        const structured = result.structuredContent;
+        const src8Step = structured?.metadata?.plan?.steps?.find((step) => step.id === 'src8-1');
+        expect(src8Step?.action).toMatch(/明确差距/);
+        expect(result.content[0].text).toMatch(/memory-config/);
+        expect(result.content[0].text).not.toMatch(/docs\/src8-methodology/);
+    });
     test('返回委托式执行计划（steps）', async () => {
         const result = await startBugfix({
             error_message: 'TypeError: Cannot read property',
@@ -36,12 +48,13 @@ describe('start_bugfix 单元测试', () => {
         expect(plan).toBeTruthy();
         expect(plan.mode).toBe('delegated');
         expect(Array.isArray(plan.steps)).toBe(true);
-        expect(structured.analysisMode).toBe('tbp8');
+        expect(structured.analysisMode).toBe('src8');
         expect(structured.tbp.rootCauseStatement).toMatch(/A \+ B|因果句|待形成/);
-        const fixStep = plan.steps.find((step) => step.tool === 'fix_bug');
+        const fixStep = plan.steps.find((step) => step.id === 'src8-4');
         expect(fixStep).toBeTruthy();
-        expect(fixStep.args.error_message).toBe('TypeError: Cannot read property');
-        expect(fixStep.args.analysis_mode).toBe('tbp8');
+        expect(fixStep.action).toMatch(/把握真因/);
+        const gentestStep = plan.steps.find((step) => step.id === 'src8-7');
+        expect(gentestStep?.tool).toBe('gentest');
         const contextStep = plan.steps.find((step) => step.tool === 'init_project_context');
         expect(contextStep.outputs).toContain('docs/graph-insights/latest.md');
         expect(contextStep.outputs).toContain('docs/graph-insights/latest.json');
@@ -90,20 +103,20 @@ describe('start_bugfix 单元测试', () => {
     });
     test('template_profile 自动选择 strict（结构化输入）', async () => {
         const result = await startBugfix({
-            error_message: `## 复现步骤
-1. 打开登录页
-2. 输入错误账号
-3. 点击登录
-
-## 期望
-提示错误信息并保持页面可交互
-
-## 实际
-页面白屏，控制台报错
-
-## 环境
-- 浏览器: Chrome 120
-- 系统: Windows 11
+            error_message: `## 复现步骤
+1. 打开登录页
+2. 输入错误账号
+3. 点击登录
+
+## 期望
+提示错误信息并保持页面可交互
+
+## 实际
+页面白屏，控制台报错
+
+## 环境
+- 浏览器: Chrome 120
+- 系统: Windows 11
 - 版本: v2.3.0`,
             stack_trace: 'TypeError: Cannot read property',
             template_profile: 'auto',

package/build/tools/code_review.js

@@ -1,187 +1,172 @@
 import { parseArgs, getString } from "../utils/parseArgs.js";
-import { okText } from "../lib/response.js";
+import { okStructured } from "../lib/response.js";
 import { renderGuidanceHeader } from "../lib/guidance.js";
 import { handleToolError } from "../utils/error-handler.js";
 import { renderCodeLimits, renderBannedPatterns, CODE_LIMITS } from "../lib/quality-constraints.js";
-// code_review 工具实现
+import { resolveReviewCode, trimReviewCodeForPrompt } from "../lib/code-review-input.js";
+import { resolveWorkspaceRoot } from "../lib/workspace-root.js";
 export async function codeReview(args) {
     try {
-        // 智能参数解析，支持自然语言输入
         const parsedArgs = parseArgs(args, {
             defaultValues: {
                 code: "",
                 focus: "all",
+                file_path: "",
+                project_root: "",
             },
-            primaryField: "code", // 纯文本输入默认映射到 code 字段
+            primaryField: "code",
             fieldAliases: {
-                code: ["source", "src", "代码", "content"],
+                code: ["source", "src", "代码", "content", "diff"],
                 focus: ["type", "category", "类型", "重点"],
+                file_path: ["filePath", "filepath", "path", "文件路径"],
+                project_root: ["projectRoot", "project_path", "dir", "directory", "项目路径"],
             },
         });
-        const code = getString(parsedArgs.code);
-        const focus = getString(parsedArgs.focus) || "all"; // quality, security, performance, all
+        const focus = getString(parsedArgs.focus) || "all";
+        const inlineCode = getString(parsedArgs.code) || getString(parsedArgs.input);
+        const filePath = getString(parsedArgs.file_path);
+        const projectRoot = getString(parsedArgs.project_root);
+        const resolved = resolveReviewCode({
+            code: inlineCode,
+            filePath: filePath || undefined,
+            projectRoot: projectRoot ? resolveWorkspaceRoot(projectRoot) : undefined,
+        });
+        if (resolved.error) {
+            return okStructured(`❌ code_review 无法读取输入: ${resolved.error}`, {
+                mode: "guidance",
+                reviewInput: {
+                    received: false,
+                    error: resolved.error,
+                    file: filePath || null,
+                },
+            }, {
+                note: "指南型工具：请修正 file_path / project_root 或传入 code 后，由 Agent 按清单完成审查并输出 issues JSON",
+            });
+        }
+        const hasCode = Boolean(resolved.code.trim());
+        const promptCode = hasCode ? trimReviewCodeForPrompt(resolved.code) : "";
         const header = renderGuidanceHeader({
             tool: "code_review",
-            goal: "对代码进行全面审查，发现问题并提供改进建议。",
-            tasks: ["基于代码进行质量/安全/性能检查", "列出发现的问题和建议"],
-            outputs: ["审查报告（包含问题清单、优点、建议）"],
+            goal: "由 Agent 根据下方代码与审查清单完成审查，并输出结构化问题清单。",
+            tasks: [
+                "先阅读本次注入的 reviewInput.code（或 file_path 对应文件）",
+                "按审查清单逐项检查，不要只复述清单",
+                "在回复中输出 issues JSON（severity/category/message/suggestion）",
+            ],
+            outputs: ["审查报告（问题清单、优点、建议）— 由 Agent 生成，非 MCP 静态扫描"],
+            notes: [
+                "本工具为指南型（guidance-only），不在服务端做静态规则扫描",
+                hasCode ? `已注入待审代码（${resolved.code.split("\n").length} 行）` : "未收到 code/file_path，请先提供待审内容",
+            ],
         });
-        const message = `${header}请对以下代码进行全面审查：
-
-📝 **代码内容**：
-${code || "请提供需要审查的代码"}
-
-🎯 **审查重点**：${focus}
-
----
-
-## 代码审查清单
-
-### 1️⃣ 代码质量检查
-
-${renderCodeLimits()}
-
-**代码坏味道（Code Smells）**：
-- [ ] 重复代码（Duplicated Code）
-- [ ] 过长函数（Long Function）> ${CODE_LIMITS.maxFunctionLines} 行
-- [ ] 过长参数列表（Long Parameter List）> ${CODE_LIMITS.maxParameters} 个
-- [ ] 复杂条件判断（Complex Conditional）> ${CODE_LIMITS.maxNestingDepth} 层嵌套
-- [ ] 魔法数字（Magic Numbers）
-- [ ] 命名不清晰（Poor Naming）
-
-**设计原则**：
-- [ ] 单一职责原则（SRP）
-- [ ] 开闭原则（OCP）
-- [ ] 接口隔离原则（ISP）
-- [ ] 依赖倒置原则（DIP）
-
-### 2️⃣ 安全漏洞检查
-
-**常见漏洞**：
-- [ ] SQL 注入风险
-- [ ] XSS（跨站脚本）风险
-- [ ] CSRF（跨站请求伪造）
-- [ ] 硬编码密钥/密码
-- [ ] 不安全的随机数生成
-- [ ] 路径遍历漏洞
-- [ ] 未验证的输入
-- [ ] 敏感信息泄露
-
-**安全最佳实践**：
-- [ ] 输入验证和过滤
-- [ ] 输出编码
-- [ ] 使用参数化查询
-- [ ] 密码/密钥使用环境变量
-- [ ] HTTPS 通信
-
-### 3️⃣ 性能问题检查
-
-**性能风险**：
-- [ ] 循环内创建对象
-- [ ] 嵌套循环（O(n²) 或更差）
-- [ ] 不必要的重复计算
-- [ ] 内存泄漏风险
-- [ ] 阻塞主线程
-- [ ] 大数据量未分页
-- [ ] 同步 I/O 操作
-
-**React/Vue 性能**：
-- [ ] 未使用 useMemo/useCallback
-- [ ] 组件不必要的重渲染
-- [ ] 大列表未虚拟化
-- [ ] 状态管理不当
-
-### 4️⃣ 完整性检查
-
-${renderBannedPatterns()}
-
-### 5️⃣ 最佳实践检查
-
-**TypeScript/JavaScript**：
-- [ ] 类型定义完整（避免 any）
-- [ ] 错误处理完善（try-catch）
-- [ ] 异步操作正确处理
-- [ ] 使用 const/let 替代 var
-- [ ] 箭头函数合理使用
-
-**命名规范**：
-- [ ] 变量：驼峰命名（camelCase）
-- [ ] 常量：大写下划线（UPPER_CASE）
-- [ ] 类/接口：帕斯卡命名（PascalCase）
-- [ ] 文件：短横线命名（kebab-case）
-- [ ] 布尔值：is/has/should 前缀
-
-**注释和文档**：
-- [ ] 复杂逻辑有注释说明
-- [ ] 公共 API 有文档
-- [ ] TODO/FIXME 标记清晰
-
----
-
-## 审查报告格式
-
-**严重问题（🔴 Critical）**：
-1. [位置] 问题描述
-   - 风险：...
-   - 建议：...
-   - 修复示例：\`\`\`typescript ... \`\`\`
-
-**警告（🟡 Warning）**：
-1. [位置] 问题描述
-   - 影响：...
-   - 建议：...
-
-**建议（🟢 Suggestion）**：
-1. [位置] 改进建议
-   - 当前：...
-   - 建议：...
-   - 收益：...
-
-**优点（✅ Good）**：
-- 做得好的地方
-
----
-
----
-
-## 📤 输出格式要求
-
-请严格按以下 JSON 格式输出审查结果：
-
-\`\`\`json
-{
-  "summary": "代码整体评价（一句话）",
-  "score": 85,
-  "issues": [
-    {
-      "severity": "critical|high|medium|low",
-      "category": "quality|security|performance|style",
-      "file": "文件路径（如有）",
-      "line": 10,
-      "code": "问题代码片段",
-      "message": "问题描述",
-      "suggestion": "修复建议",
-      "fix_example": "修复示例代码"
-    }
-  ],
-  "highlights": ["做得好的地方1", "做得好的地方2"]
-}
-\`\`\`
-
-## ⚠️ 边界约束
-
-- ❌ 仅分析，不自动修改源代码
-- ❌ 不执行代码或 shell 命令
-- ❌ 不做业务逻辑正确性判断（只关注代码质量）
-- ✅ 输出结构化问题清单和改进建议
-
-现在请开始代码审查，生成详细的审查报告。`;
-        return okText(message, {
-            schema: (await import('../schemas/output/core-tools.js')).CodeReviewReportSchema,
-            note: "本工具返回代码审查指南，AI 应根据指南分析代码并输出审查报告"
+        const message = `${header}请对以下代码进行全面审查：
+
+📝 **待审代码**${resolved.file ? `（来源: ${resolved.file}）` : ""}：
+${hasCode ? `\`\`\`\n${promptCode}\n\`\`\`` : "_未提供 code / file_path，请 Agent 先读取目标文件或让用户补充后再审查_"}
+
+🎯 **审查重点**：${focus}
+
+---
+
+## 代码审查清单
+
+### 1️⃣ 代码质量检查
+
+${renderCodeLimits()}
+
+**代码坏味道（Code Smells）**：
+- [ ] 重复代码（Duplicated Code）
+- [ ] 过长函数（Long Function）> ${CODE_LIMITS.maxFunctionLines} 行
+- [ ] 过长参数列表（Long Parameter List）> ${CODE_LIMITS.maxParameters} 个
+- [ ] 复杂条件判断（Complex Conditional）> ${CODE_LIMITS.maxNestingDepth} 层嵌套
+- [ ] 魔法数字（Magic Numbers）
+- [ ] 命名不清晰（Poor Naming）
+
+**设计原则**：
+- [ ] 单一职责原则（SRP）
+- [ ] 开闭原则（OCP）
+- [ ] 接口隔离原则（ISP）
+- [ ] 依赖倒置原则（DIP）
+
+### 2️⃣ 安全漏洞检查
+
+**常见漏洞**：
+- [ ] SQL 注入风险
+- [ ] XSS（跨站脚本）风险
+- [ ] CSRF（跨站请求伪造）
+- [ ] 硬编码密钥/密码
+- [ ] 不安全的随机数生成
+- [ ] 路径遍历漏洞
+- [ ] 未验证的输入
+- [ ] 敏感信息泄露
+
+### 3️⃣ 性能问题检查
+
+**性能风险**：
+- [ ] 循环内创建对象
+- [ ] 嵌套循环（O(n²) 或更差）
+- [ ] 不必要的重复计算
+- [ ] 内存泄漏风险
+- [ ] 阻塞主线程
+- [ ] 大数据量未分页
+- [ ] 同步 I/O 操作
+
+### 4️⃣ 完整性检查
+
+${renderBannedPatterns()}
+
+### 5️⃣ 最佳实践检查
+
+**TypeScript/JavaScript**：
+- [ ] 类型定义完整（避免 any）
+- [ ] 错误处理完善（try-catch）
+- [ ] 异步操作正确处理
+
+---
+
+## 📤 Agent 必须输出的 JSON 格式
+
+\`\`\`json
+{
+  "summary": "代码整体评价（一句话）",
+  "overallScore": 85,
+  "issues": [
+    {
+      "severity": "critical|high|medium|low",
+      "category": "quality|security|performance|style",
+      "file": "文件路径（如有）",
+      "line": 10,
+      "code": "问题代码片段",
+      "message": "问题描述",
+      "suggestion": "修复建议"
+    }
+  ],
+  "strengths": ["做得好的地方"]
+}
+\`\`\`
+
+## ⚠️ 边界
+
+- MCP 仅返回指南 + 注入的待审代码，**issues 由 Agent 审查后生成**
+- 不要声称「工具已扫描完成」；应基于上方代码与清单给出真实发现
+
+现在请开始审查并输出问题清单。`;
+        return okStructured(message, {
+            mode: "guidance",
+            reviewInput: {
+                received: hasCode,
+                focus,
+                file: resolved.file ?? null,
+                lineCount: hasCode ? resolved.code.split("\n").length : 0,
+                code: hasCode ? resolved.code : null,
+                truncatedInPrompt: hasCode && resolved.code.length !== promptCode.length,
+            },
+        }, {
+            schema: (await import("../schemas/output/core-tools.js")).CodeReviewReportSchema,
+            note: "指南型工具：issues 须由 Agent 按清单审查后生成；MCP 不返回静态扫描结果",
         });
     }
     catch (error) {
-        return handleToolError(error, 'code_review');
+        return handleToolError(error, "code_review");
     }
 }

package/build/tools/fix_bug.d.ts

@@ -1,13 +1,5 @@
 /**
- * fix_bug 工具
- *
- * 功能：基于 TBP 8 步法输出真因分析与修复指南
- * 模式：指令生成器模式 - 返回详细的 RCA 与修复指南，由 AI 执行实际操作
- *
- * 流程：定义现象 → 复盘时间线 → 排除错误方向 → 找共同模式 → 定位边界 → 陈述真因 → 闭合证据链 → 设计修复方案
- */
-/**
- * fix_bug 工具实现
+ * fix_bug 工具 — SRC-8 delegated plan + 真因工作表
  */
 export declare function fixBug(args: any): Promise<import("../lib/response.js").ToolResponse | {
     content: Array<{