@zhushanwen/pi-session-manager 0.2.4 → 0.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zhushanwen/pi-session-manager",
-  "version": "0.2.4",
+  "version": "0.3.0",
   "description": "Pi extension for managing agent-managed sessions — create, send, read history, list, status, abort via ctx.ui.select channel.",
   "type": "module",
   "main": "index.ts",
@@ -30,11 +30,11 @@
     "README.md"
   ],
   "dependencies": {
-    "@zhushanwen/extension-protocol": "0.17.0",
-    "@zhushanwen/pi-ext-guards": "0.4.2",
-    "@zhushanwen/pi-extension-logger": "0.6.1",
-    "@zhushanwen/pi-notify-ledger-host": "0.2.4",
-    "@zhushanwen/subagent-core": "1.2.1"
+    "@zhushanwen/extension-protocol": "0.18.0",
+    "@zhushanwen/pi-notify-ledger-host": "0.3.0",
+    "@zhushanwen/subagent-core": "1.3.0",
+    "@zhushanwen/pi-ext-guards": "0.4.3",
+    "@zhushanwen/pi-extension-logger": "0.6.2"
   },
   "devDependencies": {
     "@types/node": "^24.0.0",
@@ -43,7 +43,7 @@
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "*",
-    "typebox": "*"
+    "typebox": "1.3.27"
   },
   "peerDependenciesMeta": {
     "@earendil-works/pi-coding-agent": {

package/src/__tests__/tool-error-handling.test.ts

@@ -11,7 +11,7 @@ describe("U5-A4 tool-error-handling", () => {
 		vi.clearAllMocks();
 	});
 
-	it("select returning undefined (user cancel/timeout) → cancelled throw（W4 范式：错误路径 throw）", async () => {
+	it("select returning undefined (user cancel/channel error) → cancelled throw（W4 范式：错误路径 throw）", async () => {
 		await expectToolRejects(
 			createToolHarness(vi.fn().mockResolvedValue(undefined)),
 			"create_managed_session",

package/src/__tests__/tool-execute.test.ts

@@ -53,8 +53,8 @@ describe("U5-A2 tool-execute", () => {
 		expect(options).toHaveLength(1);
 		const payload = JSON.parse(options[0]);
 		expect(payload.action).toBe(expectedAction);
-		// third arg has timeout
-		expect(opts).toHaveProperty("timeout");
+		// third arg: timeout 不传（ADR-0122 无包内挂死兜底，全 action 无 timer 长挂）
+		expect(opts).toHaveProperty("timeout", undefined);
 	});
 
 	it("create_managed_session includes cwd and label in payload.params", async () => {

package/src/index.ts

@@ -4,6 +4,7 @@
 import type {
 	ExtensionAPI,
 	ExtensionContext,
+	ExtensionToolContext,
 	SessionCompactEvent,
 	SessionStartEvent,
 } from "@earendil-works/pi-coding-agent";
@@ -55,27 +56,19 @@ const AbortSessionParams = Type.Object({
 
 // ── select 通道辅助 ──
 
-/** select 超时（ms）：工具等待 runtime handler respond 的最大时间（create/history 走长链路放宽）。
- *  watch 刻意缺席（notify-once D2：不传 timeout、fire-and-forget 长挂——P1 实测 `if
- *  (opts?.timeout)` 才设 timer）。形制 = `Exclude<SessionManagerAction, "watch">`：
- *  协议 action 集 6→7 后漏配键即编译期红（计划 blocker #1 的修复形态）。 */
-const SELECT_TIMEOUT_MS: Record<Exclude<SessionManagerAction, "watch">, number> = {
-	create: 60_000,
-	send: 30_000,
-	history: 60_000,
-	status: 30_000,
-	list: 30_000,
-	abort: 30_000,
-};
-
 /**
  * 通过 select 通道向 runtime handler 发送 session 管理请求（传输核走 protocol 的
  * callMarkerRpc 原语，D8）。回包为 handler respond 的 JSON 字符串（value 恒 raw）；
- * 失败四态（cancelled/timeout/channel-error/non-json）由 executeTool 统一 throw——
- * pi agent-loop 仅在 execute throw 时置 isError:true，返回值里的 isError 字段被丢弃
- * （pi-agent-core dist/agent-loop.js:453-483 executePreparedToolCall：正常 return
- * 硬编码 isError:false、catch 置 true——0.84.4 实读 :468/:470-476；语义登记 PS-56）。
+ * 失败态（cancelled/channel-error/non-json）由 executeTool 统一 throw——
+ * execute throw → pi agent-loop catch 置 isError:true（pi-agent-core dist/agent-loop.js
+ * executePreparedToolCall，1.0.0 实读 catch :581-588）；1.0.0 起返回值 isError:true 也被
+ * 尊重（同文件 :579 `isError: result.isError === true`，0.84.4 时返回值会被丢弃）——
+ * 本工具失败恒走 throw，两版语义等价（语义登记 PS-56）。
  * 通道异常与非 JSON 回包的留痕由原语经注入的 log 承担。
+ *
+ * 不传 timeout（ADR-0122：无包内挂死兜底，handler 不回包时工具调用长挂、失败直报；
+ * 原 SELECT_TIMEOUT_MS per-action 表已删——原 watch 通道 D2/P1 的无 timer 长挂形态
+ * 现为全部 action 统一形态：任意晚的 respond 按 id 精确 resolve）。
  */
 function callSessionManager(
 	ctx: ExtensionContext,
@@ -95,8 +88,7 @@ function callSessionManager(
 		ui: { select: ctx.ui.select.bind(ctx.ui) },
 	};
 	return callMarkerRpc(guiCtx, SESSION_MANAGER_MARKER, payload, {
-		// watch 不传 timeout（D2/P1：无 timer 长挂不死，任意晚的 respond 按 id 精确 resolve；PS-59）
-		timeout: action === "watch" ? undefined : SELECT_TIMEOUT_MS[action],
+		// 全 action 不传 timeout（ADR-0122：无包内挂死兜底；无 timer 长挂，任意晚的 respond 按 id 精确 resolve）
 		log: (msg, detail) => logger.error(`[session-manager] ${msg}`, detail),
 	});
 }
@@ -117,10 +109,13 @@ function asResultRecord(v: unknown): Record<string, unknown> | undefined {
 /**
  * 统一的 execute 包装：调用 select 通道并解析结果。
  * 返回标准 AgentToolResult 形状；select 取消/超时/异常/非 JSON 回包是错误路径，
- * 必须 throw（extension-conventions「禁止错误成功模式」——pi 契约里 execute 只有
- * throw 才被置 isError:true，返回值携带 isError 字段会被 agent-loop 丢弃
- * （agent-loop.js:453-483，PS-56），ask-user/scheduler/session-reader 的 W4 throw
- * 范式同款；调用方 agent 需能区分成功与失败以决定重试/放弃）。
+ * 必须 throw（extension-conventions「禁止错误成功模式」——execute throw → pi
+ * agent-loop catch 置 isError:true（pi-agent-core dist/agent-loop.js
+ * executePreparedToolCall，1.0.0 实读 catch :581-588）；1.0.0 起返回值 isError:true
+ * 也被尊重（同文件 :579 `isError: result.isError === true`，0.84.4 时返回值会被
+ * 丢弃）——本工具失败恒走 throw，两版语义等价（语义登记 PS-56）；
+ * ask-user/scheduler/session-reader 的 W4 throw 范式同款，调用方 agent 需能区分
+ * 成功与失败以决定重试/放弃）。
  */
 async function executeTool(
 	ctx: ExtensionContext,
@@ -130,13 +125,13 @@ async function executeTool(
 ): Promise<{ content: Array<{ type: "text"; text: string }>; details: undefined }> {
 	const result = await callSessionManager(ctx, action, params);
 	if (!result.ok) {
-		// 行为微变①（D8，有意——对齐 plugin-bridge 形态）：非 JSON 回包从「catch 后
+		// 行为微变①（D8，有意）：非 JSON 回包从「catch 后
 		// parsed=undefined 静默当成功文本返回」改为 throw + 提示文本（留痕由原语
-		// 经注入的 logger.error 承担）；其余三态维持原 cancelled/timeout 折叠文案。
+		// 经注入的 logger.error 承担）；不传 timeout 后剩余 cancelled/channel-error 两态折叠。
 		const text =
 			result.reason === "non-json"
 				? `Session manager ${action}: non-JSON response from runtime (protocol mismatch — redeploy same-version runtime + extension; see extension logs).`
-				: `Session manager ${action}: cancelled or timed out.`;
+				: `Session manager ${action}: cancelled or channel error.`;
 		throw new Error(text);
 	}
 	const raw = result.value;
@@ -189,7 +184,8 @@ function registerSessionTool<S extends TObject>(pi: ExtensionAPI, cfg: SessionTo
 			params: Static<S>,
 			_signal: AbortSignal | undefined,
 			_onUpdate: unknown,
-			ctx: ExtensionContext,
+			// pi 1.0.0：ToolDefinition.execute 第 5 参 ctx 收窄为 ExtensionToolContext
+			ctx: ExtensionToolContext,
 		) {
 			return executeTool(ctx, cfg.action, cfg.toParams(params), cfg.onResult);
 		},