Repository navigation
Session Handoff [review-tooling]: Suppressed Unread Check and One-Line Details Bounding Released as 2.0.1087 and 2.0.1088, Tag-Opener Primitive Next #3207
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
- Recognize a Details or Summary Opener Through One Primitive in pr_review.py #3206: recognize a
<details>or<summary>opener through one primitive inpr_review.py. Seven sites carry four different tag-name boundary rules, andHEADING's<summarybranch has none at all. Done looks like: one primitive every site builds on, and a test pinning<summaryx>and<summaryplus a no-break space as no opener at every site, and<summary>,<summaryplus a tab and an attribute, and<summary/>as an opener. - pr_review.py Passes a Clean-Verdict Copilot Round Whose Headline Sentence Names a Finding #2567: a clean-verdict round whose headline names a finding. Needs the maintainer's design call on reading headline prose before any code, so ask before starting it. It carries no
decisionlabel, so raise it as a question when it comes up. - Finish the Incidental Count Sweep, Including Counted Section Titles #3169: finish the incidental-count sweep. Carried over, lower priority.
External blockers
- CodeRabbit's free OSS review allowance ran out twice this round, each time on a promotion. It is advisory and blocks nothing, so proceed on Copilot and the local pass when its notice says the limit is reached, and do not ask it again on that pull request.
Internal dependencies
- Recognize a Details or Summary Opener Through One Primitive in pr_review.py #3206 touches the same patterns as Bound a Section Whose Summary Shares Its Details Line in pr_review.py #3204, which is released, so it starts from a current
developand needs no ordering against anything else. - pr_review.py Passes a Clean-Verdict Copilot Round Whose Headline Sentence Names a Finding #2567 waits on the maintainer's design call.
State
- Released
2.0.1087from promotion Promote the Script File-Naming Standard and the Suppressed Unread Check to Main #3203 (Report a Vetted Suppressed Summary Its Reader Misses in pr_review.py #3202 and State the Fleet Standard for Script and Utility File Names #3198, closing State the Fleet Standard for Script and Utility File Names #3140 and Check Suppressed Summaries for a Vetted-but-Unread Section in pr_review.py #3193). Released2.0.1088from promotion Promote One-Line Details Section Bounding in pr_review.py to Main #3205 (Bound a Section Whose Summary Shares Its Details Line in pr_review.py #3204, closing Bound a Suppressed Block at a One-Line Previously-Missed Section in pr_review.py #3201). - This machine's Codex and opencode skills snapshot was refreshed from
mainat0e3cb080and reads current. The host-safety kit reports CURRENT. - The primary checkout is on
develop, clean, and atorigin/develop.pr_review.py cleanupremoved both task worktrees and branches, locally and on the remote. - Detached
fm-worktrees and one named for crash-loop lint belong to another, live session.cleanup --sweepreports thefm-mainone as removable, being idle past its window and contained inmain.--applywas not run, since it reaches that session's tree, so it is the maintainer's call. - No other repository was touched.
The parked decision queue
The queue held 0 open issues with the decision label when this was written. Nothing is parked.
What the last round did
- Report a Vetted Suppressed Summary Its Reader Misses in pr_review.py #3202 extends the vetted-but-unread check to the suppressed summaries, and counts only the summary opening each block, so a block running into the next section no longer stands in for it. Closes Check Suppressed Summaries for a Vetted-but-Unread Section in pr_review.py #3193 via Promote the Script File-Naming Standard and the Suppressed Unread Check to Main #3203.
- Bound a Section Whose Summary Shares Its Details Line in pr_review.py #3204 bounds a section whose
<summary>shares its line with its<details>opener, reads a heading past that opener inheading_of, and narrows the tag-attribute separator in four patterns to HTML whitespace. Closes Bound a Suppressed Block at a One-Line Previously-Missed Section in pr_review.py #3201 via Promote One-Line Details Section Bounding in pr_review.py to Main #3205. - Promoted State the Fleet Standard for Script and Utility File Names #3198 from the closed
auto-3140lane alongside Report a Vetted Suppressed Summary Its Reader Misses in pr_review.py #3202, closing State the Fleet Standard for Script and Utility File Names #3140. - Filed Bound a Suppressed Block at a One-Line Previously-Missed Section in pr_review.py #3201 from the local review of Report a Vetted Suppressed Summary Its Reader Misses in pr_review.py #3202, and Recognize a Details or Summary Opener Through One Primitive in pr_review.py #3206 from Copilot's headline on Promote One-Line Details Section Bounding in pr_review.py to Main #3205.
What not to repeat
- A test claiming a check reports a reader's miss must cover the over-run form, not only a dropped block. The first Check Suppressed Summaries for a Vetted-but-Unread Section in pr_review.py #3193 test stubbed the reader to drop a block. The local review then showed a block running into its sibling counted that sibling's summary as read. Counting every summary a block holds was the defect.
- A pattern widened to admit markup must be matched by every reader of the same line. Bound a Section Whose Summary Shares Its Details Line in pr_review.py #3204's first cut let
HEADINGadmit a<details>opener whileheading_ofkept it, so a count-first heading lost its count. A body the base reported as unrecognized became a silent undercount. - Python's
\sis wider than HTML attribute whitespace. Use[ \t\n\f\r], the setDETAILS_OPENalready uses, after a tag name. - A status watcher must read the digest's first line.
pr_review.py statusprints thread lines after it, and atail -1watcher read a thread line as a failed check. - Copilot's lite "Needs a closer look" headline can be vague. On Promote One-Line Details Section Bounding in pr_review.py to Main #3205 the specific ask, a tag-name boundary, was only in the body's change list, so read the whole body before answering.
New learnings
- None beyond the issues filed.
Reactions are currently unavailable
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 19, 2026
Continued in #3211, round 6 on track
review-tooling.