Preserve raw capture sources during reconciliation - #2860
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Published #2860 at f1f4586, based on a063a68. Preserves raw line endings across legacy disposition, backfill and read comparison without masking genuine content changes. Full source backend: 9,172 passed / 5 skipped; current-base combined API integration: 34 passed. Independent Terra review reaffirmed SHIP against current base; no blockers. Hosted CI remains pending, broader #1999 remains open. Completed worktree removed normally after pushing and preserving all five .tmp evidence files with matching hashes under .claude/local/milestone-20260909/preserved-1999-reconciliation/. No owned runtime remains. |
Linked transcript corrections retain raw CRLF or lone-CR text in source assets while the queue stores LF. Subsequent Keep/Archive, backfill, and read fallback previously treated those line endings as changed content, replacing or hiding the original source and leaving equivalent archived rows outstanding.
Compare canonical line endings at those reconciliation decisions while retaining the raw source asset, ID, and hash. Actual text changes still follow the existing repair or archived-mismatch path. This reduces unexpected source maintenance after capture without changing review or execution authority.
Refs #1999; addresses the source reconciliation follow-up recorded in comment5606596609. This does not close the broader editor issue or restore historical superseded assets.
Validation:
dotnet test backend/Taskdeck.sln -c Release -m:1passed, 9,172 passed / 5 skipped on source commit b1f506d. Current-main merge is rechecked with the capture and private-memory integration suites; exact result is recorded below.No canonical state docs changed: their coordinator lease remains elsewhere. Human actions remain in OUTSTANDING_TASKS.md. Historical source restoration and the remaining #1999 provenance/editor acceptance work remain separate.
Current-base verification on f1f4586 (base a063a68): 34 API integration tests passed across CaptureServiceTransactionIntegrationTests, ContextFabricCaptureBackfillTests, and PrivateMemorySourceApiTests. Fresh-context review reaffirmed SHIP against this base. The full backend was not repeated after the base merge; hosted required CI remains the merge gate.