Fix Unexpected Execution Error when entity lookup returns null

[michaelstaib]

No description provided.

[Copilot]

Pull request overview

This PR addresses a Fusion execution edge case where an entity lookup returning null could surface an “Unexpected Execution Error”, by treating a clean null lookup result as a valid “not found” state and completing fields using normal GraphQL nullability semantics.

Changes:

• Update result completion to handle a null source without an associated error by completing each selected field (triggering standard non-null violations where applicable) rather than emitting a generic execution error.
• Add coverage for nested entity lookups that resolve to null, including both non-null and nullable parent-field scenarios, with new snapshots.
• (Unrelated / problematic) Add external worktree project references to src/All.slnx that appear to be machine-local paths.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
src/HotChocolate/Fusion/test/Fusion.AspNetCore.Tests/LookupTests.cs	Adds new lookup tests covering entity lookup returning null for nested entities.
src/HotChocolate/Fusion/test/Fusion.AspNetCore.Tests/snapshots/LookupTests.Lookup_Should_Return_Null_When_Nested_Entity_Not_Found.yaml	Snapshot for the non-null parent-field case (expects non-null violation instead of unexpected execution error).
src/HotChocolate/Fusion/test/Fusion.AspNetCore.Tests/snapshots/LookupTests.Lookup_Should_Null_Field_When_Nullable_Entity_Not_Found.yaml	Snapshot for the nullable parent-field case (field nulling + error semantics captured).
src/HotChocolate/Fusion/src/Fusion.Execution/Execution/Results/ValueCompletion.cs	Implements per-field completion behavior for clean null sources without errors.
src/All.slnx	Adds machine-local ../.claude/worktrees/... projects (should be removed).

---

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

[github-actions]

Patch coverage

30.7% of changed lines covered (31/101)

File	Covered	Changed	Patch %
…/Fusion/test/Fusion.AspNetCore.Tests/LookupTests.cs	0	68	0.0% 🔴
…/src/Fusion.Execution/Execution/Results/ValueCompletion.cs	31	33	93.9% 🟡

Uncovered changed lines (JSON)

{
  "sha": "552f55c9817ae7034fb337ce6caebc2f3083afa8",
  "files": [
    { "path": "src/HotChocolate/Fusion/test/Fusion.AspNetCore.Tests/LookupTests.cs", "ranges": [[147, 147], [149, 151], [153, 155], [157, 161], [164, 164], [166, 175], [177, 180], [183, 184], [188, 188], [190, 192], [194, 196], [198, 202], [205, 205], [207, 216], [218, 221], [224, 225], [366, 366], [369, 369], [374, 374], [380, 380], [387, 387], [402, 402], [405, 405], [410, 410], [416, 416], [423, 423]] },
    { "path": "src/HotChocolate/Fusion/src/Fusion.Execution/Execution/Results/ValueCompletion.cs", "ranges": [[350, 351]] }
  ]
}

---

Project coverage: 49.3% (217292/440360 lines)