Skip to content

Session Handoff [review-tooling]: Clean Headline Advisory Released as 2.0.1093, #3214 Code-Span Bound Next #3226

Closed

Description

@ptr727

Next steps, in priority order

  1. End a Code Span at an HTML Block Line in pr_review.py #3214: end a CODE_SPAN at an HTML block line. unwrap masks through CODE_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 by unwrap, with every other CODE_SPAN reader's tests still green.
  2. 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, and DETAILS_TAG each still spell their own tail beside the shared ATTRIBUTES. Done looks like a table test pinning the same opener shapes at every site.
  3. Keep an Emphasis Tag Quoted in Code When reduced Strips Emphasis in pr_review.py #3215: keep an emphasis tag quoted in code when reduced strips emphasis. It is shared by normal, unwrap, and heading_count, so the vetted, printed, and counted text must move together.
  4. Finish the Incidental Count Sweep, Including Counted Section Titles #3169: finish the incidental-count sweep. Carried over, lowest priority.

External blockers

Internal dependencies

State

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

What not to repeat

  • A Dependabot auto-merge can move the promotion head between the go-ahead and the merge. --match-head-commit refused it, as designed. The new head then had no file table, so status exited 45, 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. status returns 42, 43, or 45 ahead of the 50/0 tail. Say "changes no exit code" for any reading that does not gate.
  • status run 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

Activity

  1. added
    handoffA link in the session handoff chain, one open issue per track
    on Oct 19, 2026
  2. ptr727 commented on Oct 19, 2026

    @ptr727
    OwnerAuthor

    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.

  3. ptr727 commented on Oct 19, 2026

    @ptr727
    OwnerAuthor

    Continued in #3233, round 9 on track review-tooling.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    handoffA link in the session handoff chain, one open issue per track

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions