fix(issue #27): added an optional limit parameter#28
Conversation
…sport and stdio_streams Signed-off-by: Chojan Shang <chojan.shang@vesoft.com>
|
cc @OhadRubin, could you please review this? |
OhadRubin
left a comment
There was a problem hiding this comment.
LGTM, I do have some nitpicks but this looks fine.
Might set a more reasonable default?
Good idea, but it seems that in UI displays, most modern tools collapse content (and don't allow expansion) to avoid overwhelming users with large amounts of information. I'd like to consult @stdrc , who built kimi-cli, as it's one of the primary use cases at present. |
One concrete reasonable use case: base64 encoded image... |
Close: #27