@skyhook-io/radar-app 1.8.6 → 1.8.7

package/src/components/diagnose/DiagnoseSurface.tsx

@@ -0,0 +1,385 @@
+// The right-docked shell of the AI surface. Two layouts:
+//  - docked: a single-pane right column (app reflows left via the provider's push)
+//  - expanded: a master-detail workspace that fills ONLY the content area (does
+//    not cover the left nav rail or top bar) — recent list on the left, the
+//    selected investigation/report on the right.
+import { useState } from "react";
+import {
+  Sparkles,
+  X,
+  Maximize2,
+  Minimize2,
+  ChevronLeft,
+  Settings2,
+  MoreVertical,
+  TerminalSquare,
+  Copy,
+  Check,
+} from "lucide-react";
+import { Tooltip } from "../ui/Tooltip";
+import {
+  useDiagnose,
+  useDiagnoseLayout,
+  agentLabelFor,
+  openDiagnoseSettings,
+} from "./DiagnoseContext";
+import { InvestigationView } from "./InvestigationView";
+import { RecentList } from "./Home";
+import { ConsentCard } from "./parts";
+import { buildLaunchCommand, launchAgentLabel, openInTerminal } from "./launch";
+import { type RunSummary } from "../../api/diagnose";
+
+function capWord(s: string): string {
+  return s ? s[0].toUpperCase() + s.slice(1) : s;
+}
+
+// buildConfigLine renders the active AI config as the header subtitle. Codex shows
+// its isolation mode + effective reasoning effort (Default → medium); a model
+// override is shown for either agent. Reflects a run's recorded settings, or the
+// current defaults on Home.
+function buildConfigLine(cfg: {
+  agent?: string;
+  isolated?: boolean;
+  model?: string;
+  effort?: string;
+}): string {
+  const parts = [agentLabelFor(cfg.agent ?? "")];
+  if (cfg.agent === "codex") {
+    parts.push(cfg.isolated === false ? "My setup" : "Isolated");
+    parts.push(`${capWord(cfg.effort || "medium")} effort`);
+  }
+  if (cfg.model) parts.push(capWord(cfg.model));
+  return "via " + parts.join(" · ");
+}
+
+// InvestigationMenu hands an investigation off to the user's own full agent. The
+// PRIMARY action copies a resume command they can paste wherever they actually
+// work (terminal, tmux, IDE) — destination-agnostic. Running it in Radar's built-in
+// terminal is the secondary "run it here" convenience.
+function InvestigationMenu({ run }: { run: RunSummary }) {
+  const [open, setOpen] = useState(false);
+  const [copied, setCopied] = useState(false);
+  const label = launchAgentLabel(run);
+  const command = buildLaunchCommand(run, `${window.location.origin}/mcp`);
+  // No resumable session yet (or stale run) → nothing to hand off.
+  if (!command) return null;
+
+  const toggle = () => {
+    setCopied(false);
+    setOpen((v) => !v);
+  };
+  const copy = () => {
+    void navigator.clipboard?.writeText(command);
+    setCopied(true);
+    setTimeout(() => {
+      setCopied(false);
+      setOpen(false);
+    }, 1100);
+  };
+
+  return (
+    <div className="relative flex items-center">
+      <Tooltip content="More" position="bottom">
+        <button
+          onClick={toggle}
+          className="rounded-md p-1 text-theme-text-tertiary hover:bg-theme-hover hover:text-theme-text-primary"
+          aria-label="More actions"
+          aria-haspopup="menu"
+          aria-expanded={open}
+        >
+          <MoreVertical className="h-4 w-4" />
+        </button>
+      </Tooltip>
+      {open && (
+        <>
+          <div className="fixed inset-0 z-10" onClick={() => setOpen(false)} />
+          <div className="absolute right-0 top-full z-20 mt-1 w-64 rounded-lg border border-theme-border bg-theme-surface py-1 shadow-theme-lg">
+            <button
+              onClick={copy}
+              className="flex w-full items-start gap-2 px-3 py-1.5 text-left text-sm text-theme-text-primary hover:bg-theme-hover"
+            >
+              {copied ? (
+                <Check className="mt-0.5 h-4 w-4 shrink-0 text-emerald-500" />
+              ) : (
+                <Copy className="mt-0.5 h-4 w-4 shrink-0 text-theme-text-tertiary" />
+              )}
+              <span>
+                {copied ? "Copied ✓" : `Copy command to continue in ${label}`}
+                {!copied && (
+                  <span className="block text-[11px] text-theme-text-tertiary">
+                    Paste it wherever you run {label} — resumes this exact
+                    session.
+                  </span>
+                )}
+              </span>
+            </button>
+            <button
+              onClick={() => {
+                openInTerminal(command, "Diagnose");
+                setOpen(false);
+              }}
+              className="flex w-full items-center gap-2 px-3 py-1.5 text-left text-sm text-theme-text-primary hover:bg-theme-hover"
+            >
+              <TerminalSquare className="h-4 w-4 shrink-0 text-theme-text-tertiary" />
+              Run in a Radar terminal
+            </button>
+          </div>
+        </>
+      )}
+    </div>
+  );
+}
+
+// The panel is an ABSOLUTE slot inside the app's body frame (the column under the
+// 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.
+export function DiagnoseSurface({ topInset = 0 }: { topInset?: number }) {
+  const d = useDiagnose();
+  const {
+    maximized,
+    setMaximized,
+    panelWidth: width,
+    setPanelWidth: setWidth,
+    panelNarrow: narrow,
+    panelBounds: { min: minW, max: maxW },
+    panelWidthKey: widthKey,
+  } = useDiagnoseLayout();
+
+  const startResize = (e: React.MouseEvent) => {
+    e.preventDefault();
+    const onMove = (m: MouseEvent) =>
+      setWidth(() =>
+        Math.min(maxW, Math.max(minW, window.innerWidth - m.clientX)),
+      );
+    const onUp = () => {
+      document.removeEventListener("mousemove", onMove);
+      document.removeEventListener("mouseup", onUp);
+      setWidth((w) => {
+        try {
+          localStorage.setItem(widthKey, String(w));
+        } catch {
+          /* storage disabled — width just won't persist */
+        }
+        return w;
+      });
+    };
+    document.addEventListener("mousemove", onMove);
+    document.addEventListener("mouseup", onUp);
+  };
+
+  const activeRun = d.runs.find((r) => r.id === d.activeRunId) ?? null;
+  // A focused run shows the agent it actually ran with; Home reflects the current pick.
+  const activeAgentLabel = activeRun?.agent
+    ? agentLabelFor(activeRun.agent)
+    : d.agentLabel;
+  // Header subtitle: the config a focused run actually used (it records agent /
+  // isolation / model / effort), or the current defaults on Home. Codex shows mode
+  // + reasoning effort; model is shown only when overridden. Clicking opens Settings.
+  const configLine = buildConfigLine(
+    activeRun ?? {
+      agent: d.selectedAgent,
+      isolated: d.isolated,
+      model: d.model,
+      effort: d.effort,
+    },
+  );
+  const detailTitle = activeRun
+    ? `${activeRun.kind} ${activeRun.namespace ? `${activeRun.namespace}/` : ""}${activeRun.name}`
+    : "AI investigations";
+
+  // Absolute within the body frame: maximized fills it; docked is a right slot.
+  // topInset clears the header (the frame spans the full column incl. the header).
+  const positionStyle: React.CSSProperties = maximized
+    ? { top: topInset, left: 0, right: 0, bottom: 0 }
+    : { top: topInset, right: 0, bottom: 0, width, maxWidth: "100%" };
+
+  // The detail pane (right side when expanded; the whole body when docked).
+  // Keyed by run id so toggling Expand doesn't remount a focused run's view.
+  const detail = d.needsConsent ? (
+    <div className="flex-1 overflow-y-auto px-4 py-3">
+      <div className={maximized ? "mx-auto max-w-3xl" : ""}>
+        <ConsentCard
+          agentName={d.agentLabel}
+          agent={d.selectedAgent}
+          isolated={d.isolated}
+          onOpenSettings={openDiagnoseSettings}
+          onApprove={d.approveConsent}
+          onCancel={d.cancelConsent}
+        />
+      </div>
+    </div>
+  ) : activeRun ? (
+    <InvestigationView
+      key={activeRun.id}
+      run={activeRun}
+      agentLabel={activeAgentLabel}
+      maximized={maximized}
+    />
+  ) : d.activeRunId && !d.runsLoaded ? (
+    // Deep-linked to a run before the list has ever loaded: show the load
+    // state (the 4s poll retries while the panel is open) — never the generic
+    // placeholder, and never a premature "no longer available".
+    <div className="flex flex-1 items-center justify-center px-6 text-center text-sm text-theme-text-tertiary">
+      {d.runsLoadFailed
+        ? "Couldn't load investigations — retrying…"
+        : "Loading investigations…"}
+    </div>
+  ) : d.activeRunId && d.runsLoaded ? (
+    // A focused id that isn't in the loaded list — a deep link (?ai-run=…) to a
+    // cleared/evicted/unknown run. Say so; the generic "select an
+    // 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 no longer available — history keeps the most
+        recent investigations, and this one has been cleared.
+      </p>
+      <button
+        onClick={d.goHome}
+        className="rounded-lg border border-theme-border px-3 py-1.5 text-sm text-theme-text-secondary hover:bg-theme-hover"
+      >
+        View investigations
+      </button>
+    </div>
+  ) : d.startError ? (
+    <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">{d.startError}</p>
+      <button
+        onClick={d.dismissError}
+        className="rounded-lg border border-theme-border px-3 py-1.5 text-sm text-theme-text-secondary hover:bg-theme-hover"
+      >
+        Dismiss
+      </button>
+    </div>
+  ) : (
+    <div className="flex flex-1 items-center justify-center px-6 text-center text-sm text-theme-text-tertiary">
+      Select an investigation, or open a resource and click Diagnose.
+    </div>
+  );
+
+  const showBreadcrumb = !maximized && d.view !== "home";
+  // The maximized workspace always shows the recent-investigations list (there's
+  // room for it in full-wide) — it's the master pane of the master-detail layout.
+  const showHistory = maximized;
+
+  return (
+    <div
+      role="dialog"
+      aria-label="AI investigations"
+      className="absolute z-40 flex flex-col border-l border-theme-border bg-theme-surface shadow-drawer"
+      style={{
+        ...positionStyle,
+        animation: "slide-in-from-right 0.22s cubic-bezier(0.32,0.72,0,1)",
+      }}
+    >
+      {!maximized && !narrow && (
+        <div
+          onMouseDown={startResize}
+          className="absolute left-0 top-0 z-10 h-full w-1.5 cursor-ew-resize bg-theme-border/40 transition-colors hover:bg-accent/50"
+          role="separator"
+          aria-orientation="vertical"
+          aria-label="Resize panel"
+        />
+      )}
+
+      {/* Header */}
+      <div className="flex items-center justify-between border-b border-theme-border px-4 py-2.5">
+        <div className="flex min-w-0 items-center gap-2">
+          {!showBreadcrumb && (
+            <Sparkles className="h-4 w-4 shrink-0 text-accent" />
+          )}
+          <div className="min-w-0">
+            {showBreadcrumb && (
+              <button
+                onClick={d.goHome}
+                className="-ml-1 mb-0.5 flex items-center gap-0.5 rounded px-1 text-[11px] text-theme-text-tertiary hover:text-theme-text-primary"
+              >
+                <ChevronLeft className="h-3 w-3" />
+                Investigations
+              </button>
+            )}
+            <div className="truncate text-sm font-medium text-theme-text-primary">
+              {detailTitle}
+            </div>
+            <div className="flex items-center gap-1 text-xs text-theme-text-tertiary">
+              <span className="truncate">{configLine}</span>
+              <Tooltip content="AI settings" position="bottom">
+                <button
+                  onClick={openDiagnoseSettings}
+                  className="shrink-0 rounded p-0.5 text-theme-text-tertiary hover:text-theme-text-primary"
+                  aria-label="AI settings"
+                >
+                  <Settings2 className="h-3 w-3" />
+                </button>
+              </Tooltip>
+            </div>
+          </div>
+        </div>
+        <div className="flex shrink-0 items-center gap-0.5">
+          {activeRun && <InvestigationMenu run={activeRun} />}
+          <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 ? (
+                <Minimize2 className="h-4 w-4" />
+              ) : (
+                <Maximize2 className="h-4 w-4" />
+              )}
+            </button>
+          </Tooltip>
+          <Tooltip content="Close" position="bottom">
+            <button
+              onClick={d.close}
+              className="rounded-md p-1 text-theme-text-tertiary hover:bg-theme-hover hover:text-theme-text-primary"
+              aria-label="Close"
+            >
+              <X className="h-4 w-4" />
+            </button>
+          </Tooltip>
+        </div>
+      </div>
+
+      {/* Body. The detail wrapper keeps a stable position + key across both
+          layouts so toggling Expand doesn't remount a live InvestigationView
+          (which would discard its transcript and re-run the agent). The aside
+          only appears when expanded; keys keep the detail node identity-stable
+          as it comes and goes. */}
+      <div className="flex min-h-0 flex-1">
+        {showHistory && (
+          <aside
+            key="recent"
+            className="w-72 shrink-0 overflow-y-auto border-r border-theme-border px-3 py-3"
+          >
+            <RecentList
+              agentLabel={d.agentLabel}
+              runs={d.runs}
+              selectedId={d.activeRunId}
+              onSelect={d.openRun}
+              historyDegraded={d.historyDegraded}
+            />
+          </aside>
+        )}
+        {!maximized && d.view === "home" ? (
+          <div
+            key="main"
+            className="flex-1 overflow-y-auto overflow-x-hidden px-4 py-3"
+          >
+            <RecentList
+              agentLabel={d.agentLabel}
+              runs={d.runs}
+              onSelect={d.openRun}
+              historyDegraded={d.historyDegraded}
+            />
+          </div>
+        ) : (
+          <div key="main" className="flex min-h-0 min-w-0 flex-1 flex-col">
+            {detail}
+          </div>
+        )}
+      </div>
+    </div>
+  );
+}

