From 71c5826c35f72714af19e598b31b08ecd7c5f62c Mon Sep 17 00:00:00 2001
From: rcourtman <courtmanr@gmail.com>
Date: Mon, 11 May 2026 11:17:57 +0100
Subject: [PATCH] Hide Investigate / Verify fix on non-active findings
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Investigate's prompt instructs the LLM to "actively use the Pulse
tools to gather fresh evidence about the current state of the
affected resource." Verify fix asks "did the applied fix actually
clear the underlying condition right now." Both prompts are about
CURRENT state — irrelevant for findings whose status is resolved,
dismissed, or snoozed, where there is no current condition to
investigate or verify against.

Gates both buttons on finding.status === 'active'. Explain, Why,
and Discuss stay on closed findings — retrospective questions
("walk me through what we know," "what caused this") remain
meaningful for audit-trail review.

(The hasAppliedFix gate stays on Verify fix; the new active-status
gate just adds an additional necessary condition.)
---
 .../src/components/AI/FindingsPanel.tsx       | 36 ++++++++++---------
 1 file changed, 19 insertions(+), 17 deletions(-)

diff --git a/frontend-modern/src/components/AI/FindingsPanel.tsx b/frontend-modern/src/components/AI/FindingsPanel.tsx
index 6676d8f750..e485f2dade 100644
--- a/frontend-modern/src/components/AI/FindingsPanel.tsx
+++ b/frontend-modern/src/components/AI/FindingsPanel.tsx
@@ -1299,22 +1299,24 @@ export const FindingsPanel: Component<FindingsPanelProps> = (props) => {
               </svg>
               Explain
             </button>
-            <button
-              type="button"
-              onClick={(e) => handleInvestigateFinding(finding, e)}
-              class="px-2 py-1 rounded border border-border hover:bg-surface-hover flex items-center gap-1"
-              title="Ask Pulse Assistant to actively investigate using metrics, alerts, and state — not just summarize"
-            >
-              <svg class="w-3 h-3" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-                <path
-                  stroke-linecap="round"
-                  stroke-linejoin="round"
-                  stroke-width="2"
-                  d="M21 21l-6-6m2-5a7 7 0 11-14 0 7 7 0 0114 0z"
-                />
-              </svg>
-              Investigate
-            </button>
+            <Show when={finding.status === 'active'}>
+              <button
+                type="button"
+                onClick={(e) => handleInvestigateFinding(finding, e)}
+                class="px-2 py-1 rounded border border-border hover:bg-surface-hover flex items-center gap-1"
+                title="Ask Pulse Assistant to actively investigate using metrics, alerts, and state — not just summarize"
+              >
+                <svg class="w-3 h-3" fill="none" stroke="currentColor" viewBox="0 0 24 24">
+                  <path
+                    stroke-linecap="round"
+                    stroke-linejoin="round"
+                    stroke-width="2"
+                    d="M21 21l-6-6m2-5a7 7 0 11-14 0 7 7 0 0114 0z"
+                  />
+                </svg>
+                Investigate
+              </button>
+            </Show>
             <button
               type="button"
               onClick={(e) => handleWhyFinding(finding, e)}
@@ -1331,7 +1333,7 @@ export const FindingsPanel: Component<FindingsPanelProps> = (props) => {
               </svg>
               Why
             </button>
-            <Show when={hasAppliedFix(finding)}>
+            <Show when={hasAppliedFix(finding) && finding.status === 'active'}>
               <button
                 type="button"
                 onClick={(e) => handleVerifyFixFinding(finding, e)}