dsh-custom-mode 1.9.7 → 1.9.9

package/client.js

@@ -170,7 +170,7 @@ try {
         "msg.exported": "已导出为文件。",
         "msg.exportFailed": "导出失败",
         "delete.title": "永久删除这个助手？",
-        "delete.description": "这会删除磁盘上的模式目录，连同它的系统提示词一起消失，无法撤销。正在使用它的会话不受影响；新建会话时它不再出现在选择器里。",
+        "delete.description": "这会删除磁盘上的模式目录，连同它的系统提示词一起消失，无法撤销。正在使用它的会话不受影响；新建会话时它不再出现在选择器里。（{id}）",
         "delete.acknowledge": "我明白这个助手的提示词会被永久删除",
         "delete.confirm": "永久删除",
         "delete.close": "关闭",
@@ -200,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 工具",
@@ -377,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",
@@ -407,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",
@@ -681,6 +683,13 @@ try {
         ".cpfe-btn-ghost{border-color:transparent;background:transparent}",
         ".cpfe-input{box-sizing:border-box;width:100%;height:34px;padding:0 12px;border-radius:8px;border:.5px solid var(--dsw-alias-border-l2);background:var(--dsw-alias-bg-base);color:var(--dsw-alias-label-primary);font:inherit;font-size:13px}",
         ".cpfe-switch{display:inline-flex;align-items:center;gap:8px;cursor:pointer;font-size:13px;line-height:19px;color:var(--dsw-alias-label-primary)}",
+        // 忙碌期间禁掉草稿类控件的交互（issue #6）。
+        // 状态层已经拦住了改动（`update()` 在 busy 时直接返回），但对**受控组件**来说，
+        // 光拦状态还不够：DOM 会先显示用户敲进去的字/翻过去的开关，等下一次重渲染才被拉回来 ——
+        // 那就是"我明明改了，它自己弹回去了"。`pointer-events` 让点击根本到不了控件，
+        // 视觉上也给一个明确的"此刻不能动"。
+        ".cpfe-busy .cpfe-switch,.cpfe-busy .cpfe-row-toggle,.cpfe-busy .cpfe-input,.cpfe-busy .cpfe-field,.cpfe-busy .cpfe-editor{pointer-events:none;opacity:.65}",
+        ".cpfe-busy .cpfe-editor{cursor:progress}",
         ".cpfe-tag{font-size:11px;line-height:16px;padding:0 5px;border-radius:4px;border:.5px solid var(--dsw-alias-border-l1);color:var(--dsw-alias-label-secondary)}",
         ".cpfe-tag-success,.cpfe-tag-info{color:var(--dsw-alias-state-success-primary)}",
         ".cpfe-tag-warning{color:var(--dsw-alias-state-warn-primary)}",
@@ -1038,6 +1047,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)
@@ -1183,7 +1205,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)
@@ -1195,7 +1217,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
@@ -1215,7 +1250,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)
           }
@@ -1247,7 +1282,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)
           }
@@ -1275,7 +1310,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)
           }
@@ -1308,7 +1343,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) }))
           }
         }
 
@@ -1327,7 +1362,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) }))
           }
         }
 
@@ -1389,7 +1424,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)
           }
@@ -1452,7 +1487,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)
           }
@@ -1569,6 +1604,9 @@ try {
                   className: "cpfe-field cpfe-desc",
                   value: draft.description,
                   rows: 2,
+                  // 忙碌期间只读（issue #6）。**描述框在 DOM 里排在提示词编辑器前面**，
+                  // 所以只给 `.cpfe-editor` 加 readOnly 是不够的 —— 实测就是这么漏掉的。
+                  readOnly: busy === true,
                   placeholder: t("name.descriptionPlaceholder"),
                   "aria-label": t("name.descriptionPlaceholder"),
                   onChange: (event) => update({ description: event.target.value }),
@@ -1757,6 +1795,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 }),
                 }),
@@ -1802,7 +1845,9 @@ try {
 
         return react.createElement(
           "div",
-          { className: "cpfe" },
+          // `cpfe-busy` 是 issue #6 的兜底：面板里**所有**会改草稿的控件在忙碌期间一律不可交互。
+          // 逐个控件加 `disabled` 会漏（本轮就漏了行开关与描述框），一个根类 + 一条 CSS 反而漏不掉。
+          { className: busy === true ? "cpfe cpfe-busy" : "cpfe" },
           assistantList,
           react.createElement("p", { className: "cpfe-note" }, t("assistant.switchHint")),
           // 配了却不生效的项：主动点名，而不是让用户对着"我明明写了"发呆。
@@ -1878,7 +1923,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

@@ -281,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,
@@ -507,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 = {
@@ -600,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 + '）。现在这个模式能重新出现在选择器里。',
   }
 }

package/locales.mjs

@@ -129,7 +129,7 @@ export const zh = {
   'msg.exported': '已导出为文件。',
   'msg.exportFailed': '导出失败',
   'delete.title': '永久删除这个助手？',
-  'delete.description': '这会删除磁盘上的模式目录，连同它的系统提示词一起消失，无法撤销。正在使用它的会话不受影响；新建会话时它不再出现在选择器里。',
+  'delete.description': '这会删除磁盘上的模式目录，连同它的系统提示词一起消失，无法撤销。正在使用它的会话不受影响；新建会话时它不再出现在选择器里。（{id}）',
   'delete.acknowledge': '我明白这个助手的提示词会被永久删除',
   'delete.confirm': '永久删除',
   'delete.close': '关闭',
@@ -168,6 +168,9 @@ export const zh = {
   'msg.readFailed': '读取失败',
   'msg.saveFailed': '保存失败',
   'msg.saved': '已保存。新建会话即生效，当前会话保持原配置。',
+  // 把「一句已翻译的话 + 一个底层细节」拼起来时，中间那个标点也必须**按语言**走 ——
+  // 硬编码全角冒号会让英文界面在出错那一刻出现中文标点（issue #7）。
+  'status.detail': '{message}：{detail}',
 
   // ── row labels ────────────────────────────────────────────────────────────
   'row.persona.label': '身份（系统提示词）',
@@ -352,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',
@@ -394,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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-custom-mode",
-  "version": "1.9.7",
+  "version": "1.9.9",
   "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",