dsh-persona-forge 0.3.0 → 0.3.1

package/CHANGELOG.md

@@ -46,12 +46,36 @@ adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ### Changed
 
+- **The composer menu is a picker again, not a settings form.** It had grown to
+  five equal-weight segmented rows plus 248 characters of permanently visible
+  explanation, so picking a character meant scanning past four configuration
+  rows every time. It is now two layers: card list, intensity and send mode
+  always visible; **rewrite mode and fidelity** — per-session overrides of a
+  card's own declaration — folded behind "More settings", shut by default. The
+  collapsed row still names anything overridden, so a hidden choice cannot
+  surprise the user later. Measured: 5 rows + 248 hint chars → 2 rows + 0
+  permanent hint chars.
+
+- **Fidelity is only offered when the card may add behavioural constraints.** On
+  a voice-only card both options resolve identically, so the row was a control
+  that could not do anything, paired with a paragraph explaining why the thing
+  you would want was refused. It is now absent on `style` cards.
+
+- **The model check moved to the settings page** as a global preference rather
+  than a per-session switch in the composer menu. It answers "do I want to pay
+  for the second call", which is decided once, not re-asked on the way to every
+  send.
+
+- "No persona" moved to the end of the card list: it is an exit action, and
+  listing it first made it the first thing read in a menu whose point is choosing
+  a character.
+
 - **The model-based preservation check is now off by default** and switchable
-  from the menu. It is the only thing that can see a requirement reworded weaker,
-  or a constraint added in prose — but it costs a second model call on **every**
-  send, and the new local check already proves the common and most damaging
-  failure for free. `factCheck` in the plugin config now sets that default, and
-  only an explicit `true` enables it.
+  from the settings page. It is the only thing that can see a requirement
+  reworded weaker, or a constraint added in prose — but it costs a second model
+  call on **every** send, and the new local check already proves the common and
+  most damaging failure for free. `factCheck` in the plugin config now sets that
+  default, and only an explicit `true` enables it.
 
 - The Chinese README is the primary `README.md` and the English one is
   `README.en.md`, so GitHub, npm and the repository landing page open in Chinese,
@@ -82,6 +106,15 @@ adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
   confirmation for an edited bundled card says that its changes are discarded
   and the bundled version returns.
 
+- **A cancelled rewrite could not be retried.** `lastRewrite` was written on
+  every successful rewrite and never removed, so that the send gesture could
+  tell "the rewrite is already in the composer" from "not rewritten yet".
+  Restoring the original put the draft text back but left that memory in place,
+  so the next send matched "already rewritten" and the plugin silently let the
+  raw text through. Restoring is a rewind, not a reviewed result, so it now
+  deletes the entry — conditionally on putting back the original, because
+  sending the rewrite itself must still count as already rewritten.
+
 - **A narrowed fidelity never reached the model.** The route computed the
   narrowed value and reported it in the response, but `buildSystemPrompt` read
   `card.fidelity` directly — so the setting appeared to work and did nothing. The

package/README.en.md

@@ -126,22 +126,15 @@ you want them gone.
 
 ## Use
 
-1. Click 🎭 in the composer tool row and pick a character. **No persona** in the
-   same menu turns rewriting off — that is the state the control starts in, and
-   your choice is remembered per session.
-2. Set **intensity, rewrite mode, fidelity, model check** and **send mode**:
+1. Click 🎭 in the composer tool row and pick a character. **No persona**, at the
+   end of the same list, turns rewriting off — that is the state the control
+   starts in, and your choice is remembered per session.
+2. Set **intensity** and **send mode**:
    - Intensity is a four-rung ladder: **Light · Medium · Strong · Zealot**
      (轻 · 中 · 重 · 狂热). Light adds a touch of flavour; Medium is clear but not
      dominant; Strong gives the voice real weight while keeping the request the
      first thing the reader sees; Zealot is ritual throughout. A card carries
      its own default, and this choice overrides it for your session.
-   - **Rewrite mode**: Card default / Model rewrite / Template. A temporary
-     switch applies to this session only; if the card has no content for the
-     mode you picked (no `style`, or no `template`), the override is ignored and
-     falls back to the card's own mode.
-   - **Fidelity**: Card default / Voice only. **Narrowing only, never widening**
-     — see below.
-   - **Model check**: on / off — see *The preservation check* below.
    - **Review first** — the rewrite appears in the comparison panel below the
      composer; you read it and then press Send there. Your draft is untouched
      until you confirm.
@@ -151,6 +144,12 @@ you want them gone.
 3. With a character selected, the composer's normal **Send** button — or
    **Enter** — *is* the rewrite trigger.
 
+**More settings**, at the bottom of the menu, is folded shut by default. It holds
+**rewrite mode** and **fidelity** — per-session OVERRIDES of a card's own
+declaration that most people set once and never touch, so they do not take space
+in a menu whose job is choosing a character. When something *is* overridden, the
+collapsed row names it, so a choice you forgot about cannot hide behind the fold.
+
 The rewrite runs **once per draft**: after a rewrite lands, sending goes through
 as-is instead of rewriting the output again. Editing the draft re-arms it.
 
