@zhushanwen/pi-ask-user 7.0.12 → 7.0.14

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zhushanwen/pi-ask-user",
-  "version": "7.0.12",
+  "version": "7.0.14",
   "description": "Inline adaptive ask_user tool for Pi — single/multi-question structured input with split-pane preview, inline editor and optional comments.",
   "type": "module",
   "main": "index.ts",
@@ -29,7 +29,8 @@
     "ARCHITECTURE.md"
   ],
   "dependencies": {
-    "@xyz-agent/extension-protocol": "0.5.1"
+    "@xyz-agent/extension-protocol": "0.6.0",
+    "@zhushanwen/pi-extension-logger": "0.3.0"
   },
   "devDependencies": {
     "@earendil-works/pi-tui": "^0.84.2",

package/src/__tests__/channel-registry-register.test.ts

@@ -11,6 +11,16 @@
 //   - version !== 1 → warn + 重建为新 slot，旧 pending 丢弃
 //
 // 隔离：每个用例前 Reflect.deleteProperty(globalThis, CHANNEL_HANDSHAKE_KEY)。
+// Mock 共享 logger，让 logger.warn 可被 spy（源码已从 console.warn 改为 logger.warn）
+const { loggerMock } = vi.hoisted(() => ({
+  loggerMock: { debug: vi.fn(), warn: vi.fn(), error: vi.fn() },
+}));
+vi.mock("@zhushanwen/pi-extension-logger", () => ({
+  getLogger: () => loggerMock,
+  createLogger: () => loggerMock,
+  setPiHandle: vi.fn(),
+}));
+
 import { beforeEach,describe, expect, it, vi } from "vitest";
 
 import type { ChannelHandler } from "../channel-handler";
@@ -135,7 +145,7 @@ describe("registerAskUserChannelHandler", () => {
 
 	it("version !== 1 → warn + 重建为新 slot，旧 pending 丢弃", () => {
 		// 塞一个 version=2 的旧 slot（模拟未来协议升级 / 脏数据）
-		const spyWarn = vi.spyOn(console, "warn").mockImplementation(() => undefined);
+		loggerMock.warn.mockClear();
 		const legacyPending = [{ channel: "stale", handler: noopHandler }];
 		const legacySlot = {
 			version: 2 as const,
@@ -152,10 +162,8 @@ describe("registerAskUserChannelHandler", () => {
 		expect(slot!.version).toBe(1);
 		expect(slot!.pending).toEqual([{ channel: "ask_user", handler: noopHandler }]);
 		// warn 被调用（包含 version mismatch 提示）
-		expect(spyWarn).toHaveBeenCalledTimes(1);
-		expect(spyWarn.mock.calls[0]![0]).toContain("version mismatch");
-
-		spyWarn.mockRestore();
+		expect(loggerMock.warn).toHaveBeenCalledTimes(1);
+		expect(loggerMock.warn.mock.calls[0]![0]).toContain("version mismatch");
 	});
 
 	it("registry 就绪但 slot 已预置 pending → 仍走 register 路径（pending 不被本函数消费）", () => {

package/src/channel-registry-register.ts

@@ -14,6 +14,9 @@
 // registry 实例——仅往 slot 写 pending 或调 slot.registry.register。
 
 import type { ChannelHandler } from "./channel-handler";
+import { getLogger } from "@zhushanwen/pi-extension-logger";
+
+const logger = getLogger("ask-user");
 
 /**
  * 进程级 channel registry 握手的 globalThis key（Symbol.for 跨模块共享）。
@@ -69,9 +72,10 @@ function readSlot(): ChannelRegistryHandshake | undefined {
 		| undefined;
 	if (slot === undefined) return undefined;
 	if (slot.version !== HANDSHAKE_VERSION) {
-		console.warn(
-			`[ask-user] channel handshake slot version mismatch: expected ${HANDSHAKE_VERSION}, got ${slot.version}; discarding and rebuilding slot`,
-		);
+		logger.warn('channel handshake slot version mismatch, discarding and rebuilding', {
+			expected: HANDSHAKE_VERSION,
+			got: slot.version,
+		});
 		return undefined;
 	}
 	return slot;