Skip to content

feat(tui): render LaTeX in markdown - #3950

Merged
rumpl merged 3 commits into
docker:mainfrom
rumpl:feat/latex-rendering
Aug 7, 2026
Merged

feat(tui): render LaTeX in markdown#3950
rumpl merged 3 commits into
docker:mainfrom
rumpl:feat/latex-rendering

Conversation

@rumpl

@rumpl rumpl commented Aug 7, 2026

Copy link
Copy Markdown
Member

Summary

  • add a dependency-free LaTeX-to-Unicode terminal renderer
  • render inline and display math in TUI Markdown
  • preserve unsupported, incomplete, currency, shell-variable, and code content
  • cover renderer output with golden tests and Markdown integration tests

Validation

  • task format
  • task test (with MOONSHOT_API_KEY unset to skip the opt-in live API test)
  • task lint
  • task build

Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
@rumpl
rumpl requested a review from a team as a code owner August 7, 2026 10:04

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟡 NEEDS ATTENTION

One NOTABLE finding (medium severity, LIKELY confirmed) was identified in the new inline LaTeX rendering code.

Comment thread pkg/tui/components/markdown/fast_renderer.go Outdated
Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
@aheritier aheritier added area/tui For features/issues/fixes related to the TUI status/needs-triage For issues that need to be triaged kind/feat PR adds a new feature (maps to feat:). Use on PRs only. and removed status/needs-triage For issues that need to be triaged labels Aug 7, 2026
Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
@rumpl
rumpl merged commit 3192cb7 into docker:main Aug 7, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/tui For features/issues/fixes related to the TUI kind/feat PR adds a new feature (maps to feat:). Use on PRs only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants