use block events for llm stats#902
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
| puts "[AGENT STATS] #{output.stats}" if config.show_stats? | ||
| puts "Session ID: #{output.session}" if config.show_stats? | ||
| if config.show_stats? | ||
| Event << { block: { header: "AGENT STATS", content: "#{output.stats}\nSession ID: #{output.session}" } } |
There was a problem hiding this comment.
"Agent Stats" doesn't need to be in all caps here anymore, nor in square brackets. It was only like that because there wasn't a better way to make a "header" stand out before we had this new formatter. Maybe it's still better in all caps, but feel free to think about how you'd like to style the header line now that you've got cleaner block formatting
There was a problem hiding this comment.
I like the all caps format, because it makes it stand out from the output and for consistency with the LLM response/prompt headers. But I agree that there could be a discussion on the way we want to format headers (for all of them, including response and prompt headers). I'll mark it up as ready but if you think there could be a cleaner formatting, let me know!!
Potential candidates:
(Current for reference)
0449253 to
7ed651f
Compare
c6d10bd to
1bf8dc5
Compare
7ed651f to
076d160
Compare
1bf8dc5 to
6053669
Compare
076d160 to
13658a1
Compare
6053669 to
a080bd5
Compare
13658a1 to
14f98aa
Compare
a080bd5 to
55ad147
Compare
14f98aa to
d6b84ca
Compare
55ad147 to
bfe8c0f
Compare
d6b84ca to
47749f7
Compare
bfe8c0f to
913f50c
Compare
913f50c to
9100a12
Compare
47749f7 to
1f0d7a7
Compare
9100a12 to
d3eb9c1
Compare





Current LLM stats output (after "Repeat log prefix on each line of multi-line output" is merged):
Proposed new LLM stats output: