🤖 feat: Add comprehensive logging to init hook execution

- Added log.debug() for init hook detection and script execution
- Added log.info() for init hook start and completion with exit codes
- Added log.error() for init hook failures
- Added logging to BashExecutionService for streaming command execution
- Added process error logging for bash execution failures

This improves debuggability when init hooks don't work as expected.

Author: ammar-agent

src/services/bashExecutionService.ts

@@ -1,5 +1,6 @@
 import { spawn } from "child_process";
 import type { ChildProcess } from "child_process";
+import { log } from "./log";
 
 /**
  * Configuration for bash execution
@@ -114,6 +115,11 @@ export class BashExecutionService {
     config: BashExecutionConfig,
     callbacks: StreamingCallbacks
   ): DisposableProcess {
+    log.debug(`BashExecutionService: Executing streaming command in ${config.cwd}`);
+    log.debug(
+      `BashExecutionService: Script: ${script.substring(0, 100)}${script.length > 100 ? "..." : ""}`
+    );
+
     const spawnCommand = config.niceness !== undefined ? "nice" : "bash";
     const spawnArgs =
       config.niceness !== undefined
@@ -130,6 +136,8 @@ export class BashExecutionService {
       detached: config.detached ?? true,
     });
 
+    log.debug(`BashExecutionService: Spawned process with PID ${child.pid}`);
+
     // Line-by-line streaming with incremental buffers
     let outBuf = "";
     let errBuf = "";
@@ -160,6 +168,7 @@ export class BashExecutionService {
     });
 
     child.on("close", (code: number | null) => {
+      log.debug(`BashExecutionService: Process exited with code ${code}`);
       // Flush any remaining partial lines
       if (outBuf.trim().length > 0) {
         callbacks.onStdout(outBuf);
@@ -170,6 +179,10 @@ export class BashExecutionService {
       callbacks.onExit(code ?? 0);
     });
 
+    child.on("error", (error: Error) => {
+      log.error(`BashExecutionService: Process error:`, error);
+    });
+
     return new DisposableProcess(child);
   }
 }

src/services/ipcMain.ts

@@ -1,6 +1,7 @@
 import assert from "node:assert/strict";
 import type { BrowserWindow, IpcMain as ElectronIpcMain } from "electron";
 import { spawn, spawnSync } from "child_process";
+import * as fs from "fs";
 import * as fsPromises from "fs/promises";
 import * as path from "path";
 import type { Config, ProjectConfig } from "@/config";
@@ -65,15 +66,22 @@ export class IpcMain {
     // Non-blocking fire-and-forget; errors are reported via chat stream
     try {
       const hookPath = path.join(params.projectPath, ".cmux", "init");
-      // Check if hook exists
+
+      log.debug(`Checking for init hook at ${hookPath}`);
+
+      // Check if hook exists and is executable
       const exists = await fsPromises
-        .access(hookPath)
+        .access(hookPath, fs.constants.X_OK)
         .then(() => true)
         .catch(() => false);
+
       if (!exists) {
+        log.debug(`No init hook found at ${hookPath}`);
         return; // Nothing to do
       }
 
+      log.info(`Running init hook for workspace ${params.workspaceId}: ${hookPath}`);
+
       const session = this.getOrCreateSession(params.workspaceId);
 
       // Emit start event via chat stream
@@ -108,6 +116,9 @@ export class IpcMain {
             });
           },
           onExit: (exitCode) => {
+            log.info(
+              `Init hook completed for workspace ${params.workspaceId} with exit code ${exitCode}`
+            );
             session.emitChatEvent({
               type: "init-end",
               exitCode,
@@ -117,6 +128,7 @@ export class IpcMain {
         }
       );
     } catch (error) {
+      log.error(`Failed to run init hook for workspace ${params.workspaceId}:`, error);
       const session = this.getOrCreateSession(params.workspaceId);
       session.emitChatEvent({
         type: "init-output",