Add extensive console.log for debugging CI PTY failures

- Log session creation, spawn, and API responses
- Added logs in SessionLifecycle, manager, and API handlers
- This will help identify why command is undefined in CI

Author: MBanucu

src/plugin/pty/SessionLifecycle.ts

@@ -1,6 +1,6 @@
 import { spawn, type IPty } from 'bun-pty'
 import { RingBuffer } from './buffer.ts'
-import type { PTYSession, PTYSessionInfo, SpawnOptions } from './types.ts'
+import type { PTYSession, PTYSessionInfo, SpawnOptions, PTYStatus } from './types.ts'
 import { DEFAULT_TERMINAL_COLS, DEFAULT_TERMINAL_ROWS } from '../constants.ts'
 
 const SESSION_ID_BYTE_LENGTH = 4
@@ -16,29 +16,32 @@ export class SessionLifecycleManager {
   private sessions: Map<string, PTYSession> = new Map()
 
   private createSessionObject(opts: SpawnOptions): PTYSession {
+    console.log('Creating session object with opts:', opts)
     const id = generateId()
     const args = opts.args ?? []
     const workdir = opts.workdir ?? process.cwd()
     const title =
       opts.title ?? (`${opts.command} ${args.join(' ')}`.trim() || `Terminal ${id.slice(-4)}`)
 
     const buffer = new RingBuffer()
-    return {
+    const session = {
       id,
       title,
       description: opts.description,
       command: opts.command,
       args,
       workdir,
       env: opts.env,
-      status: 'running',
+      status: 'running' as PTYStatus,
       pid: 0, // will be set after spawn
       createdAt: new Date(),
       parentSessionId: opts.parentSessionId,
       notifyOnExit: opts.notifyOnExit ?? false,
       buffer,
       process: null, // will be set
     }
+    console.log('Session object created:', session)
+    return session
   }
 
   private spawnProcess(session: PTYSession): void {
@@ -55,6 +58,12 @@ export class SessionLifecycleManager {
       console.log('PTY process spawned with pid:', ptyProcess.pid)
       session.process = ptyProcess
       session.pid = ptyProcess.pid
+      console.log('Session after spawn:', {
+        id: session.id,
+        pid: session.pid,
+        command: session.command,
+        status: session.status,
+      })
     } catch (error) {
       console.error('Failed to spawn PTY process:', error)
       throw error
@@ -152,6 +161,8 @@ export class SessionLifecycleManager {
         pid: session.pid,
         status: session.status,
         process: !!session.process,
+        command: session.command,
+        args: session.args,
       })
     return session
   }

src/plugin/pty/manager.ts

@@ -100,7 +100,11 @@ class PTYManager {
     return withSession(
       this.lifecycleManager,
       id,
-      (session) => this.lifecycleManager.toInfo(session),
+      (session) => {
+        const info = this.lifecycleManager.toInfo(session)
+        console.log('Manager get returning info:', info)
+        return info
+      },
       null
     )
   }

src/web/server/handlers/sessions.ts

@@ -43,6 +43,7 @@ export async function getSession(req: BunRequest<'/api/sessions/:id'>): Promise<
     return new ErrorResponse('Invalid session ID', 400)
   }
   const session = manager.get(sessionId)
+  console.log('API getSession for id:', sessionId, 'session:', session)
   if (!session) {
     return new ErrorResponse('Session not found', 404)
   }