[ptr727]

• Recognize a Details or Summary Opener Through One Primitive in pr_review.py #3209 makes every reader of a <details> or <summary> opener in pr_review.py end the tag name through one primitive, tag_opener, on CommonMark's rule: HTML whitespace, the end of a line, >, or />. HEADING and heading_of had no boundary at all, so <summaryx> opened a heading line. heading_of now reads the heading through one anchored SUMMARY.match, so a block whose own summary is rejected no longer takes a nested entry's summary as its heading. TAGS, which strips only a bare tag, is filed as Strip an Attributed Details or Summary Opener Through TAGS in pr_review.py #3208.

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

Closes #3206

🤖 Generated with Claude Code

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 40 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

• Configuration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: 13777f63-fb74-45a3-b4bc-e47adca928f0

📥 Commits

Reviewing files that changed from the base of the PR and between 0e3cb08 and 9a23b7d.

📒 Files selected for processing: 2

• scripts/pr_review.py
• tests/test_pr_review.py

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

🟢 Approval recommended

All reviewed changes have no unresolved blocking issues.

No open issues 🎉

What changed in this PR

Promotes shared HTML tag-opener parsing in pr_review.py and adds regression coverage for heading selection and tag boundaries.

Changes:

• Centralizes <details> and <summary> opener recognition.
• Prevents malformed or nested summaries from being misread.
• Adds boundary and heading-selection tests.

File	Description
tests/​test_pr_review.py	Adds opener-boundary and heading regression tests.
scripts/​pr_review.py	Implements shared opener matching and anchored heading parsing.

_{Review effort: Lite}

[ptr727]

@coderabbitai review

[coderabbitai]

⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[codecov]

Codecov Report

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

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3210      +/-   ##
==========================================
+ Coverage   65.49%   65.50%   +0.01%     
==========================================
  Files          17       17              
  Lines        9758     9761       +3     
==========================================
+ Hits         6391     6394       +3     
  Misses       3367     3367              

Flag	Coverage Δ
python-3.13	65.50% <100.00%> (+0.01%)	⬆️

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.