fix patch task/utility-sweep-continued #5 / 5

AI Agent gabriel · 155 days ago · Apr 20, 2026 · Diff

fix: utility sweep — split_id, fake_id, layout constants, shelf/rev-parse test correctness

- muse/core/_types.py: add OBJECTS_DIR, DEFAULT_ALGO layout constants - muse/core/object_store.py: import from _types; add objects_algo_dir() - tests/test_gc_full.py: split_id() for hex extraction; fix sha256 subdir layout - tests/test_cmd_shard.py: --json flag, schema key, fake_id, split_id, objects_algo_dir - tests/test_security_symlink.py: blob_id(), tag_path(), objects_algo_dir, split_id - tests/test_cmd_verify_tag.py: fake_id, tag_path() for canonical layout - tests/test_verify_tag_supercharge.py: same fake_id + tag_path fixes - tests/test_core_doc_history.py: fake_id, long_id, tag_path pattern - tests/test_object_store_algo_layout.py: split_id/DEFAULT_ALGO for path assertions - tests/test_cmd_rev_parse.py: split_id; remove non-existent --format flag tests - tests/test_cmd_shelf.py: shelf list --json returns envelope; unwrap ["entries"] - docs/utility-reference.md: document OBJECTS_DIR, DEFAULT_ALGO, objects_algo_dir

sha256:b615cff8d9015afbf10750b39fda0229b13e69bbf9bf8567699a08485bd17e14 sha

sha256:47319766550ec1688724e840a1e71a8f9a7a3ec469a018c0c4c6affa8342b794 parent

+71 ~252 −13 symbols

sha256:658094b70748468fced0e141176233b4a80bbaa8ba384ed0678605ac528f1152 snapshot

+71

symbols added

~252

symbols modified

−13

symbols removed

dead code introduced

Semantic Changes 336 symbols

~ docs/utility-reference.md .md 8 symbols added, 4 symbols removed, 23 symbols modified

~ table@L166

~ table@L209

~ table@L227

~ table@L242

~ table@L265

~ table@L283

~ table@L294

~ table@L308

~ table@L340

~ table@L352

~ table@L62

~ code@L85

~ table@L41

~ table@L50

− table@L115 section table@L115 L115–154

− table@L184 section table@L184 L184–200

− object_store — Object path section muse.core.object_store — Object path L31–39

− table@L33 section table@L33 L33–36

+ table@L127 section table@L127 L127–153

+ Pattern 10 — Manual cryptographic prefix stripping / construction section Pattern 10 — Manual cryptographic prefix stripping / construction L361–368

+ table@L365 section table@L365 L365–368

+ table@L183 section table@L183 L183–196

+ _types — Object store layout constants variable section muse.core._types — Object store layout constants L33–42

+ table@L35 section table@L35 L35–39

+ object_store — Object path utilities section muse.core.object_store — Object path utilities L42–51

+ table@L44 section table@L44 L44–48

~ Muse Utility Reference & Anti-Pattern Inventory

~ Anti-Pattern Inventory

~ Pattern 1 — Manual sha256: prefix construction

~ Pattern 2 — Manual prefix stripping outside the storage boundary

~ Test code

~ Pattern 9 — Manual object path construction

~ Utility Functions

~ _types

~ table@L14

~ muse/cli/commands/midi_compare.py .py 1 symbol added, 1 symbol modified

+ short_id import import short_id L39–39

~ _load

~ muse/cli/commands/midi_harmony.py .py 1 symbol added, 1 symbol modified

+ short_id import import short_id L42–42

~ run

~ muse/cli/commands/motif_detect.py .py 1 symbol added, 1 symbol modified

+ short_id import import short_id L33–33

~ run

~ muse/cli/commands/note_blame.py .py 1 symbol added, 1 symbol modified

+ short_id import import short_id L34–34

~ run

~ muse/cli/commands/note_log.py .py 1 symbol added, 1 symbol modified

+ short_id import import short_id L37–37

~ run

~ muse/cli/commands/notes.py .py 1 symbol added, 1 symbol modified

+ short_id import import short_id L42–42

~ run

~ muse/cli/commands/piano_roll.py .py 1 symbol added, 1 symbol modified

+ short_id import import short_id L39–39

~ run

~ muse/cli/commands/tempo.py .py 1 symbol added, 1 symbol modified

+ short_id import import short_id L32–32

~ run

~ muse/cli/commands/voice_leading.py .py 1 symbol added, 1 symbol modified

+ short_id import import short_id L33–33

~ run

~ muse/core/_types.py .py 2 symbols added

+ DEFAULT_ALGO variable variable DEFAULT_ALGO L321–321

+ OBJECTS_DIR variable variable OBJECTS_DIR L320–320

~ muse/core/object_store.py .py 3 symbols added, 7 symbols modified

