quickcall-integrations 0.5.0__tar.gz → 0.6.0__tar.gz

{quickcall_integrations-0.5.0 → quickcall_integrations-0.6.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: quickcall-integrations
-Version: 0.5.0
+Version: 0.6.0
 Summary: MCP server with developer integrations for Claude Code and Cursor
 Requires-Python: >=3.10
 Requires-Dist: fastmcp>=2.13.0
@@ -252,10 +252,28 @@ The `manage_issues` tool provides full issue lifecycle management:
 | `close` | Close issue(s) |
 | `reopen` | Reopen issue(s) |
 | `comment` | Add comment to issue(s) |
+| `list_comments` | List comments with limit and order |
+| `update_comment` | Edit existing comment by ID |
+| `delete_comment` | Delete comment by ID |
 | `add_sub_issue` | Add child issue to parent |
 | `remove_sub_issue` | Remove child from parent |
 | `list_sub_issues` | List sub-issues of a parent |
 
+### Comment Management
+
+| Parameter | Description |
+|-----------|-------------|
+| `comment_id` | Comment ID for update/delete operations |
+| `comments_limit` | Max comments to return (default: 10) |
+| `comments_order` | `'asc'` (oldest first) or `'desc'` (newest first) |
+
+**Examples:**
+```
+List last 5 comments on issue #42 (newest first)
+Update comment 123456 on issue #42 with new text
+Delete comment 123456 from issue #42
+```
+
 ### List Filters
 
 | Filter | Description |

{quickcall_integrations-0.5.0 → quickcall_integrations-0.6.0}/README.md

@@ -239,10 +239,28 @@ The `manage_issues` tool provides full issue lifecycle management:
 | `close` | Close issue(s) |
 | `reopen` | Reopen issue(s) |
 | `comment` | Add comment to issue(s) |
+| `list_comments` | List comments with limit and order |
+| `update_comment` | Edit existing comment by ID |
+| `delete_comment` | Delete comment by ID |
 | `add_sub_issue` | Add child issue to parent |
 | `remove_sub_issue` | Remove child from parent |
 | `list_sub_issues` | List sub-issues of a parent |
 
+### Comment Management
+
+| Parameter | Description |
+|-----------|-------------|
+| `comment_id` | Comment ID for update/delete operations |
+| `comments_limit` | Max comments to return (default: 10) |
+| `comments_order` | `'asc'` (oldest first) or `'desc'` (newest first) |
+
+**Examples:**
+```
+List last 5 comments on issue #42 (newest first)
+Update comment 123456 on issue #42 with new text
+Delete comment 123456 from issue #42
+```
+
 ### List Filters
 
 | Filter | Description |

{quickcall_integrations-0.5.0 → quickcall_integrations-0.6.0}/mcp_server/api_clients/github_client.py

@@ -1301,6 +1301,145 @@ class GitHubClient:
             "issue_number": issue_number,
         }
 
