gabriel / muse public
patch dev
AI Agent gabriel · 108 days ago · May 21, 2026 · Diff

audit(issue-12): fix remaining stale msgpack refs and false-positive tests

- store.py read_snapshot docstring: 'msgpack structure' → 'JSON structure' - test_integrity_I10_bit_flip.py: update module docstring, section comments, and dummy path args (fake.msgpack → fake.json, snap.msgpack → snap.json) - test_integrity_I8_object_store_scale.py: rewrite TestPlanFileChecklistRegression to use read_commit/read_snapshot store API instead of old msgpack commits_dir paths (were vacuously skipping at first commit since new commits live in the object store, not .muse/commits/<id>.msgpack) - test_commit_record_committed_at_corruption.py: remove TestWriteCommitRepairsCorrpuption and TestStressRepair (dead — they wrote corrupt files to legacy commits_dir path which write_commit/read_commit never touch); remove _write_corrupt_msgpack and _corrupt_commit_file helpers; rename TestFromMsgpackCommittedAt → TestFromDictCommittedAt; drop unused _read_msgpack_dict import; fix /fake/path.msgpack → /fake/path.json

sha256:51aacf6b8943429f9256543e6a14cdab46b47956cb8e031f6c916273c4eff049 sha
+5 ~23 −17 symbols
sha256:b3bf892fa2388e81d8ca03286e29b2f53babb2916681de422f0ea75b7247aee3 snapshot
+5
symbols added
~23
symbols modified
−17
symbols removed
0
dead code introduced
Semantic Changes 45 symbols
~ muse/core/store.py .py 1 symbol modified
~ tests/test_commit_record_committed_at_corruption.py .py 3 symbols added, 17 symbols removed, 7 symbols modified
TestFromMsgpackCommittedAt class class TestFromMsgpackCommittedAt L139–254
test_empty_string_committed_at_raises method method test_empty_string_committed_at_raises L142–161
TestStressRepair class class TestStressRepair L469–516
test_100_writes_after_corruption_all_readable method method test_100_writes_after_corruption_all_readable L471–484
test_multiple_corrupt_commits_all_repaired method method test_multiple_corrupt_commits_all_repaired L486–516
TestWriteCommitRepairsCorrpuption class class TestWriteCommitRepairsCorrpuption L262–392
test_commit_hash_verification_passes_after_repair method method test_commit_hash_verification_passes_after_repair L322–332
test_commit_not_permanently_lost_after_one_write_commit method method test_commit_not_permanently_lost_after_one_write_commit L334–365
test_committed_at_corruption_is_the_repair_gap_this_fix_closes method method test_committed_at_corruption_is_the_repair_gap_this_fix_closes L367–392
test_read_commit_returns_good_data_after_repair method method test_read_commit_returns_good_data_after_repair L306–320
test_write_commit_overwrites_corrupt_committed_at method method test_write_commit_overwrites_corrupt_committed_at L265–291
test_write_commit_overwrites_garbage_committed_at method method test_write_commit_overwrites_garbage_committed_at L293–304
_corrupt_commit_file function function _corrupt_commit_file L127–131
_write_corrupt_msgpack function function _write_corrupt_msgpack L103–124
commits_dir import import commits_dir L57–57
msgpack import import msgpack L54–54
tempfile import import tempfile L52–52
+ TestFromDictCommittedAt class class TestFromDictCommittedAt L85–199
+ test_empty_string_committed_at_raises method method test_empty_string_committed_at_raises L88–106
+ muse_dir import import muse_dir L33–33
~ tests/test_integrity_I8_object_store_scale.py .py 2 symbols added, 2 symbols modified
+ read_commit import import read_commit L53–53
+ read_snapshot import import read_snapshot L53–53
← Older Oldest on dev
All commits
Newer → Latest on dev

0 comments

No comments yet. Be the first to start the discussion.

To add a comment, use the Muse CLI: muse hub commit comment sha256:51aacf6b8943429f9256543e6a14cdab46b47956cb8e031f6c916273c4eff049 --body "your comment"