+ DEFAULT_ALGO import import DEFAULT_ALGO L60–60

+ OBJECTS_DIR import import OBJECTS_DIR L60–60

+ objects_algo_dir function function objects_algo_dir L308–321

~ _DEFAULT_ALGO

~ _OBJECTS_DIR

~ blob_id

~ hash_file

~ long_id

~ short_id

~ split_id

~ muse/plugins/midi/_midi_query.py .py 1 symbol added, 1 symbol modified

+ short_id import import short_id L69–69

~ run_query

~ muse/plugins/midi/_query.py .py 1 symbol added, 2 symbols modified

+ short_id import import short_id L20–20

~ Manifest

~ load_track

~ muse/plugins/midi/entity.py .py 1 symbol added, 2 symbols modified

+ short_id import import short_id L60–60

~ load_json_file

~ write_entity_index

~ tests/test_cli_new_commands.py .py 1 symbol added, 3 symbols modified

+ split_id import import split_id L13–13

~ TestGcCli

~ test_gc_verbose_lists_objects

~ blob_id

~ tests/test_cmd_archive.py .py 1 symbol added, 2 symbols modified

+ fake_id import import fake_id L24–24

~ _init_repo

~ blob_id

~ tests/test_cmd_blame.py .py 1 symbol added, 3 symbols modified

+ fake_id import import fake_id L56–56

~ _make_repo

~ Manifest

~ blob_id

~ tests/test_cmd_checkout_symbol.py .py 1 symbol added, 2 symbols modified

+ split_id import import split_id L57–57

~ TestCheckoutSymbolJSONSchema

~ test_json_restored_from_is_short_id

~ tests/test_cmd_cherry_pick.py .py 1 symbol added, 2 symbols modified

+ fake_id import import fake_id L19–19

~ _init_repo

~ long_id

~ tests/test_cmd_clones.py .py 1 symbol added, 2 symbols modified

+ split_id import import split_id L54–54

~ TestClonesJSONSchema

~ test_json_commit_is_short_id

~ tests/test_cmd_codemap.py .py 1 symbol added, 2 symbols modified

+ split_id import import split_id L46–46

~ TestCodemapJSONSchema

~ test_json_commit_is_short_id

~ tests/test_cmd_forecast.py .py 1 symbol added, 2 symbols modified

+ fake_id import import fake_id L83–83

~ _make_repo

~ MsgpackDict

~ tests/test_cmd_format_patch.py .py 1 symbol added, 3 symbols modified

+ split_id import import split_id L24–24

~ TestPatchIdStability

~ test_different_commits_different_patch_ids

~ long_id

~ tests/test_cmd_hash_object.py .py 1 symbol added, 4 symbols modified

+ split_id import import split_id L19–19

~ TestHashBytes

~ test_returns_canonical_prefixed_id

~ blob_id

~ long_id

~ tests/test_cmd_ls_files.py .py 1 symbol added, 5 symbols modified

+ split_id import import split_id L21–21

~ TestJsonFormat

~ test_object_ids_are_sha256_prefixed

~ Manifest

~ blob_id

~ long_id

~ tests/test_cmd_reflog_hardening.py .py 1 symbol modified

~ _make_repo

~ tests/test_cmd_release_hardening.py .py 3 symbols modified

~ TestReleasePushExtended

~ test_push_success_json_schema

~ test_push_success_text_output

~ tests/test_cmd_rev_list.py .py 1 symbol added, 1 symbol modified

+ split_id import import split_id L20–20

~ test_rev_list_emits_one_id_per_line

~ tests/test_cmd_rev_parse.py .py 6 symbols added, 5 symbols removed, 20 symbols modified

− test_default_output_is_json method method test_default_output_is_json L112–119

− test_format_text_explicit method method test_format_text_explicit L139–143

− test_text_format_gives_plain_commit_id method method test_text_format_gives_plain_commit_id L121–128

− test_invalid_format_errors method method test_invalid_format_errors L345–348

− test_invalid_format_emits_json_to_stdout method method test_invalid_format_emits_json_to_stdout L459–463

+ test_default_output_is_text method method test_default_output_is_text L112–117

+ test_no_flags_output_is_not_json method method test_no_flags_output_is_not_json L119–125

+ test_text_vs_json_differ method method test_text_vs_json_differ L136–142

+ test_unrecognized_flag_errors method method test_unrecognized_flag_errors L344–347

+ test_not_found_emits_json_to_stdout method method test_not_found_emits_json_to_stdout L458–463

+ split_id import import split_id L21–21

~ TestAbbrevRef

~ test_abbrev_ref_text_format

~ TestBranchResolution

~ test_resolve_branch_text

~ TestDefaultFormat

~ TestEdgeCases

~ TestErrorJson

