@skyhook-io/k8s-ui 1.8.1 → 1.8.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skyhook-io/k8s-ui",
-  "version": "1.8.1",
+  "version": "1.8.2",
   "repository": {
     "type": "git",
     "url": "https://github.com/skyhook-io/radar",

package/src/components/applications/AppTooltips.tsx

@@ -1,4 +1,4 @@
-import { provenanceSource, HEALTH_META, type AppWorkload } from '../../utils/applications'
+import { provenanceSource, HEALTH_META, appSourceLabel, isDeclaredAppSource, APP_IDENTITY_ANNOTATION, type AppWorkload } from '../../utils/applications'
 import { midTruncate } from '../../utils/format'
 
 // Structured tooltip content for the application chips. Both render a short
@@ -105,26 +105,53 @@ export function CategoryTooltip({
   )
 }
 
-// AppIdentityTooltip — why these instances are grouped. One line per distinct
-// evidence in the ProvenanceTooltip idiom (signals as inline-code, long repo
-// refs mid-truncated), plus a small flag when the grouping is heuristic-only.
-// No member list — the env pills next to this chip already show the instances.
+// AppIdentityTooltip — why these instances are grouped, and (in the fleet view)
+// whether the grouping folds across clusters + how to make it. A clear
+// source-driven headline, the per-evidence detail lines, and — when `fleet` and
+// the identity isn't a declared cross-cluster origin — the canonical "how to
+// fold" answer, so the question and its answer share one surface.
+//
+// `source`/`portable` come from the row's identity; absent (an older host that
+// doesn't pass them) the headline degrades to the evidence lines. `fleet` gates
+// the cross-cluster "how to fold" line — single-cluster (OSS) has no clusters to
+// fold across, so it shows only the grouping evidence.
 export function AppIdentityTooltip({
   members,
+  source,
+  portable,
+  fleet,
 }: {
   identityKey?: string
   members: { name: string; env: string; confidence: string; evidence: string }[]
+  source?: string
+  portable?: boolean
+  fleet?: boolean
 }) {
   const evidences = Array.from(new Set(members.map((m) => m.evidence).filter(Boolean)))
   const heuristicOnly = members.every((m) => m.confidence !== 'high')
+  // The wire `portable` flag is authoritative; fall back to deriving it from the
+  // source when an older host doesn't pass it.
+  const folds = portable ?? isDeclaredAppSource(source)
   return (
-    <div className="max-w-xs space-y-1">
+    <div className="max-w-xs space-y-1.5">
+      {source && <div className="text-xs font-medium text-theme-text-primary">Grouped by {appSourceLabel(source)}.</div>}
       {evidences.map((e, i) => (
-        <div key={i} className="text-xs text-theme-text-primary">
+        <div key={i} className="text-xs text-theme-text-secondary">
           <EvidenceLine evidence={e} />
         </div>
       ))}
-      {heuristicOnly && (
+      {fleet && folds && (
+        <div className="text-[10px] uppercase tracking-wide text-theme-text-tertiary">folds across clusters</div>
+      )}
+      {fleet && source !== undefined && !folds && (
+        <div className="space-y-1 border-t border-theme-border pt-1.5">
+          <div className="text-[11px] leading-snug text-theme-text-secondary">
+            Shown per-cluster — a name can't prove two clusters run the same app. To fold it across clusters, set the same{' '}
+            <code className="inline-code">{APP_IDENTITY_ANNOTATION}</code> on its workloads in each cluster, or deploy it via Argo CD / Flux with the environment in the source path.
+          </div>
+        </div>
+      )}
+      {!fleet && heuristicOnly && !source && (
         <div className={`text-[10px] uppercase tracking-wide ${HEALTH_META.degraded.text}`}>heuristic · medium confidence</div>
       )}
     </div>