Keep routing activity listener through slow panel mounts

Author: gitcommit90

src/client/routing.ts

@@ -832,11 +832,19 @@ export function routingPanel(isAdmin: boolean, confirm: Dialog): HTMLElement {
     applyRoutingActivity(state, activity);
     const old = content.querySelector(".routing-fabric");
     if (old) old.replaceWith(routingFabric(state));
-    if (!shell.isConnected) setTimeout(() => {
-      if (!shell.isConnected) routingActivityListeners.delete(shellActivityListener);
-    }, 1_000);
   };
   routingActivityListeners.add(shellActivityListener);
+  // routingPanel() is built before Settings attaches it to the document. Never
+  // interpret that initial detached state as disposal: slower browsers can take
+  // more than a second to mount the panel and would silently lose live events.
+  let shellWasConnected = false;
+  const shellLifecycle = new MutationObserver(() => {
+    if (shell.isConnected) { shellWasConnected = true; return; }
+    if (!shellWasConnected) return;
+    routingActivityListeners.delete(shellActivityListener);
+    shellLifecycle.disconnect();
+  });
+  shellLifecycle.observe(document.documentElement, { childList: true, subtree: true });
   const expandedAccounts = new Set<string>();
   const expandedGroups = new Set<string>();
   const views: Array<[RoutingView, string]> = isAdmin

test/routing-ui-contract.mjs

@@ -35,6 +35,10 @@ test("provider controls expose the live dotted router flow and credential-free h
   assert.doesNotMatch(server, /pendingSystemRequests|candidate\.model === String\(event\.request\?\.model/,
     "system classification never guesses from an imminent same-model user request");
   assert.doesNotMatch(client, /routing-pyramid-tier/);
+  assert.match(client, /let shellWasConnected = false;[\s\S]*if \(shell\.isConnected\) \{ shellWasConnected = true; return; \}[\s\S]*routingActivityListeners\.delete\(shellActivityListener\)/,
+    "the Settings routing listener survives its initial detached construction and is removed only after a real mount/unmount lifecycle");
+  assert.doesNotMatch(client, /if \(!shell\.isConnected\) setTimeout/,
+    "slow Settings mounts never lose live routing events to a timer-based detached-state guess");
   assert.doesNotMatch(client.slice(client.indexOf("export async function openRoutingPopover"), client.indexOf("function sourceCatalog")), /routing\/credentials|apiKey/);
 });