fix: Make context log extra field dict

Author: vincent0426

src/mcp/server/fastmcp/server.py

@@ -1206,15 +1206,15 @@ async def log(
         message: str,
         *,
         logger_name: str | None = None,
-        **extra: Any,
+        extra: dict[str, Any] | None = None,
     ) -> None:
         """Send a log message to the client.
 
         Args:
             level: Log level (debug, info, warning, error)
             message: Log message
             logger_name: Optional logger name
-            **extra: Additional structured data to include
+            extra: Optional dictionary with additional structured data to include
         """
 
         if extra:
@@ -1248,18 +1248,20 @@ def session(self):
         return self.request_context.session
 
     # Convenience methods for common log levels
-    async def debug(self, message: str, **extra: Any) -> None:
+    async def debug(self, message: str, *, logger_name: str | None = None, extra: dict[str, Any] | None = None) -> None:
         """Send a debug log message."""
-        await self.log("debug", message, **extra)
+        await self.log("debug", message, logger_name=logger_name, extra=extra)
 
-    async def info(self, message: str, **extra: Any) -> None:
+    async def info(self, message: str, *, logger_name: str | None = None, extra: dict[str, Any] | None = None) -> None:
         """Send an info log message."""
-        await self.log("info", message, **extra)
+        await self.log("info", message, logger_name=logger_name, extra=extra)
 
-    async def warning(self, message: str, **extra: Any) -> None:
+    async def warning(
+        self, message: str, *, logger_name: str | None = None, extra: dict[str, Any] | None = None
+    ) -> None:
         """Send a warning log message."""
-        await self.log("warning", message, **extra)
+        await self.log("warning", message, logger_name=logger_name, extra=extra)
 
-    async def error(self, message: str, **extra: Any) -> None:
+    async def error(self, message: str, *, logger_name: str | None = None, extra: dict[str, Any] | None = None) -> None:
         """Send an error log message."""
-        await self.log("error", message, **extra)
+        await self.log("error", message, logger_name=logger_name, extra=extra)

tests/client/test_logging_callback.py

@@ -60,8 +60,7 @@ async def test_tool_with_log_extra(
             level=level,
             message=message,
             logger_name=logger,
-            extra_string=extra_string,
-            extra_dict=extra_dict,
+            extra={"extra_string": extra_string, "extra_dict": extra_dict},
         )
         return True