@zhushanwen/pi-pending-notifications 0.3.1 → 0.3.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zhushanwen/pi-pending-notifications",
-  "version": "0.3.1",
+  "version": "0.3.3",
   "description": "Cross-extension async operation registration/query mechanism for Pi — prevents message injection during long-running operations.",
   "type": "module",
   "main": "src/index.ts",
@@ -22,6 +22,7 @@
     "index.ts"
   ],
   "devDependencies": {
+    "@vitest/coverage-v8": "^4.1.9",
     "vitest": "^4.1.8"
   },
   "peerDependencies": {

package/src/index.ts

@@ -75,13 +75,14 @@ interface PendingToolDetails {
 /**
  * 模块级 EventBus 监听器 unsubscribe 函数列表。
  *
- * EventBus 是进程级单例（真实 SDK resource-loader 构造一次，跨 /reload、会话切换复用）。
- * 工厂函数 pendingNotificationsExtension 每次 reload 都重新执行，若不先移除旧监听器，
- * N 次 reload 后 EventBus 上会累积 N 组监听器（>11 后抛 Possible EventEmitter memory leak）。
+ * [W4 注释修正] 原 rationale（"EventBus 进程级单例 + reload 累积 N 组监听器"）在
+ * pi 0.84.1 不成立：pi.events.on 的返回值是 runtime.trackEventBusSubscription 跟踪的
+ * unsubscribe（loader.js:338-341），runtime.invalidate() 时自动全部退订——session 替换
+ * （dispose）与 /reload（invalidate 先于重新 load）都触发；且 session 替换会创建全新
+ * ResourceLoader → 全新 eventBus（agent-session-services.js:63-68），旧 bus 整体废弃。
  *
- * 用模块级变量跨 reload 持久：工厂入口先调用上一轮的 unsubscribe 清理旧监听器，
- * 再注册新的。这同时修复了多 session 串数据问题——reload 后只有当前闭包的监听器存活，
- * currentSessionId 始终是最新 session（旧闭包的过期 currentSessionId 不会再给事件打戳）。
+ * 下方的工厂入口手工清理因此是双重冗余（无害）：pi 已自动清理 tracked 订阅。
+ * 保留是防御性兜底（不依赖 pi 内部清理时机，卸载幂等）。
  */
 let unsubscribers: Array<() => void> = [];
 
@@ -115,11 +116,11 @@ export default function pendingNotificationsExtension(pi: ExtensionAPI): void {
 		}
 	}
 
-	// debug 日志：环境变量 PENDING_DEBUG=1 时输出到 console.debug。
+	// debug 日志：环境变量 XYZ_AGENT_DEBUG=1 时输出到 console.debug。
 	// 不再写入 session entry（pending:log）——session entries 是 append-only 无法 GC，
 	// 12 处 debug 日志会让长 session 的 entries 线性膨胀，而 goal before-agent-start
 	// 每 turn 全量扫描 getEntries()。状态数据（pending:register/unregister）仍写 entry。
-	const debugEnabled = process.env.PENDING_DEBUG === "1";
+	const debugEnabled = process.env.XYZ_AGENT_DEBUG === "1";
 	function debugLog(level: string, message: string, data?: unknown): void {
 		if (!debugEnabled) return;
 		console.debug(`[pending-notifications:${level}] ${message}`, data ?? "");