HelixObs / gateway / 26158274886

91%

push

github

notifier: fix duplicate GitHub issues on burst reconnect

Two bugs caused the same fingerprint to open multiple issues:

1. Race condition: a burst of events (e.g. BatchSpanProcessor flush after
   reconnect) all called FindNotificationIssue before any UpsertNotificationIssue
   completed, so every goroutine saw nil and called createIssue. Fixed by a
   per-fingerprint sync.Mutex in doSCM that serialises concurrent dispatches.

2. Context cancellation: UpsertNotificationIssue was called with the app context,
   so a gateway restart after GitHub issue creation (but before the DB write) left
   an orphaned issue with no notification_issues record. Fixed by using
   context.Background() for the DB write in createIssue.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

15 of 15 new or added lines in 2 files covered. (100.0%)

1899 of 2307 relevant lines covered (82.31%)

1970.38 hits per line