Skip to content

Conversation

@zzstoatzz
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings May 20, 2025 03:02
@github-actions github-actions bot added documentation Improvements or additions to documentation example labels May 20, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces FastMCP integration into marvin by adding a new beta adapter for FastMCP servers and updating the Agent class to perform runtime conversion of MCP server instances. Key changes include modifications to the Agent’s mcp_servers field and get_mcp_servers method, implementation of a FastMCP adapter in the integrations module, and corresponding updates to dependencies, examples, and documentation.

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/marvin/beta/init.py Exposes the FastMCP adapter (MCPServerStreamHTTP) as part of the beta public API.
src/marvin/agents/agent.py Updates the mcp_servers field type and conversion logic to support FastMCP servers.
src/marvin/_internal/integrations/mcp.py Removes a circular dependency import for cleaner type-checking.
src/marvin/_internal/integrations/fastmcp.py Implements the FastMCP adapter and conversion function for integration support.
pyproject.toml Adds fastmcp as a dependency and removes the examples workspace entry.
examples/fastmcp/hello.py Provides an example usage of FastMCP within the marvin framework.
docs/docs.json, docs/api-reference/*.mdx Updates documentation to include the new beta module and API references.

@github-actions github-actions bot added enhancement New feature or request feature Pull requests that add features labels May 20, 2025
again
@zzstoatzz zzstoatzz merged commit 7bcf2ad into main May 20, 2025
4 checks passed
@zzstoatzz zzstoatzz deleted the adapter branch May 20, 2025 04:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request example feature Pull requests that add features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants