jinzd-ai-cli 0.1.49 → 0.1.50

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
 
@@ -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-64YCGWC5.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.50";
 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-64YCGWC5.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-6DPROQJ6.js");
         const argStr = args.join(" ").trim();
         let testArgs = {};
         if (argStr) {

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

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

package/package.json

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