@mzzsfy/dsh-turn-notify 0.12.1 → 0.12.2

package/README.md

@@ -152,7 +152,7 @@ turn-notify:
 
 按数据流向分五步:
 
-1. **事件监听**:host 观察 `session/event`(回合结束、ask_user_question 工具调用与 assistant 消息)与 `approval/request` 审批请求(只观察,不拦截)。回合结束原因逐一映射:正常结束归任务完成,错误与手动停止归任务出错,审批拦截归等待审批,输出触顶归达到上限,会话收尾补发的中断归被中断;assistant 消息流经途累积回合小结(tokens 与回答前缀),回合结束随通知消费。
+1. **事件监听**:host 观察 `session/event`(回合结束、ask_user_question 工具调用与 assistant 消息)、`approval/asked` 审批事件(审批通知唯一信号:生产者先 append 后发 waterfall,会话事件流恒先达且不依赖应答方放行)。回合结束原因逐一映射:正常结束归任务完成,错误与手动停止归任务出错,审批拦截归等待审批,输出触顶归达到上限,会话收尾补发的中断归被中断;assistant 消息流经途累积回合小结(tokens 与回答前缀),回合结束随通知消费。
 2. **分类与过滤**:事件归入六类,经子代理会话、子代理相关静默(仅任务完成类)、最短回合时长三道过滤,通过即产生一条通知,并按 kindRoutes 裁定放行通道。
 3. **host 直发通道**:webhook、IM 与宿主桌面通知由 host 进程直接投递(经路由名单过滤),不依赖浏览器;宿主通知按浏览器在场(在途长轮询,来源不限本机)浏览器优先去重,离场补位;同时通知写入内存队列(环形 20 条,60 秒过期,不落盘),路由名单随单元下发。
 4. **浏览器通道**:各窗口以长轮询挂起在该队列上(cursor 续传,事件与配置变更即时唤醒,空闲期单次挂起至多数十秒,失败指数退避重连),localStorage 协调保证同一通知只有一个窗口呈现,再按本机偏好与路由名单分发到声音/系统弹窗/页内提示;等待用户动作的审批与提问超时未处理时补发一轮提醒。
@@ -192,4 +192,10 @@ node scripts/dev-link.mjs dsh-turn-notify   # 仓库根执行:归一 profile 依
 
 ## dsh 版本兼容
 
-三版本(0.1.2-rc.1 / 0.1.5-rc.3 / 0.1.7-rc.1)全部通过:通知开关与播报形态(四 tab 计数、事件开关、音效卡试听、IM 自洽)、激活 live。0.1.6 控件数 13→19 属宿主演进,不构成不兼容。
+- 0.2.0-rc.2:完全适配(装载 + 功能面逐格验证通过)
+- 0.1.7-rc.2:不崩溃底线通过(全家桶 crash-only)
+- 0.1.5-rc.3:未承诺(仅 shell-select 有宿主要求豁免;全家桶在该版本不启动,详见仓库 docs/兼容性测试/逐包测试进度.md)
+
+历史验收(Round 30,2026-09-27):0.1.7-rc.2 完全适配(设置 HTTP 写读闭环 deepMerge 语义 + 「消息通知」分区事件芯片渲染;11/11 判定面);0.1.5-rc.3 曾判不崩溃底线通过(已被 2026-09-30 复测推翻,见上);历史窗口 0.1.2-rc.1 全格通过(四 tab 计数/事件开关/音效卡试听/IM 自洽)。0.1.6 控件数 13→19 属宿主演进,不构成不兼容。
+
+更细的逐包结论见仓库 `docs/兼容性测试/逐包测试进度.md` 与本包 `兼容性测试.md`。

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@mzzsfy/dsh-turn-notify",
   "description": "消息通知:六类回合事件送达声音、系统弹窗、页内提示、webhook、IM 与宿主桌面通知;内置八种合成音色,自定义音效,按事件分类指定;多窗口只响一次",
-  "version": "0.12.1",
+  "version": "0.12.2",
   "type": "module",
   "main": "src/index.js",
   "exports": {

package/src/core.mjs

@@ -87,6 +87,11 @@ const IM_TARGET_ID_PATTERN = /^[A-Za-z0-9._:@-]{1,128}$/
 export const isValidImBotId = (value) => typeof value === 'string' && IM_BOT_ID_PATTERN.test(value)
 
 const TURN_END_KIND = 'turn/end'
+// 审批请求的会话事件类型(DSH SessionEventMap 官方定义:
+//   'approval/asked': { id; toolName; callId?; reason? })。
+// 审批通知唯一信号:生产者(dsh-user-approval request)先 append 后发 waterfall,
+// 会话事件流恒先达且不依赖应答方放行;approval/request waterfall 只读不可靠,不作信号源。
+export const APPROVAL_ASKED_KIND = 'approval/asked'
 
 // turn/end reason.kind 到通知分类的映射;未知 kind(插件可扩展)返回 null。
 const REASON_KIND_TO_CATEGORY = {
@@ -103,6 +108,7 @@ export function mapEventToCategory(type, data) {
     const kind = data && data.reason && data.reason.kind
     return Object.prototype.hasOwnProperty.call(REASON_KIND_TO_CATEGORY, kind) ? REASON_KIND_TO_CATEGORY[kind] : null
   }
+  if (type === APPROVAL_ASKED_KIND) return CATEGORY_APPROVAL
   if (type === 'tool/call' && data && data.name === 'ask_user_question') return CATEGORY_ASK
   return null
 }
@@ -468,14 +474,6 @@ export function readRawBody(req, maxBytes) {
   })
 }
 
