Skip to content

refine symbol styles#431

Merged
iceljc merged 1 commit intoSciSharp:mainfrom
iceljc:main
Mar 10, 2026
Merged

refine symbol styles#431
iceljc merged 1 commit intoSciSharp:mainfrom
iceljc:main

Conversation

@iceljc
Copy link
Collaborator

@iceljc iceljc commented Mar 10, 2026

No description provided.

@qodo-code-review
Copy link

Review Summary by Qodo

Refine markdown list item symbol replacement regex

🐞 Bug fix

Grey Divider

Walkthroughs

Description
• Refine markdown replacement regex for list items
• Replace /^-\s*/gm with /^-[ \t]+/gm for precise whitespace matching
• Ensures only spaces and tabs after dash are converted to bullets
Diagram
flowchart LR
  A["Old regex: /^-\s*/gm"] -->|"Replace with"| B["New regex: /^-[ \t]+/gm"]
  B -->|"Result"| C["More precise whitespace matching"]
Loading

Grey Divider

File Changes

1. src/lib/helpers/http.js 🐞 Bug fix +1/-1

Refine list item markdown regex pattern

• Updated replaceMarkdown() function regex pattern for list item conversion
• Changed from /^-\s*/gm to /^-[ \t]+/gm to match only spaces and tabs
• Improves specificity of markdown list item replacement logic

src/lib/helpers/http.js


Grey Divider

Qodo Logo

@qodo-code-review
Copy link

qodo-code-review bot commented Mar 10, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

@iceljc iceljc merged commit cf113c8 into SciSharp:main Mar 10, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant