fix(web-ui): satisfy session navigation lint contracts

Author: GCWing

src/web-ui/src/app/components/NavPanel/sections/sessions/SessionsSection.tsx

@@ -293,9 +293,11 @@ const SessionsSection: React.FC<SessionsSectionProps> = ({
     sessionNavStatusService.getOrderingSnapshot,
     sessionNavStatusService.getOrderingSnapshot,
   );
-  const runningSessionIds = useMemo(() => new Set(
-    [...flowChatState.sessions.keys()].filter(sessionNavStatusService.isRunning),
-  ), [flowChatState.sessions, orderingRevision]);
+  const runningSessionIds = useMemo(() => {
+    // The revision invalidates these reads from the live navigation service.
+    void orderingRevision;
+    return new Set([...flowChatState.sessions.keys()].filter(sessionNavStatusService.isRunning));
+  }, [flowChatState.sessions, orderingRevision]);
   const [scheduledJobsSessionId, setScheduledJobsSessionId] = useState<string | null>(null);
   const [batchWorkspace, setBatchWorkspace] = useState<WorkspaceSessionScope | null>(null);
   const editInputRef = useRef<HTMLInputElement>(null);
@@ -798,6 +800,8 @@ const SessionsSection: React.FC<SessionsSectionProps> = ({
   );
 
   const { topLevelSessions: allTopLevelSessions, childrenByParent } = useMemo(() => {
+    // Activity timestamps can change independently of the session records.
+    void orderingRevision;
     const childMap = new Map<string, Session[]>();
     const parents: Session[] = [];
 

src/web-ui/src/app/stores/sceneStore.ts

@@ -72,38 +72,6 @@ export interface SessionSceneNavigation {
 let sessionNavigation: SessionSceneNavigation | undefined;
 let navigationRequest = 0;
 
-export function registerSessionSceneNavigation(adapter: SessionSceneNavigation): () => void {
-  sessionNavigation = adapter;
-  return () => {
-    if (sessionNavigation === adapter) {
-      sessionNavigation = undefined;
-      navigationRequest++;
-      if (useSceneStore.getState().pendingTabId !== null) useSceneStore.setState({ pendingTabId: null });
-    }
-  };
-}
-
-/** Every navigation path, including history and close fallback, uses this gate. */
-function navigateToScene(id: SceneTabId | null, commit: () => void, session?: SessionSceneTarget): void {
-  const hadPendingNavigation = useSceneStore.getState().pendingTabId !== null;
-  const request = ++navigationRequest;
-  const adapter = sessionNavigation;
-  const target = session ?? useSceneStore.getState().openTabs.find(tab => tab.id === id)?.session;
-  const isCurrent = () => request === navigationRequest && adapter === sessionNavigation;
-  if (!target || !adapter || (!hadPendingNavigation && adapter.isActive(target))) {
-    if (useSceneStore.getState().pendingTabId !== null) useSceneStore.setState({ pendingTabId: null });
-    commit();
-    return;
-  }
-
-  useSceneStore.setState({ pendingTabId: id });
-  void adapter.activate(target, isCurrent).then(activated => {
-    if (activated && isCurrent()) commit();
-  }).finally(() => {
-    if (isCurrent()) useSceneStore.setState({ pendingTabId: null });
-  });
-}
-
 function resolveNavSceneId(sceneId: SceneTabId | null): SceneTabId | null {
   if (sceneId === null) return null;
   return getSceneNav(sceneId) ? sceneId : null;
@@ -314,6 +282,38 @@ export const useSceneStore = create<SceneState>((set, get) => ({
   },
 }));
 
+export function registerSessionSceneNavigation(adapter: SessionSceneNavigation): () => void {
+  sessionNavigation = adapter;
+  return () => {
+    if (sessionNavigation === adapter) {
+      sessionNavigation = undefined;
+      navigationRequest++;
+      if (useSceneStore.getState().pendingTabId !== null) useSceneStore.setState({ pendingTabId: null });
+    }
+  };
+}
+
+/** Every navigation path, including history and close fallback, uses this gate. */
+function navigateToScene(id: SceneTabId | null, commit: () => void, session?: SessionSceneTarget): void {
+  const hadPendingNavigation = useSceneStore.getState().pendingTabId !== null;
+  const request = ++navigationRequest;
+  const adapter = sessionNavigation;
+  const target = session ?? useSceneStore.getState().openTabs.find(tab => tab.id === id)?.session;
+  const isCurrent = () => request === navigationRequest && adapter === sessionNavigation;
+  if (!target || !adapter || (!hadPendingNavigation && adapter.isActive(target))) {
+    if (useSceneStore.getState().pendingTabId !== null) useSceneStore.setState({ pendingTabId: null });
+    commit();
+    return;
+  }
+
+  useSceneStore.setState({ pendingTabId: id });
+  void adapter.activate(target, isCurrent).then(activated => {
+    if (activated && isCurrent()) commit();
+  }).finally(() => {
+    if (isCurrent()) useSceneStore.setState({ pendingTabId: null });
+  });
+}
+
 function openSceneTarget(id: SceneTabId, session?: SessionSceneTarget): void {
   const get = useSceneStore.getState;
   const set = useSceneStore.setState;