fix(sections): keep a description's block Markdown in the description section - #713
Conversation
… section The description section replaced every blank line in the description with `<br />` and squashed runs of spaces. A list, a second paragraph or an indented code block lost its structure, and a fence in a list item fell out of the list. The description is now kept as written, apart from trimming and CRLF normalisation, and prettier formats it as block Markdown. The fence detection from #709 only existed to protect fences from the squash, so it goes too. The contract verifier builds its expected description with the same rule. A README whose action description contains a blank line changes once: paragraphs joined by `<br />` become separate paragraphs. Fixes #711 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H37LVnd9qB6PmZ5bJi5H2t
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: bitflight-devops/github-action-readme-generator/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (5)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughThe description formatter and README contract verifier now preserve block Markdown, including paragraph breaks, lists, and code blocks. The formatter trims outer whitespace and normalizes CRLF line endings. The README Usage example now references version v2.0.2. ChangesDescription Markdown preservation
README Usage example version
Priority: ➖ Normal Estimated code review effort: 3 (Moderate) | ~15 minutes Change: Bug fix · Severity of issue fixed: Medium Suggested reviewers: Merge Risk: ⚪ Minimal · up to The description changes and README example are ready to merge after normal checks. Security Architecture ReviewSecurity architecture risk: 🔵 Low · up to The generated description now retains block Markdown, but the change does not add a runtime entrypoint, privilege, or write target. No security regression was identified in the reviewed path. Retained concerns Security review detailsSecurity Blast Radius
Trust Boundaries and Controls
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
Full details: Out of Scope Changes checkExplanation The README change updates the Usage example from Full details: Docstring CoverageExplanation Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 4 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
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. Comment |
Coverage Report
File Coverage
|
||||||||||||||||||||||||||||||||||||||
1 similar comment
Coverage Report
File Coverage
|
||||||||||||||||||||||||||||||||||||||
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
## [2.0.4](v2.0.3...v2.0.4) (2026-09-29) ### Bug Fixes * **sections:** keep a description's block Markdown in the description section ([#713](#713)) ([a14c23f](a14c23f))
|
🎉 This PR is included in version 2.0.4 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Description
The description section replaced each blank line in an action description with
<br />and squashed runs of spaces. A list, a second paragraph or an indented code block lost its structure. A fence in a list item fell out of the list.The description is now kept as written: it is only trimmed and CRLF becomes LF. Prettier formats the span as block Markdown.
Type of Change
Related Issues
Changes Made
src/sections/update-description.ts:descriptionMarkdownkeeps the description as written. The fence detection from fix(sections): keep fenced code blocks in a description intact #709 only protected fences from the squash, so it is removed.scripts/verify-readme-contract.mjs: the contract verifier builds its expected description with the same rule.__tests__/update-description.test.ts: covers the rule, the list-with-fence case from the issue, the prettier-formatted section, and a stable second format.__tests__/verify-readme-contract.test.ts: the verifier accepts a block Markdown description and rejects the old<br />form.README.md: the pre-commitgenerate-docsrun set the usage version tov2.0.2. The2.0.3release commit onmainmakes the same change, so this line is a no-op after merge.Testing
npm run check,npm run test)Checklist
Additional Notes
This is filed as a non-breaking
fix, because the text between the markers belongs to the tool. Consumers still see a one-time change:<br />become separate paragraphs.🤖 Generated with Claude Code
https://claude.ai/code/session_01H37LVnd9qB6PmZ5bJi5H2t
Summary by CodeRabbit
github-action-readme-generator@v2.0.2.