jinzd-ai-cli 0.1.49 → 0.1.51

package/CLAUDE.md

@@ -350,6 +350,55 @@ const stdinLines = Array.isArray(rawStdin) ? rawStdin.map(String)
 - [x] **web_fetch DNS 解析时 SSRF 防护**（v0.1.25）：新增 `resolveAndCheck()` 函数，用 `dns.promises.lookup()` 预解析域名，检查结果 IP 是否为私有地址。初始 URL 和 redirect 目标均校验。
 - [ ] **`persistentCwd` 全局状态**：bash 工具的当前工作目录是模块级全局变量，多 session 并发时可能串扰。现阶段单 session REPL 无影响，GUI 多会话扩展时需重构为 per-session 状态。
 
+## 本轮开发完成记录（2026-03-08，v0.1.49 → v0.1.50）
+
+### 代码质量：L1 低危修复 — run-tests.ts package.json 细粒度错误处理
+
+**问题**：`detectProject()` 中 `JSON.parse(readFileSync('package.json'))` 错误处理粗糙——文件读取和 JSON 解析混在同一个 catch 中；解析失败时静默 fallthrough 可能误判为 Python/Go 项目；无 test script 时不尝试检测已安装的测试框架。
+
+**修复**（`src/tools/builtin/run-tests.ts`）：
+
+新增 `safeReadPackageJson(cwd)` 辅助函数——细粒度四层处理：
+1. **文件读取错误**：区分 `EACCES`/`EPERM`（权限问题）和其他读取错误，分别给出不同提示
+2. **UTF-8 BOM 处理**：自动剥离 BOM（Windows Notepad 等编辑器常见问题），防止 `JSON.parse` 失败
+3. **空文件检测**：`raw.trim() === ''` 时给出明确 "package.json is empty" 提示
+4. **JSON 解析错误细分**：区分语法错误（`Unexpected token`）、截断文件（`Unexpected end`）、根不是对象（数组或其他类型）
+
+新增 `detectNodeTestFramework(cwd, pkg)` 辅助函数——当 package.json 有效但无 `scripts.test` 时：
+- 从 `devDependencies` + `dependencies` + 配置文件检测 5 种测试框架：
+  - **Vitest**：`vitest` 依赖 或 `vitest.config.{ts,js,mts}`
+  - **Jest**：`jest` 依赖 或 `jest.config.{js,ts,mjs}`
+  - **Mocha**：`mocha` 依赖 或 `.mocharc.{yml,yaml,json,js}`
+  - **Ava**：`ava` 依赖
+  - **Playwright**：`@playwright/test` 依赖
+- 检测到框架时直接使用 `npx <framework>` 命令，无需 `scripts.test`
+
+更新 filter 参数处理——不同 Node 测试框架使用正确的 filter 语法：
+- vitest/jest → `-t "filter"`
+- mocha/playwright → `--grep "filter"`
+- npm test → `-- --grep "filter"`（passthrough）
+
+**改善效果**：
+- package.json 格式错误时不再静默 fallthrough，返回 `npm (package.json error)` 仍识别为 Node 项目
+- 安装了 vitest/jest/mocha 但未配置 `scripts.test` 的项目现在能自动检测并运行测试
+- 错误信息更具体、更有指导性
+
+### 版本与收尾
+- `src/core/constants.ts`：VERSION `0.1.49` → `0.1.50`
+- `package.json`：version 同步
+- 构建验证：`npm run build` 零错误（ESM + CJS 双产物）
+- 代码审查报告 L1 条目标记为 ✅ 已修复
+
+### 本轮变更文件汇总
+
+| 文件 | 变更类型 | 说明 |
+|------|---------|------|
+| `src/tools/builtin/run-tests.ts` | 修改 | safeReadPackageJson + detectNodeTestFramework + filter 语法更新 |
+| `src/core/constants.ts` | 修改 | VERSION 0.1.49 → 0.1.50 |
+| `package.json` | 修改 | version 0.1.49 → 0.1.50 |
+
+---
+
 ## 本轮开发完成记录（2026-03-08，v0.1.47 → v0.1.48）
 
 ### Tier 2 体验增强：/undo 增强 + /fork 对话分支
