feat(chat-messages): add activity trace pattern - #2557
Conversation
29e7250 to
039cec2
Compare
|
Some comments:
So I would prefer to build this in steps, i.e. for now only do content-projection and land this quickly. And address the file links in a generic way |
66ce2ab to
d219c1f
Compare
94c679e to
3cbf665
Compare
00202a4 to
5c625b4
Compare
|
General comment: I know in the UX specs we call this Activity Trace and in general for the entire thing (multiple ones) it makes sense to call it "activity trace" but individual messages should not be called trace messages IMO. This is not a term used for this and also doesn't really make sense, it should be something more generic, I like tool but maybe we can find a more generic name. |
|
Tools also can have 3 different things to display:
One "trace message" needs to support having all of these 3 at the same time, otherwise every product will need to choose their own Markdown representation which is precisely what we want to avoid. |
That's more like general feedback for #2339 ? |
5c625b4 to
1feda89
Compare
Tool, activity or action message perhaps? |
8758018 to
ca2d0eb
Compare
I like |
1cab260 to
43a487a
Compare
43a487a to
f55b096
Compare
@kfenner @Killusions I think activity message is kind of weird. I have seen the pattern described also as activity history or reasoning history, but never message. Message" implies a single unit of communication (like a chat bubble), not a time-ordered sequence of steps. Our pattern shows progress, intermediate actions, and a path to an outcome... that's a trace, not a message. If the rename is coming from how it's implemented (each step rendered as a message-like row), that's a component-level detail, not what the pattern does for the user...so is a bit tricky to then explain. I would prefer to keep "activity trace." If we want an alternative, "activity history" is the next best fit... though it leans past-tense/archival, weaker for live in-progress use. "Reasoning history" undersells it too, since we cover tool calls and file edits, not just reasoning steps. What do you guys think? Whatever the desicion, then i need to reflect it and justify it for the AI patterns team. |
@Killusions whenever you have time, lets connect on this so you can explain me a bit better this ;) thank youu |
@panch1739 History is definitely wrong here because it's a singular piece of the history. That's also what I meant above, the whole name makes sense for a "block" of messages, but the component is an individual piece of it, it's kind of like a list vs a list item, you would also not call a list item "list-message" or whatever. I understand that the UX and user facing pattern makes sense named like this (although I never saw it referred to this name anywhere either) but as a singular item it just seems confusing. |
Closes #2339
Changes
Documentation.
Examples.
Dashboards Demo.
Playwright report.
Coverage Reports: