@menghuan1918/dsh-apollo 0.5.0 → 0.5.1

package/README.md

@@ -86,13 +86,13 @@ flowchart TB
 
 ```sh
 # npm 安装（发布后）
-dsh plugin --profile <profile> add @menghuan1918/dsh-apollo@0.5.0
+dsh plugin --profile <profile> add @menghuan1918/dsh-apollo@0.5.1
 
 # 或从 GitHub main 源码安装（备选）
 dsh plugin --profile <profile> add github:Menghuan1918/dsh-apollo
 ```
 
-> **发布状态（诚实说明）**：Apollo `0.5.0` 把 `dsh-better-subagent@0.1.0` 与 `dsh-better-compact@0.1.0` 声明为**精确 npm 依赖**，但截至本实现完成，这两个版本尚未发布到 npm。Apollo 0.5.0 必须在两个子包发布并按顺序验证后再发布；本仓库的验证使用本地 tarball `npm install --no-save`，提交的 manifest 中**没有** `file:` / `link:` / Git 说明符。
+> **发布状态（诚实说明）**：Apollo `0.5.0` 已发布，`dsh-better-subagent@0.1.0` / `dsh-better-compact@0.1.0` 作为其精确依赖已可从 npm 取得。当前版本 `0.5.1` 是旧 profile `name` guard 的迁移修复版；本仓库 0.5.1 的验证使用本地 tarball `npm install --no-save`，提交的 manifest 中**没有** `file:` / `link:` / Git 说明符。
 
 安装、升级或换包都会改变运行的代码，必须重启对应 profile 的 DSH 进程；只刷新网页不够。目标行、子代理行与压缩行由 Apollo 的 bundle patch 插入，不需要再单独 `add` 三个子包。
 
@@ -102,14 +102,16 @@ dsh plugin --profile <profile> add github:Menghuan1918/dsh-apollo
 
 ```sh
 dsh plugin --profile <profile> remove dsh-apollo
-dsh plugin --profile <profile> add @menghuan1918/dsh-apollo@0.5.0
+dsh plugin --profile <profile> add @menghuan1918/dsh-apollo@0.5.1
 ```
 
 > `dsh-apollo` 只是旧入口名；unscoped 的 `dsh-apollo` 已被其他项目占用，不要从 npm 安装它。
 
 ## 从 0.4.x 升级
 
