fix: address onboarding merge blockers

Author: web3dev1337

client/app.js

@@ -9347,8 +9347,8 @@ class ClaudeOrchestrator {
 	    })();
 
 	    const setBootstrapPending = (pending) => {
-	      if (!isWindowsHost) return;
 	      if (pending) {
+	        if (!isWindowsHost) return;
 	        body?.classList?.add?.('dependency-onboarding-booting');
 	        body?.classList?.remove?.('dependency-onboarding-active');
 	        return;
@@ -10158,7 +10158,6 @@ class ClaudeOrchestrator {
 	        }
 	        const view = render();
 	        applyOnboardingLockUI();
-	        if (view.req?.coreReady) writeDismissed(false);
 
 	        const hasCompletedOnboarding = readCompleted();
 	        const coreReady = !!view.req?.coreReady;

server/index.js

@@ -454,19 +454,29 @@ workspaceSystemReady = initializeWorkspaceSystem()
     logger.info('Workspace system initialized');
     return true;
   })
-  .then(() => loadPlugins())
-  .then((status) => {
-    logger.info('Plugin loader finished', {
-      loaded: Array.isArray(status?.loaded) ? status.loaded.length : 0,
-      failed: Array.isArray(status?.failed) ? status.failed.length : 0
-    });
-    return true;
-  })
   .catch(error => {
     logger.error('Workspace system initialization failed', { error: error.message, stack: error.stack });
     return false;
   });
 
+workspaceSystemReady
+  .then((workspaceReady) => {
+    if (!workspaceReady) return null;
+    return loadPlugins()
+      .then((status) => {
+        logger.info('Plugin loader finished', {
+          loaded: Array.isArray(status?.loaded) ? status.loaded.length : 0,
+          failed: Array.isArray(status?.failed) ? status.failed.length : 0
+        });
+        return status;
+      })
+      .catch((error) => {
+        logger.error('Plugin loader failed', { error: error.message, stack: error.stack });
+        return null;
+      });
+  })
+  .catch(() => null);
+
 // WebSocket connection handling
 io.on('connection', (socket) => {
   logger.info('Client connected', { socketId: socket.id });

server/sessionManager.js

@@ -2373,6 +2373,9 @@ class SessionManager extends EventEmitter {
       let stdout = '';
       child.stdout.on('data', (d) => { stdout += d; });
       const timer = setTimeout(() => child.kill(), 2000);
+      child.on('error', () => {
+        clearTimeout(timer);
+      });
       child.on('close', () => {
         clearTimeout(timer);
         const processCount = parseInt(String(stdout || '').trim(), 10);
@@ -2397,6 +2400,9 @@ class SessionManager extends EventEmitter {
     let stdout = '';
     child.stdout.on('data', (d) => { stdout += d; });
     const timer = setTimeout(() => child.kill(), 2000);
+    child.on('error', () => {
+      clearTimeout(timer);
+    });
     child.on('close', (code) => {
       clearTimeout(timer);
       // pgrep exits with code 1 when no child process matches; treat as zero children.