+    def list_issue_comments(
+        self,
+        issue_number: int,
+        owner: Optional[str] = None,
+        repo: Optional[str] = None,
+        limit: int = 10,
+        order: str = "asc",
+    ) -> List[Dict[str, Any]]:
+        """
+        List comments on a GitHub issue.
+
+        Args:
+            issue_number: Issue number
+            owner: Repository owner
+            repo: Repository name
+            limit: Maximum comments to return (default: 10)
+            order: 'asc' for oldest first, 'desc' for newest first (default: 'asc')
+
+        Returns:
+            List of comment dicts with id, body, author, timestamps, url
+        """
+        gh_repo = self._get_repo(owner, repo)
+        issue = gh_repo.get_issue(issue_number)
+
+        comments = []
+        all_comments = list(issue.get_comments())
+
+        # Apply order
+        if order == "desc":
+            all_comments = all_comments[::-1]
+
+        # Apply limit
+        for comment in all_comments[:limit]:
+            comments.append(
+                {
+                    "id": comment.id,
+                    "body": comment.body,
+                    "author": comment.user.login if comment.user else "unknown",
+                    "created_at": comment.created_at.isoformat(),
+                    "updated_at": comment.updated_at.isoformat()
+                    if comment.updated_at
+                    else None,
+                    "html_url": comment.html_url,
+                }
+            )
+
+        return comments
+
+    def get_issue_comment(
+        self,
+        comment_id: int,
+        owner: Optional[str] = None,
+        repo: Optional[str] = None,
+    ) -> Dict[str, Any]:
+        """
+        Get a specific comment by ID.
+
+        Args:
+            comment_id: Comment ID
+            owner: Repository owner
+            repo: Repository name
+
+        Returns:
+            Comment dict with id, body, author, timestamps, url
+        """
+        gh_repo = self._get_repo(owner, repo)
+        comment = gh_repo.get_issue_comment(comment_id)
+
+        return {
+            "id": comment.id,
+            "body": comment.body,
+            "author": comment.user.login if comment.user else "unknown",
+            "created_at": comment.created_at.isoformat(),
+            "updated_at": comment.updated_at.isoformat()
+            if comment.updated_at
+            else None,
+            "html_url": comment.html_url,
+        }
+
+    def update_issue_comment(
+        self,
+        comment_id: int,
+        body: str,
+        owner: Optional[str] = None,
+        repo: Optional[str] = None,
+    ) -> Dict[str, Any]:
+        """
+        Update an existing comment.
+
+        Args:
+            comment_id: Comment ID
+            body: New comment body
+            owner: Repository owner
+            repo: Repository name
+
+        Returns:
+            Updated comment dict
+        """
+        gh_repo = self._get_repo(owner, repo)
+        comment = gh_repo.get_issue_comment(comment_id)
+        comment.edit(body)
+
+        return {
+            "id": comment.id,
+            "body": comment.body,
+            "author": comment.user.login if comment.user else "unknown",
+            "created_at": comment.created_at.isoformat(),
+            "updated_at": comment.updated_at.isoformat()
+            if comment.updated_at
+            else None,
+            "html_url": comment.html_url,
+        }
+
+    def delete_issue_comment(
+        self,
+        comment_id: int,
+        owner: Optional[str] = None,
+        repo: Optional[str] = None,
+    ) -> Dict[str, Any]:
+        """
+        Delete a comment.
+
+        Args:
+            comment_id: Comment ID
+            owner: Repository owner
+            repo: Repository name
+
+        Returns:
+            Dict with deleted comment_id
+        """
+        gh_repo = self._get_repo(owner, repo)
+        comment = gh_repo.get_issue_comment(comment_id)
+        comment.delete()
+
+        return {
+            "deleted": True,
+            "comment_id": comment_id,
+        }
+
     def get_issue(
         self,
         issue_number: int,

{quickcall_integrations-0.5.0 → quickcall_integrations-0.6.0}/mcp_server/tools/github_tools.py

@@ -571,11 +571,12 @@ def create_github_tools(mcp: FastMCP) -> None:
         action: str = Field(
             ...,
             description="Action: 'list', 'view', 'create', 'update', 'close', 'reopen', 'comment', "
+            "'list_comments', 'update_comment', 'delete_comment', "
             "'add_sub_issue', 'remove_sub_issue', 'list_sub_issues'",
         ),
         issue_numbers: Optional[List[int]] = Field(
             default=None,
-            description="Issue number(s). Required for view/update/close/reopen/comment/sub-issue/project ops.",
+            description="Issue number(s). Required for view/update/close/reopen/comment/sub-issue/comment ops.",
         ),
         title: Optional[str] = Field(
             default=None,
@@ -583,7 +584,7 @@ def create_github_tools(mcp: FastMCP) -> None:
         ),
         body: Optional[str] = Field(
             default=None,
-            description="Issue body (for 'create'/'update') or comment text (for 'comment')",
+            description="Issue body (for 'create'/'update') or comment text (for 'comment'/'update_comment')",
         ),
         labels: Optional[List[str]] = Field(
             default=None,
@@ -602,6 +603,18 @@ def create_github_tools(mcp: FastMCP) -> None:
             description="Parent issue number. For 'create': attach new issue as sub-issue. "
             "For 'add_sub_issue'/'remove_sub_issue'/'list_sub_issues': the parent issue.",
         ),
+        comment_id: Optional[int] = Field(
+            default=None,
+            description="Comment ID for 'update_comment' or 'delete_comment' actions.",
+        ),
+        comments_limit: Optional[int] = Field(
+            default=10,
+            description="Max comments to return for 'list_comments' (default: 10).",
+        ),
+        comments_order: Optional[str] = Field(
+            default="asc",
+            description="Comment order for 'list_comments': 'asc' (oldest first) or 'desc' (newest first).",
+        ),
         owner: Optional[str] = Field(
             default=None,
             description="Repository owner",
@@ -644,6 +657,10 @@ def create_github_tools(mcp: FastMCP) -> None:
         - create as sub-issue: manage_issues(action="create", title="Task 1", parent_issue=42)
         - close multiple: manage_issues(action="close", issue_numbers=[1, 2, 3])
         - comment: manage_issues(action="comment", issue_numbers=[42], body="Fixed!")
+        - list comments: manage_issues(action="list_comments", issue_numbers=[42], comments_limit=5)
+        - list newest comments: manage_issues(action="list_comments", issue_numbers=[42], comments_order="desc")
+        - update comment: manage_issues(action="update_comment", issue_numbers=[42], comment_id=123, body="Updated")
+        - delete comment: manage_issues(action="delete_comment", issue_numbers=[42], comment_id=123)
         - add sub-issues: manage_issues(action="add_sub_issue", issue_numbers=[43,44], parent_issue=42)
         - remove sub-issue: manage_issues(action="remove_sub_issue", issue_numbers=[43], parent_issue=42)
         - list sub-issues: manage_issues(action="list_sub_issues", parent_issue=42)
@@ -830,6 +847,66 @@ def create_github_tools(mcp: FastMCP) -> None:
                         }
                     )
 
