[Fusion] Fix mutation execution backlog dropping batched follow-up operations - #9847
#9847[Fusion] Fix mutation execution backlog dropping batched follow-up operations#9847tobias-tengler merged 1 commit intomainChilliCream/graphql-platform:mainfrom tte/fix-operation-batch-after-mutationChilliCream/graphql-platform:tte/fix-operation-batch-after-mutationCopy head branch name to clipboard
Conversation
There was a problem hiding this comment.
Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.
Once credits are available, push a new commit or reopen this pull request to trigger a review.
There was a problem hiding this comment.
Pull request overview
This PR fixes Fusion mutation execution scheduling so that mutation follow-up operations (e.g., batched node lookups) are not dropped when the dependent execution nodes have only optional dependencies. It also adds a regression test verifying that follow-up lookups are executed and composed into the final mutation result.
Changes:
- Fix mutation backlog initialization to treat nodes with optional dependencies as non-root nodes (so they are enqueued properly).
- Add a regression test covering mutation root execution with batched node lookups and follow-up field hydration.
- Add a snapshot capturing the expected upstream interactions and operation plan.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
src/HotChocolate/Fusion/src/Fusion.Execution/Execution/ExecutionState.cs |
Ensures mutation nodes with optional dependencies are added to the backlog (prevents follow-up ops from being skipped). |
src/HotChocolate/Fusion/test/Fusion.AspNetCore.Tests/MutationTests.cs |
Adds a regression test for mutation execution with batched node lookups and follow-up fields. |
src/HotChocolate/Fusion/test/Fusion.AspNetCore.Tests/__snapshots__/MutationTests.Mutation_Root_With_Batched_Node_Lookups_Executes_Followups.yaml |
Adds the expected snapshot output and upstream interaction trace for the new test. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.