@@ -214,10 +213,11 @@ is already proven for free by layer 1, so paying a model call on every send to
 re-check it is a waste. Turn it on when the drafts are technical enough to
 warrant it.
 
-The 🎭 menu has **Model check: on / off**, remembered per session. With it off
-the panel states plainly that the local check ran and spent no tokens, and says
-what that check does **not** cover — a narrow verdict has to declare its own
-limits.
+The switch lives in **Settings → Persona** ("Enable the model check") and is a
+global preference: it answers "do I want to pay for the second call", which is
+decided once rather than re-asked on the way to every send. With it off the panel
+states plainly that the local check ran and spent no tokens, and says what that
+check does **not** cover — a narrow verdict has to declare its own limits.
 
 Both layers are **advisory** and never block a rewrite: they annotate, not stop.
 
@@ -384,9 +384,12 @@ ever narrowed, never widened:**
   author's decision, not something to grant with a click on the way to Send. To
   widen, edit the card.
 
-The 🎭 menu's *Fidelity* defaults to **Card default**; on a `style` card the hint
-says it cannot be widened here. The server enforces the same rule, so the UI
-never offers a control it would ignore.
+The control appears under *More settings*, and **only when the card is
+`strategy`**. On a `style` card it is not rendered at all: both options would
+resolve identically there, and showing a control that cannot do anything — plus a
+paragraph explaining why the other one is refused — is worse than showing
+nothing. The server enforces the same rule, so the UI never offers a control it
+would ignore.
 
 ### `mode: template`
 

package/README.md

@@ -108,19 +108,18 @@ dsh plugin --profile web remove dsh-persona-forge
 
 ## 使用
 
-1. 点输入框工具行的 🎭，选一个角色。同一菜单里的 **「无角色」** 可关闭改写 —— 这也是控件的初始状态，选择按会话记住。
-2. 设置**强度、改写方式、保真度、模型校验、发送方式**：
+1. 点输入框工具行的 🎭，选一个角色。列表最后的 **「无角色」** 可关闭改写 —— 这也是控件的初始状态，选择按会话记住。
+2. 设置**强度和发送方式**：
    - 强度是四档阶梯：**轻 · 中 · 重 · 狂热**（Light · Medium · Strong · Zealot）。轻=只加一点味道；中=明显但不喧宾夺主；重=语气充分，需求依然一眼可见；狂热=通篇仪式化。每张卡有自己的默认值，这里的选择会覆盖它。
-   - **改写方式**：跟随卡片（默认）/ 模型改写 / 模板套用。临时切换只影响当前会话；如果卡片没有对应的内容（没有 `style` 或没有 `template`），这个覆盖会被忽略并回退到卡片自己的方式。
-   - **保真度**：跟随卡片（默认）/ 只改语气。**只能收窄，不能放宽** —— 见下方说明。
-   - **模型校验**：开/关，见下方「技术事实校验」。
    - **先审查** —— 改写结果显示在输入框下方的对比面板里；你看过之后按面板里的发送。**在确认之前，你的草稿一字不动。**
    - **直接发出** —— 改写完成后立即作为你的消息发出。面板会保留显示实际发出的内容，因为已发出的消息无法撤回。
 3. 选了角色之后，输入框**原生的发送按钮、以及 Enter 键**就是改写触发器。
 
+菜单底部的 **「更多设置」** 默认收起，里面是**改写方式**与**保真度** —— 它们是对卡片自身声明的**按会话覆盖**，多数人设一次就不会再动，所以不占主菜单。收起时若有覆盖，这一行会**写出改了什么**，不会让一个你忘了的选择藏在折叠后面。
+
 **同一份草稿只改写一次**：改写结果落到输入框后，再按发送会原样发出，不会把改写结果再改写一遍。改动草稿会重新触发改写。
 
-你的草稿会被保留：**还原原文** 可恢复，**仅回填** 只把改写结果放进输入框而不发送。
+你的草稿会被保留：**还原原文** 可恢复，且**会撤销"已改写"的记忆** —— 所以撤消后还能再改一次。**仅回填** 只把改写结果放进输入框而不发送。
 
 ### 发送拦截的原理与它唯一的风险
 
@@ -148,7 +147,7 @@ dsh plugin --profile web remove dsh-persona-forge
 
 代价是**延迟和 token 都翻倍**，所以默认关闭：最常见、最伤人的失败（丢掉文件名 / 数字 / 版本号）已经被本地检查免费证明了，为它每次发送都付一次模型调用是浪费。草稿偏技术时再开。
 
