Skip to content

Conversation

juftin
Copy link

@juftin juftin commented Sep 14, 2023

This PR adds Rich as a dependency and includes the option to print with Markdown formatting with the --rich flag.

image

Resolves: #12

Let me know if you're still interested in this and I'm happy to adapt it if you'd like

@NathanWilliams
Copy link

I just tried this and it is so nice to see!
I coupled it with #293 to add readline and together they make chat a real pleasure to use

@vividfog
Copy link

vividfog commented Oct 28, 2023

I see it's done with the .live() option, with streaming. Very nice. Previously it was possible to pipe the output to Rich to get Markdown rendering, but then streaming is lost. Live Markdown is a readability boost especially for chat mode.

@bloomsa
Copy link

bloomsa commented Mar 27, 2024

any chance this functionality could move forward? Whether from this PR or another like it. imo it'd be a boon to the UX of this tool! @simonw

@jimmybutton
Copy link

I would love this too! mods supports this, and it looks really cool! 😍

@thiswillbeyourgithub
Copy link

Looks awesome, would love to see this implemented!

@gianlucatruda
Copy link

I've been wanting to implement this myself for weeks. I use llm daily and the lack of rich formatting is the only issue I have with it. I recently discovered rich and found it delightful and robust, so was planning on adding this to llm myself, but discoverd this PR. Are we just waiting on @simonw to review or is there work to be done? I'm happy to help :)

@gianlucatruda
Copy link

UPDATE: I rebased, fixed some weird edge cases that were causing tests to fail, and resolved merged conflicts. I've opened #571 with the improvements, as this PR seems to have gone quiet 😢

@kristopolous
Copy link

Honestly, a general hooks framework with a toml config would be more useful. What would be really nice is generic interception

@gianlucatruda
Copy link

gianlucatruda commented Mar 17, 2025

@kristopolous I ultimately wrote https://github.com/gianlucatruda/richify to solve this and use it daily now.

UPDATE: I see in another thread you know about this and have built something cool of your own. Will discuss there.

@simonw simonw force-pushed the main branch 2 times, most recently from 569b316 to f5a38d3 Compare June 1, 2025 19:31
@daonb
Copy link

daonb commented Aug 12, 2025

This can be closed as the discussion moved here

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.

Markdown renderer support
9 participants