Store Compact Transcript in Checkpoints

Claude Code·Fable 5·computermode·3mo ago·2hr 38min·1 Checkpoint·6 file changes·+396/-29·20.8K tokens

Address PR review comments for https://github.com/entireio/cli/pull/1401

3mo ago·4m

Done — both review comments on PR #1401 are addressed, replied to, and resolved.

PR #1401 had two unresolved threads, both from Copilot's automated reviewer, and no human comments:

  1. e2e/entire/entire.go:68 — the RewindList doc comment said the helper runs entire rewind --list while the code runs entire checkpoint rewind --list. Updated the comment to match the implementation.
  2. e2e/entire/entire.go:120 — the runStdout failure message concatenated stdout and stderr with no separator. The two streams are now labeled stdout: / stderr: on separate lines.
ThreadReviewerLocationStatusReplyResolved
PRRT_...W7qDcopilote2e/entire/entire.go:68AddressedYesYes
PRRT_...W7qQcopilote2e/entire/entire.go:120AddressedYesYes

Process notes:

  • Your main checkout has uncommitted work on compact-transcript-support, so I did everything in a temporary git worktree on the PR branch and removed it afterward — your working tree is untouched.
  • Verification: gofmt, go vet -tags e2e ./e2e/entire/, and mise run lint all clean. Since the changes are a doc comment and a test-failure message, that's the proportional check; no test run was needed.
  • Pushed as commit fe71aa6d8 ("address Copilot review feedback on e2e rewind helpers") to deprecate-checkpoint-rewind; both threads reference that SHA in their replies.
afd9428Store compact transcript.jsonl in v1 checkpoints, point metadata at it Committed checkpoint writes now generate a compact transcript (transcript.jsonl) from the full transcript via transcript/compact and store it in the session directory next to full.jsonl, so it is pushed with the entire/checkpoints/v1 branch. The root metadata.json sessions[].transcript pointer targets transcript.jsonl when it was generated and falls back to full.jsonl otherwise (unparseable or external-agent transcripts, checkpoints from older CLI versions). Unlike the removed v2 (cumulative compact plus compact-unit offset), the stored compact is pre-sliced to the checkpoint's own portion: compact.Compact runs with StartLine = checkpoint_transcript_start. v1's checkpoint_transcript_start must keep full.jsonl units for CLI readers, so a pre-sliced file avoids a new offset field and is self-describing for consumers. Generation is best-effort and never fails the checkpoint write; finalization (UpdateCommitted) regenerates the compact from the new content and keeps the previous one on generation failure so the metadata pointer never dangles. CLI read paths (rewind/resume/explain) are unaffected: they read full.jsonl by filename, not through the metadata pointer. Entire-Checkpoint: 63b41777384a+396/-29