Skip to content

Demote createSourceEventStream to helper taking ValidatedExecutionArgs#4700

Merged
yaacovCR merged 1 commit intographql:17.x.xfrom
yaacovCR:demote-create-source-event-stream-to-helper
May 1, 2026
Merged

Demote createSourceEventStream to helper taking ValidatedExecutionArgs#4700
yaacovCR merged 1 commit intographql:17.x.xfrom
yaacovCR:demote-create-source-event-stream-to-helper

Conversation

@yaacovCR
Copy link
Copy Markdown
Contributor

@yaacovCR yaacovCR commented May 1, 2026

This PR demote rarely used createSourceEventStream() from a request-level API to a lower-level helper for previously validated execution args. subscribe() remains the request-level API and now validates execution args before delegating to createSourceEventStream().

BREAKING CHANGE: createSourceEventStream() no longer accepts ExecutionArgs. Call validateExecutionArgs() first and pass its result, or use subscribe() for the full subscription pipeline.

This PR demote rarely used createSourceEventStream() from a request-level API to a lower-level helper for previously validated execution args. subscribe() remains the request-level API and now validates execution args before delegating to createSourceEventStream().

BREAKING CHANGE: createSourceEventStream() no longer accepts ExecutionArgs. Call validateExecutionArgs() first and pass its result, or use subscribe() for the full subscription pipeline.
@yaacovCR yaacovCR added the PR: breaking change 💥 implementation requires increase of "major" version number label May 1, 2026
@vercel
Copy link
Copy Markdown

vercel Bot commented May 1, 2026

@yaacovCR is attempting to deploy a commit to the The GraphQL Foundation Team on Vercel.

A member of the Team first needs to authorize it.

@yaacovCR yaacovCR merged commit 3fe3b20 into graphql:17.x.x May 1, 2026
22 of 23 checks passed
@yaacovCR yaacovCR deleted the demote-create-source-event-stream-to-helper branch May 1, 2026 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: breaking change 💥 implementation requires increase of "major" version number

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant