feat(debug): re-add console.log for CI PTY debugging

Author: MBanucu

src/plugin/pty/SessionLifecycle.ts

@@ -16,6 +16,7 @@ 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()
@@ -39,10 +40,12 @@ export class SessionLifecycleManager {
       buffer,
       process: null, // will be set
     }
+    console.log('Session object created:', session)
     return session
   }
 
   private spawnProcess(session: PTYSession): void {
+    console.log('Spawning PTY process for command:', session.command, 'args:', session.args)
     const env = { ...process.env, ...session.env } as Record<string, string>
     try {
       const ptyProcess: IPty = spawn(session.command, session.args, {
@@ -52,8 +55,15 @@ export class SessionLifecycleManager {
         cwd: session.workdir,
         env,
       })
+      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
@@ -65,12 +75,15 @@ export class SessionLifecycleManager {
     onData: (id: string, data: string) => void,
     onExit: (id: string, exitCode: number | null) => void
   ): void {
+    console.log('Setting up event handlers for session:', session.id)
     session.process!.onData((data: string) => {
+      console.log('PTY onData for session', session.id, 'data length:', data.length)
       session.buffer.append(data)
       onData(session.id, data)
     })
 
     session.process!.onExit(({ exitCode }) => {
+      console.log('PTY onExit for session', session.id, 'exitCode:', exitCode)
       // Flush any remaining incomplete line in the buffer
       session.buffer.flush()
 
@@ -141,6 +154,16 @@ export class SessionLifecycleManager {
 
   getSession(id: string): PTYSession | null {
     const session = this.sessions.get(id) || null
+    console.log('SessionLifecycle getSession for id:', id, 'found:', !!session)
+    if (session)
+      console.log('Session details:', {
+        id: session.id,
+        pid: session.pid,
+        status: session.status,
+        process: !!session.process,
+        command: session.command,
+        args: session.args,
+      })
     return session
   }
 

src/plugin/pty/manager.ts

@@ -41,30 +41,36 @@ class PTYManager {
   }
 
   spawn(opts: SpawnOptions): PTYSessionInfo {
+    console.log('Manager spawn called with opts:', opts)
     const session = this.lifecycleManager.spawn(
       opts,
       (id, data) => {
+        console.log('Manager onData callback for id:', id, 'data length:', data.length)
         notifyRawOutput(id, data)
       },
       async (id, exitCode) => {
+        console.log('Manager onExit callback for id:', id, 'exitCode:', exitCode)
         if (onSessionUpdate) onSessionUpdate()
         const session = this.lifecycleManager.getSession(id)
         if (session && session.notifyOnExit) {
           await this.notificationManager.sendExitNotification(session, exitCode || 0)
         }
       }
     )
+    console.log('Manager spawn returning session:', session)
     if (onSessionUpdate) onSessionUpdate()
     return session
   }
 
   write(id: string, data: string): boolean {
+    console.log('Manager write called for id:', id, 'data:', data)
     const result = withSession(
       this.lifecycleManager,
       id,
       (session) => this.outputManager.write(session, data),
       false
     )
+    console.log('Manager write result:', result)
     return result
   }
 
@@ -96,6 +102,7 @@ class PTYManager {
       id,
       (session) => {
         const info = this.lifecycleManager.toInfo(session)
+        console.log('Manager get returning info:', info)
         return info
       },
       null