dsh-activity-pane 0.10.0 → 0.10.1

package/.dsh-plugin/client.js

@@ -2236,6 +2236,32 @@ function reconcileTurnStats(current, records, { closeOpenTurn = false, forceFres
 	return { ...state, watermarkSeq, watermarkTime };
 }
 
+/**
+ * 会话级登记串行化（R-01-020/AC-02）：宿主存储域的 `KvTable.put` 先异步落盘、落盘完成
+ * 后才更新内存快照，因此同一会话的两个边界事件 handler 并发执行时，后到事件会在先到
+ * 事件落盘前读到旧状态，其效果被当作「无效果」永久丢弃（实测：`ask_user_question` 的
+ * `tool/call` 与 11ms 后的 `tool/result` 交错，等待区间悬挂到 `turn/end` 强制结算，
+ * 总耗时系统性缺记整段运行过程）。返回 `(id, run) => tail`：同一会话前一个 run 完成
+ * （含其 put 的内存生效）后才执行下一个 run，不同会话互不阻塞。run 内部须自行捕获
+ * 错误、不得让拒绝外泄（调用方可安全丢弃返回的 tail）；tail 供调用方等待与测试断言。
+ * 链空闲即清理，不随会话数增长。
+ */
+function createSessionEventSerializer() {
+	const tails = new Map();
+	return (id, run) => {
+		const prev = tails.get(id) ?? Promise.resolve();
+		const tail = prev.then(run, run);
+		tails.set(id, tail);
+		tail.then(
+			() => {
+				if (tails.get(id) === tail) tails.delete(id);
+			},
+			() => {},
+		);
+		return tail;
+	};
+}
+
 /**
  * 构建历史区条目：当前非活动的**主会话**，按最后活动时间从新到旧返回完整候选集合。
  * 子代理是临时工作单元，不入历史区；归档、空会话、完成/错误提醒与委托周期中的会话

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-activity-pane",
-  "version": "0.10.0",
+  "version": "0.10.1",
   "packageManager": "pnpm@11.5.0",
   "engines": {
     "node": ">=20"

package/scripts/check.mjs

@@ -63,9 +63,12 @@ import {
 	lastTurnBusyFromEvents,
 	lastTurnDuration,
 	applyTurnEventToStats,
+	createSessionEventSerializer,
 	reconcileTurnStats,
 	normalizeBusyMs,
 	totalBusyDisplayMs,
+	turnStatsEqual,
+	turnStatsFrom,
 	pendingText,
 	progressHalfLifeSec,
 	progressOf,
@@ -1231,6 +1234,39 @@ assert.equal(fmtElapsedMs(-1), "", "负时长归一为空");
 	}
 	assert.equal(stats.busyMs, 1_000, "R-01-020/AC-04 水位检查下重复推送不重复计数");
 }
+// R-01-020/AC-02：实时登记的会话级串行化——宿主 KvTable.put 先落盘后更新内存快照，
+// 并发到达的边界事件若各自读-改-写，后到事件会在先到事件落盘前读到旧状态，其效果被
+// 当作「无效果」永久丢弃（实测：提问 tool/call 与 11ms 后的 tool/result 结算丢失，
+// 等待区间悬挂到 turn/end 强制结算，总耗时缺记整段运行过程）。串行化后即便全部 handler
+// 同步注册、put 延迟落盘，最终记账也必须与顺序重放一致。
+{
+	const events = [
+		{ type: "turn/start", seq: 1, time: 1_000 },
+		{ type: "tool/call", seq: 2, time: 2_000, data: { callId: "c1", name: "ask_user_question" } },
+		{ type: "tool/result", seq: 3, time: 2_011, data: { message: { source: { kind: "tool", callId: "c1" } } } },
+		{ type: "turn/end", seq: 4, time: 5_000, data: { reason: { kind: "completed" } } },
+	];
+	let sequential = { busyMs: null, openTurnStart: null, openWaitStart: null, openWaitKind: null, openWaitId: null, waitedMs: null };
+	for (const event of events) sequential = applyTurnEventToStats(sequential, event);
+	assert.equal(sequential.busyMs, 4_000 - 11, "R-01-020/AC-02 顺序重放基线：回合 4s 扣提问等待 11ms");
+	// 全部事件同步注册（与 session/event 同步观察者的派发序一致）；模拟宿主 KvTable：
+	// put 落盘（异步延迟）完成后才更新内存快照。
+	const serialize = createSessionEventSerializer();
+	const memory = new Map();
+	await Promise.all(
+		events.map((event) =>
+			serialize("session-under-test", async () => {
+				const current = turnStatsFrom(memory.get("session-under-test") ?? null);
+				const next = applyTurnEventToStats(current, event);
+				if (turnStatsEqual(next, current)) return;
+				await new Promise((resolve) => setTimeout(resolve, 5));
+				memory.set("session-under-test", next);
+			}),
+		),
+	);
+	assert.equal(memory.get("session-under-test")?.busyMs, 4_000 - 11, "R-01-020/AC-02 会话级串行化下并发登记与顺序重放一致");
+	assert.equal(memory.get("session-under-test")?.openTurnStart, null, "R-01-020/AC-02 串行化登记后回合闭合无残留起点");
+}
 // R-01-020/AC-01
 // R-01-020/AC-03：开放回合实时增量与起点不可得降级。
 assert.equal(totalBusyDisplayMs({ busyMs: 3_500, openTurnStart: 30_000, now: 32_000 }), 5_500, "累计加开放回合实时已耗时");

package/src/core.mjs

@@ -2230,6 +2230,32 @@ export function reconcileTurnStats(current, records, { closeOpenTurn = false, fo
 	return { ...state, watermarkSeq, watermarkTime };
 }
 
+/**
+ * 会话级登记串行化（R-01-020/AC-02）：宿主存储域的 `KvTable.put` 先异步落盘、落盘完成
+ * 后才更新内存快照，因此同一会话的两个边界事件 handler 并发执行时，后到事件会在先到
+ * 事件落盘前读到旧状态，其效果被当作「无效果」永久丢弃（实测：`ask_user_question` 的
+ * `tool/call` 与 11ms 后的 `tool/result` 交错，等待区间悬挂到 `turn/end` 强制结算，
+ * 总耗时系统性缺记整段运行过程）。返回 `(id, run) => tail`：同一会话前一个 run 完成
+ * （含其 put 的内存生效）后才执行下一个 run，不同会话互不阻塞。run 内部须自行捕获
+ * 错误、不得让拒绝外泄（调用方可安全丢弃返回的 tail）；tail 供调用方等待与测试断言。
+ * 链空闲即清理，不随会话数增长。
+ */
+export function createSessionEventSerializer() {
+	const tails = new Map();
+	return (id, run) => {
+		const prev = tails.get(id) ?? Promise.resolve();
+		const tail = prev.then(run, run);
+		tails.set(id, tail);
+		tail.then(
+			() => {
+				if (tails.get(id) === tail) tails.delete(id);
+			},
+			() => {},
+		);
+		return tail;
+	};
+}
+
 /**
  * 构建历史区条目：当前非活动的**主会话**，按最后活动时间从新到旧返回完整候选集合。
  * 子代理是临时工作单元，不入历史区；归档、空会话、完成/错误提醒与委托周期中的会话

package/src/host.mjs

@@ -26,7 +26,7 @@
 
 import { defineDomain, domainTable } from '@deepseek-ai/dsh-storage-domain'
 import { z } from 'zod'
-import { applyTurnEventToStats, isBusyBoundaryEvent, reconcileTurnStats, turnStatsEqual, turnStatsFrom, truncateErrorNote } from './core.mjs'
+import { applyTurnEventToStats, createSessionEventSerializer, isBusyBoundaryEvent, reconcileTurnStats, turnStatsEqual, turnStatsFrom, truncateErrorNote } from './core.mjs'
 
 export const name = 'dsh-activity-pane'
 export const inject = ['storageDomain', 'webServer', 'sessionQuery', 'connection']
@@ -266,52 +266,58 @@ export function apply(ctx) {
 	// tool/call）不落盘、不广播；主/子统一登记，过滤由客户端判定。
 	// 表内无记录的会话先懒回填存量回合再应用实时事件——否则首个实时事件会以 null 建立记录、
 	// 存量回合永远失去补齐机会。
-	ctx.on('session/event', async (session, event) => {
+	// 读-改-写经 createSessionEventSerializer 按会话串行（put 先落盘后更新内存，并发
+	// 读-改-写会丢先到事件效果，机制见 core.mjs 同名 JSDoc）；注册同步进行保持派发序，
+	// run 内 catch-all 保证链不因单事件失败中断。
+	const serializeBusyEvent = createSessionEventSerializer()
+	ctx.on('session/event', (session, event) => {
 		const type = event?.type
 		if (!isBusyBoundaryEvent(type)) return
 		const seq = Number(event.seq)
 		if (!Number.isFinite(seq)) return
 		const id = String(session?.id ?? '')
 		if (id === '') return
-		try {
-			await domainReady
-			if (!turnStats.get(id)) {
-				await ensureTurnStatsFresh(id)
-			} else {
-				// 回填在途时排队等待其写入完成，再按水位应用实时事件，避免回填快照
-				// 覆盖（回退）实时事件的效果或造成缺口丢失（R-01-020/AC-04）。
-				const inflight = backfills.get(id)
-				if (inflight) await inflight
-			}
-			let current = turnStats.get(id)
-			if (Number.isFinite(current?.watermarkSeq) && seq <= current.watermarkSeq) {
-				// 重编检测（R-01-020/AC-04）：稳定 seq 空间内低水位事件的时刻不可能比水位处
-				// 事件更新；时刻反而更新说明事件 seq 空间被重编（如 dsh 0.1.5 V3 迁移），
-				// 持久化水位已失效，增量口径不再封死后续事件。无 watermarkTime 的存量记录
-				//（本字段引入前写入，其水位未经重编检测校验）同样不可信——两者统一丢弃旧
-				// 记账强制全量重放自愈；重放若仍覆盖不到本事件（日志写入滞后），落回下方
-				// 正常增量路径应用。
+		serializeBusyEvent(id, async () => {
+			try {
+				await domainReady
+				if (!turnStats.get(id)) {
+					await ensureTurnStatsFresh(id)
+				} else {
+					// 回填在途时排队等待其写入完成，再按水位应用实时事件，避免回填快照
+					// 覆盖（回退）实时事件的效果或造成缺口丢失（R-01-020/AC-04）。
+					const inflight = backfills.get(id)
+					if (inflight) await inflight
+				}
+				let current = turnStats.get(id)
+				if (Number.isFinite(current?.watermarkSeq) && seq <= current.watermarkSeq) {
+					// 重编检测（R-01-020/AC-04）：稳定 seq 空间内低水位事件的时刻不可能比水位处
+					// 事件更新；时刻反而更新说明事件 seq 空间被重编（如 dsh 0.1.5 V3 迁移），
+					// 持久化水位已失效，增量口径不再封死后续事件。无 watermarkTime 的存量记录
+					//（本字段引入前写入，其水位未经重编检测校验）同样不可信——两者统一丢弃旧
+					// 记账强制全量重放自愈；重放若仍覆盖不到本事件（日志写入滞后），落回下方
+					// 正常增量路径应用。
+					const time = Number(event.time)
+					if (!Number.isFinite(time)) return
+					if (Number.isFinite(current.watermarkTime) && time <= current.watermarkTime) return
+					await ensureTurnStatsFresh(id, { forceFresh: true })
+					current = turnStats.get(id)
+					if (Number.isFinite(current?.watermarkSeq) && seq <= current.watermarkSeq) return
+				}
+				const next = applyTurnEventToStats(turnStatsFrom(current), event)
+				if (turnStatsEqual(next, current)) {
+					// 无效果事件（非提问工具调用、未配对的结算事件等）：不落盘不广播，
+					// 水位不前移——重放路径对同类事件同样不改变记账状态，口径一致。
+					return
+				}
 				const time = Number(event.time)
-				if (!Number.isFinite(time)) return
-				if (Number.isFinite(current.watermarkTime) && time <= current.watermarkTime) return
-				await ensureTurnStatsFresh(id, { forceFresh: true })
-				current = turnStats.get(id)
-				if (Number.isFinite(current?.watermarkSeq) && seq <= current.watermarkSeq) return
-			}
-			const next = applyTurnEventToStats(turnStatsFrom(current), event)
-			if (turnStatsEqual(next, current)) {
-				// 无效果事件（非提问工具调用、未配对的结算事件等）：不落盘不广播，
-				// 水位不前移——重放路径对同类事件同样不改变记账状态，口径一致。
-				return
+				// 与 reconcile 同口径：时刻无效时保留既有 watermarkTime 不前推（能走到此处的
+				// 必为有效果事件、时刻有效，null 分支仅作防御）。
+				await turnStats.put(id, { ...turnStatsFrom(next), watermarkSeq: seq, watermarkTime: Number.isFinite(time) ? time : turnStatsFrom(current).watermarkTime ?? null })
+				broadcastBusy()
+			} catch (error) {
+				ctx.logger?.warn?.(`dsh-activity-pane: 回合统计登记失败（${id}）: ${String(error)}`)
 			}
-			const time = Number(event.time)
-			// 与 reconcile 同口径：时刻无效时保留既有 watermarkTime 不前推（能走到此处的
-			// 必为有效果事件、时刻有效，null 分支仅作防御）。
-			await turnStats.put(id, { ...turnStatsFrom(next), watermarkSeq: seq, watermarkTime: Number.isFinite(time) ? time : turnStatsFrom(current).watermarkTime ?? null })
-			broadcastBusy()
-		} catch (error) {
-			ctx.logger?.warn?.(`dsh-activity-pane: 回合统计登记失败（${id}）: ${String(error)}`)
-		}
+		})
 	})
 
 	// HTTP API（前缀挂载，handler 内按子路径分发）。