Repository navigation
Add sample for using Client Roots #5
Description
Describe the solution you'd like
Here is the spec: https://spec.modelcontextprotocol.io/specification/client/roots/
Upd link: https://modelcontextprotocol.io/docs/concepts/roots
Describe the solution you'd like
Here is the spec: https://spec.modelcontextprotocol.io/specification/client/roots/
Upd link: https://modelcontextprotocol.io/docs/concepts/roots
Activity
SeanChinJunKai commentedon Mar 28, 2025
@e5l I would like to work on this issue however im unable to access the link
e5l commentedon Mar 28, 2025
Hey, thank you! It would be great 👍
It looks like the link has changed, I also
don't have access.🙂Here is the updated link: https://modelcontextprotocol.io/docs/concepts/roots
SeanChinJunKai commentedon Mar 28, 2025
Thank you @e5l. Ill read up on this first
SeanChinJunKai commentedon Mar 29, 2025
@e5l Can I clarify that for this task i should be adding on to the kotlin-mcp-client in the samples folder?
e5l commentedon Mar 29, 2025
Yep!
SeanChinJunKai commentedon Mar 29, 2025
@e5l Thank you for the clarification. Ill take some time to digest the code and try setting up and then make the change.
SeanChinJunKai commentedon Mar 29, 2025
@e5l I think this are the changes required please correct me if I am wrong:
Create a data class called Root and include validation for uri
@serializable
data class Root(
val name: String,
val uri: String
)
Modify Roots to include an array/list of Root
Modify the kotlin-mcp-client sample to declare some sample roots
e5l commentedon Mar 31, 2025
it also would be nice to show how to consume exposed roots
devcrocod commentedon Apr 1, 2025
Hi @SeanChinJunKai
Please note that this sample is used in the quickstart-client. So if you make any changes, make sure the code in the documentation remains up to date
SeanChinJunKai commentedon Apr 1, 2025
Thank you @devcrocod for the heads up
SeanChinJunKai commentedon Apr 9, 2025
I am still trying to learn about how server can consume roots
aircode610 commentedon Sep 21, 2025
@e5l @devcrocod Hello wanted to see if the issue is resolved or can I work on it?
e5l commentedon Sep 22, 2025
It looks like it is still open :)
MichielDean commentedon May 12, 2026
I've opened #763 with a sample implementation for this.
devcrocod commentedon Jun 17, 2026
Roots being deprecated in the new spec: #816