gabriel / musehub public
refactor tests task/blob-id-audit #1 / 1
AI Agent gabriel · 131 days ago · May 5, 2026 · Diff

refactor(tests): eliminate _sha/_sha256_oid/_blob_id wrappers; inline blob_id at all call sites

- Remove all def _sha, def _sha256_oid, and blob_id/_blob_id alias patterns from 34 test files - Fix test_social_api.py: bare-hex _blob_id (no prefix — a bug) → blob_id() + split_id() - Path construction now embeds algorithm: posts/sha256/{hex[:16]}.json not posts/{bare_hex[:16]}.json - Replace long_id(hashlib.sha256(X).hexdigest()) with blob_id(X) throughout - Remove now-unused 'import hashlib' statements from all affected files - factories.py: commit_id LazyFunction now uses blob_id(str(uuid4()).encode()) - conftest.py: _sha(str) helper removed; all 16 call sites inlined with blob_id(f'...'.encode())

sha256:3a29a6fd5c0da6ed1e40a498f5c491bee45c684b34936b55e64be6b76413cb52 sha
+28 ~170 −70 symbols
sha256:b03a194375d97497bc28cec8c01a3654296f5fa9e37164d7ecae776667f013ad snapshot
+28
symbols added
~170
symbols modified
−70
symbols removed
0
dead code introduced
Semantic Changes 268 symbols
~ tests/bench_push.py .py 1 symbol removed, 7 symbols modified
~ tests/conftest.py .py 1 symbol added, 3 symbols removed, 10 symbols modified
_sha function function _sha L388–389
_hashlib import import _hashlib L382–382
_long_id import import _long_id L385–385
+ blob_id import import blob_id L384–384
~ tests/factories.py .py 1 symbol removed, 1 symbol modified
_sha function function _sha L54–55
~ tests/test_id_canonical.py .py 1 symbol removed, 1 symbol modified
_long_id import import _long_id L70–70
~ tests/test_identity_belt_and_suspenders.py .py 2 symbols added, 1 symbol removed, 6 symbols modified
hashlib import import hashlib L34–34
+ blob_id import import blob_id L44–44
+ public_key_fingerprint import import public_key_fingerprint L44–44
~ tests/test_identity_repo_phase5.py .py 2 symbols added, 2 symbols removed, 4 symbols modified
hashlib import import hashlib L17–17
long_id import import long_id L25–25
+ blob_id import import blob_id L24–24
+ public_key_fingerprint import import public_key_fingerprint L24–24
~ tests/test_identity_repo_phase6.py .py 1 symbol added, 2 symbols removed, 2 symbols modified
hashlib import import hashlib L22–22
long_id import import long_id L31–31
+ blob_id import import blob_id L30–30
~ tests/test_mcp_executor_no_commit_meta.py .py 1 symbol added, 3 symbols removed, 2 symbols modified
_sha function function _sha L18–19
hashlib import import hashlib L4–4
long_id import import long_id L6–6
+ blob_id import import blob_id L5–5
~ tests/test_on_disk_refs.py .py 1 symbol added, 1 symbol removed, 2 symbols modified
_blob_id import import _blob_id L340–340
+ blob_id import import blob_id L340–340
~ tests/test_proposals_no_commit_meta.py .py 1 symbol added, 3 symbols removed, 1 symbol modified
_sha function function _sha L18–19
hashlib import import hashlib L4–4
long_id import import long_id L9–9
+ blob_id import import blob_id L8–8
~ tests/test_quorum_enforcement.py .py 1 symbol added, 2 symbols removed, 2 symbols modified
_sha256 function function _sha256 L31–32
_hashlib import import _hashlib L25–25
+ blob_id import import blob_id L25–25
~ tests/test_release_analysis_no_commit_meta.py .py 1 symbol added, 3 symbols removed, 3 symbols modified
_sha function function _sha L19–20
hashlib import import hashlib L4–4
long_id import import long_id L12–12
+ blob_id import import blob_id L11–11
~ tests/test_repository_no_commit_meta.py .py 1 symbol added, 3 symbols removed, 3 symbols modified
_sha function function _sha L19–20
hashlib import import hashlib L4–4
long_id import import long_id L12–12
+ blob_id import import blob_id L11–11
~ tests/test_snapshot_symbol_indexer.py .py 1 symbol added, 2 symbols modified
+ blob_id import import blob_id L48–48
~ tests/test_social_api.py .py 2 symbols added, 3 symbols removed, 1 symbol modified
_blob_id function function _blob_id L52–53
hashlib import import hashlib L17–17
long_id import import long_id L31–31
+ blob_id import import blob_id L30–30
+ split_id import import split_id L30–30
~ tests/test_symbol_indexer_no_commit_meta.py .py 1 symbol added, 3 symbols removed, 4 symbols modified
_sha function function _sha L19–20
hashlib import import hashlib L4–4
long_id import import long_id L10–10
+ blob_id import import blob_id L9–9
~ tests/test_ui_agents_no_commit_meta.py .py 1 symbol added, 3 symbols removed, 1 symbol modified
_sha function function _sha L18–19
hashlib import import hashlib L4–4
long_id import import long_id L6–6
+ blob_id import import blob_id L5–5
~ tests/test_ui_blame_no_commit_meta.py .py 1 symbol added, 3 symbols removed, 2 symbols modified
_sha function function _sha L18–19
hashlib import import hashlib L4–4
long_id import import long_id L9–9
+ blob_id import import blob_id L8–8
~ tests/test_ui_blob_no_commit_meta.py .py 2 symbols added, 3 symbols removed, 2 symbols modified
_sha function function _sha L23–24
hashlib import import hashlib L10–10
long_id import import long_id L16–16
+ blob_id import import blob_id L16–16
+ msgpack import import msgpack L10–10
~ tests/test_ui_commits_no_commit_meta.py .py 1 symbol added, 3 symbols removed, 1 symbol modified
_sha function function _sha L18–19
hashlib import import hashlib L4–4
long_id import import long_id L9–9
+ blob_id import import blob_id L8–8
~ tests/test_ui_proposals_no_commit_meta.py .py 1 symbol added, 3 symbols removed, 1 symbol modified
_sha function function _sha L18–19
hashlib import import hashlib L4–4
long_id import import long_id L9–9
+ blob_id import import blob_id L8–8
~ tests/test_ui_repo_no_commit_meta.py .py 1 symbol added, 3 symbols removed, 2 symbols modified
_sha function function _sha L18–19
hashlib import import hashlib L4–4
long_id import import long_id L9–9
+ blob_id import import blob_id L8–8
~ tests/test_ui_user_profile_no_commit_meta.py .py 1 symbol added, 3 symbols removed, 1 symbol modified
_sha function function _sha L18–19
hashlib import import hashlib L4–4
long_id import import long_id L9–9
+ blob_id import import blob_id L8–8
~ tests/test_ui_view_no_commit_meta.py .py 1 symbol added, 3 symbols removed, 1 symbol modified
_sha function function _sha L24–25
hashlib import import hashlib L11–11
long_id import import long_id L17–17
+ blob_id import import blob_id L16–16
~ tests/test_users_no_commit_meta.py .py 1 symbol added, 3 symbols removed, 1 symbol modified
_sha function function _sha L18–19
hashlib import import hashlib L4–4
long_id import import long_id L11–11
+ blob_id import import blob_id L10–10
~ tests/test_wire_framing.py .py 1 symbol removed, 2 symbols modified
_sha256_oid function function _sha256_oid L30–31
~ tests/test_wire_push_plain.py .py 1 symbol removed, 3 symbols modified
_sha256_oid function function _sha256_oid L60–61
~ tests/test_wire_push_stream.py .py 1 symbol removed, 35 symbols modified
_sha256_oid function function _sha256_oid L99–100
~ tests/test_wire_streaming_push.py .py 1 symbol removed, 3 symbols modified
← Older Oldest on task/blob-id-audit
All commits
Newer → Latest on task/blob-id-audit

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