From 76e0eb72be3dfffa7a7d9c840eeef47dc2fe6d47 Mon Sep 17 00:00:00 2001 From: Shubhdeep Chhabra Date: Tue, 20 Jan 2026 18:49:58 +0530 Subject: [PATCH] fix: add cleanup handlers for EventSource connection in cli --- packages/spotlight/src/server/cli/tail.ts | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/packages/spotlight/src/server/cli/tail.ts b/packages/spotlight/src/server/cli/tail.ts index 073752d4..7cc1a66e 100644 --- a/packages/spotlight/src/server/cli/tail.ts +++ b/packages/spotlight/src/server/cli/tail.ts @@ -131,6 +131,22 @@ export async function handler( try { const client = await connectUpstream(port); client.addEventListener(SENTRY_CONTENT_TYPE, event => onEnvelope!(JSON.parse(event.data))); + + // Set up cleanup handlers to close the EventSource connection + const cleanup = (): void => { + try { + if (client.readyState !== 2) { + client.close(); + } + } catch (_err) { + // Ignore errors because maybe client is already closed + } + }; + + process.on("SIGINT", cleanup); + process.on("SIGTERM", cleanup); + process.on("beforeExit", cleanup); + // Early return - don't start our own server if we can connect to an upstream one return undefined; } catch (err) {