Skip to content

Deepen webhook ingestion module#63

Merged
darthfork merged 2 commits into
mainfrom
deepen-webhook-ingestion
May 21, 2026
Merged

Deepen webhook ingestion module#63
darthfork merged 2 commits into
mainfrom
deepen-webhook-ingestion

Conversation

@darthfork
Copy link
Copy Markdown
Owner

Summary

  • add a webhook ingestion Module with an interface-level Accept path for signature validation, delivery dedupe, queue acceptance, sync dispatch, and status outcomes
  • turn githubEventsHandler into an HTTP Adapter that reads request data and delegates to the ingestion Module
  • add interface-level tests for invalid signatures, duplicate deliveries, delivery-store errors, sync dispatch, async enqueue, queue-full behavior, and HTTP header/body adaptation

TDD / Tests

  • Added failing tests first in src/webhook_ingestion_test.go, then implemented the Module and Adapter
  • Verified with: mise exec go@1.25.10 -- go test ./...

Notes

  • The repo's .go-version is 1.25.10, while plain go on PATH reported 1.25.9 in this shell, so verification used mise exec go@1.25.10.

@darthfork darthfork merged commit eae711e into main May 21, 2026
3 checks passed
@darthfork darthfork deleted the deepen-webhook-ingestion branch May 21, 2026 21:48
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