github-router 0.3.35 → 0.3.36

package/dist/browser-bridge/index.js

@@ -3639,6 +3639,64 @@ function discoveryPath() {
 	return path.join(homedir(), ".local", "share", "github-router", "browser-mcp", "bridge.json");
 }
 
+//#endregion
+//#region src/browser-bridge/pending.ts
+const pendingMap = /* @__PURE__ */ new Map();
+/**
+* Register a pending request. The TTL timer ensures the entry is removed
+* even when the extension hangs (MV3 SW dormancy, tab crash, navigation
+* interrupt) and the WS client stays connected — which would otherwise
+* leave the entry in the Map for the lifetime of a long proxy session.
+*
+* On TTL expiry the caller receives a structured timeout error so the
+* dispatcher can surface a meaningful message rather than hanging.
+*
+* @param id       Request ID (must match the extension's response).
+* @param client   WS client to send the response back on.
+* @param ttlMs    How long to wait before forcibly resolving with an error.
+* @param sendResp Callback that encodes and writes BridgeResponse to `client`.
+*/
+function pendingAdd(id, client, ttlMs, sendResp) {
+	const ttlTimer = setTimeout(() => {
+		if (!pendingMap.has(id)) return;
+		pendingMap.delete(id);
+		sendResp({
+			id,
+			ok: false,
+			error: `bridge timeout after ${ttlMs}ms`,
+			code: "timeout"
+		});
+	}, ttlMs);
+	pendingMap.set(id, {
+		resolve: sendResp,
+		client,
+		ttlTimer
+	});
+}
+/**
+* Resolve a pending request with the browser's response and cancel the
+* TTL timer. No-ops if the id is unknown (double-resolve from a
+* misbehaving extension is harmless).
+*/
+function pendingResolve(id, msg) {
+	const entry = pendingMap.get(id);
+	if (!entry) return;
+	clearTimeout(entry.ttlTimer);
+	pendingMap.delete(id);
+	entry.resolve(msg);
+}
+/**
+* Drop all pending entries belonging to a specific WS client and cancel
+* their TTL timers. Called from the WS "close" handler so we don't leak
+* entries when the dispatcher disconnects mid-flight.
+*/
+function pendingDropClient(client) {
+	for (const [id, entry] of pendingMap) if (entry.client === client) {
+		clearTimeout(entry.ttlTimer);
+		pendingMap.delete(id);
+	}
+}
+
 //#endregion
 //#region src/browser-bridge/index.ts
 try {
@@ -3646,6 +3704,7 @@ try {
 } catch {}
 const HEARTBEAT_MS = 5e3;
 const HEARTBEAT_MISS_LIMIT = 3;
+const PENDING_TTL_MS = 3e5;
 function writeDiscoveryFile(payload) {
 	const file = discoveryPath();
 	mkdirSync(path.dirname(file), { recursive: true });
@@ -3726,7 +3785,6 @@ httpServer.on("upgrade", (req, socket, head) => {
 		handleWsConnection(ws);
 	});
 });
-const pending = /* @__PURE__ */ new Map();
 function handleWsConnection(ws) {
 	let alive = true;
 	let misses = 0;
@@ -3765,15 +3823,12 @@ function handleWsConnection(ws) {
 			return;
 		}
 		if (typeof msg.id !== "string" || typeof msg.tool !== "string") return;
-		pending.set(msg.id, {
-			resolve: (resp) => ws.send(JSON.stringify(resp)),
-			client: ws
-		});
+		pendingAdd(msg.id, ws, PENDING_TTL_MS, (resp) => ws.send(JSON.stringify(resp)));
 		sendToBrowser(msg);
 	});
 	ws.on("close", () => {
 		clearInterval(heartbeat);
-		for (const [id, p] of pending) if (p.client === ws) pending.delete(id);
+		pendingDropClient(ws);
 	});
 }
 let lastBrowserContactMs = 0;
@@ -3784,10 +3839,7 @@ fromBrowserListeners.push((msg) => {
 	lastBrowserContactMs = Date.now();
 	const r = msg;
 	if (typeof r.id !== "string") return;
-	const p = pending.get(r.id);
-	if (!p) return;
-	pending.delete(r.id);
-	p.resolve(r);
+	pendingResolve(r.id, r);
 });
 httpServer.listen(0, "127.0.0.1", () => {
 	const addr = httpServer.address();