[ptr727]

Summary

Fixes the two findings promotion PR #3778's review raised against backlog-burndown.

• Checked remote delete for an abandoned group's pushed branch. "Cleanup Is the Orchestrator's" used to report such a branch to the maintainer rather than delete it, so the next round kept deferring the re-formed group. Per the maintainer's answer on backlog-burndown: cleanup leaves an abandoned group's claim and its pushed remote branch standing #1274, the orchestrator now records the local tip before removal. It deletes the remote branch only when that tip is an ancestor of origin/develop, the live remote tip equals it, and no open pull request uses the branch as its head or base. The delete is named as an outward-facing write under GOVERNANCE.md "Repository Boundaries and Write Safety". The claim is released only once reading the ref back confirms the delete. Where a check fails, the claim is kept and the group stops for the maintainer. "Bounding the Wait on a Worker" and "The Round" step 6 are aligned with it.
• The claim-lookup command's <branch> placeholder. One sentence now states that <branch> is replaced with the branch name being looked up before the command runs.

Generated trees regenerated with scripts/build_dist.py, and --check passes.

Local review

Three local-strict-review passes, the last over the pushed head with no findings. The first two raised findings that were fixed in this branch, apart from the two below.

• Deferred to Give the Orchestrator a Cleanup Path for an Abandoned Branch With No Pull Request #3733, class introduced (design, reuse): scripts/pr_review.py cleanup already holds a verify-then-delete in code but refuses an unmerged branch. A branch-named mode there, which Give the Orchestrator a Cleanup Path for an Abandoned Branch With No Pull Request #3733 already proposes for the never-pushed case, should take the pushed case too and replace this skill's hand-typed remote delete. That needs files outside this change.

Pre-existing findings, not filed

• .agents/skills/backlog-burndown/SKILL.md "Cleanup Is the Orchestrator's", the line "git branch -d refuses rather than discards where that check was wrong", class pre-existing: for a branch with an upstream, git branch -d checks against the upstream rather than develop, so it is no backstop there. The remote delete added here no longer relies on it, since it carries its own git merge-base --is-ancestor check.

Closes on promotion: #1274

🤖 Generated with Claude Code

[ptr727]

A recorded local strict-review pass covers head 4e0e438079812998c1e9430cbba64321f47cafb5, 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

• Source mode: Auto
• Repository: ptr727/ProjectTemplate
• Branch: feature/burn-3778-fix
• File: .coderabbit.yaml
• Commit: 8c15548
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: de78f81d-54fc-4e13-8186-9cf73a1b4f50

• 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

[Copilot]

🟡 Changes recommended

Active moderate and critical findings remain in the cleanup procedure, including stale-tip checks, unsafe deletion ordering, incomplete pull-request pagination, and a non-atomic remote delete.

4 open issues

• Paginate both open pull request lookups · New
• Make remote deletion conditional on the checked tip · New
• Confirm remote deletion before removing local cleanup state · New
• Refresh origin/develop before checking branch containment · New

What changed in this PR

This pull request updates backlog-burndown to clean up abandoned groups with pushed branches through checked remote deletion, clarifies claim-lookup placeholder substitution, and regenerates the distributed skill files.

Changes:

• Adds ancestry, remote-tip, open pull-request, deletion, and read-back checks for pushed branches.
• Clarifies that <branch> is replaced before running the claim lookup.
• Regenerates the GitHub and Claude skill copies and updates the source digest.
• The implementation still has unresolved issues involving stale remote state, cleanup retryability, pull-request lookup completeness, and conditional deletion safety.

Review status: This pull request requires changes.

File	Description
.github/​skills/​backlog-burndown/​SKILL.md	Generated GitHub skill distribution.
.claude-plugin/​fleet-skills/​skills/​backlog-burndown/​SKILL.md	Generated Claude fleet skill distribution.
.claude-plugin/​fleet-skills/​.source-digests/​backlog-burndown	Updated source digest for the generated skill.
.agents/​skills/​backlog-burndown/​SKILL.md	Canonical backlog cleanup and claim-lookup procedure updates.

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.34%. Comparing base (c24836e) to head (8c15548).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #3779   +/-   ##
========================================
  Coverage    82.34%   82.34%           
========================================
  Files           28       28           
  Lines         9454     9454           
========================================
  Hits          7785     7785           
  Misses        1669     1669           

Flag	Coverage Δ
python-3.13	82.34% <ø> (ø)

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.

[ptr727]

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