Sweep a Pruned Worktree Entry's Branch in the Same Run in pr_review.py

[ptr727]

Summary

This fixes a finding raised on promotion #3189. cleanup --sweep read the branches held by worktrees before pruning the entries whose directory is gone. A finished branch held only by a pruned entry was therefore skipped by the loose-branch scan until the next run.

CleanupSweep.prune now returns the entries it pruned, or under a report the ones it would prune, and those entries no longer hold their branch. An entry kept as too recent, or still listed after a failed prune, keeps holding its branch.

Tests

• test_a_pruned_entry_s_branch_is_swept_in_the_same_run and test_a_dry_run_reports_a_prunable_entry_s_branch both failed before the fix.
• test_a_too_recent_prunable_entry_still_holds_its_branch fails when held drops every prunable entry.
• The failed-prune test now carries a branch-holding entry. It fails when the failed path returns the entries anyway.
• python3 -m unittest tests.test_pr_review ran 917+ tests, all OK.

Local review

Three recorded local-strict-review passes ran at the frontier tier. Both findings were test-coverage gaps, now closed. Nothing remains open.

🤖 Generated with Claude Code

[ptr727]

A recorded local strict-review pass covers head a27f90986faf6e8ea871973d916edb01f8d6c174, the content this pull request carries at that commit against develop, and it recorded 0 findings.

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

• Configuration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: 6cbc7a14-4519-4dd6-b3e9-0dd3796081fa

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[Copilot]

🔵 Needs a closer look

Cleanup-state and dry-run behavior warrant final human review.

No open issues 🎉

What changed in this PR

Updates cleanup sweeping so branches from pruned worktree entries are evaluated in the same run.

Changes:

• Tracks pruned and report-prunable worktree entries.
• Preserves branch holds for recent or failed prunes.
• Adds regression coverage for apply, dry-run, recent, and failed-prune cases.

File	Summary
tests/​test_pr_review.py	Adds pruning and branch-sweep regression tests.
scripts/​pr_review.py	Updates pruning and branch-hold tracking.

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.46%. Comparing base (3dcb1aa) to head (a27f909).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #3190   +/-   ##
========================================
  Coverage    65.46%   65.46%           
========================================
  Files           17       17           
  Lines         9749     9750    +1     
========================================
+ Hits          6382     6383    +1     
  Misses        3367     3367           

Flag	Coverage Δ
python-3.13	65.46% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.