-0.5.0 把子代理与压缩能力移出 Apollo。旧 profile 不需要手工重填配置：Apollo 启动后在 settings/loader 可用时运行一次**幂等迁移**（[src/migrate.js](src/migrate.js)）。
+0.5.0 把子代理与压缩能力移出 Apollo，0.5.1 修复旧 profile 行的 `name` guard 迁移缺口。旧 profile 不需要手工重填配置：Apollo 启动后在 settings/loader 可用时运行一次**幂等迁移**（[src/migrate.js](src/migrate.js)）。
+
+> **0.5.1 迁移修复**：如果旧 profile patch 的 `apollo-skills` 覆盖还带历史名称 `name: dsh-apollo`，DSH 会把 `name` 当 guard 并跳过该覆盖，导致有效配置看起来仍是默认值。0.5.1 从 `configEditor.configuration().override` 恢复这些显式值（含 `false` / `0` / `""` / `[]`），再合并到迁移来源；被跳过的旧预设覆盖（`preset-eng` / `preset-eng-ptc`）同样处理。无需手工修改旧行；升级后重启 profile 即可。
 
 ```mermaid
 flowchart LR
@@ -204,7 +206,7 @@ dsh plugin --profile <profile> remove dsh-better-compact
 - 彻底移除压缩增强：从 Apollo 组合中移除 Apollo，或把预设/组合改回原生压缩。**只禁用独立 compact bundle 不会恢复 Apollo 预设内的 engine 行**，因为该 engine 由 Apollo 自己的预设 patch 声明；需要移除 Apollo，或恢复一份使用原生 `compaction-basic` 的预设覆盖。
 - 彻底移除子代理增强：移除 Apollo（或至少禁用其 `dsh-better-subagent` 行）。原生 create/fork/message/interrupt 仍可用；CLI-only 旧会话需要重新安装 better-subagent 才能续接，其它原生会话不受影响。
 - 卸载不删除原生压缩事件、goal 日志或 CLI 身份 KV；handoff 临时文件不是必需依赖，删除后已压缩会话仍可从原生 checkpoint 恢复。
-- 回退到 0.4.x 前先备份 profile 的 `cordis.patch.yml`；旧字段仍能装载，但 0.5.0 写入的新行配置不会被 0.4.x 读取。
+- 回退到 0.4.x 前先备份 profile 的 `cordis.patch.yml`；旧字段仍能装载，但 0.5.x 写入的新行配置不会被 0.4.x 读取。
 
 ## 提示词归属
 
@@ -224,7 +226,7 @@ dsh plugin --profile <profile> remove dsh-better-compact
 - `package.json` 的 `files` 只列入 `docs/DEVELOPMENT.md`，并额外用 [.npmignore](.npmignore) 排除 `docs/specs/`；
 - `tests/packaging.test.mjs` 直接运行 `npm pack --dry-run --json`，断言发布清单没有 Specs、测试、脚本、快照或 tarball。
 
-发布顺序：**先发布并验证两个独立包（`@menghuan1918/dsh-better-subagent@0.1.0`、`dsh-better-compact@0.1.0`），再发布 Apollo 0.5.0**。Apollo 的精确依赖指向尚不存在的版本时，`npm install` 会失败；不要用开发 `file:`/`link:` 依赖发布。两个子仓各自有 trusted publishing workflow；Apollo 的 [release workflow](.github/workflows/release.yml) 只发布本包。
+发布顺序：**先发布并验证两个独立包（`@menghuan1918/dsh-better-subagent@0.1.0`、`dsh-better-compact@0.1.0`），再发布 Apollo 0.5.1**。两个 0.1.0 已随 0.5.0 发布；0.5.1 可只发布本包。Apollo 的精确依赖指向尚不存在的版本时，`npm install` 会失败；不要用开发 `file:`/`link:` 依赖发布。两个子仓各自有 trusted publishing workflow；Apollo 的 [release workflow](.github/workflows/release.yml) 只发布本包。
 
 ## 验证状态（诚实记录）
 

package/docs/DEVELOPMENT.md

@@ -2,7 +2,7 @@
 
 面向维护者、集成者和需要排查装配问题的用户。用户安装与配置请看 [README](../README.md)；本文保留 DSH 插件的实现细节、旧配置迁移、组合安装边界与验证方法。
 
-当前实现版本：`0.5.0`；npm 包名是 `@menghuan1918/dsh-apollo`。纯 ESM，无构建步骤；宿主浏览器半边是直接加载的闭包工厂。
+当前实现版本：`0.5.1`；npm 包名是 `@menghuan1918/dsh-apollo`。纯 ESM，无构建步骤；宿主浏览器半边是直接加载的闭包工厂。
 
 ## 目录
 
@@ -203,6 +203,8 @@ npm install --no-save /path/to/menghuan1918-dsh-better-subagent-0.1.0.tgz /path/
 
 来源：`apollo-skills` 的**有效配置**。volatile 字段来自 `describe().value`；非 volatile 的 `trustedHosts` 来自旧行 raw config（`describe().value` 只投影 volatile 表单，不能用来判断 trustedHosts）。
 
+**0.5.1 修复**：DSH 把 profile 覆盖行的 `name` 当 guard。旧 profile 若保留历史 `name: dsh-apollo`，Loader 会跳过该覆盖，`describe().value` 只剩默认值（例如用户显式 `subagents.eng: false` 会被看成默认 `true`）。迁移现在额外读取 `configEditor.configuration()` 中该行的 `override`，递归合并到有效配置之上：普通对象逐键合并，数组与所有标量（含 `false`/`0`/`""`/`[]`）整体覆盖。这样被跳过的显式值仍会迁移。`preset-eng` / `preset-eng-ptc` 的预设覆盖也按同样方式合并（`plugins` 数组整体替换），再读取原生 `compaction-basic`。
+
 目标：`dsh-better-subagent` 的 `settings.describe()`：
 
 - `target.user` 的存在性表示显式值（含 false/0/""/[]）；
@@ -306,10 +308,10 @@ DSH_PACKAGE_ROOT="/…/node_modules/@deepseek-ai/dsh" npm run prompts
 ### npm 安装
 
 ```sh
