Skip to content

fix(webhook): remove unused payload from retained delivery state (#2861 salvage) - #763

Merged
hashbender merged 1 commit into
mainfrom
mirror/pr-56945
Jul 2, 2026
Merged

fix(webhook): remove unused payload from retained delivery state (#2861 salvage)#763
hashbender merged 1 commit into
mainfrom
mirror/pr-56945

Conversation

@hashbender

Copy link
Copy Markdown
Owner

Summary

Retained webhook delivery state no longer holds the full request payload — _delivery_info entries kept up to ~1MB of dead weight per delivery for the 1-hour idempotency TTL, and no consumer ever read it (all readers access only deliver / deliver_extra).

Salvage of NousResearch#2861 by @VolodymyrBg (stale branch, cherry-picked onto current main with authorship preserved).

Changes

  • gateway/platforms/webhook.py: drop "payload": payload from the retained agent-mode deliver_config (the request-scoped deliver-only dict is untouched)
  • tests/gateway/test_webhook_adapter.py: remove the matching fixture key
  • scripts/release.py: AUTHOR_MAP entry for @VolodymyrBg

Validation

Before After
tests/gateway/test_webhook_adapter.py all pass
payload retained per delivery up to ~1MB × 1h TTL none

Infographic

infographic

Nous Research


Mirror-of: NousResearch#56945
NousResearch#56945

@hashbender
hashbender merged commit 8642ead into main Jul 2, 2026
2 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.

1 participant