dsh-custom-mode 1.9.6 → 1.9.8

package/README.md

@@ -2,11 +2,12 @@
 
 ![dsh-custom-mode — a settings page for dsh agent modes](https://raw.githubusercontent.com/BOWLUNA/dsh-custom-mode/main/docs/images/header.png)
 
-[![npm](https://img.shields.io/npm/v/dsh-custom-mode?label=npm&style=flat-square&logo=npm&logoColor=white)](https://www.npmjs.com/package/dsh-custom-mode) [![CI](https://img.shields.io/github/actions/workflow/status/BOWLUNA/dsh-custom-mode/test.yml?label=CI&style=flat-square&logo=githubactions&logoColor=white)](https://github.com/BOWLUNA/dsh-custom-mode/actions/workflows/test.yml) [![license](https://img.shields.io/npm/l/dsh-custom-mode?label=license&style=flat-square&logo=opensourceinitiative&logoColor=white)](LICENSE)
+[![npm](https://img.shields.io/npm/v/dsh-custom-mode?label=npm&style=flat-square&logo=npm&logoColor=white&labelColor=1f2430)](https://www.npmjs.com/package/dsh-custom-mode) [![CI](https://img.shields.io/github/actions/workflow/status/BOWLUNA/dsh-custom-mode/test.yml?label=CI&style=flat-square&logo=githubactions&logoColor=white&labelColor=1f2430)](https://github.com/BOWLUNA/dsh-custom-mode/actions/workflows/test.yml) [![license](https://img.shields.io/badge/license-MIT-97ca00?style=flat-square&logo=opensourceinitiative&logoColor=white&labelColor=1f2430)](LICENSE) [![dsh](https://img.shields.io/badge/dsh-%E2%89%A50.1.5--rc.2-4d6bfe?style=flat-square&logo=deepseek&logoColor=white&labelColor=1f2430)](https://github.com/BOWLUNA/dsh-custom-mode#readme)
 
-<!-- community badge row: required in every new repository (see the storefront template in the operator SOP/DSH Plugins folder) -->
+<!-- badge rows: same two-part structure (dark label + coloured value) in both rows; every new repo copies this block -->
+[![bilibili](https://img.shields.io/badge/bilibili-videos-%2300A1D6?style=flat-square&logo=bilibili&logoColor=white&labelColor=1f2430)](https://b23.tv/qJ4Ev0W) [![Douyin](https://img.shields.io/badge/Douyin-shorts-%23FE2C55?style=flat-square&logo=tiktok&logoColor=white&labelColor=1f2430)](https://v.douyin.com/VWh0M03Fa4Y/) [![RedNote](https://img.shields.io/badge/RedNote-notes-%23FF2442?style=flat-square&logo=xiaohongshu&logoColor=white&labelColor=1f2430)](https://xhslink.cn/o/A7QtXmePBBF) [![Discord](https://img.shields.io/badge/Discord-chat-%235865F2?style=flat-square&logo=discord&logoColor=white&labelColor=1f2430)](https://discord.gg/pz97SfAfSy) [![GitHub](https://img.shields.io/github/discussions/BOWLUNA/dsh-custom-mode?label=GitHub&style=flat-square&logo=github&logoColor=white&labelColor=1f2430)](https://github.com/BOWLUNA/dsh-custom-mode/discussions)
 
-[![bilibili](https://img.shields.io/badge/bilibili-%2300A1D6?style=flat-square&logo=bilibili&logoColor=white)](https://b23.tv/qJ4Ev0W) [![Douyin](https://img.shields.io/badge/Douyin-%23FE2C55?style=flat-square&logo=tiktok&logoColor=white)](https://v.douyin.com/VWh0M03Fa4Y/) [![RedNote](https://img.shields.io/badge/RedNote-%23FF2442?style=flat-square&logo=xiaohongshu&logoColor=white)](https://xhslink.cn/o/A7QtXmePBBF) [![Discord](https://img.shields.io/badge/Discord-%235865F2?style=flat-square&logo=discord&logoColor=white)](https://discord.gg/pz97SfAfSy) [![Discussions](https://img.shields.io/github/discussions/BOWLUNA/dsh-custom-mode?label=Discussions&style=flat-square&logo=github&logoColor=white)](https://github.com/BOWLUNA/dsh-custom-mode/discussions)
+<!-- community badge row: required in every new repository (see the storefront template in the operator SOP/DSH Plugins folder) -->
 
 The **settings-page half** of [dsh-custom-mode](https://github.com/BOWLUNA/dsh-custom-mode) — a Web UI for
 [DeepSeek Harness](https://github.com/deepseek-ai/deepseek-harness) (dsh) agent modes: choose a mode's base
@@ -15,7 +16,8 @@ agent loop re-reads before every model step — **a save applies on the next ste
 session**. Several modes ("assistants") live side by side, each with its own prompt.
 
 Searched for as: custom mode · custom prompt · system-prompt editor · multi-mode / several assistants ·
-自定义模式 · 自定义提示词 · 多助手／多模式.
+multi-agent · roleplay (RP) / chat personas ·
+自定义模式 · 自定义提示词 · 多助手／多模式 · 角色扮演／聊天人格（RP）.
 
 ![Assistant manager](https://raw.githubusercontent.com/BOWLUNA/dsh-custom-mode/main/docs/images/05-assistant-manager.png)
 

package/assistants.mjs

@@ -28,7 +28,10 @@
 import { existsSync, mkdirSync, readFileSync, readdirSync, writeFileSync } from 'node:fs'
 import { dirname, join } from 'node:path'
 import { dshHome, PRESET_DIR } from './paths.mjs'
-import { readPresetMeta, writePresetMeta } from './meta.mjs'
+// `presetMetaPath` 曾被漏掉：reorderAssistant 用它做"快照 → 失败回滚"，于是两处调用都抛
+// ReferenceError 被 try/catch 吞掉，回滚循环 continue 掉每一条 —— 用户看到的"已回滚"从未发生，
+// 磁盘上留下改了一半的 order。少一个导入就能让一整段安全网变成死代码（外部评审复现）。
+import { readPresetMeta, writePresetMeta, presetMetaPath } from './meta.mjs'
 import { writeAtomic } from './atomic.mjs'
 import { unresolvableRows } from './composition.mjs'
 import { seedPreset, seedPresetWithLog } from './seed.mjs'
@@ -256,20 +259,22 @@ export function allocateId(name, taken) {
  * @returns {{ok: true, id: string, dir: string} | {ok: false, error: string}}
  */
 export function createAssistantDir({ root, id, composition, templateDir }) {
+  // 每个用户可见的结果都带 `code`（AGENTS.md 第 13 条）：没有它，页面只能显示下面这串中文，
+  // 英文界面就在这一刻掉回中文。前三条还会被设置页原样显示给用户。
   if (!PRESET_ID.test(String(id ?? ''))) {
-    return { ok: false, error: `助手标识不合法：${String(id)}（只能是 a-z0-9 与连字符）` }
+    return { ok: false, code: 'badAssistantId', params: { id: String(id) }, error: `助手标识不合法：${String(id)}（只能是 a-z0-9 与连字符）` }
   }
   const dir = join(root, id)
-  if (existsSync(dir)) return { ok: false, error: `目录已存在：${dir}` }
+  if (existsSync(dir)) return { ok: false, code: 'dirExists', params: { path: dir }, error: `目录已存在：${dir}` }
 
   const seeded = seedPreset(dir, templateDir)
   if (seeded.errors.length > 0) {
-    return { ok: false, error: '复制模式模板失败：' + seeded.errors.join('；') }
+    return { ok: false, code: 'seedFailed', params: { detail: seeded.errors.join('；') }, error: '复制模式模板失败：' + seeded.errors.join('；') }
   }
   try {
     writeFileSync(join(dir, COMPOSITION_FILE), composition, 'utf8')
   } catch (error) {
-    return { ok: false, error: '写入组成文件失败：' + describe(error) }
+    return { ok: false, code: 'writeFailed', params: { detail: describe(error) }, error: '写入组成文件失败：' + describe(error) }
   }
   return { ok: true, id, dir }
 }

package/client.js

@@ -138,6 +138,13 @@ try {
         "api.promptWriteFailed": "写入提示词失败：{detail}",
         "api.renderFailed": "生成组成文件失败：{detail}",
         "api.selfCheckFailed": "生成结果自检失败，已放弃写入：{detail}",
+        "api.internalError": "宿主内部错误：{detail}",
+        "api.bodyTooLarge": "请求体过大（上限 {max} 字节）",
+        "api.repairNotNeeded": "这个助手在本机没有无法解析的行，无需修复。",
+        "api.repaired": "已按本机这条 dsh 线关闭 {count} 个无法解析的行（{ids}）。现在这个模式能重新出现在选择器里。",
+        "api.badAssistantId": "助手标识不合法：{id}（只能用 a-z、0-9 与连字符）",
+        "api.seedFailed": "复制模式模板失败：{detail}",
+        "api.metaWriteFailed": "写入 preset.yml 失败：{detail}",
         "api.promptReadFailed": "读取提示词失败：{detail}",
         "api.noRemoveApi": "当前 DSH 版本没有 agentPresets.remove()，无法删除。",
         "api.deleteFailed": "删除失败：{detail}",
@@ -163,7 +170,7 @@ try {
         "msg.exported": "已导出为文件。",
         "msg.exportFailed": "导出失败",
         "delete.title": "永久删除这个助手？",
-        "delete.description": "这会删除磁盘上的模式目录，连同它的系统提示词一起消失，无法撤销。正在使用它的会话不受影响；新建会话时它不再出现在选择器里。",
+        "delete.description": "这会删除磁盘上的模式目录，连同它的系统提示词一起消失，无法撤销。正在使用它的会话不受影响；新建会话时它不再出现在选择器里。（{id}）",
         "delete.acknowledge": "我明白这个助手的提示词会被永久删除",
         "delete.confirm": "永久删除",
         "delete.close": "关闭",
@@ -193,6 +200,7 @@ try {
         "msg.readFailed": "读取失败",
         "msg.saveFailed": "保存失败",
         "msg.saved": "已保存。新建会话即生效，当前会话保持原配置。",
+        "status.detail": "{message}：{detail}",
         "row.persona.label": "身份（系统提示词）",
         "row.persona.note": "提示词注入点；关掉后本模式用回部署默认身份",
         "row.custom-prompt-tool.label": "custom_prompt 工具",
@@ -338,6 +346,13 @@ try {
         "api.promptWriteFailed": "Writing the prompt failed: {detail}",
         "api.renderFailed": "Rendering the composition failed: {detail}",
         "api.selfCheckFailed": "The generated composition failed its self-check, so nothing was written: {detail}",
+        "api.internalError": "Unexpected host error: {detail}",
+        "api.bodyTooLarge": "Request body too large (limit {max} bytes)",
+        "api.repairNotNeeded": "Nothing to repair — no row of this assistant is unresolvable on this machine.",
+        "api.repaired": "Disabled {count} row(s) this dsh line cannot resolve ({ids}). The mode can appear in the pickers again.",
+        "api.badAssistantId": "Invalid assistant id: {id} (a-z, 0-9 and hyphens only)",
+        "api.seedFailed": "Copying the mode template failed: {detail}",
+        "api.metaWriteFailed": "Writing preset.yml failed: {detail}",
         "api.promptReadFailed": "Reading the prompt failed: {detail}",
         "api.noRemoveApi": "This DSH build has no agentPresets.remove(), so deleting is unavailable.",
         "api.deleteFailed": "Delete failed: {detail}",
@@ -363,7 +378,7 @@ try {
         "msg.exported": "Exported to a file.",
         "msg.exportFailed": "Export failed",
         "delete.title": "Delete this assistant permanently?",
-        "delete.description": "This removes the mode directory from disk, system prompt included, and cannot be undone. Sessions already using it keep running; new sessions no longer offer it.",
+        "delete.description": "This removes the mode directory from disk, system prompt included, and cannot be undone. Sessions already using it keep running; new sessions no longer offer it. ({id})",
         "delete.acknowledge": "I understand this assistant's prompt will be deleted permanently",
         "delete.confirm": "Delete permanently",
         "delete.close": "Close",
@@ -393,6 +408,7 @@ try {
         "msg.readFailed": "Load failed",
         "msg.saveFailed": "Save failed",
         "msg.saved": "Saved. A new session picks it up; the current one keeps its configuration.",
+        "status.detail": "{message}: {detail}",
         "row.persona.label": "Identity (system prompt)",
         "row.persona.note": "Where the prompt is injected; turning it off falls back to the deployment identity",
         "row.custom-prompt-tool.label": "custom_prompt tool",
@@ -1024,6 +1040,19 @@ try {
             const result = await postJson(ROUTES.repair, { id: selected })
             if (result !== null && result.ok === true) {
               setStatus(apiText(result, "msg.repaired"))
+              // ★ 把修复折进**草稿**（issue #8）。
+              //
+              // 修复在磁盘上是生效的，但草稿里那几行仍然是"启用"。而 `reload()` 对脏草稿是
+              // **保留**的（`open()` 的 `!sameDraft(...)` 保护），所以草稿不会自己更新；
+              // 于是下一次保存会按草稿重渲染，把刚修好的行原样打开 —— 模式再次从所有选择器里消失。
+              // 实测（2026-09-21）：修复后磁盘上 ghost-row 已关闭，保存一次就又被打开了。
+              const repairedIds = Array.isArray(result.params?.repairedIds) ? result.params.repairedIds : []
+              if (repairedIds.length > 0 && draft !== null) {
+                // 草稿里的 `overrides` 是"显式**启用**"语义，所以关闭 = false。
+                const nextOverrides = { ...draft.overrides }
+                for (const rowId of repairedIds) nextOverrides[rowId] = false
+                update({ overrides: nextOverrides }, { force: true })
+              }
               await reload()
             } else {
               setFailed(true)
@@ -1169,7 +1198,7 @@ try {
             } catch (error) {
               if (alive) {
                 setFailed(true)
-                setStatus(t("msg.readFailed") + "：" + describeError(error))
+                setStatus(fillPlaceholders(t("status.detail"), { message: t("msg.readFailed"), detail: describeError(error) }))
               }
             } finally {
               if (alive) setBusy(false)
@@ -1181,7 +1210,20 @@ try {
         }, [])
 
         /** Change one field of the selected assistant's draft. */
-        const update = (patch) => {
+        const update = (patch, options) => {
+          // ★ 忙碌期间**不接受**对草稿的修改（issue #6）。
+          //
+          // 为什么必须在这里拦：保存成功后（下面 `save()` 里）会用服务器归一化的结果
+          // `value: normalized` **无条件**覆盖草稿。于是往返期间敲进去的字、翻过的行开关都会
+          // 凭空消失，而页面还显示 "Saved" —— 用户丢的是手写的系统提示词。
+          //
+          // 拦在这一层而不是逐个控件加 `disabled`，是因为它一处覆盖**所有**草稿字段
+          // （name / description / prompt / overrides），漏掉任何一个都会重新打开那条丢失路径。
+          // 视觉上控件仍可点，但点了不会改状态，所以不会出现"开关翻过去又弹回来"这种骗人的反馈。
+          //
+          // `{ force: true }` 是给**内部**调用用的（例如「按本线修复」把结果折进草稿）：
+          // 那不是用户输入，不受这条守护约束。
+          if (busy === true && options?.force !== true) return
           setEntries((previous) => {
             const current = previous[selected]
             if (current === undefined) return previous
@@ -1201,7 +1243,7 @@ try {
             await open(id)
           } catch (error) {
             setFailed(true)
-            setStatus(t("msg.readFailed") + "：" + describeError(error))
+            setStatus(fillPlaceholders(t("status.detail"), { message: t("msg.readFailed"), detail: describeError(error) }))
           } finally {
             setBusy(false)
           }
@@ -1233,7 +1275,7 @@ try {
             }
           } catch (error) {
             setFailed(true)
-            setStatus(t("msg.createFailed") + "：" + describeError(error))
+            setStatus(fillPlaceholders(t("status.detail"), { message: t("msg.createFailed"), detail: describeError(error) }))
           } finally {
             setBusy(false)
           }
@@ -1261,7 +1303,7 @@ try {
             }
           } catch (error) {
             setFailed(true)
-            setStatus(t("msg.reorderFailed") + "：" + describe(error))
+            setStatus(fillPlaceholders(t("status.detail"), { message: t("msg.reorderFailed"), detail: describe(error) }))
           } finally {
             setBusy(false)
           }
@@ -1294,7 +1336,7 @@ try {
             }
             update({ prompt: payload.text })
           } catch (error) {
-            setStatus(t("msg.loadFailed") + "：" + describeError(error))
+            setStatus(fillPlaceholders(t("status.detail"), { message: t("msg.loadFailed"), detail: describeError(error) }))
           }
         }
 
@@ -1313,7 +1355,7 @@ try {
             setStatus(t("msg.exported"))
           } catch (error) {
             setFailed(true)
-            setStatus(t("msg.exportFailed") + "：" + describe(error))
+            setStatus(fillPlaceholders(t("status.detail"), { message: t("msg.exportFailed"), detail: describe(error) }))
           }
         }
 
@@ -1375,7 +1417,7 @@ try {
             }
           } catch (error) {
             setFailed(true)
-            setStatus(t("msg.deleteFailed") + "：" + describeError(error))
+            setStatus(fillPlaceholders(t("status.detail"), { message: t("msg.deleteFailed"), detail: describeError(error) }))
           } finally {
             setBusy(false)
           }
@@ -1438,7 +1480,7 @@ try {
             }
           } catch (error) {
             setFailed(true)
-            setStatus(t("msg.saveFailed") + "：" + describeError(error))
+            setStatus(fillPlaceholders(t("status.detail"), { message: t("msg.saveFailed"), detail: describeError(error) }))
           } finally {
             setBusy(false)
           }
@@ -1743,6 +1785,11 @@ try {
                   className: "cpfe-editor",
                   value: draft.prompt,
                   spellCheck: false,
+                  // 忙碌期间只读（issue #6）：`update()` 那一层已经会丢弃改动，这里再给一个
+                  // **看得见**的信号 —— 否则用户会以为自己在编辑，而字根本没进去。
+                  // 用 readOnly 而不是 disabled：仍然可以选中/复制，只是改不了。
+                  readOnly: busy === true,
+                  "aria-busy": busy === true,
                   "aria-label": t("prompt.heading"),
                   onChange: (event) => update({ prompt: event.target.value }),
                 }),
@@ -1864,7 +1911,7 @@ try {
             ? react.createElement(A.RiskConfirmation, {
                 open: deleteOpen,
                 title: t("delete.title"),
-                description: t("delete.description") + "（" + draft.id + "）",
+                description: fillPlaceholders(t("delete.description"), { id: draft.id }),
                 acknowledgeLabel: t("delete.acknowledge"),
                 cancelLabel: t("btn.cancel"),
                 closeLabel: t("delete.close"),

package/composition.mjs

@@ -593,8 +593,15 @@ export function disableRowsInPlace(text, ids) {
   if (wanted.length === 0) return text
   // `applyLevel` 把 Map 的值直接交给 `setDisabled(...)`，所以值是**关闭**布尔（true = 关闭）。
   const off = new Map(wanted.map((id) => [id, true]))
-  const top = applyLevel(text, true, off, false, false)
-  return applyLevel(top, false, off, false, false)
+  // **只跑顶级这一遍。** 它已经通过 applyLevel 的 `isGroup` 分支递归进每个分组，
+  // 所以原先跟在后面的那次扁平扫描（`applyLevel(top, false, …)`）纯属重复 —— 而且有害：
+  // 它以「下一个 4 空格 `- id:`」界定片段，于是**分组的最后一个子行会吞掉后续的顶层内容**
+  // （包括下一个分组的 `name:` 头），`ownKeyIndent` 随即算出缩进 2，把 `disabled` 写进了
+  // **下一个分组**。实测（2026-09-21，4 个模式 × 每个行 id 全扫）：连带损害 7 例
+  // （关 plan-mode 会连带关掉整个 compaction 分组；关 tool-result-pruner 会连带关掉整个
+  // delegation 分组）、行数 Δ=+2 的结构异常 6 例、以及若干"目标行根本没被改到"的空操作。
+  // 删掉这一遍之后：连带 0 例、结构异常 0 例，正确生效 81 → 87。
+  return applyLevel(text, true, off, false, false)
 }
 
 export function unresolvableRows(text) {
@@ -613,15 +620,25 @@ export function unresolvableRows(text) {
     const row = /^( {0,4})- id: (.+?)\s*$/.exec(lines[index])
     if (row === null) continue
     const indent = row[1].length
+    // 这一行**自己的键**所在的列（`- id:` 再进两格）。只有这一列上的 `name:` / `disabled:`
+    // 才属于它 —— 否则会读到子树里的键。
+    const keyIndent = indent + 2
     let name
     let disabled = false
     for (let next = index + 1; next < lines.length; next += 1) {
       const line = lines[next]
-      if (line.trim() !== '' && line.search(/\S/) <= indent) break
-      if (/^\s+name: ['"]?(.+?)['"]?\s*$/.test(line)) name = /^\s+name: ['"]?(.+?)['"]?\s*$/.exec(line)[1]
+      if (line.trim() === '') continue
+      const column = line.search(/\S/)
+      if (column <= indent) break
+      // **跳过比自己深的行**（子行的键、子行的键的子行……）。
+      // 原先这里没有这一层过滤，于是顶层分组会一路走完整个子树，`name` 被**最后一个子行**覆盖：
+      // 一个健康的分组因此被报成"解析不了"（外部实测：`grp` 被报成坏行，name 取自 child-last），
+      // 后果是页面点名它、启动日志喊它、而"按本线修复"会**关掉整个分组**。
+      if (column !== keyIndent) continue
+      if (/^name: ['"]?(.+?)['"]?\s*$/.test(line.slice(keyIndent))) name = /^name: ['"]?(.+?)['"]?\s*$/.exec(line.slice(keyIndent))[1]
       // 平台条件行（`disabled: !!js …`）必须**求值**判定：只看有没有字面 `true` 会把"本平台已启用"的行
       // 误判成关闭，也会把"本平台本来就关闭"的行（如 Windows 专用的 pwsh）误报成"无法解析"。
-      const flag = /^\s+disabled:\s*(.+?)\s*$/.exec(line)
+      const flag = /^disabled:\s*(.+?)\s*$/.exec(line.slice(keyIndent))
       if (flag !== null) {
         // 字面量直接读，`!!js` 一类交给求值器（它对字面量不做布尔化）。
         const raw = flag[1].replace(/^['"]|['"]$/g, '')

package/index.mjs

@@ -196,8 +196,12 @@ export function readPrompt(directory) {
 
 /** The isolation message for an id the roster does not describe as a managed assistant. */
 function unknownAssistant(id) {
+  // 带 `code`：两个标签页同时开着、在 B 里删掉这个助手、回到 A 点保存时走的就是这条 ——
+  // 没有 code 时英文界面会显示下面这串中文，而词典里的 `api.unknownAssistant` 反而永远用不上。
   return {
     ok: false,
+    code: 'unknownAssistant',
+    params: { id: String(id) },
     error:
       '找不到助手「' +
       String(id) +
@@ -277,7 +281,19 @@ export function readState(rows, id, options = {}) {
   const meta = readPresetMeta(directory)
   // 页面要打开时顺便对账：磁盘上的文本若与日志末条不同，说明它在设置页之外被改过
   // （会话内的 custom_prompt 工具、手工编辑、别处同步）—— 补记一条 external，于是"被改过"看得见。
-  if (prompt.ok === true) recordExternalChange(directory, prompt.text)
+  //
+  // **必须包起来**：这是一次"写透"（recordPrompt → writeAtomic），而日志路径本身可能是坏的
+  // （被换成目录、被 chmod 000、一次 sudo 跑 dsh 留下的 root 属主）。1.9.7 给 readEntries 加的
+  // 守卫只覆盖了**读**那一半，于是同一个损坏场景会在这里重新把整个 GET state 打成 500 ——
+  // 页面打不开这个助手，用户既看不到也改不了自己的提示词（外部实测：EISDIR on rename）。
+  // 留痕失败降级为"这一版没记上"，与 saveState 里已经做过的处置一致。
+  if (prompt.ok === true) {
+    try {
+      recordExternalChange(directory, prompt.text)
+    } catch {
+      /* 审计留痕失败不影响读取 */
+    }
+  }
   return {
     ok: true,
     id,
@@ -414,13 +430,22 @@ export function saveState(rows, input) {
       [compositionFile(directory), composition],
       [promptFile(directory), prompt],
     ])
-    // 改动留痕：三个改动路径（设置页 / 会话内工具 / 手工编辑）里，只有设置页是"当场知道"的。
-    // 另外两条由 readState 对比补记（见 journal.mjs 的单写者说明）。
-    recordPrompt(directory, prompt, HISTORY_SOURCE.settings)
   } catch (error) {
     return { ok: false, code: 'writeFailed', params: { detail: describe(error) }, error: '写入失败：' + describe(error) }
   }
 
+  // 改动留痕：三个改动路径（设置页 / 会话内工具 / 手工编辑）里，只有设置页是"当场知道"的。
+  // 另外两条由 readState 对比补记（见 journal.mjs 的单写者说明）。
+  //
+  // **在写入 try 之外**：这是一条审计记录，它失败不该让一次已经落盘的保存报 writeFailed
+  // ——外部评审复现过：日志文件被 chmod 000 时，prompt.md 已经写成功而页面显示"保存失败"，
+  // 页面又不会重新读取，于是它永远停在旧草稿上。留痕失败只降级为"这一版没记上"。
+  try {
+    recordPrompt(directory, prompt, HISTORY_SOURCE.settings)
+  } catch {
+    /* 审计记录失败不影响保存本身 */
+  }
+
   // A name the user cleared is left alone rather than written as an empty scalar:
   // an empty `preset.yml` name is what makes a mode render as its bare id.
   if (name !== '') {
@@ -494,7 +519,14 @@ export function createAssistant(rows, input, templateDir = packagedPresetDir())
     // 显示名（`rows` 里的 name）优先于内部 id —— 与删除一致。
     const fromRow = rows.find((item) => item !== null && typeof item === 'object' && item.id === from)
     if (typeof fromRow?.name === 'string' && fromRow.name.trim() !== '') fromDisplayName = fromRow.name
-    const text = readFileSync(fromComposition, 'utf8')
+    // 与 readState 同一条纪律：**坏的源文件要给类型化错误，不能把异常抛到路由层**
+    // （抛上去就变成 500 internalError，页面只能显示 Node 的原始错误串）。
+    let text
+    try {
+      text = readFileSync(fromComposition, 'utf8')
+    } catch (error) {
+      return { ok: false, code: 'compositionMissing', params: { path: fromComposition, detail: describe(error) }, error: '读不到源助手的组成文件：' + describe(error) }
+    }
     const sourceMode = modeOf(text)
     const sourcePrompt = readPrompt(fromDir)
     source = {
@@ -511,7 +543,10 @@ export function createAssistant(rows, input, templateDir = packagedPresetDir())
       ? renderComposition('standard', new Map(), { modeName: name, assistantId: id })
       : renderComposition(source.mode, source.overrides, { modeName: name, assistantId: id })
   } catch (error) {
-    return { ok: false, error: '生成组成文件失败：' + describe(error) }
+    // 与保存路径的 `renderFailed` 同一套：**每个**用户可见的结果都要带 `code`，否则页面无从本地化
+    // —— client.js 的 apiText 在没有 code 时直接退回下面这串中文，英文界面会在这一刻掉回中文
+    // （AGENTS.md 第 13 条）。这条曾经是宿主半唯一漏掉 code 的普通返回。
+    return { ok: false, code: 'renderFailed', params: { detail: describe(error) }, error: '生成组成文件失败：' + describe(error) }
   }
 
   const created = createAssistantDir({ root, id, composition, templateDir })
@@ -584,7 +619,10 @@ export function repairComposition(rows, input) {
     ok: true,
     id,
     code: 'repaired',
-    params: { count: bad.length, ids },
+    // `ids` 是给人看的一串；`repairedIds` 是给**页面**用的数组 —— 页面必须把这几行写进它自己的
+    // 草稿，否则下一次保存会按草稿重渲染，把刚才的修复原样撤销（issue #8 实测：
+    // 修复写进了磁盘，但草稿里那几行仍是启用的，于是保存后 ghost 行又回来了）。
+    params: { count: bad.length, ids, repairedIds: bad.map((row) => row.id) },
     note: '已按本机这条 dsh 线关闭 ' + String(bad.length) + ' 个无法解析的行（' + ids + '）。现在这个模式能重新出现在选择器里。',
   }
 }
@@ -819,7 +857,9 @@ export function apply(ctx) {
         const result = await serializedWrite('delete:' + targetId, async () => deleteAssistant(await roster(), parsed, scope.agentPresets))
         return json(result, result.ok === true ? 200 : 400)
       } catch (error) {
-        return json({ ok: false, error: describe(error) }, 500)
+        // 兜底也要带 code：这是**意料之外**的异常，页面拿到的是 Node 的原始错误串，方向反着也一样糟
+        // ——中文界面里会蹦出一句英文。带 code 后页面用自己的词典渲染，原始串只作为 `detail` 参数。
+        return json({ ok: false, code: 'internalError', params: { detail: describe(error) }, error: describe(error) }, 500)
       }
     }
 

package/journal.mjs

@@ -55,8 +55,18 @@ export function historyFile(directory) {
 export function readEntries(directory) {
   const file = historyFile(directory)
   if (!existsSync(file)) return []
+  let raw = ''
+  try {
+    raw = readFileSync(file, 'utf8')
+  } catch {
+    // **坏的是文件本身**（不是某一行）：被 chmod 000（一次 sudo 跑 dsh 就会把 root 属主留在
+    // 日志上）、被改成目录、或读到一半的 EIO。此前这里裸读，于是这一个文件能同时做到两件坏事 ——
+    // 让 GET state（页面打开某个助手的入口）永远 500，以及让一次**已经写成功**的保存报 writeFailed。
+    // 模块自己的承诺是"坏一行不该赔上全部版本"，那么坏一个文件更不该赔上这个助手。
+    return []
+  }
   const entries = []
-  for (const line of readFileSync(file, 'utf8').split('\n')) {
+  for (const line of raw.split('\n')) {
     if (line.trim() === '') continue
     try {
       const parsed = JSON.parse(line)
@@ -82,13 +92,16 @@ export function readEntries(directory) {
  * @param {string} directory - one assistant's preset directory.
  * @param {string} text - the prompt content **after** the change.
  * @param {string} by - {@link HISTORY_SOURCE}.
- * @returns {{ recorded: boolean, at: string, n: number }} whether a line was appended.
+ * @returns {{ recorded: boolean, at: string, n: number }} whether a line was appended. `n` is the
+ *   revision this text already corresponds to when `recorded` is false, and `0` when the input could
+ *   not be recorded at all — never `undefined`, so a caller can feed it straight back to
+ *   {@link readVersion} without a guard that nobody would remember to write.
  */
 export function recordPrompt(directory, text, by = HISTORY_SOURCE.settings) {
-  if (typeof text !== 'string') return { recorded: false, at: '' }
+  if (typeof text !== 'string') return { recorded: false, at: '', n: 0 }
   const entries = readEntries(directory)
   const newest = entries[entries.length - 1]
-  if (newest !== undefined && newest.text === text) return { recorded: false, at: newest.at }
+  if (newest !== undefined && newest.text === text) return { recorded: false, at: newest.at, n: newest.n }
 
   const at = new Date().toISOString()
   const n = entries.length === 0 ? 1 : entries[entries.length - 1].n + 1

package/locales.mjs

@@ -97,6 +97,13 @@ export const zh = {
   'api.promptWriteFailed': '写入提示词失败：{detail}',
   'api.renderFailed': '生成组成文件失败：{detail}',
   'api.selfCheckFailed': '生成结果自检失败，已放弃写入：{detail}',
+  'api.internalError': '宿主内部错误：{detail}',
+  'api.bodyTooLarge': '请求体过大（上限 {max} 字节）',
+  'api.repairNotNeeded': '这个助手在本机没有无法解析的行，无需修复。',
+  'api.repaired': '已按本机这条 dsh 线关闭 {count} 个无法解析的行（{ids}）。现在这个模式能重新出现在选择器里。',
+  'api.badAssistantId': '助手标识不合法：{id}（只能用 a-z、0-9 与连字符）',
+  'api.seedFailed': '复制模式模板失败：{detail}',
+  'api.metaWriteFailed': '写入 preset.yml 失败：{detail}',
   'api.promptReadFailed': '读取提示词失败：{detail}',
   'api.noRemoveApi': '当前 DSH 版本没有 agentPresets.remove()，无法删除。',
   'api.deleteFailed': '删除失败：{detail}',
@@ -122,7 +129,7 @@ export const zh = {
   'msg.exported': '已导出为文件。',
   'msg.exportFailed': '导出失败',
   'delete.title': '永久删除这个助手？',
-  'delete.description': '这会删除磁盘上的模式目录，连同它的系统提示词一起消失，无法撤销。正在使用它的会话不受影响；新建会话时它不再出现在选择器里。',
+  'delete.description': '这会删除磁盘上的模式目录，连同它的系统提示词一起消失，无法撤销。正在使用它的会话不受影响；新建会话时它不再出现在选择器里。（{id}）',
   'delete.acknowledge': '我明白这个助手的提示词会被永久删除',
   'delete.confirm': '永久删除',
   'delete.close': '关闭',
@@ -161,6 +168,9 @@ export const zh = {
   'msg.readFailed': '读取失败',
   'msg.saveFailed': '保存失败',
   'msg.saved': '已保存。新建会话即生效，当前会话保持原配置。',
+  // 把「一句已翻译的话 + 一个底层细节」拼起来时，中间那个标点也必须**按语言**走 ——
+  // 硬编码全角冒号会让英文界面在出错那一刻出现中文标点（issue #7）。
+  'status.detail': '{message}：{detail}',
 
   // ── row labels ────────────────────────────────────────────────────────────
   'row.persona.label': '身份（系统提示词）',
@@ -313,6 +323,13 @@ export const en = {
   'api.promptWriteFailed': 'Writing the prompt failed: {detail}',
   'api.renderFailed': 'Rendering the composition failed: {detail}',
   'api.selfCheckFailed': 'The generated composition failed its self-check, so nothing was written: {detail}',
+  'api.internalError': 'Unexpected host error: {detail}',
+  'api.bodyTooLarge': 'Request body too large (limit {max} bytes)',
+  'api.repairNotNeeded': 'Nothing to repair — no row of this assistant is unresolvable on this machine.',
+  'api.repaired': 'Disabled {count} row(s) this dsh line cannot resolve ({ids}). The mode can appear in the pickers again.',
+  'api.badAssistantId': 'Invalid assistant id: {id} (a-z, 0-9 and hyphens only)',
+  'api.seedFailed': 'Copying the mode template failed: {detail}',
+  'api.metaWriteFailed': 'Writing preset.yml failed: {detail}',
   'api.promptReadFailed': 'Reading the prompt failed: {detail}',
   'api.noRemoveApi': 'This DSH build has no agentPresets.remove(), so deleting is unavailable.',
   'api.deleteFailed': 'Delete failed: {detail}',
@@ -338,7 +355,7 @@ export const en = {
   'msg.exported': 'Exported to a file.',
   'msg.exportFailed': 'Export failed',
   'delete.title': 'Delete this assistant permanently?',
-  'delete.description': 'This removes the mode directory from disk, system prompt included, and cannot be undone. Sessions already using it keep running; new sessions no longer offer it.',
+  'delete.description': 'This removes the mode directory from disk, system prompt included, and cannot be undone. Sessions already using it keep running; new sessions no longer offer it. ({id})',
   'delete.acknowledge': 'I understand this assistant\'s prompt will be deleted permanently',
   'delete.confirm': 'Delete permanently',
   'delete.close': 'Close',
@@ -380,6 +397,7 @@ export const en = {
   'msg.readFailed': 'Load failed',
   'msg.saveFailed': 'Save failed',
   'msg.saved': 'Saved. A new session picks it up; the current one keeps its configuration.',
+  'status.detail': '{message}: {detail}',
 
   // ── row labels ────────────────────────────────────────────────────────────
   'row.persona.label': 'Identity (system prompt)',

package/meta.mjs

@@ -86,7 +86,12 @@ export function readPresetMeta(directory = PRESET_DIR) {
 /** Quote and flatten a value into a single-line YAML scalar. */
 function yamlScalar(value) {
   const flat = String(value).replace(/\r?\n/g, ' ').trim()
-  return '"' + flat.replace(/\\/g, '\\\\').replace(/"/g, '\\"') + '"'
+  // `JSON.stringify` 恰好就是 YAML 双引号标量需要的转义：除了 `"` 与 `\`，它还会把控制字符写成
+  // `\u0007` 这类转义序列。此前手工只转前两个，于是名字里一个 BEL / VT / NUL 就会写出平台
+  // js-yaml 解析不了的 preset.yml —— 这个模式在**所有选择器**里退化成裸目录 id，description 与
+  // order 一起丢，而本插件自己的页面还显示着那个名字（外部评审实测）。
+  // 对普通名字（含中文）输出与旧实现逐字节相同，所以这次替换不会动任何既有文件。
+  return JSON.stringify(flat)
 }
 
 /**
@@ -104,7 +109,7 @@ function yamlScalar(value) {
  */
 export function writePresetMeta(name, description, directory = PRESET_DIR, options = {}) {
   const cleanName = typeof name === 'string' ? name.replace(/\r?\n/g, ' ').trim() : ''
-  if (cleanName === '') return { ok: false, error: '模式名称不能为空。' }
+  if (cleanName === '') return { ok: false, code: 'nameRequired', error: '模式名称不能为空。' }
   const lines = ['name: ' + yamlScalar(cleanName)]
   if (typeof description === 'string' && description.trim() !== '') {
     lines.push('description: ' + yamlScalar(description))
@@ -118,7 +123,11 @@ export function writePresetMeta(name, description, directory = PRESET_DIR, optio
     // 实测审阅指出这里原先用的是裸 writeFileSync，与设置页的纪律不一致；现在共用同一份实现。
     writeAtomic(presetMetaPath(directory), lines.join('\n') + '\n')
   } catch (error) {
-    return { ok: false, error: '写入 preset.yml 失败：' + String((error && error.message) || error) }
+    // 带 `code`：这条是保存路径上最后一个没有 code 的用户可见失败，英文界面此前会在这里显示中文。
+    // 它发生得尤其难受 —— 组成文件与提示词**已经写成功**了，只有 preset.yml 没写成，所以页面
+    // 说"失败"而磁盘上是新提示词 + 旧名字（外部评审复现）。
+    const detail = String((error && error.message) || error)
+    return { ok: false, code: 'metaWriteFailed', params: { detail }, error: '写入 preset.yml 失败：' + detail }
   }
   return { ok: true, name: cleanName }
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "dsh-custom-mode",
-  "version": "1.9.6",
-  "description": "Custom modes and custom prompts for DeepSeek Harness (dsh): edit a mode's system prompt on the settings page (it takes effect on the next model step), choose its base mode, switch plugins row by row, and keep several assistants side by side.",
+  "version": "1.9.8",
+  "description": "Custom modes and custom prompts for DeepSeek Harness (dsh): edit a mode's system prompt on the settings page (it takes effect on the next model step), pick its base mode, switch plugin rows one by one, and keep several assistants side by side — coding, chat or roleplay (RP) personas (multi-mode / multi-agent).",
   "license": "MIT",
   "type": "module",
   "main": "./index.mjs",
@@ -33,7 +33,10 @@
     "custom-prompt",
     "prompt-editor",
     "prompt-engineering",
-    "persona"
+    "persona",
+    "multi-agent",
+    "roleplay",
+    "chat"
   ],
   "dsh": {
     "bundle": {

package/preset/prompt-tool.mjs

@@ -27,7 +27,7 @@
 
 import { dirname, join } from 'node:path'
 import { randomBytes } from 'node:crypto'
-import { mkdirSync, readFileSync, renameSync, writeFileSync } from 'node:fs'
+import { mkdirSync, readFileSync, renameSync, rmSync, writeFileSync } from 'node:fs'
 import { fileURLToPath } from 'node:url'
 
 /** Absolute path of the prompt file this preset reads. */
@@ -216,6 +216,17 @@ function makeDefinition(modeName) {
         }
       }
 
+      // **显式白名单，不再靠"落到最后就当写入"。** 这段代码到这儿只剩整体替换，此前它对任何
+      // 不认识的 action（`replace` / `Write` / `overwrite` / 空串 / `append ` 末尾多一个空格）
+      // 都静默照做，而审批闸门只认识 write/append —— 一个词的拼写差异就能把用户的提示词整体
+      // 换掉且不弹审批（外部评审实测）。不认识的动词现在一律拒绝并说明，模型据此改对再调。
+      if (action !== 'write') {
+        return (
+          '写入被拒绝：未知的 action「' + action + '」。只接受 read / write / append' +
+          '（未指定 action 按 read 处理）；append 必须精确拼写，末尾多一个空格不算。'
+        )
+      }
+
       if (typeof args.text !== 'string' || args.text.trim() === '') {
         return '写入被拒绝：action 为 "write" 时必须提供非空的 text。'
       }
@@ -263,9 +274,17 @@ function registerApprovalGate(ctx) {
     () => ctx.on('tools/pre-execute', (exec, next) => {
       if (exec === null || typeof exec !== 'object' || exec.name !== TOOL_NAME) return next()
       const args = exec.arguments
-      const action = args !== null && typeof args === 'object' ? args.action : undefined
-      // 拦所有会改文件的动作：write 与 append 都在改系统提示词；未指定 action 时按 read 处理，不拦。
-      if (action !== 'write' && action !== 'append') return next()
+      // **只放行真正的读取。** 未指定 action 与 execute 一样按 read 处理；其余一律发问。
+      //
+      // 此前这里只拦 `write` / `append` 两个字面量，而 execute 把"既不是 read 也不是 append"的
+      // **任何** action 都当成整体替换 —— 于是 `replace` / `Write` / 空串 / 末尾多一个空格都能
+      // 绕过审批面板直接改掉用户的提示词（外部评审实测复现）。闸门要盖住"一切可能改文件的调用"，
+      // 不能只认识它恰好想到的那两个词。execute 那边同时加了显式白名单，两层各自独立。
+      const raw = args !== null && typeof args === 'object' ? args.action : undefined
+      // 只有"没提供 action"才等同 read（execute 也是这样兜的）；**提供了但不认识**（含空串）
+      // 一律发问 —— 闸门的判据是"这次调用会不会改文件"，不是"这个动词我认不认识"。
+      const action = typeof raw === 'string' ? raw : 'read'
+      if (action === 'read') return next()
       const verb = action === 'append' ? '追加到' : '整体替换为'
       const text = typeof args.text === 'string' ? args.text : ''
       const firstLine = text.split('\n').find((line) => line.trim() !== '') ?? ''