-dsh plugin --profile <profile> add @menghuan1918/dsh-apollo@0.5.0
+dsh plugin --profile <profile> add @menghuan1918/dsh-apollo@0.5.1
 ```
 
-包发布后即可用此命令安装，并带入三个精确依赖、应用三份 bundle patch。安装后需要重启对应 profile 的 DSH 进程。**首发完成前不要宣称 registry 已有 0.5.0**：两个子包的 0.1.0 必须先发布。
+包发布后即可用此命令安装，并带入三个精确依赖、应用三份 bundle patch。安装后需要重启对应 profile 的 DSH 进程。0.5.0 已发布；0.5.1 是迁移修复版本，可只发布本包。
 
 ### GitHub 源码安装
 
@@ -344,7 +346,7 @@ dsh --profile <profile> --dump-config
 
 ### 升级
 
-- 升级包后重启 profile；0.4.x → 0.5.0 的迁移见 [旧配置迁移](#旧配置迁移)；
+- 升级包后重启 profile；0.4.x → 0.5.x 的迁移见 [旧配置迁移](#旧配置迁移)；
 - 迁移写入的 compact 设置需要再重载一次才会被运行中引擎采用；
 - 升级 DSH 后先跑 `npm test`，漂移测试会按目标 DSH 安装比较预设；
 - 版本范围声明：Node `^22.19.0 || >=24.0.0`，DSH `>=0.2.1-rc.2 <0.3.0`；声明范围接受预发布版，不等于逐版验证保证。
@@ -363,7 +365,7 @@ dsh plugin --profile <profile> remove @menghuan1918/dsh-apollo
 
 1. 先发布 `@menghuan1918/dsh-better-subagent@0.1.0` 与 `dsh-better-compact@0.1.0`（各自仓库的 Release）；
 2. 在官方 registry（`https://registry.npmjs.org/`）确认两个版本可取得、可安装；
-3. 再把 Apollo 的精确依赖指向已发布版本，推 main 并发布 Apollo Release（`v0.5.0`）；
+3. 再把 Apollo 的精确依赖指向已发布版本，推 main 并发布 Apollo Release（0.5.0 已发布；本轮为 `v0.5.1`）；
 4. 子包版本可取得前，不要把指向它们的精确依赖推到 Apollo main，否则 `npm install` / GitHub 源码安装会 404。
 
 ## 测试与冒烟
@@ -390,7 +392,7 @@ DSH_PACKAGE_ROOT=/path/to/node_modules/@deepseek-ai/dsh npm test
 | [tests/dsh-home.test.mjs](../tests/dsh-home.test.mjs) | `DSH_HOME` 的默认、绝对、`~`、相对与空白处理 |
 | [tests/exit-guard.test.mjs](../tests/exit-guard.test.mjs) | job-only 判断、终态过滤、source kind、SessionId 参数、fail-open、用户中断、无豁免 |
 | [tests/migrate.test.mjs](../tests/migrate.test.mjs) | 迁移纯函数与 fake host 编排（见上） |
-| [tests/migration.runtime.test.mjs](../tests/migration.runtime.test.mjs) | 真实 temp-profile 迁移（无宿主时跳过） |
+| [tests/migration.runtime.test.mjs](../tests/migration.runtime.test.mjs) | 真实 temp-profile 迁移；含旧 `name: dsh-apollo` 被 Loader 跳过时从 `configuration().override` 恢复显式值的回归（无宿主时跳过） |
 | [tests/mode-instructions.test.mjs](../tests/mode-instructions.test.mjs) | 只含通用纪律 + job 守卫；无委派/分档/CLI/PTC 文本；不依赖 tiers |
 | [tests/owns-host.test.mjs](../tests/owns-host.test.mjs) | `declareOwnsHost` tap 注入位置、开关、去重、零路由 |
 | [tests/packaging.test.mjs](../tests/packaging.test.mjs) | 真实 `npm pack --dry-run`：Specs/tests/scripts/prompts/tarball 不在清单 |
@@ -479,7 +481,7 @@ DSH_PACKAGE_ROOT=/path/to/node_modules/@deepseek-ai/dsh node scripts/runtime-smo
 
 ### 发布
 
