feat: Add new_since parameter to bug_comments to filter results by date.

README.md

@@ -28,12 +28,13 @@ The server provides the following tools for interacting with Bugzilla:
   - **Returns**: A dictionary containing all available information about the bug (status, assignee, summary, description, attachments, etc.)
   - **Example**: `bug_info(12345)` returns complete bug details
 
-- **`bug_comments(id: int, include_private_comments: bool = False)`**: Fetches all comments associated with a given bug ID.
+- **`bug_comments(id: int, include_private_comments: bool = False, new_since: Optional[datetime] = None)`**: Fetches all comments associated with a given bug ID.
   - **Parameters**:
     - `id`: The bug ID to fetch comments for
     - `include_private_comments`: Whether to include private comments (default: `False`)
+    - `new_since`: Optional datetime object to only return comments newer than this time.
   - **Returns**: A list of comment dictionaries, each containing author, timestamp, text, and privacy status
-  - **Example**: `bug_comments(12345, include_private_comments=True)` returns all comments including private ones
+  - **Example**: `bug_comments(12345, include_private_comments=True, new_since=datetime.fromisoformat("2024-01-01T00:00:00"))` returns all comments newer than Jan 1, 2024 including private ones
 
 - **`add_comment(bug_id: int, comment: str, is_private: bool = False)`**: Adds a new comment to a specified bug.
   - **Parameters**:

src/mcp_bugzilla/mcp_utils.py

@@ -9,6 +9,7 @@
 import logging
 import os
 from typing import Any, Optional
+from datetime import datetime
 from httpx_retries import RetryTransport
 import httpx
 
@@ -109,13 +110,17 @@ async def bug_info(self, bug_id: int) -> dict[str, Any]:
         mcp_log.debug(f"[BZ-RES] {data}")
         return data
 
-    async def bug_comments(self, bug_id: int) -> list[dict[str, Any]]:
+    async def bug_comments(self, bug_id: int, new_since: Optional[datetime] = None) -> list[dict[str, Any]]:
         """Get comments of a bug"""
         url = f"/bug/{bug_id}/comment"
-        mcp_log.info(f"[BZ-REQ] GET {self.api_url}{url}")
+        params = {}
+        if new_since:
+            params["new_since"] = new_since.strftime("%Y-%m-%dT%H:%M:%SZ")
+
+        mcp_log.info(f"[BZ-REQ] GET {self.api_url}{url} params={params}")
 
         try:
-            r = await self.client.get(url)
+            r = await self.client.get(url, params=params)
             r.raise_for_status()
         except httpx.HTTPStatusError as e:
             mcp_log.error(

src/mcp_bugzilla/server.py

@@ -8,6 +8,7 @@
 
 import importlib.metadata
 from typing import Any, List
+from datetime import datetime
 
 from contextlib import asynccontextmanager
 
@@ -70,19 +71,20 @@ async def bug_info(id: int, bz: Bugzilla = Depends(get_bz)) -> dict[str, Any]:
 
 @mcp.tool()
 async def bug_comments(
-    id: int, include_private_comments: bool = False, bz: Bugzilla = Depends(get_bz)
+    id: int, include_private_comments: bool = False, new_since: Optional[datetime] = None, bz: Bugzilla = Depends(get_bz)
 ) -> List[dict[str, Any]]:
     """Returns the comments of given bug id
     Private comments are not included by default
-    but can be explicitly requested
+    but can be explicitly requested.
+    new_since allows filtering comments newer than the given date.
     """
 
     mcp_log.info(
-        f"[LLM-REQ] bug_comments(id={id}, include_private_comments={include_private_comments})"
+        f"[LLM-REQ] bug_comments(id={id}, include_private_comments={include_private_comments}, new_since={new_since})"
     )
 
     try:
-        all_comments = await bz.bug_comments(id)
+        all_comments = await bz.bug_comments(id, new_since=new_since)
 
         if include_private_comments:
             mcp_log.info(

tests/test_mcp_utils.py

@@ -3,6 +3,7 @@
 import respx
 from httpx import Response
 from mcp_bugzilla.mcp_utils import Bugzilla
+from datetime import datetime
 
 MOCK_URL = "https://bugzilla.example.com"
 MOCK_API_KEY = "secret_key"
@@ -42,7 +43,7 @@ async def test_bug_info(bz_client):
 @pytest.mark.asyncio
 async def test_bug_comments(bz_client):
     async with respx.mock(base_url=MOCK_URL) as respx_mock:
-        respx_mock.get("/rest/bug/123/comment").mock(
+        route = respx_mock.get("/rest/bug/123/comment").mock(
             return_value=Response(
                 200,
                 json={
@@ -61,6 +62,13 @@ async def test_bug_comments(bz_client):
         comments = await bz_client.bug_comments(123)
         assert len(comments) == 2
         assert comments[0]["text"] == "Comment 1"
+        assert route.called
+        assert "new_since" not in route.calls.last.request.url.params
+
+        test_dt = datetime(2000, 1, 1, 0, 0, 0)
+        comments_with_since = await bz_client.bug_comments(123, new_since=test_dt)
+        assert len(comments_with_since) == 2
+        assert route.calls.last.request.url.params["new_since"] == "2000-01-01T00:00:00Z"
 
 
 @pytest.mark.asyncio