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

@janepiejanepie commented Apr 30, 2025

Copy link
Copy Markdown
Member

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
janepieforce-pushed the feat/tool-info branch 3 times, most recently from 0ca6fea to 755f385CompareApril 30, 2025 10:15
julien-nc
julien-nc requested changes Apr 30, 2025
View reviewed changes
Comment threadsrc/components/ChattyLLM/Message.vue Outdated
@janepie
janepie requested a review from julien-ncApril 30, 2025 12:09
julien-nc
julien-nc previously requested changes Apr 30, 2025
View reviewed changes
Comment threadsrc/components/ChattyLLM/Message.vue Outdated
@janepie
feat: add tool info in Chat UI
31210b0
Signed-off-by: Jana Peper <jana.peper@nextcloud.com>
jancborchardt
jancborchardt reviewed Apr 30, 2025
View reviewed changes

@jancborchardtjancborchardt left a comment

Copy link
Copy Markdown
Member

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.

marcelklehr
marcelklehr reviewed Apr 30, 2025
View reviewed changes
Comment threadlib/Service/AssistantService.php Outdated
@janepie

janepie commented Apr 30, 2025

Copy link
Copy Markdown
MemberAuthor

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
feat: translate tool names to information source strings
ae04aaf
Signed-off-by: Jana Peper <jana.peper@nextcloud.com>
@janepie
fix: update openapi
8112bb6
Signed-off-by: Jana Peper <jana.peper@nextcloud.com>
@janepie
refactor: sources as computed prop
1feb135
Signed-off-by: Jana Peper <jana.peper@nextcloud.com>
marcelklehr
marcelklehr reviewed May 5, 2025
View reviewed changes
Comment threadlib/Listener/ChattyLLMTaskListener.php Outdated
@janepie@marcelklehr
Fix: Error handling for json_encode
395b727
Co-authored-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: janepie <49834966+janepie@users.noreply.github.com>
@jancborchardtjancborchardt moved this to 🏗️ At engineering in 🖍 Design teamMay 5, 2025
@marcelklehr
marcelklehr merged commit e1ab390 into mainMay 6, 2025
@marcelklehr
marcelklehr deleted the feat/tool-info branch May 6, 2025 09:48
@github-project-automationgithub-project-automationBot moved this from 🏗️ At engineering to 🎉 Done in 🖍 Design teamMay 6, 2025
@jancborchardtjancborchardt mentioned this pull request May 11, 2025
Open
80 tasks
@julien-ncjulien-nc mentioned this pull request Jun 26, 2025
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marcelklehrmarcelklehrmarcelklehr approved these changes
@jancborchardtjancborchardtjancborchardt approved these changes
@julien-ncjulien-ncAwaiting requested review from julien-ncjulien-nc is a code owner

Assignees

No one assigned

Labels

None yet

Projects

🖍 Design team
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@janepie@jancborchardt@marcelklehr@julien-nc