package/src/components/diagnose/Home.tsx

@@ -0,0 +1,163 @@
+// The recent-investigations list — now backed by server-side runs (the source of
+// truth), so background/running investigations appear here live. Used both as the
+// docked Home view and the master pane of the maximized workspace.
+import { Loader2, Sparkles } from "lucide-react";
+import { StatusDot, type StatusTone } from "@skyhook-io/k8s-ui";
+import { type RunSummary } from "../../api/diagnose";
+
+// Compact "3m ago" / "2h ago" / date label.
+function relativeTime(ts: number, now: number): string {
+  const s = Math.max(0, Math.round((now - ts) / 1000));
+  if (s < 60) return "just now";
+  if (s < 3600) return `${Math.floor(s / 60)}m ago`;
+  if (s < 86400) return `${Math.floor(s / 3600)}h ago`;
+  if (s < 7 * 86400) return `${Math.floor(s / 86400)}d ago`;
+  return new Date(ts).toLocaleDateString();
+}
+
+// Map a run status to the design-system status tone (StatusDot). stopped is
+// user-initiated → neutral/unknown, NOT a failure (distinct from error).
+function runTone(status: RunSummary["status"]): StatusTone {
+  switch (status) {
+    case "error":
+      return "unhealthy";
+    case "stale":
+      return "degraded";
+    case "done":
+      return "healthy";
+    default: // stopped
+      return "unknown";
+  }
+}
+
+function statusDot(status: RunSummary["status"]) {
+  if (status === "running")
+    return <Loader2 className="h-3 w-3 shrink-0 animate-spin text-accent" />;
+  return <StatusDot tone={runTone(status)} className="shrink-0" />;
+}
+
+// A short text status for terminal non-done states, so the run's outcome doesn't
+// rely on decoding a 6px colored dot (and so "I stopped it" reads differently from
+// "it failed"). Done/running are conveyed by the dot + time already.
+function statusWord(
+  status: RunSummary["status"],
+): { text: string; cls: string } | null {
+  switch (status) {
+    case "error":
+      return { text: "Failed", cls: "text-red-400" };
+    case "stopped":
+      return { text: "Stopped", cls: "text-theme-text-tertiary" };
+    case "stale":
+      // Plain words, not the internal status name: "stale" means the run was
+      // about a cluster that's no longer connected.
+      return { text: "Different cluster", cls: "text-amber-500" };
+    default:
+      return null;
+  }
+}
+
+export function RecentList({
+  agentLabel,
+  runs,
+  onSelect,
+  selectedId,
+  historyDegraded = false,
+}: {
+  agentLabel: string;
+  runs: RunSummary[];
+  onSelect: (id: string) => void;
+  selectedId?: string | null;
+  historyDegraded?: boolean;
+}) {
+  const now = Date.now();
+
+  // Persistence broke (disk error) — without this the user reasonably assumes
+  // their history survives a restart, and it won't.
+  const degradedNote = historyDegraded ? (
+    <div className="mb-2 rounded-md border border-amber-500/40 bg-amber-500/10 px-2.5 py-1.5 text-[11px] leading-snug text-theme-text-secondary">
+      History isn&apos;t being saved right now (disk error) — investigations
+      won&apos;t survive a restart.
+    </div>
+  ) : null;
+
+  if (runs.length === 0) {
+    return (
+      <div>
+      {degradedNote}
+      <div className="flex flex-col items-center px-4 py-12 text-center">
+        <Sparkles className="mb-3 h-7 w-7 text-accent" />
+        <div className="text-sm font-medium text-theme-text-primary">
+          No investigations yet
+        </div>
+        <p className="mt-1 max-w-xs text-sm text-theme-text-tertiary">
+          Open a resource and use its{" "}
+          <Sparkles className="inline h-3.5 w-3.5 align-text-bottom text-accent" />{" "}
+          action to investigate it with {agentLabel} —{" "}
+          <span className="font-medium text-theme-text-secondary">Diagnose</span>{" "}
+          a problem, or just ask about it. Investigations run in the background
+          and are kept in your history here.
+        </p>
+      </div>
+      </div>
+    );
+  }
+
+  return (
+    <div className="space-y-2">
+      {degradedNote}
+      <div className="text-[11px] font-medium uppercase tracking-wide text-theme-text-tertiary">
+        Investigations
+      </div>
+      {runs.map((r) => (
+        <button
+          key={r.id}
+          onClick={() => onSelect(r.id)}
+          className={`flex w-full flex-col gap-0.5 rounded-md border px-2.5 py-2 text-left ${
+            r.id === selectedId
+              ? "border-accent/50 bg-accent/10"
+              : "border-theme-border/60 bg-theme-base/40 hover:bg-theme-hover"
+          }`}
+        >
+          <div className="flex items-center gap-2">
+            {statusDot(r.status)}
+            <span className="min-w-0 flex-1 truncate text-sm text-theme-text-primary">
+              {r.kind} {r.namespace ? `${r.namespace}/` : ""}
+              {r.name}
+            </span>
+            <span className="shrink-0 text-[11px] text-theme-text-tertiary">
+              {r.status === "running" ? (
+                "running…"
+              ) : (
+                <>
+                  {(() => {
+                    const w = statusWord(r.status);
+                    return w ? (
+                      <span className={`font-medium ${w.cls}`}>
+                        {w.text} ·{" "}
+                      </span>
+                    ) : null;
+                  })()}
+                  {relativeTime(new Date(r.updatedAt).getTime(), now)}
+                </>
+              )}
+            </span>
+          </div>
+          {(r.status === "stale" && r.context) || r.preview ? (
+            <div className="truncate pl-3.5 text-xs text-theme-text-tertiary">
+              {/* A foreign-cluster run names its cluster — in mixed multi-
+                  context history, identical-looking rows otherwise give no way
+                  to tell WHICH cluster an investigation was about. */}
+              {r.status === "stale" && r.context ? (
+                <span className="text-amber-600/80 dark:text-amber-500/80">
+                  {r.context}
+                </span>
+              ) : null}
+              {r.status === "stale" && r.context && r.preview ? " · " : ""}
+              {r.preview}
+            </div>
+          ) : null}
+        </button>
+      ))}
+    </div>
+  );
+}