-release workflow 使用 npm trusted publishing：tag 必须等于 `package.json` 版本，先 `npm install` / `npm test` 再 `npm publish --provenance`。Apollo 0.5.0 的 release job 只有在两个子包 0.1.0 发布后才可能成功；不要硬编码 token、假发布或用开发 link 代替正式依赖。
+release workflow 使用 npm trusted publishing：tag 必须等于 `package.json` 版本，先 `npm install` / `npm test` 再 `npm publish --provenance`。Apollo 0.5.x 的 release job 只有在两个子包 0.1.0 可取得后才可能成功；不要硬编码 token、假发布或用开发 link 代替正式依赖。
 
 ## 许可与致谢
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@menghuan1918/dsh-apollo",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "DSH plugin bundle: workflow skills, engineering presets, independent better-goal / better-subagent / better-compact integration, and a minimal job exit guard. Pure ESM, no build step.",
   "license": "GPL-3.0-or-later",
   "repository": {

package/src/migrate.js

@@ -70,6 +70,35 @@ function clone(value) {
   return value === undefined ? undefined : structuredClone(value)
 }
 
+/**
+ * Deep-merge an explicit profile override over an effective config.
+ *
+ * DSH applies a profile row's `name` as a guard: an old row whose name no
+ * longer matches the active row (for example the historical
+ * `name: dsh-apollo` on `apollo-skills`) is skipped by the Loader, yet
+ * `configEditor.configuration().override` still exposes its explicit config.
+ * Merging that override back over the effective config recovers the user's
+ * explicit values during migration.
+ *
+ * Plain objects merge recursively; arrays and every scalar - including
+ * explicit `false`, `0`, `""` and `[]` - replace the base value.
+ *
+ * @param base - effective config (defaults + applied layers).
+ * @param override - raw explicit profile override; undefined means "no override".
+ * @returns a detached merged value.
+ */
+export function mergeConfigOverride(base, override) {
+  if (override === undefined) return clone(base)
+  if (!isRecord(base) || !isRecord(override)) return clone(override)
+  const result = clone(base)
+  for (const [key, value] of Object.entries(override)) {
+    result[key] = isRecord(value) && isRecord(result[key])
+      ? mergeConfigOverride(result[key], value)
+      : clone(value)
+  }
+  return result
+}
+
 /** Assign a nested path into a patch object. */
 function setPatch(patch, keys, value) {
   let node = patch
@@ -383,7 +412,26 @@ function defaultPresetOf(agentPresets) {
   }
 }
 
-/** Top-level preset declarations visible to configEditor.entries(). */
+/** Read one row's raw profile override from configEditor.configuration(). */
+function configurationOverrideOf(configEditor, id) {
+  try {
+    const rows = configEditor?.configuration?.()
+    if (!Array.isArray(rows)) return undefined
+    const row = rows.find((candidate) => candidate?.entry?.options?.id === id)
+    return isRecord(row?.override) ? row.override : undefined
+  } catch {
+    return undefined
+  }
+}
+
+/**
+ * Top-level preset declarations visible to configEditor.entries().
+ *
+ * A preset override whose `name` guard no longer matches is skipped by the
+ * Loader, so merge the configuration override back before reading its native
+ * `compaction-basic` config. Array values (the child `plugins` list) replace
+ * wholesale, matching DSH's preset-override semantics.
+ */
 function presetRowsOf(configEditor) {
   let entries
   try {
@@ -394,11 +442,16 @@ function presetRowsOf(configEditor) {
   if (!Array.isArray(entries)) return []
   return entries
     .filter((entry) => entry?.options?.name === '@deepseek-ai/dsh-agent-preset')
-    .map((entry) => ({
-      rowId: entry.options.id,
-      presetId: entry.options.config?.id,
-      config: entry.options.config,
-    }))
+    .map((entry) => {
+      const base = isRecord(entry.options.config) ? entry.options.config : {}
+      const override = configurationOverrideOf(configEditor, entry.options.id)
+      const config = override === undefined ? base : mergeConfigOverride(base, override)
+      return {
+        rowId: entry.options.id,
+        presetId: config.id,
+        config,
+      }
+    })
 }
 
 /** Find the target row entry (needed for the non-volatile trustedHosts write). */
@@ -550,13 +603,20 @@ export async function runMigration(ctx, options = {}) {
   const agentPresets = options.agentPresets ?? serviceOf(ctx, 'agentPresets')
   // describe().value only projects the *volatile* form, so read the Apollo
   // row's raw config for non-volatile fields (trustedHosts) and overlay the
-  // live volatile values on top.
+  // live volatile values on top. Finally merge the raw profile override:
+  // DSH skips a row whose old `name` guard no longer matches (historical
+  // `name: dsh-apollo`), and only configuration().override still carries the
+  // user's explicit values.
   const rawSource = targetEntryOf(configEditor, APOLLO_NS)?.options?.config
   const sourceBase = {
     ...(isRecord(options.config) ? options.config : {}),
     ...(isRecord(rawSource) ? rawSource : {}),
   }
-  const source = normalizeConfig({ ...sourceBase, ...(sourceDescriptor?.value ?? {}) })
+  const effectiveSource = { ...sourceBase, ...(sourceDescriptor?.value ?? {}) }
+  const source = normalizeConfig(mergeConfigOverride(
+    effectiveSource,
+    configurationOverrideOf(configEditor, APOLLO_NS),
+  ))
   const subagent = await migrateSubagent({ settings, configEditor, descriptors, source, log })
   const compact = await migrateCompact({ settings, configEditor, descriptors, agentPresets, log })
   const status = [subagent.status, compact.status].some((value) => value === 'error')