Validate required fields for MCP post creation

This reverts commit 7a2cf829c7.

mcp/src/openisle_mcp/search_client.py

@@ -74,7 +74,7 @@ class SearchClient:
     def _build_headers(
         self,
         *,
-        token: str | None = None,
+        token: str,
         accept: str = "application/json",
         include_json: bool = False,
     ) -> dict[str, str]:
@@ -111,9 +111,8 @@ class SearchClient:
     async def reply_to_comment(
         self,
         comment_id: int,
+        token: str,
         content: str,
-        *,
-        token: str | None = None,
         captcha: str | None = None,
     ) -> dict[str, Any]:
         """Reply to an existing comment and return the created reply."""
@@ -145,9 +144,8 @@ class SearchClient:
     async def reply_to_post(
         self,
         post_id: int,
+        token: str,
         content: str,
-        *,
-        token: str | None = None,
         captcha: str | None = None,
     ) -> dict[str, Any]:
         """Create a comment on a post and return the backend payload."""
@@ -180,7 +178,7 @@ class SearchClient:
         self,
         payload: dict[str, Any],
         *,
-        token: str | None = None,
+        token: str,
     ) -> dict[str, Any]:
         """Create a new post and return the detailed backend payload."""
 
@@ -200,7 +198,7 @@ class SearchClient:
         )
         response.raise_for_status()
         body = self._ensure_dict(response.json())
-        logger.info("Post creation succeeded with id=%s", body.get("id"))
+        logger.info("Post creation succeeded with id=%s, token=%s", body.get("id"), token)
         return body
 
     async def recent_posts(self, minutes: int) -> list[dict[str, Any]]:
@@ -251,7 +249,7 @@ class SearchClient:
         *,
         page: int = 0,
         size: int = 30,
-        token: str | None = None,
+        token: str,
     ) -> list[dict[str, Any]]:
         """Return unread notifications for the authenticated user."""
 
@@ -287,7 +285,7 @@ class SearchClient:
         self,
         ids: list[int],
         *,
-        token: str | None = None,
+        token: str
     ) -> None:
         """Mark the provided notifications as read for the authenticated user."""
 

mcp/src/openisle_mcp/server.py

@@ -509,6 +509,8 @@ async def create_post(
             raise ValueError("Category identifier must be an integer.") from exc
         if sanitized_category_id <= 0:
             raise ValueError("Category identifier must be a positive integer.")
+    if sanitized_category_id is None:
+        raise ValueError("A category identifier is required to create a post.")
 
     sanitized_tag_ids: list[int] | None = None
     if tag_ids is not None:
@@ -525,6 +527,10 @@ async def create_post(
             sanitized_tag_ids.append(converted)
         if not sanitized_tag_ids:
             sanitized_tag_ids = None
+    if not sanitized_tag_ids:
+        raise ValueError("At least one tag identifier is required to create a post.")
+    if len(sanitized_tag_ids) > 2:
+        raise ValueError("At most two tag identifiers can be provided for a post.")
 
     sanitized_post_type = post_type.strip() if isinstance(post_type, str) else None
     if sanitized_post_type == "":