Skip to content

[codex] Fix index watch failure handling - #2956

Merged
Widthdom merged 4 commits into
mainfrom
fix-issue2903-2902
Jun 2, 2026
Merged

[codex] Fix index watch failure handling#2956
Widthdom merged 4 commits into
mainfrom
fix-issue2903-2902

Conversation

@Widthdom

@Widthdom Widthdom commented Jun 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • Propagate index --watch sub-run failures by emitting failed watch batch events with exit_code, adding exit codes to human summaries, and returning the latest non-zero sub-run exit code when watch mode stops.
  • Cap pending watch path batches and collapse excessive unique paths into a full incremental rescan instead of retaining an unbounded path set.

Documentation / Changelog

  • Updated USER_GUIDE.md watch-mode behavior in English and Japanese.
  • Added changelog fragments:
    • changelog.d/unreleased/2902.fixed.md
    • changelog.d/unreleased/2903.fixed.md

Validation

  • dotnet build
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~IndexWatchRunnerTests"
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -f net9.0 --filter "FullyQualifiedName~InvokeSubRunAndEmit"
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json
  • Codex adversarial review: No blocking/actionable issues found.

Full-suite dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --no-restore was attempted; unrelated ambient workspace failures were filed as follow-up #2951.

Issues

Fixes #2902
Fixes #2903

Follow-up: #2951

@Widthdom
Widthdom marked this pull request as ready for review June 2, 2026 18:26
@Widthdom
Widthdom merged commit eb8dbf5 into main Jun 2, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue2903-2902 branch June 2, 2026 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant