Repository navigation
Session Handoff [sentence-length]: Hub Code Comment Sweep Next, Hub-Only Markdown Sweep Shipped in 2.0.1251 #3575
Copy link
Copy link
Closed
Labels
handoffA link in the session handoff chain, one open issue per trackA link in the session handoff chain, one open issue per track
Description
Next steps, in priority order
- Sweep the hub code comments and docstrings. 1662 hits at
main77d7ca35, all outside Markdown, mostly underscripts/andtests/. Re-count withpython3 scripts/prose_lint.py --check sentence-lengthand group by file before slicing. A comment-only pull request needs thecommentslabel andPROSE_ALLOW_COMMENTS=1on its commits, as Split the Over-Length Sentences in the Secrets Shell Library Comments #3581 did. Also align the "Copies left for the comment sweep" pairs listed in the bodies of Split the Over-Length Sentences in scripts/README.md #3594, Split the Over-Length Sentences in the Host Setup Docs #3595, and Split the Over-Length Sentences in the Spec and Repo-Config Docs #3597 with their Markdown splits. Use the Say What a Carrier's Local Strict Review Does With Sentence-Length Hits on Hub-Verbatim Text #3531 brief and its round 3 and round 4 addenda as the worker brief. Done: the whole tree reads zero. - Make
sentence-lengtha default check once the tree reads zero. Delete thecomment-and-doc-styletext calling it opt-in "because the existing corpus predates the cap". Done: Say What a Carrier's Local Strict Review Does With Sentence-Length Hits on Hub-Verbatim Text #3531 closes on that promotion. - Move the Dispatch-Refresh Note Out of the Docker Clause in branching-and-release-model #3588: move the dispatch-refresh note out of the Docker clause in
branching-and-release-model. One small skill edit, sharing no file with step 1. - Make pr_review.py and Merge Gate Item 2 Agree on an Earlier-Head Copilot Refusal #3571: make
pr_review.pyand Merge Gate item 2 agree on a Copilot refusal that landed on an earlier head. Independent of the steps above. - Nothing Checks That the Host CLAUDE.md Blocks Stay in Sync With Their Governance Sections #3601, filed this round: nothing checks that the host
CLAUDE.mdblocks stay in sync with their governance sections. It lists three choices and needs a design pick before any code.
External blockers
- Host-safety kit is stale on this machine. Split the Over-Length Sentences in the Agent-Safety Markdown #3596 changed
claude-md-safety.mdandclaude-md-fleet.md, soinstall.py --reportreads not current after 2.0.1251. Only the maintainer re-runs that installer, since it rewrites hooks and settings. Re-check it at resume. - Promotion coverage gaps. Copilot's table on Promote Develop: Split the Over-Length Sentences in the Hub-Only Markdown #3600 named 31 of 42 files. The maintainer accepted feature-pull-request coverage, recorded on Promote Develop: Split the Over-Length Sentences in the Hub-Only Markdown #3600. Expect the same on each sweep promotion, and bring it to the maintainer each time.
- Copilot refuses some large diffs with "default exclusions" (Split the Over-Length Sentences in TODO.md #3591, Split the Over-Length Sentences in scripts/README.md #3594). A refusal leaves Merge Gate item 2 to the maintainer. This round's acceptance covered only this round's slices.
Internal dependencies
- Step 1 before step 2. The gate fails on any remaining hit.
- Within a round, no two slices share a file. A comment that restates a Markdown sentence follows the Markdown's split. So a slice takes the pair list from the PR bodies named in step 1.
- A skills-touching slice (step 3, or any comment in a generated tree) holds the single
.source-digestline, so at most one runs at a time.
State
Re-derive each of these. Do not trust the copy here.
main77d7ca35(promotion Promote Develop: Split the Over-Length Sentences in the Hub-Only Markdown #3600) anddevelop0d4e6f1dat close. Release 2.0.1251 published frommain(run 40883205338, success).- No worktree or branch of this round remains. Every feature branch was cleaned with
pr_review.py cleanup.cleanup --sweeplists only other sessions' worktrees, kept as too recent. - This machine's skills snapshot reads current at
77d7ca35. The host-safety kit does not, per the blocker above. - Open issues in scope: Say What a Carrier's Local Strict Review Does With Sentence-Length Hits on Hub-Verbatim Text #3531 (this track), Move the Dispatch-Refresh Note Out of the Docker Clause in branching-and-release-model #3588, Make pr_review.py and Merge Gate Item 2 Agree on an Earlier-Head Copilot Refusal #3571, Nothing Checks That the Host CLAUDE.md Blocks Stay in Sync With Their Governance Sections #3601, and Carry a CLI-Usage Test Harness So Fleet Repos Stop Copying the Hub's #3502, the held carried CLI harness proposal.
The parked decision queue
Zero open decision issues at close. The maintainer answered this round's questions in session. They granted develop merges for this session's slices, Sonnet workers with opus review in two waves of four, and Markdown-only scope. They accepted the recorded local pass for Copilot-refused slices, and feature-pull-request coverage on #3600. They chose merge, Skills refresh, and release.
What the last round did
- Hub-only Markdown sweep, done. Nine pull requests took the hub-only Markdown from 1518 hits to zero: Split the Over-Length Sentences in scripts/README.md #3594, Split the Over-Length Sentences in TODO.md #3591, Split the Over-Length Sentences in reusable-workflows and fleet-map #3592, Split the Over-Length Sentences in the Hub Procedure Docs #3593, Split the Over-Length Sentences in the Host Setup Docs #3595, Split the Remaining Over-Length Sentences in the Host Setup Docs #3598, Split the Over-Length Sentences in the Agent-Safety Markdown #3596, Split the Over-Length Sentences in the Spec and Repo-Config Docs #3597, and Split the Over-Length Sentences in the Docs, Catalog READMEs, and HISTORY #3599. Promotion Promote Develop: Split the Over-Length Sentences in the Hub-Only Markdown #3600 merged to
main. - Filed Nothing Checks That the Host CLAUDE.md Blocks Stay in Sync With Their Governance Sections #3601 for the missing host-block sync check.
- Recorded this round's sweep lessons as a round 4 addendum to the Say What a Carrier's Local Strict Review Does With Sentence-Length Hits on Hub-Verbatim Text #3531 brief.
What not to repeat
- Merging on
coverage=none. Split the Over-Length Sentences in the Spec and Repo-Config Docs #3597 merged with its local pass recorded but never attested, and Copilot had reviewed only an earlier head. Readcoverage=on the status line before every merge. - Letting a worker merge. The auto-mode classifier refused two workers' develop merges as "Merge Without Review". Workers hand back at ready, and the orchestrator merges.
- A stop rule with room for a budget. Three of eight workers stopped with findings on new sentences unfixed, reading the per-sentence rule as a pass cap. Brief "no pass cap" explicitly at dispatch.
- Splitting a long list into two partial lists. Reviewers read "the first three ... the last two" as closed. Convert to a lead sentence plus bullets.
New learnings
- Recorded in the Say What a Carrier's Local Strict Review Does With Sentence-Length Hits on Hub-Verbatim Text #3531 round 4 brief addendum: no pass cap, orchestrator merges, attest before ready, and Copilot refusals on large Markdown. It also covers lists for list sentences, the copies rule, and settling a contested condition against the code.
Metadata
Metadata
Assignees
Labels
handoffA link in the session handoff chain, one open issue per trackA link in the session handoff chain, one open issue per track
Activity
ptr727 commentedon Oct 28, 2026
Added to this track, to be ranked into the next round's next steps: #3577, the 23
sentence-lengthhits inlib/sh/secrets.shatb72f50f3. The file is carried byte-locked intolib/fleet/sh, so every carrier's first carry pull request reports these hits and cannot fix them. That ranks it with the skill-source sweep (step 2) rather than with the hub-only text (step 3). The slice is that one file, and it shares no file with the split audits in flight. The Blog session filed it while carryinglib/fleet/sh(Blog #561).ptr727 commentedon Oct 28, 2026
Round 1, closed 2026-10-28: Session Handoff [sentence-length]: Split Audit of the Swept GOVERNANCE, AGENTS and WORKFLOW Text Next
Next steps, in priority order
main: Bring GOVERNANCE.md Sentences to 25 Words or Fewer #3551 (GOVERNANCE.md), Bring AGENTS.md and copilot-instructions.md Sentences to 25 Words or Fewer #3559 (AGENTS.md,.github/copilot-instructions.md), Bring WORKFLOW.md Sentences to 25 Words or Fewer #3561 (WORKFLOW.md). Compare every split each made with its original (git show <squash>) and repair any that lost meaning. Done: one PR per source file group, each split listed in its body, with a Copilot round whose table names every changed file. Each audit of a later sweep found 12 to 16 broken splits, so expect some here..agents/skills/: 484 hits across 29 files. The largest are local-strict-review, comment-and-doc-style (with itsreferences/), skill-lifecycle, repo-worktree,branching-and-release-model/references/release-publish-mechanics.md, and merge-and-release. Done: zero hits per file. Each brief requires split-by-split comparison before review, per the Say What a Carrier's Local Strict Review Does With Sentence-Length Hits on Hub-Verbatim Text #3531 brief comment.scripts/README.md,TODO.md,docs/,reports/). Lower priority, since no carrier copies it, but the ruling covers the whole hub.sentence-lengtha default check once the tree reads zero. Delete thecomment-and-doc-styletext calling it opt-in "because the existing corpus predates the cap". Done: Say What a Carrier's Local Strict Review Does With Sentence-Length Hits on Hub-Verbatim Text #3531 closes on that promotion.pr_review.pyand Merge Gate item 2 agree on a Copilot refusal that landed on an earlier head. Independent of the steps above.External blockers
Internal dependencies
GOVERNANCE.md,AGENTS.md,CODESTYLE.md, orWORKFLOW.mdsection included by a skill regenerates that skill. Read the include markers (grep -rhoE 'include: [A-Z]+\.md > [^>]*' .agents/skills) before grouping.State
Re-derive each of these. Do not trust the copy here.
mainwas1abb46f2anddevelopb72f50f3at close. No promotion is open, no release was dispatched.pr_review.py cleanup --sweepreported four detached worktrees older than 24 hours from other sessions:ProjectTemplate-lint-homepage,hub-w1493, and two Financial-Modeling scratch hubs. They were not applied, since removal needs the maintainer.main.The parked decision queue
Zero open
decisionissues at close. #3502 is a held design proposal, not a parked decision.What the last round did
Driven by the backlog-burndown skill over four rounds and four promotions. Promotion #3544 went to
main65adbf17, #3553 to54f477a6, #3562 toa5bfc9c2, and #3572 to1abb46f2.repoOwnedTrees. Tell a Transient Copilot Fetch Error From the Quota Body Before Advising No Re-Request #3510 reads the run log. State That a Python Profile Belongs to a Declared Directory, and When a Repo May Declare Several #3492 gives one build directory per deliverable, with lint-only Python beside its component. Say What the Registry's Python Profile Means Once the Profile Is Per Directory #3556 drops the registrypythonprofile. Say What a Carrier's Local Strict Review Does With Sentence-Length Hits on Hub-Verbatim Text #3531 says fix the text.What not to repeat
prose_lint.py --check sentence-length F && gh ....gh pr edit ... >/dev/null. Read every write's response.cut -c1-500hid the end ofWORKFLOW.md:142, and the orchestrator wrongly told a worker that D1.2 did not state a condition it does state.New learnings
coverage=localin the tool but unmet in the rule: Make pr_review.py and Merge Gate Item 2 Agree on an Earlier-Head Copilot Refusal #3571.[-]Session Handoff [sentence-length]: Split Audit of the Swept GOVERNANCE, AGENTS and WORKFLOW Text Next[/-][+]Session Handoff [sentence-length]: Skill-Source Sweep and lib/sh/secrets.sh Next, Split Audits Shipped in 2.0.1234[/+]ptr727 commentedon Oct 28, 2026
Closing answers for round 2, from the maintainer on 2026-10-28:
cleanup --sweep --applyremovedProjectTemplate-lint-homepageandhub-w1493. It kept the two dirty Financial-Modeling scratch hubs.install.py --reportat resume rather than assuming it reads CURRENT.Parked decision queue: zero open
decisionissues.ptr727 commentedon Oct 29, 2026
Round 2, closed 2026-10-28: Session Handoff [sentence-length]: Skill-Source Sweep and lib/sh/secrets.sh Next, Split Audits Shipped in 2.0.1234
Next steps, in priority order
.agents/skills/: 484 hits across 29 files atdevelopb82a4a0d. The largest are local-strict-review (54), comment-and-doc-style (48, plus 24 inreferences/line-endings.md), skill-lifecycle (36), repo-worktree (35),branching-and-release-model/references/release-publish-mechanics.md(34), and merge-and-release (32). Two known leftovers sit there: 12 hits in agent-conduct outside its include regions, and one 28-word sentence in the workflow-ci-contractreferences/d-guarantees.mdpreamble. Done: zero hits per file. Each worker compares every split with its original before review, per the Say What a Carrier's Local Strict Review Does With Sentence-Length Hits on Hub-Verbatim Text #3531 brief comment.lib/sh/secrets.sh. The file is carried byte-locked intolib/fleet/sh, so every carrier's first carry PR reports them and cannot fix them. One-file slice, sharing no file with step 1. Done:prose_lint.py --check sentence-length libexits 0.scripts/README.md,TODO.md,docs/,reports/). Lower priority, since no carrier copies it, but the ruling covers the whole hub. Re-count before slicing.sentence-lengtha default check once the tree reads zero. Delete thecomment-and-doc-styletext calling it opt-in "because the existing corpus predates the cap". Done: Say What a Carrier's Local Strict Review Does With Sentence-Length Hits on Hub-Verbatim Text #3531 closes on that promotion.pr_review.pyand Merge Gate item 2 agree on a Copilot refusal that landed on an earlier head. Independent of the steps above.External blockers
Internal dependencies
grep -rhoE 'include: [A-Z]+\.md > [^>]*' .agents/skills) before grouping. A skill-source slice edits text outside the include regions, so its only generated files are that skill's own copies and digest.State
Re-derive each of these. Do not trust the copy here.
main4aae620c(promotion Promote Develop: Restore Meaning Lost in the GOVERNANCE, AGENTS, and WORKFLOW Sentence Splits #3580) anddevelopb82a4a0dat close. Release 2.0.1234 published frommain(run 40774865902, success).pr_review.py cleanup --sweepstill lists two detached worktrees from other sessions it would remove (ProjectTemplate-lint-homepage,hub-w1493), and keeps two dirty Financial-Modeling scratch hubs. Removal needs the maintainer's go-ahead.4aae620c. The host-safety kit reports STALE (installed from hubf60f57d). Its installer rewrites hooks and settings, so only the maintainer runs it.lib/fleet/sh(Blog Ask for a blocked decision instead of reporting it #561). Carry an Indent Setting With lib/sh So Tab-Indented Carriers Pass shfmt #3576, a.editorconfigforlib/sh, went to thedefaulttrack (Session Handoff [default]: #3671 Thread-Aware Held Wait Released in 2.0.1286, Carried Backlog Next #2361) as a comment.The parked decision queue
Zero open
decisionissues at close.What the last round did
main. Restore Meaning Lost in the GOVERNANCE and AGENTS Sentence Splits #3579 repaired 15 broken splits from Bring GOVERNANCE.md Sentences to 25 Words or Fewer #3551 and Bring AGENTS.md and copilot-instructions.md Sentences to 25 Words or Fewer #3559 inGOVERNANCE.md,AGENTS.md, and.github/copilot-instructions.md. Seven had a reason left on one half of a split. Restore Antecedents, Reasons, and Scope Lost in the WORKFLOW.md Sentence Splits #3578 repaired 10 of the 107 splits Bring WORKFLOW.md Sentences to 25 Words or Fewer #3561 made inWORKFLOW.md. Six had lost an antecedent.develop. Promotion Promote Develop: Restore Meaning Lost in the GOVERNANCE, AGENTS, and WORKFLOW Sentence Splits #3580 merged tomainwith all 22 files in Copilot's table.default, at the Blog session's request.What not to repeat
python3on this host.build_dist.pyandpr_review.pyimportmarkdown_it, which the system Python lacks. Useuv run --no-project --script scripts/<name>.py.prose_lint.py --check sentence-length F && gh .... The promotion worker's first body draft failed the gate, and the gate stopped it.New learnings
[-]Session Handoff [sentence-length]: Skill-Source Sweep and lib/sh/secrets.sh Next, Split Audits Shipped in 2.0.1234[/-][+]Session Handoff [sentence-length]: Hub-Only Markdown and Comment Sweep Next, Skill-Source Sweep Shipped in 2.0.1242[/+]ptr727 commentedon Oct 29, 2026
Closing answers for round 3, from the maintainer on 2026-10-28:
Parked decision queue: zero open
decisionissues.ptr727 commentedon Oct 29, 2026
Round 3, closed 2026-10-29: Session Handoff [sentence-length]: Hub-Only Markdown and Comment Sweep Next, Skill-Source Sweep Shipped in 2.0.1242
Next steps, in priority order
main13c2b8b1. The largest arescripts/README.md(303),TODO.md(178),docs/reusable-workflows.md(126),STANDUP.md(83),docs/fleet-map.md(78),AUDIT.md(73),docs/host-setup.md(70), andREADME.md(61). Re-count before slicing. Done: zero hits per file. Use the Say What a Carrier's Local Strict Review Does With Sentence-Length Hits on Hub-Verbatim Text #3531 brief and its round 3 addendum as the worker brief.scripts/andtests/. A comment-only pull request needs thecommentslabel andPROSE_ALLOW_COMMENTS=1on its commits, as Split the Over-Length Sentences in the Secrets Shell Library Comments #3581 did. Done: the whole tree reads zero.sentence-lengtha default check once the tree reads zero. Delete thecomment-and-doc-styletext calling it opt-in "because the existing corpus predates the cap". Done: Say What a Carrier's Local Strict Review Does With Sentence-Length Hits on Hub-Verbatim Text #3531 closes on that promotion.branching-and-release-model. One small skill edit, sharing no file with steps 1 and 2.pr_review.pyand Merge Gate item 2 agree on a Copilot refusal that landed on an earlier head. Independent of the steps above.External blockers
Internal dependencies
grep -rhE '^<!--\s*include:' .agents/skills) before grouping. Keep a doc that feeds a skill in the same slice as that skill's regenerated copies.State
Re-derive each of these. Do not trust the copy here.
main13c2b8b1(promotion Promote Develop: Split the Over-Length Sentences in the Skill Sources and the Secrets Shell Library #3589) anddevelop7364e141at close. Release 2.0.1242 published frommain(run 40804010827, success).pr_review.py cleanup. Thecleanup --sweepreport lists only other sessions' worktrees, kept as dirty or too recent.13c2b8b1, and the host-safety kit reads CURRENT.The parked decision queue
Zero open
decisionissues at close. The maintainer answered this round's two questions in session. They accepted feature-pull-request coverage on #3589, and chose merge, Skills refresh, and release.What the last round did
lib/sh/secrets.shfrom 507 hits to zero: Split the Over-Length Sentences in the Local Review, Agent Conduct, and Check This Repo Skills #3584, Split the Over-Length Sentences in the Comment and Doc Style Skill #3586, Split the Over-Length Sentences in the Repo Procedure Skills #3587, Split the Over-Length Sentences in the Git and Worktree Skills #3582, Split the Over-Length Sentences in the Release Skills #3585, Split the Over-Length Sentences in the Codestyle and Guard Skills #3583, and Split the Over-Length Sentences in the Secrets Shell Library Comments #3581. Promotion fix Restore a Relative Pronoun and Split a Long Reference Sentence in agent-conduct #3590 answered two Copilot findings on Promote Develop: Split the Over-Length Sentences in the Skill Sources and the Secrets Shell Library #3589, and Promote Develop: Split the Over-Length Sentences in the Skill Sources and the Secrets Shell Library #3589 merged tomain.What not to repeat
[P2]items with no thread, and on Split the Over-Length Sentences in the Codestyle and Guard Skills #3583 both were real drifts.New learnings
[-]Session Handoff [sentence-length]: Hub-Only Markdown and Comment Sweep Next, Skill-Source Sweep Shipped in 2.0.1242[/-][+]Session Handoff [sentence-length]: Hub Code Comment Sweep Next, Hub-Only Markdown Sweep Shipped in 2.0.1251[/+]ptr727 commentedon Oct 29, 2026
Closing answers for round 4, 2026-10-29:
install.pyafter release 2.0.1251.install.py --reportreads CURRENT at a tree identical tomain77d7ca35, so the round 4 "Host-safety kit is stale" blocker is resolved.Parked decision queue: zero open
decisionissues.ptr727 commentedon Oct 29, 2026
Outcome: track complete (round 5, closed 2026-10-29)
The sentence-length track's goal is done. The whole hub tree reads 0
sentence-lengthhits, down from 6964 when the track started. The rule is now a default prose-gate check fleet-wide, and #3531 is closed. Shipped in release 2.0.1262, frommainf24e9f17(promotion #3619, publish run 40935484439, success).What round 5 did
sentence-lengthdefault inprose-gate.json, and dropped the opt-in text fromcomment-and-doc-style,local-strict-review,GOVERNANCE.md,OPERATIONS.md,docs/fleet-map.md, andscripts/README.md. It closed Say What a Carrier's Local Strict Review Does With Sentence-Length Hits on Hub-Verbatim Text #3531.branching-and-release-model.scripts/README.md.decision):ruff@latestin the lint-only validator turns carriers red on a ruff release.RegressionTestsdriftNote clause once PlexCleaner Tighten Dependabot Exclude-Patterns Rationale Comment #1064 reaches itsmain.pr_review.pyagainst a live Qodo PR.files.Where the remaining work lives
Each is an ordinary backlog issue now rather than a step on this track:
pr_review.pyand Merge Gate item 2 on an earlier-head Copilot refusal)qodo_open, verify on PlexCleaner)State at close
mainisf24e9f17anddevelopis487deaa6. The primary checkout is ondevelop, current, and clean.cleanup --sweeplists only other sessions' worktrees, kept as too recent.f24e9f17.host-setup/agent-safety/claude/install.py.What not to repeat
developin instead.develop, so any develop merge moves it and triggers a new Copilot round.apply.py. Namespace helpers per branch..ps1/.shcomment in two parallel slices. F1 and F2 split the same copies differently (Align the Comment Copies the Sentence Sweep Split Two Ways, and Fix Three Stale Counts #3622). Put both halves of a copy pair in one slice.New learnings
uv run pytestfails to spawn on this host.uv run --no-project --with-requirements tests/requirements.txt python tests/<file>.pyworks. This is recorded in this machine's memory.The parked decision queue
There is one open
decisionissue: #3616, ruff pinning, filed this round. It is put to the maintainer in this session, and the answer is commented here.ptr727 commentedon Oct 29, 2026
Closing answer, 2026-10-29: the maintainer decided #3616 as a Dependabot-managed ruff pin with no fleet canary, recorded on #3616, and its
decisionlabel is removed.Parked decision queue after this round: zero open
decisionissues.