removed extra logging and extra exports

Author: vijay

src/mcp/server/skills.py

@@ -28,7 +28,6 @@ async def get_skill(ctx, params):
 
 from __future__ import annotations
 
-import logging
 from collections.abc import Awaitable, Callable, Sequence
 from typing import Any
 
@@ -58,8 +57,6 @@ async def get_skill(ctx, params):
 
 __all__ = ["Skills"]
 
-logger = logging.getLogger(__name__)
-
 ListSkillsHandler = Callable[[ServerRequestContext[Any, Any], ListSkillsParams], Awaitable[ListSkillsResult]]
 GetSkillHandler = Callable[[ServerRequestContext[Any, Any], GetSkillParams], Awaitable[GetSkillResult]]
 ReadDirectoryHandler = Callable[[ServerRequestContext[Any, Any], ReadDirectoryParams], Awaitable[ReadDirectoryResult]]
@@ -116,7 +113,6 @@ async def _handle_list(self, ctx: ServerRequestContext[Any, Any], params: ListSk
             result = await self._list_skills(ctx, params)
             ListSkillsResult.model_validate(result.model_dump())
         except ValidationError:
-            logger.exception("list_skills handler returned an invalid result")
             raise MCPError(code=INTERNAL_ERROR, message="Handler returned an invalid result") from None
         return _finalize_cacheable(result, ctx.protocol_version)
 
@@ -128,10 +124,8 @@ async def _handle_get(self, ctx: ServerRequestContext[Any, Any], params: GetSkil
             result = await self._get_skill(ctx, params)
             GetSkillResult.model_validate(result.model_dump())
         except ValidationError:
-            logger.exception("get_skill handler returned an invalid result")
             raise MCPError(code=INTERNAL_ERROR, message="Handler returned an invalid result") from None
         if result.skill.uri != params.uri:
-            logger.error("get_skill handler returned %r for requested %r", result.skill.uri, params.uri)
             raise MCPError(code=INTERNAL_ERROR, message="Handler returned an invalid result")
         return _finalize_cacheable(result, ctx.protocol_version)
 
@@ -144,7 +138,6 @@ async def _handle_read_directory(
         try:
             validate_directory_result(params.uri, result)
         except ValueError:
-            logger.exception("read_directory handler returned an invalid result")
             raise MCPError(code=INTERNAL_ERROR, message="Handler returned an invalid result") from None
         return result
 

src/mcp/shared/skills.py

@@ -17,12 +17,6 @@
 from pydantic.alias_generators import to_camel
 
 __all__ = [
-    "EXTENSION_ID",
-    "METHOD_LIST",
-    "METHOD_GET",
-    "METHOD_READ_DIRECTORY",
-    "MAX_RESOURCES_PER_SKILL",
-    "MAX_TOTAL_SIZE",
     "Frontmatter",
     "SkillResource",
     "SkillResources",
@@ -33,12 +27,6 @@
     "GetSkillResult",
     "ReadDirectoryParams",
     "ReadDirectoryResult",
-    "ListSkillsRequest",
-    "GetSkillRequest",
-    "ReadDirectoryRequest",
-    "skill_name_from_uri",
-    "parse_directory_uri",
-    "validate_directory_result",
     "verify_skill_resource",
 ]