Promote the Attributed Opener Stripper in pr_review.py to Main

[ptr727]

• Strip an Attributed Details or Summary Opener Through TAGS in pr_review.py #3213 makes TAGS in pr_review.py strip an attributed or self-closed <details> or <summary> opener the same way it strips a bare one, so <details open>, <summary class="x">, and <summary/> no longer print raw in the digest. The tag name ends through tag_opener, and the attribute tail is HTML_TAG's grammar, now shared as ATTRIBUTES. So prose such as <details element's ... -> stays text, and a > inside a quoted value belongs to its tag. unwrap also finds its tags in a copy with every fence and code span masked, so a tag a finding quotes in code stays in its text. Follow-ups are filed as Share One Attribute Tail Across the Details and Summary Opener Readers in pr_review.py #3212, End a Code Span at an HTML Block Line in pr_review.py #3214, and Keep an Emphasis Tag Quoted in Code When reduced Strips Emphasis in pr_review.py #3215.

The change is to scripts and tests only, so no installed Skill changes at release.

Closes #3208

🤖 Generated with Claude Code

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

• Configuration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: 44799fcb-6416-46fa-b5b1-6ed187b23677

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

🔵 Needs a closer look

Quoted code spans are removed before masking, causing the new quoted-markup behavior to fail.

No open issues 🎉

What changed in this PR

Promotes attributed and self-closing <details>/<summary> stripping in pr_review.py, with regression coverage.

Changes:

• Shares HTML attribute parsing between tag readers.
• Updates quotation-aware unwrapping.
• Adds tests for attributes, boundaries, and quoted markup.

File	Summary
tests/​test_pr_review.py	Adds regression tests for tag handling.
scripts/​pr_review.py	Updates tag parsing and unwrapping behavior.

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.52%. Comparing base (9a23b7d) to head (d557c3d).
⚠️ Report is 398 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3216      +/-   ##
==========================================
+ Coverage   65.50%   65.52%   +0.02%     
==========================================
  Files          17       17              
  Lines        9761     9767       +6     
==========================================
+ Hits         6394     6400       +6     
  Misses       3367     3367              

Flag	Coverage Δ
python-3.13	65.52% <100.00%> (+0.02%)	⬆️

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.

[ptr727]

Answering the round's headline, "Quoted code spans are removed before masking, causing the new quoted-markup behavior to fail": declined, the claim does not hold on this head.

unwrap runs reduced before mask_quotations, and reduced is EMPHASIS.sub("", SEVERITY_BADGE.sub(badge_text, text)). EMPHASIS is </?(?:strong|b|em|i)>, and SEVERITY_BADGE reduces the severity badge markup to its word. Neither touches a backtick, so every code span reaches the masking intact. Probed against develop at d557c3d:

unwrap("Keep `<details open>` here.")      -> "Keep `<details open>` here."
unwrap("Keep **`<details open>`** here.")  -> "Keep **`<details open>`** here."
unwrap("Keep ``<details open>`` x")        -> "Keep ``<details open>`` x"
unwrap("Keep <b>`<summary>`</b> x")        -> "Keep `<summary>` x"

test_a_render_strips_an_attributed_opener_as_it_strips_a_bare_one pins the quoted cases, and it fails when the masking is reverted. The one related gap is an emphasis tag quoted in code, which reduced does strip, and that is filed as #3215.