Repository navigation
Session Handoff [review-tooling]: Clean Headline Advisory Released as 2.0.1093, #3214 Code-Span Bound Next #3226
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
- End a Code Span at an HTML Block Line in pr_review.py #3214: end a
CODE_SPANat an HTML block line.unwrapmasks throughCODE_SPAN, so a stray backtick that pairs across</details>/<details>leaves those wrappers printed raw. Done looks like the constructed case in End a Code Span at an HTML Block Line in pr_review.py #3214 stripped byunwrap, with every otherCODE_SPANreader's tests still green. - Share One Attribute Tail Across the Details and Summary Opener Readers in pr_review.py #3212: one attribute tail for the remaining opener readers.
SUMMARY,DETAILS_OPENER, andDETAILS_TAGeach still spell their own tail beside the sharedATTRIBUTES. Done looks like a table test pinning the same opener shapes at every site. - Keep an Emphasis Tag Quoted in Code When reduced Strips Emphasis in pr_review.py #3215: keep an emphasis tag quoted in code when
reducedstrips emphasis. It is shared bynormal,unwrap, andheading_count, so the vetted, printed, and counted text must move together. - Finish the Incidental Count Sweep, Including Counted Section Titles #3169: finish the incidental-count sweep. Carried over, lowest priority.
External blockers
- None for steps 1 through 4.
- CodeRabbit does not review this repository automatically. Copilot and the local pass carried Print a Clean Verdict's Headline as an Advisory Line in pr_review.py #3221 and Promote the Clean Headline Advisory and hub_fetch_run.py Rename to Main #3222.
Internal dependencies
- End a Code Span at an HTML Block Line in pr_review.py #3214 before Keep an Emphasis Tag Quoted in Code When reduced Strips Emphasis in pr_review.py #3215. Both change what
unwrapmasks and prints, and End a Code Span at an HTML Block Line in pr_review.py #3214 changes which spans are masked, so Keep an Emphasis Tag Quoted in Code When reduced Strips Emphasis in pr_review.py #3215's fix is easier to test on the corrected spans. - Share One Attribute Tail Across the Details and Summary Opener Readers in pr_review.py #3212 is independent, but touches
SUMMARY, whichheading_ofreads, so run the wholetest_pr_reviewsuite rather than the opener table alone.
State
- Released
2.0.1093from promotion Promote the Clean Headline Advisory and hub_fetch_run.py Rename to Main #3222, which carried Print a Clean Verdict's Headline as an Advisory Line in pr_review.py #3221 (pr_review.py Passes a Clean-Verdict Copilot Round Whose Headline Sentence Names a Finding #2567), Rename the Carried hub-fetch-run.py Snippet to hub_fetch_run.py #3219 (a peer lane's Rename the Carried hub-fetch-run.py Snippet to Snake Case #3143), and Dependabot's Bump streetsidesoftware/cspell-action from 9.1.0 to 9.2.0 in the actions-deps group across 1 directory #3223. - This machine's Codex and opencode skills snapshot was refreshed from
mainat5f278f2dand reads current. The host-safety kit reports CURRENT. - The primary checkout is on
develop, clean, and atorigin/develop.pr_review.py cleanupremoved the task worktree and branch, locally and on the remote. - Detached
fm-worktrees belong to another session that is live.cleanup --sweeplists two as removable, and--applywas not run. - No other repository was touched. A peer session in HomeAutomation-Config asked for Read a Compose Mount's Container Path as No Person's Home in the Prose Gate #3220 (prose gate, container path in a Compose mount) to be expedited. It belongs to no step above and was put to the maintainer at close.
The parked decision queue
The queue held 1 open issue with the decision label when this was written: #3225, filed this round. It was put to the maintainer at close.
What the last round did
- Print a Clean Verdict's Headline as an Advisory Line in pr_review.py #3221 prints a clean-verdict round's headline under
CLEAN VERDICT HEADLINEwhen the round counts no finding. The line is advisory and changes no exit code. It shares one guard with exit50through a verdict-set parameter onuncounted_verdict. Closes pr_review.py Passes a Clean-Verdict Copilot Round Whose Headline Sentence Names a Finding #2567 via Promote the Clean Headline Advisory and hub_fetch_run.py Rename to Main #3222. - Filed Decide Whether a Pin-Only Dependabot Bump Counts as Covered on a Promotion #3225, a decision on whether a pin-only Dependabot bump counts as covered on a promotion.
What not to repeat
- A Dependabot auto-merge can move the promotion head between the go-ahead and the merge.
--match-head-commitrefused it, as designed. The new head then had no file table, sostatusexited45, and the bump had no round of its own, so the promotion case did not apply. Expect a second go-ahead whenever develop gained a Dependabot merge. Decide Whether a Pin-Only Dependabot Bump Counts as Covered on a Promotion #3225 asks whether that should change. - Docs claiming an advisory "exits 0" overstate it.
statusreturns42,43, or45ahead of the50/0tail. Say "changes no exit code" for any reading that does not gate. statusrun from a checkout that lacks the change under review never shows it. The primary checkout ran pre-merge code, so the new advisory line printed only after the merge. To dogfood a reader before merge, run it from the task worktree.
New learnings
- None beyond the points above and Decide Whether a Pin-Only Dependabot Bump Counts as Covered on a Promotion #3225.
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
Answers given at the close of the round that wrote this link. #3225: keep a Dependabot bump's coverage on a promotion as the maintainer's call each time. It is recorded on #3225, which is closed. #3220: the maintainer wants it taken in a fresh session of its own, outside this track. The decision queue now holds 0 issues.
ptr727 commentedon Oct 19, 2026
Continued in #3233, round 9 on track
review-tooling.