Bug: Loader inject has no optional/timeout degrade - a third-party bundle can permanently block core agent presets while dsh only logs a warning

[yangala]

---

name: Bug
about: 记录现有预期行为的失效
type: Bug

Summary

第三方 bundle 可以通过自己的 dsh.bundle.patch.yml 用 inject 字段给 Loader 的核心条目加上硬依赖，而 Loader 没有 optional / 超时 / 降级机制。依赖一旦消失，被注入的条目永远停在 FIBER_PENDING。

由于 preset-standard 等行不在全局 required list 里，dsh 会照常启动、只打一条 warning。用户看到的是「能开机、UI 能用、文件也能读，但新建会话必失败」——故障被降级成了一条几乎没人会读的警告。

现成复现：billion-context 插件的 bundle patch（dsh.bundle.patch.yml:23-30）往 preset-standard / preset-ptc / preset-minimal / preset-cordis 四行注入 biliBasicCompactionPolicy；禁用该插件后 dsh 永久无法新建会话，且没有任何恢复路径。

Reproduction

1. profile 的 package.json → dsh.profile.bundles 里包含 billion-context
2. 启动 dsh，此时一切正常
3. 在插件管理器里禁用 billion-context（profile 的 cordis.patch.yml 追加 - id: bili-native + disabled: true）
4. 重启 dsh
5. 尝试新建任意会话 → 失败

Current behavior

~/.dsh/logs/desktop-next.log：

[warn] dsh: warning: 4 entries did not activate
[info] preset-standard (@deepseek-ai/dsh-agent-preset): pending (waiting for service: biliBasicCompactionPolicy)
[info] preset-ptc       (@deepseek-ai/dsh-agent-preset): pending (waiting for service: biliBasicCompactionPolicy)
[info] preset-minimal   (@deepseek-ai/dsh-agent-preset): pending (waiting for service: biliBasicCompactionPolicy)
[info] preset-cordis    (@deepseek-ai/dsh-agent-preset): pending (waiting for service: biliBasicCompactionPolicy)

UI 点新建会话：

新建会话失败：agent-preset/not-found: Unknown agent preset: standard

根因链：

• vendor/loader/src/index.ts:134 — Inject.resolve(fiber.entry!.options.inject, fiber.inject)，条目级 inject 完全来自 entry.options.inject
• vendor/cordis/src/registry.ts:71-72 — Inject.resolve(null) 直接 return；数组形式 inject: [a] 里每一项仍必填（registry.ts:73-76 只把 intercept config 置 null）
• vendor/cordis/src/fiber.ts:611-623 — _refresh 只要 inject 里任一键未解析就置 epoch = INACTIVE
• packages/boot/app-boot/lib/index.js:3942 — const missing = Object.keys(fiber.inject).filter(s => fiber.ctx.get(s) === undefined) → outcome.kind = "pending"
• packages/boot/app-boot/lib/index.js:4005-4008 — 只有全局 required list 里的不活跃条目才拒绝启动，其余仅 warning

EntryOptions 只有 id / name / config / group / disabled / inject（vendor/loader/src/config/entry.ts:10-23），没有任何 optionalInject / optional / required: false 之类的表达。结果就是第三方 bundle 可以为 dsh 的核心条目制造不可逆的硬依赖。

用户侧唯一可用的绕过是写 home 层 patch（~/.dsh/cordis.patch.yml，层序见 packages/boot/app-boot/src/profile-context.ts:73-80，home 层在 profile 层之后所以能赢）：

- id: preset-standard
  inject: null

inject: null 能生效是因为 applyEntryPatches（vendor/include/src/index.ts:120-123）是顶层键整体替换而非深合并。

Expected behavior

至少满足其一：

1. 条目级 inject 支持可选依赖语义——依赖缺失时激活并降级，而不是永久 pending；
2. 或 pending 超过阈值后主动放弃依赖、记录诊断、按未注入状态激活；
3. 或：非 required list 里「被注入的核心能力条目」pending 时显式失败并拒绝启动，而不是一条 warning。静默降级是这次最难排查的部分——只能从「新建会话失败」这个下游症状倒推，配置文件、YAML 全部看起来正常。

Environment

• Windows 11 / x64
• DeepSeek Harness 0.2.1-rc.2
• Node：dsh 自带运行时 v24.21.0
• 触发插件：billion-context 0.1.199
• profile：desktop（含 @deepseek-ai/dsh-web-app 预设）

