Skip to content

Add low-level MCP contrib package with tools and tests#1211

Open
nilbacardit26 wants to merge 32 commits intomasterfrom
feat/mcp-lowlevel-contrib
Open

Add low-level MCP contrib package with tools and tests#1211
nilbacardit26 wants to merge 32 commits intomasterfrom
feat/mcp-lowlevel-contrib

Conversation

@nilbacardit26
Copy link
Copy Markdown
Contributor

WIP

from guillotina.interfaces import IResource


@configure.subscriber(for_=(IResource, IObjectAddedEvent))
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.

Does not work on distributed systems

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bloodbare take another look now, please. I changed that to use redis if available with an asyncio queue to avoid spamming the invalidation of the cache too much

@nilbacardit26
Copy link
Copy Markdown
Contributor Author

nilbacardit26 commented Mar 30, 2026

@bloodbare fear feel not for the 300+ files changed. MCP has a higher dependency on uvicorn than the one we already have, so I decided to upgrade uvicorn to the latest version (0.42.0) and drop support for 3.8 and 3.9 python versions, as they are too old to be maintained with the new uvicorn and the whole stack. I added test coverage for version 3.12. Tests are passing.

I upgraded the versions of the test packages and lint tools as well: isort and black. Most of the files are due to black and isort changes. Async fixtures decorators are changed to the most modern syntax as well.

Adding MCP has been a good reason I believe to do all these upgrades

Maybe now that I am doing this, I can add support to at least 3.13 too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants