dsh-notify-say 0.4.55 → 0.4.56

package/README.md

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

package/lib/client.js

@@ -194,7 +194,8 @@ window.__ModuleLoader__.load({
       ok: { color: '#1e8449', fontSize: '12px', marginTop: '6px' },
       foot: { opacity: 0.65, marginTop: '12px', fontSize: '12.5px', lineHeight: '1.6' },
       // 事件开关：勾选框 + 可换行的说明文字
-      toggleRow: { display: 'flex', alignItems: 'flex-start', gap: '8px', margin: '8px 0' },
+      // 行间距略放大（v0.4.56）：420px 下说明文字密集，靠间距而不是缩字号。
+      toggleRow: { display: 'flex', alignItems: 'flex-start', gap: '8px', margin: '10px 0' },
       toggleBox: { marginTop: '3px', flex: '0 0 auto' },
       toggleText: { flex: '1', lineHeight: '1.5' },
       toggleLabel: { fontWeight: 500 },
@@ -598,6 +599,17 @@ window.__ModuleLoader__.load({
       var hvset = hvst[1]
       var advHover = hvst[0].id || ""
 
+      // 声音设置区的展开状态（v0.4.56）。
+      //
+      // `null` = 跟随默认：声音提醒开着就展开、关了就收起。
+      // 用户手动点过之后才固定成 true/false（不覆盖用户的显式选择）。
+      //
+      // ⚠️ 收起**只是隐藏**，配置一律保留；试听也在里面，
+      // 用户想看随时能展开 —— 不禁用任何操作。
+      var sndst = React.useState({ open: null })
+      var sndset = sndst[1]
+      var sndPref = sndst[0].open
+
       var st = React.useState({
         loaded: false,
         values: {
@@ -1087,6 +1099,31 @@ window.__ModuleLoader__.load({
       // ── 二、声音设置 ────────────────────────────────────────────────
       rows.push(section('sound', '声音设置', '音量、音色与试听；语音包可按需下载。'))
 
+      // 声音提醒关掉时，把音频控件收起来（v0.4.56）。
+      //
+      // 用户指出：两个提醒都关了，下面还铺着整套 TTS 设置，视觉噪音大。
+      // 但**不能删**配置 —— 用户可能只是暂时静音，回头还要用。
+      var voiceOff = v.voice === false
+      var sndOpen = sndPref === null ? !voiceOff : sndPref === true
+      if (voiceOff) {
+        rows.push(h('div', { key: 'snd-off-head', style: S.foldHead,
+          onClick: function () { sndset({ open: !sndOpen }) },
+          role: 'button', tabIndex: 0,
+          'aria-expanded': sndOpen ? 'true' : 'false',
+          onKeyDown: function (e) {
+            if (e.key === 'Enter' || e.key === ' ') {
+              e.preventDefault()
+              sndset({ open: !sndOpen })
+            }
+          },
+        },
+          h('span', { style: { fontSize: '13px', opacity: 0.9 } },
+            '声音提醒已关闭，原有配置已保留'),
+          h('span', { style: S.foldArrow }, sndOpen ? '▾' : '▸'),
+        ))
+      }
+
+      if (sndOpen) {
       pushIf(curSource !== 'none', h('div', { key: 'vol', style: S.row },
         h('span', { style: S.label }, '音量'),
         h('input', {
@@ -1312,10 +1349,21 @@ window.__ModuleLoader__.load({
                     },
                   }, p.installed ? '卸载' : (p.downloadable ? '下载' : '不可用')),
                 ),
-                // 署名与条款：小字，不抢视觉但随时可查
+                // 署名与条款：小字，不抢视觉但随时可查。
+                //
+                // ⚠️ 占位提示的判据是「**能下载却没有条款**」（v0.4.56）：
+                // 那才是真的数据问题，值得提示。
+                //
+                // 原来判据是「没有 termsUrl 就提示」，于是**离线**时
+                // （索引取不到 → 已安装的包也没有 termsUrl）界面会显示
+                // 「条款链接待补」，看起来像插件发布缺字段 —— 虚报。
+                //
+                // 许可安全本身不受影响：下载按钮要求 p.downloadable，
+                // 而 downloadable 只可能来自带 terms 的远程索引。
                 h('div', { style: S.subHint },
                   (p.credit ? '署名：' + p.credit : '')
-                  + (p.termsUrl ? '' : '　（条款链接待补）'),
+                  // 内联判断（这里是表达式上下文，不能写 var）
+                  + ((p.downloadable && !p.termsUrl) ? '　（条款链接待补）' : ''),
                   p.termsUrl
                     ? h('a', {
                       href: p.termsUrl,
@@ -1541,6 +1589,8 @@ window.__ModuleLoader__.load({
         notifyOnAborted: false, notifyOnApproval: true, notifyOnQuestion: true,
         subagents: false, idleGate: true,
       }
+      }   // ← 声音设置区展开/收起（v0.4.56）
+
       // ── 三、提醒规则 ────────────────────────────────────────────────
       rows.push(section('rules', '提醒规则', '哪些事件要提醒你。默认只提醒需要你关注的那些。'))
       // 按「什么时候会用到」分三组渲染（v0.4.54）。
@@ -1803,7 +1853,8 @@ window.__ModuleLoader__.load({
               var KIND_ROWS = [['Done', '任务完成'], ['Fail', '任务失败'],
                 ['Question', '向你提问'], ['Approval', '待你批准'], ['Blocked', '被拦截']]
               var FOLLOW = '__follow__'
-          rows.push(h('div', { key: 'adv-group', style: S.groupLabel }, '按事件覆盖声音来源'))
+          // 内层不再重复标题 —— 二级分组已经叫「按事件定制声音」（v0.4.56）。
+        // 只保留说明，直接接五个事件选择器。
           rows.push(h('div', { key: 'advhint', style: S.subHint },
             '默认跟随上面的「声音来源」。只有你改过的那几行才会单独生效。'))
           KIND_ROWS.forEach(function (pair) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name":  "dsh-notify-say",
-    "version":  "0.4.55",
+    "version":  "0.4.56",
     "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/ui-final.test.mjs

@@ -0,0 +1,220 @@
+/**
+ * 定稿前的收尾行为测试（v0.4.56）。
+ *
+ * 这两条来自用户评审：
+ *   ① 静音时音频设置应收起，但**不能禁用试听**
+ *   ② 「条款链接待补」不能虚报
+ */
+import { test } from 'node:test';
+import assert from 'node:assert/strict';
+import { readFileSync } 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');
+
+function fakeReact() {
+  const createElement = (type, props, ...children) => ({
+    $$typeof: Symbol.for('react.element'),
+    type,
+    props: { ...(props || {}), children },
+  });
+  const hooks = { slots: [], index: 0, eff: 0, ranEff: new Set(), rerender: null };
+  const useState = (init) => {
+    const i = hooks.index++;
+    if (hooks.slots[i] === undefined) {
+      hooks.slots[i] = { value: typeof init === 'function' ? init() : init };
+    }
+    const slot = hooks.slots[i];
+    const set = (updater) => {
+      slot.value = typeof updater === 'function' ? updater(slot.value) : updater;
+      if (typeof hooks.rerender === 'function') hooks.rerender();
+    };
+    return [slot.value, set];
+  };
+  // 只跑一次（模拟 [] 依赖）；不调 cleanup（否则 alive=false，配置永远不合并）
+  const useEffect = (fn) => {
+    const i = hooks.eff++;
+    if (hooks.ranEff.has(i)) return;
+    hooks.ranEff.add(i);
+    try { fn(); } catch { /* ignore */ }
+  };
+  const resetIndex = () => { hooks.index = 0; hooks.eff = 0; };
+  return { createElement, useState, useEffect, __hooks: hooks, __resetIndex: resetIndex };
+}
+
+const FULL = {
+  voice: true, voiceMode: 'fixed', voiceName: 'zundamon-normal', volume: 80,
+  notifyOnComplete: true, notifyOnError: true, notifyOnBlocked: true,
+  notifyOnAborted: false, notifyOnApproval: true, notifyOnQuestion: true,
+  subagents: false, idleGate: true,
+  audioSource: 'tts', showToast: true, ttsVoice: '', ttsRate: 0, devReload: false,
+  logLevel: 'info', voiceGain: 2.0, normalizeCustomAudio: false,
+  jumpWhenBackground: true, jumpWhenForeground: false,
+  debounceSeconds: 60, globalCooldownSeconds: 0,
+  audioSourceByKind: {}, ttsTexts: {}, kindSystemSound: {}, kindVoiceName: {},
+  customFiles: {},
+};
+
+function texts(node, out) {
+  const acc = out || [];
+  if (node === null || node === undefined || node === false) return acc;
+  if (typeof node === 'string' || typeof node === 'number') { acc.push(String(node)); return acc; }
+  if (Array.isArray(node)) { node.forEach((n) => texts(n, acc)); return acc; }
+  texts(node.props && node.props.children, acc);
+  return acc;
+}
+
+function elements(node, out) {
+  const acc = out || [];
+  if (!node || typeof node !== 'object') return acc;
+  if (Array.isArray(node)) { node.forEach((n) => elements(n, acc)); return acc; }
+  if (node.type) acc.push(node);
+  elements(node.props && node.props.children, acc);
+  return acc;
+}
+
+/** 按形状找主 state 槽（组件有多个 useState，不能写死索引）。 */
+function findMainSlot(card) {
+  const s = card.r.__hooks.slots.find(
+    (x) => x && x.value && typeof x.value === 'object'
+      && 'values' in x.value && 'loaded' in x.value,
+  );
+  if (!s) throw new Error('找不到主 state 槽');
+  return s;
+}
+
+/** 按形状找「声音设置展开状态」槽（{ open: ... }）。 */
+function findSoundSlot(card) {
+  const s = card.r.__hooks.slots.find(
+    (x) => x && x.value && typeof x.value === 'object'
+      && 'open' in x.value && Object.keys(x.value).length === 1,
+  );
+  if (!s) throw new Error('找不到声音区 state 槽');
+  return s;
+}
+
+async function loadCard() {
+  let captured = null;
+  new Function('window', 'document', CLIENT)(
+    { __ModuleLoader__: { load(def) { captured = def; } }, addEventListener() {}, removeEventListener() {} },
+    undefined,
+  );
+  if (!captured) throw new Error('client.js 没有注册');
+
+  const r = fakeReact();
+  const mod = captured.factory((id) => {
+    if (id === 'react') return r;
+    throw new Error('client 不应 require: ' + id);
+  });
+
+  let component = null;
+  mod.apply({
+    get(id) {
+      if (id === 'slots') return { inject(_n, fn) { fn(); }, register(_m, c) { component = c; } };
+      return undefined;
+    },
+  });
+  if (!component) throw new Error('没有注册设置卡');
+
+  globalThis.fetch = (url) => Promise.resolve({
+    ok: true, status: 200,
+    json: () => Promise.resolve(
+      String(url).includes('version')
+        ? {
+          ok: true, running: '0.4.56', installed: '0.4.56', latest: '0.4.56',
+          hasUpdate: false, needsRestart: false, npmLatest: '0.4.56',
+          githubLatest: '0.4.56', installMode: 'managed', changelogUrl: 'x',
+        }
+        : { ok: true, values: FULL, packs: [] },
+    ),
+  });
+
+  const render = () => { r.__resetIndex(); return component({}); };
+  render();
+  r.__hooks.rerender = render;
+  for (let i = 0; i < 8; i++) await Promise.resolve();
+  return { r, render };
+}
+
+const has = (text, s) => text.includes(s);
+
+/** 设 state 后渲染。 */
+function renderWith(card, main, sound) {
+  findMainSlot(card).value = { ...findMainSlot(card).value, ...main };
+  if (sound !== undefined) findSoundSlot(card).value = { open: sound };
+  const tree = card.render();
+  return { text: texts(tree).join(' '), els: elements(tree) };
+}
+
+// ── N6: 静音时收起声音设置，但不禁用试听 ────────────────────────────────
+test('N6: 声音提醒关闭时收起音频设置，试听仍可用', async () => {
+  const card = await loadCard();
+
+  // 声音开着 → 正常展开；手动把 open 置 null（跟随默认）
+  {
+    const { text } = renderWith(card,
+      { values: { ...FULL, audioSource: 'tts', voice: true } }, null);
+    assert.ok(has(text, '朗读声音'), '声音开着时应正常显示音频控件');
+    assert.ok(!has(text, '声音提醒已关闭'), '声音开着时不该出现收起提示');
+  }
+
+  // 声音关掉 → 收起，但说明"配置还在"
+  {
+    const { text } = renderWith(card,
+      { values: { ...FULL, audioSource: 'tts', voice: false } }, null);
+    assert.ok(has(text, '声音提醒已关闭，原有配置已保留'),
+      '静音时应说明「配置还在」，否则用户以为被清掉了');
+    assert.ok(!has(text, '朗读声音'), '静音时应收起音频控件');
+  }
+
+  // 用户手动展开 → 控件回来，且试听**没有**被禁用
+  {
+    const { text, els } = renderWith(card,
+      { values: { ...FULL, audioSource: 'tts', voice: false } }, true);
+    assert.ok(has(text, '朗读声音'), '手动展开后应能看到音频控件');
+
+    const pvButtons = els.filter((e) => {
+      if (e.type !== 'button') return false;
+      const kids = e.props.children || [];
+      return ['完成', '失败', '提问', '批准', '拦截'].includes(kids[0]);
+    });
+    assert.ok(pvButtons.length >= 5, `五个试听按钮都应存在（实际 ${pvButtons.length}）`);
+    assert.ok(pvButtons.every((b) => !b.props.disabled),
+      '静音**不得禁用试听** —— 试听是测试配置的行为，与通知开关无关');
+  }
+});
+
+// ── N7: 条款占位不虚报 ──────────────────────────────────────────────────
+test('N7: 条款占位只在「能下载却没条款」时出现', () => {
+  // 用户评审时看到「条款链接待补」，要求核实是不是发布缺字段。
+  //
+  // 核实结论：真实 packs.json 的 13 个音色都有 terms，服务端也转成 termsUrl
+  // （R7 用真实请求钉住）。但**离线**时索引取不到，已安装的包没有 termsUrl ——
+  // 原来会显示占位，看起来像发布缺字段。
+  //
+  // 现在判据收窄为「能下载 && 没条款」。
+  assert.match(
+    CLIENT,
+    /\(p\.downloadable && !p\.termsUrl\)/,
+    '占位提示必须同时要求 downloadable —— 否则离线时会虚报',
+  );
+  // 反面断言：不能退回「没有 termsUrl 就报」
+  assert.doesNotMatch(
+    CLIENT,
+    /\+ \(p\.termsUrl \? '' : '　（条款链接待补）'\)/,
+    '不应退回旧的「没 termsUrl 就提示」判据',
+  );
+});
+
+// ── N8: 高级区不再有重复标题 ────────────────────────────────────────────
+test('N8: 高级区四个子组没有重复的内部标题', async () => {
+  const card = await loadCard();
+  const { text } = renderWith(card, { advOpen: true }, undefined);
+  // 子组标题「按事件定制声音」只应出现一次（内部不再重复「按事件覆盖声音来源」）
+  const occurrences = text.split('按事件定制声音').length - 1;
+  assert.equal(occurrences, 1, `「按事件定制声音」应只出现一次（实际 ${occurrences}）`);
+  assert.doesNotMatch(CLIENT, /'按事件覆盖声音来源'/,
+    '内部标题应已删除 —— 与二级分组标题重复');
+});

package/test/voice-pack-route.test.mjs

@@ -182,3 +182,49 @@ test('R7: 安装索引里没有的音色返回 404', async () => {
       `应是 404 或 502，实际 ${res.statusCode}`);
   });
 });
+
+// ── R8: 索引里的 terms → termsUrl（前端读的是这个字段名）─────────────────
+test('R8: 路由把 packs.json 的 terms 映射成前端要的 termsUrl', async () => {
+  // ⚠️ 用户评审时看到界面显示「条款链接待补」，要求核实是不是发布缺字段。
+  //
+  // 核实结论：真实 packs.json 的 13 个音色**都有** terms，
+  // 但字段名是 `terms`，而前端读的是 `termsUrl` ——
+  // 转名这一步在 voice-pack-route.js 里（`termsUrl: p.terms`）。
+  //
+  // 这步一旦被删，界面会对**所有**音色显示「条款链接待补」。
+  // 用真实请求 + 打桩 fetch 把这条跨文件契约钉住。
+  await withDataDir(async () => {
+    const realFetch = globalThis.fetch;
+    const FAKE_TERMS = 'https://example.invalid/voice-terms';
+    globalThis.fetch = async () => ({
+      ok: true,
+      status: 200,
+      headers: { get: () => null },
+      json: async () => ({
+        packs: [{
+          key: 'zundamon-normal',
+          name: 'ずんだもん（ノーマル）',
+          file: 'zundamon-normal.zip',
+          bytes: 100,
+          sha256: 'a'.repeat(64),
+          wavs: 1,
+          credit: 'VOICEVOX:ずんだもん',
+          terms: FAKE_TERMS,
+        }],
+      }),
+    });
+    try {
+      const route = createVoicePackRoute({ pluginRoot: '.', logger: { info() {}, warn() {} } });
+      const res = await call(route, 'GET', 'index');
+      assert.equal(res.statusCode, 200);
+      const doc = res.json();
+      assert.equal(doc.indexOk, true, '索引应可用（fetch 已打桩）');
+      const p = (doc.packs || []).find((x) => x.key === 'zundamon-normal');
+      assert.ok(p, '应返回该音色');
+      assert.equal(p.termsUrl, FAKE_TERMS,
+        'packs.json 里叫 terms，前端读的是 termsUrl —— 名字不对会让界面显示「条款链接待补」');
+    } finally {
+      globalThis.fetch = realFetch;
+    }
+  });
+});