refactor: sha256:b5e0cb54e3d1a5cbfb53740e853a0eae6736ff89f9e64cd51fb674df6842d610 · gabriel/muse

refactor: phase 4 sweep — remove dead repo_id param from get_commits_for_branch

Three stale items in store.py fixed: - Module docstring commit schema: removed 'repo_id' from the example JSON block - _verify_commit_id docstring: removed repo_id from the compute_commit_id coverage list (compute_commit_id never included repo_id) - get_commits_for_branch: removed dead repo_id: str parameter and its docstring entry — the parameter was never read in the function body

Updated all callers to drop the now-gone argument: - muse/cli/commands/log.py (2 sites) - muse/cli/commands/shortlog.py (1 site) - tests/test_core_store.py (5 sites) - tests/test_core_coverage_gaps.py (4 sites + 1 CommitRecord straggler) - tests/test_phase5_store_linear_walks.py (2 sites) - tests/test_cmd_branch.py (3 sites) - tests/test_cmd_checkout.py (1 site) - tests/test_perf_phase3.py (2 sites)

~29

symbols modified

dead code introduced

Semantic Changes 29 symbols

~ muse/cli/commands/log.py .py 1 symbol modified

~ run

~ muse/cli/commands/shortlog.py .py 1 symbol modified

~ run

~ muse/core/store.py .py 2 symbols modified

~ _verify_commit_id

~ get_commits_for_branch

~ tests/test_cmd_branch.py .py 5 symbols modified

~ TestCreate

~ test_create_at_full_sha

~ test_create_at_partial_sha

~ TestResolveStartPoint

~ test_resolves_partial_sha

~ tests/test_cmd_checkout.py .py 2 symbols modified

~ TestDetach

~ test_detach_partial_sha_points_to_correct_commit

~ tests/test_core_coverage_gaps.py .py 6 symbols modified

~ TestStoreGaps

~ _seed_chain

~ test_get_commits_for_branch_empty

~ test_get_commits_for_branch_max_count_larger_than_chain

~ test_get_commits_for_branch_max_count_stops_early

~ test_get_commits_for_branch_max_count_zero_returns_all

~ tests/test_core_store.py .py 7 symbols modified

~ TestGetCommitsForBranch

~ test_chain

~ test_empty_branch

~ test_max_count_with_remote_ref

~ test_remote_tracking_ref

~ test_remote_tracking_ref_chain

~ test_remote_tracking_ref_missing_returns_empty

~ tests/test_perf_phase3.py .py 3 symbols modified

~ TestGetCommitsForBranchWalkCap

~ test_walk_cap_bounds_returned_records

~ test_walk_cap_memory_bounded_deep_chain

~ tests/test_phase5_store_linear_walks.py .py 2 symbols modified

~ test_p5_6_get_commits_for_branch_newest_first

~ test_p5_7_get_commits_for_branch_max_count

← Older Oldest on task/issue-9-phase4-sweep

All commits

Newer → Latest on task/issue-9-phase4-sweep

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