[ptr727]

Closes four prose-gate misreads in .github/actions/prose-gate/prose_lint.py, each with its issue's constructed case as a test that fails before and passes after.

• A heredoc label opening on a digit (<<123) is now read, but only where its terminator line follows. Bash reads <<2 as a shift inside (( )), $[ ], and an array subscript, so accepting any \w label, as the issue suggested, would have opened a false heredoc at every such shift, the existing (( x << 2 )) test among them. Bash does open a heredoc on let x=1<<2. This rule reads it as one only where a 2 line follows, which is the case the issue asked to keep closed.
• A second heredoc on one opener line (<<A 3<<B) queues its body after the first's terminator. A <<word after a quote or a # on that line is not read as a second opener, so a <<EOF inside a string cannot run the body on to a later heredoc's EOF.
• A run: step whose shell: is neither bash nor sh is no longer split at its heredocs. The new step_shell reads the step's own shell: key, past a comment line, and takes the program by its base name, as repo_gate.py's step_dialect does. A job-level or workflow-level defaults: run: shell: is not read, so such a step is split as it was on the base.
• A stray backtick (escaped, inside an HTML comment, or in a tag attribute) no longer pairs with a later one in the inline comment search. An escape, a comment, or a tag opening first keeps its backticks, which is CommonMark's leftmost precedence. The tag grammar comes from markdown-it-py's own HTML_TAG_RE, the parser this search lines up with. mask_code_and_quotes keeps CODE_SPAN unchanged, since it reads code comments rather than Markdown.

Raises the gate's size ceiling from 1840 to 1862 lines on the maintainer's decision this round. The growth is parser logic rather than vocabulary, and a scan for compaction found only cosmetic rewrites in unrelated functions.

Closes on promotion: #3430
Closes on promotion: #3435
Closes on promotion: #3428
Closes on promotion: #3429

Local review, declined with reasons:

• An escaped backtick directly followed by a code span still leaves that span unblanked, since CODE_SPAN's lookbehind sees the escaped backtick. It needs that adjacency plus a code span holding a later identical comment's text, and its failure is a comment numbered one line off, so it is declined per GOVERNANCE.md "Trust Boundaries and Hardening Effort".
• A flow-mapping step (- {shell: python, run: "..."}) is not read by step_shell, and is split as on the base.
• MARKDOWN_SPAN relies on markdown-it-py's HTML_TAG_RE holding no capturing group, so \1 stays the backtick run. Version 4.2.0 holds none, and a release that added one fails the existing code-span tests in CI, which installs the floating dependency, so no guard is added.

This change writes a comment line, so it carries the comments label.

🤖 Generated with Claude Code

[ptr727]

A recorded local strict-review pass covers head 2e271e22cf5a353274bb777a4fc241dd8f5ed638, the content this pull request carries at that commit against develop, and it recorded 1 finding.

[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: feature/prose-gate-heredoc-followups
• File: .coderabbit.yaml
• Commit: 2e271e2
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 3392b153-9030-4ce8-b3ea-25184f679127

• 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

Two moderate issues remain: shell lookup can cross YAML step boundaries, and digit-labelled single heredocs are not recognized.

1 open issue

• Match single heredocs with digit-labelled terminators · New

What changed in this PR

This pull request fixes prose-gate parsing issues involving Bash heredocs, non-shell workflow steps, and CommonMark-style inline code handling, with regression tests for the affected cases.

Changes:

• Supports digit-labelled and multiple heredocs while avoiding shift-expression false positives.
• Skips heredoc splitting for non-Bash and non-sh-compatible run: steps.
• Applies CommonMark-style precedence for escaped backticks, HTML tags, comments, and code spans.
• Raises the prose-gate size ceiling and adds regression coverage.

File	Description
.github/​actions/​prose-gate/​prose_lint.py	Extends heredoc parsing, workflow shell detection, and Markdown span masking.
tests/​test_prose_lint.py	Adds regression tests for the new parsing behaviors and updates the size ceiling.

_{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 (2e271e2).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #3437   +/-   ##
========================================
  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.