Skip to content

[Fusion] Remove locations from errors raised during subgraph execution#9729

Merged
tobias-tengler merged 1 commit into
mainfrom
tte/ignore-locations-for-execution-fusion-errors
May 18, 2026
Merged

[Fusion] Remove locations from errors raised during subgraph execution#9729
tobias-tengler merged 1 commit into
mainfrom
tte/ignore-locations-for-execution-fusion-errors

Conversation

@tobias-tengler

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings May 18, 2026 12:41

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.

Pull request overview

This PR updates Fusion’s result completion logic to stop attaching GraphQL document locations to errors produced while completing/propagating subgraph execution errors, ensuring those errors are reported with path (and other metadata) but without source locations.

Changes:

  • Stop adding a parent selection location when promoting pocketed errors.
  • Remove selection-based .AddLocation(...) calls from field error application and value/list completion error creation.
  • Keep error handling behavior (path setting, null propagation, error handler invocation) unchanged aside from locations.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tobias-tengler tobias-tengler merged commit 0401891 into main May 18, 2026
143 of 145 checks passed
@tobias-tengler tobias-tengler deleted the tte/ignore-locations-for-execution-fusion-errors branch May 18, 2026 13:22
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