Skip to content

Honor Git helper cancellation tokens - #3845

Merged
Widthdom merged 3 commits into
mainfrom
fix-issue3723
Jun 20, 2026
Merged

Honor Git helper cancellation tokens#3845
Widthdom merged 3 commits into
mainfrom
fix-issue3723

Conversation

@Widthdom

@Widthdom Widthdom commented Jun 20, 2026

Copy link
Copy Markdown
Owner

Summary

  • Pass caller cancellation tokens through Git-backed freshness, status, repository metadata, and commit-resolution paths.
  • Preserve cancellation propagation through best-effort helpers and status database handling.
  • Add regression coverage and a bilingual changelog fragment for Require explicit cancellation tokens on Git helper execution paths #3723.
  • Stabilize the MCP timeout diagnostics regression test after status diagnostics started observing request cancellation.

Validation

  • dotnet build CodeIndex.sln --no-restore
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --framework net8.0 --filter "FullyQualifiedName~McpServerTests.ProcessFrameAsync_RequestTimeout_ReturnsStructuredTimeoutError"
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~RequestTimeout_ReturnsStructuredTimeoutError"
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --no-build
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet format CodeIndex.sln --verify-no-changes --no-restore
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll index . --commits HEAD --json
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json
  • Codex adversarial review: No blocking/actionable issues found.

Fixes #3723

@Widthdom
Widthdom marked this pull request as ready for review June 20, 2026 13:33
@Widthdom
Widthdom merged commit ecd301e into main Jun 20, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue3723 branch June 20, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Require explicit cancellation tokens on Git helper execution paths

1 participant