~ test_ambiguous_prefix_has_schema

~ test_bare_hex_has_schema

~ TestJsonSchema

~ test_prefix_resolution_has_schema

~ TestSha256PrefixRequired

~ test_bare_full_hex_rejected

~ test_bare_short_hex_rejected

~ test_canonical_prefix_resolves

~ TestShaResolution

~ test_ambiguous_prefix_returns_candidates

~ test_resolve_8char_prefix

~ Manifest

~ long_id

~ tests/test_cmd_revert_hardening.py .py 2 symbols modified

~ TestHeadRef

~ test_short_id_resolves

~ tests/test_cmd_shard.py .py 3 symbols added, 2 symbols removed, 6 symbols modified

− test_json_shorthand_same_as_format_json method method test_json_shorthand_same_as_format_json L293–313

− uuid import import uuid L69–69

+ test_json_shorthand_same_as_json_long method method test_json_shorthand_same_as_json_long L292–312

+ fake_id import import fake_id L72–72

+ short_id import import short_id L72–72

~ TestShardIntegration

~ test_format_json_produces_valid_json

~ TestShardJsonSchema

~ test_all_schema_fields_present

~ test_commit_short_is_8_chars

~ repo

~ tests/test_cmd_shelf.py .py 1 symbol added, 28 symbols modified

+ split_id import import split_id L43–43

~ TestApplyJsonSchema

~ test_apply_preserves_shelf_entry

~ TestComputeShelfId

~ test_id_is_64_hex_after_prefix

~ TestDropJsonSchema

~ test_drop_removes_entry_from_list

~ TestListJsonSchema

~ test_empty_returns_empty_array

~ test_files_count_positive

~ test_filter_branch

~ test_filter_branch_no_match_empty

~ test_filter_by_creator

~ test_filter_resumable

~ test_schema_complete

~ TestPopJsonSchema

~ test_pop_removes_entry

~ TestReadJsonSchema

~ test_read_by_name

~ TestRoundTrips

~ test_shelf_persists_across_commands

~ test_stack_ordering_newest_first

~ TestSecurity

~ test_invalid_format_exits_1

~ TestStress

~ test_100_save_drop_cycles

~ test_stack_with_50_entries_then_clear

~ fake_id

~ long_id

~ tests/test_cmd_verify_commit.py .py 1 symbol added, 5 symbols modified

+ split_id import import split_id L53–53

~ TestResolveRef

~ test_bare_hex_normalised_to_sha256_prefix

~ blob_id

~ decode_sig

~ encode_sig

~ tests/test_cmd_verify_tag.py .py 2 symbols added, 15 symbols modified

+ fake_id import import fake_id L39–39

+ tag_path import import tag_path L30–30

~ _REPO_ID

~ _force_write_tag

~ _init_repo

~ _make_tag

~ CommitRecord

~ Manifest

~ SnapshotRecord

~ TagRecord

~ blob_id

~ decode_sig

~ encode_sig

~ sig_algo

~ write_commit

~ write_snapshot

~ write_tag

~ tests/test_cmd_worktree.py .py 1 symbol modified

~ _init_repo

~ tests/test_core_doc_history.py .py 3 symbols added, 21 symbols modified

+ _REPO_ID variable variable _REPO_ID L42–42

+ fake_id import import fake_id L40–40

+ long_id import import long_id L40–40

~ TestBuildCommitToVersionMap

~ test_deterministic_with_multiple_tags

~ test_empty

~ test_single_tag

~ TestGenerateChangelog

~ test_added_symbol

~ test_breaking_symbol

~ test_empty_range

~ test_sorted_output

~ test_unresolvable_to_ref

~ TestGetSymbolVersionEvents

~ test_address_not_in_index

~ test_empty_index

~ test_event_with_breaking_commit

~ test_event_with_commit_and_tag

~ test_multiple_events_ordered

~ test_single_entry_no_commit

~ _make_repo

~ _write_commit

~ _write_tag

~ Manifest

~ tests/test_gc_full.py .py 1 symbol added, 23 symbols modified

+ split_id import import split_id L82–82

~ TestCollectReachableCommits

~ test_commit_only_in_orphaned_file_not_reachable

~ test_corrupt_commit_file_skipped_gracefully

~ test_diamond_dag_no_duplicate_walk

~ test_merge_commit_both_parents_reachable

~ test_multiple_branches_union

~ test_nested_branch_ref_traversed

~ test_single_branch_single_commit

~ test_traverses_parent_chain

~ TestCollectReachableSnapshots

~ test_missing_snapshot_file_skipped

~ test_multiple_commits_same_snapshot_deduplicated

~ test_returns_blob_ids_from_manifest

~ test_returns_snapshot_ids_from_reachable_commits

~ TestListStoredMsgpack

