Skip to content

Feat: Information Source info in Chat UI#230

Merged
marcelklehr merged 5 commits into
mainfrom
feat/tool-info
May 6, 2025
Merged

Feat: Information Source info in Chat UI#230
marcelklehr merged 5 commits into
mainfrom
feat/tool-info

Conversation

@janepie
Copy link
Copy Markdown
Member

@janepie janepie commented Apr 30, 2025

The info button shows where the information the Agent used comes from
Defined per tool, every source will only be listed once

image

Needs nextcloud/context_agent#22 to show sources

@janepie janepie requested a review from jancborchardt April 30, 2025 10:03
@janepie janepie force-pushed the feat/tool-info branch 3 times, most recently from 0ca6fea to 755f385 Compare April 30, 2025 10:15
Comment thread src/components/ChattyLLM/Message.vue Outdated
@janepie janepie requested a review from julien-nc April 30, 2025 12:09
Comment thread src/components/ChattyLLM/Message.vue Outdated
Signed-off-by: Jana Peper <jana.peper@nextcloud.com>
Copy link
Copy Markdown
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

Very nice! Only tiny feedback: In other places where we use the info icon (like in admin settings) it is usually the circular one rather than the square one.

Comment thread lib/Service/AssistantService.php Outdated
@janepie
Copy link
Copy Markdown
Member Author

janepie commented Apr 30, 2025

Very nice! Only tiny feedback: In other places where we use the info icon (like in admin settings) it is usually the circular one rather than the square one.

I chose this to make it a bit more eye-catching so people actually click it, but would also be fine with a circular one. Would you prefer that?

janepie added 2 commits April 30, 2025 18:27
Signed-off-by: Jana Peper <jana.peper@nextcloud.com>
Signed-off-by: Jana Peper <jana.peper@nextcloud.com>
Signed-off-by: Jana Peper <jana.peper@nextcloud.com>
Comment thread lib/Listener/ChattyLLMTaskListener.php Outdated
Co-authored-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: janepie <49834966+janepie@users.noreply.github.com>
@jancborchardt jancborchardt moved this to 🏗️ At engineering in 🖍 Design team May 5, 2025
@marcelklehr marcelklehr merged commit e1ab390 into main May 6, 2025
16 checks passed
@marcelklehr marcelklehr deleted the feat/tool-info branch May 6, 2025 09:48
@github-project-automation github-project-automation Bot moved this from 🏗️ At engineering to 🎉 Done in 🖍 Design team May 6, 2025
@julien-nc julien-nc mentioned this pull request Jun 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants