-
Notifications
You must be signed in to change notification settings - Fork 1.5k
MCP Server instructions are currently ignored #704
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Not explicit, I didn't realize this existed. From the docs, looks like instructions is optional and it says
We could add a way for you to fetch the instructions, and then you can choose to append that to the system prompt. Open to other ideas too. |
This sounds intuitive enough to me. Shall I file a separate feature request? |
Nope, we can use this issue. PR also welcome! |
Hi @bocytko, if you haven’t started on this yet I’d be happy to implement this feature. |
@DanieleMorotti please go ahead. Thank you in advance! |
@rm-openai I can do something like this: server_result = await session.initialize()
self.mcp_instructions = server_result.instructions or None in |
@DanieleMorotti wdym? I don't think you need to do anything else. Once you do that, users will be able to do this:
|
Ok sorry, I thought it should be injected to the prompt somehow but instead users will directly choose how to adopt instructions. |
Ah yup, prefer to leave it to users - less magic that way. |
Added the `instructions` attribute to the MCP servers to solve #704 . Let me know if you want to add an example to the documentation.
Please read this first
Question
MCP Servers define instructions, for example the aws-documentation-mcp-server.
Instructions are currently ignored in the Agent execution flow. It only reads mcp tools and converts these to LLM tool descriptions. Is this an explicit design decision?
The text was updated successfully, but these errors were encountered: