merge

Author: zzstoatzz

Diff for: .github/workflows/publish.yml

@@ -5,28 +5,22 @@ on:
   workflow_dispatch:
 
 jobs:
-  publish-pypi-release:
+  pypi-publish:
+    name: Upload to PyPI
     runs-on: ubuntu-latest
-    environment: release
     permissions:
-      contents: write
-      id-token: write
+      id-token: write # For PyPI's trusted publishing
     steps:
       - name: Checkout
         uses: actions/checkout@v4
-      - name: Set up Python
-        uses: actions/setup-python@v5
         with:
-          python-version: "3.11"
-          cache: pip
-          cache-dependency-path: "**/pyproject.toml"
-      - name: Install dependencies
-        run: |
-          pip install setuptools wheel build
+          fetch-depth: 0
+
+      - name: "Install uv"
+        uses: astral-sh/setup-uv@v3
+
       - name: Build
-        run: |
-          python -m build
-      - name: Publish
-        uses: pypa/gh-action-pypi-publish@release/v1
-        with:
-          verbose: true
+        run: uv build
+
+      - name: Publish to PyPi
+        run: uv publish -v dist/*

Diff for: .gitignore

@@ -11,4 +11,8 @@ wheels/
 .DS_Store
 
 
-src/fastmcp/_version.py
+src/fastmcp/_version.py
+
+# editors
+.cursorrules
+.vscode/

Diff for: .pre-commit-config.yaml

@@ -0,0 +1,20 @@
+fail_fast: true
+
+repos:
+  - repo: https://github.com/abravalheri/validate-pyproject
+    rev: v0.23
+    hooks:
+      - id: validate-pyproject
+
+  - repo: https://github.com/pre-commit/mirrors-prettier
+    rev: v3.1.0
+    hooks:
+      - id: prettier
+        types_or: [yaml, json5]
+
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.8.0
+    hooks:
+      - id: ruff-format
+      - id: ruff
+        args: [--fix, --exit-non-zero-on-fix]

Diff for: examples/echo.py

@@ -4,7 +4,6 @@
 
 from fastmcp import FastMCP
 
-
 # Create server
 mcp = FastMCP("Echo Server")
 

Diff for: pyproject.toml

@@ -29,10 +29,12 @@ dev = [
     "copychat>=0.5.2",
     "ipython>=8.12.3",
     "pdbpp>=0.10.3",
+    "pre-commit>=3.10.1",
     "pytest-xdist>=3.6.1",
     "pytest>=8.3.3",
     "pytest-asyncio>=0.23.5",
 ]
 
 [tool.pytest.ini_options]
 asyncio_mode = "auto"
+asyncio_default_fixture_loop_scope = "session"

Diff for: src/fastmcp/resources/base.py

@@ -3,15 +3,18 @@
 import abc
 from typing import Union
 
-from pydantic import BaseModel, Field, field_validator
-from pydantic.networks import _BaseUrl
+from pydantic import BaseModel, ConfigDict, Field, ValidationInfo, field_validator
+
+from fastmcp.types import LaxAnyUrl
 
 
 class Resource(BaseModel, abc.ABC):
     """Base class for all resources."""
 
-    uri: _BaseUrl = Field(description="URI of the resource")
-    name: str = Field(description="Name of the resource", default=None)
+    model_config = ConfigDict(validate_default=True)
+
+    uri: LaxAnyUrl = Field(description="URI of the resource")
+    name: str | None = Field(description="Name of the resource", default=None)
     description: str | None = Field(
         description="Description of the resource", default=None
     )
@@ -23,13 +26,12 @@ class Resource(BaseModel, abc.ABC):
 
     @field_validator("name", mode="before")
     @classmethod
-    def set_default_name(cls, name: str | None, info) -> str:
+    def set_default_name(cls, name: str | None, info: ValidationInfo) -> str:
         """Set default name from URI if not provided."""
         if name:
             return name
         # Extract everything after the protocol (e.g., "desktop" from "resource://desktop")
-        uri = info.data.get("uri")
-        if uri:
+        if uri := info.data.get("uri"):
             uri_str = str(uri)
             if "://" in uri_str:
                 name = uri_str.split("://", 1)[1]
@@ -41,7 +43,3 @@ def set_default_name(cls, name: str | None, info) -> str:
     async def read(self) -> Union[str, bytes]:
         """Read the resource content."""
         pass
-
-    model_config = {
-        "validate_default": True,
-    }

Diff for: src/fastmcp/resources/resource_manager.py

@@ -2,7 +2,7 @@
 
 from typing import Callable, Dict, Optional, Union
 
-from pydantic.networks import _BaseUrl
+from pydantic import AnyUrl
 
 from fastmcp.resources.base import Resource
 from fastmcp.resources.templates import ResourceTemplate
@@ -64,7 +64,7 @@ def add_template(
         self._templates[template.uri_template] = template
         return template
 
-    async def get_resource(self, uri: Union[_BaseUrl, str]) -> Optional[Resource]:
+    async def get_resource(self, uri: Union[AnyUrl, str]) -> Optional[Resource]:
         """Get resource by URI, checking concrete resources first, then templates."""
         uri_str = str(uri)
         logger.debug("Getting resource", extra={"uri": uri_str})