+                # === LIST COMMENTS ACTION ===
+                elif action == "list_comments":
+                    comments = client.list_issue_comments(
+                        issue_number=issue_number,
+                        owner=owner,
+                        repo=repo,
+                        limit=comments_limit or 10,
+                        order=comments_order or "asc",
+                    )
+                    results.append(
+                        {
+                            "number": issue_number,
+                            "comments_count": len(comments),
+                            "comments": comments,
+                        }
+                    )
+
+                # === UPDATE COMMENT ACTION ===
+                elif action == "update_comment":
+                    if not comment_id:
+                        raise ToolError(
+                            "'comment_id' is required for 'update_comment' action"
+                        )
+                    if not body:
+                        raise ToolError(
+                            "'body' is required for 'update_comment' action"
+                        )
+                    updated = client.update_issue_comment(
+                        comment_id=comment_id,
+                        body=body,
+                        owner=owner,
+                        repo=repo,
+                    )
+                    results.append(
+                        {
+                            "number": issue_number,
+                            "status": "comment_updated",
+                            "comment": updated,
+                        }
+                    )
+
+                # === DELETE COMMENT ACTION ===
+                elif action == "delete_comment":
+                    if not comment_id:
+                        raise ToolError(
+                            "'comment_id' is required for 'delete_comment' action"
+                        )
+                    client.delete_issue_comment(
+                        comment_id=comment_id,
+                        owner=owner,
+                        repo=repo,
+                    )
+                    results.append(
+                        {
+                            "number": issue_number,
+                            "status": "comment_deleted",
+                            "comment_id": comment_id,
+                        }
+                    )
+
                 else:
                     raise ToolError(f"Invalid action: {action}")
 
@@ -841,6 +918,13 @@ def create_github_tools(mcp: FastMCP) -> None:
                     "issues": results,
                 }
 
+            if action == "list_comments":
+                return {
+                    "action": "list_comments",
+                    "count": len(results),
+                    "issues": results,
+                }
+
             return {"action": action, "count": len(results), "results": results}
 
         except ToolError:

{quickcall_integrations-0.5.0 → quickcall_integrations-0.6.0}/plugins/quickcall/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "quickcall",
   "description": "Integrate quickcall into dev workflows - eliminate interruptions for developers. Ask about your work, get instant answers. No more context switching.",
-  "version": "0.8.0",
+  "version": "0.9.0",
   "author": {
     "name": "Sagar Sarkale"
   }

