Skip to content

Fix backfill fold dry-run and recovery signals - #2733

Merged
Widthdom merged 7 commits into
mainfrom
codex/fix-issue1613-1703-1919
May 31, 2026
Merged

Fix backfill fold dry-run and recovery signals#2733
Widthdom merged 7 commits into
mainfrom
codex/fix-issue1613-1703-1919

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Add backfill-fold --dry-run and MCP backfill_fold dry_run / force support.
  • Make backfill responses distinguish preview, already-complete, effective FoldReady before/after, and fold metadata before/after.
  • Commit folded-row rewrites and FoldReady metadata in one transaction so crash recovery cannot leave current metadata with stale rows.

Validation

  • dotnet build CodeIndex.sln
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~IndexCommandRunnerTests.RunBackfillFold|FullyQualifiedName~McpServerTests.ToolsCall_BackfillFold|FullyQualifiedName~CliFlagSchema"
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll index . --commits f79495ace --json
  • Codex adversarial review: No blocking/actionable issues found.

Full dotnet test CodeIndex.sln was attempted, but this local environment's Git commit signing key prompts for a passphrase and causes unrelated git-backed tests to fail. Tracked as #2729.

Documentation and Changelog

  • Updated DEVELOPER_GUIDE.md.
  • Added changelog.d/unreleased/1613.fixed.md.

Issues

Fixes #1613
Fixes #1703
Fixes #1919

References #2729

@Widthdom
Widthdom merged commit 465e72b into main May 31, 2026
10 checks passed
@Widthdom
Widthdom deleted the codex/fix-issue1613-1703-1919 branch May 31, 2026 09:07
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