@moonquake2004/dsh-doctor 0.4.9 → 0.4.13

package/dsh-doctor.mjs

@@ -212,6 +212,15 @@ function readableSessionEventTypes() {
 
 const READABLE = readableSessionEventTypes();
 
+/**
+ * 是否存在值得诊断的 DSH 环境（真实用户机器有 sessions/ 或 settings.yaml；干净容器/仓库检出没有）。
+ * 以传入的 home 为准：探针应当判断"它被告知的那个环境"，而不是进程全局的 HOME ——
+ * 这样单测可以显式构造两种环境，fixture 也不会被真实 HOME 的状态污染。
+ */
+function hasDshEnvironment(home = HOME) {
+  return existsSync(join(home, 'sessions')) || existsSync(join(home, 'settings.yaml'));
+}
+
 function report(section, id, ok, detail, fix, src) {
   results.push({ section, id, ok, detail, fix, src: src ?? 'builtin' });
 }
@@ -274,6 +283,14 @@ function checkEnv() {
     for (const d of readdirSync(pnpmStore)) if (d.startsWith('node-pty@')) ptyDirs.push(join(pnpmStore, d, 'node_modules', 'node-pty'));
   }
   const plat = `${process.platform}-${process.arch}`;
+  // 无对象可查时 skip 而非 fail（2026-09 修正）：E4 的判据是"**已安装的** dsh 里 node-pty 是否完整"，
+  // 但此前在**没有任何 node_modules 的合成 HOME**（测试 fixture、全新环境）里也会报"未找到 node-pty" ——
+  // 那不是"缺失"，而是"这里根本没有安装树可查"。据此报 fail 会让 fixture 全红、也会让 CI 在
+  // 干净容器里得到无意义的 exit 2。与项目一贯纪律一致：不适用 ⇒ skip 且带 reason。
+  // "有安装树可查" = PATH 里找到了 dsh 的 node_modules，**或**本 HOME 的 profile 里有 node_modules。
+  // 二者皆无（合成 HOME、干净容器）→ 这里没有"已安装的 dsh"可言，报 fail 是无意义的。
+  const fromPath = ptyDirs.length > 0 && existsSync(ptyDirs[0]);
+  const hasInstall = fromPath || existsSync(profileNM);
   const ptyFound = ptyDirs.filter((d) => existsSync(d));
   let ptyBinary = null;
   for (const d of ptyFound) {
@@ -282,7 +299,8 @@ function checkEnv() {
     }
     if (ptyBinary) break;
   }
-  if (ptyFound.length === 0) report('env', 'E4', false, '未找到 node-pty（dsh web 终端依赖它，#1219）', '重新安装 @deepseek-ai/dsh，确保 node-pty 装全');
+  if (!hasInstall) reportSkip('env', 'E4', '未发现 DSH 安装树（无 profiles/node_modules 可查），跳过 node-pty 完整性检测');
+  else if (ptyFound.length === 0) report('env', 'E4', false, '未找到 node-pty（dsh web 终端依赖它，#1219）', '重新安装 @deepseek-ai/dsh，确保 node-pty 装全');
   else if (ptyBinary) report('env', 'E4', true, `node-pty 原生模块在位（${plat}）`, undefined);
   else report('env', 'E4', false, `node-pty 存在但缺 ${plat} 原生二进制（#1219: dsh web 启动失败）`, '重装 node-pty（npm rebuild node-pty）或从源码构建');
 
@@ -1638,8 +1656,14 @@ export function runCatalogCheck(check, ctx) {
   switch (probe.type) {
     case 'command-exists': {
       const found = findCommand(probe.cmd);
-      return found ? { ok: true, detail: check.detailOk ?? `${probe.cmd} 在 PATH: ${found}` }
-                   : { ok: false, detail: check.detailFail ?? `${probe.cmd} 不在 PATH` };
+      if (found) return { ok: true, detail: check.detailOk ?? `${probe.cmd} 在 PATH: ${found}` };
+      // 无 DSH 环境可言时 skip（2026-09）：在干净容器/仓库检出里跑本工具时，dsh 当然不在 PATH ——
+      // 那不是"用户环境有问题"，而是"这里没有被诊断的 DSH 环境"。据此报 error 会让 CI 与 fixture
+      // 得到无意义的失败（CI 三次红都源于同类检查：E4 → E1-pnpm → E7）。
+      if (!hasDshEnvironment(ctx?.home ?? HOME)) {
+        return { skipped: true, detail: `未发现 DSH 环境（无 sessions/ 与 settings.yaml），跳过 ${probe.cmd} 的 PATH 检查` };
+      }
+      return { ok: false, detail: check.detailFail ?? `${probe.cmd} 不在 PATH` };
     }
     case 'path-exists':
     case 'path-is-dir':
@@ -2012,7 +2036,13 @@ async function run() {
     console.log(JSON.stringify(out, null, 2));
     process.exit(exitCode);
   } else if (jsonOut) {
-    console.log(JSON.stringify({ ok: bad.length === 0 && secExit === 0, checks: results, catalog: catalogMeta, update: updateInfo, ...(securityMeta.enabled ? { security: securityMeta } : {}) }, null, 2));
+    // 每条检查带上 status（pass/warn/fail/skip，与 --envelope 同一词汇表）。
+    // 2026-09：此前 --json 只给 ok 布尔，消费者（含我们自己的测试助手）无法区分
+    // "warn 级失败"（如 CI 上无 pnpm → E1-pnpm warn，不翻退出码）与"error 级失败"，
+    // 于是把 warn 当成误报。状态本就不该由消费者自行推导。
+    const statusOf = (r) => (r.skip ? 'skip' : (!r.ok ? (((r.section === 'security') ? r.severity !== 'critical' : catalogSeverity.get(r.id) === 'warn') ? 'warn' : 'fail') : 'pass'));
+    const checksWithStatus = results.map((r) => ({ ...r, status: statusOf(r) }));
+    console.log(JSON.stringify({ ok: bad.length === 0 && secExit === 0, checks: checksWithStatus, catalog: catalogMeta, update: updateInfo, ...(securityMeta.enabled ? { security: securityMeta } : {}) }, null, 2));
   } else {
     const sectionOrder = { env: 0, profile: 1, session: 2, catalog: 3 };
     const ordered = [...results].sort((a, b) => (sectionOrder[a.section] ?? 9) - (sectionOrder[b.section] ?? 9));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moonquake2004/dsh-doctor",
-  "version": "0.4.9",
+  "version": "0.4.13",
   "description": "Offline diagnostic for DeepSeek Harness — 28 built-in + 5 catalog checks across env/profile/session (Layer A checks-as-data), self-update (Layer B), and a semi-automatic LLM observer (Layer C, --observe); Doctor panel in web UI settings.",
   "main": "lib/index.js",
   "files": [