Skip to content

Consolidate event dispatch policy#67

Merged
darthfork merged 1 commit into
mainfrom
consolidate-event-dispatch
Jun 5, 2026
Merged

Consolidate event dispatch policy#67
darthfork merged 1 commit into
mainfrom
consolidate-event-dispatch

Conversation

@darthfork
Copy link
Copy Markdown
Owner

Summary

  • add a GitHub event dispatcher module that owns supported event registration and unsupported-event outcomes
  • route both synchronous webhook fallback and async worker processing through the same dispatcher policy
  • update async tests and integration tests to inject event handlers through the dispatcher seam
  • bump .go-version to Go 1.25.11 so security scans use the patched standard library

TDD notes

  • RED: added dispatcher interface tests for supported routing and unknown-event rejection
  • GREEN: introduced the dispatcher module and wired synchronous webhook dispatch through it
  • RED: added async processor coverage proving async processing uses the shared dispatcher
  • GREEN/REFACTOR: removed the async-only handler map and routed worker processing through the dispatcher

Cardinality guardrails

  • no Prometheus collector definitions in src/metrics.go were changed
  • metric recorder cardinality guard tests remain in place

Verification

  • make test-all
  • make lint
  • make security

@darthfork darthfork marked this pull request as ready for review June 5, 2026 01:58
@darthfork darthfork merged commit f4c1ee4 into main Jun 5, 2026
3 checks passed
@darthfork darthfork deleted the consolidate-event-dispatch branch June 5, 2026 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant