feat(conversations): expose disconnect method for clean process exit (#340)

The SessionManager already has a disconnect() method but it was not
exposed through the public ConversationService API. CLI consumers that
use startSession/endSession had no way to close the underlying WebSocket,
causing the process to hang indefinitely due to socket.io reconnection
timers keeping the event loop alive.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: dsuresh-ap

src/models/conversational-agent/conversations/conversations.models.ts

@@ -364,6 +364,20 @@ export interface ConversationServiceModel {
    * @internal
    */
   onConnectionStatusChanged(handler: ConnectionStatusChangedHandler): () => void;
+
+  /**
+   * Closes the WebSocket connection and releases all session resources.
+   *
+   * In Node.js the WebSocket keeps the event loop alive until disconnected,
+   * so call this to allow the process to exit cleanly. In the browser the
+   * runtime handles socket cleanup on page unload, so this is effectively a no-op.
+   *
+   * @example
+   * ```typescript
+   * conversationalAgent.conversations.disconnect();
+   * ```
+   */
+  disconnect(): void;
 }
 
 /**

src/services/conversational-agent/conversations/conversations.ts

@@ -471,6 +471,23 @@ export class ConversationService extends BaseService implements ConversationServ
     return this._sessionManager.onConnectionStatusChanged(handler);
   }
 
+  /**
+   * Closes the WebSocket connection and releases all session resources.
+   *
+   * In Node.js the WebSocket keeps the event loop alive until disconnected,
+   * so call this to allow the process to exit cleanly. In the browser the
+   * runtime handles socket cleanup on page unload, so this is effectively a no-op.
+   *
+   * @example
+   * ```typescript
+   * conversationalAgent.conversations.disconnect();
+   * ```
+   */
+  @track('ConversationalAgent.Conversations.Disconnect')
+  disconnect(): void {
+    this._sessionManager.disconnect();
+  }
+
   // ==================== Private Methods ====================
 
   private _getEvents(): ConversationEventHelperManager {

tests/unit/services/conversational-agent/conversations.test.ts

@@ -36,6 +36,7 @@ vi.mock('@/services/conversational-agent/conversations/session/session-manager',
     setLogLevel: vi.fn(),
     setEventDispatcher: vi.fn(),
     emitEvent: vi.fn(),
+    disconnect: vi.fn(),
   }))
 }));
 
@@ -651,5 +652,13 @@ describe('ConversationalAgent.conversations Unit Tests', () => {
       expect(cleanup).toBeDefined();
       expect(typeof cleanup).toBe('function');
     });
+
+    it('should call disconnect on the session manager', () => {
+      const sessionManager = (conversationalAgent.conversations as any)._sessionManager;
+
+      conversationalAgent.conversations.disconnect();
+
+      expect(sessionManager.disconnect).toHaveBeenCalledTimes(1);
+    });
   });
 });