@skyhook-io/k8s-ui 1.14.4 → 1.14.5

package/src/components/topology/K8sResourceNode.tsx

@@ -168,6 +168,34 @@ function getIssueTooltip(issue: string | undefined): React.ReactNode {
   );
 }
 
+// Full port list for the Service subtitle's hover affordance — the subtitle
+// itself only ever shows the first port + a count, so this is the only place
+// a multi-port Service's other ports are visible without opening the detail
+// page. Formatting (hide targetPort when it matches port) mirrors
+// ServicePortCards in ServiceRenderer.tsx so a Service's ports read the same
+// whether glanced at in the graph or opened in the full resource view.
+function servicePortsTooltip(ports: ServicePortEntry[]): React.ReactNode | null {
+  if (ports.length < 2) return null;
+  return (
+    <div className="max-w-xs space-y-0.5">
+      {ports.map((p, i) => {
+        const target =
+          p.targetPort != null && p.targetPort !== String(p.port)
+            ? ` → ${p.targetPort}`
+            : "";
+        const proto = p.appProtocol || p.protocol;
+        return (
+          <div key={i} className="text-[11px]">
+            {p.name && <span className="text-theme-text-tertiary">{p.name} </span>}
+            <span className="font-medium">{p.port}{target}</span>
+            {proto && <span className="text-theme-text-tertiary"> ({proto})</span>}
+          </div>
+        );
+      })}
+    </div>
+  );
+}
+
 // Default dimensions for unknown CRD kinds
 export const DEFAULT_NODE_DIMENSIONS = { width: 260, height: 84 };
 
@@ -322,6 +350,16 @@ const SUMMARY_POD_KINDS = new Set<NodeKind>([
   "Service",
 ]);
 
