Skip to content

[Fusion] Fix the operation planner dropping requirement chains - #10485

Merged
michaelstaib merged 13 commits into
mainfrom
mst/planner-issues
Oct 8, 2026
Merged

michaelstaib merged 13 commits into
mainfrom
mst/planner-issues

Conversation

@michaelstaib

Copy link
Copy Markdown
Member

No description provided.

Copilot AI balanced review requested due to automatic review settings October 6, 2026 02:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

github-actions Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Patch coverage

92.1% of changed lines covered (430/467)

File Covered Changed Patch %
…/Planning/OperationPlanner.BuildExecutionTree.cs 198 226 87.6% 🟡
…/Fusion/src/Fusion.Execution/Planning/OperationPlanner.cs 107 115 93.0% 🟡
…/Fusion/src/Fusion.Execution/Planning/PlanningBudget.cs 61 62 98.4% 🟡
…/Fusion/src/Fusion.Execution/Execution/ThrowHelper.cs 2 2 100.0% 🟢
…/src/Fusion.Execution/Planning/OperationPlanner.Defer.cs 36 36 100.0% 🟢
…/src/Fusion.Execution/Planning/Steps/OperationPlanStep.cs 26 26 100.0% 🟢
Uncovered changed lines (JSON)
{
  "sha": "8202bb8d77b5948127d5006f32d5c84f695e2ec7",
  "files": [
    { "path": "src/HotChocolate/Fusion/src/Fusion.Execution/Planning/OperationPlanner.BuildExecutionTree.cs", "ranges": [[147, 148], [345, 345], [436, 436], [455, 458], [467, 469], [483, 484], [486, 488], [490, 490], [494, 494], [623, 625], [628, 632], [634, 635]] },
    { "path": "src/HotChocolate/Fusion/src/Fusion.Execution/Planning/OperationPlanner.cs", "ranges": [[2952, 2958], [3192, 3192]] },
    { "path": "src/HotChocolate/Fusion/src/Fusion.Execution/Planning/PlanningBudget.cs", "ranges": [[56, 56]] }
  ]
}

Project coverage: 58.6% (314159/536364 lines)

@michaelstaib michaelstaib changed the title [Fusion] Fix planner issues [Fusion] Fix the operation planner dropping requirement chains Oct 8, 2026
@michaelstaib
michaelstaib merged commit 8fbb242 into main Oct 8, 2026
7 of 9 checks passed
@michaelstaib
michaelstaib deleted the mst/planner-issues branch October 8, 2026 03:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants