@fyeeme/pi-peon-ping 1.1.0 → 1.1.2

package/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 ## [Unreleased]
 
+## [1.1.2] - 2026-09-09
+
+### Fixed
+
+- peon.sh discovery (including the up-to-3s `brew --prefix` probe) no longer runs at extension load time; it is deferred to the first session event per the extensions doc's factory rule.
+- `PermissionRequest` (input.required) now fires from `ui_prompt_start` — any blocking `ctx.ui` dialog — instead of sniffing the omp-only `ask_user_question` tool name, which does not exist in pi.
+- Tab-title project name and event payloads use `ctx.cwd` instead of a stale factory-time `process.cwd()`.
+
+## [1.1.1] - 2026-08-08
+
+### Changed
+
+- Tab 标题改用 provided API：删除手写 ANSI `setTabTitle`（`process.stdout.write(\x1b]0;...\x07)`），四处调用点（`session_start`/`before_agent_start`/`agent_settled`/`tool_result`，均在 `ctx.hasUI` 守卫下）改为 `ctx.ui.setTitle(...)`——mode 感知，print/json 模式自动 no-op，不再向 stdout 写裸 ANSI。
+- `Stop` 事件映射从 `agent_end` 改为 `agent_settled`（与 ExtensionAPI 实际事件名一致）。
+
 ## [1.1.0] - 2026-07-26
 
 ### Added

package/README.md

@@ -97,9 +97,9 @@ pi-peon-ping auto-discovers the installed `peon.sh` with the following priority
 
 When a `.ps1` file is found, it is automatically run with `powershell` instead of `bash`.
 
-### peon-ping 配置
+### peon-ping configuration
 
