mingdao-harness 0.6.2 → 0.6.4

package/src/web/routes/domains/sessions.js

@@ -50,7 +50,7 @@ export async function handle({ req, res, method, p, url }, deps, shared) {
       const ws = workspaceForDir(wsDir);
       if (wsDir && path.resolve(wsDir) !== path.resolve(state.workingDir)) {
         state.workingDir = wsDir; // 审计 P2-5：仅在目录不同时才切换，减少全局状态抖动
-        if (ws) touchWorkspace(ws.name);
+        if (ws) await touchWorkspace(ws.name);
       }
       json(res, 200, {
         ok: true,
@@ -74,13 +74,13 @@ export async function handle({ req, res, method, p, url }, deps, shared) {
       const title = sanitizeTitle(String(body.title || '会话'));
       const renamed = renameSessionFile(fs, path, home, { file: full }, title);
       if (!renamed) return json(res, 500, { error: '重命名失败（可能存在同名会话）' });
-      moveSessionWorkspace(file, path.basename(renamed));
+      await moveSessionWorkspace(file, path.basename(renamed));
       return json(res, 200, { ok: true, file: path.basename(renamed) });
     }
     if (body.action === 'delete') {
       try {
         fs.unlinkSync(full);
-        removeSessionWorkspace(file);
+        await removeSessionWorkspace(file);
         return json(res, 200, { ok: true });
       } catch (/** @type {any} */ err) {
         return json(res, 500, { error: String(err?.message || err) });

package/src/web/routes/domains/sync.js

@@ -52,7 +52,8 @@ export async function handle({ req, res, method, p, url }, deps, shared) {
     if (body.action === 'push') {
       const r = /** @type {any} */ (await syncPush(body.name));
       if (r.error) return json(res, 400, { error: r.error });
-      return json(res, 200, { ok: true, pushed: r.pushed.length, conflicts: r.conflicts });
+      // v0.6.3（审计 H-2）：把"上传前已脱敏"的会话数透给前端，避免用户以为远端是原文
+      return json(res, 200, { ok: true, pushed: r.pushed.length, conflicts: r.conflicts, redacted: (r.redacted || []).length });
     }
     if (body.action === 'pull') {
       const r = /** @type {any} */ (await syncPull(body.name));

package/src/web/routes/domains/workspace.js

@@ -36,13 +36,39 @@ export async function handle({ req, res, method, p, url }, deps, shared) {
     os.tmpdir(),
   ];
   const normPath = (/** @type {string} */ x) => (process.platform === 'win32' ? x.toLowerCase() : x);
+
+  // v0.6.3（P0-3）：围栏必须按**真实路径**判定，否则一个符号链接就够了。
+  // 原实现只用 path.resolve（纯字符串消解 `..`）做前缀比较，而后续 statSync/readdirSync 会
+  // **跟随符号链接**：`ln -s / /tmp/root` 之后 `?dir=/tmp/root` 通过前缀检查、stat 落到 `/`，
+  // 于是任意目录枚举；再用 `POST /api/workspaces {dir:'/tmp/root'}` 登记，Agent 的
+  // bash/write/edit 就整体跑到围栏之外。`os.tmpdir()` 被无条件算作允许根，门槛更低。
+  //
+  // 两处都归一化才有意义：目标 realpath + 根 realpath（macOS 的 /Users、Linux 的 /home
+  // 常是符号链接，只归一化一边会把合法目录误判越界）。
+  /** 归一化到真实路径；目标不存在时回退到「最近的存在祖先」的 realpath 再接回剩余段 */
+  const realpathDeep = (/** @type {string} */ p0) => {
+    let cur = path.resolve(String(p0));
+    /** @type {string[]} */
+    const rest = [];
+    for (;;) {
+      try {
+        const real = fs.realpathSync(cur);
+        return rest.length ? path.join(real, ...rest.slice().reverse()) : real;
+      } catch {
+        const parent = path.dirname(cur);
+        if (parent === cur) return path.resolve(String(p0)); // 连根都读不到：退回原路径
+        rest.push(path.basename(cur));
+        cur = parent;
+      }
+    }
+  };
   const allowedRoots = () =>
     [...baseRoots, startupCwd, state.workingDir, ...(Array.isArray(cfg?.web?.browseRoots) ? cfg.web.browseRoots : [])]
       .filter(Boolean)
-      .map((r) => normPath(path.resolve(String(r))));
-  /** 目标目录是否落在允许根内（含根自身） */
+      .map((r) => normPath(realpathDeep(String(r))));
+  /** 目标目录是否落在允许根内（含根自身）——按 realpath 比较，符号链接逃逸会在这里被拒 */
   const withinAllowed = (/** @type {string} */ dir) => {
-    const d = normPath(dir);
+    const d = normPath(realpathDeep(dir));
     return allowedRoots().some((r) => d === r || d.startsWith(r + path.sep));
   };
   // 显式放开（默认 false）：确需登记家目录之外的位置（外置卷/网络盘）时由用户显式开启
@@ -58,7 +84,7 @@ export async function handle({ req, res, method, p, url }, deps, shared) {
     const name = String(body.name || '').trim();
     if (body.action === 'add') {
       if (!name) return json(res, 400, { error: '名称不能为空' });
-      const target = path.resolve(body.dir || state.workingDir);
+      const target = realpathDeep(path.resolve(body.dir || state.workingDir)); // P0-3：登记前先消解符号链接
       // v0.4.7（T1）：登记工作空间 == 授权它可被目录浏览（fs-browse 的基目录含 state.workingDir）。
       // 若允许登记任意绝对路径，围栏就能被「先 add 再 set」一步自行解除（登记 / 即可枚举全盘）。
       // 因此登记与自动建目录都限定在允许根内；需要家目录之外的位置请显式配置
@@ -76,12 +102,12 @@ export async function handle({ req, res, method, p, url }, deps, shared) {
           return json(res, 400, { error: `无法创建目录：${target}（${e.message}）` });
         }
       }
-      const r = addWorkspace(name, target);
+      const r = await addWorkspace(name, target);
       if (r.error) return json(res, 400, { error: r.error });
       return json(res, 200, { ok: true, name: r.name, dir: r.dir, created: body.create !== false });
     }
     if (body.action === 'rename') {
-      const r = renameWorkspace(name, body.newName);
+      const r = await renameWorkspace(name, body.newName);
       if (r.error) return json(res, 400, { error: r.error });
       return json(res, 200, { ok: true, name: r.name });
     }
@@ -90,13 +116,13 @@ export async function handle({ req, res, method, p, url }, deps, shared) {
       // 携带 file 时同时把当前会话的工作空间切过去（P3-4：会话跟随显式切换）。
       if (body.dir) {
         // v0.4.7（T1）：改目录同样受允许根约束——否则「先登记一个合法目录，再 set 到 /」即可绕过上面的闸门
-        const t2 = path.resolve(String(body.dir));
+        const t2 = realpathDeep(path.resolve(String(body.dir))); // P0-3：同上
         if (!allowAnyDir && !withinAllowed(t2)) {
           return json(res, 400, {
             error: `目录 ${t2} 不在允许范围内（家目录 / 启动目录 / 当前工作目录 / web.browseRoots）。确需切换请配置 web.allowAnyWorkspaceDir: true。`,
           });
         }
-        const r = setWorkspaceDir(name, body.dir);
+        const r = await setWorkspaceDir(name, body.dir);
         if (r.error) return json(res, 400, { error: r.error });
       }
       const dir = workspacePath(name);
@@ -106,15 +132,15 @@ export async function handle({ req, res, method, p, url }, deps, shared) {
       } catch (/** @type {any} */ e) {
         return json(res, 400, { error: `无法创建目录：${dir}（${e.message}）` });
       }
-      touchWorkspace(name);
+      await touchWorkspace(name);
       state.workingDir = dir;
       // 不再 process.chdir：运行中任务的 cwd 在创建时已固定，全局切换只影响新会话
-      if (body.file) setSessionWorkspace(String(body.file), dir, /** @type {any} */ (name));
+      if (body.file) await setSessionWorkspace(String(body.file), dir, /** @type {any} */ (name));
       return json(res, 200, { ok: true, name, dir, current: name });
     }
     if (body.action === 'remove') {
       if (!name) return json(res, 400, { error: '缺少名称' });
-      const rm = removeWorkspace(name);
+      const rm = await removeWorkspace(name);
       if (rm && /** @type {any} */ (rm).error) return json(res, 500, { error: /** @type {any} */ (rm).error });
       return json(res, 200, { ok: rm === true });
     }
@@ -128,7 +154,7 @@ export async function handle({ req, res, method, p, url }, deps, shared) {
     if (!path.isAbsolute(dir)) return json(res, 400, { error: '需要绝对路径' });
     // 评估 6.1（v0.4.3）：先 path.resolve 消解 .. 段，再做前缀比较与 stat/readdir——此前字符串
     // 前缀比较用未规范化的 dir，`/home/u/../../etc` 能通过 startsWith('/home/u/') 但 stat 解析到 /etc。
-    dir = path.resolve(dir);
+    dir = realpathDeep(dir); // v0.6.3（P0-3）：消解符号链接——`path.resolve` 只消解 `..`，挡不住 `ln -s /`
     // 质检 A3：目录浏览限定基目录，拒绝越界。Windows（CodeArts 报告）：家目录覆盖整个用户配置树
     // （AppData 等）——收紧为 桌面/文档/下载 三常用目录 + 启动目录 + 工作目录 + web.browseRoots 显式授权；
     // 路径比较在 win32 下大小写归一（D:\\ vs d:\\ 不再误拒）。
@@ -144,13 +170,17 @@ export async function handle({ req, res, method, p, url }, deps, shared) {
       if (!st.isDirectory()) return json(res, 400, { error: '不是目录' });
       const entries = fs
         .readdirSync(dir, { withFileTypes: true })
-        .filter((e) => e.isDirectory() && !e.name.startsWith('.'))
+        // P0-3：符号链接条目一律不列出。**如实说明**：`Dirent.isDirectory()` 走的是 lstat 语义，
+        // 对符号链接本就返回 false，所以这一条 `!e.isSymbolicLink()` 是双保险而**不是**主要防线
+        // （实测：去掉它也列不出链接条目）。真正的防线是上面的 realpath 判定——
+        // 即便有人把链接路径直接喂进来，也会因为 realpath 落在允许根之外而被 403。
+        .filter((e) => e.isDirectory() && !e.isSymbolicLink() && !e.name.startsWith('.'))
         .map((e) => ({ name: e.name, path: path.join(dir, e.name) }))
         .sort((a, b) => a.name.localeCompare(b.name, 'zh-CN'))
         .slice(0, 300);
       let parent = path.dirname(dir);
-      // 父目录同样不得越出基目录
-      if (!browseRoots.some((r) => norm(parent) === r || norm(parent).startsWith(r + path.sep))) parent = /** @type {any} */ (null);
+      // 父目录同样不得越出基目录（与上面的判定共用同一个 realpath 版 withinAllowed，避免两处漂移）
+      if (!withinAllowed(parent)) parent = /** @type {any} */ (null);
       json(res, 200, { ok: true, path: dir, parent: parent === dir ? null : parent, entries });
     } catch (/** @type {any} */ err) {
       json(res, 400, { error: String(err?.message || err) });

package/src/web/server.js

@@ -22,14 +22,14 @@ import { createApiDispatch } from './routes/api.js';
 import { ensureHome, loadConfig, saveConfig, mingdaoHome } from '../config.js';
 import { setStoredKey, removeStoredKey, getStoredKey, maskKey } from '../credentials.js';
 import { availableModels, fetchProviderModels, providerHasKey } from '../model-discovery.js';
-import { createProvider, resolveProviderConfig, helperProvider } from '../providers/index.js';
+import { createProvider, resolveProviderConfig, helperProvider, resolveVisionSupport } from '../providers/index.js';
 import { MODELS, modelPreset, PROVIDERS } from '../models.js';
 import { routeTask, routingConfig } from '../routing.js';
 import { buildUserContent } from './attachments.js';
 import { MAX_CONCURRENT, SECURITY_HEADERS } from './constants.js';
 import { createAgent } from '../agent.js';
 import { createPermission } from '../permissions.js';
-import { isPrivateHost as sharedIsPrivateHost } from '../tools/fetch.js';
+import { isPrivateHost as sharedIsPrivateHost, isMetadataHost } from '../tools/fetch.js';
 import { buildSystemPrompt } from '../prompts.js';
 import { loadProjectMemory, loadProjectMemoryEntries, retrieveRelevant, extractAndAppendProjectMemory } from '../memory.js';
 import { saveTaskStateMerge, clearTaskState, loadTaskState, resumePrompt, checkpointHint } from '../task-state.js';
@@ -315,6 +315,11 @@ export async function runWebServer({ host = '127.0.0.1', port = 3820, authToken,
       return { error: '地址必须是合法的 http(s) URL' };
     }
     if (u.protocol !== 'http:' && u.protocol !== 'https:') return { error: '仅支持 http/https 地址' };
+    // v0.6.3（P1-8）：云元数据端点**无条件拒绝**——它不在"本机可信模型服务"那一类里，
+    // 因此回环绑定与 web.allowPrivateEndpoints 都不放行（原实现把整段校验都跳过了）。
+    if (isMetadataHost(String(u.hostname || ''))) {
+      return { error: `拒绝访问云元数据端点（${u.hostname}）——SSRF 防护，此地址无任何放行开关` };
+    }
     // 威胁模型：本机回环绑定时（默认/桌面版），本机模型服务（如 Ollama）属可信场景，放行；
     // 对外监听（0.0.0.0）时拒绝私网目标——除非显式 allowPrivateEndpoints。
     // 域名目标：DNS 解析后逐条校验（2026-09-02 加固，防 DNS 重绑定把域名指向内网绕过字面量检查）。
@@ -393,7 +398,11 @@ export async function runWebServer({ host = '127.0.0.1', port = 3820, authToken,
     }, 5000);
     const userMessage = String(body.message ?? '').trim();
     entry.message = (userMessage || '[附件]').slice(0, 40);
-    const visionSupported = Boolean(modelPreset(modelName)?.supportsVision || cfg.customModels?.[modelName]?.vision);
+    // v0.6.3（下游 Dify 工作流反馈）：门控必须**也咨询自定义 Provider**。
+    // 原式只看内置预设与 customModels，于是自定义 Provider（Dify/网关适配器）永远被判不支持图片；
+    // 而为了让门控通过去写 customModels，又会把 provider 解析劫持到 openai-compatible 直连
+    // （见 providers/index.js 的 isEndpointDeclaration 注释）。现在统一走 resolveVisionSupport。
+    const visionSupported = await resolveVisionSupport(cfg, modelName);
     const built = buildUserContent(userMessage, body.attachments, visionSupported);
     if (built.error) {
       entry.status = 'failed';
@@ -479,7 +488,7 @@ export async function runWebServer({ host = '127.0.0.1', port = 3820, authToken,
     const sessionWsDir = getSessionWorkspace(sessionName);
     const taskDir = sessionWsDir || workingDir;
     if (!sessionWsDir) {
-      setSessionWorkspace(sessionName, taskDir, currentWorkspace(workingDir)?.name || null);
+      await setSessionWorkspace(sessionName, taskDir, currentWorkspace(workingDir)?.name || null);
     }
     // v0.3.0 P0-3：项目记忆会话内快照——本会话全程用同一份快照（前缀稳定），
     // 每轮结束提取的新条目只写文件、不回灌当前会话。
@@ -670,7 +679,7 @@ export async function runWebServer({ host = '127.0.0.1', port = 3820, authToken,
             const oldName = path.basename(session.file);
             const renamed = renameSessionFile(fs, path, home, session, title);
             if (renamed) {
-              moveSessionWorkspace(oldName, path.basename(renamed)); // 会话改名 → 工作空间映射跟随
+              await moveSessionWorkspace(oldName, path.basename(renamed)); // 会话改名 → 工作空间映射跟随
               claimSessionKey(session.file); // 忙锁键同步迁移，否则新文件名发起的回合不会被判「忙」
             }
           }
@@ -796,6 +805,14 @@ export async function runWebServer({ host = '127.0.0.1', port = 3820, authToken,
       // 质检 L6：实际 trustedHost 白名单只放行回环 Host（远程浏览器带 LAN IP 的 Host 会 403），
       // 文案按真实行为修正——非浏览器客户端/伪造 Host 仍可达，务必启用令牌
       console.warn('  ⚠ 警告：当前监听 ' + host + '（非本机回环）且未启用令牌，浏览器直接访问会被拒绝（Host 白名单），但非浏览器客户端仍可直连。强烈建议：mingdao web --auth-token <令牌>。');
+    } else {
+      // v0.6.3（H-5）：回环 + 未配置令牌时**把"本机信任"这个决定说出来**。
+      // 静默无认证的危险不在于它一定被利用，而在于用户不知道自己处在这个模式里：
+      //   · 跨站浏览器请求（含 <img>/no-cors 盲打）已在 api.js 用 Sec-Fetch-Site 一律拒绝；
+      //   · 但**同机的其它进程/其它用户**仍可直连 API（多用户机器、共享 CI runner 上尤其要注意）。
+      // 一句话给出两个可选动作，而不是替用户决定（改默认会打断"浏览器直接打开 127.0.0.1"的日常用法）。
+      console.log('  ℹ 本机信任模式：未配置访问令牌，本机其它进程可直接访问 /api/*。');
+      console.log('    多用户机器 / 共享 CI 上建议启用令牌：mingdao web --auth-token <令牌>，或 config.json 的 web.token。');
     }
     console.log(`  模型: ${modelName} · 权限: ${cfg.permission ?? 'ask'} · 工作目录: ${workingDir}`);
     if (cfg.mcpServers && Object.keys(cfg.mcpServers).length) console.log('  MCP:  后台连接中，/api/state 可查看状态');

package/src/workspace.js

@@ -1,3 +1,16 @@
+// v0.6.2（自评 P2-7 阻塞面）：本模块**全部**加锁函数已迁到异步版 `withFileLock`。
+//
+// 为什么先迁这一组：它正好在 WebUI 的请求路径上（开会话设目录、切换工作空间、会话改名），
+// 而同步锁等待期间 `Atomics.wait` 会让**整个事件循环停摆**——实测持锁方存活 2.6 秒时，
+// 一个 100ms 的定时器在锁返回前根本没触发，表现为所有并发会话/权限确认/SSE 流一起卡住。
+// 且这组函数的 11 个调用点**全部**已在 async 处理函数里（web 路由 / 命令处理 / 会话改名），
+// 迁移只是加 `await`，不产生 async 传染。
+//
+// 仍用同步版的调用点（tasks 2 处 / schedule 12 处 / cachestats 1 处 / sync 1 处）见
+// docs/AUDIT-v0.6.1-第三方报告登记.md §3.26 的说明：它们要么在纯同步读-改-写链里
+// （强行异步化会传染 24 个调用点，回归风险大于收益），要么在调度守护进程内部
+// （阻塞只会推迟定时任务，不会冻结用户请求）。
+//
 // 工作空间：项目目录注册表（参考 WorkBuddy 的项目组织思路）——
 // 为常做的项目登记名称与目录，一键回到对应项目，配置与记忆随目录（AGENTS.md / .mingdao/skills 等）自然跟随。
 // 注册表：<mingdao-home>/workspaces.json → { "<名称>": { dir, createdAt, lastUsed } }
@@ -6,7 +19,7 @@
 import fs from 'node:fs';
 import path from 'node:path';
 import { mingdaoHome, ensureHome } from './config.js';
-import { atomicWriteFileSync, withFileLockSync } from './atomic-write.js';
+import { atomicWriteFileSync, atomicWritePrivateSync, withFileLock } from './atomic-write.js';
 
 export function workspacesFile() {
   return path.join(mingdaoHome(), 'workspaces.json');
@@ -33,14 +46,14 @@ export function saveWorkspaces(/** @type {any} */ ws) {
   try {
     ensureHome();
     // 原子写（评估 6.6）：随机 tmp 名 + rename，避免崩溃冲空与跨进程共名 tmp 串扰
-    atomicWriteFileSync(workspacesFile(), JSON.stringify(ws, null, 2) + '\n');
+    atomicWritePrivateSync(workspacesFile(), JSON.stringify(ws, null, 2) + '\n');
     return { ok: true, error: null };
   } catch (err) {
     return { ok: false, error: String(/** @type {any} */ (err)?.message ?? err) };
   }
 }
 
-export function addWorkspace(/** @type {any} */ name, /** @type {any} */ dir) {
+export async function addWorkspace(/** @type {any} */ name, /** @type {any} */ dir) {
   const key = String(name).trim();
   if (!key) return { error: '名称不能为空' };
   if (/[\\/]/.test(key)) return { error: '名称不能包含路径分隔符' };
@@ -48,7 +61,7 @@ export function addWorkspace(/** @type {any} */ name, /** @type {any} */ dir) {
   if (!fs.existsSync(target)) return { error: `目录不存在：${target}` };
   // v0.4.7（T19）：读-改-写必须在跨进程锁内——WebUI 每次建会话都会 touch 工作空间，
   // 与 CLI 的 add/remove 并发时未加锁会丢更新（注册表少一条工作空间）。
-  return withFileLockSync(workspacesFile() + '.lock', () => {
+  return withFileLock(workspacesFile() + '.lock', () => {
     const ws = loadWorkspaces();
     ws[key] = { dir: target, createdAt: ws[key]?.createdAt || Date.now(), lastUsed: Date.now() };
     const saved = saveWorkspaces(ws);
@@ -57,8 +70,8 @@ export function addWorkspace(/** @type {any} */ name, /** @type {any} */ dir) {
   });
 }
 
-export function removeWorkspace(/** @type {any} */ name) {
-  return withFileLockSync(workspacesFile() + '.lock', () => {
+export async function removeWorkspace(/** @type {any} */ name) {
+  return withFileLock(workspacesFile() + '.lock', () => {
     const ws = loadWorkspaces();
     if (!ws[name]) return false;
     delete ws[name];
@@ -70,12 +83,12 @@ export function removeWorkspace(/** @type {any} */ name) {
   });
 }
 
-export function renameWorkspace(/** @type {any} */ name, /** @type {any} */ newName) {
+export async function renameWorkspace(/** @type {any} */ name, /** @type {any} */ newName) {
   const key = String(newName).trim();
   if (!key) return { error: '新名称不能为空' };
   if (/[\\/]/.test(key)) return { error: '名称不能包含路径分隔符' };
   if (key === name) return { name: key }; // 原样改名：无操作，避免自删条目
-  return withFileLockSync(workspacesFile() + '.lock', () => {
+  return withFileLock(workspacesFile() + '.lock', () => {
     const ws = loadWorkspaces();
     if (!ws[name]) return { error: `工作空间 ${name} 不存在` };
     if (ws[key]) return { error: `名称 ${key} 已存在` };
@@ -88,7 +101,7 @@ export function renameWorkspace(/** @type {any} */ name, /** @type {any} */ newN
 }
 
 // 修改目录：登记同名即可覆盖目录
-export function setWorkspaceDir(/** @type {any} */ name, /** @type {any} */ dir) {
+export async function setWorkspaceDir(/** @type {any} */ name, /** @type {any} */ dir) {
   return addWorkspace(name, dir);
 }
 
@@ -96,9 +109,9 @@ export function workspacePath(/** @type {any} */ name) {
   return loadWorkspaces()[name]?.dir || null;
 }
 
-export function touchWorkspace(/** @type {any} */ name) {
+export async function touchWorkspace(/** @type {any} */ name) {
   // v0.4.7（T19）：同上，加锁防并发丢更新
-  return withFileLockSync(workspacesFile() + '.lock', () => {
+  return withFileLock(workspacesFile() + '.lock', () => {
     const ws = loadWorkspaces();
     if (!ws[name]) return false;
     ws[name].lastUsed = Date.now();
@@ -160,7 +173,7 @@ export function saveSessionWorkspaces(/** @type {any} */ map) {
   try {
     ensureHome();
     // 原子写（评估 6.6）：随机 tmp 名 + rename
-    atomicWriteFileSync(sessionWorkspacesFile(), JSON.stringify(map, null, 2) + '\n', { mode: 0o600 });
+    atomicWritePrivateSync(sessionWorkspacesFile(), JSON.stringify(map, null, 2) + '\n');
     return { ok: true, error: null };
   } catch (err) {
     const msg = String(/** @type {any} */ (err)?.message ?? err);
@@ -180,17 +193,17 @@ export function getSessionWorkspace(/** @type {any} */ sessionName) {
   return loadSessionWorkspaces()[sessionName]?.dir || null;
 }
 
-export function setSessionWorkspace(/** @type {any} */ sessionName, /** @type {any} */ dir, wsName = null) {
+export async function setSessionWorkspace(/** @type {any} */ sessionName, /** @type {any} */ dir, wsName = null) {
   // v0.4.7（T19）：多标签页/多任务并行时会话级映射同样会被并发改写
-  return withFileLockSync(sessionWorkspacesFile() + '.lock', () => {
+  return withFileLock(sessionWorkspacesFile() + '.lock', () => {
     const map = loadSessionWorkspaces();
     map[sessionName] = { dir: path.resolve(dir), name: wsName || workspaceForDir(dir)?.name || null, at: Date.now() };
     saveSessionWorkspaces(map);
   });
 }
 
-export function removeSessionWorkspace(/** @type {any} */ sessionName) {
-  return withFileLockSync(sessionWorkspacesFile() + '.lock', () => {
+export async function removeSessionWorkspace(/** @type {any} */ sessionName) {
+  return withFileLock(sessionWorkspacesFile() + '.lock', () => {
     const map = loadSessionWorkspaces();
     if (!map[sessionName]) return false;
     delete map[sessionName];
@@ -200,8 +213,8 @@ export function removeSessionWorkspace(/** @type {any} */ sessionName) {
 }
 
 // 会话改名时迁移映射（记录保留）
-export function moveSessionWorkspace(/** @type {any} */ oldName, /** @type {any} */ newName) {
-  return withFileLockSync(sessionWorkspacesFile() + '.lock', () => {
+export async function moveSessionWorkspace(/** @type {any} */ oldName, /** @type {any} */ newName) {
+  return withFileLock(sessionWorkspacesFile() + '.lock', () => {
     const map = loadSessionWorkspaces();
     if (!map[oldName]) return false;
     map[newName] = map[oldName];