@zhushanwen/pi-unified-hooks 0.0.4 → 0.0.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zhushanwen/pi-unified-hooks",
-  "version": "0.0.4",
+  "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/tool-error-handler.ts

@@ -22,6 +22,9 @@ export function setupToolErrorHandler(pi: ExtensionAPI): void {
   pi.on("tool_execution_end", async (event: unknown) => {
     const e = event as ToolExecutionEndLikeEvent;
     if (!e.isError) return;
-    console.log(`[unified-hooks] ${e.toolName} error (callId=${e.toolCallId})`);
+    // 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

@@ -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(", ")}` : ""
       }`