fix: add clearRawOutputCallbacks and use in tests to prevent interference

Author: MBanucu

.github/workflows/ci.yml

@@ -48,7 +48,7 @@ jobs:
         run: bun run build:all:prod
 
       - name: Run tests
-        run: bun test --max-concurrency=1
+        run: bun test
 
   dependency-review:
     runs-on: ubuntu-24.04

src/plugin/pty/manager.ts

@@ -19,6 +19,10 @@ export function onRawOutput(callback: RawOutputCallback): void {
   rawOutputCallbacks.push(callback)
 }
 
+export function clearRawOutputCallbacks(): void {
+  rawOutputCallbacks.length = 0
+}
+
 function notifyRawOutput(sessionId: string, rawData: string): void {
   for (const callback of rawOutputCallbacks) {
     try {

test/pty-echo.test.ts

@@ -1,5 +1,10 @@
 import { describe, it, expect, beforeEach, afterEach } from 'bun:test'
-import { initManager, manager, onRawOutput } from '../src/plugin/pty/manager.ts'
+import {
+  initManager,
+  manager,
+  onRawOutput,
+  clearRawOutputCallbacks,
+} from '../src/plugin/pty/manager.ts'
 
 describe('PTY Echo Behavior', () => {
   const fakeClient = {
@@ -17,6 +22,7 @@ describe('PTY Echo Behavior', () => {
   afterEach(() => {
     // Clean up any sessions
     manager.clearAllSessions()
+    clearRawOutputCallbacks()
   })
 
   it('should echo input characters in interactive bash session', async () => {

test/web-server.test.ts

@@ -1,6 +1,6 @@
 import { describe, it, expect, beforeEach, afterEach } from 'bun:test'
 import { startWebServer, stopWebServer, getServerUrl } from '../src/web/server/server.ts'
-import { initManager, manager } from '../src/plugin/pty/manager.ts'
+import { initManager, manager, clearRawOutputCallbacks } from '../src/plugin/pty/manager.ts'
 
 describe('Web Server', () => {
   const fakeClient = {
@@ -18,6 +18,7 @@ describe('Web Server', () => {
   afterEach(() => {
     stopWebServer()
     manager.cleanupAll() // Ensure cleanup after each test
+    clearRawOutputCallbacks()
   })
 
   describe('Server Lifecycle', () => {