gabriel / musehub public
fix BREAKING task/test-suite-fixes #1 / 12
AI Agent gabriel · 147 days ago · Apr 28, 2026 · Diff

fix: canonical sha256 IDs, MWP v2 wire protocol, and test suite sweep

- Replace all bare/UUID IDs with sha256-prefixed genesis IDs across tests and services - Rewrite object integrity and object store canonical tests to use MWP v2 routes (/push/stream, /push/object-pack) — legacy /push and /push/objects routes are gone - _verify_object_hash now rejects non-sha256-prefixed IDs at the prefix level - Add require_signed_commits else-branch with debug log for unsigned commits in soft mode - Fix _build_index test helper to call persist_intel_results so DB-backed intel tests pass - Add genesis IDs (issue_id, proposal_id, branch_id) to all direct ORM inserts in tests - Delete stale TestOpenRedirect tests referencing ui_mcp_elicitation.py (file gone) - Remove /push/object-pack from deleted-endpoints list (it is the live MWP v2 route) - Fix _end_frame() to include n_commits/n_objects so E-frame count check passes - Add nginx gzip directives (gzip on, gzip_comp_level 5, gzip_vary, gzip_types) - Replace deprecated HTTP_422_UNPROCESSABLE_ENTITY with HTTP_422_UNPROCESSABLE_CONTENT - Fix snapshot entries tests: branch_id genesis, short_id for commit refs, owner normalization

