mirror of
https://github.com/rcourtman/Pulse.git
synced 2026-09-24 12:13:28 +00:00
rcourtman
33af1627f4
The main issue was that finding IDs included the title, which the LLM generates differently each time. 'High CPU on minipc' vs 'Node minipc experiencing high CPU load' got different IDs, making dismissals useless. Changes: 1. LLM findings now get IDs based on resource+category only, not title 2. Add() now checks if finding is suppressed before adding as new 3. Add() now checks dismissed findings and only reactivates on severity escalation 4. IsSuppressed() now matches by resource+category only, not title 5. Added isSuppressedInternal() for use when lock is already held Now when you dismiss 'performance issues on minipc', any future patrol finding about performance on minipc will be recognized as the same issue and stay dismissed.