checkpoint: collapse persistent reads to generic Read/List

698e477→main·

Soph·3w ago·30 files·+100 added/-100 removed

Rename the committed read methods to the generic surface differentiated by return type: (PersistentStore).ReadCommitted -> Read (returns *CheckpointSummary) (PersistentStore).ListCommitted -> List ReadCommittedCheckpoint (helper) -> ReadCheckpoint

Sets up sto.Persistent.Read(...) once the facade is reshaped. No behavior change.

Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com

Sessions

88d8a34b0471View transcript

Changes

30

• cmd/entire/cli

• Mattach.go+2/-2

• Mattach_test.go+9/-9

• Mattribution.go+2/-2

• Mattribution_test.go+1/-1

• checkpoint

• Mbackwards_compat_test.go+2/-2

• Mcheckpoint_test.go+34/-34

• Mcommitted.go+6/-6

• Mcommitted_phantom_paths_test.go+2/-2

• Mcommitted_reader_resolve.go+6/-6

• Mcommitted_reader_resolve_test.go+3/-3

• Mcommitted_update_test.go+4/-4

• Mcommitted_write_test.go+3/-3

• Mconfig_test.go+1/-1

• dispatch

• Mmode_local.go+2/-2

• Mexplain.go+3/-3

• Mexplain_export.go+2/-2

• Mexplain_export_test.go+1/-1

• Mexplain_test.go+3/-3

• Mhead_checkpoint_flags.go+1/-1

• Mresume.go+1/-1

• Mresume_test.go+1/-1

• Mreview_context.go+1/-1

• Mreview_context_test.go+1/-1

• strategy

• Mcleanup.go+1/-1

• Mcommon.go+1/-1

• Mmanual_commit_condensation.go+2/-2

• Mmanual_commit_hooks.go+1/-1

• Mmanual_commit_rewind.go+1/-1

• Mmanual_commit_test.go+1/-1

• Mphase_postcommit_test.go+2/-2

Mcmd/entire/cli/attach.go+2/-2

Mcmd/entire/cli/attach_test.go+9/-9

Mcmd/entire/cli/attribution.go+2/-2

Mcmd/entire/cli/attribution_test.go+1/-1

Mcmd/entire/cli/checkpoint/backwards_compat_test.go+2/-2

Mcmd/entire/cli/checkpoint/checkpoint_test.go+34/-34

Mcmd/entire/cli/checkpoint/committed.go+6/-6

Mcmd/entire/cli/checkpoint/committed_phantom_paths_test.go+2/-2

Mcmd/entire/cli/checkpoint/committed_reader_resolve.go+6/-6

Mcmd/entire/cli/checkpoint/committed_reader_resolve_test.go+3/-3

Mcmd/entire/cli/checkpoint/committed_update_test.go+4/-4

Mcmd/entire/cli/config_test.go+1/-1

Mcmd/entire/cli/dispatch/mode_local.go+2/-2

Mcmd/entire/cli/explain.go+3/-3

Mcmd/entire/cli/explain_export.go+2/-2

Mcmd/entire/cli/explain_export_test.go+1/-1

Mcmd/entire/cli/explain_test.go+3/-3

Mcmd/entire/cli/head_checkpoint_flags.go+1/-1

Mcmd/entire/cli/resume.go+1/-1

Mcmd/entire/cli/resume_test.go+1/-1

Mcmd/entire/cli/review_context.go+1/-1

Mcmd/entire/cli/review_context_test.go+1/-1

Mcmd/entire/cli/strategy/cleanup.go+1/-1

Mcmd/entire/cli/strategy/common.go+1/-1

Mcmd/entire/cli/strategy/manual_commit_condensation.go+2/-2