[Bug] Switching a blank session to an already-loaded preset drops subagent tools

[higekibaka]

环境

• DSH：0.1.5-rc.1
• Commit：183f08e9c6dde7e36cd2318eaee70b0da08fb35e
• Node.js：24.20.0
• 使用内置 standard、ptc 预设
• 已启用子代理模型选择，并配置允许的模型

复现条件

在同一个 Host 中，目标预设的加载顺序早于来源预设。

已通过真实 Cordis／AgentLoop／AgentPresets 的隔离环境验证以下原生 API 顺序：

1. 先调用 agentPresets.standingKeyFor('ptc')，加载目标预设。
2. 创建使用 standard 的空白 Agent，尚未发送消息。
3. 调用 agentPresets.select(agent, 'ptc')。
4. 等待异步操作完成后，检查 tools.schemas(agent)。

实际结果

切换前存在：

subagent_fork
list_subagent_models
subagent

切换后只剩：

subagent_fork

同时日志出现：

tool "list_subagent_models" is already registered in this scope

会话保存的 subagentModelSelectionPolicy 仍然存在，但模型选择和创建子代理的工具消失。两个独立空白 Agent 均复现，验证过程没有模型请求。

对照结果

• 先加载 standard、再首次加载并切到 ptc：正常。
• 直接创建使用 ptc 的会话，不经过切换：正常。
• 出现上述故障后，在同一 Host 中直接创建新的 ptc 会话：仍正常。

预期结果

空白会话切换预设后，应保留新预设应提供的完整工具。若重新注册失败，应明确报错或恢复一致状态，而不是切换成功但静默丢失工具。

可能原因

tool-subagent 的各预设实例独立监听 tools/change，分别安装或移除会话级工具。

当目标预设的监听器先执行时，旧工具尚未注销，新注册发生重名冲突；随后来源预设清理旧工具，导致两者都不存在。失败的安装 fiber 又可能仍被 scopedInstalls 当作已安装，阻止重试。

这是根据报错、对照实验及相关注册逻辑提出的原因分析，供维护者进一步确认。

临时规避

让新会话直接使用目标预设创建，避免先用默认预设创建空白会话再切换。

[Dawn388887]

补充验证：官方 npm 0.1.5-rc.2 仍可复现本帖描述的缺陷。查重后发现这里已经有相同加载顺序及根因分析，因此补充在本帖，不再新开重复讨论。另见相同现象的 #6339。

测试环境与未打补丁证明

• Windows x64，Node.js v24.14.0。
• 通过 npm install -g @deepseek-ai/dsh@0.1.5-rc.2 升级。
• CLI 及实际解析的 dsh-tool-subagent、dsh-web-app、dsh-agent-loop、dsh-subagent 均核实为 0.1.5-rc.2。
• 另用 npm pack @deepseek-ai/dsh-tool-subagent@0.1.5-rc.2 --ignore-scripts 下载官方原包，与实际安装的 lib/index.js 比对，SHA256 完全一致：674E8008E9DCB8009C8662B50E4D4971A7E02B6E1AE6A8972434713D43060F00。测试安装没有应用本地补丁；该工具入口也与此前未修改的 rc.1 入口摘要一致。

实际结果

before: [ 'list_subagent_models', 'subagent' ]
after: []
after another tools/change: []
AssertionError: subagent must survive preset reparenting

先加载目标 scope，再加载来源 scope；Agent 从来源重绑目标，两个工具同时消失，再触发 tools/change 仍不能恢复。测试没有注册真实模型适配器，没有模型请求、凭据读取、实际子代理委派或持久化会话写入。

可直接运行的最小复现

将以下代码保存为 repro-subagent.mjs，参数传入官方已安装的 @deepseek-ai/dsh 包目录：

node repro-subagent.mjs /path/to/node_modules/@deepseek-ai/dsh

这里使用真实 Cordis、AgentLoop、scope、tool-subagent 等插件，以及预设重组所用的 rebind / tools/change 原语；这是进程内生命周期复现，不是浏览器点击或完整 Loader 入口复现。

import assert from 'node:assert/strict';
import { createRequire } from 'node:module';
import { resolve } from 'node:path';
import { pathToFileURL } from 'node:url';

