Skip to content

fix: DeferStreamDirectiveOnValidOperations fragment tracking#4762

Merged
yaacovCR merged 3 commits into
17.x.xfrom
robrichard/defer-subscription-rule
Jun 2, 2026
Merged

fix: DeferStreamDirectiveOnValidOperations fragment tracking#4762
yaacovCR merged 3 commits into
17.x.xfrom
robrichard/defer-subscription-rule

Conversation

@robrichard

@robrichard robrichard commented May 21, 2026

Copy link
Copy Markdown
Contributor

Tracks all fragment spreads back to root usage

also properly handles skip/include on fragments

@vercel

vercel Bot commented May 21, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
graphql-js Ignored Ignored Preview Jun 2, 2026 7:49am

Request Review

Tracks all fragment spreads back to root usage
@yaacovCR yaacovCR changed the base branch from next to 17.x.x June 2, 2026 07:08
@yaacovCR yaacovCR force-pushed the robrichard/defer-subscription-rule branch from 60373ac to b342805 Compare June 2, 2026 07:09
@yaacovCR yaacovCR added the PR: bug fix 🐞 requires increase of "patch" version number label Jun 2, 2026
@yaacovCR yaacovCR changed the title Rewrite defer/stream on valid operations rule fix: DeferStreamDirectiveOnValidOperations must track all fragments back to root Jun 2, 2026
@yaacovCR yaacovCR changed the title fix: DeferStreamDirectiveOnValidOperations must track all fragments back to root fix: DeferStreamDirectiveOnValidOperations to track fragments Jun 2, 2026
@yaacovCR yaacovCR changed the title fix: DeferStreamDirectiveOnValidOperations to track fragments fix: DeferStreamDirectiveOnValidOperations fragment tracking Jun 2, 2026
@yaacovCR yaacovCR merged commit c3e5513 into 17.x.x Jun 2, 2026
44 checks passed
@yaacovCR yaacovCR deleted the robrichard/defer-subscription-rule branch June 2, 2026 07:55
yaacovCR added a commit that referenced this pull request Jun 2, 2026
## v17.0.0-rc.0 (2026-06-02)

#### New Feature 🚀
* [#4670](#4670) feat(v17): Implement tracing channels ([@logaretm](https://github.com/logaretm))

#### Bug Fix 🐞
* [#4761](#4761) fix: DeferStreamDirectiveOnRootField forbids fragments on abstract types ([@robrichard](https://github.com/robrichard))
* [#4762](#4762) fix: DeferStreamDirectiveOnValidOperations fragment tracking ([@robrichard](https://github.com/robrichard))
* [#4784](#4784) fix: do not copy over stack from error cause ([@Malien](https://github.com/Malien))

#### Docs 📝
* [#4781](#4781) docs: improve deprecation information regarding originalError and cause ([@yaacovCR](https://github.com/yaacovCR))

#### Committers: 4
* Abdelrahman Awad([@logaretm](https://github.com/logaretm))
* Rob Richard([@robrichard](https://github.com/robrichard))
* Yaacov Rydzinski ([@yaacovCR](https://github.com/yaacovCR))
* Yaroslav Petryk([@Malien](https://github.com/Malien))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: bug fix 🐞 requires increase of "patch" version number

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants