session-manager.test.ts 42 KB

12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214121512161217121812191220122112221223122412251226122712281229123012311232123312341235123612371238123912401241124212431244124512461247124812491250125112521253125412551256125712581259126012611262126312641265126612671268126912701271127212731274127512761277127812791280128112821283128412851286128712881289129012911292129312941295129612971298129913001301130213031304130513061307130813091310131113121313131413151316131713181319132013211322132313241325132613271328132913301331133213331334133513361337133813391340134113421343134413451346134713481349135013511352135313541355135613571358135913601361136213631364136513661367136813691370137113721373137413751376137713781379138013811382138313841385138613871388138913901391139213931394139513961397139813991400140114021403140414051406140714081409141014111412141314141415141614171418141914201421142214231424142514261427142814291430143114321433143414351436143714381439144014411442144314441445144614471448144914501451145214531454145514561457145814591460146114621463146414651466146714681469
  1. import { afterEach, beforeEach, describe, expect, mock, test } from 'bun:test';
  2. import { BackgroundJobBoard } from '../utils/background-job-board';
  3. import { BackgroundJobCoordinator } from '../utils/background-job-coordinator';
  4. import {
  5. MultiplexerSessionManager,
  6. resetMultiplexerSessionManagerState,
  7. } from './session-manager';
  8. const originalFetch = globalThis.fetch;
  9. let mockSessionStatuses: Record<string, { type: string }> = {};
  10. const mockFetch = mock(
  11. async () =>
  12. new Response(JSON.stringify(mockSessionStatuses), {
  13. headers: { 'content-type': 'application/json' },
  14. status: 200,
  15. }),
  16. );
  17. // Define the mock multiplexer
  18. const mockMultiplexer = {
  19. type: 'tmux' as const,
  20. isAvailable: mock(async () => true),
  21. isInsideSession: mock(() => true),
  22. spawnPane: mock(async () => ({
  23. success: true,
  24. paneId: '%mock-pane',
  25. })),
  26. closePane: mock(async () => true),
  27. applyLayout: mock(async () => {}),
  28. };
  29. // Mock the multiplexer module
  30. mock.module('../multiplexer', () => ({
  31. getMultiplexer: () => mockMultiplexer,
  32. isServerRunning: mock(async () => true),
  33. startAvailabilityCheck: () => {},
  34. }));
  35. // Mock the plugin context
  36. function createMockContext(overrides?: {
  37. sessionStatusResult?: { data?: Record<string, { type: string }> };
  38. directory?: string;
  39. serverUrl?: string;
  40. }) {
  41. const defaultPort = process.env.OPENCODE_PORT ?? '4096';
  42. return {
  43. client: {
  44. session: {
  45. status: mock(
  46. async () => overrides?.sessionStatusResult ?? { data: {} },
  47. ),
  48. },
  49. },
  50. directory: overrides?.directory ?? '/test/directory',
  51. serverUrl: new URL(
  52. overrides?.serverUrl ?? `http://localhost:${defaultPort}`,
  53. ),
  54. } as any;
  55. }
  56. function setMockSessionStatuses(statuses: Record<string, { type: string }>) {
  57. mockSessionStatuses = statuses;
  58. }
  59. const defaultMultiplexerConfig = {
  60. type: 'tmux' as const,
  61. layout: 'main-vertical' as const,
  62. main_pane_size: 60,
  63. zellij_pane_mode: 'agent-tab' as const,
  64. };
  65. function createDeferred<T>() {
  66. let resolve!: (value: T) => void;
  67. let reject!: (reason?: unknown) => void;
  68. const promise = new Promise<T>((res, rej) => {
  69. resolve = res;
  70. reject = rej;
  71. });
  72. return { promise, resolve, reject };
  73. }
  74. describe('MultiplexerSessionManager', () => {
  75. beforeEach(() => {
  76. resetMultiplexerSessionManagerState();
  77. mockSessionStatuses = {};
  78. mockFetch.mockClear();
  79. globalThis.fetch = mockFetch as typeof fetch;
  80. mockMultiplexer.spawnPane.mockReset();
  81. mockMultiplexer.spawnPane.mockResolvedValue({
  82. success: true,
  83. paneId: '%mock-pane',
  84. });
  85. mockMultiplexer.closePane.mockReset();
  86. mockMultiplexer.closePane.mockResolvedValue(true);
  87. mockMultiplexer.isInsideSession.mockReset();
  88. mockMultiplexer.isInsideSession.mockReturnValue(true);
  89. });
  90. afterEach(() => {
  91. globalThis.fetch = originalFetch;
  92. });
  93. describe('constructor', () => {
  94. test('initializes with config', () => {
  95. const ctx = createMockContext();
  96. const manager = new MultiplexerSessionManager(
  97. ctx,
  98. defaultMultiplexerConfig,
  99. );
  100. expect(manager).toBeDefined();
  101. });
  102. });
  103. describe('onSessionCreated', () => {
  104. test('spawns pane for child sessions', async () => {
  105. const ctx = createMockContext();
  106. const manager = new MultiplexerSessionManager(
  107. ctx,
  108. defaultMultiplexerConfig,
  109. );
  110. await manager.onSessionCreated({
  111. type: 'session.created',
  112. properties: {
  113. info: {
  114. id: 'child-123',
  115. parentID: 'parent-456',
  116. title: 'Test Worker',
  117. },
  118. },
  119. });
  120. expect(mockMultiplexer.spawnPane).toHaveBeenCalled();
  121. expect(mockMultiplexer.spawnPane).toHaveBeenCalledWith(
  122. 'child-123',
  123. 'Test Worker',
  124. `http://localhost:${process.env.OPENCODE_PORT ?? '4096'}/`,
  125. '/test/directory',
  126. );
  127. });
  128. test('ignores sessions without parentID', async () => {
  129. const ctx = createMockContext();
  130. const manager = new MultiplexerSessionManager(
  131. ctx,
  132. defaultMultiplexerConfig,
  133. );
  134. await manager.onSessionCreated({
  135. type: 'session.created',
  136. properties: {
  137. info: {
  138. id: 'root-session',
  139. title: 'Main Chat',
  140. },
  141. },
  142. });
  143. expect(mockMultiplexer.spawnPane).not.toHaveBeenCalled();
  144. });
  145. test('prefers child session directory when present', async () => {
  146. const ctx = createMockContext({ directory: '/parent/directory' });
  147. const manager = new MultiplexerSessionManager(
  148. ctx,
  149. defaultMultiplexerConfig,
  150. );
  151. await manager.onSessionCreated({
  152. type: 'session.created',
  153. properties: {
  154. info: {
  155. id: 'child-456',
  156. parentID: 'parent-456',
  157. title: 'Nested Worker',
  158. directory: '/child/directory',
  159. },
  160. },
  161. });
  162. expect(mockMultiplexer.spawnPane).toHaveBeenCalledWith(
  163. 'child-456',
  164. 'Nested Worker',
  165. `http://localhost:${process.env.OPENCODE_PORT ?? '4096'}/`,
  166. '/child/directory',
  167. );
  168. });
  169. test('ignores if disabled in config', async () => {
  170. const ctx = createMockContext();
  171. const manager = new MultiplexerSessionManager(ctx, {
  172. ...defaultMultiplexerConfig,
  173. type: 'none',
  174. });
  175. await manager.onSessionCreated({
  176. type: 'session.created',
  177. properties: {
  178. info: { id: 'child', parentID: 'parent' },
  179. },
  180. });
  181. expect(mockMultiplexer.spawnPane).not.toHaveBeenCalled();
  182. });
  183. test('does not spawn twice for duplicate create events while spawning', async () => {
  184. const ctx = createMockContext();
  185. const manager = new MultiplexerSessionManager(
  186. ctx,
  187. defaultMultiplexerConfig,
  188. );
  189. const deferred = createDeferred<{ success: true; paneId: string }>();
  190. mockMultiplexer.spawnPane.mockImplementationOnce(() => deferred.promise);
  191. const event = {
  192. type: 'session.created',
  193. properties: {
  194. info: {
  195. id: 'child-race',
  196. parentID: 'parent-race',
  197. title: 'Race Worker',
  198. },
  199. },
  200. };
  201. const firstCreate = manager.onSessionCreated(event);
  202. const secondCreate = manager.onSessionCreated(event);
  203. await Promise.resolve();
  204. expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
  205. deferred.resolve({ success: true, paneId: 'p-race' });
  206. await Promise.all([firstCreate, secondCreate]);
  207. expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
  208. });
  209. });
  210. describe('polling and closure', () => {
  211. test('closes pane when session becomes idle', async () => {
  212. const ctx = createMockContext();
  213. mockMultiplexer.spawnPane.mockResolvedValue({
  214. success: true,
  215. paneId: 'p-1',
  216. });
  217. const manager = new MultiplexerSessionManager(
  218. ctx,
  219. defaultMultiplexerConfig,
  220. );
  221. // Register session
  222. await manager.onSessionCreated({
  223. type: 'session.created',
  224. properties: { info: { id: 'c1', parentID: 'p1' } },
  225. });
  226. setMockSessionStatuses({ c1: { type: 'idle' } });
  227. await (manager as any).pollSessions();
  228. expect(mockMultiplexer.closePane).toHaveBeenCalledWith('p-1');
  229. });
  230. test('closes pane immediately on session.idle event', async () => {
  231. const ctx = createMockContext();
  232. mockMultiplexer.spawnPane.mockResolvedValue({
  233. success: true,
  234. paneId: 'p-idle-event',
  235. });
  236. const manager = new MultiplexerSessionManager(
  237. ctx,
  238. defaultMultiplexerConfig,
  239. );
  240. await manager.onSessionCreated({
  241. type: 'session.created',
  242. properties: { info: { id: 'idle-event-child', parentID: 'parent' } },
  243. });
  244. await manager.onSessionStatus({
  245. type: 'session.idle',
  246. properties: { sessionID: 'idle-event-child' },
  247. });
  248. expect(mockMultiplexer.closePane).toHaveBeenCalledWith('p-idle-event');
  249. expect(mockFetch).not.toHaveBeenCalled();
  250. });
  251. test('does not close another manager instance pane on idle event', async () => {
  252. const ctx = createMockContext();
  253. mockMultiplexer.spawnPane.mockResolvedValue({
  254. success: true,
  255. paneId: 'p-shared-idle',
  256. });
  257. const spawningManager = new MultiplexerSessionManager(
  258. ctx,
  259. defaultMultiplexerConfig,
  260. );
  261. const idleManager = new MultiplexerSessionManager(
  262. ctx,
  263. defaultMultiplexerConfig,
  264. );
  265. await spawningManager.onSessionCreated({
  266. type: 'session.created',
  267. properties: { info: { id: 'shared-child', parentID: 'parent' } },
  268. });
  269. await idleManager.onSessionStatus({
  270. type: 'session.idle',
  271. properties: { sessionID: 'shared-child' },
  272. });
  273. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  274. });
  275. test('respawns resumed known session from a different manager instance', async () => {
  276. const ctx = createMockContext();
  277. mockMultiplexer.spawnPane
  278. .mockResolvedValueOnce({
  279. success: true,
  280. paneId: 'p-first',
  281. })
  282. .mockResolvedValueOnce({
  283. success: true,
  284. paneId: 'p-resumed',
  285. });
  286. const firstManager = new MultiplexerSessionManager(
  287. ctx,
  288. defaultMultiplexerConfig,
  289. );
  290. const secondManager = new MultiplexerSessionManager(
  291. ctx,
  292. defaultMultiplexerConfig,
  293. );
  294. await firstManager.onSessionCreated({
  295. type: 'session.created',
  296. properties: {
  297. info: {
  298. id: 'resumed-child',
  299. parentID: 'parent',
  300. title: 'Resumed Worker',
  301. directory: '/resumed/dir',
  302. },
  303. },
  304. });
  305. await firstManager.onSessionStatus({
  306. type: 'session.idle',
  307. properties: { sessionID: 'resumed-child' },
  308. });
  309. await secondManager.onSessionStatus({
  310. type: 'session.status',
  311. properties: {
  312. sessionID: 'resumed-child',
  313. status: { type: 'busy' },
  314. },
  315. });
  316. expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(2);
  317. expect(mockMultiplexer.spawnPane).toHaveBeenLastCalledWith(
  318. 'resumed-child',
  319. 'Resumed Worker',
  320. `http://localhost:${process.env.OPENCODE_PORT ?? '4096'}/`,
  321. '/resumed/dir',
  322. );
  323. });
  324. test('does not close running background child pane on idle event', async () => {
  325. const ctx = createMockContext();
  326. const board = new BackgroundJobBoard();
  327. board.registerLaunch({
  328. taskID: 'running-idle-child',
  329. parentSessionID: 'parent-1',
  330. agent: 'explorer',
  331. });
  332. mockMultiplexer.spawnPane.mockResolvedValue({
  333. success: true,
  334. paneId: 'p-running-idle-child',
  335. });
  336. const manager = new MultiplexerSessionManager(
  337. ctx,
  338. defaultMultiplexerConfig,
  339. board,
  340. );
  341. await manager.onSessionCreated({
  342. type: 'session.created',
  343. properties: {
  344. info: { id: 'running-idle-child', parentID: 'parent-1' },
  345. },
  346. });
  347. await manager.onSessionStatus({
  348. type: 'session.status',
  349. properties: {
  350. sessionID: 'running-idle-child',
  351. status: { type: 'idle' },
  352. },
  353. });
  354. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  355. });
  356. test('timed out running jobs still close after safe recovery and completion', async () => {
  357. const ctx = createMockContext();
  358. const board = new BackgroundJobBoard();
  359. const coordinator = new BackgroundJobCoordinator(board);
  360. board.registerLaunch({
  361. taskID: 'timedout-child',
  362. parentSessionID: 'parent-1',
  363. agent: 'explorer',
  364. });
  365. board.updateStatus({
  366. taskID: 'timedout-child',
  367. state: 'running',
  368. timedOut: true,
  369. now: 100,
  370. });
  371. mockMultiplexer.spawnPane.mockResolvedValue({
  372. success: true,
  373. paneId: 'p-timedout-child',
  374. });
  375. const manager = new MultiplexerSessionManager(
  376. ctx,
  377. defaultMultiplexerConfig,
  378. coordinator,
  379. );
  380. coordinator.addTerminalStateListener((sessionId) => {
  381. void manager.closeSessionFromCoordinator(sessionId);
  382. });
  383. await manager.onSessionCreated({
  384. type: 'session.created',
  385. properties: {
  386. info: { id: 'timedout-child', parentID: 'parent-1' },
  387. },
  388. });
  389. await manager.onSessionStatus({
  390. type: 'session.status',
  391. properties: {
  392. sessionID: 'timedout-child',
  393. status: { type: 'idle' },
  394. },
  395. });
  396. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  397. await manager.onSessionStatus({
  398. type: 'session.status',
  399. properties: {
  400. sessionID: 'timedout-child',
  401. status: { type: 'busy' },
  402. },
  403. });
  404. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  405. board.updateStatus({
  406. taskID: 'timedout-child',
  407. state: 'completed',
  408. resultSummary: 'done',
  409. });
  410. await Promise.resolve();
  411. expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
  412. 'p-timedout-child',
  413. );
  414. });
  415. test('deferred idle closes retry on terminal status updates', async () => {
  416. for (const state of ['completed', 'error', 'cancelled'] as const) {
  417. resetMultiplexerSessionManagerState();
  418. mockMultiplexer.spawnPane.mockClear();
  419. mockMultiplexer.closePane.mockClear();
  420. const ctx = createMockContext();
  421. const board = new BackgroundJobBoard();
  422. const coordinator = new BackgroundJobCoordinator(board);
  423. const sessionId = `deferred-${state}`;
  424. board.registerLaunch({
  425. taskID: sessionId,
  426. parentSessionID: 'parent-1',
  427. agent: 'explorer',
  428. });
  429. mockMultiplexer.spawnPane.mockResolvedValueOnce({
  430. success: true,
  431. paneId: `p-${state}`,
  432. });
  433. const manager = new MultiplexerSessionManager(
  434. ctx,
  435. defaultMultiplexerConfig,
  436. coordinator,
  437. );
  438. coordinator.addTerminalStateListener((sessionId) => {
  439. void manager.closeSessionFromCoordinator(sessionId);
  440. });
  441. await manager.onSessionCreated({
  442. type: 'session.created',
  443. properties: { info: { id: sessionId, parentID: 'parent-1' } },
  444. });
  445. await manager.onSessionStatus({
  446. type: 'session.status',
  447. properties: { sessionID: sessionId, status: { type: 'idle' } },
  448. });
  449. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  450. board.updateStatus({ taskID: sessionId, state });
  451. await Promise.resolve();
  452. expect(mockMultiplexer.closePane).toHaveBeenCalledWith(`p-${state}`);
  453. }
  454. });
  455. test('deferred idle close retries on markCancelled', async () => {
  456. const ctx = createMockContext();
  457. const board = new BackgroundJobBoard();
  458. const coordinator = new BackgroundJobCoordinator(board);
  459. board.registerLaunch({
  460. taskID: 'deferred-cancel',
  461. parentSessionID: 'parent-1',
  462. agent: 'explorer',
  463. });
  464. mockMultiplexer.spawnPane.mockResolvedValue({
  465. success: true,
  466. paneId: 'p-deferred-cancel',
  467. });
  468. const manager = new MultiplexerSessionManager(
  469. ctx,
  470. defaultMultiplexerConfig,
  471. coordinator,
  472. );
  473. coordinator.addTerminalStateListener((sessionId) => {
  474. void manager.closeSessionFromCoordinator(sessionId);
  475. });
  476. await manager.onSessionCreated({
  477. type: 'session.created',
  478. properties: { info: { id: 'deferred-cancel', parentID: 'parent-1' } },
  479. });
  480. await manager.onSessionStatus({
  481. type: 'session.status',
  482. properties: {
  483. sessionID: 'deferred-cancel',
  484. status: { type: 'idle' },
  485. },
  486. });
  487. board.markCancelled('deferred-cancel');
  488. await Promise.resolve();
  489. expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
  490. 'p-deferred-cancel',
  491. );
  492. });
  493. test('terminal status without deferred idle close does not close pane', async () => {
  494. const ctx = createMockContext();
  495. const board = new BackgroundJobBoard();
  496. const coordinator = new BackgroundJobCoordinator(board);
  497. board.registerLaunch({
  498. taskID: 'terminal-without-defer',
  499. parentSessionID: 'parent-1',
  500. agent: 'explorer',
  501. });
  502. mockMultiplexer.spawnPane.mockResolvedValue({
  503. success: true,
  504. paneId: 'p-terminal-without-defer',
  505. });
  506. const manager = new MultiplexerSessionManager(
  507. ctx,
  508. defaultMultiplexerConfig,
  509. coordinator,
  510. );
  511. coordinator.addTerminalStateListener((sessionId) => {
  512. void manager.closeSessionFromCoordinator(sessionId);
  513. });
  514. await manager.onSessionCreated({
  515. type: 'session.created',
  516. properties: {
  517. info: { id: 'terminal-without-defer', parentID: 'parent-1' },
  518. },
  519. });
  520. board.updateStatus({
  521. taskID: 'terminal-without-defer',
  522. state: 'completed',
  523. });
  524. await Promise.resolve();
  525. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  526. });
  527. test('deleted clears deferred idle close and later terminal update is no-op', async () => {
  528. const ctx = createMockContext();
  529. const board = new BackgroundJobBoard();
  530. const coordinator = new BackgroundJobCoordinator(board);
  531. board.registerLaunch({
  532. taskID: 'deleted-deferred',
  533. parentSessionID: 'parent-1',
  534. agent: 'explorer',
  535. });
  536. mockMultiplexer.spawnPane.mockResolvedValue({
  537. success: true,
  538. paneId: 'p-deleted-deferred',
  539. });
  540. const manager = new MultiplexerSessionManager(
  541. ctx,
  542. defaultMultiplexerConfig,
  543. coordinator,
  544. );
  545. coordinator.addTerminalStateListener((sessionId) => {
  546. void manager.closeSessionFromCoordinator(sessionId);
  547. });
  548. await manager.onSessionCreated({
  549. type: 'session.created',
  550. properties: { info: { id: 'deleted-deferred', parentID: 'parent-1' } },
  551. });
  552. await manager.onSessionStatus({
  553. type: 'session.status',
  554. properties: { sessionID: 'deleted-deferred', status: { type: 'idle' } },
  555. });
  556. await manager.onSessionDeleted({
  557. type: 'session.deleted',
  558. properties: { sessionID: 'deleted-deferred' },
  559. });
  560. board.updateStatus({ taskID: 'deleted-deferred', state: 'completed' });
  561. await Promise.resolve();
  562. expect(mockMultiplexer.closePane).toHaveBeenCalledTimes(1);
  563. expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
  564. 'p-deleted-deferred',
  565. );
  566. });
  567. test('retry while still running keeps deferred idle close', async () => {
  568. const ctx = createMockContext();
  569. const board = new BackgroundJobBoard();
  570. const coordinator = new BackgroundJobCoordinator(board);
  571. board.registerLaunch({
  572. taskID: 'still-running-deferred',
  573. parentSessionID: 'parent-1',
  574. agent: 'explorer',
  575. });
  576. mockMultiplexer.spawnPane.mockResolvedValue({
  577. success: true,
  578. paneId: 'p-still-running-deferred',
  579. });
  580. const manager = new MultiplexerSessionManager(
  581. ctx,
  582. defaultMultiplexerConfig,
  583. coordinator,
  584. );
  585. coordinator.addTerminalStateListener((sessionId) => {
  586. void manager.closeSessionFromCoordinator(sessionId);
  587. });
  588. await manager.onSessionCreated({
  589. type: 'session.created',
  590. properties: {
  591. info: { id: 'still-running-deferred', parentID: 'parent-1' },
  592. },
  593. });
  594. await manager.onSessionStatus({
  595. type: 'session.status',
  596. properties: {
  597. sessionID: 'still-running-deferred',
  598. status: { type: 'idle' },
  599. },
  600. });
  601. // The coordinator's terminal state listener will handle the close
  602. // when the job completes, so we don't need to call retryDeferredIdleClose directly
  603. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  604. board.updateStatus({
  605. taskID: 'still-running-deferred',
  606. state: 'completed',
  607. });
  608. await Promise.resolve();
  609. expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
  610. 'p-still-running-deferred',
  611. );
  612. });
  613. test('disabled manager does not retry deferred idle close', async () => {
  614. const ctx = createMockContext();
  615. const board = new BackgroundJobBoard();
  616. const coordinator = new BackgroundJobCoordinator(board);
  617. board.registerLaunch({
  618. taskID: 'disabled-retry-deferred',
  619. parentSessionID: 'parent-1',
  620. agent: 'explorer',
  621. });
  622. mockMultiplexer.spawnPane.mockResolvedValue({
  623. success: true,
  624. paneId: 'p-disabled-retry-deferred',
  625. });
  626. const manager = new MultiplexerSessionManager(
  627. ctx,
  628. defaultMultiplexerConfig,
  629. coordinator,
  630. );
  631. await manager.onSessionCreated({
  632. type: 'session.created',
  633. properties: {
  634. info: { id: 'disabled-retry-deferred', parentID: 'parent-1' },
  635. },
  636. });
  637. await manager.onSessionStatus({
  638. type: 'session.status',
  639. properties: {
  640. sessionID: 'disabled-retry-deferred',
  641. status: { type: 'idle' },
  642. },
  643. });
  644. mockMultiplexer.isInsideSession.mockReturnValue(false);
  645. const _disabledManager = new MultiplexerSessionManager(
  646. ctx,
  647. defaultMultiplexerConfig,
  648. coordinator,
  649. );
  650. // The coordinator's terminal state listener will handle the close
  651. // when the job completes, but the disabled manager should not close
  652. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  653. });
  654. test('explicit non-idle status event clears stale deferred idle close', async () => {
  655. const ctx = createMockContext();
  656. const board = new BackgroundJobBoard();
  657. const coordinator = new BackgroundJobCoordinator(board);
  658. board.registerLaunch({
  659. taskID: 'retry-event-deferred',
  660. parentSessionID: 'parent-1',
  661. agent: 'explorer',
  662. });
  663. mockMultiplexer.spawnPane.mockResolvedValue({
  664. success: true,
  665. paneId: 'p-retry-event-deferred',
  666. });
  667. const manager = new MultiplexerSessionManager(
  668. ctx,
  669. defaultMultiplexerConfig,
  670. coordinator,
  671. );
  672. coordinator.addTerminalStateListener((sessionId) => {
  673. void manager.closeSessionFromCoordinator(sessionId);
  674. });
  675. await manager.onSessionCreated({
  676. type: 'session.created',
  677. properties: {
  678. info: { id: 'retry-event-deferred', parentID: 'parent-1' },
  679. },
  680. });
  681. await manager.onSessionStatus({
  682. type: 'session.status',
  683. properties: {
  684. sessionID: 'retry-event-deferred',
  685. status: { type: 'idle' },
  686. },
  687. });
  688. await manager.onSessionStatus({
  689. type: 'session.status',
  690. properties: {
  691. sessionID: 'retry-event-deferred',
  692. status: { type: 'retry' },
  693. },
  694. });
  695. board.updateStatus({
  696. taskID: 'retry-event-deferred',
  697. state: 'completed',
  698. });
  699. await Promise.resolve();
  700. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  701. await manager.onSessionStatus({
  702. type: 'session.status',
  703. properties: {
  704. sessionID: 'retry-event-deferred',
  705. status: { type: 'idle' },
  706. },
  707. });
  708. expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
  709. 'p-retry-event-deferred',
  710. );
  711. });
  712. test('explicit non-idle poll clears stale deferred idle close', async () => {
  713. const ctx = createMockContext();
  714. const board = new BackgroundJobBoard();
  715. const coordinator = new BackgroundJobCoordinator(board);
  716. board.registerLaunch({
  717. taskID: 'resumed-deferred',
  718. parentSessionID: 'parent-1',
  719. agent: 'explorer',
  720. });
  721. mockMultiplexer.spawnPane.mockResolvedValue({
  722. success: true,
  723. paneId: 'p-resumed-deferred',
  724. });
  725. const manager = new MultiplexerSessionManager(
  726. ctx,
  727. defaultMultiplexerConfig,
  728. coordinator,
  729. );
  730. coordinator.addTerminalStateListener((sessionId) => {
  731. void manager.closeSessionFromCoordinator(sessionId);
  732. });
  733. await manager.onSessionCreated({
  734. type: 'session.created',
  735. properties: { info: { id: 'resumed-deferred', parentID: 'parent-1' } },
  736. });
  737. await manager.onSessionStatus({
  738. type: 'session.status',
  739. properties: {
  740. sessionID: 'resumed-deferred',
  741. status: { type: 'idle' },
  742. },
  743. });
  744. setMockSessionStatuses({ 'resumed-deferred': { type: 'busy' } });
  745. await (manager as any).pollSessions();
  746. board.updateStatus({ taskID: 'resumed-deferred', state: 'completed' });
  747. await Promise.resolve();
  748. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  749. await manager.onSessionStatus({
  750. type: 'session.status',
  751. properties: {
  752. sessionID: 'resumed-deferred',
  753. status: { type: 'idle' },
  754. },
  755. });
  756. expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
  757. 'p-resumed-deferred',
  758. );
  759. });
  760. test('does not close on transient status absence', async () => {
  761. const ctx = createMockContext();
  762. const manager = new MultiplexerSessionManager(
  763. ctx,
  764. defaultMultiplexerConfig,
  765. );
  766. await manager.onSessionCreated({
  767. type: 'session.created',
  768. properties: { info: { id: 'c1', parentID: 'p1' } },
  769. });
  770. setMockSessionStatuses({});
  771. await (manager as any).pollSessions();
  772. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  773. });
  774. test('keeps background child pane open while status is running until deleted', async () => {
  775. const ctx = createMockContext();
  776. const manager = new MultiplexerSessionManager(
  777. ctx,
  778. defaultMultiplexerConfig,
  779. );
  780. mockMultiplexer.spawnPane.mockResolvedValueOnce({
  781. success: true,
  782. paneId: 'p-background-child',
  783. });
  784. await manager.onSessionCreated({
  785. type: 'session.created',
  786. properties: {
  787. info: {
  788. id: 'background-child',
  789. parentID: 'parent-1',
  790. title: 'Background Worker',
  791. },
  792. },
  793. });
  794. setMockSessionStatuses({ 'background-child': { type: 'running' } });
  795. await (manager as any).pollSessions();
  796. await (manager as any).pollSessions();
  797. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  798. await manager.onSessionDeleted({
  799. type: 'session.deleted',
  800. properties: { info: { id: 'background-child' } },
  801. });
  802. expect(mockMultiplexer.closePane).toHaveBeenCalledTimes(1);
  803. expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
  804. 'p-background-child',
  805. );
  806. });
  807. test('missing status does not close never-seen pane', async () => {
  808. const ctx = createMockContext();
  809. mockMultiplexer.spawnPane.mockResolvedValue({
  810. success: true,
  811. paneId: 'p-never-seen-orphan',
  812. });
  813. const manager = new MultiplexerSessionManager(
  814. ctx,
  815. defaultMultiplexerConfig,
  816. );
  817. await manager.onSessionCreated({
  818. type: 'session.created',
  819. properties: { info: { id: 'never-seen-orphan', parentID: 'p1' } },
  820. });
  821. setMockSessionStatuses({});
  822. await (manager as any).pollSessions();
  823. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  824. });
  825. test('ignores empty session status response without closing panes', async () => {
  826. const ctx = createMockContext();
  827. mockMultiplexer.spawnPane.mockResolvedValue({
  828. success: true,
  829. paneId: 'p-empty-status',
  830. });
  831. const manager = new MultiplexerSessionManager(
  832. ctx,
  833. defaultMultiplexerConfig,
  834. );
  835. await manager.onSessionCreated({
  836. type: 'session.created',
  837. properties: { info: { id: 'empty-status', parentID: 'p1' } },
  838. });
  839. mockFetch.mockImplementationOnce(
  840. async () => new Response('', { status: 200 }),
  841. );
  842. await (manager as any).pollSessions();
  843. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  844. });
  845. test('previously seen then missing does not close pane', async () => {
  846. const ctx = createMockContext();
  847. mockMultiplexer.spawnPane.mockResolvedValue({
  848. success: true,
  849. paneId: 'p-seen-before-missing',
  850. });
  851. const manager = new MultiplexerSessionManager(
  852. ctx,
  853. defaultMultiplexerConfig,
  854. );
  855. await manager.onSessionCreated({
  856. type: 'session.created',
  857. properties: { info: { id: 'seen-before-missing', parentID: 'p1' } },
  858. });
  859. setMockSessionStatuses({ 'seen-before-missing': { type: 'busy' } });
  860. await (manager as any).pollSessions();
  861. setMockSessionStatuses({});
  862. await (manager as any).pollSessions();
  863. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  864. });
  865. test('missing then busy does not duplicate respawn', async () => {
  866. const ctx = createMockContext();
  867. mockMultiplexer.spawnPane.mockResolvedValue({
  868. success: true,
  869. paneId: 'p-missing-then-busy',
  870. });
  871. const manager = new MultiplexerSessionManager(
  872. ctx,
  873. defaultMultiplexerConfig,
  874. );
  875. await manager.onSessionCreated({
  876. type: 'session.created',
  877. properties: { info: { id: 'missing-then-busy', parentID: 'p1' } },
  878. });
  879. setMockSessionStatuses({});
  880. await (manager as any).pollSessions();
  881. await manager.onSessionStatus({
  882. type: 'session.status',
  883. properties: {
  884. sessionID: 'missing-then-busy',
  885. status: { type: 'busy' },
  886. },
  887. });
  888. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  889. expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
  890. });
  891. test('polls the actual serverUrl instead of the plugin SDK default URL', async () => {
  892. const ctx = createMockContext({
  893. serverUrl: 'http://127.0.0.1:63871/',
  894. sessionStatusResult: { data: {} },
  895. });
  896. const manager = new MultiplexerSessionManager(
  897. ctx,
  898. defaultMultiplexerConfig,
  899. );
  900. await manager.onSessionCreated({
  901. type: 'session.created',
  902. properties: { info: { id: 'child-live', parentID: 'parent-live' } },
  903. });
  904. setMockSessionStatuses({ 'child-live': { type: 'busy' } });
  905. await (manager as any).pollSessions();
  906. expect(ctx.client.session.status).not.toHaveBeenCalled();
  907. expect(mockFetch).toHaveBeenCalledWith(
  908. new URL('http://127.0.0.1:63871/session/status'),
  909. expect.any(Object),
  910. );
  911. expect(mockMultiplexer.closePane).not.toHaveBeenCalled();
  912. });
  913. test('respawns pane on later busy after idle close for resumable session', async () => {
  914. const ctx = createMockContext();
  915. const manager = new MultiplexerSessionManager(
  916. ctx,
  917. defaultMultiplexerConfig,
  918. );
  919. mockMultiplexer.spawnPane
  920. .mockResolvedValueOnce({
  921. success: true,
  922. paneId: 'p-1',
  923. })
  924. .mockResolvedValueOnce({
  925. success: true,
  926. paneId: 'p-2',
  927. });
  928. await manager.onSessionCreated({
  929. type: 'session.created',
  930. properties: {
  931. info: {
  932. id: 'child-789',
  933. parentID: 'parent-789',
  934. title: 'Worker',
  935. directory: '/task/dir',
  936. },
  937. },
  938. });
  939. setMockSessionStatuses({ 'child-789': { type: 'idle' } });
  940. await (manager as any).pollSessions();
  941. await manager.onSessionStatus({
  942. type: 'session.status',
  943. properties: {
  944. sessionID: 'child-789',
  945. status: { type: 'busy' },
  946. },
  947. });
  948. expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(2);
  949. expect(mockMultiplexer.spawnPane).toHaveBeenCalledWith(
  950. 'child-789',
  951. 'Worker',
  952. `http://localhost:${process.env.OPENCODE_PORT ?? '4096'}/`,
  953. '/task/dir',
  954. );
  955. expect(mockMultiplexer.closePane).toHaveBeenCalledWith('p-1');
  956. expect(mockMultiplexer.closePane).toHaveBeenCalledTimes(1);
  957. });
  958. test('respawns after in-flight idle close when busy resumes same session', async () => {
  959. const ctx = createMockContext();
  960. const manager = new MultiplexerSessionManager(
  961. ctx,
  962. defaultMultiplexerConfig,
  963. );
  964. const closeDeferred = createDeferred<boolean>();
  965. mockMultiplexer.spawnPane
  966. .mockResolvedValueOnce({
  967. success: true,
  968. paneId: 'p-close-race',
  969. })
  970. .mockResolvedValueOnce({
  971. success: true,
  972. paneId: 'p-close-race-resumed',
  973. });
  974. mockMultiplexer.closePane.mockImplementationOnce(
  975. () => closeDeferred.promise,
  976. );
  977. await manager.onSessionCreated({
  978. type: 'session.created',
  979. properties: {
  980. info: {
  981. id: 'child-close-race',
  982. parentID: 'parent-close-race',
  983. title: 'Worker',
  984. },
  985. },
  986. });
  987. const idlePromise = manager.onSessionStatus({
  988. type: 'session.status',
  989. properties: {
  990. sessionID: 'child-close-race',
  991. status: { type: 'idle' },
  992. },
  993. });
  994. await Promise.resolve();
  995. const busyPromise = manager.onSessionStatus({
  996. type: 'session.status',
  997. properties: {
  998. sessionID: 'child-close-race',
  999. status: { type: 'busy' },
  1000. },
  1001. });
  1002. expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
  1003. closeDeferred.resolve(true);
  1004. await Promise.all([idlePromise, busyPromise]);
  1005. expect(mockMultiplexer.closePane).toHaveBeenCalledTimes(1);
  1006. expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(2);
  1007. expect(mockMultiplexer.spawnPane).toHaveBeenLastCalledWith(
  1008. 'child-close-race',
  1009. 'Worker',
  1010. `http://localhost:${process.env.OPENCODE_PORT ?? '4096'}/`,
  1011. '/test/directory',
  1012. );
  1013. });
  1014. test('does not respawn after in-flight close if session is deleted', async () => {
  1015. const ctx = createMockContext();
  1016. const manager = new MultiplexerSessionManager(
  1017. ctx,
  1018. defaultMultiplexerConfig,
  1019. );
  1020. const closeDeferred = createDeferred<boolean>();
  1021. mockMultiplexer.spawnPane
  1022. .mockResolvedValueOnce({
  1023. success: true,
  1024. paneId: 'p-delete-race',
  1025. })
  1026. .mockResolvedValueOnce({
  1027. success: true,
  1028. paneId: 'p-should-not-respawn',
  1029. });
  1030. mockMultiplexer.closePane.mockImplementationOnce(
  1031. () => closeDeferred.promise,
  1032. );
  1033. await manager.onSessionCreated({
  1034. type: 'session.created',
  1035. properties: {
  1036. info: {
  1037. id: 'child-delete-race',
  1038. parentID: 'parent-delete-race',
  1039. title: 'Worker',
  1040. },
  1041. },
  1042. });
  1043. const idlePromise = manager.onSessionStatus({
  1044. type: 'session.status',
  1045. properties: {
  1046. sessionID: 'child-delete-race',
  1047. status: { type: 'idle' },
  1048. },
  1049. });
  1050. await Promise.resolve();
  1051. const busyPromise = manager.onSessionStatus({
  1052. type: 'session.status',
  1053. properties: {
  1054. sessionID: 'child-delete-race',
  1055. status: { type: 'busy' },
  1056. },
  1057. });
  1058. const deletedPromise = manager.onSessionDeleted({
  1059. type: 'session.deleted',
  1060. properties: {
  1061. sessionID: 'child-delete-race',
  1062. },
  1063. });
  1064. closeDeferred.resolve(true);
  1065. await Promise.all([idlePromise, busyPromise, deletedPromise]);
  1066. expect(mockMultiplexer.closePane).toHaveBeenCalledTimes(1);
  1067. expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
  1068. });
  1069. test('closes pane on session.deleted using info.id', async () => {
  1070. const ctx = createMockContext();
  1071. const manager = new MultiplexerSessionManager(
  1072. ctx,
  1073. defaultMultiplexerConfig,
  1074. );
  1075. mockMultiplexer.spawnPane.mockResolvedValueOnce({
  1076. success: true,
  1077. paneId: 'p-info-id',
  1078. });
  1079. await manager.onSessionCreated({
  1080. type: 'session.created',
  1081. properties: {
  1082. info: {
  1083. id: 'child-info-id',
  1084. parentID: 'parent-info-id',
  1085. },
  1086. },
  1087. });
  1088. await manager.onSessionDeleted({
  1089. type: 'session.deleted',
  1090. properties: {
  1091. info: { id: 'child-info-id' },
  1092. },
  1093. });
  1094. expect(mockMultiplexer.closePane).toHaveBeenCalledWith('p-info-id');
  1095. await manager.onSessionStatus({
  1096. type: 'session.status',
  1097. properties: {
  1098. sessionID: 'child-info-id',
  1099. status: { type: 'busy' },
  1100. },
  1101. });
  1102. expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
  1103. });
  1104. test('closes deleted pane even when current instance is not owner', async () => {
  1105. const ctx = createMockContext();
  1106. const manager = new MultiplexerSessionManager(
  1107. ctx,
  1108. defaultMultiplexerConfig,
  1109. );
  1110. mockMultiplexer.spawnPane.mockResolvedValueOnce({
  1111. success: true,
  1112. paneId: 'p-non-owner-delete',
  1113. });
  1114. await manager.onSessionCreated({
  1115. type: 'session.created',
  1116. properties: {
  1117. info: {
  1118. id: 'child-non-owner-delete',
  1119. parentID: 'parent-non-owner-delete',
  1120. },
  1121. },
  1122. });
  1123. const tracked = (manager as any).sessions.get('child-non-owner-delete');
  1124. tracked.ownerInstanceId = 'other-instance';
  1125. await manager.onSessionDeleted({
  1126. type: 'session.deleted',
  1127. properties: { sessionID: 'child-non-owner-delete' },
  1128. });
  1129. expect(mockMultiplexer.closePane).toHaveBeenCalledWith(
  1130. 'p-non-owner-delete',
  1131. );
  1132. expect((manager as any).sessions.has('child-non-owner-delete')).toBe(
  1133. false,
  1134. );
  1135. });
  1136. test('closes pane returned by a stale spawn after session deleted', async () => {
  1137. const ctx = createMockContext();
  1138. const manager = new MultiplexerSessionManager(
  1139. ctx,
  1140. defaultMultiplexerConfig,
  1141. );
  1142. const spawnDeferred = createDeferred<{ success: true; paneId: string }>();
  1143. mockMultiplexer.spawnPane.mockImplementationOnce(
  1144. () => spawnDeferred.promise,
  1145. );
  1146. const createPromise = manager.onSessionCreated({
  1147. type: 'session.created',
  1148. properties: {
  1149. info: {
  1150. id: 'child-stale-spawn',
  1151. parentID: 'parent-stale-spawn',
  1152. },
  1153. },
  1154. });
  1155. await Promise.resolve();
  1156. await manager.onSessionDeleted({
  1157. type: 'session.deleted',
  1158. properties: {
  1159. info: { id: 'child-stale-spawn' },
  1160. },
  1161. });
  1162. spawnDeferred.resolve({ success: true, paneId: 'p-stale-spawn' });
  1163. await createPromise;
  1164. expect(mockMultiplexer.closePane).toHaveBeenCalledWith('p-stale-spawn');
  1165. });
  1166. test('does nothing on busy for unknown session', async () => {
  1167. const ctx = createMockContext();
  1168. const manager = new MultiplexerSessionManager(
  1169. ctx,
  1170. defaultMultiplexerConfig,
  1171. );
  1172. await manager.onSessionStatus({
  1173. type: 'session.status',
  1174. properties: {
  1175. sessionID: 'unknown-session',
  1176. status: { type: 'busy' },
  1177. },
  1178. });
  1179. expect(mockMultiplexer.spawnPane).not.toHaveBeenCalled();
  1180. });
  1181. test('does not respawn while initial pane spawn is still in progress', async () => {
  1182. const ctx = createMockContext();
  1183. const manager = new MultiplexerSessionManager(
  1184. ctx,
  1185. defaultMultiplexerConfig,
  1186. );
  1187. const deferred = createDeferred<{ success: true; paneId: string }>();
  1188. mockMultiplexer.spawnPane.mockImplementationOnce(() => deferred.promise);
  1189. const createPromise = manager.onSessionCreated({
  1190. type: 'session.created',
  1191. properties: {
  1192. info: {
  1193. id: 'child-busy-race',
  1194. parentID: 'parent-busy-race',
  1195. title: 'Busy Worker',
  1196. directory: '/task/dir',
  1197. },
  1198. },
  1199. });
  1200. await manager.onSessionStatus({
  1201. type: 'session.status',
  1202. properties: {
  1203. sessionID: 'child-busy-race',
  1204. status: { type: 'busy' },
  1205. },
  1206. });
  1207. expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
  1208. deferred.resolve({ success: true, paneId: 'p-busy-race' });
  1209. await createPromise;
  1210. expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(1);
  1211. });
  1212. });
  1213. describe('cleanup', () => {
  1214. test('closes all tracked panes concurrently', async () => {
  1215. const ctx = createMockContext();
  1216. mockMultiplexer.spawnPane
  1217. .mockResolvedValueOnce({ success: true, paneId: 'p1' })
  1218. .mockResolvedValueOnce({ success: true, paneId: 'p2' });
  1219. const manager = new MultiplexerSessionManager(
  1220. ctx,
  1221. defaultMultiplexerConfig,
  1222. );
  1223. await manager.onSessionCreated({
  1224. type: 'session.created',
  1225. properties: { info: { id: 's1', parentID: 'p1' } },
  1226. });
  1227. await manager.onSessionCreated({
  1228. type: 'session.created',
  1229. properties: { info: { id: 's2', parentID: 'p2' } },
  1230. });
  1231. await manager.cleanup();
  1232. expect(mockMultiplexer.closePane).toHaveBeenCalledTimes(2);
  1233. expect(mockMultiplexer.closePane).toHaveBeenCalledWith('p1');
  1234. expect(mockMultiplexer.closePane).toHaveBeenCalledWith('p2');
  1235. });
  1236. test('clears spawning sessions during cleanup', async () => {
  1237. const ctx = createMockContext();
  1238. const manager = new MultiplexerSessionManager(
  1239. ctx,
  1240. defaultMultiplexerConfig,
  1241. );
  1242. const deferred = createDeferred<{ success: true; paneId: string }>();
  1243. mockMultiplexer.spawnPane.mockImplementationOnce(() => deferred.promise);
  1244. const event = {
  1245. type: 'session.created',
  1246. properties: {
  1247. info: {
  1248. id: 'cleanup-spawn',
  1249. parentID: 'parent-cleanup',
  1250. title: 'Cleanup Worker',
  1251. },
  1252. },
  1253. };
  1254. const createPromise = manager.onSessionCreated(event);
  1255. await Promise.resolve();
  1256. await manager.cleanup();
  1257. await manager.onSessionCreated(event);
  1258. deferred.resolve({ success: true, paneId: 'p-cleanup' });
  1259. await createPromise;
  1260. expect(mockMultiplexer.spawnPane).toHaveBeenCalledTimes(2);
  1261. });
  1262. });
  1263. });
  1264. // Backward compatibility test
  1265. describe('TmuxSessionManager (backward compatibility)', () => {
  1266. test('TmuxSessionManager is alias for MultiplexerSessionManager', async () => {
  1267. const { TmuxSessionManager } = await import('./session-manager');
  1268. expect(TmuxSessionManager).toBe(MultiplexerSessionManager);
  1269. });
  1270. });