@zhushanwen/pi-unified-hooks 0.0.3 → 0.0.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zhushanwen/pi-unified-hooks",
-  "version": "0.0.3",
+  "version": "0.0.5",
   "description": "Unified hooks extension - collect scattered hooks in one place for easy maintenance",
   "main": "index.ts",
   "type": "module",

package/src/hooks/network-timeout-guard.ts

@@ -36,12 +36,23 @@ function detectNetworkCommand(command: string): string | null {
   return null;
 }
 
+/**
+ * Subset of `BashToolCallEvent` fields used by this hook.
+ * Local interface because the SDK's full event type is not re-exported
+ * by the CI ambient type stubs in `shared/types/mariozechner/index.d.ts`.
+ */
+interface BashToolCallLikeEvent {
+  toolName: string;
+  input: { command: string; timeout?: number };
+}
+
 export function setupNetworkTimeoutGuard(pi: ExtensionAPI): void {
-  pi.on("tool_call", async (event: any) => {
+  pi.on("tool_call", async (event: unknown) => {
     // Only intercept bash tool calls
-    if (event.toolName !== "bash") return;
+    const e = event as BashToolCallLikeEvent;
+    if (e.toolName !== "bash") return;
 
-    const { command, timeout } = event.input as { command: string; timeout?: number };
+    const { command, timeout } = e.input;
 
     // No network command detected — let it through
     const networkLabel = detectNetworkCommand(command);

package/src/hooks/test-timeout-guard.ts

@@ -71,11 +71,22 @@ function detectTestCommand(command: string): string | null {
   return null;
 }
 
+/**
+ * Subset of `BashToolCallEvent` fields used by this hook.
+ * Local interface because the SDK's full event type is not re-exported
+ * by the CI ambient type stubs in `shared/types/mariozechner/index.d.ts`.
+ */
+interface BashToolCallLikeEvent {
+  toolName: string;
+  input: { command: string; timeout?: number };
+}
+
 export function setupTestTimeoutGuard(pi: ExtensionAPI): void {
-  pi.on("tool_call", async (event: any) => {
-    if (event.toolName !== "bash") return;
+  pi.on("tool_call", async (event: unknown) => {
+    const e = event as BashToolCallLikeEvent;
+    if (e.toolName !== "bash") return;
 
-    const { command, timeout } = event.input as { command: string; timeout?: number };
+    const { command, timeout } = e.input;
 
     const testLabel = detectTestCommand(command);
     if (!testLabel) return;

package/src/hooks/tool-error-handler.ts

@@ -7,9 +7,24 @@
 
 import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
 
+/**
+ * Subset of `ToolExecutionEndEvent` fields used by this hook.
+ * Local interface because the SDK's full event type is not re-exported
+ * by the CI ambient type stubs in `shared/types/mariozechner/index.d.ts`.
+ */
+interface ToolExecutionEndLikeEvent {
+  isError: boolean;
+  toolName: string;
+  toolCallId: string;
+}
+
 export function setupToolErrorHandler(pi: ExtensionAPI): void {
-  pi.on("tool_execution_end", async (event: any) => {
-    if (!event.isError) return;
-    console.log(`[unified-hooks] ${event.toolName} error (callId=${event.toolCallId})`);
+  pi.on("tool_execution_end", async (event: unknown) => {
+    const e = event as ToolExecutionEndLikeEvent;
+    if (!e.isError) return;
+    // Tool error diagnostics are internal — surfacing to the terminal
+    // would leak to the input area. ctx.ui.notify is unnecessary because
+    // tool errors are already surfaced by Pi's own error rendering.
+    console.warn(`[unified-hooks] ${e.toolName} error (callId=${e.toolCallId})`);
   });
 }

package/src/index.ts

@@ -8,13 +8,13 @@
 import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
 
 // Re-export hook modules for easy access
-export { setupToolErrorHandler } from "./hooks/tool-error-handler";
 export { setupNetworkTimeoutGuard } from "./hooks/network-timeout-guard";
 export { setupTestTimeoutGuard } from "./hooks/test-timeout-guard";
+export { setupToolErrorHandler } from "./hooks/tool-error-handler";
 
-import { setupToolErrorHandler } from "./hooks/tool-error-handler";
 import { setupNetworkTimeoutGuard } from "./hooks/network-timeout-guard";
 import { setupTestTimeoutGuard } from "./hooks/test-timeout-guard";
+import { setupToolErrorHandler } from "./hooks/tool-error-handler";
 
 /**
  * Extension factory - registers all unified hooks
@@ -41,11 +41,13 @@ export default function unifiedHooksExtension(pi: ExtensionAPI): void {
     }
   }
 
-  // Log hook status on session start for debugging
+  // Hook status is an internal diagnostic — surfacing to the terminal
+  // would leak to the input area. Use console.warn (stderr) for
+  // development debugging; production should be silent.
   pi.on("session_start", () => {
     const enabled = hooks.filter((h) => h.enabled).map((h) => h.name);
     const disabled = hooks.filter((h) => !h.enabled).map((h) => h.name);
-    console.log(
+    console.warn(
       `[unified-hooks] Loaded: ${enabled.join(", ") || "(none)"}${
         disabled.length ? ` | Failed: ${disabled.join(", ")}` : ""
       }`