Keep skill resource verification in the shared API

Author: Kludex

docs_src/skills/tutorial001_client.py

@@ -1,7 +1,8 @@
 import anyio
 
 from mcp import Client
-from mcp.client.skills import Skills, verify_skill_resource
+from mcp.client.skills import Skills
+from mcp.shared.skills import verify_skill_resource
 from mcp.types import TextResourceContents
 
 

src/mcp/client/skills.py

@@ -37,9 +37,8 @@
     Skill,
     validate_directory_result,
 )
-from mcp.shared.skills import verify_skill_resource as verify_skill_resource
 
-__all__ = ["BoundSkills", "Skills", "verify_skill_resource"]
+__all__ = ["BoundSkills", "Skills"]
 
 
 class Skills(ClientExtension):

tests/client/test_skills.py

@@ -9,7 +9,7 @@
 from mcp_types import INVALID_PARAMS, Resource, TextResourceContents
 
 from mcp.client.client import Client
-from mcp.client.skills import BoundSkills, verify_skill_resource
+from mcp.client.skills import BoundSkills
 from mcp.client.skills import Skills as ClientSkills
 from mcp.server.context import ServerRequestContext
 from mcp.server.extension import Extension, MethodBinding
@@ -28,6 +28,7 @@
     ReadDirectoryResult,
     Skill,
     SkillResource,
+    verify_skill_resource,
 )
 
 pytestmark = pytest.mark.anyio

tests/docs_src/test_skills.py

@@ -5,8 +5,9 @@
 
 from docs_src.skills import tutorial001
 from mcp import Client
-from mcp.client.skills import Skills, verify_skill_resource
+from mcp.client.skills import Skills
 from mcp.shared.exceptions import MCPError
+from mcp.shared.skills import verify_skill_resource
 
 pytestmark = pytest.mark.anyio