{quickcall_integrations-0.5.0 → quickcall_integrations-0.6.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "quickcall-integrations"
-version = "0.5.0"
+version = "0.6.0"
 description = "MCP server with developer integrations for Claude Code and Cursor"
 readme = "README.md"
 requires-python = ">=3.10"

quickcall_integrations-0.6.0/tests/test_comment_management.py

@@ -0,0 +1,271 @@
+#!/usr/bin/env python3
+"""
+Integration test for Issue Comment Management.
+
+This test performs a full round-trip:
+1. Creates a test issue
+2. Adds multiple comments
+3. Lists comments (asc and desc order)
+4. Updates a comment
+5. Deletes a comment
+6. Closes/cleans up the test issue
+
+Run with: uv run python tests/test_comment_management.py
+
+Requires:
+- GITHUB_TOKEN env var or .quickcall.env with GitHub PAT
+- Access to quickcall-dev/quickcall-integrations repo
+"""
+
+import sys
+import time
+
+
+def run_integration_test():
+    """Run the full comment management integration test."""
+    from mcp_server.api_clients.github_client import GitHubClient
+    from mcp_server.auth import get_github_pat
+
+    print("=" * 60)
+    print("Issue Comment Management Integration Test")
+    print("=" * 60)
+
+    # Check for credentials
+    pat_token, source = get_github_pat()
+    if not pat_token:
+        print("\n❌ No GitHub PAT found!")
+        print("   Set GITHUB_TOKEN env var or create .quickcall.env")
+        return False
+
+    print(f"\n✅ Using PAT from: {source}")
+
+    client = GitHubClient(token=pat_token)
+    username = client.get_authenticated_user()
+    print(f"✅ Authenticated as: {username}")
+
+    # Configuration for test
+    TEST_ORG = "quickcall-dev"
+    TEST_REPO = "quickcall-integrations"
+    TEST_ISSUE_TITLE = "[TEST] Comment Management Test - Safe to Delete"
+
+    created_issue_number = None
+    comment_ids = []
+
+    try:
+        # Step 1: Create a test issue
+        print("\n--- Step 1: Create test issue ---")
+        issue = client.create_issue(
+            title=TEST_ISSUE_TITLE,
+            body="This is an automated test issue for comment management.\n\n"
+            "**This issue will be closed automatically after the test.**",
+            labels=["test"],
+            owner=TEST_ORG,
+            repo=TEST_REPO,
+        )
+        created_issue_number = issue["number"]
+        print(f"✅ Created issue #{created_issue_number}")
+        print(f"   URL: {issue['html_url']}")
+
+        time.sleep(1)
+
+        # Step 2: Add multiple comments
+        print("\n--- Step 2: Add 3 comments ---")
+        for i in range(1, 4):
+            comment = client.comment_on_issue(
+                issue_number=created_issue_number,
+                body=f"Test comment #{i}",
+                owner=TEST_ORG,
+                repo=TEST_REPO,
+            )
+            comment_ids.append(comment["id"])
+            print(f"✅ Added comment #{i} (ID: {comment['id']})")
+            time.sleep(0.5)
+
+        # Step 3: List comments (ascending - oldest first)
+        print("\n--- Step 3: List comments (oldest first) ---")
+        comments_asc = client.list_issue_comments(
+            issue_number=created_issue_number,
+            owner=TEST_ORG,
+            repo=TEST_REPO,
+            limit=10,
+            order="asc",
+        )
+        print(f"✅ Found {len(comments_asc)} comments (asc)")
+        for c in comments_asc:
+            print(f"   - {c['body'][:50]} (by {c['author']})")
+
+        # Step 4: List comments (descending - newest first)
+        print("\n--- Step 4: List comments (newest first) ---")
+        comments_desc = client.list_issue_comments(
+            issue_number=created_issue_number,
+            owner=TEST_ORG,
+            repo=TEST_REPO,
+            limit=10,
+            order="desc",
+        )
+        print(f"✅ Found {len(comments_desc)} comments (desc)")
+        for c in comments_desc:
+            print(f"   - {c['body'][:50]} (by {c['author']})")
+
+        # Verify order is reversed
+        if comments_asc and comments_desc:
+            assert comments_asc[0]["id"] == comments_desc[-1]["id"], (
+                "Order should be reversed"
+            )
+            print("✅ Order verification passed")
+
+        # Step 5: List with limit
+        print("\n--- Step 5: List with limit=2 ---")
+        comments_limited = client.list_issue_comments(
+            issue_number=created_issue_number,
+            owner=TEST_ORG,
+            repo=TEST_REPO,
+            limit=2,
+            order="asc",
+        )
+        print(f"✅ Found {len(comments_limited)} comments (limited to 2)")
+        assert len(comments_limited) == 2, "Should return exactly 2 comments"
+
+        # Step 6: Update a comment
+        print("\n--- Step 6: Update comment ---")
+        comment_to_update = comment_ids[1]  # Update the second comment
+        updated = client.update_issue_comment(
+            comment_id=comment_to_update,
+            body="Test comment #2 (UPDATED)",
+            owner=TEST_ORG,
+            repo=TEST_REPO,
+        )
+        print(f"✅ Updated comment ID {comment_to_update}")
+        print(f"   New body: {updated['body']}")
+        assert "UPDATED" in updated["body"], "Comment should be updated"
+
+        # Step 7: Get specific comment
+        print("\n--- Step 7: Get specific comment ---")
+        fetched = client.get_issue_comment(
+            comment_id=comment_to_update,
+            owner=TEST_ORG,
+            repo=TEST_REPO,
+        )
+        print(f"✅ Fetched comment ID {fetched['id']}")
+        print(f"   Body: {fetched['body']}")
+
+        # Step 8: Delete a comment
+        print("\n--- Step 8: Delete comment ---")
+        comment_to_delete = comment_ids[2]  # Delete the third comment
+        result = client.delete_issue_comment(
+            comment_id=comment_to_delete,
+            owner=TEST_ORG,
+            repo=TEST_REPO,
+        )
+        print(f"✅ Deleted comment ID {comment_to_delete}")
+        assert result["deleted"] is True, "Should return deleted=True"
+
+        # Verify deletion
+        time.sleep(1)
+        comments_after = client.list_issue_comments(
+            issue_number=created_issue_number,
+            owner=TEST_ORG,
+            repo=TEST_REPO,
+            limit=10,
+        )
+        print(f"✅ Verified: now {len(comments_after)} comments (was 3)")
+        assert len(comments_after) == 2, "Should have 2 comments after deletion"
+
+        # Step 9: Cleanup - close issue
+        print("\n--- Step 9: Close test issue ---")
+        client.close_issue(created_issue_number, owner=TEST_ORG, repo=TEST_REPO)
+        print(f"✅ Closed issue #{created_issue_number}")
+
+        print("\n" + "=" * 60)
+        print("✅ ALL COMMENT MANAGEMENT TESTS PASSED!")
+        print("=" * 60)
+        return True
+
+    except Exception as e:
+        print(f"\n❌ Test failed with error: {e}")
+        import traceback
+
+        traceback.print_exc()
+
+        # Cleanup
+        if created_issue_number:
+            try:
+                print(f"\n--- Cleanup: Closing issue #{created_issue_number} ---")
+                client.close_issue(created_issue_number, owner=TEST_ORG, repo=TEST_REPO)
+                print(f"✅ Closed issue #{created_issue_number}")
+            except Exception as cleanup_error:
+                print(f"⚠️  Cleanup failed: {cleanup_error}")
+
+        return False
+
+
+def test_error_handling():
+    """Test error handling for invalid comment operations."""
+    from mcp_server.api_clients.github_client import GitHubClient
+    from github import GithubException, UnknownObjectException
+    from mcp_server.auth import get_github_pat
+
+    print("\n" + "=" * 60)
+    print("Test: Error Handling")
+    print("=" * 60)
+
+    pat_token, _ = get_github_pat()
+    if not pat_token:
+        print("⚠️  Skipping - no PAT")
+        return True
+
+    client = GitHubClient(token=pat_token)
+
+    # Test 1: Get non-existent comment
+    print("\nTest 1: Get non-existent comment")
+    try:
+        client.get_issue_comment(
+            comment_id=999999999,
+            owner="quickcall-dev",
+            repo="quickcall-integrations",
+        )
+        print("❌ Should have raised an exception")
+        return False
+    except (GithubException, UnknownObjectException):
+        print("✅ Correctly raised exception for non-existent comment")
+
+    # Test 2: Update non-existent comment
+    print("\nTest 2: Update non-existent comment")
+    try:
+        client.update_issue_comment(
+            comment_id=999999999,
+            body="test",
+            owner="quickcall-dev",
+            repo="quickcall-integrations",
+        )
+        print("❌ Should have raised an exception")
+        return False
+    except (GithubException, UnknownObjectException):
+        print("✅ Correctly raised exception for update")
+
+    print("\n✅ All error handling tests passed!")
+    return True
+
+
+if __name__ == "__main__":
+    results = []
+
+    # Run main integration test
+    results.append(("Full Comment Management Test", run_integration_test()))
+
+    # Run error handling test
+    results.append(("Error Handling", test_error_handling()))
+
+    # Summary
+    print("\n" + "=" * 60)
+    print("TEST SUMMARY")
+    print("=" * 60)
+    all_passed = True
+    for name, passed in results:
+        status = "✅ PASSED" if passed else "❌ FAILED"
+        print(f"  {status}: {name}")
+        if not passed:
+            all_passed = False
+
+    print("=" * 60)
+    sys.exit(0 if all_passed else 1)