@skyhook-io/radar-app 1.13.2 → 1.13.4

package/src/components/diagnose/DiagnoseSurface.test.tsx

@@ -1,15 +1,14 @@
 import { describe, expect, it, vi } from "vitest";
 import {
   DIAGNOSE_SURFACE_FRAME_CLASS,
-  MAXIMIZED_COMPACT_HISTORY_VISIBILITY_CLASS,
-  MAXIMIZED_HOME_DETAIL_VISIBILITY_CLASS,
-  MAXIMIZED_HOME_RUN_HEADER_VISIBILITY_CLASS,
   INVESTIGATION_HISTORY_MIN_WIDTH,
   MAXIMIZED_RUN_META_VISIBILITY_CLASS,
-  canStartNewInvestigation,
+  canRerunInvestigation,
   canCopyRunLink,
+  investigationHistoryIsPersistent,
   investigationHeaderPresentation,
   openInvestigationEvidenceResource,
+  unavailableInvestigationMessage,
 } from "./DiagnoseSurface";
 import {
   canContinueInvestigation,
@@ -18,7 +17,7 @@ import {
 } from "./investigationState";
 import type { RunSummary } from "../../api/diagnose";
 
-// The "new investigation" button dispatches an agent and spends the user's own
+// The re-run button dispatches an agent and spends the user's own
 // tokens, so every one of these clauses is load-bearing rather than cosmetic.
 // Each case below is a way it misfired before the gate existed.
 function run(status: RunSummary["status"]): RunSummary {
@@ -35,41 +34,40 @@ function run(status: RunSummary["status"]): RunSummary {
   };
 }
 
-describe("canStartNewInvestigation", () => {
+describe("canRerunInvestigation", () => {
   it("offers a new investigation on a finished run", () => {
-    expect(canStartNewInvestigation("investigation", run("done"), false)).toBe(
+    expect(canRerunInvestigation("investigation", run("done"), false)).toBe(
       true,
     );
   });
 
   it("stays hidden on the investigations list", () => {
-    // goHome() leaves activeRunId set, so the header still has a run to read.
-    // Without the view check the click starts an agent on a resource the user
-    // navigated away from, over a list of unrelated investigations.
-    expect(canStartNewInvestigation("home", run("done"), false)).toBe(false);
+    // Home is a fresh-entry surface and must never expose a token-spending
+    // re-run action, even if a stale caller hands the header a run.
+    expect(canRerunInvestigation("home", run("done"), false)).toBe(false);
   });
 
   it("stays hidden while a turn is in flight", () => {
     // A start would be handed back the live run, so the button does nothing.
     expect(
-      canStartNewInvestigation("investigation", run("running"), false),
+      canRerunInvestigation("investigation", run("running"), false),
     ).toBe(false);
   });
 
   it("stays hidden on a stale run", () => {
     // A closed session must not start a new investigation against a resource
     // that may not exist in the active cluster.
-    expect(canStartNewInvestigation("investigation", run("stale"), false)).toBe(
+    expect(canRerunInvestigation("investigation", run("stale"), false)).toBe(
       false,
     );
   });
 
   it("blocks fresh starts while a human turn stops, but allows a separate human run from an automatic one", () => {
     expect(
-      canStartNewInvestigation("investigation", run("stopping"), false),
+      canRerunInvestigation("investigation", run("stopping"), false),
     ).toBe(false);
     expect(
-      canStartNewInvestigation(
+      canRerunInvestigation(
         "investigation",
         { ...run("running"), trigger: "background" },
         false,
@@ -78,27 +76,39 @@ describe("canStartNewInvestigation", () => {
   });
 
   it("stays hidden while consent is pending", () => {
-    expect(canStartNewInvestigation("investigation", run("done"), true)).toBe(
+    expect(canRerunInvestigation("investigation", run("done"), true)).toBe(
       false,
     );
   });
 
   it("stays hidden with no focused run", () => {
-    expect(canStartNewInvestigation("investigation", null, false)).toBe(false);
+    expect(canRerunInvestigation("investigation", null, false)).toBe(false);
   });
 
   it("offers one on an errored or stopped run", () => {
     // Those are the runs a person most wants to start over from.
-    expect(canStartNewInvestigation("investigation", run("error"), false)).toBe(
+    expect(canRerunInvestigation("investigation", run("error"), false)).toBe(
       true,
     );
     expect(
-      canStartNewInvestigation("investigation", run("stopped"), false),
+      canRerunInvestigation("investigation", run("stopped"), false),
     ).toBe(true);
   });
 });
 
 describe("investigation history navigation", () => {
+  it("keeps hosted recovery language out of standalone Radar", () => {
+    expect(unavailableInvestigationMessage(false)).toBe(
+      "This investigation is unavailable. It may have been removed, or its saved history may have been cleared.",
+    );
+    expect(unavailableInvestigationMessage(false)).not.toMatch(
+      /account|organization|creator|access/,
+    );
+    expect(unavailableInvestigationMessage(true)).toMatch(
+      /account and organization/,
+    );
+  });
+
   it("restores the docked surface before opening an evidence resource", () => {
     const events: string[] = [];
     const onOpenResource = vi.fn(() => events.push("open"));
@@ -117,12 +127,13 @@ describe("investigation history navigation", () => {
       setMaximized,
       closeDiagnose,
       false,
+      "run-1",
     );
 
     expect(events).toEqual(["dock", "open"]);
     expect(setMaximized).toHaveBeenCalledWith(false);
     expect(closeDiagnose).not.toHaveBeenCalled();
-    expect(onOpenResource).toHaveBeenCalledWith(ref);
+    expect(onOpenResource).toHaveBeenCalledWith(ref, "run-1");
   });
 
   it("closes an overlay before opening an evidence resource", () => {
@@ -147,7 +158,7 @@ describe("investigation history navigation", () => {
     expect(events).toEqual(["close", "open"]);
     expect(setMaximized).not.toHaveBeenCalled();
     expect(closeDiagnose).toHaveBeenCalledOnce();
-    expect(onOpenResource).toHaveBeenCalledWith(ref);
+    expect(onOpenResource).toHaveBeenCalledWith(ref, null);
   });
 
   it("keeps document overflow out of the bounded Diagnose frame", () => {
@@ -159,20 +170,28 @@ describe("investigation history navigation", () => {
 
   it("reserves the history rail for wider investigation surfaces", () => {
     expect(INVESTIGATION_HISTORY_MIN_WIDTH).toBe(1750);
-    expect(MAXIMIZED_COMPACT_HISTORY_VISIBILITY_CLASS).toBe(
-      "@min-[1750px]/diagnose-surface:hidden",
-    );
-    expect(MAXIMIZED_HOME_DETAIL_VISIBILITY_CLASS).toBe(
-      "hidden @min-[1750px]/diagnose-surface:flex",
-    );
-    expect(MAXIMIZED_HOME_RUN_HEADER_VISIBILITY_CLASS).toBe(
-      "hidden @min-[1750px]/diagnose-surface:block",
-    );
     expect(MAXIMIZED_RUN_META_VISIBILITY_CLASS).toBe(
       "hidden @min-[1750px]/diagnose-surface:flex",
     );
   });
 
+  it("keeps history visible on Home even when the workspace is compact", () => {
+    expect(
+      investigationHistoryIsPersistent({
+        maximized: true,
+        view: "home",
+        surfaceWidth: 700,
+      }),
+    ).toBe(true);
+    expect(
+      investigationHistoryIsPersistent({
+        maximized: true,
+        view: "investigation",
+        surfaceWidth: 700,
+      }),
+    ).toBe(false);
+  });
+
   it("keeps docked Home generic and removes actions for its retained run", () => {
     expect(
       investigationHeaderPresentation({
@@ -187,7 +206,7 @@ describe("investigation history navigation", () => {
     });
   });
 
-  it("swaps generic Home identity for the labeled retained detail at the wide breakpoint", () => {
+  it("keeps Home generic even if a stale caller supplies retained detail", () => {
     expect(
       investigationHeaderPresentation({
         view: "home",
@@ -195,9 +214,9 @@ describe("investigation history navigation", () => {
         hasVisibleRunDetail: true,
       }),
     ).toEqual({
-      genericIdentityClass: MAXIMIZED_COMPACT_HISTORY_VISIBILITY_CLASS,
-      detailIdentityClass: MAXIMIZED_HOME_RUN_HEADER_VISIBILITY_CLASS,
-      runActionsClass: MAXIMIZED_HOME_DETAIL_VISIBILITY_CLASS,
+      genericIdentityClass: "",
+      detailIdentityClass: null,
+      runActionsClass: null,
     });
   });
 

package/src/components/diagnose/DiagnoseSurface.tsx

@@ -20,7 +20,7 @@ import {
   TerminalSquare,
   Copy,
   Check,
-  Plus,
+  RotateCcw,
   PanelLeftOpen,
   Link,
   Lock,
@@ -39,8 +39,15 @@ import {
   type DiagnoseView,
 } from "./DiagnoseContext";
 import { useDiagnoseCustomization } from "../../context/DiagnoseCustomization";
+import { useNavCustomization } from "../../context/NavCustomization";
 import { InvestigationView } from "./InvestigationView";
-import { RecentList, absoluteTime, statusWord } from "./Home";
+import type { InvestigationTimelineScope } from "./InvestigationEvidencePane";
+import {
+  InvestigationHome,
+  RecentList,
+  absoluteTime,
+  statusWord,
+} from "./Home";
 import { AgentSetupNotice } from "./AgentSetupNotice";
 import { ConsentCard } from "./parts";
 import { buildLaunchCommand, launchAgentLabel, openInTerminal } from "./launch";
@@ -334,12 +341,10 @@ function VisibilityControl({
 // header, right of the nav rail) — App renders it there and passes topInset (the
 // header height; 0 in chromeless embeds). It shares that frame with the resource/
 // Helm drawers, so it no longer floats viewport-fixed or DOM-measures the chrome.
-// Whether the header offers a new investigation on the focused run's resource.
+// Whether the header offers a fresh re-run of the focused investigation.
 // Every clause is a failure this button actually had:
 //
-//   view          goHome() leaves activeRunId set, so the header keeps rendering
-//                 the last focused run. Without this the button dispatches an
-//                 agent — real tokens — from a screen showing an unrelated list.
+//   view          Home is the fresh-entry surface, never a re-run action.
 //   run           nothing to take a resource from.
 //   running/stopping human starts reuse the live run. Automatic investigations
 //                 are immutable, so they retain a separate fresh human start.
@@ -347,7 +352,7 @@ function VisibilityControl({
 //                 the resource may not exist in the active context. Do not
 //                 silently start a different-cluster investigation from here.
 //   needsConsent  the consent card owns the surface until it's answered.
-export function canStartNewInvestigation(
+export function canRerunInvestigation(
   view: DiagnoseView,
   run: RunSummary | null,
   needsConsent: boolean,
@@ -355,6 +360,8 @@ export function canStartNewInvestigation(
   return (
     view === "investigation" &&
     !!run &&
+    !!run.kind &&
+    !!run.name &&
     ((run.status !== "running" && run.status !== "stopping") ||
       run.trigger === "background") &&
     run.status !== "stale" &&
@@ -366,12 +373,6 @@ export function canStartNewInvestigation(
 // Keep these Tailwind literals aligned with the measured rail threshold.
 // Detail uses one stable history toggle; below this width it opens an overlay.
 export const INVESTIGATION_HISTORY_MIN_WIDTH = 1750;
-export const MAXIMIZED_COMPACT_HISTORY_VISIBILITY_CLASS =
-  "@min-[1750px]/diagnose-surface:hidden";
-export const MAXIMIZED_HOME_DETAIL_VISIBILITY_CLASS =
-  "hidden @min-[1750px]/diagnose-surface:flex";
-export const MAXIMIZED_HOME_RUN_HEADER_VISIBILITY_CLASS =
-  "hidden @min-[1750px]/diagnose-surface:block";
 export const MAXIMIZED_RUN_META_VISIBILITY_CLASS =
   "hidden @min-[1750px]/diagnose-surface:flex";
 // The panel is a bounded absolute frame whose descendants own scrolling. If
@@ -382,6 +383,24 @@ export const MAXIMIZED_RUN_META_VISIBILITY_CLASS =
 export const DIAGNOSE_SURFACE_FRAME_CLASS =
   "@container/diagnose-surface absolute z-40 flex min-h-0 flex-col overflow-hidden border-l border-theme-border bg-theme-surface shadow-drawer";
 
+export function unavailableInvestigationMessage(embedded: boolean): string {
+  return embedded
+    ? "This investigation is unavailable. It may be private, your access may have changed, or its history may have been cleared. Check your account and organization, or ask the creator for access."
+    : "This investigation is unavailable. It may have been removed, or its saved history may have been cleared.";
+}
+
+export function investigationHistoryIsPersistent(input: {
+  maximized: boolean;
+  view: DiagnoseView;
+  surfaceWidth: number;
+}): boolean {
+  return (
+    input.maximized &&
+    (input.view === "home" ||
+      input.surfaceWidth >= INVESTIGATION_HISTORY_MIN_WIDTH)
+  );
+}
+
 export function investigationHeaderPresentation(input: {
   view: DiagnoseView;
   maximized: boolean;
@@ -398,17 +417,10 @@ export function investigationHeaderPresentation(input: {
       runActionsClass: input.hasVisibleRunDetail ? "" : null,
     };
   }
-  const hasWideRetainedDetail = input.maximized && input.hasVisibleRunDetail;
   return {
-    genericIdentityClass: hasWideRetainedDetail
-      ? MAXIMIZED_COMPACT_HISTORY_VISIBILITY_CLASS
-      : "",
-    detailIdentityClass: hasWideRetainedDetail
-      ? MAXIMIZED_HOME_RUN_HEADER_VISIBILITY_CLASS
-      : null,
-    runActionsClass: hasWideRetainedDetail
-      ? MAXIMIZED_HOME_DETAIL_VISIBILITY_CLASS
-      : null,
+    genericIdentityClass: "",
+    detailIdentityClass: null,
+    runActionsClass: null,
   };
 }
 
@@ -468,32 +480,56 @@ function DiagnoseHeaderIdentity({
 
 export function openInvestigationEvidenceResource(
   ref: DiagnosisResourceRef,
-  onOpenResource: (ref: DiagnosisResourceRef) => void,
+  onOpenResource: (ref: DiagnosisResourceRef, runID?: string | null) => void,
+  setMaximized: (maximized: boolean) => void,
+  dismissDiagnose: () => void,
+  dockedPanelWouldOverlay: boolean,
+  activeRunID?: string,
+) {
+  revealInvestigationDestination(
+    setMaximized,
+    dismissDiagnose,
+    dockedPanelWouldOverlay,
+  );
+  if (dockedPanelWouldOverlay) onOpenResource(ref, null);
+  else if (activeRunID) onOpenResource(ref, activeRunID);
+  else onOpenResource(ref);
+}
+
+// A destination must be visible after the handoff. On a wide canvas, restoring
+// the docked panel leaves Radar and the investigation side by side. At tighter
+// widths that same panel overlays the host content, so close it before
+// navigating instead of making the click appear to do nothing.
+function revealInvestigationDestination(
   setMaximized: (maximized: boolean) => void,
-  closeDiagnose: () => void,
+  dismissDiagnose: () => void,
   dockedPanelWouldOverlay: boolean,
 ) {
-  // A resource destination must be visible after the handoff. On a wide canvas,
-  // restoring the docked panel leaves Radar and the investigation side by side.
-  // At tighter widths that same panel overlays the host content, so close it
-  // before navigating instead of making the click appear to do nothing.
   if (dockedPanelWouldOverlay) {
     // Closing already exposes the destination; retain the user's maximized
     // preference for the next time they open investigations.
-    closeDiagnose();
+    dismissDiagnose();
   } else {
     setMaximized(false);
   }
-  onOpenResource(ref);
 }
 
 export function DiagnoseSurface({
   topInset = 0,
   onOpenResource,
+  onOpenTimeline,
+  onOpenWorkspace,
+  onBrowseIssues,
 }: {
   topInset?: number;
   /** Resolves an evidence subject into the embedding Radar surface. */
-  onOpenResource?: (ref: DiagnosisResourceRef) => void;
+  onOpenResource?: (ref: DiagnosisResourceRef, runID?: string | null) => void;
+  /** Opens the embedding Radar Timeline for one resource's change history. */
+  onOpenTimeline?: (scope: InvestigationTimelineScope) => void;
+  /** Lets a host whose drawer lives outside Radar's route tree cross into it. */
+  onOpenWorkspace?: () => void;
+  /** Opens the canonical Issues surface for a first focused investigation. */
+  onBrowseIssues?: () => void;
 }) {
   const d = useDiagnose();
   const { data: contexts } = useContexts();
@@ -518,6 +554,7 @@ export function DiagnoseSurface({
   // null = hide the gear + links.
   const { consentCopy, onOpenSettings: hostOpenSettings } =
     useDiagnoseCustomization();
+  const { embedded } = useNavCustomization();
   const openSettings =
     hostOpenSettings === undefined ? openDiagnoseSettings : hostOpenSettings;
   const {
@@ -529,10 +566,16 @@ export function DiagnoseSurface({
     panelBounds: { min: minW, max: maxW },
     panelWidthKey: widthKey,
   } = useDiagnoseLayout();
-  const wideHistory =
-    maximized && surfaceWidth >= INVESTIGATION_HISTORY_MIN_WIDTH;
+  // Home has no detail pane competing for width, so its history is always part
+  // of the workspace. Only a focused run collapses history into a drawer when
+  // the two-pane layout would become cramped.
+  const persistentHistory = investigationHistoryIsPersistent({
+    maximized,
+    view: d.view,
+    surfaceWidth,
+  });
   const historyOverlay =
-    !wideHistory && historyOverlayOpen && d.view !== "home";
+    !persistentHistory && historyOverlayOpen;
   const { shouldRender: historyOverlayPresent, isOpen: historySlideOpen } =
     useAnimatedUnmount(historyOverlay);
   const dismissHistory = useCallback(() => {
@@ -541,7 +584,7 @@ export function DiagnoseSurface({
   }, []);
   useEffect(() => {
     setHistoryOverlayOpen(false);
-  }, [wideHistory, maximized, d.view]);
+  }, [persistentHistory, maximized, d.view]);
   useEffect(() => {
     if (historyOverlay) {
       const target =
@@ -558,11 +601,30 @@ export function DiagnoseSurface({
         ref,
         onOpenResource,
         setMaximized,
-        d.close,
+        d.dismissForNavigation,
         narrow,
+        d.activeRunId ?? undefined,
       );
     },
-    [d.close, narrow, onOpenResource, setMaximized],
+    [
+      d.activeRunId,
+      d.dismissForNavigation,
+      narrow,
+      onOpenResource,
+      setMaximized,
+    ],
+  );
+  const openEvidenceTimeline = useCallback(
+    (scope: InvestigationTimelineScope) => {
+      if (!onOpenTimeline) return;
+      revealInvestigationDestination(
+        setMaximized,
+        d.dismissForNavigation,
+        narrow,
+      );
+      onOpenTimeline(scope);
+    },
+    [d.dismissForNavigation, narrow, onOpenTimeline, setMaximized],
   );
 
   const startResize = (e: React.MouseEvent) => {
@@ -660,6 +722,7 @@ export function DiagnoseSurface({
       agentLabel={activeAgentLabel}
       maximized={maximized}
       onOpenResource={onOpenResource ? openEvidenceResource : undefined}
+      onOpenTimeline={onOpenTimeline ? openEvidenceTimeline : undefined}
     />
   ) : d.activeRunId && !d.runsLoaded ? (
     // Deep-linked to a run before the list has ever loaded: show the load
@@ -676,9 +739,7 @@ export function DiagnoseSurface({
     // investigation" placeholder would read as a broken link.
     <div className="flex flex-1 flex-col items-center justify-center gap-3 px-6 text-center">
       <p className="text-sm text-theme-text-secondary">
-        This investigation is unavailable. It may be private, your access may
-        have changed, or its history may have been cleared. Check your account
-        and organization, or ask the creator for access.
+        {unavailableInvestigationMessage(embedded === true)}
       </p>
       <button
         onClick={d.goHome}
@@ -706,25 +767,38 @@ export function DiagnoseSurface({
       Select an investigation, or open a resource and click Investigate.
     </div>
   );
-  const compactHistory = (
-    <>
-      {setupPending && <AgentSetupNotice setupState={d.setupState} />}
-      {(!setupPending || d.runs.length > 0) && (
-        <RecentList
-          currentContext={currentContext}
-          agentLabel={d.agentLabel}
-          runs={d.runs}
-          onSelect={d.openRun}
-          historyDegraded={d.historyDegraded}
-        />
-      )}
-    </>
+  const home = d.needsConsent ? (
+    <div className="mx-auto w-full max-w-3xl px-4 py-8 sm:px-6">
+      <ConsentCard
+        agentName={d.agentLabel}
+        agent={d.selectedAgent}
+        profile={d.profile}
+        copy={consentCopy}
+        onOpenSettings={openSettings ?? undefined}
+        error={d.consentError}
+        onApprove={d.approveConsent}
+        onCancel={d.cancelConsent}
+      />
+    </div>
+  ) : setupPending ? (
+    <div className="mx-auto w-full max-w-3xl px-4 py-8 sm:px-6">
+      <AgentSetupNotice setupState={d.setupState} />
+    </div>
+  ) : (
+    <InvestigationHome
+      agentLabel={d.agentLabel}
+      onBrowseIssues={
+        d.runsLoaded && d.runs.length === 0 ? onBrowseIssues : undefined
+      }
+    />
   );
 
   const showHistory = !setupPending || d.runs.length > 0;
   const historyVisible =
     showHistory &&
-    (wideHistory ? !historyCollapsed || d.view === "home" : historyOverlay);
+    (persistentHistory
+      ? d.view === "home" || !historyCollapsed
+      : historyOverlay);
 
   useLayoutEffect(() => {
     if (
@@ -764,9 +838,9 @@ export function DiagnoseSurface({
       )}
 
       {/* Header */}
-      <div className="flex items-center justify-between border-b border-theme-border px-4 py-2.5">
+      <div className="relative z-30 flex items-center justify-between border-b border-theme-border bg-theme-surface px-4 py-2.5">
         <div className="flex min-w-0 flex-1 items-center gap-2">
-          {d.view !== "home" && showHistory ? (
+          {showHistory && (d.view !== "home" || !persistentHistory) ? (
             <Tooltip
               content={
                 historyVisible
@@ -783,7 +857,7 @@ export function DiagnoseSurface({
                 aria-expanded={historyVisible}
                 aria-controls="investigation-history"
                 onClick={() =>
-                  wideHistory
+                  persistentHistory
                     ? setHistoryCollapsed((value) => !value)
                     : historyOverlay
                       ? dismissHistory()
@@ -821,9 +895,9 @@ export function DiagnoseSurface({
         </div>
         <div className="flex shrink-0 items-center gap-0.5">
           {activeRun &&
-            canStartNewInvestigation(d.view, activeRun, d.needsConsent) && (
+            canRerunInvestigation(d.view, activeRun, d.needsConsent) && (
               <Tooltip
-                content="Start fresh — ignore earlier findings"
+                content="Re-run this investigation from scratch"
                 position="bottom"
               >
                 <button
@@ -838,9 +912,9 @@ export function DiagnoseSurface({
                     })
                   }
                   className="rounded-md p-1 text-theme-text-tertiary hover:bg-theme-hover hover:text-theme-text-primary"
-                  aria-label="Start fresh — ignore earlier findings"
+                  aria-label="Re-run this investigation from scratch"
                 >
-                  <Plus className="h-4 w-4" />
+                  <RotateCcw className="h-4 w-4" />
                 </button>
               </Tooltip>
             )}
@@ -862,19 +936,29 @@ export function DiagnoseSurface({
               <InvestigationMenu run={visibleRunDetail} />
             </div>
           )}
-          <Tooltip content={maximized ? "Restore" : "Expand"} position="bottom">
-            <button
-              onClick={() => setMaximized((v) => !v)}
-              className="rounded-md p-1 text-theme-text-tertiary hover:bg-theme-hover hover:text-theme-text-primary"
-              aria-label={maximized ? "Restore" : "Expand"}
+          {(!maximized || d.canRestoreWorkspace) && (
+            <Tooltip
+              content={maximized ? "Restore" : "Expand"}
+              position="bottom"
             >
-              {maximized ? (
-                <Minimize2 className="h-4 w-4" />
-              ) : (
-                <Maximize2 className="h-4 w-4" />
-              )}
-            </button>
-          </Tooltip>
+              <button
+                onClick={
+                  maximized
+                    ? d.restoreWorkspace
+                    : (onOpenWorkspace ??
+                      (() => d.openWorkspace(d.activeRunId)))
+                }
+                className="rounded-md p-1 text-theme-text-tertiary hover:bg-theme-hover hover:text-theme-text-primary"
+                aria-label={maximized ? "Restore" : "Expand"}
+              >
+                {maximized ? (
+                  <Minimize2 className="h-4 w-4" />
+                ) : (
+                  <Maximize2 className="h-4 w-4" />
+                )}
+              </button>
+            </Tooltip>
+          )}
           <Tooltip content="Close" position="bottom">
             <button
               onClick={d.close}
@@ -893,7 +977,7 @@ export function DiagnoseSurface({
           only appears when expanded; keys keep the detail node identity-stable
           as it comes and goes. */}
       <div className="relative flex min-h-0 flex-1">
-        {!wideHistory && historyOverlayPresent && (
+        {!persistentHistory && historyOverlayPresent && (
           <div
             aria-hidden="true"
             onClick={dismissHistory}
@@ -909,7 +993,7 @@ export function DiagnoseSurface({
             aria-hidden={!historyVisible}
             inert={!historyVisible}
             id="investigation-history"
-            className={`${historyVisible || (!wideHistory && historyOverlayPresent) ? "block" : "hidden"} ${wideHistory ? "" : `absolute inset-y-0 left-0 z-20 max-w-[calc(100%-2rem)] shadow-drawer ${TRANSITION_DRAWER} motion-reduce:transition-none ${historySlideOpen ? "translate-x-0 opacity-100" : "-translate-x-full opacity-0"}`} w-72 shrink-0 overflow-y-auto border-r border-theme-border bg-theme-surface px-3 py-3 outline-none`}
+            className={`${historyVisible || (!persistentHistory && historyOverlayPresent) ? "block" : "hidden"} ${persistentHistory ? "" : `absolute inset-y-0 left-0 z-20 max-w-[calc(100%-2rem)] ${TRANSITION_DRAWER} motion-reduce:transition-none ${historySlideOpen ? "translate-x-0 opacity-100" : "-translate-x-full opacity-0"}`} w-72 shrink-0 overflow-y-auto border-r border-theme-border bg-theme-surface px-3 py-3 outline-none`}
           >
             <RecentList
               currentContext={currentContext}
@@ -925,22 +1009,13 @@ export function DiagnoseSurface({
           </aside>
         )}
         {d.view === "home" ? (
-          <>
-            <div
-              key="history"
-              className={`flex-1 overflow-y-auto overflow-x-hidden px-4 py-3 ${maximized ? MAXIMIZED_COMPACT_HISTORY_VISIBILITY_CLASS : ""}`}
-            >
-              {compactHistory}
-            </div>
-            {maximized && (
-              <div
-                key="main"
-                className={`${MAXIMIZED_HOME_DETAIL_VISIBILITY_CLASS} min-h-0 min-w-0 flex-1 flex-col`}
-              >
-                {detail}
-              </div>
-            )}
-          </>
+          <div
+            key="home"
+            inert={historyOverlay}
+            className="flex-1 overflow-y-auto overflow-x-hidden"
+          >
+            {home}
+          </div>
         ) : (
           <div
             key="main"