@skyhook-io/k8s-ui 1.5.10 → 1.5.12

package/src/components/resources/renderers/OrderRenderer.tsx

@@ -1,6 +1,7 @@
 import { ShoppingCart, AlertTriangle, Globe, Shield } from 'lucide-react'
 import { clsx } from 'clsx'
 import { Section, PropertyList, Property, ConditionsSection } from '../../ui/drawer-components'
+import { BADGE_INACTIVE } from '../../../utils/badge-colors'
 
 function getOrderStateBadge(state: string): { color: string; text: string } {
   switch (state?.toLowerCase()) {
@@ -17,7 +18,7 @@ function getOrderStateBadge(state: string): { color: string; text: string } {
     case 'errored':
       return { text: 'Errored', color: 'bg-red-500/20 text-red-400' }
     default:
-      return { text: state || 'Unknown', color: 'bg-gray-500/20 text-gray-400' }
+      return { text: state || 'Unknown', color: BADGE_INACTIVE }
   }
 }
 

package/src/components/resources/renderers/PodRenderer.tsx

@@ -2,7 +2,7 @@ import { useState, type ReactNode, type JSX } from 'react'
 import { Server, HardDrive, Terminal as TerminalIcon, FileText, Activity, CirclePlay, FolderOpen, List, Eye, EyeOff } from 'lucide-react'
 import { clsx } from 'clsx'
 import { Section, PropertyList, Property, ConditionsSection, CopyHandler, AlertBanner, ResourceLink } from '../../ui/drawer-components'
-import { formatResources, formatDuration, getPodProblems, SEVERITY_DOT_COLOR } from '../resource-utils'
+import { formatResources, formatDuration, getPodProblems, getPodPhaseDisplay, healthColors, SEVERITY_DOT_COLOR } from '../resource-utils'
 import { getResourceStatusColor, SEVERITY_BADGE_BORDERED } from '../../../utils/badge-colors'
 import type { ResolvedEnvFrom } from '../../../types'
 import { Tooltip } from '../../ui/Tooltip'
@@ -333,7 +333,28 @@ export function PodRenderer({
       {/* Status section */}
       <Section title="Status" icon={Server}>
         <PropertyList>
-          <Property label="Phase" value={data.status?.phase} />
+          {(() => {
+            const phaseDisplay = getPodPhaseDisplay(data)
+            const node = (
+              <span className={clsx(healthColors[phaseDisplay.level])}>
+                {phaseDisplay.text}
+              </span>
+            )
+            return (
+              <Property
+                label="Phase"
+                value={
+                  phaseDisplay.hint ? (
+                    <Tooltip content={phaseDisplay.hint} position="right">
+                      {node}
+                    </Tooltip>
+                  ) : (
+                    node
+                  )
+                }
+              />
+            )
+          })()}
           <Property label="Node" value={
             data.spec?.nodeName ? <ResourceLink name={data.spec.nodeName} kind="nodes" onNavigate={onNavigate} /> : undefined
           } copyable onCopy={onCopy} copied={copied} />

package/src/components/resources/renderers/RoleBindingRenderer.tsx

@@ -1,6 +1,7 @@
 import { Shield, Users } from 'lucide-react'
 import { clsx } from 'clsx'
 import { Section, PropertyList, Property, ResourceLink, AlertBanner } from '../../ui/drawer-components'
+import { BADGE_INACTIVE } from '../../../utils/badge-colors'
 
 interface RoleBindingRendererProps {
   data: any
@@ -16,7 +17,7 @@ function getSubjectKindBadgeClass(kind: string): string {
     case 'Group':
       return 'bg-purple-500/20 text-purple-400'
     default:
-      return 'bg-gray-500/20 text-gray-400'
+      return BADGE_INACTIVE
   }
 }
 
@@ -27,7 +28,7 @@ function getRoleRefKindBadgeClass(kind: string): string {
     case 'ClusterRole':
       return 'bg-purple-500/20 text-purple-400'
     default:
-      return 'bg-gray-500/20 text-gray-400'
+      return BADGE_INACTIVE
   }
 }
 

package/src/components/resources/renderers/RolloutRenderer.tsx

@@ -2,6 +2,7 @@ import { Server, GitBranch, AlertTriangle, Network } from 'lucide-react'
 import { clsx } from 'clsx'
 import { Section, PropertyList, Property, ConditionsSection, PodTemplateSection } from '../../ui/drawer-components'
 import { formatAge } from '../resource-utils'
+import { BADGE_INACTIVE } from '../../../utils/badge-colors'
 
 interface RolloutRendererProps {
   data: any
@@ -292,7 +293,7 @@ export function RolloutRenderer({ data }: RolloutRendererProps) {
                       'w-5 h-5 rounded-full flex items-center justify-center text-xs shrink-0',
                       isCompleted && 'bg-green-500/20 text-green-400',
                       isCurrent && 'bg-blue-500/20 text-blue-400',
-                      isPending && 'bg-gray-500/20 text-gray-400'
+                      isPending && BADGE_INACTIVE
                     )}
                   >
                     {isCompleted ? '\u2713' : isCurrent ? '\u25CF' : '\u25CB'}

package/src/components/resources/renderers/SecretRenderer.tsx

@@ -34,7 +34,11 @@ export function SecretRenderer({ data, certificateInfo, resourceData, onSaveSecr
   function toggleReveal(key: string) {
     setRevealed(prev => {
       const next = new Set(prev)
-      next.has(key) ? next.delete(key) : next.add(key)
+      if (next.has(key)) {
+        next.delete(key)
+      } else {
+        next.add(key)
+      }
       return next
     })
   }
@@ -153,6 +157,16 @@ export function SecretRenderer({ data, certificateInfo, resourceData, onSaveSecr
         </>
       )}
 
+      {dataKeys.length > 0 && (
+        <div className="flex items-center gap-2 p-3 bg-amber-500/10 border border-amber-500/30 rounded text-amber-400 text-sm">
+          <AlertTriangle className="w-4 h-4 flex-shrink-0" />
+          <span>
+            Secret values are sensitive. Reveal carefully — anyone who can see your
+            screen will see the plaintext.
+          </span>
+        </div>
+      )}
+
       <Section title="Data" defaultExpanded>
         <div className="space-y-2">
           {dataKeys.map((key) => {
@@ -250,11 +264,6 @@ export function SecretRenderer({ data, certificateInfo, resourceData, onSaveSecr
         </div>
       </Section>
 
-      <div className="flex items-center gap-2 p-3 bg-red-500/10 border border-red-500/30 rounded text-red-400 text-sm">
-        <AlertTriangle className="w-4 h-4" />
-        Secret values are sensitive. Be careful when revealing.
-      </div>
-
       {editingKey && (
         <ConfirmDialog
           open={showSaveConfirm}

package/src/components/resources/renderers/aws-capi-cells.tsx

@@ -2,6 +2,7 @@
 
 import { clsx } from 'clsx'
 import { Tooltip } from '../../ui/Tooltip'
+import { CAPACITY_TYPE_BADGE } from '../../../utils/badge-colors'
 import {
   getAWSMCPStatus, getAWSMCPEKSClusterName, getAWSMCPRegion, getAWSMCPVersion,
   getAWSMMPStatus, getAWSMMPInstanceType, getAWSMMPReplicas, getAWSMMPCapacityType,
@@ -57,10 +58,7 @@ export function AWSManagedMachinePoolCell({ resource, column }: { resource: any;
     case 'capacityType': {
       const ct = getAWSMMPCapacityType(resource)
       return (
-        <span className={clsx('badge badge-sm', ct === 'spot'
-          ? 'bg-amber-100 text-amber-800 border-amber-300 dark:bg-amber-950/50 dark:text-amber-400 dark:border-amber-700/40'
-          : 'bg-sky-100 text-sky-700 border-sky-300 dark:bg-sky-950/50 dark:text-sky-400 dark:border-sky-700/40'
-        )}>{ct === 'onDemand' ? 'On-Demand' : ct}</span>
+        <span className={clsx('badge badge-sm', ct === 'spot' ? CAPACITY_TYPE_BADGE.spot : CAPACITY_TYPE_BADGE.onDemand)}>{ct === 'onDemand' ? 'On-Demand' : ct}</span>
       )
     }
     default:

package/src/components/resources/renderers/azure-capi-cells.tsx

@@ -2,6 +2,7 @@
 
 import { Tooltip } from '../../ui/Tooltip'
 import { clsx } from 'clsx'
+import { CAPACITY_TYPE_BADGE, NODEPOOL_MODE_BADGE } from '../../../utils/badge-colors'
 import {
   getAzureMCPStatus, getAzureMCPLocation, getAzureMCPVersion, getAzureMCPResourceGroup,
   getAzureMMPStatus, getAzureMMPSKU, getAzureMMPMode, getAzureMMPReplicas, getAzureMMPScaleSetPriority,
@@ -40,20 +41,14 @@ export function AzureManagedMachinePoolCell({ resource, column }: { resource: an
     case 'mode': {
       const mode = getAzureMMPMode(resource)
       return (
-        <span className={clsx('badge badge-sm', mode === 'System'
-          ? 'bg-purple-100 text-purple-800 border-purple-300 dark:bg-purple-950/50 dark:text-purple-400 dark:border-purple-700/40'
-          : 'bg-sky-100 text-sky-700 border-sky-300 dark:bg-sky-950/50 dark:text-sky-400 dark:border-sky-700/40'
-        )}>{mode}</span>
+        <span className={clsx('badge badge-sm', NODEPOOL_MODE_BADGE[mode] || NODEPOOL_MODE_BADGE.User)}>{mode}</span>
       )
     }
     case 'replicas': return <TextCell value={getAzureMMPReplicas(resource)} />
     case 'priority': {
       const p = getAzureMMPScaleSetPriority(resource)
       return (
-        <span className={clsx('badge badge-sm', p === 'Spot'
-          ? 'bg-amber-100 text-amber-800 border-amber-300 dark:bg-amber-950/50 dark:text-amber-400 dark:border-amber-700/40'
-          : 'bg-sky-100 text-sky-700 border-sky-300 dark:bg-sky-950/50 dark:text-sky-400 dark:border-sky-700/40'
-        )}>{p}</span>
+        <span className={clsx('badge badge-sm', p === 'Spot' ? CAPACITY_TYPE_BADGE.spot : CAPACITY_TYPE_BADGE.regular)}>{p}</span>
       )
     }
     default: return <span className="text-sm text-theme-text-tertiary">-</span>

package/src/components/resources/renderers/istio-cells.tsx

@@ -1,6 +1,7 @@
 // Istio cell components for ResourcesView table
 
 import { clsx } from 'clsx'
+import { BADGE_INACTIVE } from '../../../utils/badge-colors'
 import {
   getVirtualServiceStatus,
   getVirtualServiceHosts,
@@ -164,7 +165,7 @@ export function PeerAuthenticationCell({ resource, column }: { resource: any; co
       return (
         <span className={clsx(
           'badge',
-          modeColors[mode] || 'bg-gray-500/20 text-gray-400'
+          modeColors[mode] || BADGE_INACTIVE
         )}>
           {mode}
         </span>
@@ -194,7 +195,7 @@ export function AuthorizationPolicyCell({ resource, column }: { resource: any; c
       return (
         <span className={clsx(
           'badge',
-          actionColors[action] || 'bg-gray-500/20 text-gray-400'
+          actionColors[action] || BADGE_INACTIVE
         )}>
           {action}
         </span>

package/src/components/resources/renderers/karpenter-cells.tsx

@@ -2,8 +2,8 @@
 
 import { clsx } from 'clsx'
 import { Tooltip } from '../../ui/Tooltip'
+import { CAPACITY_TYPE_BADGE } from '../../../utils/badge-colors'
 import {
-  CAPACITY_TYPE_BADGE,
   getNodePoolStatus,
   getNodePoolNodeClassRef,
   getNodePoolLimits,

package/src/components/resources/resource-utils-karpenter.ts

@@ -3,12 +3,6 @@
 import type { StatusBadge } from './resource-utils'
 import { healthColors } from './resource-utils'
 
-// Shared capacity type badge colors for NodeClaim renderer and table cells
-export const CAPACITY_TYPE_BADGE: Record<string, string> = {
-  spot: 'bg-amber-100 text-amber-800 border-amber-300 dark:bg-amber-950/50 dark:text-amber-400 dark:border-amber-700/40',
-  'on-demand': 'bg-sky-100 text-sky-700 border-sky-300 dark:bg-sky-950/50 dark:text-sky-400 dark:border-sky-700/40',
-}
-
 // ============================================================================
 // KARPENTER NODEPOOL UTILITIES
 // ============================================================================

package/src/components/resources/resource-utils.ts

@@ -130,6 +130,114 @@ export function getPodStatus(pod: any): StatusBadge {
   }
 }
 
+/**
+ * Pod phase enriched with readiness/restart/crash signals so the Phase
+ * row doesn't show a bare "Running" on a 0/1 or crash-looping pod.
+ */
+export interface PodPhaseDisplay {
+  /** Verbatim `pod.status.phase` (or "Unknown") — never lost. */
+  phase: string
+  /** Phase + derived qualifier, e.g. "Running — Not Ready (0/1)". */
+  text: string
+  /** Severity tier for tinting / icon choice, mirrors getPodStatus. */
+  level: HealthLevel
+  /** Optional one-line explanation surfaced as a tooltip / muted suffix. */
+  hint?: string
+}
+
+const RESTART_CYCLING_THRESHOLD = 5
+
+export function getPodPhaseDisplay(pod: any): PodPhaseDisplay {
+  const phase: string = pod?.status?.phase || 'Unknown'
+  const containerStatuses: any[] = pod?.status?.containerStatuses || []
+  const totalContainers = containerStatuses.length
+  const readyContainers = containerStatuses.filter((c) => c?.ready).length
+  const restartTotal = containerStatuses.reduce(
+    (sum, c) => sum + (c?.restartCount || 0),
+    0
+  )
+
+  if (pod?.metadata?.deletionTimestamp) {
+    return {
+      phase,
+      text: `${phase} — Terminating`,
+      level: 'degraded',
+      hint: 'Pod has a deletionTimestamp set; awaiting graceful termination.',
+    }
+  }
+
+  // Container-state failures take precedence over phase: a CrashLoopBackOff
+  // pod can still report phase: Running. Skip for Succeeded — a Job pod whose
+  // sidecar was OOMKilled before the main container completed should not be
+  // shown as unhealthy after the pod has reached terminal success.
+  if (phase !== 'Succeeded') {
+    for (const cs of containerStatuses) {
+      const waitingReason = cs?.state?.waiting?.reason
+      if (
+        waitingReason === 'CrashLoopBackOff' ||
+        waitingReason === 'ImagePullBackOff' ||
+        waitingReason === 'ErrImagePull' ||
+        waitingReason === 'CreateContainerConfigError'
+      ) {
+        return {
+          phase,
+          text: `${phase} — ${waitingReason}`,
+          level: 'unhealthy',
+          hint: `Container "${cs.name}" is stuck in ${waitingReason}.`,
+        }
+      }
+      if (cs?.state?.terminated?.reason === 'OOMKilled') {
+        return {
+          phase,
+          text: `${phase} — OOMKilled`,
+          level: 'unhealthy',
+          hint: `Container "${cs.name}" was OOMKilled.`,
+        }
+      }
+    }
+  }
+
+  switch (phase) {
+    case 'Running': {
+      const notReady = totalContainers > 0 && readyContainers < totalContainers
+      const cycling = restartTotal > RESTART_CYCLING_THRESHOLD
+      if (notReady && cycling) {
+        return {
+          phase,
+          text: `Running — Not Ready (${readyContainers}/${totalContainers}), ${restartTotal} restarts`,
+          level: 'unhealthy',
+          hint: 'Containers report not-ready and have restarted many times — likely crash-looping.',
+        }
+      }
+      if (notReady) {
+        return {
+          phase,
+          text: `Running — Not Ready (${readyContainers}/${totalContainers})`,
+          level: 'degraded',
+          hint: 'Pod is in the Running phase but at least one container is not ready (probes failing or still starting).',
+        }
+      }
+      if (cycling) {
+        return {
+          phase,
+          text: `Running — Restarting (${restartTotal} restarts)`,
+          level: 'degraded',
+          hint: 'Containers are ready right now but have restarted many times — investigate stability.',
+        }
+      }
+      return { phase, text: 'Running', level: 'healthy' }
+    }
+    case 'Succeeded':
+      return { phase, text: 'Completed', level: 'neutral' }
+    case 'Pending':
+      return { phase, text: 'Pending', level: 'degraded' }
+    case 'Failed':
+      return { phase, text: 'Failed', level: 'unhealthy' }
+    default:
+      return { phase, text: phase, level: 'unknown' }
+  }
+}
+
 export function getPodProblems(pod: any): PodProblem[] {
   const problems: PodProblem[] = []
   const containerStatuses = pod.status?.containerStatuses || []
@@ -1528,6 +1636,8 @@ export function getCellFilterValue(resource: any, column: string, kind: string):
   const kindLower = kind.toLowerCase()
 
   switch (column) {
+    case 'namespace':
+      return resource.metadata?.namespace || ''
     case 'type':
       if (kindLower === 'secrets' || kindLower === 'sealedsecrets') return getSecretType(resource).type
       if (kindLower === 'services') return resource.spec?.type || ''

package/src/components/resources/resources-search-sidebar-hint.test.ts

@@ -0,0 +1,85 @@
+import { describe, it, expect } from 'vitest'
+import { pluralize } from '../../utils/pluralize'
+import type { SelectedKindInfo } from './ResourcesSidebar'
+
+// Mirrors the empty-state hint rule inside ResourcesView (kept in
+// sync by reuse) — pure so we can pin the lookup + render contract
+// without rendering the React component. See
+// `packages/k8s-ui/src/components/resources/ResourcesView.tsx`
+// search for "the count is unfiltered".
+function sidebarHint(
+  counts: Record<string, number>,
+  selectedKind: SelectedKindInfo,
+  searchTerm: string,
+): string | null {
+  if (!searchTerm) return null
+  const key = selectedKind.group ? `${selectedKind.group}/${selectedKind.kind}` : selectedKind.kind
+  const totalForKind = counts[key] ?? 0
+  if (totalForKind === 0) return null
+  return `The sidebar shows ${pluralize(totalForKind, selectedKind.kind)} in the cluster — the count is unfiltered.`
+}
+
+describe('ResourcesView empty-search sidebar hint (SKY-828 bug 46)', () => {
+  // The bug: sidebar count badge stays unfiltered and shows the
+  // cluster-wide total per kind. When a search returns zero rows
+  // (e.g. "5 Pods" in sidebar, "No Pods found" in pane) the user
+  // reads the badge as a lie. The empty-state appends a sentence
+  // making the unfiltered nature explicit. The rule must:
+  //   1. Suppress entirely when there's no active search.
+  //   2. Suppress when the cluster total for the kind is 0
+  //      (no badge to explain → no hint).
+  //   3. Look up the count under `${group}/${kind}` for grouped
+  //      kinds, plain `kind` for core kinds (matches sidebar key).
+  //   4. Pluralize via pluralize() so "Ingress" → "Ingresses",
+  //      "NetworkPolicy" → "NetworkPolicies" (regression of the
+  //      naive `${kind}s` Cursor Bugbot caught in 80eb64b).
+  //   5. Singular for n===1: "1 Pod", not "1 Pods".
+
+  const pod: SelectedKindInfo = { name: 'pods', kind: 'Pod', group: '' }
+
+  it('returns null when there is no active search', () => {
+    expect(sidebarHint({ Pod: 232 }, pod, '')).toBeNull()
+  })
+
+  it('returns null when the cluster total for the kind is 0', () => {
+    expect(sidebarHint({ Pod: 0 }, pod, 'xyz')).toBeNull()
+  })
+
+  it('returns null when the kind is missing from counts', () => {
+    expect(sidebarHint({}, pod, 'xyz')).toBeNull()
+  })
+
+  it('formats the sentence with the cluster total and pluralized kind', () => {
+    expect(sidebarHint({ Pod: 232 }, pod, 'xyz')).toBe(
+      'The sidebar shows 232 Pods in the cluster — the count is unfiltered.',
+    )
+  })
+
+  it('uses singular noun when total is 1', () => {
+    expect(sidebarHint({ Pod: 1 }, pod, 'xyz')).toBe(
+      'The sidebar shows 1 Pod in the cluster — the count is unfiltered.',
+    )
+  })
+
+  it('looks up grouped kinds under `${group}/${kind}`', () => {
+    const ar: SelectedKindInfo = { name: 'rollouts', kind: 'Rollout', group: 'argoproj.io' }
+    const counts = { Rollout: 999, 'argoproj.io/Rollout': 4 }
+    expect(sidebarHint(counts, ar, 'xyz')).toBe(
+      'The sidebar shows 4 Rollouts in the cluster — the count is unfiltered.',
+    )
+  })
+
+  it('pluralizes Ingress correctly (regression: not "Ingresss")', () => {
+    const ing: SelectedKindInfo = { name: 'ingresses', kind: 'Ingress', group: '' }
+    expect(sidebarHint({ Ingress: 3 }, ing, 'xyz')).toBe(
+      'The sidebar shows 3 Ingresses in the cluster — the count is unfiltered.',
+    )
+  })
+
+  it('pluralizes NetworkPolicy correctly (regression: not "NetworkPolicys")', () => {
+    const np: SelectedKindInfo = { name: 'networkpolicies', kind: 'NetworkPolicy', group: 'networking.k8s.io' }
+    expect(sidebarHint({ 'networking.k8s.io/NetworkPolicy': 2 }, np, 'xyz')).toBe(
+      'The sidebar shows 2 NetworkPolicies in the cluster — the count is unfiltered.',
+    )
+  })
+})

package/src/components/timeline/shared.tsx

@@ -244,21 +244,20 @@ export function EventMarker({ event, x, selected, onClick, dimmed, small }: Even
       return 'bg-theme-hover/30 border-2 border-dashed border-theme-border-light'
     }
 
-    const opacity = dimmed ? '/50' : ''
     if (isProblematic) {
-      return `bg-amber-500${opacity}`
+      return dimmed ? 'bg-amber-500/50' : 'bg-amber-500'
     }
     if (isChange) {
       switch (event.eventType) {
         case 'add':
-          return `bg-green-500${opacity}`
+          return dimmed ? 'bg-green-500/50' : 'bg-green-500'
         case 'delete':
-          return `bg-red-500${opacity}`
+          return dimmed ? 'bg-red-500/50' : 'bg-red-500'
         case 'update':
-          return `bg-blue-500${opacity}`
+          return dimmed ? 'bg-blue-500/50' : 'bg-blue-500'
       }
     }
-    return `bg-theme-text-tertiary${opacity}`
+    return dimmed ? 'bg-theme-text-tertiary/50' : 'bg-theme-text-tertiary'
   }
 
   const getOperationLabel = () => {

package/src/components/topology/TopologySearch.tsx

@@ -9,6 +9,20 @@ interface TopologySearchProps {
   nodes: TopologyNode[]
   onNodeSelect?: (node: TopologyNode) => void
   onZoomToNode?: (nodeId: string) => void
+  /**
+   * Optional unfiltered node set. When the parent has applied a
+   * view-mode filter (e.g. Fleet mode hides Pods/Deployments and
+   * only shows CAPI kinds), it should still pass the full topology
+   * here so the empty-state can tell users "your query matches X
+   * resources hidden by the current view" rather than the
+   * misleading "No resources found." (SKY-828 bug 45)
+   */
+  allNodes?: TopologyNode[]
+  /**
+   * Human-readable label for the current view mode. Used in the
+   * empty-state hint above. Defaults to "current view".
+   */
+  viewModeLabel?: string
 }
 
 // Icon mapping for different resource kinds
@@ -50,33 +64,47 @@ function getKindColor(kind: string): string {
   }
 }
 
-export function TopologySearch({ nodes, onNodeSelect, onZoomToNode }: TopologySearchProps) {
+export function TopologySearch({ nodes, onNodeSelect, onZoomToNode, allNodes, viewModeLabel }: TopologySearchProps) {
   const [isOpen, setIsOpen] = useState(false)
   const [query, setQuery] = useState('')
   const [selectedIndex, setSelectedIndex] = useState(0)
   const inputRef = useRef<HTMLInputElement>(null)
   const resultsRef = useRef<HTMLDivElement>(null)
 
+  const matchesQuery = useCallback((node: TopologyNode, lowerQuery: string) => {
+    const name = node.name.toLowerCase()
+    const kind = node.kind.toLowerCase()
+    const namespace = (node.data.namespace as string || '').toLowerCase()
+    return (
+      name.includes(lowerQuery) ||
+      kind.includes(lowerQuery) ||
+      namespace.includes(lowerQuery) ||
+      `${kind}/${name}`.includes(lowerQuery) ||
+      `${namespace}/${name}`.includes(lowerQuery)
+    )
+  }, [])
+
   // Filter nodes based on query
   const filteredNodes = useMemo(() => {
     if (!query.trim()) return []
-
     const lowerQuery = query.toLowerCase()
     return nodes
-      .filter(node => {
-        const name = node.name.toLowerCase()
-        const kind = node.kind.toLowerCase()
-        const namespace = (node.data.namespace as string || '').toLowerCase()
-        return (
-          name.includes(lowerQuery) ||
-          kind.includes(lowerQuery) ||
-          namespace.includes(lowerQuery) ||
-          `${kind}/${name}`.includes(lowerQuery) ||
-          `${namespace}/${name}`.includes(lowerQuery)
-        )
-      })
+      .filter(node => matchesQuery(node, lowerQuery))
       .slice(0, 10) // Limit results
-  }, [nodes, query])
+  }, [nodes, query, matchesQuery])
+
+  // Count of matches in the unfiltered topology that the current
+  // view-mode filter is hiding. We only compute this when there's
+  // a query AND the visible set returned zero results, so the cost
+  // is bounded to actual no-result interactions.
+  // (SKY-828 bug 45: Fleet view hides Pods, so searching pod names
+  // returned "No resources found" — misleading.)
+  const hiddenMatchCount = useMemo(() => {
+    if (!query.trim() || filteredNodes.length > 0 || !allNodes) return 0
+    const lowerQuery = query.toLowerCase()
+    const visibleIds = new Set(nodes.map(n => n.id))
+    return allNodes.filter(n => !visibleIds.has(n.id) && matchesQuery(n, lowerQuery)).length
+  }, [query, filteredNodes.length, allNodes, nodes, matchesQuery])
 
   // Reset selection when results change
   useEffect(() => {
@@ -216,7 +244,12 @@ export function TopologySearch({ nodes, onNodeSelect, onZoomToNode }: TopologySe
               {query && filteredNodes.length === 0 && (
                 <div className="px-4 py-8 text-center text-theme-text-tertiary">
                   <Search className="w-8 h-8 mx-auto mb-2 opacity-50" />
-                  <p>No resources found for "{query}"</p>
+                  <p>No resources found for "{query}"{viewModeLabel ? ` in ${viewModeLabel}` : ''}</p>
+                  {hiddenMatchCount > 0 && (
+                    <p className="mt-2 text-xs text-amber-400">
+                      {hiddenMatchCount} {hiddenMatchCount === 1 ? 'match is' : 'matches are'} hidden by the current view{viewModeLabel ? ` (${viewModeLabel})` : ''}. Switch view to see them.
+                    </p>
+                  )}
                 </div>
               )}