From b1199b3cbf75555e36ce5bff872f22c2cd19cbb8 Mon Sep 17 00:00:00 2001
From: rcourtman <courtmanr@gmail.com>
Date: Wed, 10 Dec 2025 23:31:21 +0000
Subject: [PATCH] fix(ai): Use context.Background() for forced patrol runs

The ForcePatrol() function was using the HTTP request context, which gets
cancelled immediately when the API response is sent. This caused LLM analysis
to fail with 'context canceled' before it could complete.

Now uses context.Background() so the goroutine runs independently of the
HTTP request lifecycle.

Also fixed dropdown hover gap issue in the dismiss menu.
---
 frontend-modern/src/pages/Alerts.tsx | 128 ++++++++++++++-------------
 internal/ai/patrol.go                |   3 +-
 2 files changed, 67 insertions(+), 64 deletions(-)

diff --git a/frontend-modern/src/pages/Alerts.tsx b/frontend-modern/src/pages/Alerts.tsx
index 19d47f0..fbf6a69 100644
--- a/frontend-modern/src/pages/Alerts.tsx
+++ b/frontend-modern/src/pages/Alerts.tsx
@@ -2657,76 +2657,78 @@ function OverviewTab(props: {
                                     <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M19 9l-7 7-7-7" />
                                   </svg>
                                 </button>
-                                {/* Dropdown menu */}
-                                <div class="absolute right-0 mt-1 w-48 bg-white dark:bg-gray-800 rounded-lg shadow-lg border border-gray-200 dark:border-gray-700 opacity-0 invisible group-hover:opacity-100 group-hover:visible transition-all z-50">
-                                  <button
-                                    class="w-full px-3 py-2 text-left text-xs hover:bg-gray-100 dark:hover:bg-gray-700 rounded-t-lg transition-colors"
-                                    onClick={async (e) => {
-                                      e.stopPropagation();
-                                      try {
-                                        await dismissFinding(finding.id, 'not_an_issue');
-                                        showSuccess('Dismissed - AI will not raise this again');
-                                        fetchAiData();
-                                      } catch (err) {
-                                        showError('Failed to dismiss finding');
-                                      }
-                                    }}
-                                  >
-                                    <span class="font-medium text-gray-700 dark:text-gray-300">Not an Issue</span>
-                                    <p class="text-gray-500 dark:text-gray-500 mt-0.5">This isn't actually a problem</p>
-                                  </button>
-                                  <button
-                                    class="w-full px-3 py-2 text-left text-xs hover:bg-gray-100 dark:hover:bg-gray-700 transition-colors"
-                                    onClick={async (e) => {
-                                      e.stopPropagation();
-                                      const note = prompt('Why is this expected? (optional - helps AI understand your environment)');
-                                      try {
-                                        await dismissFinding(finding.id, 'expected_behavior', note || undefined);
-                                        showSuccess('Dismissed - AI will not raise this again');
-                                        fetchAiData();
-                                      } catch (err) {
-                                        showError('Failed to dismiss finding');
-                                      }
-                                    }}
-                                  >
-                                    <span class="font-medium text-gray-700 dark:text-gray-300">Expected Behavior</span>
-                                    <p class="text-gray-500 dark:text-gray-500 mt-0.5">This is intentional/by design</p>
-                                  </button>
-                                  <button
-                                    class="w-full px-3 py-2 text-left text-xs hover:bg-gray-100 dark:hover:bg-gray-700 transition-colors"
-                                    onClick={async (e) => {
-                                      e.stopPropagation();
-                                      try {
-                                        await dismissFinding(finding.id, 'will_fix_later');
-                                        showSuccess('Acknowledged - AI will check again later');
-                                        fetchAiData();
-                                      } catch (err) {
-                                        showError('Failed to dismiss finding');
-                                      }
-                                    }}
-                                  >
-                                    <span class="font-medium text-gray-700 dark:text-gray-300">Will Fix Later</span>
-                                    <p class="text-gray-500 dark:text-gray-500 mt-0.5">I know about it, will address</p>
-                                  </button>
-                                  <div class="border-t border-gray-200 dark:border-gray-700">
+                                {/* Dropdown menu - pt-2 creates visual gap while maintaining hover area */}
+                                <div class="absolute right-0 top-full pt-1 w-48 opacity-0 invisible group-hover:opacity-100 group-hover:visible transition-all z-50">
+                                  <div class="bg-white dark:bg-gray-800 rounded-lg shadow-lg border border-gray-200 dark:border-gray-700">
                                     <button
-                                      class="w-full px-3 py-2 text-left text-xs hover:bg-red-50 dark:hover:bg-red-900/30 rounded-b-lg transition-colors text-red-600 dark:text-red-400"
+                                      class="w-full px-3 py-2 text-left text-xs hover:bg-gray-100 dark:hover:bg-gray-700 rounded-t-lg transition-colors"
                                       onClick={async (e) => {
                                         e.stopPropagation();
-                                        if (confirm('Permanently suppress this type of finding for this resource?\n\nThe AI will never raise this issue again.')) {
-                                          try {
-                                            await suppressFinding(finding.id);
-                                            showSuccess('Suppressed - AI will never raise this again');
-                                            fetchAiData();
-                                          } catch (err) {
-                                            showError('Failed to suppress finding');
-                                          }
+                                        try {
+                                          await dismissFinding(finding.id, 'not_an_issue');
+                                          showSuccess('Dismissed - AI will not raise this again');
+                                          fetchAiData();
+                                        } catch (err) {
+                                          showError('Failed to dismiss finding');
                                         }
                                       }}
                                     >
-                                      <span class="font-medium">Never Alert Again</span>
-                                      <p class="text-red-500/80 dark:text-red-400/80 mt-0.5">Permanently suppress for this resource</p>
+                                      <span class="font-medium text-gray-700 dark:text-gray-300">Not an Issue</span>
+                                      <p class="text-gray-500 dark:text-gray-500 mt-0.5">This isn't actually a problem</p>
                                     </button>
+                                    <button
+                                      class="w-full px-3 py-2 text-left text-xs hover:bg-gray-100 dark:hover:bg-gray-700 transition-colors"
+                                      onClick={async (e) => {
+                                        e.stopPropagation();
+                                        const note = prompt('Why is this expected? (optional - helps AI understand your environment)');
+                                        try {
+                                          await dismissFinding(finding.id, 'expected_behavior', note || undefined);
+                                          showSuccess('Dismissed - AI will not raise this again');
+                                          fetchAiData();
+                                        } catch (err) {
+                                          showError('Failed to dismiss finding');
+                                        }
+                                      }}
+                                    >
+                                      <span class="font-medium text-gray-700 dark:text-gray-300">Expected Behavior</span>
+                                      <p class="text-gray-500 dark:text-gray-500 mt-0.5">This is intentional/by design</p>
+                                    </button>
+                                    <button
+                                      class="w-full px-3 py-2 text-left text-xs hover:bg-gray-100 dark:hover:bg-gray-700 transition-colors"
+                                      onClick={async (e) => {
+                                        e.stopPropagation();
+                                        try {
+                                          await dismissFinding(finding.id, 'will_fix_later');
+                                          showSuccess('Acknowledged - AI will check again later');
+                                          fetchAiData();
+                                        } catch (err) {
+                                          showError('Failed to dismiss finding');
+                                        }
+                                      }}
+                                    >
+                                      <span class="font-medium text-gray-700 dark:text-gray-300">Will Fix Later</span>
+                                      <p class="text-gray-500 dark:text-gray-500 mt-0.5">I know about it, will address</p>
+                                    </button>
+                                    <div class="border-t border-gray-200 dark:border-gray-700">
+                                      <button
+                                        class="w-full px-3 py-2 text-left text-xs hover:bg-red-50 dark:hover:bg-red-900/30 rounded-b-lg transition-colors text-red-600 dark:text-red-400"
+                                        onClick={async (e) => {
+                                          e.stopPropagation();
+                                          if (confirm('Permanently suppress this type of finding for this resource?\n\nThe AI will never raise this issue again.')) {
+                                            try {
+                                              await suppressFinding(finding.id);
+                                              showSuccess('Suppressed - AI will never raise this again');
+                                              fetchAiData();
+                                            } catch (err) {
+                                              showError('Failed to suppress finding');
+                                            }
+                                          }
+                                        }}
+                                      >
+                                        <span class="font-medium">Never Alert Again</span>
+                                        <p class="text-red-500/80 dark:text-red-400/80 mt-0.5">Permanently suppress for this resource</p>
+                                      </button>
+                                    </div>
                                   </div>
                                 </div>
                               </div>
diff --git a/internal/ai/patrol.go b/internal/ai/patrol.go
index 1ff0af6..ea39717 100644
--- a/internal/ai/patrol.go
+++ b/internal/ai/patrol.go
@@ -1099,8 +1099,9 @@ func (p *PatrolService) GetFindingsHistory(startTime *time.Time) []*Finding {
 
 // ForcePatrol triggers an immediate patrol run
 // The deep parameter is kept for API backwards compatibility but is ignored
+// Uses context.Background() since this runs async after the HTTP response
 func (p *PatrolService) ForcePatrol(ctx context.Context, deep bool) {
-	go p.runPatrol(ctx)
+	go p.runPatrol(context.Background())
 }
 
 // analyzePBSInstance checks a PBS backup server for issues