refactor: Extract markdown parsing to utility and fix regex greediness - #272
Conversation
…iness - Extracted Markdown content parsing to src/utils/markdownParser.ts - Fixed greedy regex issue preventing parsing of multiple formatting elements on the same line - Added tests for markdownParser.ts - Updated ArticlePage to use the new utility Co-authored-by: albertoivo <1276520+albertoivo@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
|
Visit the preview URL for this PR (updated for commit eccb63f): https://pyexplorer-cd32d--pr272-refactor-markdown-pa-tuqjclr3.web.app (expires Sat, 12 Sep 2026 10:59:40 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 916b4905e0cdc420f3368c8eb19b2f2e90c3be61 |
|
This is already in production. |
Acknowledged. I understand this PR is meant to address bugs and runtime inconsistencies, not just pass linting and tests. |
…iness - Extracted Markdown content parsing to src/utils/markdownParser.ts - Fixed greedy regex issue preventing parsing of multiple formatting elements on the same line - Added tests for markdownParser.ts - Updated ArticlePage to use the new utility Co-authored-by: albertoivo <1276520+albertoivo@users.noreply.github.com>
|
Eu quero que reveja se esse código. |
…iness - Extracted Markdown content parsing to src/utils/markdownParser.ts - Fixed greedy regex issue preventing parsing of multiple formatting elements on the same line - Fixed bug where `- ✅` checkmarks were being incorrectly parsed as regular lists - Added tests for markdownParser.ts - Updated ArticlePage to use the new utility Co-authored-by: albertoivo <1276520+albertoivo@users.noreply.github.com>
1 participant
Extract markdown parsing logic from ArticlePage to a new utility file (
src/utils/markdownParser.ts) to adhere to the Single Responsibility Principle and improve testability. Fixes a bug related to regex greediness for bold and italic text parsing when multiple elements are on the same line. Includes unit tests for the new utility function.PR created automatically by Jules for task 13229913192782790059 started by @albertoivo