@zhushanwen/pi-base-tool-enhance 0.5.1 → 0.5.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zhushanwen/pi-base-tool-enhance",
-  "version": "0.5.1",
+  "version": "0.5.2",
   "description": "Base tool enhancement - override builtin bash tool: foreground delegates to pi official factory, incremental background mode (design: docs/design/base-tool-enhance.md)",
   "type": "module",
   "main": "src/index.ts",
@@ -28,10 +28,10 @@
     "skills/"
   ],
   "dependencies": {
-    "@xyz-agent/extension-protocol": "0.8.2",
-    "@zhushanwen/pi-extension-logger": "0.4.1",
-    "@zhushanwen/pi-file-lock": "0.2.1",
-    "@zhushanwen/pi-llm-shared": "0.6.0"
+    "@xyz-agent/extension-protocol": "0.9.0",
+    "@zhushanwen/pi-extension-logger": "0.6.0",
+    "@zhushanwen/pi-file-lock": "0.4.0",
+    "@zhushanwen/pi-llm-shared": "0.7.0"
   },
   "devDependencies": {
     "@vitest/coverage-v8": "^4.1.9",
@@ -40,7 +40,7 @@
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "^0.84.4",
     "typebox": "*",
-    "@zhushanwen/pi-pending-notifications": "0.6.0"
+    "@zhushanwen/pi-pending-notifications": "0.7.0"
   },
   "peerDependenciesMeta": {
     "@zhushanwen/pi-pending-notifications": {

package/src/__tests__/index.test.ts

@@ -121,7 +121,7 @@ describe("session_start chain: reconcile (M3, reap sunk into runtime by u-bte-re
 					status: "cancelled",
 				}),
 			);
