Skip to content

Fix graceful Ctrl-C cancellation for index - #2308

Merged
Widthdom merged 3 commits into
mainfrom
fix-issue1809
May 17, 2026
Merged

Fix graceful Ctrl-C cancellation for index#2308
Widthdom merged 3 commits into
mainfrom
fix-issue1809

Conversation

@Widthdom

@Widthdom Widthdom commented May 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • Wire index runs to a Ctrl-C/SIGINT cancellation source so the first Ctrl-C requests cooperative cancellation and the second can force-exit.
  • Return E012_INTERRUPTED / exit 130 with a partial-progress message instead of stamping success readiness after interruption.
  • Add regression coverage for the handler and interrupted index runner path.

Fixes #1809

Validation

  • dotnet build
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~IndexCommandRunnerTests"
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet test

Documentation / Changelog

  • Added changelog.d/unreleased/1809.fixed.md.

Review

  • Adversarial review attempted with codex exec, but it failed due account usage limit.
  • Manual adversarial pass found an optimize/readiness stamping hole and fixed it; second pass found no further blocking/actionable issues.

Follow-up

@Widthdom
Widthdom merged commit 0bf05a0 into main May 17, 2026
7 checks passed
@Widthdom
Widthdom deleted the fix-issue1809 branch May 17, 2026 06:56
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.

Wire Ctrl-C handler to in-flight index CancellationTokenSource

1 participant