dsh-session-conductor 1.4.4 → 1.4.5

package/lib/features/member-model/override.js

@@ -63,6 +63,41 @@ export async function setMemberModelOverride(ctx, sessionId, provider, model) {
   memberModelCache.set(sessionId, { provider, model });
 }
 
+/**
+ * 删除一个会话的 override（domain 与内存镜像同时清）。
+ *
+ * 【为什么需要】override 一旦存在，`agent/request` 就会**每轮**按它改写请求模型，
+ * 优先级压过官方 UI / `model/selection` 链路。因此「不带 override 参数」的切换
+ * 不能只是不写 —— 必须把存量条目删掉，官方链路才能接管；否则表现为
+ * 「切换成功但请求仍走旧模型」（selection 写入后，请求仍按 override 的值发出）。
+ *
+ * 幂等：不存在时返回 {removed:false}，不报错。
+ *
+ * @param {object} ctx 插件 context
+ * @param {string} sessionId 会话 id
+ * @returns {Promise<{removed: boolean}>}
+ */
+export async function deleteMemberModelOverride(ctx, sessionId) {
+  if (!sessionId) return { removed: false };
+  const hadCache = memberModelCache.delete(sessionId);
+  let removedInDomain = false;
+  try {
+    const domain = await loadPluginDomain(ctx);
+    const state = domain.global.get() ?? {};
+    const overrides = { ...(state.memberModelOverrides ?? {}) };
+    if (Object.prototype.hasOwnProperty.call(overrides, sessionId)) {
+      delete overrides[sessionId];
+      await domain.global.set({ ...state, memberModelOverrides: overrides });
+      removedInDomain = true;
+    }
+  } catch (error) {
+    // domain 不可用（如工作区自测环境缺宿主包）：内存镜像已清，但落盘那条还在。
+    // 必须如实把原因带回调用方 —— 否则「切换成功、请求仍走旧模型」无从排查。
+    return { removed: hadCache, error: String(error?.message ?? error) };
+  }
+  return { removed: hadCache || removedInDomain };
+}
+
 /**
  * 把成员模型 override 应用到一次请求配置上（`agent/request` hook 的核心逻辑，单独导出便于单测）。
  * 丢掉继承的 reasoningEffort：换模型后应回到目标模型自己的 provider 默认行为。

package/lib/features/member-model/switch.js

@@ -14,7 +14,7 @@ import { cfg } from "../../core/config.js";
 import { hasOpenTurn } from "../../sessions/turn.js";
 import { foldLastModelSelection } from "../../sessions/route.js";
 import { lazyZstd, lazyRepair } from "../../core/lazy.js";
-import { setMemberModelOverride } from "./override.js";
+import { setMemberModelOverride, deleteMemberModelOverride } from "./override.js";
 
 /**
  * 等待一个成员进入空闲（当前回合结束/暂停）。
@@ -171,29 +171,38 @@ async function repairDescriptorIfStale(ctx, filePath, sessionId, provider, model
   const descriptor = found.descriptor;
   if (!descriptor || typeof descriptor !== "object") return { skipped: "不是子代理会话（无 descriptor）" };
 
-  // 取当前可用 provider 列表；拿不到就不猜、不动日志
-  const sc = ctx.get("sessionController");
-  if (typeof sc?.modelCatalog !== "function") return { skipped: "modelCatalog 不可用，跳过 descriptor 校验" };
-  let groups;
-  try {
-    const catalog = await sc.modelCatalog();
-    groups = catalog?.groups ?? [];
-  } catch (error) {
-    return { skipped: `modelCatalog 调用失败，跳过：${String(error?.message ?? error)}` };
+  // 何时改写 descriptor：目标模型 ≠ descriptor 当前值就改 ——
+  // 子代理冷恢复（reason=resume）的请求模型直接取 descriptor，不经过官方
+  // model/selection 投影链路，selection 事件对它无效（只写 selection 时 resume 仍用
+  // descriptor 旧模型）。要「不带 override 也能切子代理」，必须把 descriptor 同步到目标，
+  // 两条通道（挂载期 selection / 冷恢复 descriptor）才都生效。
+  const driftsFromTarget = descriptor.agentProvider === undefined
+    || descriptor.agentModel === undefined
+    || descriptor.agentProvider !== provider
+    || descriptor.agentModel !== model;
+  if (!driftsFromTarget) return { skipped: "descriptor 已指向目标模型" };
+
+  // modelCatalog 只用于**校验目标合法**（防止把 descriptor 写成一个不存在的路由，
+  // 那会让成员永久唤不醒）。它不可用时不再直接放弃同步 —— 早先版本把它当硬前置，
+  // 结果「目录服务不可用」= 同步静默失效，成员冷恢复又回落旧模型。
+  const sc = ctx.get?.("sessionController");
+  if (typeof sc?.modelCatalog === "function") {
+    try {
+      const catalog = await sc.modelCatalog();
+      const groups = catalog?.groups ?? [];
+      if (groups.length > 0) {
+        const group = groups.find((g) => g.id === provider);
+        if (!group) return { skipped: `目标 provider ${provider} 不在模型目录，跳过（避免写入失效路由）` };
+        const known = (group.models ?? []).some((m) => m.id === model);
+        if (!known) return { skipped: `目标模型 ${model} 不在 ${provider} 目录，跳过（避免写入失效路由）` };
+      }
+    } catch (error) {
+      // 目录调用失败不阻断同步：目标值由调用方给出，宿主侧还有组合校验兜底
+      ctx.logger?.info?.(`dsh-session-conductor: descriptor 同步：模型目录调用失败，仍按目标写入：${String(error?.message ?? error)}`);
+    }
   }
-  if (groups.length === 0) return { skipped: "模型目录为空，跳过 descriptor 校验" };
-
-  const providerIds = new Set(groups.map((g) => g.id));
-  const staleProvider = descriptor.agentProvider !== undefined && !providerIds.has(descriptor.agentProvider);
-  const staleModel = (() => {
-    if (descriptor.agentProvider === undefined || descriptor.agentModel === undefined) return false;
-    const group = groups.find((g) => g.id === descriptor.agentProvider);
-    if (!group) return false;
-    return !(group.models ?? []).some((m) => m.id === descriptor.agentModel);
-  })();
-  if (!staleProvider && !staleModel) return { skipped: "descriptor 路由有效" };
-
-  // 整份重写（descriptor 在文件头部，追加帧改不了它）；这条路径只在路由失效时走一次，代价可接受
+
+  // 整份重写（descriptor 在文件头部，追加帧改不了它）；这条路径只在路由失效或目标漂移时走一次，代价可接受
   const { encodeSessionText } = await lazyRepair();
   const { decodeAllFrames } = await import("../../zstd-frames.js");
   const fullText = buf.length > 0 ? await decodeAllFrames(buf) : "";
@@ -226,7 +235,7 @@ async function repairDescriptorIfStale(ctx, filePath, sessionId, provider, model
   const backup = backupSwitchLog(filePath, sessionId);
   if (backup.error) return { error: backup.error };
   replaceFileSync(filePath, await encodeSessionText(lines.join("\n") + "\n"), { tmpSuffix: ".descfixtmp" });
-  log(ctx, `切换写日志：descriptor 路由失效（${staleProvider ? "provider 已不存在" : "model 不在目录"}），已改为 ${provider}/${model}`);
+  log(ctx, `切换写日志：descriptor 已同步为 ${provider}/${model}（原 ${descriptor.agentProvider ?? "?"}/${descriptor.agentModel ?? "?"}，防冷恢复回落）`);
   return { repaired: true };
 }
 
@@ -251,6 +260,17 @@ export async function appendSelectionEventToLog(ctx, sessionId, provider, model)
     count: events.length,
   });
 
+  // descriptor 同步放在「selection 是否已相同」的判定**之前**：
+  // 两者是独立通道（selection 管挂载期、descriptor 管冷恢复），日志里 selection 已是目标值
+  // 并不代表 descriptor 也对齐了。早先放在追加帧之后，遇到「重复切换同一模型」就提前返回，
+  // 于是 descriptor 永远修不到，成员下次 resume 仍回落旧模型。
+  const descFixEarly = await repairDescriptorIfStale(ctx, filePath, sessionId, provider, model);
+  if (descFixEarly.repaired) {
+    log(ctx, `切换写日志：descriptor 已同步到 ${provider}/${model}（防冷恢复回落）`);
+  } else if (descFixEarly.error) {
+    log(ctx, `切换写日志：descriptor 校验失败（不影响本次切换）：${descFixEarly.error}`);
+  }
+
   for (let attempt = 0; attempt < 3; attempt += 1) {
     const { events } = logInfo;
     const last = foldLastModelSelection(events);
@@ -279,9 +299,8 @@ export async function appendSelectionEventToLog(ctx, sessionId, provider, model)
     const written = await appendEventFrame(filePath, event);
     log(ctx, `切换写日志：追加帧 ${Date.now() - writeStartedAt}ms（seq ${event.seq}）`);
     if (!written.ok) return { error: `写入会话日志失败：${written.error}` };
-    // descriptor 路由失效会让成员永久唤不醒（宿主报 subagent/not-resumable），这里顺带修
-    const descFix = await repairDescriptorIfStale(ctx, filePath, sessionId, provider, model);
-    if (descFix.error) log(ctx, `切换写日志：descriptor 校验失败（不影响本次切换）：${descFix.error}`);
+    // descriptor 同步已在进入循环前做过一次（那时 selection 可能是「已是该模型」而提前返回），
+    // 这里不再重复整份重写 —— 只有追加帧改变了文件尾部，descriptor 在头部、不受影响。
 
     // 附带排查提示：本机实测「成员唤不醒」的真凶是部署侧的「自动审查」开关被关掉，
     // 与 descriptor / 帧校验和 / 写锁全都无关（详见 docs/VERSIONS.md 1.3.15）。
@@ -335,15 +354,29 @@ export async function appendSelectionEventToLog(ctx, sessionId, provider, model)
  * @returns {Promise<{note: string}>} note 直接拼进回执
  */
 async function persistSwitchIntent(ctx, sessionId, provider, model, options = {}) {
-  // override 是**显式参数**（/model-switch … override / set_member_model 的 override:true）：
-  // 默认不写 —— 官方 model/selection 链路当轮立即生效且跨轮持久（干净会话实测），
-  // 每轮强制改写的 override 会压过 UI 的选择，表现为「界面切了、请求不动」。
-  if (options?.override !== true) return { note: "，未写 override（走官方 model/selection 链路）" };
+  // override 是**显式参数**：带 `override` → 写入/更新（旧行为，每轮强制改写）；
+  // 不带 → **删除该成员的存量 override**，让官方 model/selection 链路接管。
+  // 为什么必须删而不是「不写」：override 一旦存在，agent/request 会每轮按它改写请求，
+  // 优先级压过新写的 model/selection，结果就是「selection 写入成功但下一条请求仍用旧模型」。
+  // 只删该成员这一条，不做批量清理。
+  if (options?.override === true) {
+    try {
+      await setMemberModelOverride(ctx, sessionId, provider, model);
+      return { note: "，已写 override（每轮强制改写，旧行为；UI 切换依赖同步机制）" };
+    } catch (error) {
+      return { note: `，override 持久化失败：${String(error?.message ?? error)}` };
+    }
+  }
   try {
-    await setMemberModelOverride(ctx, sessionId, provider, model);
-    return { note: "，已写 override（每轮强制改写，旧行为；UI 切换依赖同步机制）" };
+    const { removed, error } = await deleteMemberModelOverride(ctx, sessionId);
+    if (error) return { note: `，清除 override 失败（旧 override 仍会压制切换）：${error}` };
+    return {
+      note: removed
+        ? "，已清除该成员的 override（改走官方 model/selection 链路）"
+        : "，未写 override（走官方 model/selection 链路）",
+    };
   } catch (error) {
-    return { note: `，意图持久化失败：${String(error?.message ?? error)}` };
+    return { note: `，清除 override 失败：${String(error?.message ?? error)}` };
   }
 }
 
