@skyhook-io/radar-app 1.12.3 → 1.13.0

package/src/components/diagnose/parts.tsx

@@ -33,6 +33,9 @@ import {
 import { StatusDot } from "@skyhook-io/k8s-ui";
 import { Markdown } from "../ui/Markdown";
 
+const CURSOR_FULL_LOCAL_WARNING =
+  "Radar passes Cursor --force, which auto-approves its built-in tools and every MCP server it loads, including your global servers. Cursor’s sandbox does not reliably confine those tools to Radar’s temporary workspace.";
+
 // Segmented two-or-more-way selector — shared shape for the agent and execution
 // profile pickers.
 function Segmented<T extends string | boolean>({
@@ -309,9 +312,11 @@ export function AgentControls({
                     configured tools and MCP servers. Radar cannot constrain that
                     external tooling; it may access local files or the network
                     and may be able to change your cluster.{" "}
-                    {isClaude
-                      ? "Claude uses the permissions from your setup; Radar does not override them."
-                      : "Radar still enables the agent CLI’s own sandbox, but that sandbox does not constrain external MCP servers."}{" "}
+                    {isCursor
+                      ? CURSOR_FULL_LOCAL_WARNING
+                      : isClaude
+                        ? "Claude uses the permissions from your setup; Radar does not override them."
+                        : "Radar still enables the agent CLI’s own sandbox, but that sandbox does not constrain external MCP servers."}{" "}
                     Choose this only when you need that setup.
                   </span>
                 </div>
@@ -336,10 +341,14 @@ export function AgentControls({
                 Radar must use this agent&apos;s normal setup. Radar cannot
                 constrain its external tools or MCP servers; they may access
                 local files or the network and may be able to change your
-                cluster. Radar still enables the agent CLI&apos;s own sandbox,
-                but that sandbox does not constrain external MCP servers.
-                {isCursor &&
-                  " Cursor always loads your global MCP servers, so Radar cannot exclude them."}
+                cluster. {isCursor ? (
+                  CURSOR_FULL_LOCAL_WARNING
+                ) : (
+                  <>
+                    Radar still enables the agent CLI&apos;s own sandbox, but that
+                    sandbox does not constrain external MCP servers.
+                  </>
+                )}
               </span>
             </div>
           )}
@@ -832,7 +841,9 @@ export function ConsentCard({
                 MCP servers. They may access local files or the network and may
                 be able to change your cluster.
               </>,
-              agent === "claude" ? (
+              agent === "cursor-agent" ? (
+                CURSOR_FULL_LOCAL_WARNING
+              ) : agent === "claude" ? (
                 <>
                   Claude uses the permissions from your setup; Radar does not
                   override them.
@@ -841,13 +852,6 @@ export function ConsentCard({
                 <>
                   Radar still enables the agent CLI&apos;s own sandbox, but that
                   sandbox does not constrain external MCP servers.
-                  {agent === "cursor-agent" && (
-                    <>
-                      {" "}
-                      Cursor always loads your global MCP servers; Radar cannot
-                      exclude them.
-                    </>
-                  )}
                 </>
               ),
             ]

package/src/components/home/CostCard.tsx

@@ -8,6 +8,7 @@ import {
   formatProjectedMonthlyCost,
   formatProjectedMonthlyRate,
 } from '../cost/format'
+import { costFreshnessLabel } from '../cost/source'
 
 export function CostCard({ onNavigate }: { onNavigate?: () => void }) {
   const { data } = useOpenCostSummary()
@@ -87,13 +88,13 @@ function CostCardContent({ data, onNavigate }: { data: OpenCostSummary; onNaviga
 
         <div className="px-4 py-1.5 border-t border-theme-border/50 flex items-center justify-between">
           <span className="text-[10px] text-theme-text-tertiary">
-            {currency} &middot; projected monthly from {data.window ?? '1h'} window
+            {currency} &middot; projected from {costFreshnessLabel(data.source, data.window, data.dataThrough)}
             {currency !== DEFAULT_COST_CURRENCY && (
               <> &middot; no conversion</>
             )}
           </span>
           <span className="flex items-center gap-1.5 text-[10px] font-semibold uppercase tracking-wider text-accent-text">
-            OpenCost
+            {data.source === 'kubecost' ? 'Kubecost' : 'OpenCost'}
           </span>
         </div>
       </div>

package/src/components/rightsizing/RightsizingScanView.tsx

@@ -33,11 +33,17 @@ import {
   type ResourceSignal,
   type RightsizingActionTone,
 } from './presentation'
+import { useNavCustomization } from '../../context/NavCustomization'
 
 export const RIGHTSIZING_SCAN_DESCRIPTION =
   'Find CPU and memory requests to increase, reduce, or review. Radar never changes them.'
 export const RIGHTSIZING_SCAN_METHODOLOGY =
   'Based on 7 days of history: CPU P95 and memory maximum, plus 15% headroom. Memory reductions require verifiable restart history.'
+export const RIGHTSIZING_METRICS_REQUIRED_TITLE = 'Metrics history is required'
+export const RIGHTSIZING_METRICS_REQUIRED_BODY =
+  'Rightsizing needs 7 days of Kubernetes workload history from a PromQL-compatible metrics backend (Prometheus, VictoriaMetrics, Thanos, or Mimir).\nCost Overview remains available without it.'
+export const RIGHTSIZING_EMBEDDED_METRICS_REQUIRED_BODY =
+  `${RIGHTSIZING_METRICS_REQUIRED_BODY}\nConfigure metrics for this cluster in the host application or Radar deployment.`
 
 export type RightsizingScanSurfaceState =
   | 'discovering'
@@ -97,6 +103,7 @@ const ACTION_META: Record<
 
 export function RightsizingScanView({ namespaces }: RightsizingScanViewProps) {
   useAutoPromConnect()
+  const settingsAvailable = !useNavCustomization().embedded
   const navigate = useNavigate()
   const [params, setParams] = useSearchParams()
   const { data: clusterInfo } = useClusterInfo()
@@ -242,16 +249,33 @@ export function RightsizingScanView({ namespaces }: RightsizingScanViewProps) {
           />
         ) : surfaceState === 'prometheus_required' ? (
           <CenteredState
-            title="Prometheus is required"
-            body="Rightsizing uses Prometheus history. Cost Overview can still use OpenCost independently."
+            title={RIGHTSIZING_METRICS_REQUIRED_TITLE}
+            body={settingsAvailable
+              ? RIGHTSIZING_METRICS_REQUIRED_BODY
+              : RIGHTSIZING_EMBEDDED_METRICS_REQUIRED_BODY}
             action={
-              <button
-                type="button"
-                onClick={() => retryPrometheus()}
-                className="btn-brand px-3 py-1.5 text-xs font-medium"
-              >
-                Check again
-              </button>
+              <div className="flex flex-wrap items-center justify-center gap-3">
+                {settingsAvailable && (
+                  <button
+                    type="button"
+                    onClick={() =>
+                      window.dispatchEvent(
+                        new CustomEvent('radar:open-settings', { detail: { section: 'prometheus' } }),
+                      )
+                    }
+                    className="btn-brand px-4 py-2 text-sm font-medium"
+                  >
+                    Configure metrics
+                  </button>
+                )}
+                <button
+                  type="button"
+                  onClick={() => retryPrometheus()}
+                  className="rounded-lg border border-theme-border px-4 py-2 text-sm font-medium text-theme-text-secondary transition-colors hover:bg-theme-hover hover:text-theme-text-primary"
+                >
+                  Check again
+                </button>
+              </div>
             }
           />
         ) : surfaceState === 'first_run' ? (
@@ -860,15 +884,15 @@ function CenteredState({
   action?: React.ReactNode
 }) {
   return (
-    <div className="flex min-h-48 items-center justify-center rounded-xl border border-theme-border bg-theme-surface">
-      <div className="flex max-w-lg flex-col items-center px-6 text-center">
+    <div className="flex min-h-56 items-center justify-center rounded-xl border border-theme-border bg-theme-surface px-6 py-8">
+      <div className="flex w-full max-w-xl flex-col items-center text-center">
         {loading ? (
           <Loader2 className="h-8 w-8 animate-spin text-theme-text-tertiary" />
         ) : (
           <Gauge className="h-8 w-8 text-theme-text-tertiary" />
         )}
         <h2 className="mt-3 text-base font-semibold text-theme-text-primary">{title}</h2>
-        <p className="mt-1 text-sm text-theme-text-secondary">{body}</p>
+        <p className="mt-1 whitespace-pre-line text-sm text-theme-text-secondary">{body}</p>
         {action && <div className="mt-4">{action}</div>}
       </div>
     </div>

package/src/components/rightsizing/copy.test.ts

@@ -1,6 +1,9 @@
 import { describe, expect, it } from 'vitest'
 import {
   getRightsizingScanSurfaceState,
+  RIGHTSIZING_EMBEDDED_METRICS_REQUIRED_BODY,
+  RIGHTSIZING_METRICS_REQUIRED_BODY,
+  RIGHTSIZING_METRICS_REQUIRED_TITLE,
   RIGHTSIZING_SCAN_DESCRIPTION,
   RIGHTSIZING_SCAN_METHODOLOGY,
 } from './RightsizingScanView'
@@ -17,6 +20,22 @@ describe('rightsizing scan copy', () => {
     expect(copy.toLowerCase()).not.toContain('savings')
   })
 
+  it('describes the metrics contract without assuming one provider or cost source', () => {
+    const copy = `${RIGHTSIZING_METRICS_REQUIRED_TITLE} ${RIGHTSIZING_METRICS_REQUIRED_BODY}`
+    expect(copy).toContain('Metrics history')
+    expect(copy).toContain('PromQL-compatible metrics backend')
+    expect(copy).toContain('7 days')
+    expect(copy).toContain('Prometheus, VictoriaMetrics, Thanos, or Mimir')
+    expect(RIGHTSIZING_METRICS_REQUIRED_BODY).toContain('\nCost Overview')
+    expect(copy).not.toContain('OpenCost')
+    expect(copy).not.toContain('Kubecost')
+  })
+
+  it('routes embedded configuration through the host instead of standalone Settings', () => {
+    expect(RIGHTSIZING_EMBEDDED_METRICS_REQUIRED_BODY).toContain('host application')
+    expect(RIGHTSIZING_EMBEDDED_METRICS_REQUIRED_BODY).not.toContain('Settings')
+  })
+
   it('retains a prior snapshot after a failed rerun but treats a first-run failure as fatal', () => {
     expect(
       getRightsizingScanSurfaceState({