dsh-custom-mode 1.0.0 → 1.0.2

package/README.md

@@ -1,41 +1,43 @@
 # dsh-custom-mode
 
-The **settings-page half** of [dsh-custom-mode](https://github.com/BOWLUNA/dsh-custom-mode): a Web UI
-for a [DeepSeek Harness](https://github.com/deepseek-ai/deepseek-harness) (dsh) agent preset whose
-**system prompt is a plain file you can edit, taking effect on the next model step** — no restart, no
-new session.
+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
+composition, toggle the plugin rows it mounts, and edit its **system prompt**, which is a plain file the
+agent loop re-reads before every model step — **a save applies on the next step, with no restart and no new
+session**. Several modes ("assistants") live side by side, each with its own prompt.
 
-![Settings → Custom mode](https://raw.githubusercontent.com/BOWLUNA/dsh-custom-mode/main/docs/images/01-mode-switch.png)
+Searched for as: custom mode · custom prompt · system-prompt editor · multi-mode / several assistants ·
+自定义模式 · 自定义提示词 · 多助手／多模式.
 
-## What is in this package, and what is not
+![Assistant manager](https://raw.githubusercontent.com/BOWLUNA/dsh-custom-mode/main/docs/images/05-assistant-manager.png)
 
-This package is **only the editor**: one private HTTP route plus the browser half that renders the
-settings section (base-mode picker, per-row plugin switches, prompt editor, mode rename).
+![Mode name and base mode](https://raw.githubusercontent.com/BOWLUNA/dsh-custom-mode/main/docs/images/01-mode-switch.png)
 
-The **agent preset itself is not an npm package** — it is a directory of files
-(`preset/agent.cordis.yml`, `prompt-reader.mjs`, `prompt-tool.mjs`, `prompt.md`) that dsh discovers
-under `$DSH_HOME/.agent-presets/<id>/`. It comes from the GitHub repository.
+![Plugin switches](https://raw.githubusercontent.com/BOWLUNA/dsh-custom-mode/main/docs/images/02-plugin-switches.png)
 
-So:
+## Install
 
-- **First-time install** → clone the repository and run `./install.sh`. It copies the preset *and*
-  installs this package into your profile.
-- **Already have the preset, want to update the editor** → install this package directly:
+```sh
+dsh plugin --profile web add dsh-custom-mode
+```
 
-  ```sh
-  dsh plugin --profile web add dsh-custom-mode
-  ```
+**That one command is the whole install.** The package carries the agent preset, and the host half writes it
+into `$DSH_HOME/.agent-presets/` on first activation — filling in only what is missing, so a `prompt.md` you
+wrote yourself is never overwritten.
 
-- The mode is **web-profile only**: the `agent-presets` service, which mounts presets at all, ships
-  with dsh's `web` profile. In `tui` / `headless` this package activates but registers nothing.
+To keep the source around as well, clone the repository and run `./install.sh` instead; it does the same two
+things explicitly, and `./uninstall.sh` undoes them (keeping your prompt unless you pass `--purge`).
+
+The mode is **web-profile only**: `agent-presets`, the service that mounts presets at all, ships with dsh's
+`web` profile. In `tui` / `headless` this package activates but registers nothing.
 
 ## Requirements
 
-- `@deepseek-ai/dsh` `>=0.1.2-alpha.1` (declared as an *optional* peer: this package never imports it,
-  it only reads the host services dsh injects)
-- The version is this package's own line (`1.0.0`, `1.0.1`, …). Which DSH it supports is declared in
-  `engines.dsh` and the peer range above; see the repository README's "Versioning" section.
-- Source is published as-is: **no build step, no dependencies** beyond Node's standard library.
+- **dsh `>=0.1.6-alpha.1`** — declared in `engines.dsh` and as an *optional* peer range. This package never
+  imports `@deepseek-ai/dsh`; it only reads the host services dsh injects.
+- **No build step, no dependencies**: the source is published as-is and uses only Node's standard library.
+- The package version is its own line (`1.0.0`, `1.0.1`, …). Which dsh it supports is declared in
+  `engines.dsh`; see the repository README's "Versioning" section.
 
 ## Documentation
 
@@ -45,18 +47,18 @@ The full documentation is bilingual in the repository:
   [中文说明](https://github.com/BOWLUNA/dsh-custom-mode/blob/main/README.zh.md)
 - [Troubleshooting](https://github.com/BOWLUNA/dsh-custom-mode/blob/main/docs/TROUBLESHOOTING.md) —
   every failure that was actually reproduced, with symptoms, cause and a way out
-- [Measured behaviour](https://github.com/BOWLUNA/dsh-custom-mode/blob/main/docs/%E5%AE%9E%E6%B5%8B%E8%AE%B0%E5%BD%95.md) —
-  the commands and raw output behind every claim
-- [Architecture](https://github.com/BOWLUNA/dsh-custom-mode/blob/main/docs/ARCHITECTURE.md) — why
-  this had to be two artifacts, and how the host APIs it depends on can break
+- [Measurements](https://github.com/BOWLUNA/dsh-custom-mode/blob/main/docs/MEASUREMENTS.md) — the commands
+  and raw output behind every claim
+- [Architecture](https://github.com/BOWLUNA/dsh-custom-mode/blob/main/docs/ARCHITECTURE.md) — why this had
+  to be two artifacts, and which host APIs it depends on
 
 ## Security note
 
 The route this package serves runs the platform's own browser-trust check
 (`ctx.connection.requestRejection`) before anything else and **fails closed** when that service is
 unavailable. Before that check existed, the route could be read and written unauthenticated — see
-[SECURITY.md](https://github.com/BOWLUNA/dsh-custom-mode/blob/main/SECURITY.md) for the affected
-range and the mitigation.
+[SECURITY.md](https://github.com/BOWLUNA/dsh-custom-mode/blob/main/SECURITY.md) for the affected range and
+the mitigation.
 
 ## License
 

package/client.js

@@ -103,11 +103,11 @@ try {
         "name.placeholder": "自定义模式",
         "name.descriptionPlaceholder": "模式描述（显示在模式选择器里，可留空）",
         "mode.heading": "基础模式",
-        "mode.hint": "选一个官方模式作为底子，下面再按行微调。改完保存后，新建会话即生效，不需要重启。",
+        "mode.hint": "选一个官方模式作为底子，下面再按行微调。注意：底子只决定**行集合**与工具能力 —— 本模式的 persona 行始终替换掉底子那一行（提示词由你编辑，complete: false），底子的提示词语义不会被继承。改完保存后，新建会话即生效，不需要重启。",
         "rows.heading": "插件开关",
         "rows.hint": "逐行控制这个模式挂载哪些插件，和官方插件列表一样按行铺开。没拨过的行保持官方默认（含平台判断）；你手动拨了就以你的为准。",
         "prompt.heading": "系统提示词",
-        "prompt.hint": "This text is re-read before every model call, so a save applies on the next step and only affects sessions on this assistant. \"Import\" reads a file into the editor (nothing is written until you save); \"Export\" saves the current text as a .md file.",
+        "prompt.hint": "这段文本在每个模型调用前重新读取，所以保存后下一步即生效，且只影响使用这个助手的会话。「导入」把文件读进编辑器（未保存前不写入任何东西）；「导出」把当前文本存成 .md 文件。",
         "status.enabled": "已启用",
         "status.disabled": "已停用",
         "status.changed": "已改",
@@ -181,7 +181,7 @@ try {
         "base.ptc.label": "PTC 模式",
         "base.ptc.note": "在标准模式基础上启用 PTC 工具呈现（tool-presentation）",
         "base.minimal.label": "极简模式",
-        "base.minimal.note": "只有 Shell 与终端，共 7 行；没有文件、检索、技能、子代理",
+        "base.minimal.note": "只有 Shell 与终端，共 7 行；没有文件、检索、技能、子代理。系统提示词不是极简那一套（persona 行被本模式替换）",
         "base.cordis.label": "Cordis 模式",
         "base.cordis.note": "标准模式 + 读写运行时的 Cordis 工具集，可让 agent 自己改 harness"
       }
@@ -233,11 +233,11 @@ try {
         "name.placeholder": "Custom mode",
         "name.descriptionPlaceholder": "Mode description (shown in the mode picker, optional)",
         "mode.heading": "Base mode",
-        "mode.hint": "Choose an official mode as the base, then fine-tune individual rows below. After saving, a new session picks it up — no restart needed.",
+        "mode.hint": "Pick an official mode as the base, then adjust it row by row. Note: the base only decides which rows exist — this mode always replaces the base's persona row with its own reader (complete: false), so the base's prompt semantics are not inherited. A save applies to the next new session; no restart needed.",
         "rows.heading": "Plugin switches",
         "rows.hint": "Control row by row which plugins this mode mounts, laid out like the official plugin list. Untouched rows keep the official default (platform conditions included); your manual choice wins.",
         "prompt.heading": "System prompt",
-        "prompt.hint": "This text is re-read before every model call, so a save applies on the next step. It affects only sessions using this mode.",
+        "prompt.hint": "This text is re-read before every model call, so a save applies on the next step and only affects sessions on this assistant. \"Import\" reads a file into the editor (nothing is written until you save); \"Export\" saves the current text as a .md file.",
         "status.enabled": "Enabled",
         "status.disabled": "Disabled",
         "status.changed": "changed",
@@ -311,7 +311,7 @@ try {
         "base.ptc.label": "PTC",
         "base.ptc.note": "Standard plus PTC tool presentation (tool-presentation)",
         "base.minimal.label": "Minimal",
-        "base.minimal.note": "Shell and terminal only, 7 rows; no files, search, skills or subagents",
+        "base.minimal.note": "Shell and terminal only, 7 rows; no files, search, skills or subagents. The system prompt is not minimal's either (this mode replaces the persona row)",
         "base.cordis.label": "Cordis",
         "base.cordis.note": "Standard plus the Cordis toolset, letting the agent modify its own harness"
       }

package/composition.mjs

@@ -297,10 +297,16 @@ function setDisabled(segmentText, disabled) {
   const own = disabledLineAt(indent)
   const index = lines.findIndex((line) => own.test(line))
   if (index !== -1) {
+    const shipped = lines[index].trim().replace(/^disabled:\s*/, '')
     if (disabled) {
       lines[index] = `${' '.repeat(indent)}disabled: true`
+    } else if (shipped.startsWith('!!js') && evalDisabledExpression(shipped) !== true) {
+      // 显式打开，但出厂那行是平台表达式、且它在本机求值就是"开"：这不是覆盖，而是**撤销覆盖** ——
+      // 保持出厂表达式原样，这一行就回到"跟随平台"。（原先这里删掉整行，结果是文件既不是出厂原样、
+      // 也不是显式覆盖，页面却显示"未拨过" —— 文件与显示同时失真。）
+      return segmentText
     } else {
-      lines.splice(index, 1)
+      lines[index] = `${' '.repeat(indent)}disabled: false`
     }
     return lines.join('\n')
   }
@@ -344,7 +350,8 @@ function describeRow(id, segmentText, children) {
   const meta = ROW_META[id] ?? {}
   // A literal `disabled: true` is off. A `!!js` predicate is resolved for THIS
   // machine, so the page shows the state actually in force rather than "has a key".
-  const literalOff = own.present && !own.value.startsWith('!!js')
+  // 只看"有没有 disabled 键"是不够的：`disabled: false` 也是字面量，但它表示**开着**。
+  const literalOff = own.present && !own.value.startsWith('!!js') && own.value === 'true'
   const fromExpression = own.present ? evalDisabledExpression(own.value) : undefined
   return {
     id,
@@ -564,6 +571,15 @@ function flattenRows(rows, into = new Map()) {
   return into
 }
 
+/** 同 {@link flattenRows}，但保留整行对象：判断"未触碰"要看**文本形态**，不只是求值结果。 */
+function flattenRowObjects(rows, into = new Map()) {
+  for (const row of rows) {
+    into.set(row.id, row)
+    flattenRowObjects(row.children, into)
+  }
+  return into
+}
+
 /**
  * Derive the explicit overrides that turned `base` into `text`.
  *
@@ -578,13 +594,19 @@ function flattenRows(rows, into = new Map()) {
  * @returns {Record<string, boolean>} row id -> enabled.
  */
 export function overridesOf(text, modeId) {
-  const base = flattenRows(collectRows(readBaseComposition(modeId)))
-  for (const extra of EXTRA_ROWS) base.set(extra.id, false)
-  const current = flattenRows(collectRows(text))
+  const base = flattenRowObjects(collectRows(readBaseComposition(modeId)))
+  for (const extra of EXTRA_ROWS) base.set(extra.id, { id: extra.id, disabled: false, disabledExpression: null })
+  const current = flattenRowObjects(collectRows(text))
   const overrides = {}
-  for (const [id, disabled] of current) {
-    if (base.get(id) === disabled) continue
-    overrides[id] = disabled !== true
+  for (const [id, row] of current) {
+    const baseRow = base.get(id)
+    // "未触碰"的判据是**文本形态相同**：出厂行可能是 `!!js` 平台表达式，而它在本机的求值结果恰好与
+    // 某个字面量一致 —— 只看求值结果，会把"表达式被换成字面量"误判成"没动过"。
+    const sameForm = baseRow !== undefined
+      && baseRow.disabled === row.disabled
+      && baseRow.disabledExpression === row.disabledExpression
+    if (sameForm) continue
+    overrides[id] = row.disabled !== true
   }
   return overrides
 }

package/index.mjs

@@ -39,7 +39,7 @@
  * rows the user changed by diffing against the same shipped base mode.
  */
 
-import { existsSync, readFileSync, writeFileSync } from 'node:fs'
+import { existsSync, readFileSync, renameSync, writeFileSync } from 'node:fs'
 import { dirname, join } from 'node:path'
 import { PROMPT_PATH, COMPOSITION_PATH, ROUTE_PATH, PRESET_DIR } from './paths.mjs'
 import {
@@ -282,8 +282,8 @@ export function saveState(rows, input) {
   }
 
   try {
-    writeFileSync(compositionFile(directory), composition, 'utf8')
-    writeFileSync(promptFile(directory), prompt, 'utf8')
+    writeAtomic(compositionFile(directory), composition)
+    writeAtomic(promptFile(directory), prompt)
   } catch (error) {
     return { ok: false, error: '写入失败：' + describe(error) }
   }
@@ -380,7 +380,7 @@ export function createAssistant(rows, input, templateDir = packagedPresetDir())
   // The template ships a starter prompt; a duplicate replaces it with the source's.
   if (source !== null) {
     try {
-      writeFileSync(promptFile(created.dir), source.prompt, 'utf8')
+      writeAtomic(promptFile(created.dir), source.prompt)
     } catch (error) {
       return { ok: false, error: '写入提示词失败：' + describe(error) }
     }
@@ -669,6 +669,19 @@ export function apply(ctx) {
   })
 }
 
+/**
+ * 写文件：先写同目录的临时文件，再 rename 覆盖。
+ *
+ * `rename(2)` 在同一文件系统内是原子的，所以读者（提示词读取器按 mtime+size 缓存）要么看到旧内容、
+ * 要么看到新内容，不会读到写了一半的文件。原来连续两次 writeFileSync 在极端时序下可能被读成撕裂的
+ * prompt，而这个文件正是"用户的提示词"。
+ */
+function writeAtomic(file, text) {
+  const temporary = `${file}.tmp-${String(process.pid)}`
+  writeFileSync(temporary, text, 'utf8')
+  renameSync(temporary, file)
+}
+
 /** `error` as a readable string, without assuming it is an Error. */
 function describe(error) {
   return String((error && error.message) || error)

package/locales.mjs

@@ -67,14 +67,14 @@ export const zh = {
   'name.descriptionPlaceholder': '模式描述（显示在模式选择器里，可留空）',
 
   'mode.heading': '基础模式',
-  'mode.hint': '选一个官方模式作为底子，下面再按行微调。改完保存后，新建会话即生效，不需要重启。',
+  'mode.hint': '选一个官方模式作为底子，下面再按行微调。注意：底子只决定**行集合**与工具能力 —— 本模式的 persona 行始终替换掉底子那一行（提示词由你编辑，complete: false），底子的提示词语义不会被继承。改完保存后，新建会话即生效，不需要重启。',
 
   'rows.heading': '插件开关',
   'rows.hint': '逐行控制这个模式挂载哪些插件，和官方插件列表一样按行铺开。没拨过的行保持官方默认（含平台判断）；你手动拨了就以你的为准。',
 
   'prompt.heading': '系统提示词',
   'prompt.hint':
-    'This text is re-read before every model call, so a save applies on the next step and only affects sessions on this assistant. "Import" reads a file into the editor (nothing is written until you save); "Export" saves the current text as a .md file.',
+    '这段文本在每个模型调用前重新读取，所以保存后下一步即生效，且只影响使用这个助手的会话。「导入」把文件读进编辑器（未保存前不写入任何东西）；「导出」把当前文本存成 .md 文件。',
 
   'status.enabled': '已启用',
   'status.disabled': '已停用',
@@ -156,7 +156,7 @@ export const zh = {
   'base.ptc.label': 'PTC 模式',
   'base.ptc.note': '在标准模式基础上启用 PTC 工具呈现（tool-presentation）',
   'base.minimal.label': '极简模式',
-  'base.minimal.note': '只有 Shell 与终端，共 7 行；没有文件、检索、技能、子代理',
+  'base.minimal.note': '只有 Shell 与终端，共 7 行；没有文件、检索、技能、子代理。系统提示词不是极简那一套（persona 行被本模式替换）',
   'base.cordis.label': 'Cordis 模式',
   'base.cordis.note': '标准模式 + 读写运行时的 Cordis 工具集，可让 agent 自己改 harness',
 }
@@ -216,7 +216,7 @@ export const en = {
 
   'mode.heading': 'Base mode',
   'mode.hint':
-    'Choose an official mode as the base, then fine-tune individual rows below. After saving, a new session picks it up — no restart needed.',
+    'Pick an official mode as the base, then adjust it row by row. Note: the base only decides which rows exist — this mode always replaces the base\'s persona row with its own reader (complete: false), so the base\'s prompt semantics are not inherited. A save applies to the next new session; no restart needed.',
 
   'rows.heading': 'Plugin switches',
   'rows.hint':
@@ -224,7 +224,7 @@ export const en = {
 
   'prompt.heading': 'System prompt',
   'prompt.hint':
-    'This text is re-read before every model call, so a save applies on the next step. It affects only sessions using this mode.',
+    'This text is re-read before every model call, so a save applies on the next step and only affects sessions on this assistant. "Import" reads a file into the editor (nothing is written until you save); "Export" saves the current text as a .md file.',
 
   'status.enabled': 'Enabled',
   'status.disabled': 'Disabled',
@@ -306,7 +306,7 @@ export const en = {
   'base.ptc.label': 'PTC',
   'base.ptc.note': 'Standard plus PTC tool presentation (tool-presentation)',
   'base.minimal.label': 'Minimal',
-  'base.minimal.note': 'Shell and terminal only, 7 rows; no files, search, skills or subagents',
+  'base.minimal.note': 'Shell and terminal only, 7 rows; no files, search, skills or subagents. The system prompt is not minimal\'s either (this mode replaces the persona row)',
   'base.cordis.label': 'Cordis',
   'base.cordis.note': 'Standard plus the Cordis toolset, letting the agent modify its own harness',
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "dsh-custom-mode",
-  "version": "1.0.0",
-  "description": "Settings page for the dsh-custom-mode agent preset: base-mode switching, per-row plugin switches, an editable system prompt, and mode renaming. Follows the DSH version.",
+  "version": "1.0.2",
+  "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.",
   "license": "MIT",
   "type": "module",
   "main": "./index.mjs",
@@ -22,9 +22,18 @@
   "keywords": [
     "dsh",
     "deepseek-harness",
+    "dsh-plugin",
+    "cordis",
     "agent-preset",
+    "agent-mode",
+    "custom-mode",
+    "multi-mode",
+    "assistant",
     "system-prompt",
-    "plugin"
+    "custom-prompt",
+    "prompt-editor",
+    "prompt-engineering",
+    "persona"
   ],
   "dsh": {
     "bundle": {
@@ -50,7 +59,7 @@
     "test": "node ../test/run.mjs"
   },
   "peerDependencies": {
-    "@deepseek-ai/dsh": ">=0.1.6-alpha.1"
+    "@deepseek-ai/dsh": ">=0.1.6-alpha.1 <0.2.0-0"
   },
   "publishConfig": {
     "access": "public",
@@ -62,6 +71,6 @@
     }
   },
   "engines": {
-    "dsh": ">=0.1.6-alpha.1"
+    "dsh": ">=0.1.6-alpha.1 <0.2.0-0"
   }
 }