| 12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214121512161217121812191220122112221223122412251226122712281229123012311232123312341235123612371238123912401241124212431244124512461247124812491250125112521253125412551256125712581259126012611262126312641265126612671268126912701271127212731274127512761277127812791280128112821283128412851286128712881289129012911292129312941295129612971298129913001301130213031304130513061307130813091310131113121313131413151316131713181319132013211322132313241325132613271328132913301331133213331334133513361337133813391340134113421343134413451346134713481349135013511352135313541355135613571358135913601361136213631364136513661367136813691370137113721373137413751376137713781379138013811382138313841385138613871388138913901391139213931394139513961397139813991400140114021403140414051406140714081409141014111412141314141415141614171418141914201421142214231424142514261427142814291430143114321433143414351436143714381439144014411442144314441445144614471448144914501451145214531454145514561457145814591460146114621463146414651466146714681469 |
- import { afterEach, beforeEach, describe, expect, mock, test } from 'bun:test';
- import { BackgroundJobBoard } from '../utils/background-job-board';
- import { BackgroundJobCoordinator } from '../utils/background-job-coordinator';
- import {
- MultiplexerSessionManager,
- resetMultiplexerSessionManagerState,
- } from './session-manager';
- const originalFetch = globalThis.fetch;
- let mockSessionStatuses: Record<string, { type: string }> = {};
- const mockFetch = mock(
- async () =>
- new Response(JSON.stringify(mockSessionStatuses), {
- headers: { 'content-type': 'application/json' },
- status: 200,
- }),
- );
- // Define the mock multiplexer
- const mockMultiplexer = {
- type: 'tmux' as const,
- isAvailable: mock(async () => true),
- isInsideSession: mock(() => true),
- spawnPane: mock(async () => ({
- success: true,
- paneId: '%mock-pane',
- })),
- closePane: mock(async () => true),
- applyLayout: mock(async () => {}),
- };
- // Mock the multiplexer module
- mock.module('../multiplexer', () => ({
- getMultiplexer: () => mockMultiplexer,
- isServerRunning: mock(async () => true),
- startAvailabilityCheck: () => {},
- }));
- // Mock the plugin context
- function createMockContext(overrides?: {
- sessionStatusResult?: { data?: Record<string, { type: string }> };
- directory?: string;
- serverUrl?: string;
- }) {
- const defaultPort = process.env.OPENCODE_PORT ?? '4096';
- return {
- client: {
- session: {
- status: mock(
- async () => overrides?.sessionStatusResult ?? { data: {} },
- ),
- },
- },
- directory: overrides?.directory ?? '/test/directory',
- serverUrl: new URL(
- overrides?.serverUrl ?? `http://localhost:${defaultPort}`,
- ),
- } as any;
- }
- function setMockSessionStatuses(statuses: Record<string, { type: string }>) {
- mockSessionStatuses = statuses;
- }
- const defaultMultiplexerConfig = {
- type: 'tmux' as const,
- layout: 'main-vertical' as const,
- main_pane_size: 60,
- zellij_pane_mode: 'agent-tab' as const,
- };
- function createDeferred<T>() {
- let resolve!: (value: T) => void;
- let reject!: (reason?: unknown) => void;
- const promise = new Promise<T>((res, rej) => {
- resolve = res;
- reject = rej;
- });
- return { promise, resolve, reject };
- }
- describe('MultiplexerSessionManager', () => {
- beforeEach(() => {
- resetMultiplexerSessionManagerState();
- mockSessionStatuses = {};
- mockFetch.mockClear();
- globalThis.fetch = mockFetch as typeof fetch;
- mockMultiplexer.spawnPane.mockReset();
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: '%mock-pane',
- });
- mockMultiplexer.closePane.mockReset();
- mockMultiplexer.closePane.mockResolvedValue(true);
- mockMultiplexer.isInsideSession.mockReset();
- mockMultiplexer.isInsideSession.mockReturnValue(true);
- });
- afterEach(() => {
- globalThis.fetch = originalFetch;
- });
- describe('constructor', () => {
- test('initializes with config', () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- expect(manager).toBeDefined();
- });
- });
- describe('onSessionCreated', () => {
- test('spawns pane for child sessions', async () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: {
- id: 'child-123',
- parentID: 'parent-456',
- title: 'Test Worker',
- },
- },
- });
- expect(mockMultiplexer.spawnPane).toHaveBeenCalled();
- expect(mockMultiplexer.spawnPane).toHaveBeenCalledWith(
- 'child-123',
- 'Test Worker',
- `http://localhost:${process.env.OPENCODE_PORT ?? '4096'}/`,
- '/test/directory',
- );
- });
- test('ignores sessions without parentID', async () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: {
- id: 'root-session',
- title: 'Main Chat',
- },
- },
- });
- expect(mockMultiplexer.spawnPane).not.toHaveBeenCalled();
- });
- test('prefers child session directory when present', async () => {
- const ctx = createMockContext({ directory: '/parent/directory' });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: {
- id: 'child-456',
- parentID: 'parent-456',
- title: 'Nested Worker',
- directory: '/child/directory',
- },
- },
- });
- expect(mockMultiplexer.spawnPane).toHaveBeenCalledWith(
- 'child-456',
- 'Nested Worker',
- `http://localhost:${process.env.OPENCODE_PORT ?? '4096'}/`,
- '/child/directory',
- );
- });
- test('ignores if disabled in config', async () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(ctx, {
- ...defaultMultiplexerConfig,
- type: 'none',
- });
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: { id: 'child', parentID: 'parent' },
- },
- });
- expect(mockMultiplexer.spawnPane).not.toHaveBeenCalled();
- });
- test('does not spawn twice for duplicate create events while spawning', async () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- const deferred = createDeferred<{ success: true; paneId: string }>();
- mockMultiplexer.spawnPane.mockImplementationOnce(() => deferred.promise);
- const event = {
- type: 'session.created',
- properties: {
- info: {
- id: 'child-race',
- parentID: 'parent-race',
- title: 'Race Worker',
- },
- },
- };
- const firstCreate = manager.onSessionCreated(event);
- const secondCreate = manager.onSessionCreated(event);
- await Promise.resolve();
- expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
- deferred.resolve({ success: true, paneId: 'p-race' });
- await Promise.all([firstCreate, secondCreate]);
- expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
- });
- });
- describe('polling and closure', () => {
- test('closes pane when session becomes idle', async () => {
- const ctx = createMockContext();
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-1',
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- // Register session
- await manager.onSessionCreated({
- type: 'session.created',
- properties: { info: { id: 'c1', parentID: 'p1' } },
- });
- setMockSessionStatuses({ c1: { type: 'idle' } });
- await (manager as any).pollSessions();
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith('p-1');
- });
- test('closes pane immediately on session.idle event', async () => {
- const ctx = createMockContext();
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-idle-event',
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- await manager.onSessionCreated({
- type: 'session.created',
- properties: { info: { id: 'idle-event-child', parentID: 'parent' } },
- });
- await manager.onSessionStatus({
- type: 'session.idle',
- properties: { sessionID: 'idle-event-child' },
- });
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith('p-idle-event');
- expect(mockFetch).not.toHaveBeenCalled();
- });
- test('does not close another manager instance pane on idle event', async () => {
- const ctx = createMockContext();
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-shared-idle',
- });
- const spawningManager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- const idleManager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- await spawningManager.onSessionCreated({
- type: 'session.created',
- properties: { info: { id: 'shared-child', parentID: 'parent' } },
- });
- await idleManager.onSessionStatus({
- type: 'session.idle',
- properties: { sessionID: 'shared-child' },
- });
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- });
- test('respawns resumed known session from a different manager instance', async () => {
- const ctx = createMockContext();
- mockMultiplexer.spawnPane
- .mockResolvedValueOnce({
- success: true,
- paneId: 'p-first',
- })
- .mockResolvedValueOnce({
- success: true,
- paneId: 'p-resumed',
- });
- const firstManager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- const secondManager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- await firstManager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: {
- id: 'resumed-child',
- parentID: 'parent',
- title: 'Resumed Worker',
- directory: '/resumed/dir',
- },
- },
- });
- await firstManager.onSessionStatus({
- type: 'session.idle',
- properties: { sessionID: 'resumed-child' },
- });
- await secondManager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'resumed-child',
- status: { type: 'busy' },
- },
- });
- expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(2);
- expect(mockMultiplexer.spawnPane).toHaveBeenLastCalledWith(
- 'resumed-child',
- 'Resumed Worker',
- `http://localhost:${process.env.OPENCODE_PORT ?? '4096'}/`,
- '/resumed/dir',
- );
- });
- test('does not close running background child pane on idle event', async () => {
- const ctx = createMockContext();
- const board = new BackgroundJobBoard();
- board.registerLaunch({
- taskID: 'running-idle-child',
- parentSessionID: 'parent-1',
- agent: 'explorer',
- });
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-running-idle-child',
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- board,
- );
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: { id: 'running-idle-child', parentID: 'parent-1' },
- },
- });
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'running-idle-child',
- status: { type: 'idle' },
- },
- });
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- });
- test('timed out running jobs still close after safe recovery and completion', async () => {
- const ctx = createMockContext();
- const board = new BackgroundJobBoard();
- const coordinator = new BackgroundJobCoordinator(board);
- board.registerLaunch({
- taskID: 'timedout-child',
- parentSessionID: 'parent-1',
- agent: 'explorer',
- });
- board.updateStatus({
- taskID: 'timedout-child',
- state: 'running',
- timedOut: true,
- now: 100,
- });
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-timedout-child',
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- coordinator,
- );
- coordinator.addTerminalStateListener((sessionId) => {
- void manager.closeSessionFromCoordinator(sessionId);
- });
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: { id: 'timedout-child', parentID: 'parent-1' },
- },
- });
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'timedout-child',
- status: { type: 'idle' },
- },
- });
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'timedout-child',
- status: { type: 'busy' },
- },
- });
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- board.updateStatus({
- taskID: 'timedout-child',
- state: 'completed',
- resultSummary: 'done',
- });
- await Promise.resolve();
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
- 'p-timedout-child',
- );
- });
- test('deferred idle closes retry on terminal status updates', async () => {
- for (const state of ['completed', 'error', 'cancelled'] as const) {
- resetMultiplexerSessionManagerState();
- mockMultiplexer.spawnPane.mockClear();
- mockMultiplexer.closePane.mockClear();
- const ctx = createMockContext();
- const board = new BackgroundJobBoard();
- const coordinator = new BackgroundJobCoordinator(board);
- const sessionId = `deferred-${state}`;
- board.registerLaunch({
- taskID: sessionId,
- parentSessionID: 'parent-1',
- agent: 'explorer',
- });
- mockMultiplexer.spawnPane.mockResolvedValueOnce({
- success: true,
- paneId: `p-${state}`,
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- coordinator,
- );
- coordinator.addTerminalStateListener((sessionId) => {
- void manager.closeSessionFromCoordinator(sessionId);
- });
- await manager.onSessionCreated({
- type: 'session.created',
- properties: { info: { id: sessionId, parentID: 'parent-1' } },
- });
- await manager.onSessionStatus({
- type: 'session.status',
- properties: { sessionID: sessionId, status: { type: 'idle' } },
- });
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- board.updateStatus({ taskID: sessionId, state });
- await Promise.resolve();
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith(`p-${state}`);
- }
- });
- test('deferred idle close retries on markCancelled', async () => {
- const ctx = createMockContext();
- const board = new BackgroundJobBoard();
- const coordinator = new BackgroundJobCoordinator(board);
- board.registerLaunch({
- taskID: 'deferred-cancel',
- parentSessionID: 'parent-1',
- agent: 'explorer',
- });
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-deferred-cancel',
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- coordinator,
- );
- coordinator.addTerminalStateListener((sessionId) => {
- void manager.closeSessionFromCoordinator(sessionId);
- });
- await manager.onSessionCreated({
- type: 'session.created',
- properties: { info: { id: 'deferred-cancel', parentID: 'parent-1' } },
- });
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'deferred-cancel',
- status: { type: 'idle' },
- },
- });
- board.markCancelled('deferred-cancel');
- await Promise.resolve();
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
- 'p-deferred-cancel',
- );
- });
- test('terminal status without deferred idle close does not close pane', async () => {
- const ctx = createMockContext();
- const board = new BackgroundJobBoard();
- const coordinator = new BackgroundJobCoordinator(board);
- board.registerLaunch({
- taskID: 'terminal-without-defer',
- parentSessionID: 'parent-1',
- agent: 'explorer',
- });
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-terminal-without-defer',
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- coordinator,
- );
- coordinator.addTerminalStateListener((sessionId) => {
- void manager.closeSessionFromCoordinator(sessionId);
- });
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: { id: 'terminal-without-defer', parentID: 'parent-1' },
- },
- });
- board.updateStatus({
- taskID: 'terminal-without-defer',
- state: 'completed',
- });
- await Promise.resolve();
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- });
- test('deleted clears deferred idle close and later terminal update is no-op', async () => {
- const ctx = createMockContext();
- const board = new BackgroundJobBoard();
- const coordinator = new BackgroundJobCoordinator(board);
- board.registerLaunch({
- taskID: 'deleted-deferred',
- parentSessionID: 'parent-1',
- agent: 'explorer',
- });
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-deleted-deferred',
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- coordinator,
- );
- coordinator.addTerminalStateListener((sessionId) => {
- void manager.closeSessionFromCoordinator(sessionId);
- });
- await manager.onSessionCreated({
- type: 'session.created',
- properties: { info: { id: 'deleted-deferred', parentID: 'parent-1' } },
- });
- await manager.onSessionStatus({
- type: 'session.status',
- properties: { sessionID: 'deleted-deferred', status: { type: 'idle' } },
- });
- await manager.onSessionDeleted({
- type: 'session.deleted',
- properties: { sessionID: 'deleted-deferred' },
- });
- board.updateStatus({ taskID: 'deleted-deferred', state: 'completed' });
- await Promise.resolve();
- expect(mockMultiplexer.closePane).toHaveBeenCalledTimes(1);
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
- 'p-deleted-deferred',
- );
- });
- test('retry while still running keeps deferred idle close', async () => {
- const ctx = createMockContext();
- const board = new BackgroundJobBoard();
- const coordinator = new BackgroundJobCoordinator(board);
- board.registerLaunch({
- taskID: 'still-running-deferred',
- parentSessionID: 'parent-1',
- agent: 'explorer',
- });
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-still-running-deferred',
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- coordinator,
- );
- coordinator.addTerminalStateListener((sessionId) => {
- void manager.closeSessionFromCoordinator(sessionId);
- });
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: { id: 'still-running-deferred', parentID: 'parent-1' },
- },
- });
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'still-running-deferred',
- status: { type: 'idle' },
- },
- });
- // The coordinator's terminal state listener will handle the close
- // when the job completes, so we don't need to call retryDeferredIdleClose directly
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- board.updateStatus({
- taskID: 'still-running-deferred',
- state: 'completed',
- });
- await Promise.resolve();
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
- 'p-still-running-deferred',
- );
- });
- test('disabled manager does not retry deferred idle close', async () => {
- const ctx = createMockContext();
- const board = new BackgroundJobBoard();
- const coordinator = new BackgroundJobCoordinator(board);
- board.registerLaunch({
- taskID: 'disabled-retry-deferred',
- parentSessionID: 'parent-1',
- agent: 'explorer',
- });
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-disabled-retry-deferred',
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- coordinator,
- );
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: { id: 'disabled-retry-deferred', parentID: 'parent-1' },
- },
- });
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'disabled-retry-deferred',
- status: { type: 'idle' },
- },
- });
- mockMultiplexer.isInsideSession.mockReturnValue(false);
- const _disabledManager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- coordinator,
- );
- // The coordinator's terminal state listener will handle the close
- // when the job completes, but the disabled manager should not close
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- });
- test('explicit non-idle status event clears stale deferred idle close', async () => {
- const ctx = createMockContext();
- const board = new BackgroundJobBoard();
- const coordinator = new BackgroundJobCoordinator(board);
- board.registerLaunch({
- taskID: 'retry-event-deferred',
- parentSessionID: 'parent-1',
- agent: 'explorer',
- });
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-retry-event-deferred',
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- coordinator,
- );
- coordinator.addTerminalStateListener((sessionId) => {
- void manager.closeSessionFromCoordinator(sessionId);
- });
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: { id: 'retry-event-deferred', parentID: 'parent-1' },
- },
- });
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'retry-event-deferred',
- status: { type: 'idle' },
- },
- });
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'retry-event-deferred',
- status: { type: 'retry' },
- },
- });
- board.updateStatus({
- taskID: 'retry-event-deferred',
- state: 'completed',
- });
- await Promise.resolve();
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'retry-event-deferred',
- status: { type: 'idle' },
- },
- });
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
- 'p-retry-event-deferred',
- );
- });
- test('explicit non-idle poll clears stale deferred idle close', async () => {
- const ctx = createMockContext();
- const board = new BackgroundJobBoard();
- const coordinator = new BackgroundJobCoordinator(board);
- board.registerLaunch({
- taskID: 'resumed-deferred',
- parentSessionID: 'parent-1',
- agent: 'explorer',
- });
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-resumed-deferred',
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- coordinator,
- );
- coordinator.addTerminalStateListener((sessionId) => {
- void manager.closeSessionFromCoordinator(sessionId);
- });
- await manager.onSessionCreated({
- type: 'session.created',
- properties: { info: { id: 'resumed-deferred', parentID: 'parent-1' } },
- });
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'resumed-deferred',
- status: { type: 'idle' },
- },
- });
- setMockSessionStatuses({ 'resumed-deferred': { type: 'busy' } });
- await (manager as any).pollSessions();
- board.updateStatus({ taskID: 'resumed-deferred', state: 'completed' });
- await Promise.resolve();
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'resumed-deferred',
- status: { type: 'idle' },
- },
- });
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
- 'p-resumed-deferred',
- );
- });
- test('does not close on transient status absence', async () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- await manager.onSessionCreated({
- type: 'session.created',
- properties: { info: { id: 'c1', parentID: 'p1' } },
- });
- setMockSessionStatuses({});
- await (manager as any).pollSessions();
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- });
- test('keeps background child pane open while status is running until deleted', async () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- mockMultiplexer.spawnPane.mockResolvedValueOnce({
- success: true,
- paneId: 'p-background-child',
- });
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: {
- id: 'background-child',
- parentID: 'parent-1',
- title: 'Background Worker',
- },
- },
- });
- setMockSessionStatuses({ 'background-child': { type: 'running' } });
- await (manager as any).pollSessions();
- await (manager as any).pollSessions();
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- await manager.onSessionDeleted({
- type: 'session.deleted',
- properties: { info: { id: 'background-child' } },
- });
- expect(mockMultiplexer.closePane).toHaveBeenCalledTimes(1);
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
- 'p-background-child',
- );
- });
- test('missing status does not close never-seen pane', async () => {
- const ctx = createMockContext();
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-never-seen-orphan',
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- await manager.onSessionCreated({
- type: 'session.created',
- properties: { info: { id: 'never-seen-orphan', parentID: 'p1' } },
- });
- setMockSessionStatuses({});
- await (manager as any).pollSessions();
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- });
- test('ignores empty session status response without closing panes', async () => {
- const ctx = createMockContext();
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-empty-status',
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- await manager.onSessionCreated({
- type: 'session.created',
- properties: { info: { id: 'empty-status', parentID: 'p1' } },
- });
- mockFetch.mockImplementationOnce(
- async () => new Response('', { status: 200 }),
- );
- await (manager as any).pollSessions();
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- });
- test('previously seen then missing does not close pane', async () => {
- const ctx = createMockContext();
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-seen-before-missing',
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- await manager.onSessionCreated({
- type: 'session.created',
- properties: { info: { id: 'seen-before-missing', parentID: 'p1' } },
- });
- setMockSessionStatuses({ 'seen-before-missing': { type: 'busy' } });
- await (manager as any).pollSessions();
- setMockSessionStatuses({});
- await (manager as any).pollSessions();
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- });
- test('missing then busy does not duplicate respawn', async () => {
- const ctx = createMockContext();
- mockMultiplexer.spawnPane.mockResolvedValue({
- success: true,
- paneId: 'p-missing-then-busy',
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- await manager.onSessionCreated({
- type: 'session.created',
- properties: { info: { id: 'missing-then-busy', parentID: 'p1' } },
- });
- setMockSessionStatuses({});
- await (manager as any).pollSessions();
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'missing-then-busy',
- status: { type: 'busy' },
- },
- });
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
- });
- test('polls the actual serverUrl instead of the plugin SDK default URL', async () => {
- const ctx = createMockContext({
- serverUrl: 'http://127.0.0.1:63871/',
- sessionStatusResult: { data: {} },
- });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- await manager.onSessionCreated({
- type: 'session.created',
- properties: { info: { id: 'child-live', parentID: 'parent-live' } },
- });
- setMockSessionStatuses({ 'child-live': { type: 'busy' } });
- await (manager as any).pollSessions();
- expect(ctx.client.session.status).not.toHaveBeenCalled();
- expect(mockFetch).toHaveBeenCalledWith(
- new URL('http://127.0.0.1:63871/session/status'),
- expect.any(Object),
- );
- expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
- });
- test('respawns pane on later busy after idle close for resumable session', async () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- mockMultiplexer.spawnPane
- .mockResolvedValueOnce({
- success: true,
- paneId: 'p-1',
- })
- .mockResolvedValueOnce({
- success: true,
- paneId: 'p-2',
- });
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: {
- id: 'child-789',
- parentID: 'parent-789',
- title: 'Worker',
- directory: '/task/dir',
- },
- },
- });
- setMockSessionStatuses({ 'child-789': { type: 'idle' } });
- await (manager as any).pollSessions();
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'child-789',
- status: { type: 'busy' },
- },
- });
- expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(2);
- expect(mockMultiplexer.spawnPane).toHaveBeenCalledWith(
- 'child-789',
- 'Worker',
- `http://localhost:${process.env.OPENCODE_PORT ?? '4096'}/`,
- '/task/dir',
- );
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith('p-1');
- expect(mockMultiplexer.closePane).toHaveBeenCalledTimes(1);
- });
- test('respawns after in-flight idle close when busy resumes same session', async () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- const closeDeferred = createDeferred<boolean>();
- mockMultiplexer.spawnPane
- .mockResolvedValueOnce({
- success: true,
- paneId: 'p-close-race',
- })
- .mockResolvedValueOnce({
- success: true,
- paneId: 'p-close-race-resumed',
- });
- mockMultiplexer.closePane.mockImplementationOnce(
- () => closeDeferred.promise,
- );
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: {
- id: 'child-close-race',
- parentID: 'parent-close-race',
- title: 'Worker',
- },
- },
- });
- const idlePromise = manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'child-close-race',
- status: { type: 'idle' },
- },
- });
- await Promise.resolve();
- const busyPromise = manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'child-close-race',
- status: { type: 'busy' },
- },
- });
- expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
- closeDeferred.resolve(true);
- await Promise.all([idlePromise, busyPromise]);
- expect(mockMultiplexer.closePane).toHaveBeenCalledTimes(1);
- expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(2);
- expect(mockMultiplexer.spawnPane).toHaveBeenLastCalledWith(
- 'child-close-race',
- 'Worker',
- `http://localhost:${process.env.OPENCODE_PORT ?? '4096'}/`,
- '/test/directory',
- );
- });
- test('does not respawn after in-flight close if session is deleted', async () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- const closeDeferred = createDeferred<boolean>();
- mockMultiplexer.spawnPane
- .mockResolvedValueOnce({
- success: true,
- paneId: 'p-delete-race',
- })
- .mockResolvedValueOnce({
- success: true,
- paneId: 'p-should-not-respawn',
- });
- mockMultiplexer.closePane.mockImplementationOnce(
- () => closeDeferred.promise,
- );
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: {
- id: 'child-delete-race',
- parentID: 'parent-delete-race',
- title: 'Worker',
- },
- },
- });
- const idlePromise = manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'child-delete-race',
- status: { type: 'idle' },
- },
- });
- await Promise.resolve();
- const busyPromise = manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'child-delete-race',
- status: { type: 'busy' },
- },
- });
- const deletedPromise = manager.onSessionDeleted({
- type: 'session.deleted',
- properties: {
- sessionID: 'child-delete-race',
- },
- });
- closeDeferred.resolve(true);
- await Promise.all([idlePromise, busyPromise, deletedPromise]);
- expect(mockMultiplexer.closePane).toHaveBeenCalledTimes(1);
- expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
- });
- test('closes pane on session.deleted using info.id', async () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- mockMultiplexer.spawnPane.mockResolvedValueOnce({
- success: true,
- paneId: 'p-info-id',
- });
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: {
- id: 'child-info-id',
- parentID: 'parent-info-id',
- },
- },
- });
- await manager.onSessionDeleted({
- type: 'session.deleted',
- properties: {
- info: { id: 'child-info-id' },
- },
- });
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith('p-info-id');
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'child-info-id',
- status: { type: 'busy' },
- },
- });
- expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
- });
- test('closes deleted pane even when current instance is not owner', async () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- mockMultiplexer.spawnPane.mockResolvedValueOnce({
- success: true,
- paneId: 'p-non-owner-delete',
- });
- await manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: {
- id: 'child-non-owner-delete',
- parentID: 'parent-non-owner-delete',
- },
- },
- });
- const tracked = (manager as any).sessions.get('child-non-owner-delete');
- tracked.ownerInstanceId = 'other-instance';
- await manager.onSessionDeleted({
- type: 'session.deleted',
- properties: { sessionID: 'child-non-owner-delete' },
- });
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
- 'p-non-owner-delete',
- );
- expect((manager as any).sessions.has('child-non-owner-delete')).toBe(
- false,
- );
- });
- test('closes pane returned by a stale spawn after session deleted', async () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- const spawnDeferred = createDeferred<{ success: true; paneId: string }>();
- mockMultiplexer.spawnPane.mockImplementationOnce(
- () => spawnDeferred.promise,
- );
- const createPromise = manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: {
- id: 'child-stale-spawn',
- parentID: 'parent-stale-spawn',
- },
- },
- });
- await Promise.resolve();
- await manager.onSessionDeleted({
- type: 'session.deleted',
- properties: {
- info: { id: 'child-stale-spawn' },
- },
- });
- spawnDeferred.resolve({ success: true, paneId: 'p-stale-spawn' });
- await createPromise;
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith('p-stale-spawn');
- });
- test('does nothing on busy for unknown session', async () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'unknown-session',
- status: { type: 'busy' },
- },
- });
- expect(mockMultiplexer.spawnPane).not.toHaveBeenCalled();
- });
- test('does not respawn while initial pane spawn is still in progress', async () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- const deferred = createDeferred<{ success: true; paneId: string }>();
- mockMultiplexer.spawnPane.mockImplementationOnce(() => deferred.promise);
- const createPromise = manager.onSessionCreated({
- type: 'session.created',
- properties: {
- info: {
- id: 'child-busy-race',
- parentID: 'parent-busy-race',
- title: 'Busy Worker',
- directory: '/task/dir',
- },
- },
- });
- await manager.onSessionStatus({
- type: 'session.status',
- properties: {
- sessionID: 'child-busy-race',
- status: { type: 'busy' },
- },
- });
- expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
- deferred.resolve({ success: true, paneId: 'p-busy-race' });
- await createPromise;
- expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
- });
- });
- describe('cleanup', () => {
- test('closes all tracked panes concurrently', async () => {
- const ctx = createMockContext();
- mockMultiplexer.spawnPane
- .mockResolvedValueOnce({ success: true, paneId: 'p1' })
- .mockResolvedValueOnce({ success: true, paneId: 'p2' });
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- await manager.onSessionCreated({
- type: 'session.created',
- properties: { info: { id: 's1', parentID: 'p1' } },
- });
- await manager.onSessionCreated({
- type: 'session.created',
- properties: { info: { id: 's2', parentID: 'p2' } },
- });
- await manager.cleanup();
- expect(mockMultiplexer.closePane).toHaveBeenCalledTimes(2);
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith('p1');
- expect(mockMultiplexer.closePane).toHaveBeenCalledWith('p2');
- });
- test('clears spawning sessions during cleanup', async () => {
- const ctx = createMockContext();
- const manager = new MultiplexerSessionManager(
- ctx,
- defaultMultiplexerConfig,
- );
- const deferred = createDeferred<{ success: true; paneId: string }>();
- mockMultiplexer.spawnPane.mockImplementationOnce(() => deferred.promise);
- const event = {
- type: 'session.created',
- properties: {
- info: {
- id: 'cleanup-spawn',
- parentID: 'parent-cleanup',
- title: 'Cleanup Worker',
- },
- },
- };
- const createPromise = manager.onSessionCreated(event);
- await Promise.resolve();
- await manager.cleanup();
- await manager.onSessionCreated(event);
- deferred.resolve({ success: true, paneId: 'p-cleanup' });
- await createPromise;
- expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(2);
- });
- });
- });
- // Backward compatibility test
- describe('TmuxSessionManager (backward compatibility)', () => {
- test('TmuxSessionManager is alias for MultiplexerSessionManager', async () => {
- const { TmuxSessionManager } = await import('./session-manager');
- expect(TmuxSessionManager).toBe(MultiplexerSessionManager);
- });
- });
|