~ test_grace_period_protects_recent_files

~ test_grace_period_zero_includes_all

~ test_returns_msgpack_files

~ _write_commit_record

~ Manifest

~ blob_id

~ fake_id

~ long_id

~ tests/test_integrity_I3_concurrent_race.py .py 1 symbol added, 2 symbols modified

+ split_id import import split_id L41–41

~ _is_valid_cid

~ fake_id

~ tests/test_integrity_I8_object_store_scale.py .py 1 symbol added, 6 symbols modified

+ split_id import import split_id L51–51

~ TestShardScaleSmoke

~ test_object_file_name_is_correct_remainder

~ Manifest

~ blob_id

~ fake_id

~ long_id

~ tests/test_object_store_algo_layout.py .py 2 symbols added, 4 symbols modified

+ DEFAULT_ALGO import import DEFAULT_ALGO L18–18

+ split_id import import split_id L18–18

~ TestObjectPathAlgoDirectory

~ test_filename_is_remaining_hex

~ test_four_char_prefix_still_nested_under_sha256

~ blob_id

~ tests/test_perf_extreme_code_porcelain.py .py 1 symbol added, 1 symbol removed, 1 symbol modified

− uuid import import uuid L29–29

+ fake_id import import fake_id L31–31

~ large_repo

~ tests/test_perf_phase3.py .py 1 symbol added, 3 symbols modified

+ split_id import import split_id L74–74

~ TestConcurrentWriteBranchRef

~ test_100_threads_write_branch_ref_last_write_wins

~ blob_id

~ tests/test_property_merge_invariants.py .py 1 symbol modified

~ _linear_repo

~ tests/test_read_commit_supercharge.py .py 1 symbol added, 4 symbols modified

+ split_id import import split_id L26–26

~ TestTextFormatShortId

~ test_text_short_id_has_12_hex_chars

~ fake_id

~ long_id

~ tests/test_read_snapshot_supercharge.py .py 1 symbol added, 4 symbols modified

+ split_id import import split_id L24–24

~ TestTextFormatShortId

~ test_text_short_id_has_12_hex_chars

~ fake_id

~ long_id

~ tests/test_release_analysis.py .py 1 symbol added, 1 symbol removed, 3 symbols modified

− uuid import import uuid L6–6

+ fake_id import import fake_id L23–23

~ _make_release

~ Manifest

~ repo

~ tests/test_security_symlink.py .py 5 symbols added, 11 symbols modified

+ DEFAULT_ALGO import import DEFAULT_ALGO L34–34

+ blob_id import import blob_id L34–34

+ fake_id import import fake_id L34–34

+ objects_algo_dir import import objects_algo_dir L35–35

+ split_id import import split_id L34–34

~ TestCleanupSkipsSymlinks

~ test_real_shards_are_cleaned

~ test_symlinked_shard_not_entered

~ TestWriteMsgpackAtomicSymlink

~ _minimal_commit_dict

~ TestWriteObjectSymlink

~ test_symlinked_shard_dir_raises

~ _sha256_content

~ cleanup_stale_object_temps

~ write_object

~ write_object_from_path

~ tests/test_snapshot_supercharge.py .py 2 symbols added, 2 symbols modified

+ short_id import import short_id L33–33

+ split_id import import split_id L33–33

~ TestSha256PrefixRoundTrip

~ test_bare_hex_prefix_rejected

~ tests/test_stress_merge_regression.py .py 1 symbol modified

~ _init_code_repo

~ tests/test_verify_tag_supercharge.py .py 1 symbol added, 10 symbols modified

+ tag_path import import tag_path L30–30

~ _REPO_ID

~ _force_write_tag

~ _init_repo

~ _make_tag

~ CommitRecord

~ SnapshotRecord

~ TagRecord

~ write_commit

~ write_snapshot

~ write_tag

← Older Oldest on task/utility-sweep-continued

All commits Newer docs: add Pattern 11 — no --format flag, text default + --json f… sha256:a7b203163b3ef82ad0a03b758132eb2b50d22d81bdae3c799163fdc36571f1d2

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:b615cff8d9015afbf10750b39fda0229b13e69bbf9bf8567699a08485bd17e14 --body "your comment"

Provenance signed

AI Agent

claude-sonnet-4-6

Bump patch

Agent claude-code

Signature Cryptographically verified

Snapshot

ID sha256:658094b70748468fced0e141176233b4a80bbaa8ba384ed0678605ac528f1152

Files 0

Browse tree at this commit →

Commit

SHA sha256:b615cff8d9015afbf10750b39fda0229b13e69bbf9bf8567699a08485bd17e14

Parent

sha256:47319766550ec1688724e840a1e71a8f9a7a3ec469a018c0c4c6affa8342b794

Branch task/utility-sweep-continued