Skip to content

Conversation

ghhamel
Copy link

@ghhamel ghhamel commented Sep 30, 2025

Fixes

Summary

Changes

Added 'description' field to responses for list_knowledge_bases_tool. That field already exists in the AWS infrastructure and is returned via the list_knowledge_bases API. I have simply added it to the result that is returned. This provides context so that an agent can choose which knowledge base to consult for a given user prompt.

User experience

Before this change, the user prompt would need to tell an agent which knowledge base to query. After this change, the agent has additional context for each knowledge base through the provided description.

Checklist

If your change doesn't seem to apply, please leave them unchecked.

  • I have reviewed the contributing guidelines
  • I have performed a self-review of this change
  • Changes have been tested
  • Changes are documented

Is this a breaking change? (Y/N) N

RFC issue number:

Checklist:

  • Migration process documented
  • Implement warnings (if it can live side by side)

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

@ghhamel ghhamel requested review from a team as code owners September 30, 2025 20:18
@scottschreckengaust scottschreckengaust self-assigned this Oct 1, 2025
@scottschreckengaust scottschreckengaust added the backlog an accumulation of something, especially uncompleted work or matters that need to be dealt with. label Oct 1, 2025
@scottschreckengaust scottschreckengaust moved this from To triage to Ready in awslabs/mcp Project Oct 1, 2025
@scottschreckengaust scottschreckengaust moved this from Ready to Backlog in awslabs/mcp Project Oct 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backlog an accumulation of something, especially uncompleted work or matters that need to be dealt with.

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

2 participants