@skyhook-io/radar-app 1.14.6 → 1.16.0

package/README.md

@@ -41,6 +41,8 @@ When `navSlots.embedded` is true, Radar sizes itself to the host container (`hei
 
 Embedded mode renders only the active view. Radar Hub owns the sidebar, top bar, cluster selector, and navigation. `onClusterLoadStateChange` supplies the loading status for Hub's top bar.
 
+The package can be newer than the Radar it talks to. Features that need a newer Radar show an inline "needs Radar vX or newer" note instead of an error. Radar decides this from the `features` block of `/api/capabilities`, or from the Radar's version when it predates a flag. Pass `radarVersion` so Radar can skip those calls before `/api/version-check` answers. Pass `onRequestRadarUpgrade` to run your own upgrade flow from the note's "Upgrade Radar" button. Without it, the note links to the upgrade instructions.
+
 ## Tailwind
 
 Radar uses Tailwind v4 classes throughout. Your app's Tailwind config must scan Radar's source:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skyhook-io/radar-app",
-  "version": "1.14.6",
+  "version": "1.16.0",
   "description": "Radar's full web UI as a reusable React component. Used by Radar's own binary and by external consumers like Radar Cloud.",
   "repository": {
     "type": "git",
@@ -25,6 +25,7 @@
     "tsc": "node ../node_modules/typescript-7/bin/tsc --noEmit",
     "tsc:ts6": "tsc --noEmit",
     "test": "vitest run src",
+    "test:e2e:settings": "playwright test --config playwright.settings.config.ts",
     "format": "prettier --write \"src/**/*.{ts,tsx}\"",
     "lint": "eslint src && node scripts/check-no-native-title.mjs"
   },
@@ -41,7 +42,7 @@
     "yaml": "^2.9.0"
   },
   "peerDependencies": {
-    "@skyhook-io/k8s-ui": ">=1.14.8",
+    "@skyhook-io/k8s-ui": ">=1.16.0",
     "@tanstack/react-query": ">=5",
     "@xyflow/react": ">=12.0.0",
     "clsx": ">=2",
@@ -66,11 +67,11 @@
     "@xyflow/react": "^12.11.2",
     "clsx": "^2.1.1",
     "elkjs": "^0.11.1",
-    "eslint": "^10.10.0",
+    "eslint": "^10.11.0",
     "eslint-plugin-react-hooks": "^7.1.1",
     "eslint-plugin-react-refresh": "^0.5.3",
     "globals": "^17.11.0",
-    "lucide-react": "^1.37.0",
+    "lucide-react": "^1.48.0",
     "postcss": "^8.5.28",
     "prettier": "^3.9.8",
     "react": "^19.3.0",
@@ -80,7 +81,7 @@
     "tailwindcss": "^4.3.3",
     "typescript": "^6.0.2",
     "typescript-eslint": "^8.70.0",
-    "vite": "^8.2.2"
+    "vite": "^8.3.1"
   },
   "sideEffects": [
     "*.css"

package/src/App.tsx

@@ -46,9 +46,13 @@ import { ConnectionProvider, useConnection } from './context/ConnectionContext'
 import { ConnectionErrorView } from './components/ConnectionErrorView'
 import { SyncProgressPanel } from './components/SyncProgressPanel'
 import { CapabilitiesProvider, useCapabilitiesContext } from './contexts/CapabilitiesContext'
+import { RadarUpgradeProvider } from './context/RadarUpgrade'
 import { UserMenu } from './components/UserMenu'
 import { ErrorBoundary } from './components/ui/ErrorBoundary'
 import { UpdateNotification } from './components/ui/UpdateNotification'
+import { openWhatsNew, useWhatsNewStatus, WhatsNew } from './components/whats-new/WhatsNew'
+import { useUsageData, useUsageRecording } from './api/usage-data'
+import { UsageDataPrompt } from './components/usage-data/UsageDataPrompt'
 import { ShortcutHelpOverlay } from './components/ui/ShortcutHelpOverlay'
 import { DiagnosticsOverlay } from './components/ui/DiagnosticsOverlay'
 import { useEventSource } from './hooks/useEventSource'
@@ -67,7 +71,7 @@ import { Tooltip } from './components/ui/Tooltip'
 import { LargeClusterNamespacePicker } from './components/shared/LargeClusterNamespacePicker'
 import { SettingsDialog, type SettingsSectionId } from './components/settings/SettingsDialog'
 import type { APIResource, TopologyNode, GroupingMode, MainView, SelectedResource, SelectedHelmRelease, NodeKind, TopologyMode, Topology, K8sEvent } from './types'
-import { kindToPluralWithGroup, pluralToKind, openExternal, apiVersionToGroup, relatedResourcePath, searchHitToSelectedResource } from './utils/navigation'
+import { kindToPluralWithGroup, pluralToKind, openExternal, apiVersionToGroup, relatedResourcePath, searchHitToSelectedResource, withCrossViewParams } from './utils/navigation'
 import { findSelectedTopologyNode } from './utils/topology-selection'
 import { type OmnibarHandle } from './components/ui/Omnibar'
 import { RadarOmnibar } from './components/ui/RadarOmnibar'
@@ -144,6 +148,28 @@ function getViewFromPath(pathname: string): ExtendedMainView {
   return 'home'
 }
 
+// The usage-data name for the current screen: a resource list names its
+// kind's plural ("resources:deployments"); the server keeps built-in kinds
+// only, so a custom resource's name never leaves.
+function usageView(pathname: string, view: ExtendedMainView, upgrade: boolean): string {
+  if (upgrade) return 'upgrade'
+  if (view === 'resources') {
+    const plural = pathname.match(/^\/resources\/([^/]+)/)?.[1]
+    if (plural) return `resources:${plural.toLowerCase()}`
+  }
+  return view
+}
+
+// The screen a crash is counted under. Fixed names, because release builds
+// shorten component names and the component stack can't say which screen.
+const CRASH_LABELS: Record<ExtendedMainView, string> = {
+  home: 'Home', topology: 'Topology', resources: 'Resources', timeline: 'Timeline',
+  issues: 'Issues', helm: 'Helm', helmCompare: 'HelmCompare', traffic: 'Traffic',
+  cost: 'Cost', capacity: 'Capacity', checks: 'Checks', gitops: 'GitOps',
+  applications: 'Applications', workload: 'Workload', compare: 'Compare',
+  investigations: 'Investigations',
+}
+
 // The namespace scope filter is meaningful only on namespaced surfaces. On
 // cluster-scoped views it does nothing, so we disable it with an explanation
 // rather than leaving a dead control that silently ignores the pick:
@@ -417,6 +443,11 @@ function AppInner({ manageDocumentTitle = false, documentTitleSuffix, onClusterL
   const mainView = getViewFromPath(location.pathname)
   const upgradeReadinessRoute = location.pathname.startsWith('/checks/upgrade')
 
+  // Opt-in usage data. Embedded hosts own their own consent, so Radar never asks there.
+  const usageData = useUsageData(!navCustomization.embedded)
+  // A status cached by another screen must not start recording inside a host.
+  useUsageRecording(usageView(location.pathname, mainView, upgradeReadinessRoute), navCustomization.embedded ? undefined : usageData.data)
+
   // Initialize the kind→plural discovery map app-wide (not just on ResourcesView
   // mount) so the omnibar can open a CRD hit with an irregular plural from any
   // view — kindToPlural would otherwise English-guess the route before a
@@ -492,6 +523,12 @@ function AppInner({ manageDocumentTitle = false, documentTitleSuffix, onClusterL
     navigate({ pathname: path, search: newParams.toString() })
   }, [location.search, navigate, takeover, goHost])
 
+  const whatsNewStatus = useWhatsNewStatus()
+
+  const navigateToPath = useCallback((path: string) => {
+    navigate(withCrossViewParams(path, location.search))
+  }, [location.search, navigate])
+
   // The standalone rail expresses intent to leave the full-width investigation
   // workspace. Close it before routing so the destination is immediately visible;
   // docked investigations stay open across views as a persistent side panel.
@@ -777,13 +814,14 @@ function AppInner({ manageDocumentTitle = false, documentTitleSuffix, onClusterL
       metadata: { namespace: resource.namespace ?? '', name: resource.name },
     })
     if (gitOpsPath) {
-      const destination = new URL(gitOpsPath, window.location.origin)
-      if (investigationRunID) destination.searchParams.set('ai-run', investigationRunID)
+      const destination = new URL(withCrossViewParams(gitOpsPath, searchParams.toString()), window.location.origin)
+      if (investigationRunID === null) destination.searchParams.delete('ai-run')
+      else if (investigationRunID) destination.searchParams.set('ai-run', investigationRunID)
       navigate(`${destination.pathname}${destination.search}${destination.hash}`)
       return
     }
     navigateToResourceList(resource, investigationRunID)
-  }, [navigate, navigateToHelmRelease, navigateToResourceList])
+  }, [navigate, navigateToHelmRelease, navigateToResourceList, searchParams])
 
   // Collapse the over-list fullscreen back to the drawer = drop ?full=1 (and the
   // resource-scoped ?tab) in place. The button means "collapse THIS to a drawer"
@@ -1732,6 +1770,7 @@ function AppInner({ manageDocumentTitle = false, documentTitleSuffix, onClusterL
           showPinToggle={!railForcedSlim}
           onOpenSettings={() => openSettings()}
           accountSlot={<UserMenu variant="rail" pinned={navRailEffectivePinned} />}
+          whatsNew={whatsNewStatus.available ? { unread: whatsNewStatus.unread, onOpen: openWhatsNew } : undefined}
         />
       )}
       {/* `relative` makes this column the containing block for the absolute
@@ -1833,6 +1872,7 @@ function AppInner({ manageDocumentTitle = false, documentTitleSuffix, onClusterL
             onSetNamespaces={(ns) => { setNamespaces(ns); setActiveNamespace.mutate({ namespaces: ns }) }}
             onToggleTheme={toggleTheme}
             onShowDiagnostics={() => setShowDiagnostics(true)}
+            onShowWhatsNew={whatsNewStatus.available ? openWhatsNew : undefined}
             onOpenResource={(hit) => navigateToResourceList(searchHitToSelectedResource(hit))}
           />
         </div>
@@ -1977,7 +2017,10 @@ function AppInner({ manageDocumentTitle = false, documentTitleSuffix, onClusterL
         {/* Search included, not just the path: selection inside a view rides in
             the query (?resource=, ?release=), so a path-only key would still
             strand a crash on the view that produced it. */}
-        <ErrorBoundary resetKey={location.pathname + location.search}>
+        <ErrorBoundary
+          resetKey={location.pathname + location.search}
+          usageLabel={upgradeReadinessRoute ? 'Upgrade' : CRASH_LABELS[mainView]}
+        >
         {/* Initial sync in progress: views that need the full cluster dataset
             show per-kind progress instead; resource views work as kinds sync. */}
         {viewsSyncGated && connection.syncStatus && (
@@ -1994,7 +2037,8 @@ function AppInner({ manageDocumentTitle = false, documentTitleSuffix, onClusterL
             fallbackClusterLoadState={showHomeClusterLoadFallback ? clusterLoadState : undefined}
             onNavigateToView={setMainView}
             onNavigateToHelmRelease={navCustomization.embedded ? undefined : navigateToHelmRelease}
-            onNavigateToManagerPath={navCustomization.embedded || takeover.gitops ? undefined : (path) => navigate(path)}
+            onNavigateToManagerPath={navCustomization.embedded || takeover.gitops ? undefined : navigateToPath}
+            onShowWhatsNew={!navCustomization.embedded && whatsNewStatus.available ? openWhatsNew : undefined}
             // Upgrade impact lives under /checks, which a Cloud host takes
             // over wholesale — its fleet pages have no upgrade sub-route, so
             // the version line stays plain text there.
@@ -2435,6 +2479,8 @@ function AppInner({ manageDocumentTitle = false, documentTitleSuffix, onClusterL
 
       {/* Update notification — hidden in embedded mode (OSS download nudge). */}
       {!navCustomization.embedded && <UpdateNotification />}
+      {!navCustomization.embedded && <WhatsNew onNavigate={navigateToPath} usageData={usageData.data} />}
+      {!navCustomization.embedded && <UsageDataPrompt status={usageData.data} />}
 
       {/* Bottom Dock for Terminal/Logs */}
       <BottomDock />
@@ -2488,17 +2534,19 @@ function App({ manageDocumentTitle = false, documentTitleSuffix, onClusterLoadSt
   return (
     <ConnectionProvider>
       <CapabilitiesProvider>
-        <ContextSwitchProvider>
-          <DockProvider>
-            <KeyboardShortcutProvider>
-              <AppInner
-                manageDocumentTitle={manageDocumentTitle}
-                documentTitleSuffix={documentTitleSuffix}
-                onClusterLoadStateChange={onClusterLoadStateChange}
-              />
-            </KeyboardShortcutProvider>
-          </DockProvider>
-        </ContextSwitchProvider>
+        <RadarUpgradeProvider>
+          <ContextSwitchProvider>
+            <DockProvider>
+              <KeyboardShortcutProvider>
+                <AppInner
+                  manageDocumentTitle={manageDocumentTitle}
+                  documentTitleSuffix={documentTitleSuffix}
+                  onClusterLoadStateChange={onClusterLoadStateChange}
+                />
+              </KeyboardShortcutProvider>
+            </DockProvider>
+          </ContextSwitchProvider>
+        </RadarUpgradeProvider>
       </CapabilitiesProvider>
     </ConnectionProvider>
   )

package/src/RadarApp.tsx

@@ -33,9 +33,15 @@ import {
   showApiSuccess,
 } from "./components/ui/Toast";
 import { setApiBase, setBasename } from "./api/config";
+import { shouldRetryRadarQuery } from "./api/radarFeatures";
 import { NavCustomizationProvider } from "./context/NavCustomization";
 import { FilterLocationBridge } from "./filter/FilterLocationBridge";
 import type { NavCustomization } from "./context/NavCustomization";
+import { RadarUpgradeHostProvider } from "./context/RadarUpgradeHost";
+import type {
+  RadarUpgradeHost,
+  RadarUpgradeRequest,
+} from "./context/RadarUpgradeHost";
 import type { ClusterLoadState } from "./types/clusterLoadState";
 import { TimelineSourceProvider } from "./context/TimelineSource";
 import type { TimelineSourceConfig } from "./api/timelineSource";
@@ -65,6 +71,21 @@ declare module "@tanstack/react-query" {
 export interface RadarAppProps {
   /** API base URL (REST + SSE + WS). Defaults to '/api' (same-origin). */
   apiBase?: string;
+  /**
+   * The version of the Radar behind `apiBase`, as the host knows it (Radar
+   * Cloud: the cluster's `radar_version`). Radar prefers that Radar's own
+   * /version-check answer and uses this only until it arrives or when it
+   * fails, to decide up front that a feature needs a newer Radar instead of
+   * probing for it.
+   */
+  radarVersion?: string;
+  /**
+   * Called when the user asks to upgrade from an inline "needs a newer Radar"
+   * note next to a feature the connected Radar is too old to serve. The host
+   * runs its own upgrade flow (Radar Cloud: one-click for owners, "ask an
+   * owner" otherwise). Omitted → the note links to the upgrade instructions.
+   */
+  onRequestRadarUpgrade?: (request: RadarUpgradeRequest) => void;
   /** React Router basename. Defaults to '' (mounted at root). */
   basename?: string;
   /**
@@ -163,7 +184,7 @@ function makeDefaultQueryClient(): QueryClient {
     defaultOptions: {
       queries: {
         refetchOnWindowFocus: false,
-        retry: 1,
+        retry: shouldRetryRadarQuery,
       },
     },
     mutationCache: new MutationCache({
@@ -193,6 +214,8 @@ function makeDefaultQueryClient(): QueryClient {
 
 export function RadarApp({
   apiBase,
+  radarVersion,
+  onRequestRadarUpgrade,
   basename,
   router = "browser",
   queryClient,
@@ -223,32 +246,39 @@ export function RadarApp({
     [queryClient],
   );
 
+  const upgradeHost = React.useMemo<RadarUpgradeHost>(
+    () => ({ radarVersion, onRequestRadarUpgrade }),
+    [radarVersion, onRequestRadarUpgrade],
+  );
+
   const inner = (
     <ThemeProvider>
       <QueryClientProvider client={client}>
         <ToastProvider>
           <NavCustomizationProvider value={navSlots}>
-            <FilterLocationBridge>
-              <TimelineSourceProvider config={timelineSource}>
-                <DiagnoseCustomizationProvider
-                  value={renderDiagnoseAction ?? defaultDiagnoseAction}
-                  consentCopy={diagnoseConsent}
-                  renderRunActions={renderInvestigationRunActions}
-                >
-                  <DiagnoseProvider
-                    browserURLState={router !== "memory"}
-                    forceRouterURLState={router === "memory"}
-                    onFocusedRun={onInvestigationFocus}
+            <RadarUpgradeHostProvider value={upgradeHost}>
+              <FilterLocationBridge>
+                <TimelineSourceProvider config={timelineSource}>
+                  <DiagnoseCustomizationProvider
+                    value={renderDiagnoseAction ?? defaultDiagnoseAction}
+                    consentCopy={diagnoseConsent}
+                    renderRunActions={renderInvestigationRunActions}
                   >
-                    <App
-                      manageDocumentTitle={manageDocumentTitle}
-                      documentTitleSuffix={documentTitleSuffix}
-                      onClusterLoadStateChange={onClusterLoadStateChange}
-                    />
-                  </DiagnoseProvider>
-                </DiagnoseCustomizationProvider>
-              </TimelineSourceProvider>
-            </FilterLocationBridge>
+                    <DiagnoseProvider
+                      browserURLState={router !== "memory"}
+                      forceRouterURLState={router === "memory"}
+                      onFocusedRun={onInvestigationFocus}
+                    >
+                      <App
+                        manageDocumentTitle={manageDocumentTitle}
+                        documentTitleSuffix={documentTitleSuffix}
+                        onClusterLoadStateChange={onClusterLoadStateChange}
+                      />
+                    </DiagnoseProvider>
+                  </DiagnoseCustomizationProvider>
+                </TimelineSourceProvider>
+              </FilterLocationBridge>
+            </RadarUpgradeHostProvider>
           </NavCustomizationProvider>
         </ToastProvider>
       </QueryClientProvider>

package/src/api/apiResources.test.ts

@@ -1,5 +1,6 @@
 import { describe, expect, it } from 'vitest'
-import { hasKarpenterNodePools } from './apiResources'
+import { hasKarpenterNodePools, hasPolicyReports } from './apiResources'
+import type { APIResource } from '../types'
 
 describe('hasKarpenterNodePools', () => {
   it('uses exact API discovery rather than top-CRD counts or kind name alone', () => {
@@ -9,3 +10,20 @@ describe('hasKarpenterNodePools', () => {
     expect(hasKarpenterNodePools(undefined)).toBe(false)
   })
 })
+
+describe('hasPolicyReports', () => {
+  const resource = (name: string, group: string) => ({ name, group }) as APIResource
+
+  it('recognizes each report family by the resource names it actually serves', () => {
+    expect(hasPolicyReports([resource('policyreports', 'wgpolicyk8s.io')])).toBe(true)
+    expect(hasPolicyReports([resource('clusterpolicyreports', 'wgpolicyk8s.io')])).toBe(true)
+    expect(hasPolicyReports([resource('reports', 'openreports.io')])).toBe(true)
+    expect(hasPolicyReports([resource('clusterreports', 'openreports.io')])).toBe(true)
+    expect(hasPolicyReports([resource('policyreports', 'openreports.io')])).toBe(false)
+  })
+
+  it('reads a cluster without them as having no policy engine', () => {
+    expect(hasPolicyReports([resource('pods', ''), resource('policyreports', 'example.com')])).toBe(false)
+    expect(hasPolicyReports(undefined)).toBe(false)
+  })
+})

package/src/api/apiResources.ts

@@ -1,6 +1,7 @@
 import { useQuery } from "@tanstack/react-query";
 import type { APIResource } from "../types";
 import { apiUrl, getAuthHeaders, getCredentialsMode } from "./config";
+import { readErrorBody } from "./httpErrors";
 
 // Re-export pure functions from package
 export {
@@ -20,9 +21,7 @@ async function fetchJSON<T>(path: string): Promise<T> {
     headers: getAuthHeaders(),
   });
   if (!response.ok) {
-    const error = await response
-      .json()
-      .catch(() => ({ error: "Unknown error" }));
+    const error = await readErrorBody(response);
     throw new Error(error.error || `HTTP ${response.status}`);
   }
   return response.json();
@@ -37,6 +36,22 @@ export function useAPIResources() {
   });
 }
 
+// The report families the server watches (reportGroups in
+// internal/k8s/policy_reports.go): the wgpolicyk8s.io working-group API and
+// its openreports.io successor, which names the same resources differently.
+const POLICY_REPORT_RESOURCES: Record<string, readonly string[]> = {
+  "wgpolicyk8s.io": ["policyreports", "clusterpolicyreports"],
+  "openreports.io": ["reports", "clusterreports"],
+};
+
+export function hasPolicyReports(resources: APIResource[] | undefined): boolean {
+  return (
+    resources?.some((resource) =>
+      POLICY_REPORT_RESOURCES[resource.group]?.includes(resource.name),
+    ) ?? false
+  );
+}
+
 export function hasKarpenterNodePools(
   resources: APIResource[] | undefined,
 ): boolean {

package/src/api/client.admission.test.ts

@@ -0,0 +1,34 @@
+import { describe, expect, it, vi } from 'vitest'
+import { useQuery } from '@tanstack/react-query'
+import { ApiError, useKueueAdmission as admissionQuery } from './client'
+
+vi.mock('@tanstack/react-query', async (original) => ({ ...await original<typeof import('@tanstack/react-query')>(), useQuery: vi.fn(() => ({})) }))
+
+describe('Kueue admission polling', () => {
+  function options(settings?: Parameters<typeof admissionQuery>[3]) {
+    admissionQuery('ml', 'training', 'current', settings)
+    return vi.mocked(useQuery).mock.calls.at(-1)![0]
+  }
+  function interval(settings: Parameters<typeof admissionQuery>[3], state: any = {}) {
+    const poll = options(settings).refetchInterval as (query: any) => number | false
+    return poll({ state })
+  }
+  it('separates Job and JobSet identities and recreated roots', () => {
+    expect(options({ isJob: true }).queryKey).toEqual(['kueue-admission', 'batch', 'jobs', 'ml', 'training', 'current'])
+    expect(options({ isRayJob: true }).queryKey).toEqual(['kueue-admission', 'ray.io', 'rayjobs', 'ml', 'training', 'current'])
+    expect(options().queryKey).toEqual(['kueue-admission', 'jobset.x-k8s.io', 'jobsets', 'ml', 'training', 'current'])
+  })
+  it('polls quiet and terminal Jobs slowly, active admission quickly', () => {
+    expect(interval({ isJob: true })).toBe(30000)
+    expect(interval({ isJob: true, hinted: true })).toBe(5000)
+    expect(interval({ isJob: true }, { data: { workloads: [{}] } })).toBe(5000)
+    expect(interval({ isJob: true, hinted: true, terminal: true })).toBe(30000)
+    expect(interval(undefined)).toBe(5000)
+  })
+  it('stops on confirmed absence or denied requests, not transient outages or old absence', () => {
+    expect(interval({ isJob: true }, { data: { uid: 'current', installed: false } })).toBe(false)
+    expect(interval({ isJob: true, hinted: true }, { data: { uid: 'old', installed: false } })).toBe(5000)
+    expect(interval({ isJob: true }, { error: new ApiError('denied', 403) })).toBe(false)
+    expect(interval({ isJob: true, hinted: true }, { error: new ApiError('warming', 503) })).toBe(5000)
+  })
+})

package/src/api/client.capacity.test.ts

@@ -5,6 +5,7 @@ import {
   isCapacityCursorInvalidError,
   shouldRetryCapacityQuery,
 } from "./client";
+import { RadarFeatureUnsupportedError } from "./radarFeatures";
 
 describe("isCapacityCursorInvalidError", () => {
   it("uses the structured capacity error code", () => {
@@ -42,6 +43,12 @@ describe("shouldRetryCapacityQuery", () => {
     );
   });
 
+  it("never retries a feature the connected Radar predates", () => {
+    expect(
+      shouldRetryCapacityQuery(0, new RadarFeatureUnsupportedError("capacity", {})),
+    ).toBe(false);
+  });
+
   it("bounds retries for transient failures", () => {
     expect(shouldRetryCapacityQuery(0, new ApiError("unavailable", 503))).toBe(
       true,
@@ -63,6 +70,14 @@ describe("capacityRefetchInterval", () => {
     ).toBe(false);
   });
 
+  it("stops polling a Radar that predates Capacity", () => {
+    expect(
+      capacityRefetchInterval(undefined, true)({
+        state: { error: new RadarFeatureUnsupportedError("capacity", {}) },
+      }),
+    ).toBe(false);
+  });
+
   it("polls fast while the server reports syncing", () => {
     expect(
       capacityRefetchInterval(undefined, true)(withData({ state: "syncing" })),

package/src/api/client.trace.test.ts

@@ -25,9 +25,17 @@ describe('runInClusterMerged error surfacing', () => {
     await expect(runInClusterMerged('Service', 'prod', 'web')).rejects.toThrow('no eligible routes')
   })
 
+  // The banner around this message already says the in-cluster test couldn't run.
   it('rejects a non-JSON gateway failure with the status, not a parse error', async () => {
     respond(504, '<html>Gateway Timeout</html>', 'text/html')
-    await expect(runInClusterMerged('Service', 'prod', 'web')).rejects.toThrow('In-cluster test failed (504)')
+    await expect(runInClusterMerged('Service', 'prod', 'web')).rejects.toThrow('HTTP 504 (Gateway Timeout)')
+  })
+
+  it("keeps a proxy's plain-text reason", async () => {
+    respond(503, 'cluster "prod" not connected\n', 'text/plain; charset=utf-8')
+    await expect(runInClusterMerged('Service', 'prod', 'web')).rejects.toThrow(
+      'cluster "prod" not connected (HTTP 503)',
+    )
   })
 
   it('resolves a clean run with the finalized payload', async () => {