-菜单里 **模型校验：开 / 关**，选择按会话记住。关闭时面板会写明「已做本地检查（不耗 token）」，并说清它**没有**覆盖什么 —— 一个窄的结论必须自己交代边界。
+开关在**设置 → 角色扮演**（「开启模型校验」），是个全局偏好 —— 它回答的是「我要不要为第二次调用付费」，一次决定即可，不必在每次发送前重新问一遍。关闭时面板会写明「已做本地检查（不耗 token）」，并说清它**没有**覆盖什么 —— 一个窄的结论必须自己交代边界。
 
 两层都是**建议性**的，绝不阻断改写：只标注，不拦你。
 
@@ -266,7 +265,7 @@ examplesByIntensity:
 - **收窄（允许）**：一张 `strategy` 卡可以临时要求「本次只改语气」。这只是**拿走**它追加约束的权力，所以任何情况下都更安全。
 - **放宽（拒绝）**：一张 `style` 卡**不能**被要求改成 `strategy`。放宽意味着让改写往你的需求里加你没写过的约束 —— 那是卡片作者的决定，不该由「发送前顺手一点」来授权。真要放宽，去编辑那张卡。
 
-输入框菜单里的「保真度」默认**跟随卡片**；若卡片是 `style`，提示会写明这里无法放宽。服务端同样强制这条规则，所以界面上不会给你一个会被忽略的按钮。
+这个开关在「更多设置」里，且**只在卡片是 `strategy` 时出现**。`style` 卡上它不显示 —— 那种卡上两个选项效果完全一样，与其摆一个做不了事的控件、再配一段解释为什么不能放宽，不如不显示。服务端强制同一条规则，所以界面上不会给你一个会被忽略的按钮。
 
 ### `mode: template`
 

package/lib/client.js

