clideck 1.29.0 → 1.29.1

package/handlers.js

@@ -101,6 +101,33 @@ checkAvailability();
 let cfg = config.load();
 if (detectTelemetryConfig(cfg)) config.save(cfg);
 
+function extractQuotedPath(command, needle) {
+  if (!command || !needle) return '';
+  const parts = String(command).match(/"([^"]+)"/g) || [];
+  for (const part of parts) {
+    const value = part.slice(1, -1);
+    if (value.includes(needle)) return value;
+  }
+  return '';
+}
+
+function hasExistingHook(arr, hookFile, route) {
+  return !!arr?.some(h => h.hooks?.some(x => {
+    if (!x.command?.includes(hookFile) || !x.command?.includes(` ${route}`)) return false;
+    const hookPath = extractQuotedPath(x.command, hookFile);
+    return !!hookPath && existsSync(hookPath);
+  }));
+}
+
+function codexConfigLooksHealthy(content, port) {
+  if (!content.includes('[otel]') || !content.includes(`localhost:${port}`)) return false;
+  const notifyLine = content.match(/^\s*notify\s*=\s*\[(.+)\]\s*$/m)?.[1] || '';
+  if (!notifyLine.includes('notify-helper')) return false;
+  const quoted = [...notifyLine.matchAll(/"([^"]+)"/g)].map(m => m[1]);
+  const helperPath = quoted.find(v => v.includes('notify-helper'));
+  return !!helperPath && existsSync(helperPath);
+}
+
 function detectTelemetryConfig(c) {
   const home = os.homedir();
   const port = '4000';
@@ -116,24 +143,27 @@ function detectTelemetryConfig(c) {
       try {
         const s = JSON.parse(readFileSync(join(home, '.claude', 'settings.json'), 'utf8'));
         const hooks = s.hooks || {};
-        const has = (arr, path) => arr?.some(h => h.hooks?.some(x => x.command?.includes('claude-hook.js') && x.command?.includes(` ${path}`)));
-        detected = has(hooks.UserPromptSubmit, 'start') && has(hooks.Stop, 'stop') && has(hooks.StopFailure, 'stop')
-                && has(hooks.PreToolUse, 'menu')
-                && hooks.Notification?.some(h => h.matcher === 'idle_prompt' && h.hooks?.some(x => x.command?.includes('claude-hook.js') && x.command?.includes(' idle')));
+        detected = hasExistingHook(hooks.UserPromptSubmit, 'claude-hook.js', 'start')
+                && hasExistingHook(hooks.Stop, 'claude-hook.js', 'stop')
+                && hasExistingHook(hooks.StopFailure, 'claude-hook.js', 'stop')
+                && hasExistingHook(hooks.PreToolUse, 'claude-hook.js', 'menu')
+                && hooks.Notification?.some(h => h.matcher === 'idle_prompt' && hasExistingHook([h], 'claude-hook.js', 'idle'));
         if (!detected) reason = 'Needs re-patch';
       } catch {}
     } else if (preset.presetId === 'codex') {
       try {
         const content = readFileSync(join(home, '.codex', 'config.toml'), 'utf8');
-        detected = content.includes('[otel]') && /^\s*notify\s*=.*notify-helper/m.test(content) && content.includes(`localhost:${port}`);
+        detected = codexConfigLooksHealthy(content, port);
         if (!detected) reason = 'Needs re-patch';
       } catch {}
     } else if (preset.presetId === 'gemini-cli') {
       try {
         const s = JSON.parse(readFileSync(join(home, '.gemini', 'settings.json'), 'utf8'));
         const hooks = s.hooks || {};
-        const has = (arr, route) => arr?.some(h => h.hooks?.some(x => x.command?.includes('gemini-hook.js') && x.command?.includes(` ${route}`)));
-        detected = has(hooks.BeforeAgent, 'start') && has(hooks.AfterAgent, 'stop') && has(hooks.SessionEnd, 'stop') && has(hooks.BeforeTool, 'menu');
+        detected = hasExistingHook(hooks.BeforeAgent, 'gemini-hook.js', 'start')
+                && hasExistingHook(hooks.AfterAgent, 'gemini-hook.js', 'stop')
+                && hasExistingHook(hooks.SessionEnd, 'gemini-hook.js', 'stop')
+                && hasExistingHook(hooks.BeforeTool, 'gemini-hook.js', 'menu');
         if (!detected) reason = 'Needs re-patch';
       } catch {}
     } else if (preset.presetId === 'opencode') {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "clideck",
-  "version": "1.29.0",
+  "version": "1.29.1",
   "description": "One screen for all your AI coding agents — run, monitor, and manage multiple CLI agents from a single browser tab",
   "main": "server.js",
   "bin": {

package/plugin-loader.js

@@ -206,13 +206,14 @@ function buildApi(pluginId, pluginDir, state) {
     getSession(id) {
       const s = sessionsFn?.()?.get(id);
       if (!s) return null;
-      return { id, name: s.name, cwd: s.cwd, commandId: s.commandId, presetId: s.presetId || 'shell', themeId: s.themeId, projectId: s.projectId, roleName: s.roleName || null, working: !!sessionStatus.get(id) };
+      const state = sessionStatus.get(id) || '';
+      return { id, name: s.name, cwd: s.cwd, commandId: s.commandId, presetId: s.presetId || 'shell', themeId: s.themeId, projectId: s.projectId, roleName: s.roleName || null, working: state.startsWith('1:') };
     },
     getSessions() {
       const sessions = sessionsFn?.();
       if (!sessions) return [];
       return [...sessions].map(([id, s]) => ({
-        id, name: s.name, cwd: s.cwd, commandId: s.commandId, presetId: s.presetId || 'shell', themeId: s.themeId, projectId: s.projectId, roleName: s.roleName || null, working: !!sessionStatus.get(id),
+        id, name: s.name, cwd: s.cwd, commandId: s.commandId, presetId: s.presetId || 'shell', themeId: s.themeId, projectId: s.projectId, roleName: s.roleName || null, working: (sessionStatus.get(id) || '').startsWith('1:'),
       }));
     },
 

package/sessions.js

@@ -101,19 +101,28 @@ function spawnSession(id, cmd, parts, cwd, name, themeId, commandId, savedToken,
   if (preset?.telemetryEnv) telemetry.watchSession(id, bin);
   if (preset?.bridge === 'opencode') opencodeBridge.watchSession(id, cwd);
 
+  function injectRolePrompt() {
+    if (!session.pendingRolePrompt) return;
+    transcript.recordInjectedInput(id, session.pendingRolePrompt);
+    term.write(session.pendingRolePrompt);
+    setTimeout(() => term.write('\r'), 150);
+    console.log(`Session ${id.slice(0, 8)}: injected role prompt`);
+    delete session.pendingRolePrompt;
+    delete session._rolePromptTimer;
+  }
+
   term.onData((data) => {
-    // Inject role prompt once after agent starts producing output
+    // Role prompts should be injected only when the agent is likely ready for
+    // input. For Codex, use the first OTLP startup event instead of a blind
+    // fixed startup delay; other agents keep the existing delayed path.
     if (session.pendingRolePrompt && !session._rolePromptTimer) {
-      session._rolePromptTimer = setTimeout(() => {
-        if (session.pendingRolePrompt) {
-          transcript.recordInjectedInput(id, session.pendingRolePrompt);
-          term.write(session.pendingRolePrompt);
-          setTimeout(() => term.write('\r'), 150);
-          console.log(`Session ${id.slice(0, 8)}: injected role prompt`);
-          delete session.pendingRolePrompt;
-          delete session._rolePromptTimer;
-        }
-      }, 3000);
+      if (session.presetId === 'codex') {
+        if (telemetry.hasEvents(id)) injectRolePrompt();
+      } else {
+        session._rolePromptTimer = setTimeout(() => {
+          if (session.pendingRolePrompt) injectRolePrompt();
+        }, 3000);
+      }
     }
     session.chunks.push(data);
     session.chunksSize += data.length;
@@ -363,7 +372,7 @@ function restart(msg, ws, cfg) {
   }
 
   const savedToken = s.sessionToken;
-  const { name, cwd, commandId, projectId } = s;
+  const { name, cwd, commandId, projectId, roleName, muted, lastPreview, lastActivityAt } = s;
 
   activity.clear(id);
   telemetry.clear(id);
@@ -380,6 +389,14 @@ function restart(msg, ws, cfg) {
     return;
   }
 
+  const next = sessions.get(id);
+  if (next) {
+    next.roleName = roleName || null;
+    next.muted = !!muted;
+    next.lastPreview = lastPreview || '';
+    next.lastActivityAt = lastActivityAt || null;
+  }
+
   broadcast({ type: 'session.restarted', id, resumed: !!canResume });
 }
 

package/transcript-builder.js

@@ -14,7 +14,7 @@ function cleanAgentText(presetId, text) {
     out = out.replace(/\n\s*.*\(running stop hook\)[\s\S]*$/, '').trim();
     out = out.replace(/\n\s*\?\s*for shortcuts[\s\S]*$/, '').trim();
     out = out.replace(/\n\s*esc to interrupt[\s\S]*$/, '').trim();
-    out = out.replace(/\n\s*[✻✢✣✤✥✦✧]\s+[^\n]*$/, '').trim();
+    out = out.replace(/\n\n\s*[✻✢✣✤✥✦✧][\s\S]*$/, '').trim();
   }
   return out;
 }