Repository navigation
feat: scoped drafts, event feed and conversion reporting (package 3) - #87
Draft
nikolajIvanov wants to merge 4 commits into
Draft
nikolajIvanov wants to merge 4 commits into
nikolajIvanov wants to merge 4 commits into
Conversation
|
@nikolajIvanov is attempting to deploy a commit to the diwenne's projects Team on Vercel. A member of the Team first needs to authorize it. |
This branch has not been deployed
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
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.
Package 3: scoped draft and event integrations (dependent draft)
Prerequisites: read-only API/MCP #84, creator workflow #85 and durable delivery #86.
This branch contains those prerequisite changes, including a cherry-pick of #84. The upstream diff is therefore cumulative, not a claim that all files belong to this package.
The package-only review is this comparison or the single feature commit.
Merge the prerequisites first, then rebase this draft; do not merge the shared changes twice.
Changes
drafts:write,events:read,conversions:write. Existing keys retain their scopes and exact expiration dates; new keys default to read-only.IntegrationEventmigration only in this package.Verification
Risks and release notes
Apply prerequisite and new migrations before starting web and worker. See
docs/integration-writes.md.The event feed is at-least-once: complete pagination, checkpoint nextSince, replay the 15-minute overlap and dedupe by eventId plus status. Transactions beyond the overlap need a wider replay; this is not an immutable audit of every intermediate status.
Integration events include the caller-supplied opaque conversion reference/value, so grant events:read only to trusted consumers. Delivery projections exclude recipient IDs, raw payloads and errors.
No OAuth/DCR server, automatic CRM push, auth/SMTP, Docker, worker or dependency changes in the package-specific commit. Existing upstream dependency findings are unchanged.
Maintainer CI/preview authorization and eventual merge are separate from this local verification.