-// 审批观察器包装:notify 异步投递,next() 同步立即放行,不阻塞 waterfall。
-export function createApprovalTap(notify, schedule) {
-  return function approvalTap(_req, next) {
-    schedule(() => notify({ category: CATEGORY_APPROVAL }))
-    return next()
-  }
-}
-
 // webhook 直发:未配置跳过;任何失败不抛出(fire-and-forget,不重试)。
 // 返回真实投递结果供测试按钮呈现,真实通知路径以 void 忽略。
 export async function sendWebhook({ url, payload, fetchImpl = fetch }) {

package/src/index.js

@@ -1,4 +1,4 @@
-// dsh-turn-notify Host 半区:回合通知单源决策。观察 session/event 与 approval/request,
+// dsh-turn-notify Host 半区:回合通知单源决策。观察 session/event,
 // 命中分类即发 webhook、写入内存投影;webServer 路由供浏览器半区轮询投影与管理音效。
 // 音效持久化在 ~/.dsh/dsh-turn-notify/sounds/,投影不落盘。
 
@@ -9,6 +9,7 @@ import { homedir } from 'node:os'
 import { join } from 'node:path'
 import z from '@deepseek-ai/schemastery'
 import {
+  APPROVAL_ASKED_KIND,
   AUDIO_EXTS,
   CATEGORIES,
   CATEGORY_ASK,
@@ -21,7 +22,6 @@ import {
   buildWebhookPayload,
   collectAssistantTail,
   collectSessionEvents,
-  createApprovalTap,
   createProjection,
   isSubagent,
   isSubagentWakeTurn,
@@ -446,20 +446,16 @@ export function apply(ctx, config) {
     const category = mapEventToCategory(event.type, event.data)
     if (category === null) return
     const reasonKind = event.type === TURN_END_KIND && event.data.reason ? event.data.reason.kind : null
-    // tool/call 的提问事件与 turn/end 分类共用入口;提问通知不受碎轮过滤
-    const kind = category === CATEGORY_ASK ? TOOL_CALL_KIND : TURN_END_KIND
+    // 即时送达类事件(tool/call 提问、approval/asked 审批)用自身 kind,
+    // 不落入 turn/end 的碎轮时长过滤;其余归类沿用 turn/end。
+    const kind = category === CATEGORY_ASK
+      ? TOOL_CALL_KIND
+      : category === CATEGORY_APPROVAL && event.type === APPROVAL_ASKED_KIND
+        ? APPROVAL_ASKED_KIND
+        : TURN_END_KIND
     notifyUnit({ category, kind, reasonKind, session, ...(endedTurn ?? {}) })
   })
 
-  // 审批 waterfall 观察者:next() 同步放行,通知异步投递
-  ctx.on('approval/request', createApprovalTap(
-    ({ category }) => {
-      // waterfall 请求不携带会话对象,标题与工作区留空,仅分类与时间有效
-      notifyUnit({ category, kind: 'approval/request', reasonKind: category, session: { header: {} } })
-    },
-    (fn) => { fn() },
-  ))
-
   ctx.inject(['settings'], (settingsCtx) => {
     // 方法面守卫:settings 服务缺 register 面(0.1.7 已移除)即走新形态分支;
     // 0.1.7 关闭原生自动设置页,设置面由本包自带分区承担(maintain 同构)

package/test/core.test.mjs

@@ -2,6 +2,7 @@
 import test from 'node:test'
 import assert from 'node:assert/strict'
 import {
+  APPROVAL_ASKED_KIND,
   CATEGORIES,
   CATEGORY_DONE,
   CATEGORY_ERROR,
@@ -41,7 +42,6 @@ import {
   TITLE_MAX_CHARS,
   readRawBody,
   sessionTitle,
-  createApprovalTap,
   sendWebhook,
   TONE_BELL,
   TONE_UP_ARPEGGIO,
@@ -129,6 +129,36 @@ test('非回合结束事件仅 ask_user_question tool/call 命中提问分类',
   assert.equal(mapEventToCategory('turn/start', {}), null)
 })
 
+test('approval/asked 会话事件命中审批分类(宿主 waterfall 被 UI 独占时的唯一信号)', () => {
+  assert.equal(
+    mapEventToCategory('approval/asked', { id: 'a-1', toolName: 'bash', reason: 'escalate' }),
+    CATEGORY_APPROVAL,
+  )
+  // 同一命名空间下的其他事件不误判为审批
+  assert.equal(mapEventToCategory('approval/decided', { id: 'a-1', outcome: 'allowed-once' }), null)
+  assert.equal(mapEventToCategory('approval/policy', { policy: 'ask' }), null)
+})
+
+test('审批通知即时送达,不受最短回合时长过滤', () => {
+  const settings = { enabled: { approval: true }, rootsOnly: true, minTurnDurationMs: 5000 }
+  // 即使时长为 0(< 阈值),approval/asked 仍应放行
+  assert.equal(shouldNotify({
+    category: CATEGORY_APPROVAL,
+    kind: APPROVAL_ASKED_KIND,
+    durationMs: 0,
+    settings,
+    header: {},
+  }), true)
+  // 对照:turn/end 类仍受碎轮过滤
+  assert.equal(shouldNotify({
+    category: CATEGORY_DONE,
+    kind: 'turn/end',
+    durationMs: 0,
+    settings,
+    header: {},
+  }), false)
+})
+
 test('rootsOnly 按子代理会话过滤', () => {
   const subByOrigin = { origin: 'subagent' }
   const subByDepth = { delegationDepth: 1 }
@@ -164,7 +194,7 @@ test('碎轮过滤仅作用于 turn/end 类,时长恰等边界放行', () => {
     category: CATEGORY_DONE, kind: 'turn/end', durationMs: MIN_TURN_MS, settings, header: top,
   }), true)
   assert.equal(shouldNotify({
-    category: CATEGORY_APPROVAL, kind: 'approval/request', durationMs: null, settings, header: top,
+    category: CATEGORY_APPROVAL, kind: APPROVAL_ASKED_KIND, durationMs: null, settings, header: top,
   }), true)
   assert.equal(shouldNotify({
     category: CATEGORY_ASK, kind: 'tool/call', durationMs: null, settings, header: top,
@@ -547,20 +577,6 @@ test('标题提取:首个用户文本并截断', () => {
   assert.equal(sessionTitle([]), null)
 })
 
-test('审批观察器 next() 立即放行且只通知一次', () => {
-  const notified = []
-  const scheduled = []
-  const tap = createApprovalTap((unit) => { notified.push(unit) }, (fn) => { scheduled.push(fn) })
-  let nextCalled = 0
-  const returned = tap({ toolName: 'bash' }, () => { nextCalled += 1; return 'next-result' })
-  assert.equal(nextCalled, 1)
-  assert.equal(returned, 'next-result')
-  assert.equal(notified.length, 0)
-  for (const fn of scheduled) fn()
-  assert.equal(notified.length, 1)
-  assert.equal(notified[0].category, CATEGORY_APPROVAL)
-})
-
 test('webhook 发送吞错不抛出', async () => {
   const calls = []
   const failing = async () => { calls.push(1); throw new Error('unreachable') }

package/test/event-wiring.test.mjs

@@ -462,13 +462,13 @@ test('Given 非提问的 tool/call When 事件到达 Then 不产生通知', asyn
   assert.equal(units.length, 0)
 })
 
-test('Given approval/request 观察者 When 触发 Then 投影审批单元且 next 同步放行', async () => {
+test('Given approval/asked 会话事件 When 到达 Then 投影恰好一条审批单元且 approval/request 无观察者', async () => {
   const { ctx, routes, handlers } = makeCtx()
   apply(ctx)
-  const tap = handlers.get('approval/request')
-  assert.equal(typeof tap, 'function')
-  const returned = tap({}, () => 'next-value')
-  assert.equal(returned, 'next-value')
+  // 回归锁定:审批通知单源为 session/event,waterfall 观察者不得回潮(双路重复送达)
+  assert.equal(handlers.get('approval/request'), undefined)
+  const onEvent = handlers.get('session/event')
+  onEvent(MAIN, { type: 'approval/asked', data: { id: 'a-1', toolName: 'bash' } })
   const units = await projectionUnits(routes)
   assert.equal(units.filter((unit) => unit.category === 'approval').length, 1)
 })
@@ -508,8 +508,8 @@ test('Given 会话无事件日志可读 When 通知投递 Then 不抛并回落
     sessionTitle: { get: () => { throw new Error('no event log') } },
   })
   apply(ctx)
-  const tap = handlers.get('approval/request')
-  tap({}, () => 'next')
+  const onEvent = handlers.get('session/event')
+  onEvent(MAIN, { type: 'approval/asked', data: { id: 'a-1', toolName: 'bash' } })
   const units = await projectionUnits(routes)
   const approval = units.find((unit) => unit.category === 'approval')
   assert.equal(approval.session, null)