---

English summary

A third-party bundle can turn a core Loader entry's service dependency into a hard requirement via inject, and the Loader has no optional / timeout / degrade path. Once the provider disappears the entry stays FIBER_PENDING forever. Because preset-standard and friends are not in the global required list, dsh still boots and only emits a warning — so the user gets a working UI that cannot create a single session.

Reproducer: billion-context's dsh.bundle.patch.yml:23-30 injects biliBasicCompactionPolicy into preset-standard / preset-ptc / preset-minimal / preset-cordis. Disable the plugin and dsh can never create a session again.

Chain: vendor/loader/src/index.ts:134 → vendor/cordis/src/registry.ts:71-76 (every array item is still required) → vendor/cordis/src/fiber.ts:611-623 (any unresolved key ⇒ INACTIVE) → app-boot/lib/index.js:3942 (pending) → :4005-4008 (only the global required list fails startup). EntryOptions has no way to express an optional dependency.

[PerryLink]

机制你说对了，只有一处归属要挪一下：「没有 optional」不是 loader 缺的，是 cordis 缺的；而「只警告不失败」这一半，其实已经有现成开关，只是没打开。

1. inject 没有 optional，这不是 loader 的选择

vendor/loader/src/index.ts:134 就是你引的那行：

Inject.resolve(fiber.entry!.options.inject, fiber.inject)

它只是把 options.inject 归一化后塞进 fiber。归一化在 vendor/cordis/src/registry.ts:71-88：数组 → 每个名字映射到 null，对象 → 每个名字映射到它的 intercept 配置。整个结构里没有 required / optional 之分，所以任何写进 inject 的名字都是硬依赖。要让 bundle 能声明「有就用、没有也要起」，得给 cordis 的 Inject 加一层语义，那是 API 变更，不是补丁。

2. 「只警告」是可配置的，缺的是把这些 id 传进去

packages/boot/app-boot/src/index.ts：

• :273 readonly requiredIds?: readonly string[]
• :323 / :331 默认 [] —— 也就是默认没有任何条目是必须的
• :399-405 判定「这次重载是否引入了新失败」：

if (requiredIds.includes(failure.entry.options.id)) return true
// An entry waiting for a service starts once a provider appears.
if (failure.outcome.kind === 'pending') return false

顺序是关键：requiredIds 在 pending 排除之前。也就是说，一个卡在 pending 的条目，只要它的 id 在 requiredIds 里，就会被算作「引入的失败」， :407 直接抛 activationDiagnostic(...)；否则才走到 :412 变成 warnings，也就是你看到的那句 warning: 4 entries did not activate（文案在 :1098）。

所以最小的修法不是给 loader 加超时，而是：把这些 preset 的 id 加进启动时传给 recompose 的 requiredIds。这四行条目一旦起不来，dsh 就不该正常启动 —— 因为「能开机但新建会话必失败」比「开不了机」更难排查，你说的正是这一点。

3. 但直接变 fatal 未必是用户想要的

把 preset-* 变成 required 之后，受害者的处境从「UI 能用但新建会话失败」变成「完全起不来」。对一个只想把某个插件关掉的用户，后者的可操作性更差。所以更好的形态可能是两步：

• 失败信息要能指名道姓。pending 的 outcome 已经带了 missing: string[]（:1072-1076），至少能列出缺的是哪个 service；再往前一步把「是谁 inject 的」也带出来（loader 的 entry options 里有 patch 来源），用户才知道该去 profile 里删哪一行。
• 给一条恢复路径：启动失败时提示可以直接用 cordis.patch.yml 关掉注入方，或者干脆在检测到 pending 的 preset 时以降级模式启动（禁用注入方那一行）并显式告知。

这两条都不需要动 cordis。真要支持「optional 依赖」，那才是他说的那个 API 级改动。

4. 关于责任归属

billion-context 往四个 preset 行注入自己的 service，这本身就是把核心条目绑到可选插件上 —— 插件侧这样写是不该的。但「第三方 bundle 能让核心条目永久 pending，而 dsh 只打一行几乎没人看的 warning」这个结构性缺口，确实在宿主侧。两边都该改，只是改法不同：插件侧去掉硬注入，宿主侧让这件事至少响亮。