dsh-notify-say 0.4.54 → 0.4.55

package/README.md

@@ -330,7 +330,7 @@ npm run test:route    # 需要 --import loader 的接线测试
 npm run test:audio-wire
 ```
 
-**410 条测试通过。** 绝大多数是纯逻辑测试，克隆仓库后 `npm test` 即可跑。
+**414 条测试通过。** 绝大多数是纯逻辑测试，克隆仓库后 `npm test` 即可跑。
 
 少数用例需要真实的 DSH 宿主（用于验证 Cordis 上下文行为）。
 找不到宿主时它们会**自动 skip** —— 注意 skip 不等于通过。

package/lib/client.js

@@ -266,6 +266,56 @@ window.__ModuleLoader__.load({
         opacity: 0.78, fontSize: '12.5px',
       },
 
+      // ── 试听按钮组（v0.4.55）────────────────────────────────────────
+      //
+      // 用户反馈：420px 下五个按钮挤成 4+1，最后一个「拦截」孤零零一行。
+      //
+      // 改法：整组**独占一行**（flex-basis 100%），用 grid 自动定列数。
+      //   · 420px：内容宽约 364 → 排得下 5 列（每列约 63px）→ 一行五个
+      //   · 更窄时自动降列 → 自然换行，行距列距一致（都是 6px）
+      //   · maxWidth 340：720px 下不让按钮被拉成大块头
+      //
+      // 不缩小按钮：minmax 下限 56px 比原来的约 51px 还宽一点。
+      pvGroup: {
+        display: 'grid',
+        gridTemplateColumns: 'repeat(auto-fit, minmax(56px, 1fr))',
+        gap: '6px',
+        flex: '1 1 100%',
+        maxWidth: '340px',
+        marginTop: '2px',
+      },
+
+      // ── 中性提示框（v0.4.55）────────────────────────────────────────
+      //
+      // 用于「不是错误、但用户应当知道」的状态（如弹窗与声音都关掉）。
+      // 刻意**不用红/黄**：这不是故障，是用户自己的选择，只需告知结果。
+      noticeBox: {
+        margin: '8px 0 2px',
+        padding: '8px 10px',
+        borderLeft: '3px solid rgba(128,128,128,.5)',
+        background: 'rgba(128,128,128,.08)',
+        fontSize: '12.5px',
+        lineHeight: '1.6',
+        opacity: 0.92,
+      },
+
+      // 可点击的折叠标题（v0.4.55）：整行可点，箭头固定在最右。
+      foldHead: {
+        display: 'flex',
+        alignItems: 'center',
+        justifyContent: 'space-between',
+        gap: '8px',
+        cursor: 'pointer',
+        userSelect: 'none',
+        margin: '16px 0 2px',
+        padding: '2px 4px',
+        marginLeft: '-4px',
+        borderRadius: '4px',
+      },
+      // 悬停/键盘焦点时的反馈（同一套，键盘用户也要看得见）。
+      foldHeadActive: { background: 'rgba(128,128,128,.14)' },
+      foldArrow: { flex: '0 0 auto', opacity: 0.6, fontSize: '11px' },
+
       // 说明文字：弱化但保持可读对比度（不要低到看不清）。
       fieldHint: { fontSize: '12.5px', opacity: 0.72, lineHeight: '1.6', flex: '1 1 100%', margin: '0 0 7px' },
 
@@ -416,6 +466,54 @@ window.__ModuleLoader__.load({
       'shikoku-metang', 'ameharu-hau', 'namiine-ritsu', 'shirakami-kotaro',
     ]
 
+    /**
+     * 音色友好显示名（v0.4.55）。
+     *
+     * 用户反馈：下拉里显示 `zundamon-normal` 是**开发者内部标识**，
+     * 普通用户看不懂。改为「角色名 · 风格名」；原 key 只在 title 悬停提示
+     * 与诊断日志里出现。
+     *
+     * ⚠️ 只影响**显示**：保存提交的仍是原 key（voiceName / kindVoiceName），
+     * 配置格式、默认值、后端一字未动。
+     *
+     * ⚠️ 名字与 `tools/build-voice-packs.ps1` 的 $Approved.Name 同源，
+     * 由测试 cross-check 防漂移（见 test/voice-label.test.mjs）。
+     */
+    var VOICE_LABELS = {
+      'zundamon-normal': 'ずんだもん · ノーマル',
+      'zundamon-amama': 'ずんだもん · あまあま',
+      'ankomon': 'あんこもん',
+      'goki-nuigurumi': '後鬼 · ぬいぐるみ',
+      'kyushu-sora': '九州そら',
+      'manbetsu-hanamaru-genki': '満別花丸 · 元気',
+      'meimei-meria': '冥鳴ひまり',
+      'shikoku-metang': '四国めたん',
+      'ameharu-hau': '雨晴はう',
+      'namiine-ritsu': '波音リツ',
+      'shirakami-kotaro': '白上虎太郎',
+    }
+
+    /**
+     * key → 友好名。未知 key **原样返回**（不猜、不美化）——
+     * 用户手填的自定义 key 显示成它自己，比显示一个错的别名安全。
+     */
+    function voiceLabel(key) {
+      if (!key) return ''
+      return VOICE_LABELS[key] || key
+    }
+
+    /**
+     * 语音包元数据里的官方名（`ずんだもん（ノーマル）`）
+     * → 统一成「角色名 · 风格名」，与 VOICE_LABELS 风格一致。
+     *
+     * 否则「语音包列表」用括号、「音色下拉」用点，同一件事两种叫法。
+     */
+    function prettyPackName(name) {
+      var s = String(name || '')
+      var m = /^(.+?)（(.+?)）$/.exec(s)
+      return m ? (m[1] + ' · ' + m[2]) : s
+    }
+
     /** 事件开关定义（键 → 中文名）。 */
     // 事件开关：[键, 名称, 一句话说明]。
     //
@@ -493,6 +591,13 @@ window.__ModuleLoader__.load({
       var avset = avst[1]
       var advSec = avst[0]
 
+      // 高级子组标题的悬停/焦点高亮（v0.4.55）。
+      // 独立 state：它纯粹是瞬时交互反馈，与配置无关，
+      // 也不该因为保存配置（25 处 set 重建主 state）被重置。
+      var hvst = React.useState({ id: "" })
+      var hvset = hvst[1]
+      var advHover = hvst[0].id || ""
+
       var st = React.useState({
         loaded: false,
         values: {
@@ -935,6 +1040,17 @@ window.__ModuleLoader__.load({
         h('span', { style: S.hint }, '关掉只弹 Toast'),
       ))
 
+      // ⚠️ 两个都关掉 = 完全没有任何提醒（v0.4.55）。
+      //
+      // 用户指出这个歧义：不禁止这样设置（那是用户的选择），
+      // 但必须**如实告知结果**，否则用户以为"我配过了怎么没反应"。
+      // 用中性样式，不用红/黄 —— 这不是故障。
+      if (v.showToast === false && v.voice === false) {
+        rows.push(h('div', { key: 'all-silent', style: S.noticeBox },
+          '弹窗与声音都已关闭 —— 插件不会产生任何可见或可听的提醒。'
+            + '（提醒规则里的事件仍会照常触发，只是没有任何表现。）'))
+      }
+
       // ── 音源选择（v0.4.4）─────────────────────────────────────────────────
       //
       // 这是本插件最核心的可配置项：用户想听系统语音、想听语音包、想要系统音效、
@@ -1083,7 +1199,7 @@ window.__ModuleLoader__.load({
             ['done', '完成'], ['fail', '失败'], ['question', '提问'],
             ['approval', '批准'], ['blocked', '拦截'],
           ]
-          return h('div', { style: { display: 'flex', gap: '6px', flexWrap: 'wrap', flex: '1' } },
+                      return h('div', { style: S.pvGroup },
             KINDS.map(function (pair) {
               var isBusy = pvState.kind === pair[0]
               return h('button', {
@@ -1175,7 +1291,8 @@ window.__ModuleLoader__.load({
               // 条款要**下载前**可见 —— 不能只靠 zip 里的 TERMS.md（那要下载后才看得到）。
               return h('div', { key: 'vp-item-' + p.key, style: { display: 'flex', flexDirection: 'column', gap: '2px', padding: '3px 0' } },
                 h('div', { style: { display: 'flex', alignItems: 'center', gap: '8px' } },
-                  h('span', { style: { flex: '1' } }, (p.name || p.key) + (mb ? '（' + mb + '）' : '')),
+                  h('span', { style: { flex: '1' }, title: p.key },
+              (prettyPackName(p.name) || p.key) + (mb ? '（' + mb + '）' : '')),
                   p.installed ? h('span', { style: { color: '#2a7' } }, '已安装') : null,
                   h('button', {
                     type: 'button',
@@ -1242,7 +1359,10 @@ window.__ModuleLoader__.load({
             value: v.voiceName || 'zundamon-normal',
             disabled: s.busy || !fixedMode,
             onChange: function (e) { apply({ voiceName: e.target.value }, '音色已更新') },
-          }, keyOpts.map(function (k) { return h('option', { key: k, value: k }, k) })),
+          }, keyOpts.map(function (k) {
+            // value 仍是原 key（保存用它），只改显示文本；原 key 放 title
+            return h('option', { key: k, value: k, title: k }, voiceLabel(k))
+          })),
           fixedMode ? null : h('span', { style: S.hint }, '随机模式下不生效'),
         ))
       }
@@ -1647,13 +1767,33 @@ window.__ModuleLoader__.load({
         ADV_SECTIONS.forEach(function (sec) {
           var id = sec[0]
           var isOpen = advSec[id] === true
+          // 整行可点 + 箭头固定最右 + 悬停/焦点反馈 + 键盘可操作（v0.4.55）。
+          //
+          // 用户反馈：原来箭头跟在文字后面（`高级设置 ▸`），不明显能点。
+          // 现在整行是点击区、箭头靠右、Enter/Space 也能展开 ——
+          // 键盘用户与鼠标用户看到同一套反馈（foldHeadActive）。
+          var headActive = advHover === id
           rows.push(h('div', {
             key: 'adv-sec-' + id,
-            style: Object.assign({}, S.groupLabel, {
-              cursor: 'pointer', userSelect: 'none', margin: '16px 0 2px',
-            }),
+            role: 'button',
+            tabIndex: 0,
+            'aria-expanded': isOpen ? 'true' : 'false',
+            style: Object.assign({}, S.foldHead, headActive ? S.foldHeadActive : null),
             onClick: function () { toggleAdvSec(id) },
-          }, sec[1] + (isOpen ? '  ▾' : '  ▸')))
+            onMouseEnter: function () { hvset({ id: id }) },
+            onMouseLeave: function () { hvset({ id: advHover === id ? '' : advHover }) },
+            onFocus: function () { hvset({ id: id }) },
+            onBlur: function () { hvset({ id: advHover === id ? '' : advHover }) },
+            onKeyDown: function (e) {
+              if (e.key === 'Enter' || e.key === ' ') {
+                e.preventDefault()
+                toggleAdvSec(id)
+              }
+            },
+          },
+            h('span', { style: { fontWeight: 600, fontSize: '13px' } }, sec[1]),
+            h('span', { style: S.foldArrow }, isOpen ? '▾' : '▸'),
+          ))
           rows.push(h('div', { key: 'adv-sec-desc-' + id, style: S.subHint }, sec[2]))
 
           if (isOpen) {
@@ -1871,7 +2011,9 @@ window.__ModuleLoader__.load({
                     return function (e) { var o = {}; o[k] = e.target.value; apply(o, '已更新') }
                   })(nKey),
                 }, [h('option', { key: '', value: '' }, '跟随全局')].concat(
-                  kOpts2.map(function (x) { return h('option', { key: x, value: x }, x) }),
+                  kOpts2.map(function (x) {
+                    return h('option', { key: x, value: x, title: x }, voiceLabel(x))
+                  }),
                 )),
                 mCur === 'random' ? h('span', { style: S.hint }, '随机模式下不生效') : null,
               ))

package/package.json

@@ -1,6 +1,6 @@
 {
     "name":  "dsh-notify-say",
-    "version":  "0.4.54",
+    "version":  "0.4.55",
     "description":  "Desktop notifications for DSH Desktop that actually speak: Windows TTS out of the box (zero download), VOICEVOX voice packs optional, a selectable system-sound fallback, and an idle gate that waits for running subagents before saying the turn is done. Includes a settings-page GUI (sound source, per-event overrides, jump-back-to-session, cooldowns) and a self hot-reload for development.",
     "license":  "MIT",
     "type":  "module",

package/test/voice-label.test.mjs

@@ -0,0 +1,123 @@
+/**
+ * 音色友好显示名的一致性测试（v0.4.55）。
+ *
+ * ## 为什么要这些测试
+ *
+ * 用户反馈：下拉里显示 `zundamon-normal` 是开发者内部标识，应显示「角色名 · 风格名」。
+ *
+ * 这个改动本身很小，但它引入了**第二份音色名字**：
+ *   ① `tools/build-voice-packs.ps1` 的 `$Approved[].Name`（打包用，权威）
+ *   ② `lib/client.js` 的 `VOICE_LABELS`（显示用）
+ *
+ * 两份名字一旦漂移，用户会在「语音包列表」看到一个名字、
+ * 在「音色下拉」看到另一个名字 —— 同一件事两种叫法。
+ * 所以必须用测试把它们钉在一起。
+ *
+ * ## 关键的"不能改"约束
+ *
+ * 只改显示。**保存提交的仍是原 key** —— 如果哪天有人把 option 的 value
+ * 改成友好名，用户的配置就会写入一个后端不认识的字符串。
+ * Q3 专门盯这一条。
+ */
+import { test } from 'node:test';
+import assert from 'node:assert/strict';
+import { readFileSync, existsSync } from 'node:fs';
+import { resolve, dirname, join } from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const ROOT = resolve(dirname(fileURLToPath(import.meta.url)), '..');
+const CLIENT = readFileSync(join(ROOT, 'lib', 'client.js'), 'utf8');
+const BUILD_PS1 = join(ROOT, 'tools', 'build-voice-packs.ps1');
+const hasBuild = existsSync(BUILD_PS1);
+
+/** 解析 client.js 里的 VOICE_KEYS。 */
+function voiceKeys() {
+  const m = CLIENT.match(/var VOICE_KEYS = \[([\s\S]*?)\n    \]/);
+  assert.ok(m, '应能定位 VOICE_KEYS');
+  return [...m[1].matchAll(/'([a-z0-9-]+)'/g)].map((x) => x[1]);
+}
+
+/** 解析 client.js 里的 VOICE_LABELS。 */
+function voiceLabels() {
+  const m = CLIENT.match(/var VOICE_LABELS = \{([\s\S]*?)\n    \}/);
+  assert.ok(m, '应能定位 VOICE_LABELS');
+  const out = {};
+  for (const mm of m[1].matchAll(/'([a-z0-9-]+)':\s*'([^']+)'/g)) out[mm[1]] = mm[2];
+  return out;
+}
+
+/** 解析打包脚本的 $Approved（key → 官方 Name）。 */
+function approvedNames() {
+  const src = readFileSync(BUILD_PS1, 'utf8');
+  const out = {};
+  for (const m of src.matchAll(/'([a-z0-9-]+)'\s*=\s*@\{\s*Name\s*=\s*'([^']+)'/g)) {
+    out[m[1]] = m[2];
+  }
+  return out;
+}
+
+/** `ずんだもん（ノーマル）` → `ずんだもん`；`四国めたん` → `四国めたん`。 */
+function characterOf(official) {
+  const m = /^(.+?)（(.+?)）$/.exec(official);
+  return m ? m[1] : official;
+}
+
+// ── Q1: VOICE_KEYS 每一项都有友好名 ─────────────────────────────────────
+test('Q1: 每个可选音色都有友好显示名', () => {
+  const keys = voiceKeys();
+  const labels = voiceLabels();
+  assert.ok(keys.length >= 8, `音色数量偏少（${keys.length}）`);
+
+  const missing = keys.filter((k) => !labels[k]);
+  assert.deepEqual(missing, [],
+    '这些音色还显示着英文 key（用户看不懂），应补进 VOICE_LABELS：\n' + missing.join('\n'));
+});
+
+// ── Q2: 友好名与打包脚本的官方名同源 ────────────────────────────────────
+test('Q2: 友好名的角色部分与 build-voice-packs.ps1 一致', () => {
+  if (!hasBuild) {
+    // 发布包里没有维护者工具，跳过（与 D6/D7 同款处理）
+    return;
+  }
+  const labels = voiceLabels();
+  const approved = approvedNames();
+  assert.ok(Object.keys(approved).length >= 8, '应能解析出打包脚本的音色表');
+
+  const problems = [];
+  for (const [key, official] of Object.entries(approved)) {
+    const label = labels[key];
+    if (!label) continue;                       // Q1 管缺失
+    const wantChar = characterOf(official);
+    const gotChar = label.split(' · ')[0];
+    if (gotChar !== wantChar) {
+      problems.push(`${key}: 显示「${label}」，但打包脚本的角色名是「${wantChar}」`);
+    }
+  }
+  assert.deepEqual(problems, [],
+    '两份音色名字漂移了 —— 用户会在不同位置看到不同叫法：\n' + problems.join('\n'));
+});
+
+// ── Q3: 保存的仍是原 key，不是友好名 ───────────────────────────────────
+test('Q3: 音色下拉的 value 仍是原 key（只改显示）', () => {
+  // 如果 value 被改成友好名，用户的配置会写入后端不认识的字符串。
+  assert.match(CLIENT, /h\('option', \{ key: k, value: k, title: k \}, voiceLabel\(k\)\)/,
+    '固定音色的 option 必须 value=原key、显示=友好名');
+  assert.match(CLIENT, /h\('option', \{ key: x, value: x, title: x \}, voiceLabel\(x\)\)/,
+    '按事件覆盖的音色 option 同理');
+  // 原 key 要能在悬停提示里看到（用户排查问题时需要）
+  assert.match(CLIENT, /title: k/, '原 key 应放在 title 里（悬停可见）');
+});
+
+// ── Q4: option 显示文本不再直接是 key ──────────────────────────────────
+test('Q4: 音色 option 的显示文本走 voiceLabel，不是裸 key', () => {
+  // 反面断言：不能再出现 `h('option', { key: k, value: k }, k)` 这种裸 key 显示
+  assert.doesNotMatch(CLIENT, /h\('option', \{ key: k, value: k \}, k\)/,
+    '固定音色不应再直接显示英文 key');
+  assert.doesNotMatch(CLIENT, /h\('option', \{ key: x, value: x \}, x\)/,
+    '按事件覆盖的音色不应再直接显示英文 key');
+
+  // 语音包列表也要用统一风格（括号 → 点）
+  assert.match(CLIENT, /prettyPackName\(p\.name\)/,
+    '语音包列表的官方名应统一成「角色 · 风格」风格');
+  assert.match(CLIENT, /function prettyPackName/, '应有 prettyPackName 工具函数');
+});