fix: Update @mcp.resource to use function documentation as default descrip… (#489)

This commit is contained in:
gaojingyu
2025-05-15 18:14:50 +08:00
committed by GitHub
parent c2f8730d6d
commit 1bdeed33c2
4 changed files with 73 additions and 9 deletions

View File

@@ -11,7 +11,7 @@ import anyio.to_thread
import httpx
import pydantic
import pydantic_core
from pydantic import Field, ValidationInfo
from pydantic import AnyUrl, Field, ValidationInfo, validate_call
from mcp.server.fastmcp.resources.base import Resource
@@ -68,6 +68,31 @@ class FunctionResource(Resource):
except Exception as e:
raise ValueError(f"Error reading resource {self.uri}: {e}")
@classmethod
def from_function(
cls,
fn: Callable[..., Any],
uri: str,
name: str | None = None,
description: str | None = None,
mime_type: str | None = None,
) -> "FunctionResource":
"""Create a FunctionResource from a function."""
func_name = name or fn.__name__
if func_name == "<lambda>":
raise ValueError("You must provide a name for lambda functions")
# ensure the arguments are properly cast
fn = validate_call(fn)
return cls(
uri=AnyUrl(uri),
name=func_name,
description=description or fn.__doc__ or "",
mime_type=mime_type or "text/plain",
fn=fn,
)
class FileResource(Resource):
"""A resource that reads from a file.

View File

@@ -148,9 +148,11 @@ class FastMCP:
self._mcp_server = MCPServer(
name=name or "FastMCP",
instructions=instructions,
lifespan=lifespan_wrapper(self, self.settings.lifespan)
if self.settings.lifespan
else default_lifespan,
lifespan=(
lifespan_wrapper(self, self.settings.lifespan)
if self.settings.lifespan
else default_lifespan
),
)
self._tool_manager = ToolManager(
warn_on_duplicate_tools=self.settings.warn_on_duplicate_tools
@@ -465,16 +467,16 @@ class FastMCP:
uri_template=uri,
name=name,
description=description,
mime_type=mime_type or "text/plain",
mime_type=mime_type,
)
else:
# Register as regular resource
resource = FunctionResource(
uri=AnyUrl(uri),
resource = FunctionResource.from_function(
fn=fn,
uri=uri,
name=name,
description=description,
mime_type=mime_type or "text/plain",
fn=fn,
mime_type=mime_type,
)
self.add_resource(resource)
return fn