lsm / neokai / 28214742412

82%

Build Type

push

github

Committed by web-flow

Commit Message

fix(external-events): dedupe polled PR metadata on head sha, not updated_at (#2191)

GitHub bumps a PR's updated_at on every touch (comment, check_run, review,
push), so keying the polling dedupe suffix on updated_at re-delivered the
same PR metadata event every poll cycle during a review storm. The deduper
never collapsed the row because the suffix changed each cycle, and the
volatile updated_at also fed deliveryId/externalId.

Key the pulls dedupe on the PR head sha instead, so a metadata event is
re-emitted only on a real push; comments and checks already arrive via
their own dedicated polling events. Fall back to updatedAt only when the
head is missing (deleted-head PRs) so the row still dedupes within a cycle.
occurredAt is unchanged. issue_comments/review_comments are unaffected
(comments are immutable, their updated_at is stable).

Adds unit tests for normalizeGitHubPollingRow covering the three cases.

Coverage Stats

11380 of 16404 branches covered (69.37%)

Branch coverage included in aggregate %.

4 of 4 new or added lines in 1 file covered. (100.0%)

1 existing line in 1 file now uncovered.

95523 of 114175 relevant lines covered (83.66%)

299.86 hits per line