From 521092d66b11eed5dbaccc2b7a998fe69927b025 Mon Sep 17 00:00:00 2001
From: Pierre Kisters <1524059+lhns@users.noreply.github.com>
Date: Thu, 1 Oct 2026 04:07:39 +0200
Subject: [PATCH] UI patches: no re-injection timer for a session that closed
 while attaching

The attach handler set the 15 s interval after its awaits even when the
socket had closed meanwhile (onclose already ran), leaking a timer per
such session. Also documents the optional stateDir config key.
---
 modules/steam-ui-patches/injector.mjs | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/modules/steam-ui-patches/injector.mjs b/modules/steam-ui-patches/injector.mjs
index af8c2a0..9c19d58 100644
--- a/modules/steam-ui-patches/injector.mjs
+++ b/modules/steam-ui-patches/injector.mjs
@@ -7,6 +7,7 @@
 //     "reinjectMs": 15000,   // periodic re-injection
 //     "debounceMs": 3000,    // delay after a burst of new JS contexts
 //     "unpatchTimeoutMs": 2000,
+//     "stateDir": "...",     // optional, for manual runs (default below)
 //     "patches": [ { "name": "launcher-menu-order",
 //                    "endpoint": "http://127.0.0.1:8080",
 //                    "target": { "title": "SharedJSContext" }, // or titleRegex / urlRegex
@@ -150,7 +151,8 @@ function open(endpoint, target, list) {
     await call('Runtime.enable');
     if (stateful.size) await call('Runtime.addBinding', { name: BINDING });
     await inject().catch((e) => console.error(`${tag} inject:`, e.message));
-    timer = setInterval(() => inject().catch(() => {}), REINJECT);
+    // Closed while attaching: onclose already ran, so no timer.
+    if (ws.readyState === WebSocket.OPEN) timer = setInterval(() => inject().catch(() => {}), REINJECT);
   };
   ws.onmessage = (e) => {
     const m = JSON.parse(e.data);