@moonquake2004/dsh-security 0.1.3 → 0.1.5

package/README.md

@@ -2,9 +2,9 @@
 
 [![License: MIT](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
 [![npm](https://img.shields.io/npm/v/@moonquake2004/dsh-security)](https://www.npmjs.com/package/@moonquake2004/dsh-security)
-[![Tests](https://img.shields.io/badge/tests-80%2F80%20passing-brightgreen)](#)
+[![Tests](https://img.shields.io/badge/tests-92%2F92%20passing-brightgreen)](#)
 
-**DSH 生态统一安全检查框架** — 覆盖插件全生命周期（发现→安装→运行→更新→退役），19 个内置检查 + 4 个外部工具集成。
+**DSH 生态统一安全检查框架** — 覆盖插件全生命周期（发现→安装→运行→更新→退役），21 个内置检查 + 4 个外部工具集成。
 
 > Community security tool. Not an official DeepSeek project.
 
@@ -76,9 +76,11 @@ dsh-security 的目标：**建立 DSH 生态的统一安全检查层**，让任
 │  SP6: 已知漏洞匹配 (OSV/CVE/GHSA)                        │
 │  SP7: client 产物语法预检 (#2752 白屏源 boot 前拦截)      │
 │  SP8: dist-tag 异常检测 (#2763 broken latest)            │
+│  SP9: CLI 核心包泄漏检测 (#4640 symbol 分裂)              │
 │  SS1: 凭据泄露检测 (会话日志)                             │
 │  SS2: PII 数据暴露检测                                    │
 │  SS3: 插件输出敏感数据检测                                │
+│  SS4: 会话日志完整性校验 (截断JSON/孤儿tool-call)          │
 ├──────────────────────────────────────────────────────────┤
 │  Layer 2: Runtime Checks（运行时，基于会话日志）           │
 │  SR1: 沙箱逃逸检测 (#1769 mount remount 等)              │
@@ -251,7 +253,7 @@ node --test test/*.mjs
 dsh-security/
 ├── src/
 │   ├── protocol/          # Layer 0: Check Protocol
-│   ├── checks/            # 19 个内置检查
+│   ├── checks/            # 21 个内置检查
 │   ├── integrations/      # 4 个外部工具集成
 │   ├── session-reader.mjs # 会话日志读取（明文 + zstd）
 │   ├── registry.mjs       # Check Registry（支持 setConfig 注入配置）

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moonquake2004/dsh-security",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "description": "Unified security check framework for DeepSeek Harness ecosystem — covers the full plugin lifecycle (discovery → install → runtime → update → retirement)",
   "type": "module",
   "main": "src/index.mjs",

package/src/checks/index.mjs

@@ -7,6 +7,7 @@ export { sp5Check } from './sp5-permission-model.mjs';
 export { sp6Check } from './sp6-vuln-match.mjs';
 export { sp7Check } from './sp7-client-syntax.mjs';
 export { sp8Check } from './sp8-dist-tag-health.mjs';
+export { sp9Check } from './sp9-dual-instance-guard.mjs';
 
 // Layer 2: Runtime Checks
 export { sr1Check } from './sr1-sandbox-violation.mjs';
@@ -24,3 +25,4 @@ export { sl4Check } from './sl4-release-compat.mjs';
 export { ss1Check } from './ss1-credential-leak.mjs';
 export { ss2Check } from './ss2-pii-exposure.mjs';
 export { ss3Check } from './ss3-sensitive-output.mjs';
+export { ss4Check } from './ss4-session-integrity.mjs';

package/src/checks/sp9-dual-instance-guard.mjs

@@ -0,0 +1,105 @@
+/**
+ * SP9: Dual-Instance Guard — CLI 核心包泄漏检测
+ *
+ * 出处：#4640 + zoahdev/dsh-ecosystem 家族 4——profile 的 node_modules
+ * 中若出现 @deepseek-ai/dsh-* 包（尤其是 dsh-tools），会导致同一进程
+ * 两份副本 → TOOL_RUNTIME_SCHEDULER 唯一 symbol 分裂 →
+ * undefined.prepare → 所有工具调用失败 → 会话不可恢复。
+ *
+ * 核心规则：profile 只承载 surfaces 和 plugins；core packages 由 CLI
+ * 自身的依赖树提供。profile 中出现任何 @deepseek-ai/dsh-* 包即为异常。
+ *
+ * 做法：扫描 profile/node_modules/@deepseek-ai/ 下的目录，
+ * 检查是否存在 dsh-* 包（排除纯客户端包如 dsh-client-*，
+ * 因为它们可能被插件合法引用；只报 dsh-tools/dsh-agent-loop
+ * 等核心运行时包）。
+ *
+ * Severity: CRITICAL（直接导致工具调用全灭）
+ * Phase: POST_INSTALL
+ */
+
+import { readFileSync, readdirSync, existsSync } from 'node:fs';
+import { join } from 'node:path';
+import { Severity } from '../protocol/severity.mjs';
+import { CheckPhase } from '../protocol/phase.mjs';
+import { pass, fail, skip } from '../protocol/check.mjs';
+
+/**
+ * CLI 核心运行时包——出现在 profile 中即为异常。
+ * 客户端包（dsh-client-*）被插件合法引用，不在此列。
+ */
+const CORE_RUNTIME_PKGS = new Set([
+  'dsh-tools',
+  'dsh-agent-loop',
+  'dsh-sandbox-local',
+  'dsh-subprocess-local',
+]);
+
+export async function run(profileDir) {
+  const id = 'SP9';
+  const nmDir = join(profileDir, 'node_modules');
+  if (!existsSync(nmDir)) return pass(id, Severity.CRITICAL, '无 node_modules，跳过核心包泄漏检测');
+
+  const dsDir = join(nmDir, '@deepseek-ai');
+  if (!existsSync(dsDir)) return pass(id, Severity.CRITICAL, 'profile 中无 @deepseek-ai 包，无泄漏风险');
+
+  // 枚举 @deepseek-ai/* 下的包
+  let entries;
+  try { entries = readdirSync(dsDir, { withFileTypes: true }); } catch { entries = []; }
+
+  const leaked = [];
+  for (const entry of entries) {
+    if (!entry.isDirectory()) continue;
+    const pkgName = `@deepseek-ai/${entry.name}`;
+    const pjPath = join(dsDir, entry.name, 'package.json');
+    if (!existsSync(pjPath)) continue;
+
+    let version = '?';
+    try {
+      const pkg = JSON.parse(readFileSync(pjPath, 'utf8'));
+      version = pkg.version || '?';
+    } catch { /* read error */ }
+
+    if (entry.name.startsWith('dsh-')) {
+      leaked.push({ name: pkgName, version, isCore: CORE_RUNTIME_PKGS.has(entry.name) });
+    }
+  }
+
+  if (leaked.length === 0) {
+    return pass(id, Severity.CRITICAL, 'profile 中无 @deepseek-ai/dsh-* 泄漏，核心包仅由 CLI 提供');
+  }
+
+  const coreLeaks = leaked.filter(l => l.isCore);
+  const otherLeaks = leaked.filter(l => !l.isCore);
+
+  // 核心运行时包泄漏 = CRITICAL（直接导致 Symbol 分裂 + 工具调用全灭）
+  if (coreLeaks.length > 0) {
+    const details = coreLeaks.map(l => `  ${l.name}@${l.version}`).join('\n');
+    const otherNote = otherLeaks.length > 0
+      ? `\n另有 ${otherLeaks.length} 个非核心 @deepseek-ai/dsh-* 包在 profile 中：${otherLeaks.map(l => l.name).join(', ')}`
+      : '';
+    return fail(id, Severity.CRITICAL,
+      `检测到 ${coreLeaks.length} 个 CLI 核心运行时包泄漏到 profile（#4640：双实例 symbol 分裂 → 工具调用全灭）：\n${details}${otherNote}\n修复: dsh plugin --profile <name> remove <泄漏包名>（或检查安装来源是否误将 CLI 依赖写入 profile）`,
+      '核心包（dsh-tools/dsh-agent-loop 等）不应出现在 profile 的 node_modules 中；它们由 CLI 自身依赖树提供',
+      ['#4640']
+    );
+  }
+
+  // 只有非核心包泄漏 = HIGH（潜在风险但不一定立即崩溃）
+  const details = otherLeaks.map(l => `  ${l.name}@${l.version}`).join('\n');
+  return fail(id, Severity.HIGH,
+    `检测到 ${otherLeaks.length} 个 @deepseek-ai/dsh-* 包泄漏到 profile（非核心包，暂未触发 symbol 分裂，但增加未来冲突风险）：\n${details}`,
+    '清理 profile 中不需要的 @deepseek-ai/dsh-* 包；若为客户端包且被插件依赖，确认无版本冲突',
+    ['#4640']
+  );
+}
+
+export const sp9Check = {
+  id: 'SP9',
+  name: 'dual-instance-guard',
+  severity: Severity.CRITICAL,
+  phase: CheckPhase.POST_INSTALL,
+  description: 'CLI 核心包泄漏检测——profile 中不应出现 @deepseek-ai/dsh-tools 等核心包（#4640 symbol 分裂）',
+  src: 'builtin',
+  runner: (profileDir) => run(profileDir),
+};

package/src/checks/ss1-credential-leak.mjs

@@ -124,7 +124,7 @@ export async function run(sessionFile) {
     .map(f => `行 ${f.line} — ${f.type}（${f.snippet}）`)
     .join('\n');
 
-  const fix = '使用 dsh-redact 脱敏后再分享会话日志：npx dsh-redact <session.jsonl> --out redacted.jsonl';
+  const fix = '轮换泄露的密钥（不可逆），然后清理会话日志：可用 zoahdev/dsh-redact（GitHub: github.com/zoahdev/dsh-redact）或手动删除含密钥的行';
 
   return fail(id, Severity.CRITICAL, `检测到 ${findings.length} 个凭据泄露（${summary}）：\n${details}`, fix, ['#962']);
 }

package/src/checks/ss2-pii-exposure.mjs

@@ -74,7 +74,7 @@ export async function run(sessionFile) {
   for (const f of findings) { if (!byType[f.type]) byType[f.type] = 0; byType[f.type]++; }
   const summary = Object.entries(byType).map(([t, c]) => `${t}: ${c}`).join(', ');
   const details = findings.slice(0, 10).map(f => `行${f.line} — ${f.type}: ${f.snippet}`).join('\n');
-  return fail(id, Severity.MEDIUM, `检测到 ${findings.length} 个 PII 暴露（${summary}）：\n${details}`, '使用 dsh-redact 脱敏后再分享会话日志');
+  return fail(id, Severity.MEDIUM, `检测到 ${findings.length} 个 PII 暴露（${summary}）：\n${details}`, '分享会话日志前用 zoahdev/dsh-redact（github.com/zoahdev/dsh-redact）脱敏，或手动删除含 PII 的行');
 }
 
 export const ss2Check = { id: 'SS2', name: 'pii-exposure', severity: Severity.MEDIUM, phase: CheckPhase.POST_INSTALL, description: 'PII 数据暴露检测', src: 'builtin', runner: (f) => run(f) };

package/src/checks/ss4-session-integrity.mjs

@@ -0,0 +1,117 @@
+/**
+ * SS4: Session Integrity — 会话日志完整性校验
+ *
+ * 出处：zoahdev/dsh-ecosystem 家族 3（坏工件隔离）+ 家族 11（out-of-tree
+ * 会话事件信封缺口）——截断的 tool-call 参数、torn session tail、空文件
+ * 会导致会话永久砖化或 resume 失败。
+ *
+ * 检查项：
+ * 1. JSON 语法：逐行 parse，无效行 = 截断/损坏
+ * 2. tool/call 配对：每个 tool/call 应有对应 tool/result（孤儿调用）
+ * 3. 空/极小文件：0 行或 <10 行 = 可能未正常写入
+ *
+ * Severity: HIGH
+ * Phase: SESSION
+ */
+
+import { existsSync, statSync } from 'node:fs';
+import { Severity } from '../protocol/severity.mjs';
+import { CheckPhase } from '../protocol/phase.mjs';
+import { pass, fail, skip } from '../protocol/check.mjs';
+import { scanSessionLines } from '../session-reader.mjs';
+
+const MIN_LINES = 10;
+
+export async function run(sessionFile) {
+  const id = 'SS4';
+
+  if (!sessionFile || !existsSync(sessionFile)) {
+    return skip(id, Severity.HIGH, '无会话日志文件，跳过完整性校验');
+  }
+
+  // 检查文件大小
+  let fileSize = 0;
+  try { fileSize = statSync(sessionFile).size; } catch { /* ignore */ }
+  if (fileSize === 0) {
+    return fail(id, Severity.HIGH, '会话日志文件为空（0 字节）——可能未正常写入', '检查 dsh 进程是否正常退出；若为预期空会话可忽略', ['#675']);
+  }
+
+  let totalLines = 0;
+  let invalidJson = 0;
+  let firstInvalidLine = null;
+  const orphanCalls = new Map(); // callId → line number
+  let resultCount = 0;
+
+  await scanSessionLines(sessionFile, (line, lineNum) => {
+    totalLines++;
+
+    // 1. JSON 语法检查
+    let parsed;
+    try {
+      parsed = JSON.parse(line);
+    } catch {
+      invalidJson++;
+      if (!firstInvalidLine) firstInvalidLine = lineNum;
+      return; // 无法 parse 的行跳过后续检查
+    }
+
+    // 2. tool/call 与 tool/result 配对检查
+    const type = parsed.type;
+    if (type === 'tool/call') {
+      const callId = parsed.data?.callId;
+      if (callId) orphanCalls.set(callId, lineNum);
+    } else if (type === 'tool/result') {
+      const callId = parsed.data?.callId;
+      if (callId) {
+        orphanCalls.delete(callId); // 配对成功
+        resultCount++;
+      }
+    }
+  });
+
+  // 3. 极小文件检查
+  if (totalLines < MIN_LINES && totalLines > 0) {
+    return fail(id, Severity.HIGH,
+      `会话日志仅 ${totalLines} 行（阈值 ${MIN_LINES}）——可能未完整记录`,
+      '检查会话是否正常结束；若为极短会话可忽略',
+      ['#675']
+    );
+  }
+
+  const issues = [];
+  const refs = [];
+
+  // 汇总孤儿 tool/call
+  if (orphanCalls.size > 0) {
+    const samples = [...orphanCalls.entries()].slice(0, 5).map(([id, ln]) => `  行${ln}: callId=${id.slice(0, 16)}…`);
+    issues.push(`${orphanCalls.size} 个孤儿 tool/call（有调用无结果）：\n${samples.join('\n')}`);
+    refs.push('#3234');
+  }
+
+  // 汇总无效 JSON
+  if (invalidJson > 0) {
+    issues.push(`${invalidJson} 行无效 JSON（首个在行 ${firstInvalidLine}）——可能截断或损坏`);
+    refs.push('#675', '#1047');
+  }
+
+  if (issues.length === 0) {
+    return pass(id, Severity.HIGH,
+      `会话日志完整性通过：${totalLines} 行，JSON 全有效，${resultCount} 个 tool/result 均已配对`);
+  }
+
+  return fail(id, Severity.HIGH,
+    `检测到会话日志完整性问题（${totalLines} 行）：\n${issues.join('\n\n')}`,
+    '截断的 tool/call 会导致会话砖化（#3234）；无效 JSON 行可能是损坏的事件（#675/#1047）；考虑用 dsh-shelf 归档后重新开始会话',
+    refs
+  );
+}
+
+export const ss4Check = {
+  id: 'SS4',
+  name: 'session-integrity',
+  severity: Severity.HIGH,
+  phase: CheckPhase.POST_INSTALL,
+  description: '会话日志完整性校验——检测截断 JSON、孤儿 tool/call、空/极小文件（家族 3/11）',
+  src: 'builtin',
+  runner: (sessionFile) => run(sessionFile),
+};

package/src/index.mjs

@@ -1,7 +1,7 @@
 /**
  * DSH Security Framework — 统一安全检查框架
  *
- * 为 DSH 生态提供全生命周期安全检查（19 个检查项，4 层架构）。
+ * 为 DSH 生态提供全生命周期安全检查（21 个检查项，4 层架构）。
  *
  * @example
  * import { createDefaultRegistry } from '@moonquake2004/dsh-security';
@@ -24,6 +24,7 @@ export { sp5Check } from './checks/sp5-permission-model.mjs';
 export { sp6Check } from './checks/sp6-vuln-match.mjs';
 export { sp7Check } from './checks/sp7-client-syntax.mjs';
 export { sp8Check } from './checks/sp8-dist-tag-health.mjs';
+export { sp9Check } from './checks/sp9-dual-instance-guard.mjs';
 
 // Layer 2: Runtime Checks
 export { sr1Check } from './checks/sr1-sandbox-violation.mjs';
@@ -41,6 +42,7 @@ export { sl4Check } from './checks/sl4-release-compat.mjs';
 export { ss1Check } from './checks/ss1-credential-leak.mjs';
 export { ss2Check } from './checks/ss2-pii-exposure.mjs';
 export { ss3Check } from './checks/ss3-sensitive-output.mjs';
+export { ss4Check } from './checks/ss4-session-integrity.mjs';
 
 // Registry
 export { SecurityCheckRegistry, createDefaultRegistry } from './registry.mjs';

package/src/registry.mjs

@@ -147,8 +147,10 @@ export async function createDefaultRegistry(loadExternal = true, options = {}) {
     import('./checks/ss1-credential-leak.mjs'),
     import('./checks/ss2-pii-exposure.mjs'),
     import('./checks/ss3-sensitive-output.mjs'),
+    import('./checks/ss4-session-integrity.mjs'),
     import('./checks/sp7-client-syntax.mjs'),
     import('./checks/sp8-dist-tag-health.mjs'),
+    import('./checks/sp9-dual-instance-guard.mjs'),
   ]);
   for (const mod of modules) {
     for (const val of Object.values(mod)) {