+// Shape of one entry in a Service node's nodeData.ports (pkg/topology/builder.go's
+// serviceTopologyPorts). name/appProtocol are omitted server-side when unset.
+export interface ServicePortEntry {
+  name?: string;
+  port: number;
+  targetPort?: string;
+  protocol?: string;
+  appProtocol?: string;
+}
+
 export function baseSubtitle(kind: NodeKind, nodeData: Record<string, unknown>): string {
   if (nodeData.deploymentMembership === 'source-only') {
     return `Declared by ${nodeData.deploymentSourceLabel ?? 'deployment source'}`
@@ -369,8 +407,10 @@ export function baseSubtitle(kind: NodeKind, nodeData: Record<string, unknown>):
       return (nodeData.phase as string) || "Unknown";
     case "Service": {
       const svcType = (nodeData.type as string) || "ClusterIP";
-      const port = nodeData.port;
-      return port ? `${svcType} :${port}` : svcType;
+      const ports = (nodeData.ports as ServicePortEntry[] | undefined) || [];
+      if (ports.length === 0) return svcType;
+      const more = ports.length > 1 ? ` +${ports.length - 1} more` : "";
+      return `${svcType} :${ports[0].port}${more}`;
     }
     case "CalicoNetworkPolicy":
     case "CalicoGlobalNetworkPolicy":
@@ -513,6 +553,9 @@ export const K8sResourceNode = memo(function K8sResourceNode({
   const { ownerColorIndex } = ownershipOf(nodeData)
   const hue = ownerColorIndex !== null ? workloadHue(ownerColorIndex) : undefined
   const subtitle = getSubtitle(kind, nodeData)
+  const portsTooltip = kind === 'Service'
+    ? servicePortsTooltip((nodeData.ports as ServicePortEntry[] | undefined) || [])
+    : null
   const isInternet = kind === 'Internet'
   const isPodGroup = kind === 'PodGroup'
   const isSmallNode = kind === 'ConfigMap' || kind === 'Secret' || kind === 'ServiceAccount' || kind === 'SealedSecret' || kind === 'ServiceMonitor' || kind === 'PodMonitor' || kind === 'HorizontalPodAutoscaler'
@@ -730,9 +773,17 @@ export const K8sResourceNode = memo(function K8sResourceNode({
 
           {/* Subtitle */}
           {subtitle && (
-            <div className="text-xs text-theme-text-secondary truncate mt-0.5">
-              {subtitle}
-            </div>
+            portsTooltip ? (
+              <Tooltip content={portsTooltip} position="bottom" wrapperClassName="max-w-full mt-0.5">
+                <div className="text-xs text-theme-text-secondary truncate cursor-help">
+                  {subtitle}
+                </div>
+              </Tooltip>
+            ) : (
+              <div className="text-xs text-theme-text-secondary truncate mt-0.5">
+                {subtitle}
+              </div>
+            )
           )}
         </div>
       </div>

package/src/components/ui/Badge.tsx

@@ -23,7 +23,10 @@ export type BadgeSize = 'sm' | 'default'
 //   "tell these sibling options apart". The only place sharing is intended.
 // - structural: ports/paths/hosts/weights/names — neutral data fragments, not a
 //   status. Its own name so it can diverge from severity-neutral later.
-export type BadgeTone = 'note' | 'accent1' | 'accent2' | 'accent3' | 'structural'
+// - agent:    words an AI agent attached to a Radar fact (evidence roles such
+//   as "Cause" / "Rules out"). Muted brand accent on purpose: never a severity
+//   hue, so an agent's framing can never be mistaken for a Radar finding.
+export type BadgeTone = 'note' | 'accent1' | 'accent2' | 'accent3' | 'structural' | 'agent'
 
 interface BadgeProps {
   /** Severity-based coloring (status badges) */
@@ -231,6 +234,7 @@ const TONE: Record<BadgeTone, string> = {
   accent3:    'bg-teal-100 text-teal-800 border-teal-300 dark:bg-teal-950/50 dark:text-teal-400 dark:border-teal-700/40',
   // neutral data fragment (ports/paths/hosts/names)
   structural: 'bg-theme-elevated text-theme-text-secondary border-theme-border',
+  agent:      'bg-accent-muted text-accent-text border-accent/30',
 }
 
 // Structure classes

package/src/components/ui/index.ts

@@ -33,6 +33,11 @@ export { InClusterConsentDialog } from './InClusterConsentDialog'
 export { ToastProvider, useToast, showApiError, showApiSuccess } from './Toast'
 export { CodeViewer } from './CodeViewer'
 export { YamlEditor, YamlDiffEditor } from './YamlEditor'
+// Boots the self-hosted Monaco runtime (bundled editor, local workers) without
+// pulling monaco-editor into the main bundle. Anything that mounts a bare
+// `Editor` must await this first, or @monaco-editor/react reaches for its CDN.
+export const ensureMonacoRuntime = () =>
+  import('./monacoRuntime').then(({ ensureMonaco }) => ensureMonaco())
 export type {
   YamlDiagnostic,
   YamlDocumentIdentity,

package/src/components/ui/monacoRuntime.ts

@@ -2,7 +2,21 @@
 
 import { loader } from '@monaco-editor/react'
 import * as monaco from 'monaco-editor/esm/vs/editor/editor.api'
+// Tokenizers only — the set the YAML editor and the pod file viewer hand to
+// Monaco. JSON deliberately absent: its language lives in vs/language/json and
+// spawns a validation worker this runtime does not ship; viewers highlight
+// JSON with the YAML tokenizer, which covers it.
 import 'monaco-editor/esm/vs/basic-languages/yaml/yaml.contribution'
+import 'monaco-editor/esm/vs/basic-languages/xml/xml.contribution'
+import 'monaco-editor/esm/vs/basic-languages/html/html.contribution'
+import 'monaco-editor/esm/vs/basic-languages/css/css.contribution'
+import 'monaco-editor/esm/vs/basic-languages/javascript/javascript.contribution'
+import 'monaco-editor/esm/vs/basic-languages/typescript/typescript.contribution'
+import 'monaco-editor/esm/vs/basic-languages/python/python.contribution'
+import 'monaco-editor/esm/vs/basic-languages/shell/shell.contribution'
+import 'monaco-editor/esm/vs/basic-languages/markdown/markdown.contribution'
+import 'monaco-editor/esm/vs/basic-languages/ini/ini.contribution'
+import 'monaco-editor/esm/vs/basic-languages/dockerfile/dockerfile.contribution'
 import 'monaco-editor/esm/vs/editor/contrib/find/browser/findController.js'
 import 'monaco-editor/esm/vs/editor/contrib/folding/browser/folding.js'
 import 'monaco-editor/esm/vs/editor/contrib/format/browser/formatActions.js'

package/src/monaco-deep.d.ts

@@ -3,3 +3,13 @@ declare module 'monaco-editor/esm/vs/editor/editor.api' {
 }
 
 declare module 'monaco-editor/esm/vs/basic-languages/yaml/yaml.contribution'
+declare module 'monaco-editor/esm/vs/basic-languages/xml/xml.contribution'
+declare module 'monaco-editor/esm/vs/basic-languages/html/html.contribution'
+declare module 'monaco-editor/esm/vs/basic-languages/css/css.contribution'
+declare module 'monaco-editor/esm/vs/basic-languages/javascript/javascript.contribution'
+declare module 'monaco-editor/esm/vs/basic-languages/typescript/typescript.contribution'
+declare module 'monaco-editor/esm/vs/basic-languages/python/python.contribution'
+declare module 'monaco-editor/esm/vs/basic-languages/shell/shell.contribution'
+declare module 'monaco-editor/esm/vs/basic-languages/markdown/markdown.contribution'
+declare module 'monaco-editor/esm/vs/basic-languages/ini/ini.contribution'
+declare module 'monaco-editor/esm/vs/basic-languages/dockerfile/dockerfile.contribution'

package/src/types/core.ts

@@ -516,6 +516,9 @@ export interface ContextInfo {
   /** Source kubeconfig label (e.g. "kube-cluster-paris"). Set by the backend
    *  for contexts loaded through the isolated source registry. */
   source?: string
+  /** AWS profile extracted from the exec plugin's --profile arg or AWS_PROFILE
+   *  env var. Present only for EKS contexts that pin a profile. */
+  awsProfile?: string
 }
 
 // Namespace
@@ -683,24 +686,34 @@ export type HPADiagnosisState =
   | 'stabilized'
   | 'unknown'
 
-export interface HPADiagnosis {
+export interface HPABounds {
+  min: number
+  max: number
+  current: number
+  desired: number
+  observedGeneration?: number
+  generation?: number
+}
+
+// The subset of an HPA diagnosis that HPADiagnosisSummary presents. Producers
+// that carry a richer or looser envelope (Radar's own HPADiagnosis, the
+// investigation evidence projection) extend this so the presentation rules —
+// state severity, state label, reason redundancy — cannot drift per surface.
+export interface HPADiagnosisView {
   state: HPADiagnosisState
   summary: string
+  bounds?: HPABounds
+  reasons?: HPAReasonSummary[]
+}
+
+export interface HPADiagnosis extends HPADiagnosisView {
   target: {
     apiVersion?: string
     kind?: string
     name?: string
   }
-  bounds: {
-    min: number
-    max: number
-    current: number
-    desired: number
-    observedGeneration?: number
-    generation?: number
-  }
+  bounds: HPABounds
   metrics?: HPAMetricSummary[]
-  reasons?: HPAReasonSummary[]
 }
 
 export interface HPAReasonSummary {