Skip to content

Fix ordered-list spacing in TUI markdown renderer#731

Open
rksharma-owg wants to merge 1 commit into
usestrix:mainfrom
rksharma-owg:codex/fix-numbered-list-spacing
Open

Fix ordered-list spacing in TUI markdown renderer#731
rksharma-owg wants to merge 1 commit into
usestrix:mainfrom
rksharma-owg:codex/fix-numbered-list-spacing

Conversation

@rksharma-owg

Copy link
Copy Markdown

Summary

  • fix ordered-list markdown rendering to avoid double spaces after numeric markers
  • add a regression test for 1., 2), and bullet-list rendering
  • keep the change scoped to the TUI markdown renderer

Testing

  • source .venv/bin/activate && pytest tests/test_agent_message_renderer.py

Closes #684

@greptile-apps

greptile-apps Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR fixes spacing for ordered lists in the TUI markdown renderer. The main changes are:

  • Skips the full ordered-list marker before rendering item text.
  • Normalizes 2) style markers to dot-style output.
  • Adds tests for ordered-list and bullet-list spacing.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.
  • The renderer slice now matches the ordered-list marker length.
  • The added tests cover the spacing behavior changed by this PR.

Important Files Changed

Filename Overview
strix/interface/tui/renderers/agent_message_renderer.py Updates ordered-list rendering so the content starts after the full three-character marker.
tests/test_agent_message_renderer.py Adds focused tests for dot-style ordered lists, paren-style ordered lists, and bullet spacing.

Reviews (1): Last reviewed commit: "Fix ordered-list spacing in TUI markdown..." | Re-trigger Greptile

@0xallam
0xallam force-pushed the main branch 3 times, most recently from 44e87ca to daf39a2 Compare July 13, 2026 23:50
@0xallam
0xallam force-pushed the codex/fix-numbered-list-spacing branch from d5463fc to b4c26db Compare July 14, 2026 00:00
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.

Numbered-list markdown renders a double space after the marker

1 participant