gabriel / musehub public
fix minor bugfix/soft-delete-owner-slug #1 / 1
gabriel · 166 days ago · Apr 13, 2026 · Diff

fix: filter soft-deleted repos from get_repo_row_by_owner_slug

Soft-deleted repos were still returned via the owner/slug lookup path because deleted_at IS NULL was missing from get_repo_row_by_owner_slug. The UUID path (get_repo) already filtered correctly.

Added regression suite TestSoftDeleteOwnerSlugRegression with 4 tests: - get_repo_by_owner_slug returns None after delete - get_repo_row_by_owner_slug returns None after delete - live repo unaffected when sibling is deleted - HTTP /owner/slug returns 404 for deleted repo

sha256:d7c39230b08efc76ef28110fc809fc17a460e18d59f8de898df373b33c9504d5 sha
+5 ~1 symbols
sha256:190cfe31c3f25c5da496b6899a480e7983878806b59e7add2407e098243b5397 snapshot
+5
symbols added
~1
symbol modified
0
dead code introduced
Semantic Changes 6 symbols
~ tests/test_repository_service.py .py 5 symbols added
+ TestSoftDeleteOwnerSlugRegression class class TestSoftDeleteOwnerSlugRegression L956–1045
+ 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 L966–983
+ 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 L986–1003
+ test_live_repo_still_returned_after_another_is_deleted method async_method test_live_repo_still_returned_after_another_is_deleted L1006–1023
+ test_owner_slug_http_returns_404_for_deleted_repo method async_method test_owner_slug_http_returns_404_for_deleted_repo L1026–1045
← Older Oldest on bugfix/soft-delete-owner-slug
All commits
Newer → Latest on bugfix/soft-delete-owner-slug

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