dsh-custom-mode 1.9.14 → 1.9.15

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

@@ -733,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 = (() => {
@@ -745,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 + '」。正在使用它的会话不受影响；新建会话时不再出现。' }
 }
@@ -912,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()
@@ -1134,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.14",
+  "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()],