Skip to content

Limit webhook poll batch size#102

Merged
matthiastjong merged 1 commit into
matthiastjong:mainfrom
steve-wecompare:fix/webhook-poll-default-limit
Jun 9, 2026
Merged

Limit webhook poll batch size#102
matthiastjong merged 1 commit into
matthiastjong:mainfrom
steve-wecompare:fix/webhook-poll-default-limit

Conversation

@steve-wecompare

Copy link
Copy Markdown
Contributor

Summary

  • default webhook polling to batches of 5 pending events
  • return pending events oldest-first so backlogs drain predictably
  • allow an optional ?limit= override, clamped to a bounded range
  • add regression coverage for default and explicit poll limits

Tests

  • pnpm check passes (0 errors, existing Svelte warnings only)
  • pnpm exec vitest run tests/integration/webhook-events.test.ts could not run locally because Testcontainers could not find a working container runtime strategy on this machine

@matthiastjong matthiastjong merged commit 2782ad6 into matthiastjong:main Jun 9, 2026
4 checks passed
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.

2 participants