Pass --repo on Documented Handoff Commands and Guard Handoff Reads Against a Full Page by ptr727 · Pull Request #1856 · ptr727/ProjectTemplate

ptr727 · 2026-09-26T03:01:32Z

Fixes the three defects #1847 raised against the handoff and burndown skills when a downstream repository carried them.

--repo on every documented command. In unattended-handoff and session-handoff, every handoff.py invocation (chain, new, resume) and every gh issue view/gh issue list read now passes --repo. A session run from a hub checkout otherwise reads the hub's issues. The picker's handoff.py new also names the --title and --body-file flags it requires.
No ranking of a truncated list. The open-handoff reads in both skills, and the picker's backlog read, now re-read with a larger limit whenever a full page comes back.
handoff plus blocked. backlog-burndown "Ranking" now says the handoff rule controls for a link that also carries blocked, so that link is neither ranked nor counted.

The generated trees were regenerated and build_dist.py --check is clean. A local-strict-review pass is recorded.

Closes on promotion: #1847

🤖 Generated with Claude Code

Every documented handoff.py and gh issue invocation in unattended-handoff and session-handoff now carries --repo, since a session run from a hub checkout otherwise reads the hub's issues. Both open-handoff reads now re-read with a larger limit when a full page comes back rather than ranking a truncated list. backlog-burndown states that the handoff rule controls for a link that also carries blocked. Closes on promotion: #1847 Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

…Page Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

coderabbitai · 2026-09-26T03:01:41Z

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: 048a607e-074a-4e88-907c-fa328c2c79c7

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

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

Copilot

Copilot review overview

🟢 Approval recommended

The changes address the documented downstream-usage defects, with only minor documentation reflow nits remaining.

Review effort: Lite
Findings: 2

Open (2)

Start the blocked-link sentence on a new wrapped line · New
Wrap the gh issue view sentence for consistent hard wrapping · New

What changed in this PR

This PR updates the handoff- and burndown-related Skills documentation so scripted and manual gh/handoff.py reads consistently target the intended downstream repository and do not silently rank or count from a truncated first page of results.

Changes:

Document --repo on handoff.py invocations and gh issue list/view reads to avoid accidentally querying the hub repo when running from a hub checkout.
Add guidance to re-read with a larger --limit when an issues listing returns a full page, avoiding ranking decisions based on a potentially truncated result set.
Clarify that handoff takes precedence over blocked for counting/ranking in backlog-burndown.

File	Description
.agents/skills/unattended-handoff/SKILL.md	Canonical doc updates: add `--repo` and truncated-page re-read guidance for picker/worker steps.
.agents/skills/session-handoff/SKILL.md	Canonical doc updates: add `--repo` to reads and guard against ranking from a full page.
.agents/skills/backlog-burndown/SKILL.md	Canonical doc update: specify `handoff` overrides `blocked` for ranking/counting.
.github/skills/unattended-handoff/SKILL.md	Regenerated distributed copy of the `unattended-handoff` skill doc.
.github/skills/session-handoff/SKILL.md	Regenerated distributed copy of the `session-handoff` skill doc.
.github/skills/backlog-burndown/SKILL.md	Regenerated distributed copy of the `backlog-burndown` skill doc.
.claude-plugin/fleet-skills/skills/unattended-handoff/SKILL.md	Regenerated plugin copy of the `unattended-handoff` skill doc.
.claude-plugin/fleet-skills/skills/session-handoff/SKILL.md	Regenerated plugin copy of the `session-handoff` skill doc.
.claude-plugin/fleet-skills/skills/backlog-burndown/SKILL.md	Regenerated plugin copy of the `backlog-burndown` skill doc.
.claude-plugin/fleet-skills/.source-digests/unattended-handoff	Updated digest for regenerated plugin `unattended-handoff` skill.
.claude-plugin/fleet-skills/.source-digests/session-handoff	Updated digest for regenerated plugin `session-handoff` skill.
.claude-plugin/fleet-skills/.source-digests/backlog-burndown	Updated digest for regenerated plugin `backlog-burndown` skill.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

Copilot

Copilot review overview

🟡 Changes recommended

The new truncation-handling guidance is currently unbounded in two places and should be capped to avoid repeated reads or selecting from an incomplete set.

Review effort: Lite
Findings: 2

Open (2)

Retry loop lacks a max limit and can return partial counts · New
Unbounded issue-list retry can loop on capped results · New

Resolved since last review (2)

Wrap the gh issue view sentence for consistent hard wrapping
Start the blocked-link sentence on a new wrapped line

