repro: add even more logging to HookRunner and TestRig

integration-tests/hooks-system.test.ts

@@ -100,7 +100,10 @@ describe('Hooks System Integration', () => {
 
       const scriptPath = rig.createScript(
         'stderr_block_hook.cjs',
-        "process.stderr.write('File writing blocked by security policy'); process.exit(2);",
+        `
+process.stderr.write('File writing blocked by security policy');
+process.exit(2);
+`,
       );
 
       const normalizedCmd = normalizePath(`node "${scriptPath}"`);

packages/core/src/hooks/hookRunner.ts

@@ -339,6 +339,12 @@ export class HookRunner {
         stderr += data.toString();
       });
 
+      child.on('exit', (code, signal) => {
+        if (process.env['CI'] === 'true' || process.env['VERBOSE'] === 'true') {
+          console.log(`[HookRunner] Hook exit. code: ${code}, signal: ${signal}`);
+        }
+      });
+
       // Handle process exit
       child.on('close', (exitCode) => {
         clearTimeout(timeoutHandle);
@@ -406,6 +412,10 @@ export class HookRunner {
         clearTimeout(timeoutHandle);
         const duration = Date.now() - startTime;
 
+        if (process.env['CI'] === 'true' || process.env['VERBOSE'] === 'true') {
+          console.log(`[HookRunner] Hook process error: ${error.message}`);
+        }
+
         resolve({
           hookConfig,
           eventName,

packages/test-utils/src/test-rig.ts

@@ -65,7 +65,11 @@ export function sanitizeTestName(name: string) {
  * On Windows, this converts backslashes to forward slashes.
  */
 export function normalizePath(p: string): string {
-  return p.replace(/\\/g, '/');
+  const result = p.replace(/\\/g, '/');
+  if (process.env['CI'] === 'true' || process.env['VERBOSE'] === 'true') {
+    console.log(`[normalizePath] "${p}" -> "${result}"`);
+  }
+  return result;
 }
 
 // Helper to create detailed error messages