mingdao-harness 0.4.2 → 0.4.3

package/README.md

@@ -1,6 +1,8 @@
 # MingDao Harness
 
 > 开源智能体框架（Agent Harness）：**零运行时依赖、开箱即用**，针对 DeepSeek-V4 首发深度优化，开放主流模型接入。一条命令安装，终端与浏览器双界面，命令：`mingdao`（简写 `mdh`）。
+>
+> 项目简称 **MDH**（MingDao Harness），自 v0.4.3 起文档/日志/发布说明统一使用。
 
 轻量的「模型循环 + 工具 + 权限」内核，能力以 ESM 库导出、接口全部开放。架构见 [docs/ARCHITECTURE.md](docs/ARCHITECTURE.md)。
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mingdao-harness",
-  "version": "0.4.2",
+  "version": "0.4.3",
   "description": "MingDao Harness —— 开源智能体框架（Agent Harness）。零依赖、开箱即用，针对 DeepSeek-V4 系列优化，开放主流模型接入。",
   "type": "module",
   "bin": {

package/src/agent.js

@@ -45,7 +45,7 @@ export function createAgent({ provider, permission, io, modelName, workingDir, c
   const toolResultCap = Math.min(20000, Math.max(2000, Math.floor(caps.contextWindow / 16)));
   const temperature = cfg.temperature ?? preset.temperature ?? 0.6;
   const reasoningEffort = cfg.reasoningByModel?.[modelName] ?? cfg.reasoningEffort ?? preset.reasoningEffort?.default ?? undefined;
-  const hooks = createHooks(cfg.hooks, workingDir);
+  const hooks = createHooks(cfg.hooks, workingDir, cfg);
   const todos = /** @type {any[]} */ ([]);
   // 会话级共享：调用方传入则复用（/model 切换、子代理均共享，undo 不丢失）
   const undo = undoStore || { backups: new Map() };
@@ -789,6 +789,12 @@ export function createAgent({ provider, permission, io, modelName, workingDir, c
     }
     // 理论不可达（for 循环末轮必 return）；给 tsc 一个兜底，保证 runTurn 恒有返回值
     return { text: null, reasoning: '', usage, steps, finish, truncated: true, aborted: false, capHit: true, durationMs: Date.now() - startedAt, perf: perf() };
+    } catch (/** @type {any} */ err) {
+      // 审计 P2-6（v0.4.2）：工具管线异常（hooks.pre / permission.check / prepTool 等）沿大 try 上抛时，
+      // assistant tool_calls 已 push 进 messages 却无对应 tool 回填——会话恢复后 API 因孤儿 tool_call_id 400。
+      // 清理孤儿调用后再抛（stripOrphanCalls 此前只在中断/收尾路径调用）。
+      stripOrphanCalls();
+      throw err;
     } finally {
       currentAc = null;
       offSigint();

package/src/cachestats.js

@@ -6,6 +6,7 @@ import path from 'node:path';
 import { mingdaoHome, ensureHome } from './config.js';
 import { estimateCost, cacheSplit, beijingDayStart, beijingParts } from './pricing.js';
 import { modelPreset } from './models.js';
+import { withFileLockSync, atomicWriteFileSync } from './atomic-write.js';
 
 export function cacheStatsFile() {
   return path.join(mingdaoHome(), 'cache-stats.jsonl');
@@ -41,12 +42,16 @@ export function recordCacheStats(/** @type {any} */ entry) {
   } catch {}
   if (cacheStatsCount > MAX_LINES && cacheStatsCount % 200 === 0) {
     try {
-      const raw = fs.readFileSync(cacheStatsFile(), 'utf8');
-      const lines = raw.split('\n').filter(Boolean);
-      if (lines.length > MAX_LINES) {
-        fs.writeFileSync(cacheStatsFile(), lines.slice(-KEEP_LINES).join('\n') + '\n');
-        cacheStatsCount = KEEP_LINES;
-      }
+      // 审计 P2-3（v0.4.2）：轮转 read-modify-write 加跨进程锁——web/CLI/worker 多进程并发轮转时，
+      // 读与写之间他人追加的行会被覆写丢失；锁内重读再瘦身，写用原子替换。
+      withFileLockSync(cacheStatsFile() + '.lock', () => {
+        const raw = fs.readFileSync(cacheStatsFile(), 'utf8');
+        const lines = raw.split('\n').filter(Boolean);
+        if (lines.length > MAX_LINES) {
+          atomicWriteFileSync(cacheStatsFile(), lines.slice(-KEEP_LINES).join('\n') + '\n');
+          cacheStatsCount = KEEP_LINES;
+        }
+      });
     } catch {}
   }
 }

package/src/cli.js

@@ -172,6 +172,9 @@ async function main() {
   }
   // 最近会话日志默认不注入（新会话全新开始）；--journal 显式带上
   const withJournal = Boolean(opts.journal);
+  // 审计 P2-5（v0.4.2）：jsonMode 提前到 opts 解析后判定——此前在单次提问分支内才判定，
+  // 预设/tools/MCP 等提示在判定前已打到 stdout，污染 --format json 的单行 JSON 输出。
+  const jsonMode = opts.format === 'json';
 
   // —— 命令分发（已拆至 src/commands/，评估 P0-1 拆 cli.js）——
   // 各 handler 返回 true = 已处理；false = 按普通提问继续（保留词劫持防护）。
@@ -366,20 +369,20 @@ async function main() {
     activePreset = loadPreset(workingDir, opts.preset);
     if (!activePreset) {
       const names = listPresets(workingDir).map((/** @type {any} */ p) => p.name).join(', ') || '（无可用预设）';
-      io.print(style(`⚠ 预设 "${opts.preset}" 不存在。可用：${names}`, C.yellow));
+      if (!jsonMode) io.print(style(`⚠ 预设 "${opts.preset}" 不存在。可用：${names}`, C.yellow));
     } else {
       presetOverlay = { ...presetConfigOverrides(activePreset), presetName: activePreset.name };
       // P0（v0.4.1）：预设 permission 提权防护——预设不得把 ask/readonly 静默改成 auto
       const permOv = presetPermissionOverride(activePreset, cfg.permission ?? 'ask');
       if (permOv.escalated) {
         delete presetOverlay.permission;
-        io.print(style(`⚠ 预设 "${activePreset.name}" 声明 permission=${activePreset.permission} 属提权（当前 ${cfg.permission ?? 'ask'}），已忽略并保持 ${permOv.permission}。`, C.yellow));
+        if (!jsonMode) io.print(style(`⚠ 预设 "${activePreset.name}" 声明 permission=${activePreset.permission} 属提权（当前 ${cfg.permission ?? 'ask'}），已忽略并保持 ${permOv.permission}。`, C.yellow));
       } else if (activePreset.permission !== undefined) {
         presetOverlay.permission = permOv.permission;
       }
       if (!opts.model && presetOverlay.model) modelName = presetOverlay.model;
       presetBlock = presetSystemBlock(activePreset);
-      io.print(style(`▣ 已应用智能体预设：${activePreset.name}${activePreset.label ? '（' + activePreset.label + '）' : ''}`, C.cyan));
+      if (!jsonMode) io.print(style(`▣ 已应用智能体预设：${activePreset.name}${activePreset.label ? '（' + activePreset.label + '）' : ''}`, C.cyan));
     }
   }
   // agent 使用的配置 = cfg + 预设 overlay（presetTools/permission/参数按预设生效，cfg 本体保持干净）
@@ -406,7 +409,7 @@ async function main() {
   {
     const { mountConfigTools } = await import('./tools/index.js');
     const mounted = mountConfigTools(cfg);
-    if (mounted.length) io.print(style(`🔧 已挂载声明式工具（config.tools）：${mounted.join(', ')}`, C.dim));
+    if (mounted.length && !jsonMode) io.print(style(`🔧 已挂载声明式工具（config.tools）：${mounted.join(', ')}`, C.dim));
   }
   // 会话级 undo 备份仓：模型切换、子代理均共享，撤销记录不丢失
   const sessionUndoStore = { backups: new Map() };
@@ -432,10 +435,10 @@ async function main() {
     if (!mcpManager) mcpStartP.then((/** @type {any} */ m) => { if (m) m.stop(); });
     if (mcpManager) {
       const ready = mcpManager.status().filter((/** @type {any} */ s) => s.ok).length;
-      if (io && !opts.prompt.length) {
+      if (io && !opts.prompt.length && !jsonMode) {
         io.print(style(`✓ MCP 就绪：${ready}/${mcpManager.status().length} 个服务器，共 ${mcpManager.toolSchemas().length} 个工具`, C.dim));
       }
-    } else if (io && !opts.prompt.length) {
+    } else if (io && !opts.prompt.length && !jsonMode) {
       io.print(style('⚠ MCP 连接超时（6s）：本会话不注入 MCP 工具（重启 mingdao 可重试）', C.dim));
     }
   }
@@ -463,7 +466,6 @@ async function main() {
 
   // —— 单次提问模式 ——
   if (opts.prompt.length > 0) {
-    const jsonMode = opts.format === 'json';
     const question = opts.prompt.join(' ');
     // 自动路由：规划类任务切 planner，执行类走 executor（JSON 模式静默）
     const route = await routeTask({ cfg, provider, currentModel: modelName, text: question });
@@ -504,12 +506,16 @@ async function main() {
       const res = await turnAgent.runTurn(messages);
       appendMessages(session.file, messages.slice(oneShotPersisted));
       if (!jsonMode && cfg.autoTitle !== false && res.text) {
-        const tModel = titleModel(cfg, modelName);
-        const title = await generateTitle(await helperProvider(cfg, tModel, provider), tModel, question);
-        if (title) {
-          const renamed = renameSessionFile(fs, path, home, session, title);
-          if (renamed) io.print(style(`✓ 会话标题：${path.basename(renamed)}`, C.dim));
-        }
+        // 审计 P2-4（v0.4.2）：autoTitle 独立 try/catch——此前 titleModel 目标服务商无 Key 时
+        // helperProvider 抛错落入外层 catch，成功回答已输出却被误报失败（exitCode=2）。
+        try {
+          const tModel = titleModel(cfg, modelName);
+          const title = await generateTitle(await helperProvider(cfg, tModel, provider), tModel, question);
+          if (title) {
+            const renamed = renameSessionFile(fs, path, home, session, title);
+            if (renamed) io.print(style(`✓ 会话标题：${path.basename(renamed)}`, C.dim));
+          }
+        } catch {}
       }
       // v0.3.0 P0-2：单次提问跑满步数/中断落检查点（--continue 可续跑），正常完成清除
       if (res.capHit || res.aborted) {

package/src/commands/repl.js

@@ -173,8 +173,12 @@ export async function runRepl(ctx) {
   if (cfg.web?.autoStart && !process.env.MINGDAO_NO_WEB_AUTOSTART) {
     try {
       const { spawn } = await import('node:child_process');
+      const path = await import('node:path');
       const { fileURLToPath } = await import('node:url');
-      const child = spawn(process.execPath, [fileURLToPath(import.meta.url), 'web'], {
+      // 审计 P1-1（v0.4.2）：此前 spawn 的是 repl.js 自身（纯模块无 main 入口），web 参数被静默
+      // 丢弃、进程即退——自启 100% 失效还误报「后台启动中」。改为 spawn cli.js web（命令分发入口）。
+      const cliEntry = path.join(path.dirname(fileURLToPath(import.meta.url)), '..', 'cli.js');
+      const child = spawn(process.execPath, [cliEntry, 'web'], {
         detached: true,
         stdio: 'ignore',
         env: process.env,

package/src/commands/skill.js

@@ -39,7 +39,8 @@ export async function handleSkill(/** @type {any} */ cmd, /** @type {any} */ arg
     return true;
   }
   if (sub === 'install') {
-    const r = await installSkill(arg);
+    // CLI 显式输入 URL 属用户自担意图：放行内网地址（与 curl 等价）；WebUI 路径默认拦截（SSRF 防护）
+    const r = await installSkill(arg, { allowPrivateUrl: true });
     if (r.error) {
       console.log('[错误] ' + (/** @type {any} */ (r)).error);
       process.exitCode = 1;

package/src/hooks.js

@@ -9,6 +9,7 @@
 //  - matcher 支持精确工具名、逗号分隔多个名、'*' 通配。
 
 import { spawn } from 'node:child_process';
+import { isSensitiveEnv } from './tools/bash.js';
 
 function normalize(/** @type {any} */ list) {
   if (!Array.isArray(list)) return [];
@@ -25,16 +26,27 @@ function match(/** @type {any} */ hook, /** @type {any} */ toolName) {
   });
 }
 
-export function createHooks(hooksCfg = {}, /** @type {any} */ workingDir) {
+export function createHooks(hooksCfg = {}, /** @type {any} */ workingDir, /** @type {any} */ cfg = {}) {
   const pre = normalize((/** @type {any} */ (hooksCfg))?.PreToolUse);
   const post = normalize((/** @type {any} */ (hooksCfg))?.PostToolUse);
+  // 审计 P3-6（v0.4.2）：hook 子进程不再全量透传 process.env（含 API Key）——与 bash 工具同口径
+  // 默认过滤敏感变量；config.bashEnvKeep 按名放行、bashEnvFilter=false 整体关闭。
+  const keepEnv = new Set((cfg?.bashEnvKeep || []).map(String));
+  let childEnv = process.env;
+  if (cfg?.bashEnvFilter !== false) {
+    const filtered = /** @type {any} */ ({});
+    for (const [k, v] of Object.entries(process.env)) {
+      if (!isSensitiveEnv(k) || keepEnv.has(k)) filtered[k] = v;
+    }
+    childEnv = filtered;
+  }
 
   function run(/** @type {any} */ hook, /** @type {any} */ payload) {
     return new Promise((resolve) => {
       const child = spawn(hook.cmd, {
         shell: true,
         cwd: workingDir,
-        env: process.env,
+        env: childEnv,
         stdio: ['pipe', 'pipe', 'pipe'],
       });
       let out = '';

package/src/model-caps.js

@@ -3,6 +3,10 @@
 // 供预算推导、超时、工具截断统一引用——避免各层各自猜一份 128000 默认，
 // 本地小模型（窗口小/内存少）自动收紧预算与超时，不撑爆、不误杀。
 import { modelPreset } from './models.js';
+// 审计 P3-3（v0.4.2）：本地判定复用 fetch.js 的 isPrivateHost（IPv4 私网/回环/CGNAT/多播 + IPv6
+// fc00::/7、fe80::/10、::、::1、IPv4-mapped）——此前只查 IPv4 与 ::1，IPv6 本地模型被误判远程
+// （超时档位错），且与 fetch 工具/SSRF 判定各维护一份、口径漂移。
+import { isPrivateHost } from './tools/fetch.js';
 
 // 兜底：未知模型默认上下文窗口。本地小模型宁可保守（不撑爆）也不乐观。
 export const UNKNOWN_LOCAL_WINDOW = 32768;
@@ -20,14 +24,8 @@ export const EDGE_RATIO = 0.85;
 export function isLocalBaseUrl(/** @type {any} */ baseUrl) {
   try {
     const u = new URL(String(baseUrl || ''));
-    const h = u.hostname.toLowerCase();
-    if (!h) return false;
-    if (h === 'localhost' || h === '::1') return true;
-    const m = h.match(/^(\d{1,3})\.(\d{1,3})\.(\d{1,3})\.(\d{1,3})$/);
-    if (!m) return false;
-    const a = Number(m[1]);
-    const b = Number(m[2]);
-    return a === 10 || a === 127 || a === 0 || (a === 169 && b === 254) || (a === 172 && b >= 16 && b <= 31) || (a === 192 && b === 168);
+    if (!u.hostname) return false;
+    return isPrivateHost(u.hostname);
   } catch {
     return false;
   }

package/src/presets.js

@@ -78,8 +78,9 @@ export function validatePreset(/** @type {any} */ obj) {
  */
 export function listPresets(/** @type {any} */ workingDir) {
   ensureHome();
+  // 审计 P3-2（v0.4.2）：遮蔽 key 用预设的 name 字段而非文件名——此前两个不同文件名声明同名
+  // 预设会都被列出（违背「同名遮蔽」契约），且非法 JSON 在遮蔽判定时不可见。
   const seen = new Map();
-  const order = /** @type {string[]} */ ([]);
   for (const { dir, source } of presetLocations(workingDir)) {
     let files = [];
     try {
@@ -88,34 +89,31 @@ export function listPresets(/** @type {any} */ workingDir) {
       continue;
     }
     for (const f of files) {
-      if (seen.has(f)) continue;
-      seen.set(f, { dir, source });
-      order.push(f);
-    }
-  }
-  const out = [];
-  for (const f of order) {
-    const { dir, source } = seen.get(f);
-    try {
-      const obj = JSON.parse(fs.readFileSync(path.join(dir, f), 'utf8'));
+      const file = path.join(dir, f);
+      let obj;
+      try {
+        obj = JSON.parse(fs.readFileSync(file, 'utf8'));
+      } catch {
+        continue; // JSON 解析失败：跳过
+      }
       const v = validatePreset(obj);
       if (!v.ok) continue; // 非法预设跳过并静默（不阻塞会话）；diagnose 可查
-      out.push({
-        name: String(obj.name),
-        label: String(obj.label || obj.name),
-        description: String(obj.description || ''),
-        source,
-        file: path.join(dir, f),
-        ...(obj.systemPrompt ? { systemPrompt: obj.systemPrompt } : {}),
-        ...(Array.isArray(obj.tools) ? { tools: obj.tools } : {}),
-        ...(obj.permission ? { permission: String(obj.permission) } : {}),
-        ...(obj.model ? { model: String(obj.model) } : {}),
-      });
-    } catch {
-      // JSON 解析失败：跳过
+      const key = String(obj.name);
+      if (seen.has(key)) continue; // 同名遮蔽：项目 → 用户 → 内置（先发现者胜）
+      seen.set(key, { obj, source, file });
     }
   }
-  return out;
+  return [...seen.values()].map(({ obj, source, file }) => ({
+    name: String(obj.name),
+    label: String(obj.label || obj.name),
+    description: String(obj.description || ''),
+    source,
+    file,
+    ...(obj.systemPrompt ? { systemPrompt: obj.systemPrompt } : {}),
+    ...(Array.isArray(obj.tools) ? { tools: obj.tools } : {}),
+    ...(obj.permission ? { permission: String(obj.permission) } : {}),
+    ...(obj.model ? { model: String(obj.model) } : {}),
+  }));
 }
 
 /**

package/src/pricing.js

@@ -16,20 +16,23 @@ export const PRICE_DATA_AS_OF = '2026-08';
 // cfg.pricing.source 拉取（TTL 默认 7 天，cfg.pricing.ttlDays 可调）；TTL 内覆盖内置表，
 // 过期自动回退内置并置 stale 标记（/cost 与费用标签会提示）。
 function pricingFilePath() { return path.join(mingdaoHome(), 'pricing.json'); }
-/** @type {{ mtime: number, data: any, stale: boolean }} */
-let extCache = { mtime: -1, data: null, stale: false };
+/** @type {{ mtime: number, ttlDays: number, data: any, stale: boolean }} */
+let extCache = { mtime: -1, ttlDays: 7, data: null, stale: false };
 function externalPricing() {
   try {
+    // 审计 P2-2（v0.4.2）：此前直接读 tzCache.ttlDays——首次调用若早于 peakCfg()（isPeakHour），
+    // tzCache 还是初值（无 ttlDays），用户配的 pricing.ttlDays 被忽略按 7 天判过期且缓存整个进程寿命。
+    // 改走 peakCfg()（mtime 缓存，代价极低）；ttlDays 变化也触发重算（不依赖 pricing.json mtime 变化）。
+    const ttlDays = Number(peakCfg().ttlDays || 7);
     const f = pricingFilePath();
     const st = fs.statSync(f);
-    if (st.mtimeMs !== extCache.mtime) {
+    if (st.mtimeMs !== extCache.mtime || ttlDays !== extCache.ttlDays) {
       const d = JSON.parse(fs.readFileSync(f, 'utf8'));
-      const ttlDays = Number(tzCache.ttlDays ?? 7);
       const age = Date.now() - Number(d?.fetchedAt || 0);
-      extCache = { mtime: st.mtimeMs, data: d, stale: !Number.isFinite(age) || age > ttlDays * 86400000 };
+      extCache = { mtime: st.mtimeMs, ttlDays, data: d, stale: !Number.isFinite(age) || age > ttlDays * 86400000 };
     }
   } catch {
-    extCache = { mtime: -1, data: null, stale: false };
+    extCache = { mtime: -1, ttlDays: 7, data: null, stale: false };
   }
   return extCache;
 }
@@ -61,7 +64,7 @@ export async function refreshPricingFromSource(cfg) {
   const file = pricingFilePath();
   fs.mkdirSync(path.dirname(file), { recursive: true });
   fs.writeFileSync(file, JSON.stringify({ fetchedAt: Date.now(), source: src, models }, null, 2));
-  extCache = { mtime: -1, data: null, stale: false };
+  extCache = { mtime: -1, ttlDays: 7, data: null, stale: false };
   const names = Object.keys(models).join('、');
   return { ok: true, lines: ['✓ 价格表已刷新（' + names + '），TTL 内费用估算/护栏/避峰自动跟随'] };
 }

package/src/redact.js

@@ -21,6 +21,10 @@ export function redactSensitive(/** @type {any} */ text) {
   s = s.replace(/\b(?:10|127)(?:\.\d{1,3}){3}\b|\b192\.168(?:\.\d{1,3}){2}\b|\b169\.254(?:\.\d{1,3}){2}\b|\b172\.(?:1[6-9]|2\d|3[01])(?:\.\d{1,3}){2}\b|\b100\.(?:6[4-9]|[7-9]\d|1[01]\d|12[0-7])(?:\.\d{1,3}){2}\b/g, '[私网IP]');
   s = s.replace(/(?:fe80:[\da-f:]+|::1|::)/gi, '[链路本地/回环IPv6]');
   const home = os.homedir();
-  if (home && home.length > 1) s = s.split(home).join('~');
+  // 审计 P3-1（v0.4.2）：split().join() 无边界——/home/user2/xxx 会被误脱敏为 ~2/xxx。
+  // 正则要求家目录后跟路径分隔符或字符串结尾（如 /home/user2 的 2 紧跟目录名不匹配）。
+  if (home && home.length > 1) {
+    s = s.replace(new RegExp(home.replace(/[.*+?^${}()|[\]\\]/g, '\\$&') + '(?=/|$)', 'g'), '~');
+  }
   return s;
 }

package/src/skill-lib.js

@@ -13,9 +13,11 @@ import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
 import crypto from 'node:crypto';
-import { spawnSync } from 'node:child_process';
+import { spawn } from 'node:child_process';
+import { lookup } from 'node:dns/promises';
 import { fileURLToPath } from 'node:url';
 import { mingdaoHome, ensureHome } from './config.js';
+import { isPrivateHost } from './tools/fetch.js';
 
 const LIB_DIR = fileURLToPath(new URL('../skills-lib', import.meta.url));
 
@@ -250,7 +252,7 @@ export function installFromDir(dir) {
 /**
  * @param {any} url
  */
-export async function installFromUrl(url) {
+export async function installFromUrl(url, { allowPrivate = false } = {}) {
   let u;
   try {
     u = new URL(url);
@@ -264,7 +266,41 @@ export async function installFromUrl(url) {
   const timer = setTimeout(() => ctrl.abort(), 30000);
   let text;
   try {
-    const res = await fetch(u, { signal: ctrl.signal, redirect: 'follow' });
+    // 审计 P2-1（v0.4.2）：SSRF 防护——与 fetch 工具/validateRemoteUrl 同口径：
+    // 初始与每一跳重定向都做私网/回环字面量判定 + DNS 复检（防域名重绑定），跳数上限 5。
+    // allowPrivate（CLI 显式输入 URL 时开启）：本地用户自担意图，内网地址可安装；WebUI 默认拦截。
+    let cur = u;
+    let res = /** @type {any} */ (null);
+    for (let hop = 0; hop <= 5; hop++) {
+      const ch = String(cur.hostname || '').toLowerCase();
+      let blocked = !allowPrivate && isPrivateHost(ch);
+      if (!blocked && ch && ch !== 'localhost' && !/^\d{1,3}(\.\d{1,3}){3}$/.test(ch)) {
+        try {
+          const addrs = await lookup(ch, { all: true, verbatim: true });
+          blocked = !allowPrivate && addrs.some((/** @type {any} */ a) => isPrivateHost(a.address));
+        } catch {
+          // DNS 解析失败：放行，连接阶段会报错
+        }
+      }
+      if (blocked) return { error: `拒绝访问内网/本机地址（${ch}）——SSRF 防护。` };
+      res = await fetch(cur, { signal: ctrl.signal, redirect: 'manual' });
+      if (res.status >= 300 && res.status < 400) {
+        if (hop >= 5) return { error: '重定向次数超过上限（5 跳）。' };
+        const loc = res.headers.get('location');
+        if (!loc) break;
+        try {
+          cur = new URL(loc, cur);
+        } catch {
+          return { error: `非法重定向地址：${loc}` };
+        }
+        if (cur.protocol !== 'http:' && cur.protocol !== 'https:') {
+          return { error: '重定向到非 http(s) 地址，已拒绝。' };
+        }
+        continue;
+      }
+      break;
+    }
+    if (!res) return { error: '下载失败：无响应' };
     if (!res.ok) return { error: `下载失败：HTTP ${res.status}` };
     text = await res.text();
     if (text.length > 512 * 1024) return { error: 'SKILL.md 超过 512KB 上限' };
@@ -290,23 +326,34 @@ export async function installFromUrl(url) {
   return r;
 }
 
+/** 异步 spawn（审计 P1-3）：child_process.spawn + Promise，返回 { error?, code, signal }。 */
+function runSpawn(/** @type {string} */ cmd, /** @type {string[]} */ args, /** @type {{ timeoutMs?: number }} */ { timeoutMs } = {}) {
+  return new Promise((resolve) => {
+    const child = spawn(cmd, args, { stdio: 'ignore', timeout: timeoutMs });
+    child.on('error', (/** @type {any} */ err) => resolve({ error: err }));
+    child.on('close', (code, signal) => resolve({ code, signal }));
+  });
+}
+
 /**
  * @param {any} gitUrl
  */
-export function installFromGit(gitUrl) {
+export async function installFromGit(gitUrl) {
   if (typeof gitUrl !== 'string' || gitUrl.trim().startsWith('-')) {
     return { error: 'git 地址不能以 - 开头（防选项注入）' };
   }
-  const check = spawnSync('git', ['--version'], { stdio: 'ignore' });
-  if (check.error || check.status !== 0) {
+  // 审计 P1-3（v0.4.2）：spawnSync 最长阻塞 120s 冻结整个 Node 事件循环（WebUI 全部并发会话/
+  // 权限确认/SSE 流无响应）。改异步 spawn，与 v0.4.1 mountConfigTools 修复同口径。
+  const check = await runSpawn('git', ['--version']);
+  if (check.error || check.code !== 0) {
     return { error: '未找到 git（git 仓库安装需要系统 git，可用 URL 安装单文件技能）' };
   }
   const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'mingdao-skill-git-'));
   // -- 分隔符：gitUrl 即使形似选项也只按路径处理
-  const r = spawnSync('git', ['clone', '--depth', '1', '--', gitUrl, tmp], { stdio: 'ignore', timeout: 120000 });
-  if (r.error || r.status !== 0) {
+  const r = await runSpawn('git', ['clone', '--depth', '1', '--', gitUrl, tmp], { timeoutMs: 120000 });
+  if (r.error || r.code !== 0) {
     fs.rmSync(tmp, { recursive: true, force: true });
-    return { error: `git clone 失败：${r.error?.message || `退出码 ${r.status}`}` };
+    return { error: `git clone 失败：${r.error?.message || (r.signal ? `超时/被终止（${r.signal}）` : `退出码 ${r.code}`)}` };
   }
   const found = [];
   const stack = [tmp];
@@ -388,15 +435,16 @@ export async function reinstallSkill(name) {
 // 统一入口：自动识别 库名 | 本地目录 | SKILL.md URL | git 仓库
 /**
  * @param {any} arg
+ * @param {{ allowPrivateUrl?: boolean }} [opts] allowPrivateUrl=true 时 URL 安装放行内网地址（CLI 显式输入场景）
  */
-export async function installSkill(arg) {
+export async function installSkill(arg, opts = {}) {
   const a = String(arg || '').trim();
   if (!a) return { error: '缺少参数：mingdao skill install <库名|目录|SKILL.md URL|git 仓库地址>' };
   const libHit = libraryList().find((s) => s.name === a);
   if (libHit) return installFromLibrary(a);
   if (fs.existsSync(path.resolve(a))) return installFromDir(a);
   if (/^https?:\/\//i.test(a)) {
-    if (/\.md(#.*)?$/i.test(a.split('?')[0])) return installFromUrl(a);
+    if (/\.md(#.*)?$/i.test(a.split('?')[0])) return installFromUrl(a, { allowPrivate: opts.allowPrivateUrl === true });
     return installFromGit(a);
   }
   if (/^git@/.test(a)) return installFromGit(a);

package/src/sync-server.js

@@ -347,10 +347,10 @@ function doPush(username, body) {
   const dir = sessionsDir(username);
   fs.mkdirSync(dir, { recursive: true, mode: 0o700 });
   const file = path.join(dir, name);
-  const tmp = file + '.tmp';
-  fs.writeFileSync(tmp, content, { mode: 0o600 });
-  fs.renameSync(tmp, file);
+  // 审计 P1-2（v0.4.2）：内容落盘必须在锁内且原子——此前固定 .tmp 名 + 锁外写，
+  // 并发同名 push 互相覆盖 tmp、rename 竞态，内容与 meta size 可能错位。
   return withWriteLock(() => {
+    atomicWriteFileSync(file, content, { mode: 0o600 });
     const meta = readJson(metaFile(username), {});
     meta[name] = { mtime: Date.now(), size: Buffer.byteLength(content) };
     writeJson(metaFile(username), meta);

package/src/tools/bash.js

@@ -15,7 +15,8 @@ const MAX_TIMEOUT_SECONDS = 600;
 // 整体关闭（回到完全透传）。
 const SENSITIVE_ENV_PAIR = /(api[_-]?key|access[_-]?key|client[_-]?secret|private[_-]?key)/i;
 const SENSITIVE_ENV_SEGMENT = /(^|_)(token|secret|password|passwd|credential|authorization|auth)(_|$)/i;
-const isSensitiveEnv = (/** @type {any} */ k) => SENSITIVE_ENV_PAIR.test(k) || SENSITIVE_ENV_SEGMENT.test(k);
+// 审计 P3-6（v0.4.2）：导出供 hooks.js 复用——hook 子进程 env 与 bash 工具同口径过滤敏感变量。
+export const isSensitiveEnv = (/** @type {any} */ k) => SENSITIVE_ENV_PAIR.test(k) || SENSITIVE_ENV_SEGMENT.test(k);
 
 function buildChildEnv(/** @type {any} */ ctx, /** @type {any} */ filterSensitive) {
   if (!filterSensitive) return process.env;
@@ -87,7 +88,9 @@ export function runBash(/** @type {any} */ args, /** @type {any} */ ctx) {
   if (!command.trim()) return { ok: false, error: 'command 参数为空。' };
   const timeoutSec = Math.min(Number(args.timeout) || 120, MAX_TIMEOUT_SECONDS);
   // 配置优先：模型不能通过传 sandbox:'off' 自行降级（配置里选了 safe/readonly 就必须沙箱）
-  const mode = String(ctx?.cfg?.sandbox ?? args.sandbox ?? 'off');
+  // 审计 P3-9（v0.4.2）：cfg.sandbox=''（空串）时 ?? 不触发，mode 为空串落入 readonly 沙箱分支——
+  // || 'off' 归一化空串回默认 off。
+  const mode = String(ctx?.cfg?.sandbox ?? args.sandbox ?? 'off') || 'off';
   const shell = process.platform === 'win32' ? 'cmd.exe' : '/bin/bash';
   const shellArgs = process.platform === 'win32' ? ['/d', '/s', '/c', command] : ['-lc', command];
 

package/src/tools/fetch.js

@@ -2,7 +2,9 @@
 // SSRF 防护：字面量私网/回环拒绝 + DNS 解析复检（防域名重绑定），口径与 server.js validateRemoteUrl 一致。
 import { lookup } from 'node:dns/promises';
 
-function isPrivateHost(/** @type {string} */ hostname) {
+// 审计 4.1（v0.4.2）：isPrivateHost 导出复用——skill-lib.installFromUrl 等外联入口共用同一判定，
+// 避免「漏一处」的 SSRF 防护缺口（此前 fetch 工具与 server.validateRemoteUrl 各自维护一份）。
+export function isPrivateHost(/** @type {string} */ hostname) {
   let h = String(hostname || '').toLowerCase();
   if (!h) return true;
   h = h.replace(/^\[|\]$/g, '');

package/src/web/app.js

@@ -398,7 +398,12 @@ async function send(){
       else if(ev.type==='error'){ console.log('[MingDao] error 事件：' + ev.message); onActivity(); const d=document.createElement('div'); d.className='errline'; d.textContent=ev.message; msg.appendChild(d); scroll(); }
       else if(ev.type==='done'){ console.log('[MingDao] done 事件：session=' + ev.session); onActivity(); if(ev.budget){ const b=ev.budget; if(hintTextEl) hintTextEl.textContent='预算 '+Math.round(b.used/1000)+'K/'+Math.round(b.total/1000)+'K（'+Math.round(b.used/b.total*100)+'%）· 本轮完成 · 提示栏右侧为今日费用与命中率'; } refreshStatusBar(); if(ev.stats&&ev.stats.deliverables&&ev.stats.deliverables.length){ const card=document.createElement('div'); card.className='deliver'; card.innerHTML='<div class="t">📦 交付物（'+ev.stats.deliverables.length+' 个文件）</div>'+ev.stats.deliverables.map(f=>'<div class="i">'+esc(f)+(f.toLowerCase().endsWith('.html')?' <span style="color:var(--accent2)">— 浏览器打开即可运行</span>':'')+'</div>').join(''); msg.appendChild(card); } if(ev.note){ const d=document.createElement('div'); d.className='errline'; d.style.color='var(--warn)'; d.textContent=ev.note; msg.appendChild(d); } currentSession=ev.session; update(); refreshSessions(); updateTasksPanel(); }
     });
-  }catch(e){ onActivity(); const d=document.createElement('div'); d.className='errline'; d.textContent=(e&&e.name==='AbortError')?(killedByWatchdog?'响应超时已中断（120 秒无任何响应），请重试':'已中断'):(e&&e.message)||'网络错误'; msg.appendChild(d); scroll(); }
+  }catch(e){ onActivity(); // 诊断（v0.4.2 network error 排查）：静默中断此前无任何日志，无法区分
+    // 「手点停止 / 看门狗 / fetch 流静默断裂」。补记完整错误形态 + 回合状态，下次失败可定位。
+    console.error('[MingDao] chat 流异常', { name: e && e.name, message: e && e.message, err: String(e), taskId, rawLen: raw.length, steps: stepsCount, elapsedS: Math.round((Date.now() - workT0) / 1000) });
+    const d=document.createElement('div'); d.className='errline';
+    d.textContent=(e&&e.name==='AbortError')?(killedByWatchdog?'响应超时已中断（120 秒无任何响应），请重试':'已中断'):((e&&e.message)||('连接中断，本轮未完成。已执行的工作已保存检查点——直接发送「继续」即可从断点续跑（' + Math.round((Date.now() - workT0) / 1000) + 's · ' + stepsCount + ' 步）。'));
+    msg.appendChild(d); scroll(); }
   finally{ clearInterval(hintTimer); disarm(); generating=false; setBtn(); curTaskId=null; curPhase='模型推理中'; console.log('[MingDao] 回合收尾：generating=false，按钮恢复发送'); hintEl.classList.remove('working'); if(hintTextEl) hintTextEl.textContent=defaultHint; pending=false; content.innerHTML=renderMarkdown(raw); attachTrajMeta(msg); activeAiMsg=null; curWorkT0=0; renderWorkStatus(); scroll(); updateTasksPanel(); }
 }
 

package/src/web/server.js

@@ -516,6 +516,11 @@ export async function runWebServer({ host = '127.0.0.1', port = 3820, authToken
     });
 
     res.on('close', () => {
+      // 诊断（v0.4.2 network error 排查）：区分「服务端正常收尾」与「客户端中途断开」——
+      // writableEnded=false 且任务仍在跑 = 浏览器/渲染层静默断连（此前无任何日志，根因不可见）。
+      if (!res.writableEnded && entry.status === 'running') {
+        srvlog('chat 客户端断连 ' + taskId + ' 已跑=' + Math.round((Date.now() - entry.startedAt) / 1000) + 's status=' + entry.status + ' writableEnded=' + res.writableEnded);
+      }
       // 浏览器断开：中止正在跑的生成（否则白白烧 token），挂起的权限确认按拒绝处理
       if (entry.pendingAsk) {
         entry.pendingAsk.resolve('');