[ptr727]

Refs #3531

This is a promotion fix for review thread PRRT_kwDOQ5caqM6xQ88w on #3572. That thread is the P1 on unattended-handoff picker step 1, where the full-page check no longer said which reads it applies to.

The #3570 sentence-length sweep split sentences in unattended-handoff, backlog-burndown, and session-handoff. I compared each split in git show 8d390acf with its original. This change repairs each one that was ungrammatical or had lost its subject, an antecedent, or the scope of a qualifier or reason. Each repaired sentence keeps the original meaning and is 25 words or fewer. Text #3570 did not change is left alone, apart from line rewrapping.

Splits repaired

unattended-handoff:

• Picker step 1, the full-page check (the P1). Picker step 3 and the worker's decision read are the subject again. The bold term is kept, so step 1's other references still resolve. The check is defined, and the re-read applies only where a read returns a full page.
• Auto-resolvable, "Nothing on it waits on the maintainer". "And no comment asks..." had come loose after the canonical-sweep aside. It is joined back to the label clause, and the aside now names canonical-sweep rather than "the last".
• Auto-resolvable, "Nothing has worked it". The open-handoff and claim-comment clauses are one sentence again, so "That is because each marks work that has no pull request yet" covers both of them.
• Worker step 1. "Follow it" is back inside the "Where a parking comment names a decision issue" condition.
• Worker step 3, narrowed round. "so each closes" had lost its antecedent. It now reads "so each issue closes".
• Worker step 3, unnarrowed round. The hand-off to merge-and-release is back under "where the round is not narrowed".
• Parking step 2. "and the handoff it belongs to" no longer reads as an object of "would do to". "Comment onto it" is back under the existing-issue condition ("In that case").

backlog-burndown:

• Dispatching a Worker, "No worker inherits another's worktree". The "That is why" reason covers both dispositions again. The third clause carries it as "for the same reason".
• Assembling the Promotion Body. "amend the body when it lands" had "it" binding to the body. It now reads "when the fix lands".

session-handoff:

• The Attended Session step 6. Handing the lane back is back under "Where work remains" ("In that case").
• Closing a Session step 4. The elliptical "Never the round's state..." had come to follow the example sentence. "What goes there is saved as the session's final act" now comes before it.
• Running the Chain, unlisted forks. The example sentence had moved directly in front of "Missing the label there". That made "there" mean only upstream-contribution forks, and the new clause had lost the condition. The scope is unlisted forks again, and the read clause and the new clause are one sentence again, as in the original.

Local review

local-strict-review ran three recorded passes at the frontier tier. The first raised 4 findings and the second raised 1, all fixed in the two edit rounds. The third raised 1, recorded below.

Open introduced findings

• .agents/skills/unattended-handoff/SKILL.md worker step 1, class introduced. The rejoined sentence reads "since it unblocked the lane" where the original read "since it is what unblocked the lane". The meaning is identical, and the reviewer confirmed that. Bring the Handoff Skills' Sentences to 25 Words or Fewer #3570 split this sentence, so its second half is Bring the Handoff Skills' Sentences to 25 Words or Fewer #3570's text. Rejoining it at the original wording runs to 27 words. The reviewer's alternative, "Follow that answer, since...", would restore the split and leave "Follow" outside the condition, which is the scope gap this repair closes. Disposition: declined on that evidence.

Dropped findings

• .agents/skills/session-handoff/SKILL.md "Running the Chain", unlisted-fork paragraph, class style. Copilot's round on 0b58dfed raised no thread, but its overview mentions "a minor non-blocking grammar nit ... regarding the missing-label condition". The nearest candidate is the participle "Missing the label there, a read prints...". That phrase is identical in git show 8d390acf^:.agents/skills/session-handoff/SKILL.md line 359. Neither Bring the Handoff Skills' Sentences to 25 Words or Fewer #3570 nor this change altered it beyond rewrapping, so it is out of this change's scope.

🤖 Generated with Claude Code

[ptr727]

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

[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/3531-handoff-split-grammar
• File: .coderabbit.yaml
• Commit: 0b58dfe
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: dd0e46bb-9f4b-472a-9a47-60dbbb4d0dce

• 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]

🟢 Approval recommended

The affected canonical skills were repaired, both generated distributions were regenerated, and the corresponding source digests were refreshed, leaving only a minor non-blocking grammar nit.

No open issues 🎉

What changed in this PR

This pull request restores subjects, antecedents, and conditional scope lost when the handoff skills were split into shorter sentences. It updates the canonical skill sources, generated distributions, and source digests.

Changes:

• Repairs sentence scope and grammatical references in unattended-handoff, session-handoff, and backlog-burndown.
• Regenerates the GitHub Copilot and Claude Code skill distributions.
• Refreshes the three corresponding source digests.
• A minor grammar nit remains in the canonical session-handoff source regarding the missing-label condition.

File	Description
.github/​skills/​unattended-handoff/​SKILL.md	Generated GitHub Copilot distribution of the repaired skill.
.github/​skills/​session-handoff/​SKILL.md	Generated GitHub Copilot distribution of the repaired skill.
.github/​skills/​backlog-burndown/​SKILL.md	Generated GitHub Copilot distribution of the repaired skill.
.claude-plugin/​fleet-skills/​skills/​unattended-handoff/​SKILL.md	Generated Claude Code distribution of the repaired skill.
.claude-plugin/​fleet-skills/​skills/​session-handoff/​SKILL.md	Generated Claude Code distribution of the repaired skill.
.claude-plugin/​fleet-skills/​skills/​backlog-burndown/​SKILL.md	Generated Claude Code distribution of the repaired skill.
.claude-plugin/​fleet-skills/​.source-digests/​unattended-handoff	Updated source digest for the generated skill.
.claude-plugin/​fleet-skills/​.source-digests/​session-handoff	Updated source digest for the generated skill.
.claude-plugin/​fleet-skills/​.source-digests/​backlog-burndown	Updated source digest for the generated skill.
.agents/​skills/​unattended-handoff/​SKILL.md	Canonical repairs to picker, worker, parking, and decision-following instructions.
.agents/​skills/​session-handoff/​SKILL.md	Canonical repairs to lane handoff, session closing, and chain instructions; includes the remaining missing-label grammar nit.
.agents/​skills/​backlog-burndown/​SKILL.md	Canonical repairs to worktree disposition and promotion-body instructions.

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.60%. Comparing base (130499c) to head (0b58dfe).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #3573   +/-   ##
========================================
  Coverage    81.60%   81.60%           
========================================
  Files           26       26           
  Lines         9070     9070           
========================================
  Hits          7402     7402           
  Misses        1668     1668           

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

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.