@@ -415,7 +464,7 @@ const stdinLines = Array.isArray(rawStdin) ? rawStdin.map(String)
 ### 下一步建议
 
 #### Tier 2 — 体验增强（剩余）
-1. **L1 低危**：`run-tests.ts` 的 `JSON.parse(package.json)` 细粒度错误处理
+1. ~~**L1 低危**~~：✅ 已在 v0.1.50 修复（safeReadPackageJson + detectNodeTestFramework）
 2. **IDE 集成**：VS Code 扩展（架构已准备就绪，core/providers/tools 无终端依赖）
 3. **OAuth/浏览器登录**：无需手动填 API Key，打开浏览器完成 OAuth 流程自动保存 token
 
@@ -608,7 +657,7 @@ const stdinLines = Array.isArray(rawStdin) ? rawStdin.map(String)
 3. **`/diff` 命令**：显示当前 session 内所有文件修改的汇总 diff，便于 AI 操作后快速审查变更
 
 #### Tier 2 — 体验增强
-4. **L1 低危**：`run-tests.ts` 的 `JSON.parse(package.json)` 细粒度错误处理
+4. ~~**L1 低危**~~：✅ 已在 v0.1.50 修复（safeReadPackageJson + detectNodeTestFramework）
 5. **IDE 集成**：VS Code 扩展（架构已准备就绪，core/providers/tools 无终端依赖）
 6. **OAuth/浏览器登录**：无需手动填 API Key，打开浏览器完成 OAuth 流程自动保存 token
 7. ~~**`/undo` 增强**~~：✅ 已在 v0.1.48 实现（bash 文件追踪 + /undo list + /undo <n>）
@@ -687,7 +736,7 @@ const stdinLines = Array.isArray(rawStdin) ? rawStdin.map(String)
 ### 下一步建议
 1. ~~**Extended Thinking**~~：✅ 已在 v0.1.38 实现
 2. ~~**theme 迁移扩展**~~：✅ 已在 v0.1.38 全覆盖迁移
-3. **L1 低危**：`run-tests.ts` 的 `JSON.parse(package.json)` 细粒度错误处理
+3. ~~**L1 低危**~~：✅ 已在 v0.1.50 修复
 4. **IDE 集成**：VS Code 扩展（架构已准备就绪）
 5. **OAuth/浏览器登录**：无需手动填 API Key
 
@@ -777,7 +826,7 @@ const stdinLines = Array.isArray(rawStdin) ? rawStdin.map(String)
 2. ~~**`--resume <id>` 启动参数**~~：✅ 已在 v0.1.37 实现
 3. ~~**Word wrap 配置**~~：✅ 已在 v0.1.37 实现
 4. ~~**主题/颜色自定义**~~：✅ 已在 v0.1.37 实现
-5. **L1 低危**：`run-tests.ts` 的 `JSON.parse(package.json)` 细粒度错误处理
+5. ~~**L1 低危**~~：✅ 已在 v0.1.50 修复
 
 ---
 
@@ -851,7 +900,7 @@ const stdinLines = Array.isArray(rawStdin) ? rawStdin.map(String)
 ### 下一步建议
 1. **P0 功能缺口**：并行工具调用、`/add-dir` 命令
 2. **P1 功能缺口**：`/memory` 命令、`/doctor` 健康检查、`/bug` 反馈
-3. **L1 低危**：`run-tests.ts` 的 `JSON.parse(package.json)` 细粒度错误处理
+3. ~~**L1 低危**~~：✅ 已在 v0.1.50 修复
 
 ---
 
@@ -1437,7 +1486,7 @@ const stdinLines = Array.isArray(rawStdin) ? rawStdin.map(String)
 
 | # | 文件 | 问题描述 |
 |---|------|---------|
-| L1 | `src/tools/builtin/run-tests.ts:35` | `JSON.parse(package.json)` 无细粒度错误处理 |
+| L1 | `src/tools/builtin/run-tests.ts` | ✅ v0.1.50 已修复：`safeReadPackageJson()` 细粒度错误处理 + BOM + 框架自动检测 |
 | L2 | `src/tools/builtin/web-fetch.ts` | 恶意 HTML 大量 script 标签时正则性能下降 |
 | L3 | `src/session/session.ts` | `new Date(d.created)` 非法字符串返回 Invalid Date，比较失败 |
 | L4 | `src/tools/builtin/ask-user.ts` / `google-search.ts` | 模块级全局上下文，多会话架构下会串扰 |

