Join Slack
Powered by
Hey :wave: 3 PRs stacked on top of each other to ...
# proj-automation
m
modern-fireman-99755
08/24/2026, 2:18 PM
Hey ๐ 3 PRs stacked on top of each other to lay the groundwork for webhooks in the automation service. Please review them in the listed order: 1.
refactor: extract a transport-neutral accept_event() from the webhook handler
2.
feat: replace the parse-only source registry with a provider descriptor and verifier registry
3.
feat: persist accepted events to deduplicate redeliveries and expose events that matched nothing
Tested DB migration, changes are not breaking other (cron) automations, small fake webhook automation, in the VM and locally. For instance, everything works fine.
โ 1
l
limited-student-10747
08/24/2026, 6:56 PM
Tangentially related comment, have you tried the official github stacked PRs? theyโre pretty handy:
github.github.com/gh-stack
m
modern-fireman-99755
08/24/2026, 6:57 PM
No but I should do that....
๐ 1
modern-fireman-99755
08/24/2026, 7:23 PM
@quick-window-91705
Thanks for the reviews. I answered your comment ๐
โ 1
Open in Slack
Previous
Next