fix(studio-bridge): resolve tsc --build type errors

Remove unused imports/variables, add missing barrel exports (StudioState,
DataModelInstance, OutputLevel), fix mock type signatures in tests, and
import Socket from net instead of http.

Author: Quenty

tools/studio-bridge/src/bridge/bridge-session.test.ts

@@ -6,7 +6,6 @@
 import { describe, it, expect, vi } from 'vitest';
 import { EventEmitter } from 'events';
 import { BridgeSession } from './bridge-session.js';
-import type { TransportHandle } from './internal/session-tracker.js';
 import type { SessionInfo } from './types.js';
 import { SessionDisconnectedError } from './types.js';
 import type { PluginMessage, ServerMessage } from '../server/web-socket-protocol.js';
@@ -15,13 +14,10 @@ import type { PluginMessage, ServerMessage } from '../server/web-socket-protocol
 // Helpers
 // ---------------------------------------------------------------------------
 
-class MockTransportHandle extends EventEmitter implements TransportHandle {
+class MockTransportHandle extends EventEmitter {
   private _isConnected: boolean;
 
-  sendActionAsync = vi.fn(async <TResponse>(): Promise<TResponse> => {
-    return {} as TResponse;
-  });
-
+  sendActionAsync = vi.fn(async () => ({})) as any;
   sendMessage = vi.fn();
 
   constructor(connected = true) {

tools/studio-bridge/src/bridge/index.ts

@@ -26,6 +26,9 @@ export type {
   LogOptions,
   QueryDataModelOptions,
   LogFollowOptions,
+  StudioState,
+  DataModelInstance,
+  OutputLevel,
 } from './types.js';
 
 // Error classes

tools/studio-bridge/src/bridge/internal/__tests__/failover-graceful.test.ts

@@ -6,11 +6,11 @@
  * Uses real BridgeHost and HandOffManager instances with ephemeral ports.
  */
 
-import { describe, it, expect, afterEach, vi } from 'vitest';
+import { describe, it, expect, afterEach } from 'vitest';
 import { WebSocket } from 'ws';
 import { BridgeHost } from '../bridge-host.js';
-import { HandOffManager, type HandOffDependencies } from '../hand-off.js';
-import { decodeHostMessage, encodeHostMessage } from '../host-protocol.js';
+import { HandOffManager } from '../hand-off.js';
+import { decodeHostMessage } from '../host-protocol.js';
 
 // ---------------------------------------------------------------------------
 // Helpers

tools/studio-bridge/src/bridge/internal/bridge-client.test.ts

@@ -4,14 +4,12 @@
  * event handling.
  */
 
-import { describe, it, expect, vi, afterEach } from 'vitest';
+import { describe, it, expect, afterEach } from 'vitest';
 import { WebSocketServer, WebSocket } from 'ws';
 import { BridgeClient } from './bridge-client.js';
 import {
   encodeHostMessage,
   decodeHostMessage,
-  type HostEnvelope,
-  type ListSessionsRequest,
   type HostProtocolMessage,
 } from './host-protocol.js';
 import type { SessionInfo } from '../types.js';
@@ -89,7 +87,6 @@ async function createMockHostWithSessions(
   const host = await createMockHost();
 
   // Override the message handler to respond with sessions
-  const originalConnection = host.wss.listeners('connection');
   host.wss.removeAllListeners('connection');
 
   host.wss.on('connection', (ws) => {

tools/studio-bridge/src/bridge/internal/bridge-client.ts

@@ -13,19 +13,16 @@
 
 import { EventEmitter } from 'events';
 import { randomUUID } from 'crypto';
-import { TransportClient, type TransportClientOptions } from './transport-client.js';
+import { TransportClient } from './transport-client.js';
 import {
   encodeHostMessage,
   decodeHostMessage,
   type HostEnvelope,
   type HostResponse,
   type ListSessionsRequest,
   type ListSessionsResponse,
-  type ListInstancesRequest,
   type ListInstancesResponse,
   type SessionEvent,
-  type HostProtocolMessage,
-  type HostTransferNotice,
 } from './host-protocol.js';
 import { HandOffManager } from './hand-off.js';
 import type { TransportHandle } from './session-tracker.js';
@@ -140,7 +137,7 @@ export class BridgeClient extends EventEmitter {
     this._isConnected = false;
 
     // Cancel all pending requests
-    for (const [requestId, entry] of this._pendingRequests) {
+    for (const [, entry] of this._pendingRequests) {
       clearTimeout(entry.timer);
       entry.reject(new Error('Client disconnected'));
     }

tools/studio-bridge/src/bridge/internal/bridge-host.ts

@@ -7,7 +7,7 @@
 import { EventEmitter } from 'events';
 import type { WebSocket, RawData } from 'ws';
 import type { IncomingMessage } from 'http';
-import { TransportServer, type TransportServerOptions } from './transport-server.js';
+import { TransportServer } from './transport-server.js';
 import { createHealthHandler } from './health-endpoint.js';
 import { encodeHostMessage, type HostTransferNotice } from './host-protocol.js';
 import {

tools/studio-bridge/src/bridge/internal/hand-off.test.ts

@@ -4,15 +4,15 @@
  * dependencies (no real network).
  */
 
-import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 import {
   HandOffManager,
   computeTakeoverJitterMs,
   type HandOffDependencies,
   type HandOffLogEntry,
 } from './hand-off.js';
 import { HostUnreachableError } from '../types.js';
-import { createHealthHandler, type HealthInfo } from './health-endpoint.js';
+import { createHealthHandler } from './health-endpoint.js';
 
 // ---------------------------------------------------------------------------
 // Helpers
@@ -257,7 +257,7 @@ describe('HandOffManager', () => {
       await manager.onHostDisconnectedAsync();
 
       // Should have logs: host-transfer-notice, graceful-disconnect, bind-success
-      const reasons = logger.mock.calls.map((c: [HandOffLogEntry]) => c[0].reason);
+      const reasons = logger.mock.calls.map((c: any[]) => c[0].reason);
       expect(reasons).toContain('host-transfer-notice');
       expect(reasons).toContain('graceful-disconnect');
       expect(reasons).toContain('bind-success');
@@ -275,7 +275,7 @@ describe('HandOffManager', () => {
       await manager.onHostDisconnectedAsync();
       randomSpy.mockRestore();
 
-      const reasons = logger.mock.calls.map((c: [HandOffLogEntry]) => c[0].reason);
+      const reasons = logger.mock.calls.map((c: any[]) => c[0].reason);
       expect(reasons).toContain('crash-jitter');
       expect(reasons).toContain('crash-detected');
     });
@@ -297,7 +297,7 @@ describe('HandOffManager', () => {
       await manager.onHostDisconnectedAsync();
 
       const retryEntries = logger.mock.calls
-        .map((c: [HandOffLogEntry]) => c[0])
+        .map((c: any[]) => c[0])
         .filter((e: HandOffLogEntry) => e.reason === 'retry');
       expect(retryEntries.length).toBe(2);
       expect(retryEntries[0].data?.attempt).toBe(0);
@@ -316,7 +316,7 @@ describe('HandOffManager', () => {
       manager.onHostTransferNotice();
       await expect(manager.onHostDisconnectedAsync()).rejects.toThrow(HostUnreachableError);
 
-      const reasons = logger.mock.calls.map((c: [HandOffLogEntry]) => c[0].reason);
+      const reasons = logger.mock.calls.map((c: any[]) => c[0].reason);
       expect(reasons).toContain('retries-exhausted');
     });
 
@@ -371,12 +371,10 @@ describe('Health endpoint observability fields', () => {
 
     // Simulate an HTTP response object
     let statusCode = 0;
-    let headers: Record<string, string> = {};
     let body = '';
     const res = {
-      writeHead(code: number, hdrs: Record<string, string | number>) {
+      writeHead(code: number, _hdrs: Record<string, string | number>) {
         statusCode = code;
-        headers = hdrs as Record<string, string>;
       },
       end(data: string) {
         body = data;

tools/studio-bridge/src/bridge/internal/health-endpoint.test.ts

@@ -3,7 +3,7 @@
  * and the checkHealthAsync client function.
  */
 
-import { describe, it, expect, vi, afterEach } from 'vitest';
+import { describe, it, expect, afterEach } from 'vitest';
 import http from 'http';
 import { checkHealthAsync, createHealthHandler, type HealthInfo } from './health-endpoint.js';
 

tools/studio-bridge/src/bridge/internal/session-tracker.test.ts

@@ -5,8 +5,7 @@
 
 import { describe, it, expect, vi, beforeEach } from 'vitest';
 import { SessionTracker, type TransportHandle, type TrackedSession } from './session-tracker.js';
-import type { SessionInfo, InstanceInfo, SessionContext } from '../types.js';
-import type { PluginMessage, ServerMessage } from '../../server/web-socket-protocol.js';
+import type { SessionInfo, InstanceInfo } from '../types.js';
 import { EventEmitter } from 'events';
 
 // ---------------------------------------------------------------------------
@@ -19,7 +18,7 @@ function createMockHandle(connected = true): TransportHandle {
     sendActionAsync: vi.fn(async () => ({}) as any),
     sendMessage: vi.fn(),
     isConnected: connected,
-    on: emitter.on.bind(emitter) as TransportHandle['on'],
+    on: emitter.on.bind(emitter) as unknown as TransportHandle['on'],
   };
 }
 

tools/studio-bridge/src/bridge/internal/transport-server.ts

@@ -5,7 +5,8 @@
  * routes connections by URL path.
  */
 
-import { createServer, type IncomingMessage, type ServerResponse, type Server, type Socket } from 'http';
+import { createServer, type IncomingMessage, type ServerResponse, type Server } from 'http';
+import type { Socket } from 'net';
 import { WebSocketServer, WebSocket } from 'ws';
 import { URL } from 'url';