package/dist/{chunk-P4XKHPXU.js → chunk-U4BEPHO5.js}

@@ -8,7 +8,7 @@ import { platform } from "os";
 import chalk from "chalk";
 
 // src/core/constants.ts
-var VERSION = "0.1.49";
+var VERSION = "0.1.51";
 var APP_NAME = "ai-cli";
 var CONFIG_DIR_NAME = ".aicli";
 var CONFIG_FILE_NAME = "config.json";
@@ -80,6 +80,73 @@ var REPO_URL = "https://gitee.com/jinzhengdong/ai-courses";
 
 // src/tools/builtin/run-tests.ts
 var IS_WINDOWS = platform() === "win32";
+function detectNodeTestFramework(cwd, pkg) {
+  const devDeps = pkg.devDependencies ?? {};
+  const deps = pkg.dependencies ?? {};
+  const allDeps = { ...deps, ...devDeps };
+  if ("vitest" in allDeps || existsSync(join(cwd, "vitest.config.ts")) || existsSync(join(cwd, "vitest.config.js")) || existsSync(join(cwd, "vitest.config.mts"))) {
+    return { type: "node", framework: "vitest", command: "npx vitest run" };
+  }
+  if ("jest" in allDeps || existsSync(join(cwd, "jest.config.js")) || existsSync(join(cwd, "jest.config.ts")) || existsSync(join(cwd, "jest.config.mjs"))) {
+    return { type: "node", framework: "jest", command: "npx jest" };
+  }
+  if ("mocha" in allDeps || existsSync(join(cwd, ".mocharc.yml")) || existsSync(join(cwd, ".mocharc.yaml")) || existsSync(join(cwd, ".mocharc.json")) || existsSync(join(cwd, ".mocharc.js"))) {
+    return { type: "node", framework: "mocha", command: "npx mocha" };
+  }
+  if ("ava" in allDeps) {
+    return { type: "node", framework: "ava", command: "npx ava" };
+  }
+  if ("@playwright/test" in allDeps) {
+    return { type: "node", framework: "playwright", command: "npx playwright test" };
+  }
+  return null;
+}
+function safeReadPackageJson(cwd) {
+  const filePath = join(cwd, "package.json");
+  let raw;
+  try {
+    raw = readFileSync(filePath, "utf-8");
+  } catch (err) {
+    const code = err.code;
+    if (code === "EACCES" || code === "EPERM") {
+      process.stderr.write(`[Warning] Permission denied reading package.json (${code})
+`);
+    } else {
+      process.stderr.write(
+        `[Warning] Cannot read package.json: ${err instanceof Error ? err.message : String(err)}
+`
+      );
+    }
+    return null;
+  }
+  if (raw.charCodeAt(0) === 65279) {
+    raw = raw.slice(1);
+  }
+  if (!raw.trim()) {
+    process.stderr.write("[Warning] package.json is empty\n");
+    return null;
+  }
+  try {
+    const parsed = JSON.parse(raw);
+    if (typeof parsed !== "object" || parsed === null || Array.isArray(parsed)) {
+      process.stderr.write("[Warning] package.json root is not a JSON object\n");
+      return null;
+    }
+    return parsed;
+  } catch (err) {
+    const msg = err instanceof Error ? err.message : String(err);
+    if (msg.includes("Unexpected token")) {
+      process.stderr.write(`[Warning] package.json has syntax error: ${msg}
+`);
+    } else if (msg.includes("Unexpected end")) {
+      process.stderr.write("[Warning] package.json is truncated or incomplete\n");
+    } else {
+      process.stderr.write(`[Warning] package.json parse failed: ${msg}
+`);
+    }
+    return null;
+  }
+}
 function detectProject(cwd) {
   if (existsSync(join(cwd, "pom.xml"))) {
     return { type: "java", framework: "Maven (JUnit)", command: IS_WINDOWS ? "mvn.cmd test" : "mvn test" };
@@ -90,17 +157,16 @@ function detectProject(cwd) {
     return { type: "java", framework: "Gradle (JUnit)", command: cmd };
   }
   if (existsSync(join(cwd, "package.json"))) {
-    try {
-      const pkg = JSON.parse(readFileSync(join(cwd, "package.json"), "utf-8"));
-      const testScript = pkg?.scripts?.test;
+    const pkg = safeReadPackageJson(cwd);
+    if (pkg) {
+      const testScript = pkg.scripts?.test;
       if (testScript && testScript !== 'echo "Error: no test specified" && exit 1') {
         return { type: "node", framework: "npm", command: "npm test" };
       }
-    } catch (err) {
-      process.stderr.write(
-        `[Warning] Failed to parse package.json: ${err instanceof Error ? err.message : String(err)}
-`
-      );
+      const frameworkDetected = detectNodeTestFramework(cwd, pkg);
+      if (frameworkDetected) return frameworkDetected;
+    } else {
+      return { type: "node", framework: "npm (package.json error)", command: "npm test" };
     }
   }
   if (existsSync(join(cwd, "pyproject.toml")) || existsSync(join(cwd, "setup.py")) || existsSync(join(cwd, "pytest.ini"))) {
@@ -268,7 +334,17 @@ async function executeTests(args) {
       } else if (detected.type === "go") {
         command = `go test ./... -run "${filter}"`;
       } else if (detected.type === "node") {
-        command += ` -- --grep "${filter}"`;
+        if (detected.framework === "vitest") {
+          command += ` -t "${filter}"`;
+        } else if (detected.framework === "jest") {
+          command += ` -t "${filter}"`;
+        } else if (detected.framework === "mocha") {
+          command += ` --grep "${filter}"`;
+        } else if (detected.framework === "playwright") {
+          command += ` --grep "${filter}"`;
+        } else {
+          command += ` -- --grep "${filter}"`;
+        }
       }
     }
   }

package/dist/index.js

@@ -29,7 +29,7 @@ import {
   SUBAGENT_MAX_ROUNDS_LIMIT,
   VERSION,
   runTestsTool
-} from "./chunk-P4XKHPXU.js";
+} from "./chunk-U4BEPHO5.js";
 
 // src/index.ts
 import { program } from "commander";
@@ -4288,7 +4288,7 @@ ${hint}` : "")
       description: "Run project tests and show structured report",
       usage: "/test [command|filter]",
       async execute(args, _ctx) {
-        const { executeTests } = await import("./run-tests-JJVZWQKI.js");
+        const { executeTests } = await import("./run-tests-AWR6YAYZ.js");
         const argStr = args.join(" ").trim();
         let testArgs = {};
         if (argStr) {
@@ -5234,21 +5234,19 @@ ${pdfText}`;
       const dir = dirname3(normalizedPath);
       const nameNoExt = basename2(normalizedPath, ext);
       const textAlts = [".md", ".txt", ".html"].map((e) => resolve3(dir, nameNoExt + e)).filter(existsSync8);
-      return `[Binary file: ${filePath}]
-PDF \u6587\u672C\u63D0\u53D6\u5931\u8D25\uFF08pdftotext \u548C pdfminer \u5747\u4E0D\u53EF\u7528\uFF09\u3002
-` + (textAlts.length > 0 ? `\u627E\u5230\u53EF\u66FF\u4EE3\u7684\u6587\u672C\u7248\u672C\uFF1A
-${textAlts.map((p) => `  \u2192 ${basename2(p)}`).join("\n")}
-\u8BF7\u4F7F\u7528 read_file \u8BFB\u53D6\u4E0A\u8FF0\u6587\u4EF6\u3002` : `\u5982\u9700\u4F7F\u7528\u5176\u5185\u5BB9\uFF0C\u8BF7\u4F7F\u7528 bash \u5DE5\u5177\u5B89\u88C5\u5E76\u8C03\u7528 pdftotext\uFF1A
-  pip install pdfminer.six  # \u5B89\u88C5 Python PDF \u5E93
-\u6216\u5C06 PDF \u624B\u52A8\u8F6C\u6362\u4E3A .md / .txt \u6587\u4EF6\u540E\u518D\u8BFB\u53D6\u3002`);
+      if (textAlts.length > 0) {
+        return `[PDF file: ${filePath}]
+\u6B64 PDF \u6587\u4EF6\u5F53\u524D\u73AF\u5883\u65E0\u6CD5\u81EA\u52A8\u63D0\u53D6\u6587\u672C\uFF0C\u4F46\u627E\u5230\u53EF\u66FF\u4EE3\u7684\u6587\u672C\u7248\u672C\uFF1A
+` + textAlts.map((p) => `  \u2192 ${basename2(p)}`).join("\n") + `
+\u8BF7\u4F7F\u7528 read_file \u8BFB\u53D6\u4E0A\u8FF0\u6587\u4EF6\u3002`;
+      }
+      return `[PDF file: ${filePath}]
+\u6B64 PDF \u6587\u4EF6\u5F53\u524D\u73AF\u5883\u65E0\u6CD5\u81EA\u52A8\u63D0\u53D6\u6587\u672C\uFF08\u9700\u5B89\u88C5 pdftotext \u6216 pdfminer.six\uFF09\u3002
+\u5EFA\u8BAE\uFF1A\u76F4\u63A5\u53C2\u8003\u9879\u76EE\u4E2D\u5DF2\u6709\u7684\u6587\u672C\u7248\u672C\uFF08.md / .txt\uFF09\uFF0C\u6216\u7528 bash \u5B89\u88C5\u63D0\u53D6\u5DE5\u5177\u540E\u91CD\u8BD5\u3002`;
     }
     if (BINARY_EXTENSIONS.has(ext)) {
-      return `[Binary file: ${filePath}]
-\u6B64\u6587\u4EF6\u4E3A\u4E8C\u8FDB\u5236\u683C\u5F0F\uFF08${ext}\uFF09\uFF0C\u65E0\u6CD5\u4F5C\u4E3A\u6587\u672C\u8BFB\u53D6\u3002
-\u5982\u9700\u4F7F\u7528\u5176\u5185\u5BB9\uFF0C\u8BF7\u8003\u8651\uFF1A
-  1. \u5C06\u6587\u4EF6\u8F6C\u6362\u4E3A\u6587\u672C\u683C\u5F0F\u540E\u518D\u8BFB\u53D6
-  2. \u4F7F\u7528 bash \u5DE5\u5177\u8C03\u7528\u5916\u90E8\u8F6C\u6362\u7A0B\u5E8F\uFF08\u5982 pandoc \u7B49\uFF09
-  3. \u82E5\u6709\u5BF9\u5E94\u7684\u7EAF\u6587\u672C\u7248\u672C\uFF08.md / .txt\uFF09\uFF0C\u8BF7\u76F4\u63A5\u8BFB\u53D6\u90A3\u4E2A\u6587\u4EF6`;
+      return `[Binary file: ${filePath} (${ext})]
+\u6B64\u6587\u4EF6\u4E3A\u4E8C\u8FDB\u5236\u683C\u5F0F\uFF0C\u4E0D\u652F\u6301\u76F4\u63A5\u6587\u672C\u8BFB\u53D6\u3002\u5982\u9879\u76EE\u4E2D\u6709\u5BF9\u5E94\u7684\u6587\u672C\u7248\u672C\uFF08.md / .txt\uFF09\uFF0C\u8BF7\u76F4\u63A5\u8BFB\u53D6\u3002`;
     }
     const buf = readFileSync5(normalizedPath);
     if (encoding === "base64") {

package/dist/{run-tests-JJVZWQKI.js → run-tests-AWR6YAYZ.js}

@@ -2,7 +2,7 @@
 import {
   executeTests,
   runTestsTool
-} from "./chunk-P4XKHPXU.js";
+} from "./chunk-U4BEPHO5.js";
 export {
   executeTests,
   runTestsTool

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jinzd-ai-cli",
-  "version": "0.1.49",
+  "version": "0.1.51",
   "description": "Cross-platform REPL-style AI CLI with multi-provider support",
   "type": "module",
   "main": "./dist/index.js",