Skip to content

Add notification digest preferences to contract#82

Merged
haasonsaas merged 1 commit intomainfrom
codex/notifications-digest-preferences
Apr 15, 2026
Merged

Add notification digest preferences to contract#82
haasonsaas merged 1 commit intomainfrom
codex/notifications-digest-preferences

Conversation

@haasonsaas
Copy link
Copy Markdown
Contributor

Summary

  • add digest preferences to NotificationPreferences
  • add queued delivery status for batched notifications
  • regenerate Go, Python, and TypeScript notifications bindings

Testing

  • buf lint
  • go test ./... -count=1
  • npm ci
  • npm run check:package

@cursor
Copy link
Copy Markdown

cursor bot commented Apr 15, 2026

PR Summary

Medium Risk
Updates the notifications protobuf contract by adding a new preferences message/field and a new delivery status enum value, which can break or require updates in downstream clients/servers that switch on enums or expect stable message shapes.

Overview
Expands the notifications API contract to support digest/batching configuration and a new delivery lifecycle state.

NotificationPreferences now includes repeated digest_preferences entries (new DigestPreference message with category/channel, window, and enabled flag), and DeliveryStatus adds DELIVERY_STATUS_QUEUED for batched notifications. Regenerates the Go, Python, and TypeScript generated bindings to reflect these schema changes.

Reviewed by Cursor Bugbot for commit 9f0c1e7. Bugbot is set up for automated code reviews on this repo. Configure here.

@haasonsaas haasonsaas merged commit 617a72d into main Apr 15, 2026
6 checks passed
@haasonsaas haasonsaas deleted the codex/notifications-digest-preferences branch April 15, 2026 16:38
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