…and Overnight Fixes to Main (#1850) Promotes `develop` to `main`. ## Carried - #1846: exits `repo-config/configure.sh` early, writing nothing, for a repository whose registry `status` is `archived`. - #1848: fails the validator's C# unit test step when the run wrote no non-empty Cobertura report, clearing `./coverage` first. It declines #1134's `ref` input with evidence, since a bare checkout already validates `github.sha`, and D1.2 now says so. #1134's third gap moved to #1800. - #1851: asserts that the two planted registration defects are themselves reported. - #1854, #1858: correct #1846's test docstrings, which claimed the script makes no `gh` call before the archived check in cases where it does. Raised by Copilot on this pull request. - #1856: passes `--repo` on documented handoff commands and guards handoff reads against a full page, per #1847. - #1859, #1863: fall back to the default search path when `PATH` is unset in #1848's and #1846's test harnesses. Raised by Copilot on this pull request. - #1861: scopes `VerifyReferenceAotCompatibility` to an AOT publish in `dotnet-codestyle`, per #1857. - #1867: establishes the `PATH` order `tool_shadow_path` names, per #1644. - #1870: makes the installer's dirty-checkout tests independent of the real checkout's state, per #1641. - #1873: pins and decodes git's quoting in `repo_gate.py`'s `ls-files` read, per #1580 and #1872. - #1878: folds typographic punctuation in `pr_review.py reply --match`, per #1299. - #1883: routes `configure.sh`'s `ruleset_id()` through `jqr`, per #1253. - #1885: states the pin comment as the release tag and defines `$/`, per #1805. - #1888: distinguishes `./` from `$/` resolution in the pin rule's prose, per #1886. - #1893: drops the issue reference from `repo-config/README.md`'s archived-exemption note, which Copilot flagged on six rounds of this pull request. - #1895: describes IL3058 in `dotnet-codestyle` as a referenced assembly lacking `IsAotCompatible` metadata set to `true`, and drops the unversioned package examples. Raised by CodeRabbit on this pull request. Callers that pin a hub release get the new C# check on their next pin bump. A test project that runs `dotnet test --coverage` without writing a report now fails its step rather than passing silently. Closes #1134 Closes #1847 Closes #1857 Closes #1644 Closes #1641 Closes #1580 Closes #1872 Closes #1299 Closes #1253 Closes #1805 Closes #1886 🤖 Generated with [Claude Code](https://claude.com/claude-code)  ## Summary by CodeRabbit * **Bug Fixes** * Backlog counts and rankings now consistently exclude handoff issues, including those also marked blocked. * Repository configuration commands now exit without writing when a repository is archived. * Review-thread matching handles typographic punctuation, and no-match responses report the unresolved-thread count. * Tool setup handles PATH entries more precisely, and repository checks report unusual file paths without crashing. * **Reliability** * Validation now fails when C# or Python tests produce no coverage report. * Agent setup can use an explicit dirty-checkout override. * Workflow and repository guidance clarifies reference resolution, release-tag pinning, and AOT configuration.

ptr727 and others added 3 commits September 25, 2026 19:57

Pass --repo on the Picker's Backlog Read and Guard It Against a Full …

a592256

…Page Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

Name the Required Title and Body Flags on the Picker's handoff.py new

ddceb5a

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

Copilot AI lite review requested due to automatic review settings September 26, 2026 03:01

Copilot started reviewing on behalf of ptr727 September 26, 2026 03:02 View session

Copilot AI reviewed Sep 26, 2026

View reviewed changes

Comment thread .agents/skills/session-handoff/SKILL.md Outdated

Comment thread .agents/skills/unattended-handoff/SKILL.md Outdated

ptr727 and others added 2 commits September 25, 2026 20:07

Rewrap the Paragraphs the --repo Edits Left Ragged

11aa0af

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

Apply the Full-Page Check to Every Re-Derived Count

5af642e

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

Copilot AI review requested due to automatic review settings September 26, 2026 03:09

Copilot started reviewing on behalf of ptr727 September 26, 2026 03:10 View session

Copilot AI reviewed Sep 26, 2026

View reviewed changes

Comment thread .agents/skills/session-handoff/SKILL.md

Comment thread .agents/skills/unattended-handoff/SKILL.md

ptr727 merged commit eda453a into develop Sep 26, 2026
9 checks passed

ptr727 deleted the feature/auto-1847 branch September 26, 2026 03:17

This was referenced Sep 26, 2026

Session Handoff [auto-1847]: Fix Three Defects in the Handoff and Burndown Skills Raised on a Downstream Carry (#1847) #1855

Closed

Promote the C# Coverage Report Check, Archived-Repository Exemption, and Overnight Fixes to Main #1850

Merged

Navigation Menu

Repository navigation

Pass --repo on Documented Handoff Commands and Guard Handoff Reads Against a Full Page - #1856

Pass --repo on Documented Handoff Commands and Guard Handoff Reads Against a Full Page#1856
ptr727 merged 5 commits into
developfrom
feature/auto-1847

ptr727 commented Sep 26, 2026

Uh oh!

coderabbitai Bot commented Sep 26, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

ptr727 commented Sep 26, 2026

Uh oh!

coderabbitai Bot commented Sep 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟢 Approval recommended

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟡 Changes recommended

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

coderabbitai Bot commented Sep 26, 2026 •

edited

Loading