gabriel / musehub public
BREAKING task/issue-67-grid #2 / 2
AI Agent gabriel · 119 days ago · May 15, 2026 · Diff

Remove dead threshold guards and fix test models for global content-addressing

- Delete FETCH_PRESIGN_OBJECT_THRESHOLD / FETCH_PRESIGN_SIZE_THRESHOLD constants - Remove dead inline-path branches from wire_fetch_bundle and wire_fetch_presign (presign is now unconditional — BlobBackend.supports_presign always True) - Fix _make_commit helpers: MusehubCommit and MusehubSnapshot no longer have repo_id (globally content-addressed); insert MusehubCommitRef, MusehubCommitGraph, and MusehubSnapshotRef rows for repo membership instead - Delete BC1 test (inline path no longer exists) - Update FP0/FP3/FP8 assertions to expect presign=True (no threshold routing) - Fix FPST1 raise trigger to be within 5-object test range

sha256:495d26b0c4730a92aec91ee340f4630c1554646bc47b1525697b4d39addef927 sha
sha256:ffff02ea552b6bb1dc5ca4820ec806d3c688dfaf712db3dc20771e7eef1e4301 parent
+6 ~15 −6 symbols
sha256:08efba49324930959c865cb4adf8344ca510744f04732dcab615c26878d86e22 snapshot
+6
symbols added
~15
symbols modified
−6
symbols removed
0
dead code introduced
Semantic Changes 27 symbols
~ musehub/services/musehub_wire.py .py 2 symbols removed, 2 symbols modified
− FETCH_PRESIGN_OBJECT_THRESHOLD variable variable FETCH_PRESIGN_OBJECT_THRESHOLD L101–101
− FETCH_PRESIGN_SIZE_THRESHOLD variable variable FETCH_PRESIGN_SIZE_THRESHOLD L102–102 → moved to tests/test_wire_fetch_presign.py
~ wire_fetch_bundle
~ wire_fetch_presign
~ tests/test_fetch_bundle_cleanup.py .py 3 symbols added, 1 symbol removed, 7 symbols modified
− test_bc1_no_delete_on_inline_path function async_function test_bc1_no_delete_on_inline_path L183–212
+ MusehubCommitGraph import import MusehubCommitGraph L23–23
+ MusehubCommitRef import import MusehubCommitRef L23–23
+ MusehubSnapshotRef import import MusehubSnapshotRef L23–23
~ _make_commit
~ _stub_backend
~ MusehubBranch
~ MusehubCommit
~ MusehubObject
~ MusehubObjectRef
~ MusehubSnapshot
~ tests/test_wire_fetch_presign.py .py 3 symbols added, 3 symbols removed, 6 symbols modified
− FETCH_PRESIGN_OBJECT_THRESHOLD import import FETCH_PRESIGN_OBJECT_THRESHOLD L87–87
− FETCH_PRESIGN_SIZE_THRESHOLD import import FETCH_PRESIGN_SIZE_THRESHOLD L87–87
− test_fp0_below_threshold_returns_presign_false function async_function test_fp0_below_threshold_returns_presign_false L173–186
+ FETCH_PRESIGN_OBJECT_THRESHOLD variable variable FETCH_PRESIGN_OBJECT_THRESHOLD L93–93
+ FETCH_PRESIGN_SIZE_THRESHOLD variable variable FETCH_PRESIGN_SIZE_THRESHOLD L94–94 ← moved from musehub/services/musehub_wire.py
+ test_fp0_single_object_always_presigns function async_function test_fp0_single_object_always_presigns L194–207
~ _make_commit
~ wire_fetch_presign
~ test_fp3_have_set_excludes_known_objects
~ test_fp8_route_small_public_repo
~ test_fpp0_all_presign_calls_complete_above_semaphore_limit
~ test_fpst1_presign_get_exception_propagates
← Older Oldest on task/issue-67-grid
All commits Newer Fix test helpers for global content-addressing in fetch bundle tests sha256:38150d49e5fafa5068489ccb3e64799926a63574dc38b7d1c5600eb097e59ed7

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