@morlay/dsh-preset 0.0.3 → 0.0.5

package/README.md

@@ -6,12 +6,16 @@
 
 ## 内容
 
-| 文件                       | 作用                                                                              |
-| -------------------------- | --------------------------------------------------------------------------------- |
-| `cordis.patch.yml`         | bundle patch：禁用官方 preset、注册本包 preset 为默认、声明个人 `llm-pi-ai` route |
-| `tool/generate-presets.ts` | 从上游生成 preset 的模块 + tsdown hooks                                           |
-| `dist/presets/standard/`   | 构建产物：自定义 preset「标准模式」（由上游 `standard` 生成）                     |
-| `dist/presets/ptc/`        | 构建产物：自定义 preset「PTC 模式」（由上游 `ptc` 生成）                          |
+| 文件                       | 作用                                                                                                                       |
+| -------------------------- | -------------------------------------------------------------------------------------------------------------------------- |
+| `cordis.patch.yml`         | bundle patch：禁用官方 preset、注册本包 preset 为默认、声明个人 `llm-pi-ai` route、覆盖沙箱规则、插入 `prompt-reminder` 行 |
+| `tool/generate-presets.ts` | 从上游生成 preset 的模块 + tsdown hooks                                                                                    |
+| `dist/presets/standard/`   | 构建产物：自定义 preset「标准模式」（由上游 `standard` 生成）                                                              |
+| `dist/presets/ptc/`        | 构建产物：自定义 preset「PTC 模式」（由上游 `ptc` 生成）                                                                   |
+
+`cordis.patch.yml` 里插入的 `prompt-reminder`（`@morlay/dsh-prompt-reminder`）把被
+裁掉的系统提示词 section 降级为 `<system-reminder>` user 消息，见「工具说明降级为
+reminder」。
 
 ## 禁用官方 preset
 
@@ -69,11 +73,40 @@ section 在插件构造时注册；dev 模式重启 `just custom dev`/`just cust
 roster（id 沿用官方名以走客户端语言字典），以及桌面形态下 preset 必须落在 dsh 包的
 `config/agent-presets` 挂载点（见 dsh-desktopify 的 `dsh.desktop.agentPresets`）。
 
+## 工具说明降级为 reminder，不进系统提示词
+
+上游把每个工具的跨调用说明（`tool:bash`、`tool:read`、…）、Agent Teams 协作规则、
+harness 源码位置、Web GUI 说明都注册成系统提示词的 section。本部署实测它们占
+9200 字符里的约 7400：条数多、篇幅大、挤在上下文最前面，稀释真正要遵守的那几行
+纪律。`cordis.patch.yml` 因此在 host plane 插一行：
+
+```yaml
+- insert:
+    - id: prompt-reminder
+      name: "@morlay/dsh-prompt-reminder"
+```
+
+一处定义覆盖全部 preset（standard / ptc 以及后续新增的）——这是与具体 preset 无关
+的部署级策略，preset composition 不各自带这一行，生成器也就不必碰它。该插件注册
+在 root scope，会收到每个 agent 的装配与 pre-step 事件（含子 agent）：
+
+- 装配（`system-prompt/assemble` 瀑布）时保留 `keep` 名单——默认是部署级
+  `deployment:persona-prefix` / `deployment:persona-suffix`——内的 section，其余
+  非空 section 从 assembly 移除并按 order 捕获文本；
+- pre-step 时把捕获的文本作为一条 `<system-reminder>` user 消息注入，紧随本轮
+  用户消息之后；文本不变不重复注入，`plan:policy` / `tools:ptc-only` 这类按模式
+  渲染的 section 变化时追加一条新的全量 reminder（首行声明最新一条覆盖更早的）。
+
+于是系统提示词只剩首尾两段个人提示词，其余内容仍在上下文里、但不再占系统提示词
+的位置，也不再随模式切换改动。**为什么不用 `persona` 行的 `complete: true`**：那会
+连 suffix 一起丢，且子 agent 自己注册同名 persona 时 `complete` 语义消失，保留 /
+降级会不一致；本插件按 `keep` 名单匹配 section 名，两种 agent 行为一致。完整行为与
+配置见 `packages/preset/dsh-prompt-reminder/README.md`。
+
 ## 工作区指令走官方行为，候选收紧为 AGENTS 系列
 
-preset 里的 `agent-instructions` 行仍是官方 `@deepseek-ai/dsh-agent-instructions`：
-本仓库的 fork `@morlay/dsh-agent-instructions-as-prompt` 暂不接入默认打包，baseline
-仍按官方语义作为 user 消息注入一次。生成器只覆盖该行 config 的两个字段：
+preset 里的 `agent-instructions` 行是官方 `@deepseek-ai/dsh-agent-instructions`：
+baseline 按官方语义作为 user 消息注入一次。生成器只覆盖该行 config 的两个字段：
 
 ```yaml
 instructionFileCandidates: ["AGENTS.md"] # 上游默认 ['AGENTS.md','CLAUDE.md']
@@ -82,8 +115,10 @@ localInstructionFileCandidates: ["AGENTS.local.md"] # 上游默认还含 CLAUDE.
 
 即**不读 CLAUDE 系列**（本部署不需要 CLAUDE 兼容）；`maxBytes` 等其余字段跟随上游。
 
-fork 源码保留在 `packages/preset/dsh-agent-instructions-as-prompt/`；后续要恢复时按
-它的 README 操作：生成器把该行 `name` 换成 fork 名，profile 侧再禁用上游行。
+本仓库曾有一个把 baseline 落点改成 system-prompt section 的 fork
+（`@morlay/dsh-agent-instructions-as-prompt`），一直未接入默认打包；提示词分层定为
+「系统提示词只留 persona、其余走 reminder」之后它已删除——baseline 留在 user 消息
+通道与这个分层一致。
 
 ## preset 由构建生成，不是手工副本
 
@@ -102,10 +137,11 @@ pnpm --filter @morlay/dsh-preset run generate-presets [outDir]
 ```
 
 脚本把上游 composition 当**数据**读入：`js-yaml` 用 include 的
-`entryListSchema` 解析（`!!js` 标签保留为表达式节点）→ 在 JS 里改一处（删掉
-`persona` 行）→ `dump` 回 YAML。因此上游任何结构性改动（新增 / 重命名 row、改字段）
-都自动跟随，不依赖易碎的文本锚点；上游若删掉 `persona` 行，目标（preset 不自带
-persona）本就达成，脚本照常通过。
+`entryListSchema` 解析（`!!js` 标签保留为表达式节点）→ 在 JS 里改三处（删掉
+`persona` 行、把 `agent-instructions` 行的候选收紧为 AGENTS 系列、插入
+`prompt-reminder` 行）→ `dump` 回 YAML。因此上游任何结构性改动（新增 / 重命名
+row、改字段）都自动跟随，不依赖易碎的文本锚点；上游若删掉 `persona` 行，目标
+（preset 不自带 persona）本就达成，脚本照常通过。
 
 脚本先**整目录清空**输出目录再生成：产物完全派生自脚本，残留目录（改过
 `source`、旧命名）不该留下——否则 discovery 会把它们当有效 preset 扫出来。
@@ -118,7 +154,23 @@ persona）本就达成，脚本照常通过。
 
 1. 升级 `DEEPSEEK_HARNESS_VERSION` 并 sync/patch/build。
 2. `pnpm --filter @morlay/dsh-preset run build`（build:done 会重新生成）
-3. `pnpm exec vitest run packages/preset/dsh-preset` 确认无 drift。
+3. `pnpm exec vitest run packages/preset` 确认无 drift（含 reminder 插件的行为测试）。
+
+## 沙箱装配与规则（部署级）
+
+本 bundle 的 `cordis.patch.yml` 里一次做完三件事：禁用官方 `sandbox` / `fs-sandbox`
+两行、插入 `- id: sandbox-local` 行、写上 `rw` / `r-` / `--` 条目（唯一真源，本 README
+不复制一份以免两处漂移）；条目语法、优先级与各平台表达能力见
+[`@morlay/dsh-sandbox-local` 的 README](../../sandbox/dsh-sandbox-local/README.md)。
+
+放本 bundle 而不是示例 app 的 `cordis.patch.yml`：bundle patch 在 dev（web / desktop）
+与打包形态都会应用，app 的 patch 只在打包时作为 seed 的 profile patch 生效。
+
+两个前置条件：
+
+- **本包的 `dependencies` 必须声明 `@morlay/dsh-sandbox-local`**：插入行的 `name` 要能在
+  profile 的依赖树里解析（profile 的 `nodeLinker: hoisted` 由本包的依赖把它带进去）。
+- **规则变化要重启**：profile 在启动时装载，`access` 在插件构造时解析（模板也是）。
 
 ## 装配
 
@@ -159,6 +211,13 @@ profile 里把内容物化到该挂载点；`includeShippedRoot: false` 两种
 ## 维护注意
 
 - `package.json` 的 `files` 必须含 `dist`（preset 在其中）与 `tool`，否则发布产物缺内容。
+- `cordis.patch.yml` 插入的 `@morlay/dsh-prompt-reminder` 必须能被 **profile** 的依赖
+  树解析：示例 app 已在 `dependencies` 声明；换工作区时要一并声明，否则该行加载失败、
+  工具说明会留在系统提示词里。
+- 沙箱装配（禁用官方两行 + 插入 `sandbox-local` 行 + 规则）全在本 bundle 的 patch 里，
+  app 的 `dsh.profile.bundles` 不需要额外列 `@morlay/dsh-sandbox-local`；
+  `@morlay/dsh-sandbox-local` 自带的 bundle patch 供把该包作为独立 bundle 采用的部署
+  使用，与本 bundle 同时上线会重复插入同一行。
 - 生成器显式设 `quotingType: '"'`：`yaml.dump` 默认单引号而仓库 oxfmt 偏好双引号，
   不指定会让生成器与 formatter 来回改；产物在 gitignore 的 dist 里，本就不参与 fmt。
 - **dev 模式需要先构建**：`just dev` / `just desktop` / `just bundle` 都先跑

package/cordis.patch.yml

@@ -2,21 +2,16 @@
   config:
     includeHarnessIdentity: false
     includeRuntimeContext: false
-    personaSuffix: 你的工作目录在 `{{cwd}}`
+    personaSuffix: |-
+      你的工作目录在 `{{cwd}}`，工作目录内的  AGENTS.md 的规则非常重要，必须严格遵守
     personaPrefix: |-
       你是一个经验丰富的编程专家，YAGNI 是你的编程哲学，PDCA 是你的行为规范
 
-      ## 指令遵循（强制）
+      ## 语言和行为
 
-      以下指令都是**约束**而非建议；冲突时按 用户的直接指令 > 工作区 `AGENTS.md` > 全局提示词 取舍并说明：
-
-      - **工作区 `AGENTS.md`**（仓库根与目录链，随 `<system-reminder>` 注入）：「红线」与命令约定必须遵守。模板里的 "may be relevant"、"as guidance" 是通用保守措辞，不表示可以跳过。
-      - **全局提示词**：
-        - 所有思考、分析、推理过程、工具调用描述等必须使用中文，专有名词除外
-        - 思考不要陷入重复循环，一旦循环，立即退出
-        - 思考聚焦需求理解与方案设计（要点、流程、验证策略），不预演具体代码实现；代码在 Do 阶段基于实际文件状态输出，正确性由 Check 验证
-
-      协议：动手前读相关 `AGENTS.md`（不凭记忆）；过程中自检、违反即纠正；交付前核对红线与验证证据，未验证项必须标注；无法遵守时立即说明（不静默忽略、不自行判定「不适用」）；派发子代理时把约束写进任务说明。
+      - 所有思考、分析、推理过程、工具调用描述等必须使用中文，专有名词除外
+      - 思考不要陷入重复循环，一旦循环，立即退出
+      - 思考聚焦需求理解与方案设计（要点、流程、验证策略），不预演具体代码实现；代码在 Do 阶段基于实际文件状态输出，正确性由 Check 验证
 
 - id: agent-presets
   config:
@@ -62,3 +57,34 @@
               low: low
               high: high
               max: max
+
+# 沙箱装配（唯一真源，就地完成）：官方 `sandbox`（ctx.sandbox）与 `fs-sandbox`
+# （ctx.fs）两行在这里禁用，由 `@morlay/dsh-sandbox-local` 注册同名服务；规则随本
+# insert 行一起写死，不再依赖任何额外 bundle 层（漏装配时规则会静默失效，只剩
+# workspace + /tmp 可写）。
+- id: sandbox
+  disabled: true
+
+- id: fs-sandbox
+  disabled: true
+
+# 沙箱规则：`rw <path>` 追加工作区之外的可写根，`-- <pattern>` 拒绝访问（读 + 写）。
+- insert:
+    - id: sandbox-local
+      name: "@morlay/dsh-sandbox-local"
+      config:
+        access: |-
+          rw /tmp
+          rw {{ env.XDG_CACHE_HOME }}
+          rw {{ env.XDG_STATE_HOME }}
+          rw {{ env.XDG_DATA_HOME }}
+          r- {{ env.XDG_CONFIG_HOME }}
+          -- mise.*.toml
+          -- **/*.pem
+
+# 提示词分层：系统提示词只留上面的 persona，工具说明 / Agent Teams 规则 / plan
+# 规则等由该插件降级为 <system-reminder> user 消息。部署级策略，在这里定义一次
+# 即覆盖全部 preset（含子 agent），preset composition 不再各自带这一行。
+- insert:
+    - id: prompt-reminder
+      name: "@morlay/dsh-prompt-reminder"

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@morlay/dsh-preset",
-  "version": "0.0.3",
-  "description": "Personal dsh profile bundle: disables the shipped agent presets, builds generated standard / ptc presets into dist, and declares personal llm-pi-ai provider routes.",
+  "version": "0.0.5",
+  "description": "Personal dsh profile bundle: disables the shipped agent presets, builds generated standard / ptc presets into dist, declares personal llm-pi-ai provider routes, and carries the deployment sandbox access rules.",
   "keywords": [
     "dsh",
     "dsh-bundle",
@@ -25,6 +25,10 @@
     "./package.json": "./package.json",
     "./cordis.patch.yml": "./cordis.patch.yml"
   },
+  "dependencies": {
+    "@morlay/dsh-prompt-reminder": "^0.0.2",
+    "@morlay/dsh-sandbox-local": "^0.0.2"
+  },
   "devDependencies": {
     "@deepseek-ai/cordis-plugin-include": "^1.0.7",
     "@deepseek-ai/dsh-agent-presets": "^0.1.5-rc.2",

package/src/__tests__/generated-presets.spec.ts

@@ -29,7 +29,7 @@ import {
 /** 上游 preset 里承载工作区指令的行 id。 */
 const INSTRUCTIONS_ROW_ID = "agent-instructions";
 
-/** 上游那行指向的官方插件（fork 暂未接入，产物须保持官方名）。 */
+/** 上游那行指向的官方插件（产物须保持官方名；提示词分层由 profile 的 patch 承担）。 */
 const UPSTREAM_INSTRUCTIONS_PLUGIN = "@deepseek-ai/dsh-agent-instructions";
 
 /** 产物里指令候选的文件名（字面量：不读 CLAUDE 系列）。 */
@@ -105,7 +105,7 @@ describe("generated presets", () => {
       expect(productRow?.config?.localInstructionFileCandidates).toEqual(["AGENTS.local.md"]);
       // 其余 config 字段跟随上游（如 maxBytes 的字节预算）。
       expect(productRow?.config?.maxBytes).toBe(upstreamRow?.config?.maxBytes);
-      // 产物里不应出现本仓库的插件行。
+      // 提示词分层（prompt-reminder）是 profile 的 bundle patch，不进 preset 产物。
       expect(
         product.some((row) => typeof row.name === "string" && row.name.startsWith("@morlay/")),
       ).toBe(false);

package/src/__tests__/patch.spec.ts

@@ -0,0 +1,96 @@
+// 装配守卫：prompt-reminder 是 host plane 的唯一装配点。被误删 / 改名时提示词分层
+// 静默失效（工具说明回到系统提示词），这里 fail loud；persona 是 keep 名单的唯一
+// 提供者，删掉它系统提示词会变空。
+//
+// 沙箱同理：官方 `sandbox` / `fs-sandbox` 的禁用与本包 `sandbox-local` 行的插入必须在
+// 同一层——官方行还在时 `ctx.sandbox` / `ctx.fs` 由官方实现提供，access 规则没有生效点，
+// 命令照跑但规则全部失效（只剩 workspace + /tmp 可写），且没有任何报错。
+
+import { readFileSync } from "node:fs";
+import { join } from "node:path";
+import { entryListSchema } from "@deepseek-ai/cordis-plugin-include";
+import yaml from "js-yaml";
+import { describe, expect, it } from "vitest";
+
+const PATCH_PATH = join(process.cwd(), "packages/preset/dsh-preset/cordis.patch.yml");
+const UPSTREAM_BASE_PATCH = join(
+  process.cwd(),
+  "vendor/deepseek-harness/packages/bundle/base/cordis.patch.yml",
+);
+
+/** patch 的一行；只用到覆盖目标、禁用与 insert 的新增行。 */
+interface PatchRow {
+  id?: string;
+  disabled?: boolean;
+  config?: Record<string, unknown>;
+  insert?: { id?: string; name?: string; config?: Record<string, unknown> }[];
+}
+
+// 用 include 的 entry schema 解析：patch 里含 `!!js` 表达式，默认 schema 会拒绝。
+const rows = yaml.load(readFileSync(PATCH_PATH, "utf8"), {
+  schema: entryListSchema,
+}) as PatchRow[];
+
+/** 本层 insert 出来的行。 */
+const inserted = rows.flatMap((row) => row.insert ?? []);
+
+/** 本层的 `sandbox-local` 行（装配沙箱替换的那个）。 */
+const sandboxRow = inserted.find((row) => row.id === "sandbox-local");
+
+describe("dsh-preset patch wiring", () => {
+  it("declares the prompt-reminder row exactly once, on the local plugin", () => {
+    const reminders = inserted.filter((row) => row.id === "prompt-reminder");
+
+    expect(reminders).toHaveLength(1);
+    expect(reminders[0]?.name).toBe("@morlay/dsh-prompt-reminder");
+  });
+
+  it("keeps the deployment persona the reminder keeps in the system prompt", () => {
+    const config = rows.find((row) => row.id === "system-prompt")?.config;
+
+    expect(config?.personaPrefix).toBeTruthy();
+    expect(config?.personaSuffix).toBeTruthy();
+  });
+
+  it("disables the shipped sandbox rows and mounts the replacement in one layer", () => {
+    expect(rows.filter((row) => row.disabled === true).map((row) => row.id)).toEqual([
+      "sandbox",
+      "fs-sandbox",
+    ]);
+    expect(inserted.filter((row) => row.id === "sandbox-local")).toHaveLength(1);
+    expect(sandboxRow?.name).toBe("@morlay/dsh-sandbox-local");
+  });
+
+  it("names rows that still exist in the shipped base bundle", () => {
+    // 上游改名或移动这两行时，本层的 disabled 会被 include 静默跳过（warn+skip），
+    // 沙箱替换随之失效；这里对着 vendor 里的 base patch 校验一次。
+    const upstreamIds = new Set(
+      [...readFileSync(UPSTREAM_BASE_PATCH, "utf8").matchAll(/^\s*- id: (\S+)$/gm)].map(
+        (match) => match[1]!,
+      ),
+    );
+
+    for (const id of ["sandbox", "fs-sandbox"]) {
+      expect(upstreamIds.has(id), `missing upstream row: ${id}`).toBe(true);
+    }
+  });
+
+  it("carries the deployment sandbox rules on the inserted row", () => {
+    // 规则随本 bundle 分发（跨 dev / 打包形态一致），不再依赖另一层 bundle。
+    const access = sandboxRow?.config?.access;
+
+    expect(typeof access).toBe("string");
+    expect(String(access)).toContain("rw {{ env.XDG_CACHE_HOME }}");
+    expect(String(access)).toContain("rw {{ env.XDG_DATA_HOME }}");
+    expect(String(access)).toContain("-- mise.*.toml");
+    expect(String(access)).toContain("-- **/*.pem");
+  });
+
+  it("depends on the bundle whose service classes the inserted row mounts", () => {
+    const manifest = JSON.parse(
+      readFileSync(join(process.cwd(), "packages/preset/dsh-preset/package.json"), "utf8"),
+    ) as { dependencies?: Record<string, string> };
+
+    expect(manifest.dependencies?.["@morlay/dsh-sandbox-local"]).toBeTruthy();
+  });
+});

package/tool/generate-presets.ts

@@ -9,8 +9,8 @@
  * `agent-instructions` 行的指令候选收紧为 AGENTS 系列），再 `dump` 回 YAML。
  * 上游任何结构性改动（新增/重命名 row、改字段）都自动跟随，不依赖易碎的文本锚点。
  *
- * `agent-instructions` 行仍是上游官方插件：fork
- * （`@morlay/dsh-agent-instructions-as-prompt`）暂不接入，工作区指令走官方行为。
+ * `agent-instructions` 行是上游官方插件：工作区指令的 baseline 留在 user 消息通道，
+ * 与「系统提示词只留 persona、其余走 reminder」的分层一致。
  *
  * 产物落在 `dist/` 而非源码树：dist 是构建输出（gitignore），既不会与 oxfmt
  * 互相改格式，也不会把派生文件混进源码。