@@ -38,8 +38,10 @@ window.__ModuleLoader__.load({
       'control.busy': '改写中…',
       'control.menu.title': '选择角色',
       'control.menu.empty': '还没有角色卡。把 .yml 文件放进角色卡目录即可添加。',
-      'control.menu.manage': '管理角色卡…',
       'control.menu.reveal': '打开角色卡目录',
+      'control.more': '更多设置',
+      'control.more.open': '展开改写方式、保真度等按会话生效的设置',
+      'control.more.custom': '已改：',
       'control.intensity': '强度',
       'control.intensity.light': '轻',
       'control.intensity.medium': '中',
@@ -60,10 +62,6 @@ window.__ModuleLoader__.load({
       'control.fidelity.style': '只改语气',
       'control.fidelity.hint.style': '这张卡声明的是「只改语气」，不能在这里放宽——放宽是改卡的属性，请在设置页改。',
       'control.fidelity.hint.strategy': '这张卡允许追加行为约束。这里可以临时收窄成「只改语气」，但收不回去。',
-      'control.modelCheck': '模型校验',
-      'control.modelCheck.on': '开',
-      'control.modelCheck.off': '关',
-      'control.modelCheck.hint': '开启后每次改写会多调用一次模型，检查「需求被悄悄改弱」这类本地检查看不出的问题——代价是延迟和 token 翻倍。关闭时仍会做本地检查（标识符、路径、版本号、数字），不耗 token。',
       'panel.title': '角色改写',
       'panel.loading': '正在按「{name}」改写…',
       'panel.original': '你写的',
@@ -78,6 +76,7 @@ window.__ModuleLoader__.load({
       'panel.sent.hint': '消息已发出，无法撤回。下面是实际发出的内容。',
       'panel.meta': '{provider} / {model} · {ms}ms · 强度 {intensity}',
       'panel.template': '模板模式（未调用模型）',
+      'panel.slow': '已经 {seconds} 秒了 —— 模型可能较慢。再等一会儿通常会有结果；若一直不动，可关闭面板后重试。',
       'panel.drift': '技术事实可能被改动',
       'panel.checkUnavailable': '未完成事实校验',
       'panel.checkOk': '技术事实已保留',
@@ -88,7 +87,6 @@ window.__ModuleLoader__.load({
       'panel.stale': '草稿在改写期间有改动——结果基于改写前的文本。',
       'error.empty': '输入框是空的，先写下你的需求。',
       'error.phase': '输入框正忙（提交中），请稍后再试。',
-      'error.busy': '已有一个改写在进行中。',
       'error.noCard': '请先选择一个角色。',
       'error.unconfigured': '无法确定模型：先在本会话发一条消息，或设置一个默认模型。',
       'error.timeout': '改写超时，请重试；原输入未改动。',
@@ -104,7 +102,6 @@ window.__ModuleLoader__.load({
       'settings.ai.tip': '新建角色卡时可以让 AI 代填：写一句角色思路，风格和四档范例都会自动写好。',
       'settings.ai.model': '使用模型：{provider} / {model}（设置页没有会话上下文，用的是 harness 默认模型；改写本身跟随当前会话的模型）',
       'settings.ai.desc': '用一句话描述角色的思路，AI 会按同一套规范把表单填好：风格、四档强度的转换范例都写好。生成结果只填入表单，不会自动保存。',
-      'settings.ai.briefLabel': '角色思路（一句话或几句话都行）',
       'settings.ai.briefPlaceholder': '例如：一个脾气暴躁但极其可靠的老灯塔守夜人，说话简短、爱用航海比喻',
       'settings.ai.generate': '生成草稿',
       'settings.ai.busy': 'AI 正在撰写角色卡…',
@@ -126,7 +123,6 @@ window.__ModuleLoader__.load({
       'settings.check.none': '所有角色卡都与内置版本一致。',
       'settings.check.some': '有 {n} 张角色卡与内置版本不一致：',
       'settings.check.desc': '点一次即可查出哪些卡偏离了内置版本。只有不一致的卡才显示「还原为内置」。',
-      'settings.changed': '改动：',
       'settings.save': '保存',
       'settings.cancel': '取消',
       'settings.saved': '已保存',
@@ -162,8 +158,10 @@ window.__ModuleLoader__.load({
       'control.busy': 'Rewriting…',
       'control.menu.title': 'Choose a character',
       'control.menu.empty': 'No character cards yet. Drop a .yml file into the card directory to add one.',
-      'control.menu.manage': 'Manage cards…',
       'control.menu.reveal': 'Open card directory',
+      'control.more': 'More settings',
+      'control.more.open': 'Show rewrite mode, fidelity and other per-session settings',
+      'control.more.custom': 'Changed: ',
       'control.intensity': 'Intensity',
       'control.intensity.light': 'Light',
       'control.intensity.medium': 'Medium',
@@ -184,10 +182,6 @@ window.__ModuleLoader__.load({
       'control.fidelity.style': 'Voice only',
       'control.fidelity.hint.style': 'This card declares voice-only, and it cannot be widened here — widening is a property of the card, so change it in the settings page.',
       'control.fidelity.hint.strategy': 'This card may add behavioural constraints. You can narrow it to voice-only for this session, but not widen it back.',
-      'control.modelCheck': 'Model check',
-      'control.modelCheck.on': 'On',
-      'control.modelCheck.off': 'Off',
-      'control.modelCheck.hint': 'On: every rewrite makes one extra model call to catch what the local check cannot see, such as a requirement quietly weakened — at the cost of doubled latency and tokens. Off: the free local check still runs (identifiers, paths, versions, numbers).',
       'panel.title': 'Persona rewrite',
       'panel.loading': 'Rewriting as “{name}”…',
       'panel.original': 'You wrote',
@@ -202,6 +196,7 @@ window.__ModuleLoader__.load({
       'panel.sent.hint': 'The message is already sent and cannot be recalled. This is what was sent.',
       'panel.meta': '{provider} / {model} · {ms}ms · intensity {intensity}',
       'panel.template': 'Template mode (no model call)',
+      'panel.slow': '{seconds} s and counting — the model may just be slow. It usually returns; if it never does, close the panel and retry.',
       'panel.drift': 'Technical facts may have changed',
       'panel.checkUnavailable': 'Preservation check did not complete',
       'panel.checkOk': 'Technical facts preserved',
@@ -212,7 +207,6 @@ window.__ModuleLoader__.load({
       'panel.stale': 'The draft changed while rewriting — the result is based on the earlier text.',
       'error.empty': 'The input box is empty — write your request first.',
       'error.phase': 'The input box is busy (submitting) — try again shortly.',
-      'error.busy': 'A rewrite is already running.',
       'error.noCard': 'Choose a character first.',
       'error.unconfigured': 'No model resolved: send a message in this session first, or set a default model.',
       'error.timeout': 'The rewrite timed out. Retry; your draft is untouched.',
@@ -228,7 +222,6 @@ window.__ModuleLoader__.load({
       'settings.ai.tip': 'You can have the AI draft a card for you: describe the character in a sentence and the style text plus all four intensity example sets are written for you.',
       'settings.ai.model': 'Model: {provider} / {model} (the settings page has no session context, so this is the harness default; the rewrite itself follows the current session\'s model)',
       'settings.ai.desc': 'Describe the character in a sentence and the AI fills the form for you: the style text and conversion examples for all four intensity levels. The result only fills the form — it is never saved automatically.',
-      'settings.ai.briefLabel': 'Character idea (a sentence or a few)',
       'settings.ai.briefPlaceholder': 'e.g. a grumpy but utterly reliable old lighthouse keeper who speaks in short sentences and nautical metaphors',
       'settings.ai.generate': 'Generate draft',
       'settings.ai.busy': 'Writing the character card…',
@@ -250,7 +243,6 @@ window.__ModuleLoader__.load({
       'settings.check.none': 'Every character card matches its bundled version.',
       'settings.check.some': '{n} card(s) differ from the bundled version:',
       'settings.check.desc': 'Finds which cards have drifted from the bundled version. Only a drifted card shows a Restore bundled action.',
-      'settings.changed': 'Changed:',
       'settings.save': 'Save',
       'settings.cancel': 'Cancel',
       'settings.saved': 'Saved',
@@ -333,6 +325,15 @@ window.__ModuleLoader__.load({
 .pf-menu{position:fixed;z-index:1100;box-sizing:border-box;width:max-content;min-width:300px;max-width:min(460px,100vw - 24px);max-height:min(70vh,560px);overflow-y:auto;padding:4px;border:0;border-radius:var(--dsw-radius-lg);background:var(--dsw-menu-surface-fill,var(--dsw-alias-bg-overlay));-webkit-backdrop-filter:var(--dsw-menu-backdrop-filter,blur(40px) saturate(150%));backdrop-filter:var(--dsw-menu-backdrop-filter,blur(40px) saturate(150%));color:var(--dsw-alias-label-primary);--dsw-elevation-stroke-color:var(--dsw-alias-border-l1);box-shadow:var(--dsw-elevation-prominent);--dsh-scrollbar-thumb:var(--dsw-alias-scrollbar-bg-l2);--dsh-scrollbar-thumb-hover:var(--dsw-alias-scrollbar-hover-l2)}
 .pf-menu-title{padding:6px 8px 4px;font-size:11px;line-height:15px;color:var(--dsw-alias-label-tertiary)}
 .pf-menu-sep{height:.5px;margin:3px 2px;background:var(--dsw-alias-border-l2)}
+/* The fold for advanced settings: a quiet row, not another equal-weight control. */
+.pf-more{color:var(--dsw-alias-label-secondary)}
+.pf-more-body{padding:2px 8px 8px;display:grid;gap:6px}
+.pf-pref-row{display:flex;align-items:center;gap:6px;cursor:pointer}
+.pf-pref-row input{margin:0}
+/* The live counter during a rewrite: a muted suffix so the wait reads as
+ * progress rather than a freeze. */
+.pf-elapsed{margin-left:8px;padding:0 6px;border-radius:var(--dsw-radius-sm);background:var(--dsw-alias-interactive-bg-hover);color:var(--dsw-alias-label-secondary);font-variant-numeric:tabular-nums}
+
 .pf-item{display:flex;align-items:center;gap:6px;width:100%;min-height:34px;padding:6px 8px;border:none;border-radius:var(--dsw-radius-md);background:transparent;color:var(--dsw-alias-label-primary);text-align:left;font:inherit;font-size:13px;line-height:20px;cursor:pointer}
 .pf-item:hover:not(:disabled){background:var(--dsw-alias-interactive-bg-hover)}
 .pf-item:focus-visible{background:var(--dsw-alias-interactive-bg-hover);outline:none}
@@ -500,8 +501,6 @@ window.__ModuleLoader__.load({
     const intensityKey = (key) => `persona-forge:intensity:${key}`
     const modeKey = (key) => `persona-forge:mode:${key}`
     const fidelityKey = (key) => `persona-forge:fidelity:${key}`
-    const modelCheckKey = (key) => `persona-forge:modelCheck:${key}`
-
     /** Effective mode for a card: the session's pick when the card can honour it, else the card's. */
     function effectiveModeOf(card, override) {
       if (card === null) return 'llm'
@@ -674,6 +673,7 @@ window.__ModuleLoader__.load({
         panel: {
           sessionKey: options.sessionKey,
           phase: 'loading',
+          startedAt: Date.now(),
           cardId: options.cardId,
           cardName: options.cardName,
           original: options.original,
@@ -754,14 +754,10 @@ window.__ModuleLoader__.load({
       // stored as "unset" until the user actually picks something, so following
       // the card stays the default rather than an explicit choice.
       const [modeOverride, setModeOverride] = useState(() => readLocal(modeKey(sessionKey)))
+      // Advanced rows start folded: the menu's job is choosing a character, and
+      // these overrides are set once and then forgotten.
+      const [advancedOpen, setAdvancedOpen] = useState(false)
       const [fidelityOverride, setFidelityOverride] = useState(() => readLocal(fidelityKey(sessionKey)))
-      // Tri-state: undefined = follow the host config, true/false = this
-      // session's explicit choice. Stored as a string because localStorage has
-      // no booleans.
-      const [modelCheckOverride, setModelCheckOverride] = useState(() => {
-        const stored = readLocal(modelCheckKey(sessionKey))
-        return stored === 'on' ? true : stored === 'off' ? false : undefined
-      })
 
       useEffect(() => {
         if (shared.catalog === null && shared.catalogError === null) {
@@ -777,7 +773,20 @@ window.__ModuleLoader__.load({
       const effectiveIntensity = intensity ?? selected?.intensity ?? 'medium'
       const effectiveMode = effectiveModeOf(selected, modeOverride)
       const effectiveFidelity = effectiveFidelityOf(selected, fidelityOverride)
-      const effectiveModelCheck = modelCheckOverride ?? shared.factCheck
+      // The model check is a one-time decision about cost, so it is a GLOBAL
+      // preference (set in the settings page), not a per-session override.
+      // One source of truth: the host config. A client-side copy would be a
+      // second setting for the same thing, and the model check is off by
+      // default because it is paid for on every send.
+      const effectiveModelCheck = shared.factCheck
+
+      // What the fold would reveal, in one line. A folded-away override the user
+      // forgot about is a surprise waiting to happen, so the collapsed row still
+      // names anything that is not following the card.
+      const advancedSummary = [
+        modeOverride === 'llm' ? t('control.mode.llm') : modeOverride === 'template' ? t('control.mode.template') : '',
+        fidelityOverride === 'style' ? t('control.fidelity.style') : '',
+      ].filter((part) => part !== '').join(' · ')
 
       // The send-gesture listener is attached once per session but must read
       // CURRENT values. A ref carries them across renders without re-attaching
@@ -827,11 +836,6 @@ window.__ModuleLoader__.load({
         writeLocal(fidelityKey(sessionKey), fidelity)
       }, [sessionKey])
 
-      const setModelCheck = useCallback((enabled) => {
-        setModelCheckOverride(enabled)
-        writeLocal(modelCheckKey(sessionKey), enabled ? 'on' : 'off')
-      }, [sessionKey])
-
       // Position the menu above the button, clamped to the viewport.
       useEffect(() => {
         if (!open) return
@@ -1028,26 +1032,6 @@ window.__ModuleLoader__.load({
                 style: { left: `${menuPos.left}px`, bottom: `${menuPos.bottom}px`, width: `${menuPos.width}px` },
               },
               h('div', { className: 'pf-menu-title' }, t('control.menu.title')),
-              // "No persona" is a real choice, listed like any card, so the
-              // control can always be turned back off.
-              h(
-                'button',
-                {
-                  type: 'button',
-                  role: 'menuitemradio',
-                  'aria-checked': isOff ? 'true' : 'false',
-                  className: `pf-item${isOff ? ' is-selected' : ''}`,
-                  onClick: () => pickCard(OFF),
-                },
-                h('span', { className: 'pf-item-icon', 'aria-hidden': true }, '🚫'),
-                h(
-                  'span',
-                  { className: 'pf-item-body' },
-                  h('span', { className: 'pf-item-name' }, t('control.none')),
-                  h('span', { className: 'pf-item-desc' }, t('control.none.desc')),
-                ),
-                isOff ? h('span', { className: 'pf-check', 'aria-hidden': true }, '✓') : null,
-              ),
               cards.length === 0
                 ? h('div', { className: 'pf-hint' }, shared.catalogError ?? t('control.menu.empty'))
                 : cards.map((card) => h(
@@ -1071,6 +1055,28 @@ window.__ModuleLoader__.load({
                     ),
                     card.id === cardId ? h('span', { className: 'pf-check', 'aria-hidden': true }, '✓') : null,
                   )),
+              // "No persona" is a real choice, listed like any card so the
+              // control can always be turned back off — but LAST: it is an exit
+              // action, and putting it above the cards made it the first thing
+              // a user reads in a menu whose whole point is picking a character.
+              h(
+                'button',
+                {
+                  type: 'button',
+                  role: 'menuitemradio',
+                  'aria-checked': isOff ? 'true' : 'false',
+                  className: `pf-item${isOff ? ' is-selected' : ''}`,
+                  onClick: () => pickCard(OFF),
+                },
+                h('span', { className: 'pf-item-icon', 'aria-hidden': true }, '🚫'),
+                h(
+                  'span',
+                  { className: 'pf-item-body' },
+                  h('span', { className: 'pf-item-name' }, t('control.none')),
+                  h('span', { className: 'pf-item-desc' }, t('control.none.desc')),
+                ),
+                isOff ? h('span', { className: 'pf-check', 'aria-hidden': true }, '✓') : null,
+              ),
               h('div', { className: 'pf-menu-sep' }),
               h(
                 'div',
@@ -1112,80 +1118,90 @@ window.__ModuleLoader__.load({
                 ),
               ),
               h('div', { className: 'pf-hint' }, t(`control.sendMode.hint.${effectiveSendMode}`)),
-              // Mode and fidelity, per session. Both defer to the card until the
-              // user picks: following the card is the default, not a choice the
-              // user has to rediscover.
-              h(
-                'div',
-                { className: 'pf-row' },
-                h('span', { className: 'pf-row-label' }, t('control.mode')),
-                h(
-                  'span',
-                  { className: 'pf-seg' },
-                  [['', t('control.mode.card')], ['llm', t('control.mode.llm')], ['template', t('control.mode.template')]].map(([value, label]) => h(
-                    'button',
-                    {
-                      key: value === '' ? 'card' : value,
-                      type: 'button',
-                      'aria-pressed': (modeOverride ?? '') === value ? 'true' : 'false',
-                      className: (modeOverride ?? '') === value ? 'is-on' : '',
-                      onClick: () => setMode(value === '' ? undefined : value),
-                    },
-                    label,
-                  )),
-                ),
-              ),
-              h('div', { className: 'pf-hint' }, t('control.mode.hint')),
-              h(
-                'div',
-                { className: 'pf-row' },
-                h('span', { className: 'pf-row-label' }, t('control.fidelity')),
-                h(
-                  'span',
-                  { className: 'pf-seg' },
-                  [['', t('control.fidelity.card')], ['style', t('control.fidelity.style')]].map(([value, label]) => h(
-                    'button',
-                    {
-                      key: value === '' ? 'card' : value,
-                      type: 'button',
-                      'aria-pressed': (fidelityOverride ?? '') === value ? 'true' : 'false',
-                      className: (fidelityOverride ?? '') === value ? 'is-on' : '',
-                      onClick: () => setFidelity(value === '' ? undefined : value),
-                    },
-                    label,
-                  )),
-                ),
-              ),
-              // A voice-only card can never be widened from here, so the hint
-              // says which card-level rule is in force rather than offering a
-              // control that would be ignored.
-              h('div', { className: 'pf-hint' }, selected?.fidelity === 'strategy'
-                ? t('control.fidelity.hint.strategy')
-                : t('control.fidelity.hint.style')),
-              // The model preservation check costs a second call per send, so
-              // it is a visible switch rather than a hidden config file edit.
-              // The free local check runs either way and cannot be turned off.
+              // Advanced settings, folded away.
+              //
+              // Rewrite mode and fidelity are per-session OVERRIDES of a card's
+              // own declaration, so almost everyone leaves them alone. Shown
+              // flat, they gave the menu five equal-weight control rows and made
+              // it read as a settings form instead of a character picker. The
+              // folded row still reports what is overridden, so a choice the
+              // user forgot about cannot hide behind the fold.
               h(
-                'div',
-                { className: 'pf-row' },
-                h('span', { className: 'pf-row-label' }, t('control.modelCheck')),
+                'button',
+                {
+                  type: 'button',
+                  className: 'pf-item pf-more',
+                  'aria-expanded': advancedOpen ? 'true' : 'false',
+                  title: t('control.more.open'),
+                  onClick: () => setAdvancedOpen((value) => !value),
+                },
+                h('span', { className: 'pf-item-icon', 'aria-hidden': true }, advancedOpen ? '▾' : '▸'),
                 h(
                   'span',
-                  { className: 'pf-seg' },
-                  [[true, t('control.modelCheck.on')], [false, t('control.modelCheck.off')]].map(([value, label]) => h(
-                    'button',
-                    {
-                      key: String(value),
-                      type: 'button',
-                      'aria-pressed': effectiveModelCheck === value ? 'true' : 'false',
-                      className: effectiveModelCheck === value ? 'is-on' : '',
-                      onClick: () => setModelCheck(value),
-                    },
-                    label,
-                  )),
+                  { className: 'pf-item-body' },
+                  h('span', { className: 'pf-item-name' }, t('control.more')),
+                  advancedSummary !== ''
+                    ? h('span', { className: 'pf-item-desc' }, `${t('control.more.custom')}${advancedSummary}`)
+                    : null,
                 ),
               ),
-              h('div', { className: 'pf-hint' }, t('control.modelCheck.hint')),
+              advancedOpen
+                ? h(
+                    'div',
+                    { className: 'pf-more-body' },
+                    h(
+                      'div',
+                      { className: 'pf-row' },
+                      h('span', { className: 'pf-row-label' }, t('control.mode')),
+                      h(
+                        'span',
+                        { className: 'pf-seg' },
+                        [['', t('control.mode.card')], ['llm', t('control.mode.llm')], ['template', t('control.mode.template')]].map(([value, label]) => h(
+                          'button',
+                          {
+                            key: value === '' ? 'card' : value,
+                            type: 'button',
+                            'aria-pressed': (modeOverride ?? '') === value ? 'true' : 'false',
+                            className: (modeOverride ?? '') === value ? 'is-on' : '',
+                            onClick: () => setMode(value === '' ? undefined : value),
+                          },
+                          label,
+                        )),
+                      ),
+                    ),
+                    h('div', { className: 'pf-hint' }, t('control.mode.hint')),
+                    // Only a card that may add behavioural constraints has
+                    // anything to narrow. On a voice-only card both options
+                    // resolve to the same thing, and offering that choice --
+                    // with a paragraph explaining why the other one is refused --
+                    // is a control that cannot do anything.
+                    selected?.fidelity === 'strategy'
+                      ? [
+                          h(
+                            'div',
+                            { className: 'pf-row' },
+                            h('span', { className: 'pf-row-label' }, t('control.fidelity')),
+                            h(
+                              'span',
+                              { className: 'pf-seg' },
+                              [['', t('control.fidelity.card')], ['style', t('control.fidelity.style')]].map(([value, label]) => h(
+                                'button',
+                                {
+                                  key: value === '' ? 'card' : value,
+                                  type: 'button',
+                                  'aria-pressed': (fidelityOverride ?? '') === value ? 'true' : 'false',
+                                  className: (fidelityOverride ?? '') === value ? 'is-on' : '',
+                                  onClick: () => setFidelity(value === '' ? undefined : value),
+                                },
+                                label,
+                              )),
+                            ),
+                          ),
+                          h('div', { className: 'pf-hint' }, t('control.fidelity.hint.strategy')),
+                        ]
+                      : null,
+                  )
+                : null,
               h('div', { className: 'pf-menu-sep' }),
               h(
                 'button',
@@ -1218,6 +1234,23 @@ window.__ModuleLoader__.load({
       const panel = shared.panel
       const inputActions = props.inputActions
       const setDraft = typeof inputActions?.setDraft === 'function' ? inputActions.setDraft : null
+
+      // Seconds since the in-flight rewrite started, ticking once a second.
+      //
+      // Without this the panel shows "改写中" with no sense of progress, so a
+      // rewrite waiting on a slow model reads as FROZEN — the user cannot tell
+      // "working" from "hung", and reports it as stuck. A live counter answers
+      // that for nothing, and makes the host's own timeout (120s) legible
+      // instead of arbitrary.
+      const [tick, setTick] = useState(0)
+      useEffect(() => {
+        if (panel === null || panel.phase !== 'loading') return
+        const started = typeof panel.startedAt === 'number' ? panel.startedAt : Date.now()
+        setTick(Math.round((Date.now() - started) / 1000))
+        const timer = window.setInterval(() => setTick(Math.round((Date.now() - started) / 1000)), 1000)
+        return () => window.clearInterval(timer)
+      }, [panel === null, panel?.phase, panel?.startedAt])
+      const elapsedSeconds = panel !== null && panel.phase === 'loading' ? Math.max(0, tick) : null
       const submit = typeof inputActions?.submit === 'function' ? inputActions.submit : null
       const owned = panel !== null && panel.sessionKey === sessionKey ? panel : null
       // Guards the direct-send effect against firing twice for one result.
@@ -1250,6 +1283,12 @@ window.__ModuleLoader__.load({
       /** Replace the composer content, then optionally submit it. */
       const apply = (text, thenSubmit) => {
         if (setDraft === null) return
+        // Putting the ORIGINAL back has to forget that a rewrite ever happened.
+        // Without this, `lastRewrite` still records `original: A`, so the next
+        // send on the restored draft A matches "already rewritten" and the
+        // plugin silently does nothing — a rewrite you cancelled would be
+        // impossible to retry. Restoring is a rewind, not a reviewed result.
+        if (text === panel.original) lastRewrite.delete(sessionKey)
         setDraft(text)
         if (thenSubmit !== true || submit === null) {
           closePanel()
@@ -1281,6 +1320,7 @@ window.__ModuleLoader__.load({
       )
 
       if (panel.phase === 'loading') {
+        const seconds = elapsedSeconds ?? 0
         return h(
           'div',
           { className: 'pf-panel' },
@@ -1288,7 +1328,17 @@ window.__ModuleLoader__.load({
           h(
             'div',
             { className: 'pf-panel-body' },
-            h('div', { className: 'pf-meta' }, t('panel.loading', { name: panel.cardName ?? panel.cardId })),
+            h(
+              'div',
+              { className: 'pf-meta' },
+              t('panel.loading', { name: panel.cardName ?? panel.cardId }),
+              h('span', { className: 'pf-elapsed', 'aria-live': 'off' }, `${seconds}s`),
+            ),
+            // Past half the host's own timeout, say so instead of leaving the
+            // user to wonder whether anything is still happening.
+            seconds >= 45
+              ? h('div', { className: 'pf-note is-muted', style: { marginTop: '8px' } }, t('panel.slow', { seconds }))
+              : null,
           ),
         )
       }
@@ -1673,9 +1723,11 @@ window.__ModuleLoader__.load({
         // new-card flow; this line is what tells a user it exists, so they do
         // not have to press a button to find out.
         h('div', { className: 'pf-small', style: { margin: '0 0 10px' } }, t('settings.ai.tip')),
-        // The check result, stated once above the list.
+        // The check RESULT is real feedback. The instruction line is not
+        // rendered: the button's tooltip already explains the action, and
+        // repeating it on the page is the same text twice.
         drift === null
-          ? h('div', { className: 'pf-small', style: { marginBottom: '10px' } }, t('settings.check.desc'))
+          ? null
           : drift === undefined
             ? h('div', { className: 'pf-note is-muted', style: { marginBottom: '10px' } }, t('settings.check.busy'))
             : drift.length === 0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-persona-forge",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "给 DeepSeek Harness 用的角色扮演提示词改写插件：在输入框选一张角色卡，草稿就会由 harness 的模型改写成该角色的口吻（跟随当前会话正在使用的模型），然后直接发出或先人工审查。原草稿始终保留。",
   "type": "module",
   "license": "MIT",