fix: update env_schema type from str to dict in McpServerDetail and related tools

Author: pengqun

src/modelscope_mcp_server/tools/mcp.py

@@ -135,7 +135,7 @@ async def get_mcp_server_detail(
             # Additional fields
             author=server_data.get("author", ""),
             server_config=server_data.get("server_config", []),
-            env_schema=server_data.get("env_schema", ""),
+            env_schema=server_data.get("env_schema", {}),
             is_hosted=server_data.get("is_hosted", False),
             is_verified=server_data.get("is_verified", False),
             source_url=server_data.get("source_url", ""),

src/modelscope_mcp_server/types.py

@@ -159,7 +159,7 @@ class McpServerDetail(McpServer):
 
     # Configuration
     server_config: Annotated[list[dict], Field(description="Server configuration")] = []
-    env_schema: Annotated[str, Field(description="JSON schema for environment variables")]
+    env_schema: Annotated[dict, Field(description="JSON schema for environment variables")]
 
     # Status flags
     is_hosted: Annotated[bool, Field(description="Whether the server supports hosted mode")]

tests/tools/test_get_mcp_server_detail.py

@@ -52,7 +52,6 @@ def validate_server_detail_fields(server_detail):
 
     for field in required_fields:
         assert hasattr(server_detail, field), f"Server detail should have '{field}' attribute"
-
     # Validate field types
     assert isinstance(server_detail.id, str), "ID should be a string"
     assert isinstance(server_detail.name, str), "Name should be a string"
@@ -62,7 +61,6 @@ def validate_server_detail_fields(server_detail):
     assert isinstance(server_detail.view_count, int), "View count should be an integer"
     assert isinstance(server_detail.github_stars, int), "GitHub stars should be an integer"
     assert isinstance(server_detail.server_config, list), "Server config should be a list"
-    assert isinstance(server_detail.env_schema, str), "Env schema should be a string"
     assert isinstance(server_detail.readme, str), "Readme should be a string"
     assert isinstance(server_detail.logo_url, str), "Logo URL should be a string"