fix: limit [hook-trace] logging to race-condition paths only

Remove happy-path logging so [hook-trace] lines only appear when the
PostToolUse hook fires before its callback is registered.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: timvisher-dd

src/tools.ts

@@ -780,17 +780,14 @@ export const registerHookCallback = (
   },
   logger: Logger = console,
 ) => {
-  const wasPending = !!(onPostToolUseHook && pendingHooks[toolUseID]);
-  logger.log(`[hook-trace] registerHookCallback id=${toolUseID} pendingHookWaiting=${wasPending}`);
-
   toolUseCallbacks[toolUseID] = {
     onPostToolUseHook,
   };
 
   // If a PostToolUse hook is already waiting for this ID, hand it the callback.
-  if (wasPending) {
+  if (onPostToolUseHook && pendingHooks[toolUseID]) {
     logger.log(`[hook-trace] registerHookCallback resolved pending hook id=${toolUseID}`);
-    pendingHooks[toolUseID].resolve(onPostToolUseHook!);
+    pendingHooks[toolUseID].resolve(onPostToolUseHook);
     delete pendingHooks[toolUseID];
   }
 };
@@ -813,9 +810,6 @@ export const createPostToolUseHook =
       if (toolUseID) {
         const onPostToolUseHook = toolUseCallbacks[toolUseID]?.onPostToolUseHook;
         if (onPostToolUseHook) {
-          logger.log(
-            `[hook-trace] PostToolUse fired id=${toolUseID} tool=${input.tool_name} callbackReady=true`,
-          );
           await onPostToolUseHook(toolUseID, input.tool_input, input.tool_response);
           delete toolUseCallbacks[toolUseID]; // Cleanup after execution
         } else {