dsh-notify-say 0.4.58 → 0.4.59

package/CHANGELOG.md

@@ -3,6 +3,10 @@
 > 本文件由 `tools/release-build.ps1` 从开发提交自动生成，**请勿手工编辑**。
 > 只列面向使用者的变更（新增功能与修复），不含内部维护提交。
 
+## v0.4.59 — 2026-10-18
+
+- fix(notify): v0.4.59 —— 修两个 P1：弹窗开关失效 + 分叉会话漏报
+
 ## v0.4.58 — 2026-10-18
 
 - feat(ui): v0.4.58 —— 自重载入口并入「版本与更新」

package/README.md

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

package/lib/index.js

@@ -542,7 +542,25 @@ function questionSummary(toolName, rawArgs) {
 function isSubagent(session) {
   try {
     const h = session && session.header;
-    return !!(h && (h.parentSession || h.origin === 'subagent' || (h.delegationDepth || 0) > 0));
+    if (!h) return false;
+    // ⚠️ v0.4.59 修正（外部审阅指出的规则矛盾）：
+    //
+    //   原来只要有 `parentSession` 就判为子代理。但本文件 `isDescendantOf()`
+    //   的注释已经写明：`sessions.fork` 产生的**普通分叉会话** header 也携带
+    //   parentSession（指向源会话），而 origin 为空、是用户自己的顶层会话。
+    //
+    //   后果很实际：默认 `subagents: false` 时，普通 fork 会话的完成
+    //   通知会被当成"子代理的"过滤掉 —— 用户会发现"我这个会话怎么不响"。
+    //
+    //   现在与 isDescendantOf 用同一套语义：只有 origin === 'subagent'
+    //   或 delegationDepth > 0 才算子代理。
+    //
+    //   为什么不是简单删掉 parentSession：真正的子代理关系仍需识别，
+    //   只是**不能仅凭** parentSession 认定 —— 它的语义是「派生自谁」，
+    //   不是「谁的子代理」。
+    if (h.origin === 'subagent') return true;
+    if ((h.delegationDepth || 0) > 0) return true;
+    return false;
   } catch {
     return false;
   }
@@ -997,6 +1015,28 @@ export function apply(ctx, config) {
 
     // 只有 "none" 是完全不出声；其余都走声音路径
     const wantSound = cfg.voice && source !== 'none';
+    // ── 视觉通道（v0.4.59）────────────────────────────────────────────
+    //
+    // ⚠️ 外部审阅指出的真 bug：原来只有 TTS 分支看 `cfg.showToast`，
+    // 语音包 / 系统提示音 / 自定义音频一律让 ps1 弹窗 —— 于是
+    // **关掉「显示通知弹窗」后仍会弹 Toast**。
+    //
+    // 更糟的是两个都关的情况：旧代码照样 spawn ps1（带 -NoSound），
+    // ps1 在没有 -SoundOnly 时就会弹窗 —— 与设置页刚加的
+    // 「弹窗与声音都已关闭」提示直接矛盾。
+    //
+    // 现在把两个通道彻底分开：
+    //   wantToast=false → 传 -SoundOnly（只出声，不弹窗）
+    //   wantSound=false → 传 -NoSound（只弹窗，不出声）
+    //   两个都 false   → **不启动任何进程**
+    //
+    // 默认值按 `!== false` 判：配置缺字段时按"开着"处理，
+    // 不会因为 undefined 把用户的提醒静默掉。
+    const wantToast = cfg.showToast !== false;
+    if (!wantToast && !wantSound) {
+      appendNotifyLog('[skip] 弹窗与声音都已关闭，不启动提醒进程');
+      return;
+    }
     // 本事件对应的事件类（done/fail/question/approval/blocked），
     // 用于查"按事件覆盖"的各种参数（v0.4.9）。
     const aKind = audioKindOf(psKind) || 'done';
@@ -1036,7 +1076,7 @@ export function apply(ctx, config) {
       ];
       if (vVoice) ttsArgs.push('-VoiceName', vVoice);
       // TTS 自己会出声，所以 notify.ps1 只负责弹窗（-NoSound）
-      if (cfg.showToast) spawnNotify(notifyArgs(psKind, title, body, true));
+      if (wantToast) spawnNotify(notifyArgs(psKind, title, body, true));
       spawnTts(ttsArgs);
       return;
     }
@@ -1055,7 +1095,7 @@ export function apply(ctx, config) {
     // **不传 -Voice/-Mode**，并且 ps1 侧见到 -SystemSound 就跳过语音池
     // （否则 -Kind fail 会先选中 fail 池的 wav，永远走不到系统音效）。
     if (wantSound && source === 'system') {
-      const a = baseNotifyArgs(psKind, title, body, false);
+      const a = baseNotifyArgs(psKind, title, body, false, !wantToast);
       const explicitTone = kindOr(cfg.kindSystemSound, aKind, '');
       a.push('-SystemSound', explicitTone || 'auto');
       spawnNotify(a);
@@ -1071,7 +1111,7 @@ export function apply(ctx, config) {
       const kind = audioKindOf(psKind);
       const file = resolveCustomFile(kind, cfg);
       if (file) {
-        const a = baseNotifyArgs(psKind, title, body, false);
+        const a = baseNotifyArgs(psKind, title, body, false, !wantToast);
         a.push('-CustomFile', file);
         // v0.4.16：自定义音频的"标准化"是**可选**的。
         //
@@ -1085,23 +1125,28 @@ export function apply(ctx, config) {
       }
       // 没配文件：不要静默无声，退到系统提示音（audio-source 的回退链已算过，
       // 但这里再兜一次，防止 effectiveSource 的 hasCustomFile 判断与读取不一致）
-      const a2 = baseNotifyArgs(psKind, title, body, false);
+      const a2 = baseNotifyArgs(psKind, title, body, false, !wantToast);
       a2.push('-SystemSound', 'auto');
       spawnNotify(a2);
       return;
     }
 
     // ── 其余音源（bundled）走 dsh-notify.ps1 ──────────────────────────────
-    spawnNotify(notifyArgs(psKind, title, body, !wantSound, source));
+    // 第 5 个参数是 soundOnly（v0.4.59）。原来这里传的是 source，
+    // 而 notifyArgs 只认 4 个参数 —— 那个多传的值一直被静默忽略。
+    spawnNotify(notifyArgs(psKind, title, body, !wantSound, !wantToast));
   }
 
   /** 只有基础参数的 notify 调用（供 system 音源拼装）。 */
-  function baseNotifyArgs(psKind, title, body, noSound) {
+  function baseNotifyArgs(psKind, title, body, noSound, soundOnly) {
     const args = [
       '-NoProfile', '-ExecutionPolicy', 'Bypass', '-WindowStyle', 'Hidden', '-File',
       SCRIPT_PATH, '-Kind', psKind, '-Title', title, '-Body', body,
     ];
     if (noSound) args.push('-NoSound');
+    // 只出声不弹窗（v0.4.59）。与 -NoSound 互斥，调用方保证。
+    if (soundOnly) args.push('-SoundOnly');
+
   // v0.4.29：把「用户目录;插件目录」传给 ps1，让它两个区域都查
   const roots = voiceRootsArg();
   if (roots) args.push('-VoiceRoot', roots);
@@ -1112,12 +1157,14 @@ export function apply(ctx, config) {
   }
 
   /** 构造 dsh-notify.ps1 的参数。noSound=true 表示只弹窗不出声。 */
-  function notifyArgs(psKind, title, body, noSound) {
+  function notifyArgs(psKind, title, body, noSound, soundOnly) {
     const args = [
       '-NoProfile', '-ExecutionPolicy', 'Bypass', '-WindowStyle', 'Hidden', '-File',
       SCRIPT_PATH, '-Kind', psKind, '-Title', title, '-Body', body,
     ];
     if (noSound) args.push('-NoSound');
+    // 只出声不弹窗（v0.4.59）。与 -NoSound 互斥，调用方保证。
+    if (soundOnly) args.push('-SoundOnly');
   // v0.4.29：把「用户目录;插件目录」传给 ps1，让它两个区域都查
   const roots = voiceRootsArg();
   if (roots) args.push('-VoiceRoot', roots);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name":  "dsh-notify-say",
-    "version":  "0.4.58",
+    "version":  "0.4.59",
     "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/toast-sound-channels.test.mjs

@@ -0,0 +1,226 @@
+/**
+ * 视觉通道与听觉通道的独立性（v0.4.59）。
+ *
+ * ## 为什么这个文件必须存在
+ *
+ * 外部审阅在 v0.4.56 上指出一个真 bug：
+ *
+ *   `showToast=false` **只在 TTS 分支被处理**；语音包 / 系统提示音 /
+ *   自定义音频一律让 `dsh-notify.ps1` 自己弹窗 —— 于是"关掉弹窗"对
+ *   这三种音源根本没生效。
+ *
+ * 而当时的 `audio-wiring.test.mjs` 的 A3 用例**只测了 TTS**，
+ * 所以这个 bug 一直绿灯。我改完代码后测试也全绿 —— 再次印证：
+ * **没覆盖到的组合等于没测。**
+ *
+ * 这个文件按"五种音源 × 两个开关的全部组合"做参数级断言。
+ *
+ * ## 关键契约
+ *
+ * | showToast | voice | 期望 |
+ * |-----------|-------|------|
+ * | true      | true  | 弹窗 + 出声（notify 不带任何抑制开关） |
+ * | true      | false | 只弹窗（notify 带 `-NoSound`） |
+ * | false     | true  | 只出声（notify 带 `-SoundOnly`） |
+ * | false     | false | **不启动任何进程** |
+ *
+ * 注意 `-NoSound` 与 `-SoundOnly` **互斥**，同时出现说明逻辑错了。
+ *
+ * 需带 loader 跑：`npm run test:toast`
+ */
+import { test } from 'node:test';
+import assert from 'node:assert/strict';
+import { mkdtempSync, rmSync } from 'node:fs';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+
+const HOME = mkdtempSync(join(tmpdir(), 'notify-channels-'));
+process.env.DSH_HOME = HOME;
+process.on('exit', () => { try { rmSync(HOME, { recursive: true, force: true }); } catch { /* ignore */ } });
+
+const { apply } = await import('../lib/index.js');
+
+function stubActive() {
+  return globalThis.__STUB_CHILD_PROCESS__ === true;
+}
+
+function skipReason() {
+  if (!process.env.DSH_ROUTE_LOADER && !stubActive()) return '需 npm run test:toast（带 --import loader）';
+  if (!stubActive()) return 'loader stub 未生效';
+  return false;
+}
+
+function mkCtx() {
+  const handlers = new Map();
+  return {
+    on(name, fn) { handlers.set(name, fn); return { dispose() { handlers.delete(name); } }; },
+    effect(fn) { try { const d = fn(); return { dispose() { try { if (typeof d === 'function') d(); } catch { /* ignore */ } } }; } catch { return { dispose() { /* ignore */ } }; } },
+    inject(_names, fn) { try { fn({ webServer: null }); } catch { /* ignore */ } return { dispose() { /* ignore */ } }; },
+    get() { return undefined; },
+    logger: { info() { /* silent */ }, warn() { /* silent */ }, error() { /* silent */ } },
+    emit(name, ...args) { const h = handlers.get(name); if (h) h(...args); },
+  };
+}
+
+/** 触发一次"完成"，返回 spawn 的 argv 列表。 */
+function runDone(pluginCfg) {
+  globalThis.__SPAWN_CALLS = [];
+  const ctx = mkCtx();
+  apply(ctx, { idleGate: false, subagents: false, ...pluginCfg });
+
+  const agent = { id: 'agent-1', session: { id: 'sess-1' }, status: 'running' };
+  ctx.emit('session/event', agent.session, { type: 'turn/end', data: { turn: 1, reason: { kind: 'completed' } } });
+  agent.status = 'idle';
+  ctx.emit('agent/status', { agent, status: 'running' });
+  ctx.emit('agent/status', { agent, status: 'idle' });
+
+  return (globalThis.__SPAWN_CALLS || []).map((c) => (c && (c.args || c.argv)) || []);
+}
+
+function scriptOf(argv) {
+  const arr = Array.isArray(argv) ? argv : [];
+  const i = arr.indexOf('-File');
+  return i >= 0 && arr[i + 1] ? String(arr[i + 1]).replace(/\\/g, '/').split('/').pop() : '';
+}
+
+/** 只留下 dsh-notify.ps1 的调用（负责 Toast/系统音/自定义音频）。 */
+function notifyCalls(calls) {
+  return calls.filter((a) => scriptOf(a) === 'dsh-notify.ps1');
+}
+
+/** 只留下 tts-speak.ps1 的调用。 */
+function ttsCalls(calls) {
+  return calls.filter((a) => scriptOf(a) === 'tts-speak.ps1');
+}
+
+const SOURCES = ['tts', 'bundled', 'system', 'custom', 'none'];
+
+// ── M1: 两个都关 → 一个进程都不该启动 ──────────────────────────────────
+test('M1: 弹窗与声音都关时，不启动任何进程（五种音源）', { skip: skipReason() }, () => {
+  for (const source of SOURCES) {
+    const calls = runDone({ audioSource: source, voice: false, showToast: false });
+    assert.equal(calls.length, 0,
+      `source=${source} 且 voice/showToast 都关时不该 spawn 任何东西，实际 ${JSON.stringify(calls.map(scriptOf))}`);
+  }
+});
+
+// ── M2: 只出声 → notify 必须带 -SoundOnly，且不得带 -NoSound ────────────
+test('M2: showToast=false 且 voice=true 时，notify 带 -SoundOnly', { skip: skipReason() }, () => {
+  // 这条就是审阅指出的 bug：语音包/系统音/自定义音频原来不带这个开关，
+  // 于是"关掉弹窗"形同虚设。
+  for (const source of ['bundled', 'system', 'custom']) {
+    const calls = runDone({ audioSource: source, voice: true, showToast: false });
+    const n = notifyCalls(calls);
+    assert.equal(n.length, 1, `source=${source} 应恰好调一次 notify.ps1，实际 ${n.length}`);
+    assert.ok(n[0].includes('-SoundOnly'),
+      `source=${source} 应传 -SoundOnly（只出声不弹窗），实际参数: ${JSON.stringify(n[0])}`);
+    assert.ok(!n[0].includes('-NoSound'),
+      `source=${source} 不应同时传 -NoSound（与 -SoundOnly 互斥）`);
+  }
+});
+
+// ── M3: 只弹窗 → notify 必须带 -NoSound，且不得带 -SoundOnly ────────────
+test('M3: showToast=true 且 voice=false 时，notify 带 -NoSound', { skip: skipReason() }, () => {
+  for (const source of ['bundled', 'system', 'custom', 'none']) {
+    const calls = runDone({ audioSource: source, voice: false, showToast: true });
+    const n = notifyCalls(calls);
+    assert.equal(n.length, 1, `source=${source} 应恰好调一次 notify.ps1，实际 ${n.length}`);
+    assert.ok(n[0].includes('-NoSound'),
+      `source=${source} 应传 -NoSound（只弹窗不出声），实际: ${JSON.stringify(n[0])}`);
+    assert.ok(!n[0].includes('-SoundOnly'),
+      `source=${source} 不应同时传 -SoundOnly`);
+  }
+});
+
+// ── M4: 两个都开 → 两个抑制开关都不该出现 ──────────────────────────────
+test('M4: showToast=true 且 voice=true 时，两个抑制开关都不出现', { skip: skipReason() }, () => {
+  for (const source of ['bundled', 'system', 'custom']) {
+    const calls = runDone({ audioSource: source, voice: true, showToast: true });
+    const n = notifyCalls(calls);
+    assert.equal(n.length, 1, `source=${source} 应调一次 notify.ps1`);
+    assert.ok(!n[0].includes('-NoSound'), `source=${source} 不该抑制声音`);
+    assert.ok(!n[0].includes('-SoundOnly'), `source=${source} 不该抑制弹窗`);
+  }
+});
+
+// ── M5: TTS 走两个进程，各管一个通道 ───────────────────────────────────
+test('M5: TTS 由两个进程分别负责声音与弹窗', { skip: skipReason() }, () => {
+  // 两个都开：tts-speak.ps1 出声、notify.ps1 弹窗（带 -NoSound 避免重复出声）
+  {
+    const calls = runDone({ audioSource: 'tts', voice: true, showToast: true });
+    assert.equal(ttsCalls(calls).length, 1, '应调一次 tts-speak.ps1');
+    const n = notifyCalls(calls);
+    assert.equal(n.length, 1, '应调一次 notify.ps1 负责弹窗');
+    assert.ok(n[0].includes('-NoSound'), 'TTS 自己出声，notify 应带 -NoSound');
+  }
+  // 只出声：不调 notify
+  {
+    const calls = runDone({ audioSource: 'tts', voice: true, showToast: false });
+    assert.equal(ttsCalls(calls).length, 1, '应仍调 TTS');
+    assert.equal(notifyCalls(calls).length, 0, 'showToast=false 时不该调 notify.ps1');
+  }
+  // 只弹窗：不调 TTS
+  {
+    const calls = runDone({ audioSource: 'tts', voice: false, showToast: true });
+    assert.equal(ttsCalls(calls).length, 0, 'voice=false 时不该调 TTS');
+    const n = notifyCalls(calls);
+    assert.equal(n.length, 1, '应调一次 notify.ps1 弹窗');
+    assert.ok(n[0].includes('-NoSound'), '不该出声');
+  }
+});
+
+// ── M6: none 音源：不出声但可弹窗 ──────────────────────────────────────
+test('M6: audioSource=none 只弹窗、不出声', { skip: skipReason() }, () => {
+  const calls = runDone({ audioSource: 'none', voice: true, showToast: true });
+  assert.equal(ttsCalls(calls).length, 0, 'none 不该调 TTS');
+  const n = notifyCalls(calls);
+  assert.equal(n.length, 1, 'none 仍应弹窗');
+  assert.ok(n[0].includes('-NoSound'), 'none 不该出声');
+});
+
+// ── M7: 普通 fork 会话不能被当成子代理过滤掉 ────────────────────────────
+test('M7: 普通 fork（有 parentSession、无 origin）仍会提醒', { skip: skipReason() }, () => {
+  // ⚠️ 外部审阅指出的规则矛盾：isSubagent() 原来只看 parentSession。
+  // 但 sessions.fork 产生的**普通分叉会话** header 也带 parentSession
+  // （origin 为空、是用户自己的顶层会话），于是默认 subagents:false 时
+  // 它的完成通知会被当成"子代理的"过滤掉 —— 用户觉得"我这个会话怎么不响"。
+  globalThis.__SPAWN_CALLS = [];
+  const ctx = mkCtx();
+  apply(ctx, { idleGate: false, subagents: false, audioSource: 'tts', voice: true, showToast: true });
+
+  // fork 会话：parentSession 有值，origin 为空
+  const forked = {
+    id: 'agent-fork',
+    session: { id: 'sess-fork', header: { parentSession: 'root-1' } },
+    status: 'running',
+  };
+  ctx.emit('session/event', forked.session, { type: 'turn/end', data: { turn: 1, reason: { kind: 'completed' } } });
+  forked.status = 'idle';
+  ctx.emit('agent/status', { agent: forked, status: 'running' });
+  ctx.emit('agent/status', { agent: forked, status: 'idle' });
+
+  const calls = (globalThis.__SPAWN_CALLS || []).map((c) => (c && (c.args || c.argv)) || []);
+  assert.ok(calls.length > 0,
+    '普通 fork 会话的完成**应该**提醒；被当成子代理过滤掉了（isSubagent 误判）');
+});
+
+test('M8: 真子代理（origin=subagent）在 subagents=false 时仍被过滤', { skip: skipReason() }, () => {
+  // 负例的另一半：不能为了修 fork 误判，把真子代理也放进来。
+  globalThis.__SPAWN_CALLS = [];
+  const ctx = mkCtx();
+  apply(ctx, { idleGate: false, subagents: false, audioSource: 'tts', voice: true, showToast: true });
+
+  const sub = {
+    id: 'agent-sub',
+    session: { id: 'sess-sub', header: { origin: 'subagent', parentSession: 'root-1' } },
+    status: 'running',
+  };
+  ctx.emit('session/event', sub.session, { type: 'turn/end', data: { turn: 1, reason: { kind: 'completed' } } });
+  sub.status = 'idle';
+  ctx.emit('agent/status', { agent: sub, status: 'running' });
+  ctx.emit('agent/status', { agent: sub, status: 'idle' });
+
+  const calls = (globalThis.__SPAWN_CALLS || []).map((c) => (c && (c.args || c.argv)) || []);
+  assert.equal(calls.length, 0,
+    'subagents=false 时真子代理不该提醒（过滤规则不能因为修 fork 而失效）');
+});