const root = process.argv[2];
assert(root, '请传入已安装的 @deepseek-ai/dsh 包目录');
const require = createRequire(pathToFileURL(resolve(root, 'package.json')));
const load = async name => import(pathToFileURL(require.resolve('@deepseek-ai/' + name)).href);
const { Context } = await load('cordis');
const { createScope, scopeOf, bindScopeParent } = await load('dsh-scope');
const tool = await load('dsh-tool-subagent');
const ctx = new Context();
let handle;
try {
  for (const name of ['dsh-llm', 'dsh-session', 'dsh-session-projection', 'dsh-system-prompt', 'dsh-tools', 'dsh-agent']) {
    await ctx.plugin((await load(name)).default);
  }
  await ctx.plugin((await load('dsh-agent-loop')).default, { agents: [] });
  await ctx.plugin((await load('dsh-subagent')).default);
  await ctx.plugin(await load('dsh-subagent-spawn-in-process'), { providerName: 'spawn' });
  await ctx.plugin((await load('dsh-tool-subagent/model-selection-settings')).default, {
    enabled: true, allowedModels: [{ provider: 'synthetic', model: 'never-called' }],
  });
  const first = createScope(ctx, { preset: 'first' });
  const second = createScope(ctx, { preset: 'second' });
  for (const preset of [first, second]) {
    await preset.ctx.plugin(tool, {
      provider: 'spawn', modelSelectionSettings: true, backgroundMode: 'continuable',
    });
  }
  let binding;
  handle = await ctx.agents.create({
    sessionId: 'synthetic-recompose',
    setup(agentCtx) { binding = bindScopeParent(scopeOf(agentCtx), scopeOf(second.ctx)); },
  });
  const settle = () => new Promise(resolve => setImmediate(resolve));
  const names = () => ctx.tools.schemas(handle.agent).map(tool => tool.name);
  await settle();
  console.log('before:', names());
  assert(names().includes('subagent'));
  binding.rebind(scopeOf(first.ctx));
  ctx.emit('tools/change');
  await settle(); await settle();
  console.log('after:', names());
  ctx.emit('tools/change');
  await settle();
  console.log('after another tools/change:', names());
  assert(names().includes('subagent'), 'subagent must survive preset reparenting');
  assert(names().includes('list_subagent_models'), 'model discovery must survive preset reparenting');
} finally {
  if (handle) await handle.dispose();
  await ctx.fiber.dispose();
}

修复方向的补充

与楼主分析一致：各预设实例分别维护 scopedInstalls，旧 fiber 的异步清理尚未完成，新 fiber 已开始注册同名工具；失败安装仍被记录，后续 reconcile 跳过重试。

此前在 rc.1 上做过一个本地候选修复：替换安装等待已离开预设的整个 fiber 清理完成，等待后排除已经销毁的安装回调，预设卸载也等待已开始的清理。候选通过了类型构建和该包 130 项合成测试（含新增双向切换、快速往返、延迟清理、卸载重挂回归）；该候选未应用到本次复测的官方 rc.2 安装，也不是已提交或已合并的上游修复。

不建议只在各预设自己的 removeScoped() 内增加 await：另一个预设实例的监听器仍可能抢先安装。等待范围也应覆盖 provider 回调后注册的工具，不能仅清理首次安装时捕获的定义。

[malyjacob]

在 0.2.0-rc.2 上实测了同一条，来交叉引用一下：切换后的会话拿到 13 个工具（subagent_fork 在，subagent 和 list_subagent_models 都不在），同一 preset 直接新建的是 15 个。

你那个 repro-subagent.mjs 我还没跑，先说一下我这边对得上的部分：你说的两个点——「失败的 install fiber 留在 scopedInstalls 里被当成已安装，挡住后续重试」和「目标 preset 的监听先跑、旧名还没释放就撞重名」——我按源码又核了一遍，都能坐实，并且顺带确认这几段代码在 rc.1 和 rc.2 之间逐字没变（packages/core/scope、packages/bundle/web-app/presets、vendor/ 的 diff 全空，tool-subagent/src/index.ts 只改了 4 处提示词）。所以它不是 rc.2 引进的回归，是老缺陷在新时序下被踩出来。

完整的代码定位写在 #6751 的这条评论里：#6751 (comment)

那个 in-process 复现脚本是这几条报告里唯一不依赖 GUI 的，维护者要定位的话应该会先用它，多谢留下。