[ptr727]

Summary

The semicolon rule split a Markdown table row on every pipe, so an escaped \| cut its cell in two. A comma before the escaped pipe then no longer excused the semicolon list after it, and the rule reported semicolons the whole cell allows.

A cell now ends only at a pipe no backslash escapes, the same escape markdown-it-py honors when it splits a row.

Why Not the Parser's Cell Tokens

The issue proposed reading the th_open and td_open inline tokens the sentence-length rule reads. A local review pass found that loses text the row split reads today.

• A table inside a blockquote yields no cell units, since the extractor skips blockquotes, so its rows would be read whole.
• markdown-it-py drops any body cell past the header's column count, so a semicolon there would never be read.

Keeping the row split and fixing only where it ends a cell closes the issue's case and keeps both of those.

Changes

• .github/actions/prose-gate/prose_lint.py: rule_semicolon splits a row with CELL_BREAK, a pipe no backslash precedes.
• tests/test_prose_lint.py: an escaped pipe stays inside its cell, at top level and inside a blockquote. The row-per-cell test now uses a real table.
• scripts/README.md: states the escape rule.

Verification

• The issue's repro now reports no semicolon violation, and reported two before.
• The prose gate suite passes, and both new tests fail against the old split.
• A whole-tree semicolon and sentence-length run is unchanged.

Local Review

Three recorded local-strict-review passes. The first raised the two regressions above, which moved the fix off the parser tokens. The second raised one introduced overstatement in the docstring and README, which was narrowed. The third raised nothing.

The pre-existing finding that the split runs after code spans and quotations are blanked is filed as #3657.

Closes on promotion: #3655

🤖 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: feature/auto-3655
• File: .coderabbit.yaml
• Commit: 221e158
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 51cf8732-268c-4edf-a647-2620b1c26791

• 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.}

[ptr727]

A recorded local strict-review pass covers head 221e158825e0dcb921b3abcfd4260573e98b2095, the content this pull request carries at that commit against develop, and it recorded 0 findings.

[Copilot]

🟡 Changes recommended

An active moderate bug remains in prose_lint.py: pipe escaping checks only the immediately preceding backslash, so even backslash runs can misclassify delimiters and produce incorrect lint results.

1 open issue

• Handle even backslash runs when detecting escaped pipes · New

What changed in this PR

This pull request updates the semicolon rule to split Markdown table rows only at unescaped pipes, preserving escaped pipes within cells. It adds regression coverage and documents the behavior.

Changes:

• Adds escaped-pipe-aware cell splitting to rule_semicolon.
• Adds top-level and blockquoted table tests.
• Documents the table-cell escape rule.

The pull request has an unresolved moderate issue: the splitter treats a pipe preceded by an even-length backslash run as escaped. Two minor issues also remain regarding docstring formatting and ineffective blockquote coverage.

File	Summary
.github/​actions/​prose-gate/​prose_lint.py	Adds unescaped-pipe table-cell splitting; requires correction for even backslash runs and has a docstring formatting nit.
tests/​test_prose_lint.py	Adds escaped-pipe regression coverage; the blockquote case does not exercise the changed splitter.
scripts/​README.md	Documents the escaped-pipe cell-boundary behavior.

This pull request requires changes.

_{Review effort: Lite}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.70%. Comparing base (b38e084) to head (221e158).
⚠️ Report is 27 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3658      +/-   ##
===========================================
+ Coverage    81.60%   81.70%   +0.09%     
===========================================
  Files           26       26              
  Lines         9070     9119      +49     
===========================================
+ Hits          7402     7451      +49     
  Misses        1668     1668              

Flag	Coverage Δ
python-3.13	81.70% <ø> (+0.09%)	⬆️

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.