dsh-custom-mode 1.9.13 → 1.9.15

package/assistants.mjs

@@ -25,7 +25,7 @@
  * file work, so it can be tested without a running harness.
  */
 
-import { existsSync, mkdirSync, readFileSync, readdirSync, writeFileSync } from 'node:fs'
+import { existsSync, mkdirSync, readFileSync, readdirSync, rmSync, writeFileSync } from 'node:fs'
 import { dirname, join } from 'node:path'
 import { dshHome, PRESET_DIR } from './paths.mjs'
 // `presetMetaPath` 曾被漏掉：reorderAssistant 用它做"快照 → 失败回滚"，于是两处调用都抛
@@ -267,14 +267,31 @@ export function createAssistantDir({ root, id, composition, templateDir }) {
   const dir = join(root, id)
   if (existsSync(dir)) return { ok: false, code: 'dirExists', params: { path: dir }, error: `目录已存在：${dir}` }
 
+  // ★ 失败必须**撤干净**（issue #4）。上面的 existsSync 已经证明这个目录是本函数刚创建的，所以
+  //   失败时把它整体删掉是安全的。不这么做的话：用户看到"没建成"，磁盘上却留下一个半成品目录，
+  //   重启后它作为助手出现在选择器里，而且这个名字被**永久占用**（再建同名直接报"目录已存在"）。
+  //   删除本身也做失败兜底：删不掉时如实说明"留下了什么"，而不是假装干净。
+  const rollback = (result) => {
+    try {
+      rmSync(dir, { recursive: true, force: true })
+      return result
+    } catch (error) {
+      return {
+        ...result,
+        params: { ...result.params, leftover: dir, cleanupFailed: describe(error) },
+        error: result.error + '；且未能清理半成品目录 ' + dir + '（' + describe(error) + '）',
+      }
+    }
+  }
+
   const seeded = seedPreset(dir, templateDir)
   if (seeded.errors.length > 0) {
-    return { ok: false, code: 'seedFailed', params: { detail: seeded.errors.join('；') }, error: '复制模式模板失败：' + seeded.errors.join('；') }
+    return rollback({ 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, code: 'writeFailed', params: { detail: describe(error) }, error: '写入组成文件失败：' + describe(error) }
+    return rollback({ ok: false, code: 'writeFailed', params: { detail: describe(error) }, error: '写入组成文件失败：' + describe(error) })
   }
   return { ok: true, id, dir }
 }

package/client.js

@@ -148,6 +148,7 @@ try {
         "api.metaWriteFailed": "写入 preset.yml 失败：{detail}",
         "api.promptReadFailed": "读取提示词失败：{detail}",
         "api.noRemoveApi": "当前 DSH 版本没有 agentPresets.remove()，无法删除。",
+        "api.noDirectory": "删除失败：本机找不到这个助手的目录，页面上的列表可能是旧的。刷新一次再试。",
         "api.deleteFailed": "删除失败：{detail}",
         "api.versionMissing": "找不到这个版本（历史可能已被上限裁剪）。",
         "api.badJson": "请求体不是合法 JSON",
@@ -361,6 +362,7 @@ try {
         "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.noDirectory": "Delete failed: this assistant's directory is gone from disk, so the list on screen is stale. Refresh and try again.",
         "api.deleteFailed": "Delete failed: {detail}",
         "api.versionMissing": "That version is gone (the history is capped).",
         "api.badJson": "The request body is not valid JSON",

package/index.mjs

@@ -402,10 +402,12 @@ function serializedWrite(key, work) {
  * 的行集合也就不变，这是所有选项里破坏性最小的一个。
  *
  * @param {string} directory - the assistant's preset directory.
- * @param {{id: string, mode: string, prompt: string, name: string, description: string}} input - the validated save.
+ * @param {{id: string, mode: string, prompt: string, name: string, description: string, displayName?: string}} input
+ *   - the validated save; `name` decides whether `preset.yml` is written, `displayName` is what the response
+ *   reports (the disk name when the request omitted one — issue #9).
  * @returns {object} the shape `saveState` returns, with `code: 'savedPromptOnly'`.
  */
-export function savePromptOnly(directory, { id, mode, prompt, name, description }) {
+export function savePromptOnly(directory, { id, mode, prompt, name, description, displayName }) {
   // 与正常保存同一条纪律：提示词与 preset.yml 一起换名，失败则两个都不动。
   let metaText = null
   if (name !== '') {
@@ -431,7 +433,8 @@ export function savePromptOnly(directory, { id, mode, prompt, name, description
     id,
     mode,
     code: 'savedPromptOnly',
-    params: { name: name === '' ? id : name, mode },
+    // 响应里的名字用**磁盘上的那个**（issue #9）：请求省略 name 时不该把 params.name 报成裸目录 id。
+    params: { name: (displayName === undefined || displayName === '' ? name : displayName) === '' ? id : (displayName === undefined || displayName === '' ? name : displayName), mode },
     note: '已保存系统提示词（本机取不到基础模式的出厂组成，插件开关与基础模式未改动）。新建会话即生效。',
   }
 }
@@ -462,6 +465,12 @@ export function saveState(rows, input) {
   }
   // 请求里**没带** description 时保留原值：API 调用方只改提示词，不该顺手把描述清空（外部评审实测）。
   const meta = readPresetMeta(directory)
+  // ★ 请求里没带 name（或页面清空）时，渲染组成仍要用**磁盘上那个名字**（issue #9）。
+  //   组成里 custom-prompt-tool 行的 config.modeName 是"会话内工具知道自己在改哪个助手"的唯一凭据；
+  //   用空串渲染会把它整块删掉，而 preset.yml 又（有意）保留旧名字 —— 两处就此不一致，
+  //   同时成功响应还会把 params.name 报成裸目录 id。请求**没带** name 与"用户把名字清空"在这里
+  //   是同一个语义：保持现状，而不是把它抹掉。
+  const effectiveName = name !== '' ? name : typeof meta.name === 'string' ? meta.name : ''
   const rawDescription =
     input !== null && typeof input === 'object' && typeof input.description === 'string'
       ? input.description
@@ -481,14 +490,14 @@ export function saveState(rows, input) {
 
   let composition
   try {
-    composition = renderComposition(mode, overrides, { modeName: name, assistantId: id })
+    composition = renderComposition(mode, overrides, { modeName: effectiveName, assistantId: id })
   } catch (error) {
     // 拿不到出厂组成：这一页**降级**但不能全废 —— 提示词仍可单独保存（组成文件原样不动）。
     // 页面在降级态下把基础模式与开关都设为不可编辑，所以开关集合必为空；HTTP API 直接调用
     // 还想改开关时明确拒绝，而不是悄悄丢掉用户的意图。
     if (isBaseCompositionUnavailable(error)) {
       if (overrides.size === 0 && existsSync(compositionFile(directory))) {
-        return savePromptOnly(directory, { id, mode, prompt, name, description })
+        return savePromptOnly(directory, { id, mode, prompt, name, description, displayName: effectiveName })
       }
       return {
         ok: false,
@@ -547,7 +556,7 @@ export function saveState(rows, input) {
     id,
     mode,
     code: 'saved',
-    params: { name: name === '' ? id : name, mode },
+    params: { name: effectiveName === '' ? id : effectiveName, mode },
     note: '已保存（' + (name === '' ? id : name) + '，基础模式 ' + mode + '）。新建会话即生效，当前会话保持原配置。',
   }
 }
@@ -724,11 +733,14 @@ export function repairComposition(rows, input) {
   }
 }
 
-export async function deleteAssistant(rows, input, agentPresets) {
+export async function deleteAssistant(rows, input, remover) {
   const id = input !== null && typeof input === 'object' && typeof input.id === 'string' ? input.id : ''
-  if (assistantDir(rows, id) === undefined) return unknownAssistant(id)
-  if (typeof agentPresets?.remove !== 'function') {
-    return { ok: false, code: 'noRemoveApi', error: '当前 DSH 版本没有 agentPresets.remove()，无法删除。' }
+  const directory = assistantDir(rows, id)
+  if (directory === undefined) return unknownAssistant(id)
+  // `remover` 是**按后端选出来的**删除入口（见 {@link presetRemover}）：旧线是平台的
+  // `agentPresets.remove()`，新线是声明式后端的 dispose + 删目录。两边都没有时才是真的做不到。
+  if (remover === undefined || remover === null || typeof remover.remove !== 'function') {
+    return { ok: false, code: 'noRemoveApi', error: '当前 DSH 版本没有可用的删除入口（agentPresets.remove() 与声明式后端都不可用），无法删除。' }
   }
   // 文案用**显示名**，不用内部目录 id（复制/删除的状态行曾把 id 暴露给用户，审阅点名）。
   const displayName = (() => {
@@ -736,9 +748,17 @@ export async function deleteAssistant(rows, input, agentPresets) {
     return typeof row?.name === 'string' && row.name.trim() !== '' ? row.name : id
   })()
   try {
-    await agentPresets.remove(id)
+    const outcome = await remover.remove(id, directory)
+    if (outcome !== null && typeof outcome === 'object' && outcome.ok === false) {
+      // 字面量 code（不用三元表达式）：test/editor-route.test.mjs 会扫 `code:` 后面的字符串字面量，
+      // 要求每个都在词典里中英各一条 —— 三元里的 'string' 会被它当成一个 code。
+      if (outcome.code === 'noDirectory') {
+        return { ok: false, code: 'noDirectory', params: { path: directory }, error: '删除失败：找不到「' + displayName + '」的目录。' }
+      }
+      return { ok: false, code: 'deleteFailed', params: { detail: describe(outcome.code ?? 'unknown'), path: directory }, error: '删除失败：' + describe(outcome.code ?? 'unknown') }
+    }
   } catch (error) {
-    return { ok: false, code: 'deleteFailed', params: { detail: describe(error) }, error: '删除失败：' + describe(error) }
+    return { ok: false, code: 'deleteFailed', params: { detail: describe(error), path: directory }, error: '删除失败：' + describe(error) }
   }
   return { ok: true, id, code: 'deleted', params: { name: displayName }, note: '已删除「' + displayName + '」。正在使用它的会话不受影响；新建会话时不再出现。' }
 }
@@ -903,6 +923,18 @@ export function apply(ctx) {
         ? createDeclarativeBackend({ scope, log: console.log, warn: console.error })
         : null
 
+    /**
+     * 删除入口，按后端选。新线上服务**没有** `remove()`（唯一手段是 `register()` 的 disposer），
+     * 所以由声明式后端自己 dispose + 删目录；旧线继续用平台的文件系统语义。
+     *
+     * 这里返回 `undefined` 只发生在"两套都没有"的线上，那时才允许报 `noRemoveApi`。
+     */
+    const presetRemover = () => {
+      if (declarative !== null) return { remove: (id, dir) => declarative.remove(id, dir) }
+      if (typeof scope.agentPresets?.remove === 'function') return { remove: (id) => scope.agentPresets.remove(id) }
+      return undefined
+    }
+
     /** 原始 roster：只读、无副作用。同步逻辑走它，避免与 {@link roster} 互相递归。 */
     const rawRoster = async () => {
       const rows = await scope.agentPresets.list()
@@ -1125,7 +1157,7 @@ export function apply(ctx) {
           const result = await serializedWrite('tree', async () => reorderAssistant(await roster(), parsed))
           return json(result, result.ok === true ? 200 : 400)
         }
-        const result = await serializedWrite('delete:' + targetId, async () => deleteAssistant(await roster(), parsed, scope.agentPresets))
+        const result = await serializedWrite('delete:' + targetId, async () => deleteAssistant(await roster(), parsed, presetRemover()))
         return json(result, result.ok === true ? 200 : 400)
       } catch (error) {
         // 兜底也要带 code：这是**意料之外**的异常，页面拿到的是 Node 的原始错误串，方向反着也一样糟

package/locales.mjs

@@ -108,6 +108,7 @@ export const zh = {
   'api.metaWriteFailed': '写入 preset.yml 失败：{detail}',
   'api.promptReadFailed': '读取提示词失败：{detail}',
   'api.noRemoveApi': '当前 DSH 版本没有 agentPresets.remove()，无法删除。',
+  'api.noDirectory': '删除失败：本机找不到这个助手的目录，页面上的列表可能是旧的。刷新一次再试。',
   'api.deleteFailed': '删除失败：{detail}',
   'api.versionMissing': '找不到这个版本（历史可能已被上限裁剪）。',
   'api.badJson': '请求体不是合法 JSON',
@@ -339,6 +340,7 @@ export const en = {
   '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.noDirectory': "Delete failed: this assistant's directory is gone from disk, so the list on screen is stale. Refresh and try again.",
   'api.deleteFailed': 'Delete failed: {detail}',
   'api.versionMissing': 'That version is gone (the history is capped).',
   'api.badJson': 'The request body is not valid JSON',

package/package.json

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

package/preset-backend/declarative.mjs

@@ -12,7 +12,8 @@
  *     相对路径；旧线上它相对 preset 目录解析，而 `register()` 在 profile 的 baseUrl 下挂载。
  *     实测：相对路径 → `broken: … never started`；绝对 `file://` URL → 正常 mount。
  *     ⇒ `toPluginRows()` 必须把相对说明符改写成绝对 URL。
- *  2. **`register()` 的返回值是 disposer**，注销唯一手段；服务上没有 `remove()`。
+ *  2. **`register()` 的返回值是 disposer**，注销唯一手段；服务上没有 `remove()` —— 所以"删除助手"
+ *     由本后端的 {@link remove} 自己做（dispose + 删目录），宿主那半不再要求平台提供 `remove()`。
  *  3. **同一个 id 重复注册会抛 `Duplicate agent preset`**
  *     ⇒ 重新挂载前必须先 dispose 旧的。
  *
@@ -22,7 +23,7 @@
  * `agent.cordis.yml` 仍然写盘 —— 它是用户可见、可迁移、可手改的真相，也是旧线唯一的输入。
  */
 
-import { existsSync, readFileSync } from 'node:fs'
+import { existsSync, readFileSync, rmSync } from 'node:fs'
 import { join } from 'node:path'
 import { pathToFileURL } from 'node:url'
 import { parseComposition } from './parse-composition.mjs'
@@ -163,6 +164,29 @@ export function createDeclarativeBackend({ scope, log = console.log, warn = cons
     for (const id of [...mounted.keys()]) await unmount(id)
   }
 
+  /**
+   * 删除一个助手：**先注销，再删目录**。
+   *
+   * 这条线上服务没有 `remove()`（唯一手段是 `register()` 返回的 disposer），所以"删除"必须由本插件
+   * 自己完成 —— 而这正是设置页在 0.1.7 上**根本删不掉助手**的原因：宿主那半只认 `agentPresets.remove()`，
+   * 拿不到就回一个类型化的 `noRemoveApi`，页面弹完确认框只显示"删除失败"。
+   *
+   * 顺序不能反：先注销，选择器立刻不再提供它；再删目录。删目录失败时**如实抛出**（调用方转成
+   * `deleteFailed` 并把路径带给用户），而不是报告"已删除"却把目录留在磁盘上 —— 那样下次同步
+   * 会把它重新挂回来，用户看到的是"删了又回来了"。
+   *
+   * @param {string} id - assistant id.
+   * @param {string} [dir] - its directory; when omitted, the one remembered at mount time is used.
+   */
+  async function remove(id, dir) {
+    const remembered = mounted.get(id)?.dir
+    await unmount(id)
+    const target = typeof dir === 'string' && dir !== '' ? dir : remembered
+    if (typeof target !== 'string' || target === '') return { ok: false, code: 'noDirectory', id }
+    rmSync(target, { recursive: true, force: true })
+    return { ok: true, id, dir: target }
+  }
+
   /**
    * Fetch the shipped base compositions the host declares, as entry-list YAML.
    *
@@ -205,6 +229,7 @@ export function createDeclarativeBackend({ scope, log = console.log, warn = cons
     sync,
     mountOne,
     unmount,
+    remove,
     disposeAll,
     fetchBaseCompositions,
     mountedIds: () => [...mounted.keys()],