Skip to content

Tape parity: limited-read comparison uses the snapshot window - #994

Merged
ReganBell merged 2 commits into
mainfrom
parity-limited-coverage-clamp
Sep 8, 2026
Merged

Tape parity: limited-read comparison uses the snapshot window#994
ReganBell merged 2 commits into
mainfrom
parity-limited-coverage-clamp

Conversation

@ReganBell

@ReganBell ReganBell commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

The full-corpus parity check compares a session's entries and tape from one snapshot, but the limited-read comparison calls the live transcript source, which re-reads the store — so a turn landing between the script's snapshot and the serving read makes the served view newer than the expected list, and the trailing rows count as real mismatches. On an active fleet this produces a steady band of false reals per run (reproducibly, since the settle-recheck races the same ongoing activity).

Fix: compare only the intersection window — served rows clamped to the snapshot's latest seq, expected entries clamped to the tape's covered seq. A race-faithful pinning test (snapshot, then a live turn, then the limited comparison with the stale snapshot) is red on the old code.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

ReganBell and others added 2 commits September 8, 2026 14:52
…shot's latest seq and expected to the tape's covered seq

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@ReganBell
ReganBell merged commit 323780c into main Sep 8, 2026
20 checks passed
@ReganBell
ReganBell deleted the parity-limited-coverage-clamp branch September 8, 2026 22:05
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.

1 participant