@@ -475,6 +508,19 @@ async function applyModelSwitch(ctx, sessionId, provider, model, agent, timer, o
   //      （memberView 读 live.options.model），不跟着改就会一直显示创建时的老模型。
   const synced = syncLiveAgentOptions(ctx, live, provider, model);
   timer.mark("同步视图");
+  // ④-补2：descriptor 同步 —— 挂载中切换只写 selection，但成员空闲卸载后冷恢复（resume）
+  //       的请求模型直接取 descriptor，不改它就会「下次唤醒回落旧模型」。与未挂载路径
+  //       （appendSelectionEventToLog 里的修复）保持同一语义：目标 ≠ descriptor 就改写。
+  try {
+    const { findSessionLogById, sessionsRootOf } = await import("../../sessions/events.js");
+    const logPath = findSessionLogById(sessionId, sessionsRootOf());
+    if (logPath) {
+      const descFix = await repairDescriptorIfStale(ctx, logPath, sessionId, provider, model);
+      if (descFix.repaired) log(ctx, `成员模型切换：descriptor 已同步到 ${provider}/${model}（防冷恢复回落）`);
+    }
+  } catch (error) {
+    log(ctx, `成员模型切换：descriptor 同步失败（不影响本次切换）: ${String(error?.message ?? error)}`);
+  }
   let via = `model/selection 事件（官方投影接管）${eventNote}${overrideNote}，下一回合生效`
     + (synced ? "；已同步成员视图的模型显示" : "");
   // ⑤ 官方接口优先（若该部署暴露）：

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-session-conductor",
-  "version": "1.4.4",
+  "version": "1.4.5",
   "description": "DSH session management enhancement: archive/restore sessions, batch & conditional delete, AI auto-rename, interrupted-session auto-resume, release idle sessions, workspace grouping, undo last message, repair damaged session logs, value analysis, full-text search, per-member model switching (list_models / set_member_model), and plan/closing template injection (system-prompt + context).",
   "type": "module",
   "main": "lib/index.js",