-所有 peon-ping 的功能配置通过 `peon` CLI 进行：
+All peon-ping feature configuration is done through the `peon` CLI:
 
 ```bash
 peon packs use glados          # Switch sound pack

package/index.ts

@@ -14,8 +14,13 @@
  * Event mapping (pi ExtensionAPI → peon.sh hook_event_name):
  *   session_start                       → SessionStart
  *   turn_start                          → UserPromptSubmit
- *   turn_end                            → Stop
+ *   agent_settled                       → Stop
  *   tool_result (event.isError === true) → PostToolUseFailure
+ *   ui_prompt_start                     → PermissionRequest (input.required:
+ *                                         any blocking ctx.ui dialog —
+ *                                         replaces omp's ask_user_question
+ *                                         tool_call sniff, which doesn't
+ *                                         exist in pi)
  *   session_shutdown                    → SessionEnd
  *
  * peon.sh discovery (first match wins):
@@ -131,13 +136,9 @@ export function resolveShellAndScript(
 }
 
 // ============================================================================
-// Terminal tab title helpers
+// Terminal tab title (via pi's mode-aware ctx.ui.setTitle)
 // ============================================================================
 
-export function setTabTitle(title: string): void {
-	process.stdout.write(`\x1b]0;${title}\x07`);
-}
-
 // ============================================================================
 // Session ID helpers
 // ============================================================================
@@ -301,18 +302,23 @@ export async function execPeonCli(
 // ============================================================================
 
 export default function peonPingExtension(pi: ExtensionAPI): void {
-	const peonPath = findPeonSh();
-	if (!peonPath) {
-		console.warn("[pi-peon-ping] peon.sh not found. Install peon-ping first:");
-		console.warn("  brew install PeonPing/tap/peon-ping");
-		console.warn("  # or: curl -fsSL peonping.com/install | bash");
-		console.warn("  # or: set PEON_SH=/path/to/peon.sh");
-		return;
-	}
-
-	const cliPath = findPeonCli(peonPath);
-	const cwd = process.cwd();
-	const projectName = basename(cwd) || "pi";
+	// Doc (Long-lived resources): extension factories may run in invocations
+	// that never start a session — discovery (which may execFileSync `brew`,
+	// up to 3s) must not run here. Resolve lazily on first session_start and
+	// memoize per extension instance.
+	let peonPath: string | null | undefined;
+	const resolvePeon = (): string | null => {
+		if (peonPath === undefined) {
+			peonPath = findPeonSh();
+			if (!peonPath) {
+				console.warn("[pi-peon-ping] peon.sh not found. Install peon-ping first:");
+				console.warn("  brew install PeonPing/tap/peon-ping");
+				console.warn("  # or: curl -fsSL peonping.com/install | bash");
+				console.warn("  # or: set PEON_SH=/path/to/peon.sh");
+			}
+		}
+		return peonPath;
+	};
 
 	// -----------------------------------------------------------------------
 	// Lifecycle event forwarding (respect peon-ping category toggles)
@@ -321,65 +327,84 @@ export default function peonPingExtension(pi: ExtensionAPI): void {
 	// -----------------------------------------------------------------------
 
 	pi.on("session_start", async (_event, ctx) => {
-		const config = readPeonConfig(peonPath);
+		const peon = resolvePeon();
+		if (!peon) return;
+		const config = readPeonConfig(peon);
 		const sessionId = getSessionId(ctx);
-		if (ctx.hasUI) setTabTitle(`● ${projectName}: ready`);
-		firePeon(peonPath, "SessionStart", cwd, sessionId, config);
+		const projectName = basename(ctx.cwd) || "pi";
+		if (ctx.hasUI) ctx.ui.setTitle(`● ${projectName}: ready`);
+		firePeon(peon, "SessionStart", ctx.cwd, sessionId, config);
 	});
 
 	pi.on("before_agent_start", async (_event, ctx) => {
-		const config = readPeonConfig(peonPath);
+		const peon = resolvePeon();
+		if (!peon) return;
+		const config = readPeonConfig(peon);
 		const sessionId = getSessionId(ctx);
-		if (ctx.hasUI) setTabTitle(`● ${projectName}: working...`);
-		firePeon(peonPath, "UserPromptSubmit", cwd, sessionId, config);
+		const projectName = basename(ctx.cwd) || "pi";
+		if (ctx.hasUI) ctx.ui.setTitle(`● ${projectName}: working...`);
+		firePeon(peon, "UserPromptSubmit", ctx.cwd, sessionId, config);
 	});
 
 	pi.on("session_before_compact", async (_event, ctx) => {
-		const config = readPeonConfig(peonPath);
+		const peon = resolvePeon();
+		if (!peon) return;
+		const config = readPeonConfig(peon);
 		const sessionId = getSessionId(ctx);
-		firePeon(peonPath, "PreCompact", cwd, sessionId, config);
+		firePeon(peon, "PreCompact", ctx.cwd, sessionId, config);
 	});
 
-	pi.on("agent_end", async (_event, ctx) => {
-		const config = readPeonConfig(peonPath);
+	pi.on("agent_settled", async (_event, ctx) => {
+		const peon = resolvePeon();
+		if (!peon) return;
+		const config = readPeonConfig(peon);
 		const sessionId = getSessionId(ctx);
-		if (ctx.hasUI) setTabTitle(`✓ ${projectName}: done`);
-		firePeon(peonPath, "Stop", cwd, sessionId, config);
+		const projectName = basename(ctx.cwd) || "pi";
+		if (ctx.hasUI) ctx.ui.setTitle(`✓ ${projectName}: done`);
+		firePeon(peon, "Stop", ctx.cwd, sessionId, config);
 	});
 
 	pi.on("tool_result", async (event, ctx) => {
 		if (!event.isError) return;
-		const config = readPeonConfig(peonPath);
+		const peon = resolvePeon();
+		if (!peon) return;
+		const config = readPeonConfig(peon);
 		const sessionId = getSessionId(ctx);
-		if (ctx.hasUI) setTabTitle(`✗ ${projectName}: error`);
-		firePeon(peonPath, "PostToolUseFailure", cwd, sessionId, config);
+		const projectName = basename(ctx.cwd) || "pi";
+		if (ctx.hasUI) ctx.ui.setTitle(`✗ ${projectName}: error`);
+		firePeon(peon, "PostToolUseFailure", ctx.cwd, sessionId, config);
 	});
 
-	// Agent is asking the user a question (e.g. ask_user_question tool) →
-	// peon-ping "input.required" category. Fires before the tool runs.
-	pi.on("tool_call", async (event, ctx) => {
-		if (event.toolName !== "ask_user_question") return;
-		const config = readPeonConfig(peonPath);
+	// pi is asking the user something (any blocking ctx.ui dialog: select /
+	// confirm / input / editor / custom — ask_user, permission gates, …) →
+	// peon-ping "input.required" category. ui_prompt_start fires for every
+	// such span, so no per-tool sniffing is needed.
+	pi.on("ui_prompt_start", async (_event, ctx) => {
+		const peon = resolvePeon();
+		if (!peon) return;
+		const config = readPeonConfig(peon);
 		const sessionId = getSessionId(ctx);
-		firePeon(peonPath, "PermissionRequest", cwd, sessionId, config);
+		firePeon(peon, "PermissionRequest", ctx.cwd, sessionId, config);
 	});
 
 	pi.on("session_shutdown", async (_event, ctx) => {
-		const config = readPeonConfig(peonPath);
+		const peon = resolvePeon();
+		if (!peon) return;
+		const config = readPeonConfig(peon);
 		const sessionId = getSessionId(ctx);
-		firePeon(peonPath, "SessionEnd", cwd, sessionId, config);
+		firePeon(peon, "SessionEnd", ctx.cwd, sessionId, config);
 	});
 
 	// -----------------------------------------------------------------------
 	// Commands
 	// -----------------------------------------------------------------------
 
-	if (!cliPath) return;
-
 	pi.registerCommand("peon-ping-toggle", {
 		description: "Toggle peon-ping mute on/off",
 		handler: async (_args, ctx) => {
-			const result = await execPeonCli(cliPath, ["toggle"]);
+			const cli = findPeonCli(resolvePeon());
+			if (!cli) return;
+			const result = await execPeonCli(cli, ["toggle"]);
 			if (result.exitCode !== 0) {
 				ctx.ui.notify(result.stdout || "toggle failed", "error");
 			} else {
@@ -391,12 +416,14 @@ export default function peonPingExtension(pi: ExtensionAPI): void {
 	pi.registerCommand("peon-ping-use", {
 		description: "Switch sound pack: /peon-ping-use <name>",
 		handler: async (args, ctx) => {
+			const cli = findPeonCli(resolvePeon());
+			if (!cli) return;
 			const name = args.trim();
 			if (!name) {
 				ctx.ui.notify("Usage: /peon-ping-use <pack-name>", "warning");
 				return;
 			}
-			const result = await execPeonCli(cliPath, ["packs", "use", "--install", name]);
+			const result = await execPeonCli(cli, ["packs", "use", "--install", name]);
 			if (result.exitCode !== 0) {
 				ctx.ui.notify(result.stdout || `Failed to switch to ${name}`, "error");
 			} else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fyeeme/pi-peon-ping",
-  "version": "1.1.0",
+  "version": "1.1.2",
   "description": "peon-ping adapter for pi — routes lifecycle events through peon.sh for sound packs, notifications, and trainer reminders.",
   "type": "module",
   "license": "MIT",
@@ -44,10 +44,10 @@
     "typecheck": "tsc"
   },
   "peerDependencies": {
-    "@earendil-works/pi-coding-agent": ">=0.77.0"
+    "@earendil-works/pi-coding-agent": ">=0.84.4"
   },
   "devDependencies": {
-    "@earendil-works/pi-coding-agent": "0.77.0",
+    "@earendil-works/pi-coding-agent": "0.84.4",
     "@types/node": "22.19.19",
     "typescript": "5.9.3"
   }