Bug fix

Author: maltekuehl

pyproject.toml

@@ -4,7 +4,7 @@ requires = [ "hatchling" ]
 
 [project]
 name = "protocol-mcp"
-version = "0.1.0"
+version = "0.1.1"
 description = "MCP that connects to wetlab protocol resources, including protocols.io"
 readme = "README.md"
 license = { file = "LICENSE" }

server.json

@@ -1,7 +1,7 @@
 {
 	"name": "protocol-mcp",
 	"description": "MCP server that connects to wetlab protocol resources, including protocols.io",
-	"version": "0.1.0",
+	"version": "0.1.1",
 	"repository": "https://github.com/biocontext-ai/protocol-mcp",
 	"tools": [
 		{
@@ -67,8 +67,8 @@
 		]
 	},
 	"installation": {
-		"uv": "uvx protocol-mcp",
-		"pip": "pip install protocol-mcp"
+		"uv": "uvx protocol_mcp",
+		"pip": "pip install protocol_mcp"
 	},
 	"run": {
 		"command": "protocol_mcp",

src/protocol_mcp/clients/protocols_io.py

@@ -83,7 +83,9 @@ async def search_protocols(
         query: str,
         page_size: int = 10,
         page_id: int = 1,
-        filter_params: dict[str, Any] | None = None,
+        filter_type: str = "public",
+        order_field: str = "date",
+        order_dir: str = "desc",
     ) -> dict[str, Any]:
         """Search for protocols using v3 API.
 
@@ -92,11 +94,17 @@ async def search_protocols(
         query : str
             Search query string.
         page_size : int
-            Number of results per page.
+            Number of results per page (1-100).
         page_id : int
             Page number (1-indexed).
-        filter_params : dict[str, Any] | None
-            Additional filter parameters.
+        filter_type : str
+            Protocol filter type. One of: 'public', 'user_public',
+            'user_private', 'shared_with_user'. Defaults to 'public'.
+        order_field : str
+            Sort field: 'activity', 'date', 'name', or 'id'.
+            Defaults to 'date'.
+        order_dir : str
+            Sort direction: 'asc' or 'desc'. Defaults to 'desc'.
 
         Returns
         -------
@@ -105,13 +113,13 @@ async def search_protocols(
         """
         url = f"{self.config.base_url_v3}/protocols"
         params = {
+            "filter": filter_type,
             "key": query,
             "page_size": page_size,
             "page_id": page_id,
-            "order_field": "relevance",
+            "order_field": order_field,
+            "order_dir": order_dir,
         }
-        if filter_params:
-            params.update(filter_params)
         return await self._request("GET", url, params=params)
 
     async def get_protocol(

src/protocol_mcp/services/protocols_io.py

@@ -121,21 +121,12 @@ async def search_protocols(
     ProtocolSearchResponse
         Search results with pagination info.
     """
-    filter_params = {}
-    if peer_reviewed_only:
-        filter_params["filter"] = "peer_reviewed"
-    else:
-        # Default to searching public protocols
-        filter_params["filter"] = "public"
-
-    # Use date ordering instead of relevance (which has a DB error)
-    filter_params["order_field"] = "date"
-    filter_params["order_dir"] = "desc"
+    filter_type = "peer_reviewed" if peer_reviewed_only else "public"
 
     response = await client.search_protocols(
         query=query,
         page_size=max_results,
-        filter_params=filter_params,
+        filter_type=filter_type,
     )
 
     items = [_parse_search_item(item) for item in response.get("items", [])]