feat: move GitHub webhook processing into Restate#5099
Draft
Flo4604 wants to merge 2 commits intofeat/webhook-out-of-order-guardfrom
Draft
feat: move GitHub webhook processing into Restate#5099Flo4604 wants to merge 2 commits intofeat/webhook-out-of-order-guardfrom
Flo4604 wants to merge 2 commits intofeat/webhook-out-of-order-guardfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
This was referenced Feb 19, 2026
Member
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
19 tasks
Flo4604
changed the base branch from
feat/webhook-dedup
to
graphite-base/5099
Flo4604
force-pushed
the
graphite-base/5099
branch
from
9523364 to
6040a5b
Compare
Flo4604
force-pushed
the
feat/webhook-restate
branch
from
5c98b6f to
26c30cf
Compare
Flo4604
force-pushed
the
feat/webhook-restate
branch
from
26c30cf to
95cbcaf
Compare
Flo4604
force-pushed
the
graphite-base/5099
branch
from
6040a5b to
9abe5bd
Compare
Flo4604
changed the base branch from
graphite-base/5099
to
feat/webhook-out-of-order-guard
Flo4604
force-pushed
the
feat/webhook-out-of-order-guard
branch
from
9abe5bd to
c810ba7
Compare
Flo4604
force-pushed
the
feat/webhook-restate
branch
from
95cbcaf to
ef4aafd
Compare
Flo4604
force-pushed
the
feat/webhook-out-of-order-guard
branch
from
c810ba7 to
d1856a5
Compare
Flo4604
force-pushed
the
feat/webhook-restate
branch
from
ef4aafd to
7aef4ae
Compare
19 tasks
Flo4604
force-pushed
the
feat/webhook-out-of-order-guard
branch
from
d1856a5 to
ecacf4a
Compare
Flo4604
force-pushed
the
feat/webhook-restate
branch
from
7aef4ae to
8351dac
Compare
Thins the HTTP handler to: validate signature → parse payload → send to Restate → return 200. All DB work moves into a new GitHubWebhookService (VIRTUAL_OBJECT keyed by installation_id/repo_id) that serializes processing per repository and retries automatically on failure. X-GitHub-Delivery is passed as the Restate idempotency key for natural deduplication of retried deliveries.
Replace FindAppByProjectAndSlug("default") with FindAppById(repo.AppID)
for app-scoped deployments. Change all per-connection error paths from
return nil,err to continue with logging, so one bad connection cannot
prevent other connections from deploying.
Flo4604
force-pushed
the
feat/webhook-out-of-order-guard
branch
from
ecacf4a to
ee71809
Compare
Flo4604
force-pushed
the
feat/webhook-restate
branch
from
8351dac to
c38a87c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Fixes # (issue)
If there is not an issue for this, please create one first. This is used to tracking purposes and also helps us understand why this PR exists
Type of change
How should this be tested?
Checklist
Required
pnpm buildpnpm fmtmake fmton/godirectoryconsole.logsgit pull origin mainAppreciated