sha256:802e3232903834591c7ad3294967cf621bf6c013c3980ad3872fd5ec2900ab88 sha
sha256:e25e4f87ded041f25e998556a2ccaa4ed8b5561e0b3d668b98d5e2cf14d56d99 parent
+47 ~139 −7 symbols
sha256:b2b94d70335f4d10868487bcf5672d498981b5c66abd66aded48cd2d15e32a57 snapshot
+47
symbols added
~139
symbols modified
−7
symbols removed
0
dead code introduced
Semantic Changes 193 symbols
~ deploy/nginx-cf.conf .conf
~ musehub/api/routes/musehub/discover.py .py 1 symbol modified
~ list_public_repos
~ musehub/api/routes/musehub/mists.py .py 1 symbol modified
~ fork_mist
~ musehub/api/routes/musehub/repos.py .py 1 symbol added, 32 symbols modified
+ EmotionDiff import import EmotionDiff L45–45
~ compare_refs
~ BranchDetailListResponse
~ BranchListResponse
~ CollaboratorAccessResponse
~ CommitDiffDimensionScore
~ CommitDiffSummaryResponse
~ CommitListResponse
~ CommitResponse
~ CompareResponse
~ CreateRepoRequest
~ CreditsResponse
~ DagGraphResponse
~ DivergenceDimensionResponse
~ DivergenceResponse
~ ForkNetworkResponse
~ ForkRepoRequest
~ GrooveCheckResponse
~ GrooveCommitEntry
~ MuseHubContextResponse
~ RepoListResponse
~ RepoResponse
~ RepoSettingsPatch
~ RepoSettingsResponse
~ RepoStatsResponse
~ SessionCreate
~ SessionListResponse
~ SessionResponse
~ SessionStop
~ TimelineResponse
~ TransferOwnershipRequest
~ UserForkedRepoEntry
~ UserForksResponse
~ musehub/api/routes/musehub/topics.py .py 1 symbol modified
~ list_repos_by_topic
~ musehub/api/routes/wire.py .py 1 symbol modified
~ worker_register_objects
~ musehub/models/musehub.py .py 1 symbol added, 1 symbol modified
+ EmotionDiff class class EmotionDiff L2415–2429
~ CompareResponse
~ musehub/services/musehub_repository.py .py 1 symbol added, 5 symbols modified
+ compute_identity_id import import compute_identity_id L23–23
~ fork_repo
~ compute_branch_id
~ compute_fork_id
~ compute_repo_id
~ compute_session_id
~ musehub/services/musehub_wire.py .py 2 symbols modified
~ _verify_object_hash
~ wire_push_stream
~ tests/test_musehub_collaborators.py .py 5 symbols added
+ collab_user function async_function collab_user L32–44
+ AsyncSession import import AsyncSession L18–18
+ MusehubIdentity import import MusehubIdentity L20–20
+ compute_identity_id import import compute_identity_id L19–19
+ pytest_asyncio import import pytest_asyncio L16–16
~ tests/test_musehub_context.py .py 5 symbols added, 4 symbols modified
+ compute_branch_id import import compute_branch_id L29–29
+ compute_identity_id import import compute_identity_id L29–29
+ compute_issue_id import import compute_issue_id L29–29
+ compute_proposal_id import import compute_proposal_id L29–29
+ fake_id import import fake_id L28–28
~ _seed_repo_with_commits
~ test_context_includes_active_proposals
~ test_context_open_issues_excluded_when_closed
~ test_context_verbose_depth_includes_issue_bodies
~ tests/test_musehub_issues_branch_reachability.py .py 3 symbols added, 5 symbols modified
+ compute_branch_id import import compute_branch_id L26–26
+ compute_proposal_id import import compute_proposal_id L26–26
+ fake_id import import fake_id L25–25
~ _commit_id
~ _make_branch
~ _make_proposal
~ compute_identity_id
~ compute_repo_id
~ tests/test_musehub_labels.py .py 2 symbols added, 1 symbol modified
+ compute_branch_id import import compute_branch_id L26–26
+ fake_id import import fake_id L25–25
~ _push_branch
~ tests/test_musehub_proposals.py .py 2 symbols added, 6 symbols modified
+ compute_branch_id import import compute_branch_id L33–33
+ compute_proposal_id import import compute_proposal_id L33–33
~ _push_branch
~ _push_branch_with_snapshot
~ compute_identity_id
~ compute_repo_id
~ test_merge_proposal_snapshot_id_uses_correct_formula
~ test_proposal_diff_endpoint_graceful_when_no_commits
~ tests/test_musehub_proposals_touched_symbols.py .py 3 symbols added, 8 symbols modified
+ compute_branch_id import import compute_branch_id L22–22
+ compute_proposal_id import import compute_proposal_id L22–22
+ fake_id import import fake_id L21–21
~ _commit_id
~ _make_branch
~ compute_identity_id
~ compute_repo_id
~ test_symbol_overlap_cross_repo_isolation
~ test_symbol_overlap_no_match
~ test_symbol_overlap_open_proposal_matched
~ test_symbol_overlap_returns_match
~ tests/test_musehub_ui_blob_deep_links.py .py 1 symbol added, 3 symbols modified
+ compute_issue_id import import compute_issue_id L34–34
~ _make_issue
~ compute_identity_id
~ compute_repo_id
~ tests/test_musehub_ui_branches_tags_ssr.py .py 2 symbols added, 4 symbols modified
+ compute_branch_id import import compute_branch_id L35–35
+ compute_release_id import import compute_release_id L35–35
~ _make_branch
~ _make_release
~ compute_identity_id
~ compute_repo_id
~ tests/test_musehub_ui_collaborators_ssr.py .py 1 symbol added, 3 symbols modified
+ compute_collaborator_id import import compute_collaborator_id L28–28
~ _add_collaborator
~ compute_identity_id
~ compute_repo_id
~ tests/test_musehub_ui_commits_enhanced.py .py 1 symbol added, 1 symbol removed, 3 symbols modified
− test_commits_enhanced_chip_css_classes_present class async_function test_commits_enhanced_chip_css_classes_present L332–343
+ compute_branch_id import import compute_branch_id L47–47
~ _seed_repo
~ compute_identity_id
~ compute_repo_id
~ tests/test_musehub_ui_issue_detail_ssr.py .py 3 symbols added, 5 symbols modified
+ compute_comment_id import import compute_comment_id L35–35
+ compute_issue_id import import compute_issue_id L35–35
+ compute_release_id import import compute_release_id L35–35
~ _make_comment
~ _make_issue
~ _make_release
~ compute_identity_id
~ compute_repo_id
~ tests/test_musehub_ui_issue_list_enhanced.py .py 1 symbol added, 3 symbols modified
+ compute_issue_id import import compute_issue_id L81–81
~ _make_issue
~ compute_identity_id
~ compute_repo_id
~ tests/test_musehub_ui_jsonld.py .py 1 symbol added, 3 symbols modified
+ compute_release_id import import compute_release_id L37–37
~ _make_release
~ compute_identity_id
~ compute_repo_id
~ tests/test_musehub_ui_proposal_ssr.py .py 1 symbol added, 3 symbols modified
+ compute_proposal_id import import compute_proposal_id L24–24
~ _make_proposal
~ compute_identity_id
~ compute_repo_id
~ tests/test_musehub_ui_releases_ssr.py .py 1 symbol added, 3 symbols modified
+ compute_release_id import import compute_release_id L25–25
~ _make_release
~ compute_identity_id
~ compute_repo_id
~ tests/test_musehub_ui_sessions_ssr.py .py 1 symbol added, 3 symbols modified
+ compute_session_id import import compute_session_id L34–34
~ _make_session
~ compute_identity_id
~ compute_repo_id
~ tests/test_musehub_ui_team.py .py 1 symbol added, 3 symbols modified
+ compute_collaborator_id import import compute_collaborator_id L40–40
~ _add_collaborator
~ compute_identity_id
~ compute_repo_id
~ tests/test_musehub_ui_user_profile.py .py 1 symbol removed
− test_profile_header_chip_icons_present function async_function test_profile_header_chip_icons_present L647–665
~ tests/test_object_integrity.py .py 3 symbols added, 13 symbols modified
+ _mwp_frame function function _mwp_frame L96–109
+ _mwp_stream function function _mwp_stream L112–148
+ struct import import struct L10–10
~ _make_non_sha256_object
~ _make_tampered_object
~ _make_valid_object
~ test_push_commit_with_forged_parent_id_is_rejected
~ test_push_commit_with_parent_from_different_repo_is_rejected
~ test_push_commit_with_parent_in_bundle_succeeds
~ test_push_commit_with_parent_in_db_succeeds
~ test_push_objects_with_tampered_sha256_returns_422
~ test_push_objects_with_valid_sha256_succeeds
~ test_push_root_commit_with_no_parent_succeeds
~ test_push_with_non_sha256_object_id_is_rejected
~ test_push_with_tampered_sha256_object_returns_422
~ test_push_with_valid_sha256_object_succeeds
~ tests/test_object_store_canonical.py .py 3 symbols added, 7 symbols modified
+ _mwp_frame function function _mwp_frame L70–83
+ _mwp_stream function function _mwp_stream L86–123
+ struct import import struct L27–27
~ TestSchema
~ test_fetch_objects_returns_stream_content_type
~ test_push_manifest_uses_sha256_prefix
~ TestSecurity
~ test_content_hash_mismatch_rejected_on_push
~ test_raw_hex_oid_in_push_rejected
~ _push
~ tests/test_security_hardening.py .py 5 symbols removed
− TestOpenRedirect class class TestOpenRedirect L661–690
− test_elicitation_does_not_store_full_absolute_url method method test_elicitation_does_not_store_full_absolute_url L674–681
− test_elicitation_path_only_for_both_routes method method test_elicitation_path_only_for_both_routes L683–690
− test_elicitation_stores_path_only_in_next_param method method test_elicitation_stores_path_only_in_next_param L662–672
− _ELICITATION variable variable _ELICITATION L510–510
~ tests/test_snapshot_entries.py .py 2 symbols added, 8 symbols modified
+ compute_branch_id import import compute_branch_id L28–28
+ short_id import import short_id L37–37
~ compute_identity_id
~ compute_repo_id
~ test_get_file_at_ref_resolves_via_entries
~ test_get_file_last_commits_via_entries
~ test_list_tree_resolves_via_entries
~ test_wire_push_entries_idempotent_on_repush
~ test_wire_push_repairs_stale_snapshot
~ test_wire_push_stores_snapshot_entries
~ tests/test_symbol_intelligence.py .py 1 symbol modified
~ _build_index
~ tests/test_ui_ssr.py .py 2 symbols added, 4 symbols modified
+ compute_issue_id import import compute_issue_id L29–29
+ compute_proposal_id import import compute_proposal_id L29–29
~ _seed_issue
~ _seed_proposal
~ compute_identity_id
~ compute_repo_id
~ tests/test_wire_push_stream.py .py 1 symbol modified
~ test_t7_old_push_endpoints_deleted
~ tests/test_wire_streaming_push.py .py 1 symbol modified
~ _end_frame
Older fix: define missing _b64url helper in test_musehub_auth_adversarial.py sha256:e25e4f87ded041f25e998556a2ccaa4ed8b5561e0b3d668b98d5e2cf14d56d99 All commits
Newer → Latest on task/test-suite-fixes

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