Simplify Patrol findings header

This commit is contained in:
rcourtman
2026-03-25 16:36:42 +00:00
parent 2ac8a1fef9
commit b138e182b3
3 changed files with 14 additions and 4 deletions
@@ -186,6 +186,11 @@ Patrol page, `frontend-modern/src/components/AI/FindingsPanel.tsx` should
explain the absence of active findings without repeating `Last activity`,
`Next run`, or interval schedule details that already belong to the header and
verification hierarchy above.
That same findings surface should keep its section chrome functional rather
than promotional. Inside the Patrol findings tab, the card header should use a
direct label such as `Patrol findings` and should not add extra product
restatement or marketing subtitles that compete with the page-level Patrol
header above.
When Patrol is currently running, that strip should still stay factual rather
than switching to another verdict label: the runtime may add an explicit
in-progress indicator, but the primary activity label remains recent activity
@@ -2,7 +2,7 @@
* FindingsPanel
*
* Separated view showing:
* - Pulse Patrol Findings (AI-discovered insights)
* - Patrol findings
* - Threshold Alerts (user-configured rules)
* Each section has severity-based sorting and quick actions
*
@@ -1060,7 +1060,7 @@ export const FindingsPanel: Component<FindingsPanelProps> = (props) => {
</div>
</Show>
{/* Pulse Patrol Findings Section */}
{/* Patrol findings section */}
<Show when={!aiIntelligenceStore.findingsLoading}>
<Card padding="none" class="overflow-hidden">
{/* Header */}
@@ -1080,14 +1080,13 @@ export const FindingsPanel: Component<FindingsPanelProps> = (props) => {
d="M9.663 17h4.673M12 3v1m6.364 1.636l-.707.707M21 12h-1M4 12H3m3.343-5.657l-.707-.707m2.828 9.9a5 5 0 117.072 0l-.548.547A3.374 3.374 0 0014 18.469V19a2 2 0 11-4 0v-.531c0-.895-.356-1.754-.988-2.386l-.548-.547z"
/>
</svg>
<span class="font-medium text-base-content">Pulse Patrol Findings</span>
<span class="font-medium text-base-content">Patrol findings</span>
<Show when={patrolFindings().length > 0}>
<span class="px-2 py-0.5 text-xs font-medium bg-blue-200 dark:bg-blue-700 text-blue-800 dark:text-blue-200 rounded-full">
{patrolFindings().length}
</span>
</Show>
</div>
<span class="text-xs text-muted">AI-discovered insights</span>
</div>
</div>
{/* Content */}
@@ -165,6 +165,12 @@ describe('aiFindingPresentation', () => {
expect(findingsPanelSource).not.toContain('Runs every');
expect(findingsPanelSource).not.toContain('Next: ');
});
it('keeps the findings card header functional instead of repeating product marketing copy', () => {
expect(findingsPanelSource).toContain('Patrol findings');
expect(findingsPanelSource).not.toContain('Pulse Patrol Findings');
expect(findingsPanelSource).not.toContain('AI-discovered insights');
});
});
describe('sourceColors', () => {