Skip to content

Conversation

maxbuchan
Copy link
Contributor

Key Info

  • Implementation plan: link
  • Priority: normal

What changed?

Problem: Ordered lists rendered as “1., 1., 1.”. Our custom CSS-counter list implementation in markdownComponents reset the counter per item, and CommonMark doesn’t treat “1)” as an ordered list marker.

With whatever fix we ended up moving forward with here, we should also apply this fix to the design system because this issue is present in Codex as well (check AI Response section in slide)

  • Added a minimal pre-parse normalization that converts leading “N) ” → “N. ” before rendering so model output is recognized as an ordered list.

Before:

before

After:

after

What do you want the reviewer(s) to focus on?


Checklist

  • Did you link the GitHub issue?
  • Did you follow deployment steps or bump the version if needed?
  • Did you add/update tests?
  • What QA did you do?
    • Tested...

@maxbuchan maxbuchan requested a review from dogmar September 23, 2025 16:20
@maxbuchan maxbuchan merged commit 70598ff into main Sep 23, 2025
4 checks passed
@maxbuchan maxbuchan deleted the fix/normalize-ordered-list-markers branch September 23, 2025 16:57
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.

2 participants