Remove debug console.log statements from production code

Author: MBanucu

src/plugin/pty/SessionLifecycle.ts

@@ -16,7 +16,6 @@ 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()
@@ -40,12 +39,10 @@ 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, {
@@ -55,15 +52,8 @@ 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
@@ -75,15 +65,12 @@ 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()
 
@@ -154,16 +141,6 @@ 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,36 +41,30 @@ 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
   }
 
@@ -102,7 +96,6 @@ class PTYManager {
       id,
       (session) => {
         const info = this.lifecycleManager.toInfo(session)
-        console.log('Manager get returning info:', info)
         return info
       },
       null

src/web/server/handlers/sessions.ts

@@ -43,7 +43,6 @@ 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)
   }

test/pty-echo.test.ts

@@ -24,7 +24,6 @@ describe('PTY Echo Behavior', () => {
 
     // Subscribe to raw output events
     onRawOutput((_sessionId, rawData) => {
-      console.log('Received raw data:', rawData)
       receivedOutputs.push(rawData)
     })