Merge pull request #42 from meilisearch/fix/remove-unnecessary-async

Fix: Remove unnecessary async declarations from synchronous methods

Author: tpayet

src/meilisearch_mcp/client.py

@@ -28,23 +28,23 @@ def __init__(
         self.keys = KeyManager(self.client)
         self.monitoring = MonitoringManager(self.client)
 
-    async def health_check(self) -> bool:
+    def health_check(self) -> bool:
         """Check if Meilisearch is healthy"""
         try:
             response = self.client.health()
             return response.get("status") == "available"
         except Exception:
             return False
 
-    async def get_version(self) -> Dict[str, Any]:
+    def get_version(self) -> Dict[str, Any]:
         """Get Meilisearch version information"""
         return self.client.get_version()
 
-    async def get_stats(self) -> Dict[str, Any]:
+    def get_stats(self) -> Dict[str, Any]:
         """Get database stats"""
         return self.client.get_all_stats()
 
-    async def search(
+    def search(
         self,
         query: str,
         index_uid: Optional[str] = None,
@@ -97,7 +97,7 @@ async def search(
         except Exception as e:
             raise Exception(f"Search failed: {str(e)}")
 
-    async def get_indexes(self) -> Dict[str, Any]:
+    def get_indexes(self) -> Dict[str, Any]:
         """Get all indexes"""
         indexes = self.client.get_indexes()
         # Convert Index objects to serializable dictionaries

src/meilisearch_mcp/documents.py

@@ -8,7 +8,7 @@ class DocumentManager:
     def __init__(self, client: Client):
         self.client = client
 
-    async def get_documents(
+    def get_documents(
         self,
         index_uid: str,
         offset: Optional[int] = None,
@@ -57,7 +57,7 @@ async def get_documents(
         except Exception as e:
             raise Exception(f"Failed to get documents: {str(e)}")
 
-    async def get_document(
+    def get_document(
         self, index_uid: str, document_id: Union[str, int]
     ) -> Dict[str, Any]:
         """Get a single document"""
@@ -67,7 +67,7 @@ async def get_document(
         except Exception as e:
             raise Exception(f"Failed to get document: {str(e)}")
 
-    async def add_documents(
+    def add_documents(
         self,
         index_uid: str,
         documents: List[Dict[str, Any]],
@@ -80,7 +80,7 @@ async def add_documents(
         except Exception as e:
             raise Exception(f"Failed to add documents: {str(e)}")
 
-    async def update_documents(
+    def update_documents(
         self, index_uid: str, documents: List[Dict[str, Any]]
     ) -> Dict[str, Any]:
         """Update documents in an index"""
@@ -90,7 +90,7 @@ async def update_documents(
         except Exception as e:
             raise Exception(f"Failed to update documents: {str(e)}")
 
-    async def delete_document(
+    def delete_document(
         self, index_uid: str, document_id: Union[str, int]
     ) -> Dict[str, Any]:
         """Delete a single document"""
@@ -100,7 +100,7 @@ async def delete_document(
         except Exception as e:
             raise Exception(f"Failed to delete document: {str(e)}")
 
-    async def delete_documents(
+    def delete_documents(
         self, index_uid: str, document_ids: List[Union[str, int]]
     ) -> Dict[str, Any]:
         """Delete multiple documents by ID"""
@@ -110,7 +110,7 @@ async def delete_documents(
         except Exception as e:
             raise Exception(f"Failed to delete documents: {str(e)}")
 
-    async def delete_all_documents(self, index_uid: str) -> Dict[str, Any]:
+    def delete_all_documents(self, index_uid: str) -> Dict[str, Any]:
         """Delete all documents in an index"""
         try:
             index = self.client.index(index_uid)

src/meilisearch_mcp/indexes.py

@@ -17,7 +17,7 @@ class IndexManager:
     def __init__(self, client: Client):
         self.client = client
 
-    async def create_index(
+    def create_index(
         self, uid: str, primary_key: Optional[str] = None
     ) -> Dict[str, Any]:
         """Create a new index"""
@@ -26,35 +26,35 @@ async def create_index(
         except Exception as e:
             raise Exception(f"Failed to create index: {str(e)}")
 
-    async def get_index(self, uid: str) -> Dict[str, Any]:
+    def get_index(self, uid: str) -> Dict[str, Any]:
         """Get index information"""
         try:
             return self.client.get_index(uid)
         except Exception as e:
             raise Exception(f"Failed to get index: {str(e)}")
 
-    async def list_indexes(self) -> List[Dict[str, Any]]:
+    def list_indexes(self) -> List[Dict[str, Any]]:
         """List all indexes"""
         try:
             return self.client.get_indexes()
         except Exception as e:
             raise Exception(f"Failed to list indexes: {str(e)}")
 
-    async def delete_index(self, uid: str) -> Dict[str, Any]:
+    def delete_index(self, uid: str) -> Dict[str, Any]:
         """Delete an index"""
         try:
             return self.client.delete_index(uid)
         except Exception as e:
             raise Exception(f"Failed to delete index: {str(e)}")
 
-    async def update_index(self, uid: str, primary_key: str) -> Dict[str, Any]:
+    def update_index(self, uid: str, primary_key: str) -> Dict[str, Any]:
         """Update index primary key"""
         try:
             return self.client.update_index(uid, {"primaryKey": primary_key})
         except Exception as e:
             raise Exception(f"Failed to update index: {str(e)}")
 
-    async def swap_indexes(self, indexes: List[List[str]]) -> Dict[str, Any]:
+    def swap_indexes(self, indexes: List[List[str]]) -> Dict[str, Any]:
         """Swap indexes"""
         try:
             return self.client.swap_indexes(indexes)

src/meilisearch_mcp/keys.py

@@ -9,7 +9,7 @@ class KeyManager:
     def __init__(self, client: Client):
         self.client = client
 
-    async def get_keys(
+    def get_keys(
         self, parameters: Optional[Dict[str, Any]] = None
     ) -> Dict[str, Any]:
         """Get list of API keys"""
@@ -18,28 +18,28 @@ async def get_keys(
         except Exception as e:
             raise Exception(f"Failed to get keys: {str(e)}")
 
-    async def get_key(self, key: str) -> Dict[str, Any]:
+    def get_key(self, key: str) -> Dict[str, Any]:
         """Get information about a specific key"""
         try:
             return self.client.get_key(key)
         except Exception as e:
             raise Exception(f"Failed to get key: {str(e)}")
 
-    async def create_key(self, options: Dict[str, Any]) -> Dict[str, Any]:
+    def create_key(self, options: Dict[str, Any]) -> Dict[str, Any]:
         """Create a new API key"""
         try:
             return self.client.create_key(options)
         except Exception as e:
             raise Exception(f"Failed to create key: {str(e)}")
 
-    async def update_key(self, key: str, options: Dict[str, Any]) -> Dict[str, Any]:
+    def update_key(self, key: str, options: Dict[str, Any]) -> Dict[str, Any]:
         """Update an existing API key"""
         try:
             return self.client.update_key(key, options)
         except Exception as e:
             raise Exception(f"Failed to update key: {str(e)}")
 
-    async def delete_key(self, key: str) -> None:
+    def delete_key(self, key: str) -> None:
         """Delete an API key"""
         try:
             return self.client.delete_key(key)

src/meilisearch_mcp/monitoring.py

@@ -32,7 +32,7 @@ class MonitoringManager:
     def __init__(self, client: Client):
         self.client = client
 
-    async def get_health_status(self) -> HealthStatus:
+    def get_health_status(self) -> HealthStatus:
         """Get comprehensive health status"""
         try:
             # Get various stats to build health picture
@@ -62,7 +62,7 @@ async def get_health_status(self) -> HealthStatus:
         except Exception as e:
             raise Exception(f"Failed to get health status: {str(e)}")
 
-    async def get_index_metrics(self, index_uid: str) -> IndexMetrics:
+    def get_index_metrics(self, index_uid: str) -> IndexMetrics:
         """Get detailed metrics for an index"""
         try:
             index = self.client.index(index_uid)
@@ -77,7 +77,7 @@ async def get_index_metrics(self, index_uid: str) -> IndexMetrics:
         except Exception as e:
             raise Exception(f"Failed to get index metrics: {str(e)}")
 
-    async def get_system_information(self) -> Dict[str, Any]:
+    def get_system_information(self) -> Dict[str, Any]:
         """Get system-level information"""
         try:
             version = self.client.get_version()