-			// 对账之外尽力补一次 emit（listener 内存视图同步，失败无害）
+			// 对账之外尽力补一次 emit（幂等兜底，纯日志性质；失败无害）
 			expect(pi.events.emit).toHaveBeenCalledWith("pending:unregister", {
 				id: "bt-1700000000-idx001",
 				reason: "cancelled",

package/src/__tests__/maintenance-once.test.ts

@@ -5,23 +5,23 @@
 //  - reconcilePendingEntries 是 session 级豁免类，每 session_start 派发都执行
 //    （startup/resume/new 多派发 ×N——含 factory 二调 handler 累积的真实形态）
 //  - 触发面消失：reaper.ts 模块文件已删除（文件级断言，见首条用例）+ 维护链不再触发
-//    全局文件锁（withFileLock——原 reapOrphanedTasks 的 reaper.lock 路径）
+//    全局文件锁（原 reapOrphanedTasks 的 reaper.lock 路径随模块删除一并消失）
 //  - 入口无条件 debug 日志按派发次数出现，detail 仅含 reason（reapSkipped 字段
 //    随 reap 调用移除，S6 观测通道语义更新）
-// 断言方式：reconcile / logger / file-lock 全部间谍注入，观察调用次数与参数。
+// 断言方式：reconcile / logger 间谍注入观察调用次数与参数；file-lock 为 mock
+// 缺键守卫 + withFileLockSync throw 哨兵（不计数）。
 import { existsSync } from "node:fs";
 import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 
 import { describe, expect, it, vi, beforeEach } from "vitest";
 
-const { reconcileMock, loggerMock, officialBashFactoryMock, dataDirRef, withFileLockMock } =
+const { reconcileMock, loggerMock, officialBashFactoryMock, dataDirRef } =
 	vi.hoisted(() => ({
 		reconcileMock: vi.fn(),
 		loggerMock: { debug: vi.fn(), warn: vi.fn(), error: vi.fn() },
 		officialBashFactoryMock: vi.fn(),
 		dataDirRef: { dir: "/tmp/bte-fake-agent-dir" },
-		withFileLockMock: vi.fn(),
 	}));
 
 vi.mock("@earendil-works/pi-coding-agent", () => ({
@@ -31,10 +31,10 @@ vi.mock("@earendil-works/pi-coding-agent", () => ({
 vi.mock("@zhushanwen/pi-extension-logger", () => ({
 	getLogger: () => loggerMock,
 }));
-// 全局扫描探针：withFileLock 是原 reapOrphanedTasks 持 reaper.lock 的唯一锁路径
-// （registry 写走的是 withFileLockSync）——维护链触发它 = 全局扫描回流
+// 守卫形态 = mock 工厂缺键：包入口已删除的 async 锁导出不在工厂注册——维护链若
+// 误回流，对已删导出的任何访问（调用与 typeof 均计）即抛错（vitest mock 缺键语义；
+// 模块导入本身不抛）。withFileLockSync 哨兵保留：registry 写路径误入维护链即抛
 vi.mock("@zhushanwen/pi-file-lock", () => ({
-	withFileLock: withFileLockMock,
 	withFileLockSync: vi.fn(() => {
 		throw new Error("unexpected registry write in maintenance chain");
 	}),
@@ -86,7 +86,6 @@ beforeEach(() => {
 		parameters: {},
 		execute: vi.fn(),
 	});
-	withFileLockMock.mockReset();
 	loggerMock.debug.mockClear();
 	loggerMock.warn.mockClear();
 	loggerMock.error.mockClear();
@@ -118,8 +117,7 @@ describe("session_start maintenance chain after reap sink (u-bte-remove)", () =>
 		expect(reconcileMock).toHaveBeenCalledTimes(3);
 		expect(reconcileMock).toHaveBeenNthCalledWith(1, expect.anything(), dataDirRef.dir, "sid-once", []);
 		expect(reconcileMock).toHaveBeenLastCalledWith(expect.anything(), dataDirRef.dir, "sid-once", []);
-		// 无全局扫描：维护链不触发原 reaper 的跨进程锁路径，也无告警
-		expect(withFileLockMock).not.toHaveBeenCalled();
+		// 无全局扫描回流：误触发已删锁导出的任何访问会被 mock 缺键直接抛错（守卫见工厂注释），无告警
 		expect(loggerMock.warn).not.toHaveBeenCalled();
 	});
 

package/src/background/pending-reconcile.ts

@@ -8,12 +8,14 @@
  *  ③ fork 后任务完成通知写进新 session，旧 session 文件的 register 成僵尸。
  *
  * 收尾写法（权威路径）：直接 pi.appendEntry("pending:unregister", {id, reason,
- * status})——**不走 bus emit 作为权威**：pending-notifications 的 unregister listener
- * 落盘条件是其内存 registry 该 id active（其 registry 只在自身 session_start rebuild
- * 后非空），两个 extension 的加载/派发顺序（CLI --extension 顺序用户可控）无保障，
- * 顺序反转时 emit 被静默吞、对账失效。差集消费方 goal 从持久化 entries 算差集
- * （agent-end.ts getEntries()，不读 pending 内存 registry），appendEntry 对守卫直接
- * 生效。appendEntry 之外尽力补一次 emit（listener 就绪时同步其内存视图，失败无害）。
+ * status})——**不走 bus emit 作为权威**：appendEntry 不依赖 listener 存活，两个
+ * extension 的加载/派发顺序（CLI --extension 顺序用户可控）无保障，顺序反转时
+ * emit 无人接收、对账失效。差集消费方 goal 从持久化 entries 算差集
+ * （agent-end.ts getEntries()），appendEntry 同步入账（pi dist 实证）对守卫直接
+ * 生效，无不一致窗口。appendEntry 之外尽力补一次 emit（幂等兜底，纯日志性质：
+ * pending-notifications 的 unregister listener 落盘前置判断 isPendingActive 对
+ * getEntries() 现算——其内存 registry/rebuild 已随 ext-simplify-12 删除，emit
+ * 到达时该 id 已注销即跳过，失败无害）。
  *
  * 终态判据：registry state ∈ {exited, orphaned}，或（state=running/killing 且
  * kill(pid,0) 判死：收殓/写盘失败遗留的 running 条目按事实终态处理）。
@@ -112,7 +114,7 @@ export function reconcilePendingEntries(
 		if (entry === undefined) {
 			// registry 无条目：终态无从判定（LRU 淘汰的终态条目其 unregister entry 应已
 			// 落盘，差集里还出现 = spawn 后 registry 写失败等罕见路径）——保守不动作，
-			// 差集残留交给 pending 自身 TTL 之外的 next-session 对账重查
+			// pending-notifications 已无任何 TTL 清理，差集残留由 next-session 对账重查收口
 			result.skipped.push(id);
 			continue;
 		}
@@ -130,8 +132,9 @@ export function reconcilePendingEntries(
 			});
 			continue;
 		}
-		// 尽力补 emit（listener 就绪时同步 pending 内存视图，缩短 pending_notifications
-		// 工具列表的不一致窗口；失败无害——appendEntry 已是权威路径）
+		// 尽力补 emit（幂等兜底，纯日志性质：listener 落盘前置 isPendingActive 对
+		// entries 现算，该 id 已注销即跳过——appendEntry 同步入账已无不一致窗口；
+		// 失败无害）
 		try {
 			pi.events.emit("pending:unregister", { id, reason: pendingReason });
 		} catch (err) {