@skyhook-io/k8s-ui 1.10.0 → 1.10.2

package/src/types/core.ts

@@ -1,3 +1,5 @@
+import type { CapacityIntegrationState } from './capacity'
+
 // Topology types matching the Go backend
 
 // Per-resource-type RBAC permissions. Field names must match the JSON keys
@@ -57,6 +59,12 @@ export interface WorkloadWritePermissions {
   rollouts: boolean
 }
 
+export interface IntegrationCapability {
+  state: CapacityIntegrationState
+  reasonCode?: string
+  cacheUnavailable?: boolean
+}
+
 // Feature capabilities based on RBAC permissions
 export interface Capabilities {
   exec: boolean           // Terminal feature (pods/exec)
@@ -69,6 +77,10 @@ export interface Capabilities {
   nodeWrite: boolean      // Node write operations (cordon, uncordon, drain)
   workloadWrites?: WorkloadWritePermissions // Workload patch permissions (restart/scale controls)
   mcpEnabled: boolean     // MCP server is running
+  // Karpenter discovery and NodePool read state. Optional on the wire for the
+  // same newer-frontend/older-backend reason as `deployment` below — consumers
+  // must treat absence as "unknown" and fall back to discovery signals.
+  karpenter?: IntegrationCapability
   // How / where this Radar binary is running. Optional on the wire so a
   // newer frontend (e.g. radar-hub-web bundling a fresher @skyhook-io/radar-app)
   // doesn't crash against an older backend that hasn't shipped the field yet —
@@ -462,6 +474,62 @@ export interface ResolvedEnvFromEntry {
 export type ResolvedEnvFromKey = `configmap:${string}` | `secret:${string}`
 export type ResolvedEnvFrom = Partial<Record<ResolvedEnvFromKey, ResolvedEnvFromEntry>>
 
+export type PodEnvironmentValueState = 'resolved' | 'masked' | 'unavailable' | 'missing' | 'denied'
+
+export interface PodEnvironmentSource {
+  kind: string
+  name?: string
+  key?: string
+  variable?: string
+}
+
+export interface PodEnvironmentEvidence {
+  kind: 'modified' | 'removed' | 'added'
+  changedAt: string
+  message?: string
+}
+
+export interface PodEnvironmentRow {
+  name: string
+  value?: string
+  state: PodEnvironmentValueState
+  sensitive?: boolean
+  source: PodEnvironmentSource
+  dependencies?: PodEnvironmentSource[]
+  shadowedSources?: PodEnvironmentSource[]
+  message?: string
+  optional?: boolean
+  missingImpact?: 'startupBlocked' | 'restartBlocked'
+  runtimeDependent?: boolean
+  currentPodValue?: boolean
+  placeholder?: boolean
+  evidence?: PodEnvironmentEvidence
+}
+
+export interface PodEnvironmentContainer {
+  name: string
+  role: 'container' | 'init' | 'sidecar'
+  rows: PodEnvironmentRow[]
+  truncated?: boolean
+}
+
+export interface PodEnvironmentResponse {
+  containers: PodEnvironmentContainer[]
+  coverage: {
+    observedSince?: string
+    degraded?: boolean
+    degradedReason?: string
+    saturated?: boolean
+  }
+  partial?: boolean
+  truncated?: boolean
+}
+
+export interface PodEnvironmentRevealResponse {
+  value: string
+  encoding: 'utf8' | 'base64'
+}
+
 // Resource reference (for relationships)
 export interface ResourceRef {
   kind: string
@@ -1060,6 +1128,7 @@ export interface TopNodeMetrics {
   name: string
   cpu: number              // nanocores (usage)
   memory: number           // bytes (usage)
+  observedAt?: string      // exact metrics sample time; absent when no sample exists
   podCount: number         // pods scheduled on this node
   cpuAllocatable: number   // nanocores
   memoryAllocatable: number // bytes
@@ -1231,11 +1300,11 @@ export interface TrafficFilters {
   timeRange: string
 }
 
-// Main view type now includes 'traffic', 'cost', 'checks', 'gitops'.
+// Main view type now includes 'traffic', 'cost', 'capacity', 'checks', 'gitops'.
 // Library consumers (Radar Hub) get all GitOps surfaces — the package
 // IS the public surface, so adding new top-level views must extend
 // this type rather than rely on app-local extensions.
-export type ExtendedMainView = MainView | 'traffic' | 'cost' | 'checks' | 'gitops' | 'issues' | 'applications'
+export type ExtendedMainView = MainView | 'traffic' | 'cost' | 'capacity' | 'checks' | 'gitops' | 'issues' | 'applications'
 
 // ============================================================================
 // Image Filesystem Types
@@ -1302,6 +1371,7 @@ export interface WorkloadPodInfo {
   containers: string[]
   ready: boolean
   phase?: string
+  nodeName?: string
   healthLevel?: HealthStatus
   reason?: string
   message?: string

package/src/types/index.ts

@@ -4,3 +4,4 @@ export * from './gitops'
 export * from './gitops-tree'
 export * from './gitops-insights'
 export * from './rbac'
+export * from './capacity'

package/src/utils/format.test.ts

@@ -0,0 +1,35 @@
+import { describe, expect, it } from 'vitest'
+import { formatCPUString, formatMemoryString, parseMemoryToBytes } from './format'
+
+describe('formatCPUString', () => {
+  it('renders exact zero as zero, not a fabricated near-zero', () => {
+    expect(formatCPUString('0')).toBe('0 cores')
+  })
+
+  it('preserves the sign of negative quantities (over-limit headroom)', () => {
+    expect(formatCPUString('-2')).toBe('-2 cores')
+    expect(formatCPUString('-500m')).toBe('-0.50 cores')
+  })
+
+  it('keeps sub-centicore positives as an honest bound', () => {
+    expect(formatCPUString('5m')).toBe('<0.01 cores')
+  })
+})
+
+describe('formatMemoryString', () => {
+  it('preserves the sign of negative quantities', () => {
+    expect(formatMemoryString('-4Gi')).toBe('-4.00 GiB')
+    expect(formatMemoryString('-256Mi')).toBe('-256 MiB')
+  })
+
+  it('renders zero as zero', () => {
+    expect(formatMemoryString('0')).toBe('0 B')
+  })
+})
+
+describe('parseMemoryToBytes', () => {
+  it('parses signed quantities instead of silently zeroing them', () => {
+    expect(parseMemoryToBytes('-1Ki')).toBe(-1024)
+    expect(parseMemoryToBytes('1Ki')).toBe(1024)
+  })
+})

package/src/utils/format.ts

@@ -13,6 +13,10 @@
  * - < 1 core: 2 decimal places (e.g., "0.05 cores")
  */
 function formatCoresValue(cores: number): string {
+  // Zero and negatives are real values (a pool over its limit has negative
+  // headroom) — collapsing them into "<0.01 cores" fabricates a near-zero.
+  if (cores === 0) return '0 cores'
+  if (cores < 0) return `-${formatCoresValue(-cores)}`
   if (cores >= 1) {
     // Round to 1 decimal
     const rounded = Math.round(cores * 10) / 10
@@ -91,6 +95,7 @@ export function formatCPUString(cpuString: string): string {
  * @returns Formatted string like "1.5 GiB" or "256 MiB"
  */
 export function formatMemoryBytes(bytes: number): string {
+  if (bytes < 0) return `-${formatMemoryBytes(-bytes)}`
   if (bytes >= 1024 * 1024 * 1024) {
     const gib = bytes / (1024 * 1024 * 1024)
     return gib >= 10 ? `${gib.toFixed(1)} GiB` : `${gib.toFixed(2)} GiB`
@@ -115,7 +120,7 @@ export function parseMemoryToBytes(memString: string): number {
   if (!memString) return 0
 
   const str = memString.trim()
-  const match = str.match(/^(\d+(?:\.\d+)?)\s*([A-Za-z]*)$/)
+  const match = str.match(/^(-?\d+(?:\.\d+)?)\s*([A-Za-z]*)$/)
   if (!match) return 0
 
   const num = parseFloat(match[1])

package/src/utils/navigation.test.ts

@@ -189,6 +189,15 @@ describe('refToSelectedResource', () => {
       group: 'cert-manager.io',
     })
   })
+
+  test('normalizes an omitted namespace for cluster-scoped references', () => {
+    expect(refToSelectedResource({ kind: 'NodePool', name: 'spot' })).toEqual({
+      kind: 'nodepools',
+      name: 'spot',
+      namespace: '',
+      group: undefined,
+    })
+  })
 })
 
 describe('lane identity helpers', () => {

package/src/utils/navigation.ts

@@ -140,10 +140,12 @@ export function pluralToKind(plural: string): string {
  * Convert a ResourceRef (from backend relationships) to a SelectedResource (for navigation).
  * Handles kind singular→plural conversion.
  */
-export function refToSelectedResource(ref: ResourceRef): SelectedResource {
+export function refToSelectedResource(
+  ref: Pick<ResourceRef, 'kind' | 'name' | 'group'> & { namespace?: string },
+): SelectedResource {
   return {
     kind: kindToPlural(ref.kind),
-    namespace: ref.namespace,
+    namespace: ref.namespace ?? '',
     name: ref.name,
     group: ref.group,
   }