gabriel / musehub public
refactor BREAKING task/sweep-bundle-to-mpack #1 / 1
AI Agent gabriel · 118 days ago · May 16, 2026 · Diff

refactor: replace wire-protocol bundle terminology with mpack throughout musehub

bundle in MuseHub context always meant mpack. muse bundle is the separate offline-transfer CLI concept and is untouched.

Source: - musehub_wire.py: fetch_bundle → fetch_mpack in log format strings; comments - musehub_snapshot.py: push bundles → push mpacks in docstring - 0059_pack_index.py: bundle → mpack in migration docstring

Tests (26 files): - _push_bundle → _push_mpack helpers - _make_tampered_bundle → _make_tampered_mpack - bundle_dict/bundle_hex/bundle_hash → mpack_dict/mpack_hex/mpack_id - commits_in_bundle → commits_in_mpack - fetch_bundle_ms → fetch_mpack_ms in phase4 timing labels and assertions - test function names: test_*bundle* → test_*mpack* throughout - unit/test_phase2_inMemory_walk_dag.py: removed dead P2-6/P2-10/P2-11 tests for _is_ancestor_in_bundle which was replaced by _is_fast_forward

sha256:31703a39a7206c1ec8eca013a19c7538f84c60b48fdd019731f181103049de5e sha
sha256:c763733e98504ba53ad394dcce33375795ae9e254f0e2de0e00a49ec3d171aea parent
+8 ~61 −11 symbols
sha256:24e266b7756eba5cc6ab135935d2a63bf5c6f93e12527a35ff5e9b8284419083 snapshot
+8
symbols added
~61
symbols modified
−11
symbols removed
0
dead code introduced
Semantic Changes 80 symbols
~ alembic/versions/0059_pack_index.py .py
~ musehub/services/musehub_snapshot.py .py
~ musehub/services/musehub_wire.py .py 3 symbols modified
~ FetchMPackResult
~ process_mpack_index_job
~ wire_fetch_mpack
~ tests/test_cdn_reads_phase5.py .py 1 symbol modified
~ test_cdn5_wire_fetch_bundle_pack_urls_use_cdn
~ tests/test_data_integrity.py .py 1 symbol modified
~ test_ingest_push_accepts_parent_in_same_bundle
~ tests/test_fetch_mpack_cleanup.py .py 1 symbol modified
~ test_bc0_bundle_deleted_after_ttl
~ tests/test_fetch_mpack_route.py .py 2 symbols modified
~ test_rb1_small_public_repo_returns_inline_bundle
~ test_rb2_route_preserves_content_addressing_proof
~ tests/test_ltier_fetch_regression.py .py 1 symbol modified
~ test_lf2_wire_fetch_bundle_1000_commits
~ tests/test_mpack_content_scanning_phase3.py .py 2 symbols added, 2 symbols removed, 4 symbols modified
~ _push_bundle
− test_blocked_object_quarantines_bundle function async_function test_blocked_object_quarantines_bundle L215–250
− test_clean_bundle_bypasses_blocklist_unimpeded function async_function test_clean_bundle_bypasses_blocklist_unimpeded L320–342
+ test_blocked_object_quarantines_mpack function async_function test_blocked_object_quarantines_mpack L215–250
+ test_clean_mpack_bypasses_blocklist_unimpeded function async_function test_clean_mpack_bypasses_blocklist_unimpeded L320–342
~ test_blocked_check_fires_before_minio_puts
~ test_content_scan_jobs_enqueued_after_indexing
~ test_multiple_blocked_objects_all_reported
~ tests/test_mpack_delta_e2e.py .py 2 symbols modified
~ test_bundle_is_delta_encoded
~ test_bundle_push_delta_e2e
~ tests/test_mpack_fetch_readiness.py .py 1 symbol added, 1 symbol removed, 4 symbols modified
~ _push_bundle
− test_large_bundle_skips_content_cache function async_function test_large_bundle_skips_content_cache L311–348
+ test_large_mpack_skips_content_cache function async_function test_large_mpack_skips_content_cache L311–348
~ _make_small_repo
~ test_content_cache_cleared_after_job
~ test_fetch_object_before_worker_returns_bytes
~ tests/test_mpack_index_job.py .py 8 symbols modified
~ _push_bundle
~ _make_repo
~ test_job_is_idempotent
~ test_musehub_objects_rows_after_job
~ test_no_per_object_sha256_verification
~ test_object_refs_after_job
~ test_objects_written_to_minio_after_job
~ test_return_dict_includes_counts_and_timing
~ tests/test_mpack_index_job_phase3.py .py 4 symbols modified
~ _push_bundle
~ test_failed_job_resets_to_pending_for_retry
~ test_partial_minio_failure_retry_produces_correct_state
~ test_worker_full_lifecycle_pending_to_done
~ tests/test_mpack_index_job_phase4.py .py 6 symbols modified
~ _push_bundle
~ test_all_phase_timings_are_non_negative
~ test_mpack_size_bytes_in_return_dict
~ test_phase_timings_sum_within_elapsed
~ test_return_dict_has_per_phase_timing_keys
~ test_summary_log_contains_phase_breakdown
~ tests/test_mpack_push_async.py .py 2 symbols modified
~ test_bundle_index_job_populates_commits_and_snapshots
~ test_unpack_bundle_returns_fast
~ tests/test_mpack_rate_limiting_phase4.py .py 2 symbols modified
~ test_record_bundle_bytes_accumulates_across_calls
~ test_record_bundle_bytes_increments_daily_total
~ tests/test_mpack_size_gates.py .py 6 symbols modified
~ test_presign_accepts_bundle_at_limit
~ test_presign_rejects_oversized_bundle
~ test_unpack_bundle_accepts_counts_at_limit
~ test_unpack_bundle_rejects_absurd_commits_count
~ test_unpack_bundle_rejects_absurd_objects_count
~ test_unpack_bundle_rejects_oversized_wire_bytes
~ tests/test_mpack_validation_phase2.py .py 3 symbols added, 3 symbols removed, 6 symbols modified
~ _make_tampered_bundle
~ _push_bundle
− test_count_mismatch_raises_bundle_validation_error function async_function test_count_mismatch_raises_bundle_validation_error L278–302
− test_valid_bundle_passes_all_validation_checks function async_function test_valid_bundle_passes_all_validation_checks L455–471
− test_zip_bomb_raises_bundle_validation_error function async_function test_zip_bomb_raises_bundle_validation_error L253–274
+ test_count_mismatch_raises_mpack_validation_error function async_function test_count_mismatch_raises_mpack_validation_error L278–302
+ test_valid_mpack_passes_all_validation_checks function async_function test_valid_mpack_passes_all_validation_checks L455–471
+ test_zip_bomb_raises_mpack_validation_error function async_function test_zip_bomb_raises_mpack_validation_error L253–274
~ test_process_mpack_index_job_sets_quarantined_on_validation_error
~ test_quarantine_job_sets_status_and_reason
~ test_validation_failure_writes_no_commits_to_db
~ test_validation_failure_writes_no_objects_to_minio
~ tests/test_pack_fetch_phase3.py .py
~ tests/test_pack_gc_phase4.py .py 1 symbol modified
~ test_pg3_consolidated_bundle_contains_all_objects
~ tests/test_pack_index_phase1.py .py 1 symbol modified
~ test_pi2_pack_id_matches_bundle_key
~ tests/test_push_xs_unit.py .py 2 symbols modified
~ test_p1_bundle_key_matches_wire_bytes
~ test_p1_bundle_objects_round_trip
~ tests/test_wire_fetch_mpack.py .py 1 symbol added, 1 symbol removed, 3 symbols modified
~ test_fb0_single_commit_returns_bundle
~ test_fb1_bundle_id_equals_sha256_of_bytes
− test_fb2_bundle_contains_commits_and_objects function async_function test_fb2_bundle_contains_commits_and_objects L237–271
+ test_fb2_mpack_contains_commits_and_objects function async_function test_fb2_mpack_contains_commits_and_objects L237–271
~ test_fb3_have_excludes_known_commits
~ tests/test_wire_step3_upload.py .py 1 symbol modified
~ test_upload_step3_performance_gate
~ tests/test_wire_step4c_scale.py .py 1 symbol added, 1 symbol removed
− test_scale_unpack_bundle function async_function test_scale_unpack_bundle L192–285
+ test_scale_unpack_mpack function async_function test_scale_unpack_mpack L192–285
~ tests/unit/test_phase2_inMemory_walk_dag.py .py 3 symbols removed
− test_p2_10_is_ancestor_in_bundle_true function function test_p2_10_is_ancestor_in_bundle_true L218–230
− test_p2_11_is_ancestor_in_bundle_false function function test_p2_11_is_ancestor_in_bundle_false L237–248
− test_p2_6_is_ancestor_in_bundle_uses_walk_dag function function test_p2_6_is_ancestor_in_bundle_uses_walk_dag L122–132
← Older Oldest on task/sweep-bundle-to-mpack
All commits
Newer → Latest on task/sweep-bundle-to-mpack

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