gabriel / musehub public
feat BREAKING task/hard-delete-repos #1 / 1
gabriel · 164 days ago · Apr 14, 2026 · Diff

feat: hard-delete repos — drop deleted_at, convert soft-delete to CASCADE

- Migration 0047: drop deleted_at column and ix_musehub_repos_deleted_at index - MusehubRepo ORM: remove deleted_at field - delete_repo(): session.delete() + flush() instead of UPDATE SET deleted_at - Remove all deleted_at.is_(None) query filters across services and routes - Update tests to use hard-delete patterns; drop soft-delete test cases - Update migration count/head assertions to 47

sha256:6cc6c26f84cf0c1ced2c799bbb9ad0a948a7f4eb4db795eb006e2ecb02490411 sha
sha256:6fa1f603c7810b4dfea38cd7d3d3ab73ec254b9dd36f9ff5afda1955da8a0583 parent
+17 ~68 −15 symbols
sha256:1fe17f92e30a571c54e95c714a0d7f666202dc9147dd85490a2641df47b2a5a6 snapshot
+17
symbols added
~68
symbols modified
−15
symbols removed
0
dead code introduced
Semantic Changes 100 symbols
~ alembic/versions/0047_hard_delete_repos.py .py 9 symbols added
+ branch_labels variable variable branch_labels L23–23
+ depends_on variable variable depends_on L24–24
+ down_revision variable variable down_revision L22–22
+ downgrade function function downgrade L32–37
+ annotations import import annotations L16–16
+ op import import op L19–19
+ sa import import sa L18–18
+ revision variable variable revision L21–21
+ upgrade function function upgrade L27–29
~ musehub/api/routes/api/identities.py .py 1 symbol modified
~ list_identity_repos
~ musehub/api/routes/musehub/sitemap.py .py 1 symbol modified
~ _fetch_sitemap_entries
~ musehub/api/routes/musehub/topics.py .py 2 symbols modified
~ list_repos_by_topic
~ list_topics
~ musehub/api/routes/musehub/ui.py .py 1 symbol modified
~ explore_page
~ musehub/api/routes/musehub/ui_topics.py .py 2 symbols modified
~ _fetch_all_topics
~ _fetch_topic_repos
~ musehub/api/routes/musehub/ui_user_profile.py .py 8 symbols modified
~ _build_heatmap
~ _compute_provenance_badges
~ _compute_provenance_stats
~ _fetch_activity
~ _fetch_agent_fleet
~ _fetch_repos
~ _fetch_symbol_footprint
~ _resolve_identity
~ musehub/api/routes/musehub/users.py .py 1 symbol removed, 4 symbols modified
− sa_update import import sa_update L25–25
~ _query_agent_fleet
~ delete_my_account
~ export_my_data
~ get_agent_fleet
~ musehub/db/musehub_models.py .py 1 symbol modified
~ MusehubRepo
~ musehub/services/musehub_cross_repo.py .py 1 symbol modified
~ _load_owner_repos
~ musehub/services/musehub_discover.py .py 1 symbol modified
~ list_public_repos
~ musehub/services/musehub_domains.py .py 1 symbol modified
~ list_repos_for_domain
~ musehub/services/musehub_mcp_executor.py .py 3 symbols modified
~ execute_muse_push
~ execute_read_cross_repo_impact
~ execute_read_workspace_intel
~ musehub/services/musehub_profile.py .py 3 symbols modified
~ get_contribution_graph
~ get_public_repos
~ get_session_credits
~ musehub/services/musehub_repository.py .py 11 symbols modified
~ _unique_slug_for_owner
~ delete_repo
~ fork_repo
~ get_repo
~ get_repo_row_by_owner_slug
~ get_user_forks
~ global_search
~ list_repo_forks
~ list_repo_forks_flat
~ list_repos_for_user
~ transfer_repo_ownership
~ musehub/services/musehub_wire.py .py 9 symbols modified
~ wire_confirm_objects
~ wire_fetch
~ wire_negotiate
~ wire_presign
~ wire_push
~ wire_push_object_pack
~ wire_push_objects
~ wire_refs
~ wire_repair_object
~ tests/test_compliance.py .py 1 symbol removed, 1 symbol modified
− test_delete_soft_deletes_repos method method test_delete_soft_deletes_repos L356–359
~ TestGdprEndpointsExist
~ tests/test_cross_repo.py .py 2 symbols removed, 1 symbol modified
− datetime import import datetime L21–21
− timezone import import timezone L21–21
~ _db_repo
~ tests/test_domains.py .py 1 symbol modified
~ _db_repo
~ tests/test_mcp_read_tools.py .py 2 symbols added, 2 symbols removed, 1 symbol modified
− test_list_repos_excludes_soft_deleted method async_method test_list_repos_excludes_soft_deleted L1293–1308
− test_list_repos_total_excludes_soft_deleted method async_method test_list_repos_total_excludes_soft_deleted L1310–1324
+ test_list_repos_excludes_deleted method async_method test_list_repos_excludes_deleted L1293–1307
+ test_list_repos_total_excludes_deleted method async_method test_list_repos_total_excludes_deleted L1309–1322
~ TestListReposComprehensive
~ tests/test_migrations.py .py 2 symbols added, 2 symbols removed, 5 symbols modified
− test_all_46_migrations_present method method test_all_46_migrations_present L267–270
− test_head_revision_is_0046 method method test_head_revision_is_0046 L272–292
+ test_all_47_migrations_present method method test_all_47_migrations_present L267–270
+ test_head_revision_is_0047 method method test_head_revision_is_0047 L272–292
~ TestMigrationUnit
~ _ALL_REVISIONS
~ _EXPECTED_HEAD_PREFIX
~ _EXPECTED_MIGRATION_COUNT
~ _HEAD
~ tests/test_musehub_alembic.py .py 2 symbols modified
~ _EXPECTED_HEAD
~ _EXPECTED_MIGRATION_COUNT
~ tests/test_musehub_repos.py .py 1 symbol added, 1 symbol removed
− test_delete_repo_service_sets_deleted_at function async_function test_delete_repo_service_sets_deleted_at L1402–1421
+ test_delete_repo_service_hard_deletes_row function async_function test_delete_repo_service_hard_deletes_row L1402–1422
~ tests/test_repository_service.py .py 3 symbols added, 6 symbols removed, 1 symbol modified
− test_soft_delete_hides_repo_but_preserves_db_row method async_method test_soft_delete_hides_repo_but_preserves_db_row L714–730
− TestSoftDeleteOwnerSlugRegression class class TestSoftDeleteOwnerSlugRegression L957–1046
− test_get_repo_by_owner_slug_returns_none_after_soft_delete method async_method test_get_repo_by_owner_slug_returns_none_after_soft_delete L967–984
− test_get_repo_row_by_owner_slug_returns_none_after_soft_delete method async_method test_get_repo_row_by_owner_slug_returns_none_after_soft_delete L987–1004
− test_live_repo_still_returned_after_another_is_deleted method async_method test_live_repo_still_returned_after_another_is_deleted L1007–1024
− test_owner_slug_http_returns_404_for_deleted_repo method async_method test_owner_slug_http_returns_404_for_deleted_repo L1027–1046
+ test_delete_hard_deletes_row method async_method test_delete_hard_deletes_row L714–727
+ TestDeleteOwnerSlugRegression class class TestDeleteOwnerSlugRegression L954–970
+ test_owner_slug_http_returns_404_for_deleted_repo method async_method test_owner_slug_http_returns_404_for_deleted_repo L958–970
~ TestDataIntegrity
~ tests/test_security_hardening.py .py 5 symbols modified
~ TestAgentImpersonation
~ test_known_agent_not_flagged
~ test_unknown_agent_flagged_when_registry_set
~ TestCommitIdForgery
~ test_wire_push_rejects_unsigned_when_enforcement_on
~ tests/test_symbol_intelligence.py .py 2 symbols modified
~ TestSecurity
~ test_blame_404_for_deleted_repo
← Older Oldest on task/hard-delete-repos
All commits
Newer → Latest on task/hard-delete-repos

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