orchestrix-yuri 2.3.3 → 2.4.0

package/lib/gateway/config.js

@@ -20,7 +20,7 @@ const DEFAULTS = {
   engine: {
     skill: 'yuri',
     tmux_session: 'yuri-gateway',
-    startup_timeout: 30000,       // ms to wait for Claude Code to initialize
+    startup_timeout: 60000,       // ms to wait for Claude Code to initialize
     poll_interval: 2000,          // ms between capture-pane polls
     stable_count: 3,              // consecutive stable polls before declaring done
     max_retries: 3,               // session restart retries before error

package/lib/gateway/engine/claude-tmux.js

@@ -143,24 +143,28 @@ function tmuxSafe(cmd) {
 
 // ── Claude Code TUI Indicators ─────────────────────────────────────────────────
 //
-// Claude Code uses three circle symbols as primary state indicators:
+// Claude Code TUI state indicators vary by version and statusline config:
 //
-//   ○  (U+25CB) IDLE       — Claude is waiting for user input
-//   ●  (U+25CF) PROCESSING — Claude is actively generating a response
-//   ◐  (U+25D0) APPROVAL   — Claude is waiting for permission approval
+//   Idle indicators (any of these = ready for input):
+//     ○  (U+25CB) — circle idle indicator (shown with certain statusline configs)
+//     ❯           — prompt cursor (always shown when idle, most reliable)
 //
-// During processing, a Braille spinner animates: ⠋ ⠙ ⠹ ⠸ ⠼ ⠴ ⠦ ⠧ ⠇ ⠏
-// with cycling verbs like "Baking...", "Computing...", "Thinking..."
+//   Processing indicators:
+//     ●  (U+25CF) — filled circle, active generation
+//     Braille spinner: ⠋ ⠙ ⠹ ⠸ ⠼ ⠴ ⠦ ⠧ ⠇ ⠏
 //
-// Completion message format (past-tense verb + duration):
-//   "Baked for 31s", "Worked for 2m 45s", "Cooked for 1m 6s"
-//   Pattern: /[A-Z][a-z]*ed for \d+/
+//   Status line elements (NOT state indicators):
+//     ◐  — effort level indicator (e.g. "◐ medium · /effort"), NOT approval prompt
+//
+//   Completion message (past-tense verb + duration):
+//     "Baked for 31s", "Worked for 2m 45s"
+//     Pattern: /[A-Z][a-z]*ed for \d+/
 //
 // ────────────────────────────────────────────────────────────────────────────────
 
 const BRAILLE_SPINNER = /[⠋⠙⠹⠸⠼⠴⠦⠧⠇⠏]/;
 const COMPLETION_RE = /[A-Z][a-z]*ed for \d+/;
-const IDLE_RE = /○/;
+const IDLE_RE = /[○❯]/;
 const PROCESSING_RE = /●/;
 
 /**
@@ -202,10 +206,6 @@ function paneTail(name, n) {
 /**
  * Detect Claude Code's current state from pane output.
  *
- * Note: We launch with --dangerously-skip-permissions, so approval prompts (◐)
- * should never appear. The detection is kept for robustness but no auto-approve
- * action is taken — sending blind 'y' keystrokes is dangerous.
- *
  * @returns {'idle'|'processing'|'complete'|'unknown'}
  */
 function detectState(name) {
@@ -325,32 +325,43 @@ async function createSession(engineConfig) {
 
   const binary = getClaudeBinary();
   const projectRoot = resolveProjectRoot() || os.homedir();
+  log.tmux(`Binary: ${binary}`);
+  log.tmux(`Project root: ${projectRoot}`);
 
   // Ensure CLAUDE.md has channel mode instructions (survives compact)
   ensureClaudeMd(projectRoot);
 
-  // Kill existing stale session
+  // Only kill existing session if it exists — don't kill on retry
+  // so the user can `tmux attach -t yuri-gateway` to debug
   if (hasSession(sessionName)) {
+    log.tmux(`Killing existing session "${sessionName}"`);
     tmuxSafe(`kill-session -t ${sessionName}`);
   }
 
   // Create session with generous scrollback
   tmux(`new-session -d -s ${sessionName} -n claude -c "${projectRoot}"`);
   tmux(`set-option -t ${sessionName} history-limit ${HISTORY_LIMIT}`);
+  log.tmux(`Session "${sessionName}" created, launching Claude Code...`);
 
   // Set auto-compact threshold to 80% (default is 95%)
-  // This gives comfortable buffer before context pressure
   const compactPct = engineConfig.autocompact_pct || 80;
   tmux(`send-keys -t ${sessionName}:0 'export CLAUDE_AUTOCOMPACT_PCT_OVERRIDE=${compactPct}' Enter`);
 
   // Launch Claude Code in interactive mode
   tmux(`send-keys -t ${sessionName}:0 '"${binary}" --dangerously-skip-permissions' Enter`);
 
-  // Wait for Claude Code to initialize (detect idle indicator)
-  const startupTimeout = engineConfig.startup_timeout || 30000;
+  // Wait for Claude Code to initialize (detect idle indicator ○)
+  // Default 60s — Claude Code needs time to load CLAUDE.md, connect MCP servers, etc.
+  const startupTimeout = engineConfig.startup_timeout || 60000;
+  log.tmux(`Waiting for Claude Code to become idle (timeout: ${startupTimeout / 1000}s)...`);
   const started = await waitForIdle(sessionName, startupTimeout);
   if (!started) {
-    throw new Error(`Claude Code did not become idle within ${startupTimeout}ms`);
+    // Don't kill session on failure — let user debug with tmux attach
+    const tail = paneTail(sessionName, 10);
+    log.error(`Claude Code did not become idle within ${startupTimeout / 1000}s`);
+    log.error(`Last pane output:\n${tail}`);
+    log.info(`Debug: tmux attach -t ${sessionName}`);
+    throw new Error(`Claude Code did not become idle within ${startupTimeout / 1000}s`);
   }
 
   // Send L1 context as the initial system message.
@@ -358,6 +369,7 @@ async function createSession(engineConfig) {
   // so we only inject L1 global memory here to prime the session.
   const l1 = loadL1Context();
   if (l1) {
+    log.tmux('Injecting L1 context...');
     await injectMessage(sessionName, l1);
     await waitForIdle(sessionName, 120000); // allow up to 2min for L1 processing
   }
@@ -545,8 +557,14 @@ async function ensureSession(engineConfig) {
         return;
       } catch (err) {
         log.warn(`Session init attempt ${attempt}/${maxRetries} failed: ${err.message}`);
-        if (attempt === maxRetries) throw err;
-        // Brief pause before retry
+        if (attempt === maxRetries) {
+          log.error('All init attempts failed. Check Claude Code installation and tmux.');
+          log.info(`Debug: tmux attach -t ${engineConfig.tmux_session || DEFAULT_SESSION}`);
+          throw err;
+        }
+        // Kill session before retry so createSession starts fresh
+        const sn = engineConfig.tmux_session || DEFAULT_SESSION;
+        if (hasSession(sn)) tmuxSafe(`kill-session -t ${sn}`);
         await new Promise((r) => setTimeout(r, 3000));
       }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "orchestrix-yuri",
-  "version": "2.3.3",
+  "version": "2.4.0",
   "description": "Yuri — Meta-Orchestrator for Orchestrix. Drive your entire project lifecycle with natural language.",
   "main": "lib/installer.js",
   "bin": {