From 84872580a3f2b8b074c27e933e3368db9cb9800b Mon Sep 17 00:00:00 2001
From: Evan Jarrett <evan@evanjarrett.com>
Date: Wed, 2 Sep 2026 21:37:56 -0500
Subject: [PATCH] appview: show a chart's deprecation where people actually
 look

digest.html rendered a Deprecated chip, but the shared helm-metadata partial
never read .Deprecated, so the repo page carried no deprecation signal at all.
The data was already on the struct; this was a display omission.

The notice leads the metadata card, above the description: deprecation decides
whether you use the chart at all, so it has to be read before the prose that
sells it. It uses the alert/alert-warning callout the codebase already uses for
this kind of thing, including in the adjacent helm-digest-content partial,
rather than a bare badge. A lone small badge reads as a stray tag once it is
outside the digest page's row of status chips, and leaves no room to say why it
matters.

The digest page now shows deprecation twice, deliberately: its header chip is
the scannable signal above the fold, and this carries the explanation further
down. Removing the chip would push the only signal below the install command,
which is the burial this fix is meant to undo.

Note the repo page's Overview tab, which is where most people land, still shows
nothing: it renders the README and never touches chart metadata. Only the Chart
tab gains the notice here.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PDqoCE1j3njokkZ9b1C5n9
---
 pkg/appview/templates/partials/helm-metadata.html | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/pkg/appview/templates/partials/helm-metadata.html b/pkg/appview/templates/partials/helm-metadata.html
index 6401a05..c6d639e 100644
--- a/pkg/appview/templates/partials/helm-metadata.html
+++ b/pkg/appview/templates/partials/helm-metadata.html
@@ -5,6 +5,20 @@
     sparse charts don't render with a wall of "—" cells.
 */}}
 <div class="card bg-base-200 shadow-sm border border-base-300 p-6 space-y-4 min-w-0">
+    {{/* Deprecation leads the card: it changes whether you should use the
+         chart at all, so it has to be readable before the description.
+         The digest page also renders a small chip in its header; that one
+         is a scannable status label, this one carries the explanation. */}}
+    {{ if .Deprecated }}
+        <div class="alert alert-warning" role="status">
+            {{ icon "alert-triangle" "size-5 shrink-0" }}
+            <div>
+                <p class="font-medium">This chart is deprecated</p>
+                <p class="text-sm">The maintainers marked it deprecated in Chart.yaml. It may not get further updates or fixes.</p>
+            </div>
+        </div>
+    {{ end }}
+
     {{ if .Description }}
         <div>
             <h2 class="text-sm font-semibold uppercase tracking-wider text-base-content/70 mb-2">Description</h2>