fix(completions): use mcp resource URI as completion value
Carlos Alexandro Becker
created
The title might confuse the LLM, using the URI directly seems better.
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Change summary
internal/ui/model/ui.go | 9 ++-------
1 file changed, 2 insertions(+), 7 deletions(-)
Detailed changes
@@ -2556,12 +2556,7 @@ func (m *UI) insertFileCompletion(path string) tea.Cmd {
// insertMCPResourceCompletion inserts the selected resource into the textarea,
// replacing the @query, and adds the resource as an attachment.
func (m *UI) insertMCPResourceCompletion(item completions.ResourceCompletionValue) tea.Cmd {
- displayText := item.Title
- if displayText == "" {
- displayText = item.URI
- }
-
- if !m.insertCompletionText(displayText) {
+ if !m.insertCompletionText(item.URI) {
return nil
}
@@ -2601,7 +2596,7 @@ func (m *UI) insertMCPResourceCompletion(item completions.ResourceCompletionValu
return message.Attachment{
FilePath: item.URI,
- FileName: displayText,
+ FileName: item.URI,
MimeType: mimeType,
Content: data,
}