[ptr727]

Summary

Promotes the pull requests merged into develop since release 2.0.1163.

• #3437: the prose gate reads a heredoc label opening on a digit where its terminator line follows, queues a second heredoc on one opener line, skips the run-block heredoc split for a step whose shell: is neither bash nor sh, and numbers inline Markdown comments with escapes, HTML comments, and tags taking their backticks as CommonMark reads them, using markdown-it-py's own tag grammar. The gate's size ceiling moves from 1840 to 1862 lines, as the maintainer decided this round.
• #3439: fixes the two findings this promotion's review raised. Each of two heredoc bodies on one opener line is lexed alone, and a URI autolink keeps its backticks out of the code-span mask, using markdown-it-py's own autolink character classes. Three walrus rewrites hold the gate at the 1862-line ceiling.
• #3440: rewords the comment above HEREDOC to say the last, unclosed branch is the one that drops a digit label, after this promotion's review read that branch as the second-opener conditional's else branch a second time.

Closes #3428
Closes #3429
Closes #3430
Closes #3435

Constituent coverage

Each change this promotion carries had coverage on its own pull request into develop, at its merged head:

Pull request	Merged head	Coverage
#3437	2e271e22	attested local pass, and a Copilot table naming exactly the changed files, coverage=table
#3440	7bbd7780	attested local pass, and a Copilot table naming exactly the changed file, coverage=table
#3439	a53ee15a	attested local pass, and a Copilot table naming exactly the changed files, coverage=table

🤖 Generated with Claude Code

[coderabbitai]

Important

• 🔍 Trigger review

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

Run configuration

• Source mode: Auto
• Repository: ptr727/ProjectTemplate
• Branch: develop
• File: .coderabbit.yaml
• Commit: 687ccd9
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 97a99e2e-1154-4f35-bf54-89df6e0e9069

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

🟡 Changes recommended

Resolve the multiple-heredoc lexer-span issue and preserve autolink contents when masking Markdown code spans.

2 open issues

• Separate multiple heredoc bodies into distinct lexer spans · New
• Exclude CommonMark autolinks from code-span masking · New

What changed in this PR

This pull request promotes prose-gate parser fixes from develop to main, covering heredoc detection, shell-aware YAML handling, and Markdown inline-comment positioning.

Changes:

• Supports digit-labelled and multiple heredocs.
• Skips heredoc splitting for non-Bash and non-sh steps.
• Improves Markdown-aware masking for escapes, HTML, comments, and code spans.
• Adds regression coverage and raises the size ceiling to 1862 lines.

File	Summary
tests/​test_prose_lint.py	Adds regression tests for heredoc, shell-step, and Markdown parsing behavior and updates the size ceiling.
.github/​actions/​prose-gate/​prose_lint.py	Updates heredoc parsing, shell detection, and Markdown span masking; unresolved issues remain with separate lexer spans for multiple heredoc bodies and CommonMark autolinks.

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.76%. Comparing base (4c77db2) to head (687ccd9).
⚠️ Report is 418 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3438   +/-   ##
=======================================
  Coverage   80.76%   80.76%           
=======================================
  Files          23       23           
  Lines        8839     8839           
=======================================
  Hits         7139     7139           
  Misses       1700     1700           

Flag	Coverage Δ
python-3.13	80.76% <ø> (ø)

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.

[Copilot]

🟡 Changes recommended

The reviewed parser still has one critical numeric-heredoc defect and two moderate heredoc/shell-detection issues.

1 open issue

• Numeric heredoc branch rejects terminated first bodies · New

2 resolved since last review

• Exclude CommonMark autolinks from code-span masking
• Separate multiple heredoc bodies into distinct lexer spans

_{Review effort: Lite}

[Copilot]

🔵 Needs a closer look

Shell parsing still mishandles unrecognized dynamic shells and executable-qualified shell tokens, risking incorrect heredoc classification.

No open issues 🎉

1 resolved since last review

• Numeric heredoc branch rejects terminated first bodies

_{Review effort: Lite}

[ptr727]

On the round's headline, which counts no finding: both cases are declined. A dynamic shell such as shell: ${{ matrix.shell }} has no value a static read can know, so step_shell returns "" and the step is split as bash, exactly as on the base and matching GitHub's default shell. A path-qualified program such as /usr/bin/python3 {0} or pwsh.exe is classified by the base name of its first word, the rule repo_gate.py's step_dialect already uses, and test_a_non_shell_run_step_takes_no_heredoc_split covers the quoted and absolute-path forms. A Windows backslash path to bash or an env bash shell reads as non-shell and skips the split. That needs such a shell, a heredoc whose body holds an apostrophe, and a later comment, and its failure is one comment left unlinted, so it is declined per GOVERNANCE.md "Trust Boundaries and Hardening Effort".