mcp-ticketer 0.4.0__py3-none-any.whl → 0.4.2__py3-none-any.whl

mcp_ticketer/cli/mcp_configure.py

@@ -16,13 +16,21 @@ def find_mcp_ticketer_binary() -> str:
     """Find the mcp-ticketer binary path.
 
     Returns:
-        Path to mcp-ticketer binary
+        Path to mcp-ticketer binary (prefers simple 'mcp-ticketer' if in PATH)
 
     Raises:
         FileNotFoundError: If binary not found
 
     """
-    # Check if running from development environment
+    # PRIORITY 1: Check PATH first (like kuzu-memory)
+    # This allows the system to resolve the binary location
+    which_result = shutil.which("mcp-ticketer")
+    if which_result:
+        # Return just "mcp-ticketer" for PATH-based installations
+        # This is more portable and matches kuzu-memory's approach
+        return "mcp-ticketer"
+
+    # FALLBACK: Check development environment
     import mcp_ticketer
 
     package_path = Path(mcp_ticketer.__file__).parent.parent.parent
@@ -45,11 +53,6 @@ def find_mcp_ticketer_binary() -> str:
         / "mcp-ticketer",
     ]
 
-    # Check PATH
-    which_result = shutil.which("mcp-ticketer")
-    if which_result:
-        return which_result
-
     # Check possible paths
     for path in possible_paths:
         if path.exists():
@@ -183,7 +186,7 @@ def create_mcp_server_config(
     """
     config = {
         "command": binary_path,
-        "args": ["serve"],  # Use 'serve' command to start MCP server
+        "args": ["mcp", "serve"],  # Use 'mcp serve' command to start MCP server
     }
 
     # Add working directory if provided
@@ -214,6 +217,66 @@ def create_mcp_server_config(
     return config
 
 
+def remove_claude_mcp(global_config: bool = False, dry_run: bool = False) -> None:
+    """Remove mcp-ticketer from Claude Code/Desktop configuration.
+
+    Args:
+        global_config: Remove from Claude Desktop instead of project-level
+        dry_run: Show what would be removed without making changes
+
+    """
+    # Step 1: Find Claude MCP config location
+    config_type = "Claude Desktop" if global_config else "project-level"
+    console.print(f"[cyan]🔍 Removing {config_type} MCP configuration...[/cyan]")
+
+    mcp_config_path = find_claude_mcp_config(global_config)
+    console.print(f"[dim]Config location: {mcp_config_path}[/dim]")
+
+    # Step 2: Check if config file exists
+    if not mcp_config_path.exists():
+        console.print(f"[yellow]⚠ No configuration found at {mcp_config_path}[/yellow]")
+        console.print("[dim]mcp-ticketer is not configured for this platform[/dim]")
+        return
+
+    # Step 3: Load existing MCP configuration
+    mcp_config = load_claude_mcp_config(mcp_config_path)
+
+    # Step 4: Check if mcp-ticketer is configured
+    if "mcp-ticketer" not in mcp_config.get("mcpServers", {}):
+        console.print("[yellow]⚠ mcp-ticketer is not configured[/yellow]")
+        console.print(f"[dim]No mcp-ticketer entry found in {mcp_config_path}[/dim]")
+        return
+
+    # Step 5: Show what would be removed (dry run or actual removal)
+    if dry_run:
+        console.print("\n[cyan]DRY RUN - Would remove:[/cyan]")
+        console.print("  Server name: mcp-ticketer")
+        console.print(f"  From: {mcp_config_path}")
+        return
+
+    # Step 6: Remove mcp-ticketer from configuration
+    del mcp_config["mcpServers"]["mcp-ticketer"]
+
+    # Step 7: Save updated configuration
+    try:
+        save_claude_mcp_config(mcp_config_path, mcp_config)
+        console.print("\n[green]✓ Successfully removed mcp-ticketer[/green]")
+        console.print(f"[dim]Configuration updated: {mcp_config_path}[/dim]")
+
+        # Next steps
+        console.print("\n[bold cyan]Next Steps:[/bold cyan]")
+        if global_config:
+            console.print("1. Restart Claude Desktop")
+            console.print("2. mcp-ticketer will no longer be available in MCP menu")
+        else:
+            console.print("1. Restart Claude Code")
+            console.print("2. mcp-ticketer will no longer be available in this project")
+
+    except Exception as e:
+        console.print(f"\n[red]✗ Failed to update configuration:[/red] {e}")
+        raise
+
+
 def configure_claude_mcp(global_config: bool = False, force: bool = False) -> None:
     """Configure Claude Code to use mcp-ticketer.
 

mcp_ticketer/cli/platform_commands.py

@@ -29,9 +29,7 @@ def jira_list_projects():
     from rich.console import Console
 
     console = Console()
-    console.print(
-        "[yellow]JIRA platform commands are not yet implemented.[/yellow]"
-    )
+    console.print("[yellow]JIRA platform commands are not yet implemented.[/yellow]")
     console.print(
         "Use the generic ticket commands for JIRA operations:\n"
         "  mcp-ticketer ticket create 'My ticket'\n"
@@ -45,12 +43,8 @@ def jira_configure():
     from rich.console import Console
 
     console = Console()
-    console.print(
-        "[yellow]JIRA platform commands are not yet implemented.[/yellow]"
-    )
-    console.print(
-        "Use 'mcp-ticketer init --adapter jira' to configure JIRA adapter."
-    )
+    console.print("[yellow]JIRA platform commands are not yet implemented.[/yellow]")
+    console.print("Use 'mcp-ticketer init --adapter jira' to configure JIRA adapter.")
 
 
 # GitHub platform commands (placeholder)
@@ -66,9 +60,7 @@ def github_list_repos():
     from rich.console import Console
 
     console = Console()
-    console.print(
-        "[yellow]GitHub platform commands are not yet implemented.[/yellow]"
-    )
+    console.print("[yellow]GitHub platform commands are not yet implemented.[/yellow]")
     console.print(
         "Use the generic ticket commands for GitHub operations:\n"
         "  mcp-ticketer ticket create 'My issue'\n"
@@ -82,9 +74,7 @@ def github_configure():
     from rich.console import Console
 
     console = Console()
-    console.print(
-        "[yellow]GitHub platform commands are not yet implemented.[/yellow]"
-    )
+    console.print("[yellow]GitHub platform commands are not yet implemented.[/yellow]")
     console.print(
         "Use 'mcp-ticketer init --adapter github' to configure GitHub adapter."
     )

mcp_ticketer/cli/ticket_commands.py

@@ -52,7 +52,9 @@ def load_config(project_dir: Optional[Path] = None) -> dict:
                 return config
         except (OSError, json.JSONDecodeError) as e:
             logger.warning(f"Could not load project config: {e}, using defaults")
-            console.print(f"[yellow]Warning: Could not load project config: {e}[/yellow]")
+            console.print(
+                f"[yellow]Warning: Could not load project config: {e}[/yellow]"
+            )
 
     logger.info("No project-local config found, defaulting to aitrackdown adapter")
     return {"adapter": "aitrackdown", "config": {"base_path": ".aitrackdown"}}
@@ -70,7 +72,9 @@ def save_config(config: dict) -> None:
     logger.info(f"Saved configuration to: {project_config}")
 
 
-def get_adapter(override_adapter: Optional[str] = None, override_config: Optional[dict] = None):
+def get_adapter(
+    override_adapter: Optional[str] = None, override_config: Optional[dict] = None
+):
     """Get configured adapter instance."""
     config = load_config()
 
@@ -369,7 +373,9 @@ def create(
     console.print(f"  Title: {title}")
     console.print(f"  Priority: {priority}")
     console.print(f"  Adapter: {adapter_name}")
-    console.print("[dim]Use 'mcp-ticketer ticket check {queue_id}' to check progress[/dim]")
+    console.print(
+        "[dim]Use 'mcp-ticketer ticket check {queue_id}' to check progress[/dim]"
+    )
 
     # Start worker if needed with immediate feedback
     manager = WorkerManager()
@@ -606,7 +612,9 @@ def update(
     for key, value in updates.items():
         if key != "ticket_id":
             console.print(f"  {key}: {value}")
-    console.print("[dim]Use 'mcp-ticketer ticket check {queue_id}' to check progress[/dim]")
+    console.print(
+        "[dim]Use 'mcp-ticketer ticket check {queue_id}' to check progress[/dim]"
+    )
 
     # Start worker if needed
     manager = WorkerManager()
@@ -672,7 +680,9 @@ def transition(
 
     console.print(f"[green]✓[/green] Queued state transition: {queue_id}")
     console.print(f"  Ticket: {ticket_id} → {target_state}")
-    console.print("[dim]Use 'mcp-ticketer ticket check {queue_id}' to check progress[/dim]")
+    console.print(
+        "[dim]Use 'mcp-ticketer ticket check {queue_id}' to check progress[/dim]"
+    )
 
     # Start worker if needed
     manager = WorkerManager()

mcp_ticketer/mcp/server_sdk.py

@@ -0,0 +1,93 @@
+"""FastMCP-based MCP server implementation.
+
+This module implements the MCP server using the official FastMCP SDK,
+replacing the custom JSON-RPC implementation. It provides a cleaner,
+more maintainable approach with automatic schema generation and
+better error handling.
+
+The server manages a global adapter instance that is configured at
+startup and used by all tool implementations.
+"""
+
+import logging
+from typing import Any, Optional
+
+from mcp.server.fastmcp import FastMCP
+
+from ..core.adapter import BaseAdapter
+from ..core.registry import AdapterRegistry
+
+# Initialize FastMCP server
+mcp = FastMCP("mcp-ticketer")
+
+# Global adapter instance
+_adapter: Optional[BaseAdapter] = None
+
+# Configure logging
+logger = logging.getLogger(__name__)
+
+
+def configure_adapter(adapter_type: str, config: dict[str, Any]) -> None:
+    """Configure the global adapter instance.
+
+    This must be called before starting the server to initialize the
+    adapter that will handle all ticket operations.
+
+    Args:
+        adapter_type: Type of adapter to create (e.g., "linear", "jira", "github")
+        config: Configuration dictionary for the adapter
+
+    Raises:
+        ValueError: If adapter type is not registered
+        RuntimeError: If adapter configuration fails
+
+    """
+    global _adapter
+
+    try:
+        # Get adapter from registry
+        _adapter = AdapterRegistry.get_adapter(adapter_type, config)
+        logger.info(f"Configured {adapter_type} adapter for MCP server")
+    except Exception as e:
+        logger.error(f"Failed to configure adapter: {e}")
+        raise RuntimeError(f"Adapter configuration failed: {e}") from e
+
+
+def get_adapter() -> BaseAdapter:
+    """Get the configured adapter instance.
+
+    Returns:
+        The global adapter instance
+
+    Raises:
+        RuntimeError: If adapter has not been configured
+
+    """
+    if _adapter is None:
+        raise RuntimeError(
+            "Adapter not configured. Call configure_adapter() before starting server."
+        )
+    return _adapter
+
+
+# Import all tool modules to register them with FastMCP
+# These imports must come after mcp is initialized but before main()
+from . import tools  # noqa: E402, F401
+
+
+def main() -> None:
+    """Run the FastMCP server.
+
+    This function starts the server using stdio transport for
+    JSON-RPC communication with Claude Desktop/Code.
+
+    The adapter must be configured via configure_adapter() before
+    calling this function.
+
+    """
+    # Run the server with stdio transport
+    mcp.run(transport="stdio")
+
+
+if __name__ == "__main__":
+    main()

mcp_ticketer/mcp/tools/__init__.py

@@ -0,0 +1,38 @@
+"""MCP tool modules for ticket operations.
+
+This package contains all FastMCP tool implementations organized by
+functional area. Tools are automatically registered with the FastMCP
+server when imported.
+
+Modules:
+    ticket_tools: Basic CRUD operations for tickets
+    hierarchy_tools: Epic/Issue/Task hierarchy management
+    search_tools: Search and query operations
+    bulk_tools: Bulk create and update operations
+    comment_tools: Comment management
+    pr_tools: Pull request integration
+    attachment_tools: File attachment handling
+
+"""
+
+# Import all tool modules to register them with FastMCP
+# Order matters - import core functionality first
+from . import (
+    attachment_tools,  # noqa: F401
+    bulk_tools,  # noqa: F401
+    comment_tools,  # noqa: F401
+    hierarchy_tools,  # noqa: F401
+    pr_tools,  # noqa: F401
+    search_tools,  # noqa: F401
+    ticket_tools,  # noqa: F401
+)
+
+__all__ = [
+    "ticket_tools",
+    "hierarchy_tools",
+    "search_tools",
+    "bulk_tools",
+    "comment_tools",
+    "pr_tools",
+    "attachment_tools",
+]

mcp_ticketer/mcp/tools/attachment_tools.py

@@ -0,0 +1,180 @@
+"""Attachment management tools for tickets.
+
+This module implements tools for attaching files to tickets and retrieving
+attachment information. Note that file attachment functionality may not be
+available in all adapters.
+"""
+
+from typing import Any
+
+from ..server_sdk import get_adapter, mcp
+
+
+@mcp.tool()
+async def ticket_attach(
+    ticket_id: str,
+    file_path: str,
+    description: str = "",
+) -> dict[str, Any]:
+    """Attach a file to a ticket.
+
+    Uploads a file and associates it with the specified ticket. This
+    functionality may not be available in all adapters.
+
+    Args:
+        ticket_id: Unique identifier of the ticket
+        file_path: Path to the file to attach
+        description: Optional description of the attachment
+
+    Returns:
+        Attachment details including URL or ID, or error information
+
+    """
+    try:
+        adapter = get_adapter()
+
+        # Read ticket to validate it exists
+        ticket = await adapter.read(ticket_id)
+        if ticket is None:
+            return {
+                "status": "error",
+                "error": f"Ticket {ticket_id} not found",
+            }
+
+        # Check if adapter supports attachments
+        if not hasattr(adapter, "add_attachment"):
+            return {
+                "status": "error",
+                "error": f"File attachments not supported by {type(adapter).__name__} adapter",
+                "ticket_id": ticket_id,
+                "note": "Consider using ticket_comment to add a reference to the file location",
+            }
+
+        # Add attachment via adapter
+        attachment = await adapter.add_attachment(  # type: ignore
+            ticket_id=ticket_id, file_path=file_path, description=description
+        )
+
+        return {
+            "status": "completed",
+            "ticket_id": ticket_id,
+            "attachment": attachment,
+        }
+
+    except AttributeError:
+        # Fallback: Add file reference as comment
+        from ...core.models import Comment
+
+        comment_text = f"Attachment: {file_path}"
+        if description:
+            comment_text += f"\nDescription: {description}"
+
+        comment = Comment(
+            ticket_id=ticket_id,
+            content=comment_text,
+        )
+
+        created_comment = await adapter.add_comment(comment)
+
+        return {
+            "status": "completed",
+            "ticket_id": ticket_id,
+            "method": "comment_reference",
+            "file_path": file_path,
+            "comment": created_comment.model_dump(),
+            "note": "Adapter does not support direct file uploads. File reference added as comment.",
+        }
+
+    except FileNotFoundError:
+        return {
+            "status": "error",
+            "error": f"File not found: {file_path}",
+            "ticket_id": ticket_id,
+        }
+    except Exception as e:
+        return {
+            "status": "error",
+            "error": f"Failed to attach file: {str(e)}",
+            "ticket_id": ticket_id,
+        }
+
+
+@mcp.tool()
+async def ticket_attachments(
+    ticket_id: str,
+) -> dict[str, Any]:
+    """Get all attachments for a ticket.
+
+    Retrieves a list of all files attached to the specified ticket.
+    This functionality may not be available in all adapters.
+
+    Args:
+        ticket_id: Unique identifier of the ticket
+
+    Returns:
+        List of attachments with metadata, or error information
+
+    """
+    try:
+        adapter = get_adapter()
+
+        # Read ticket to validate it exists
+        ticket = await adapter.read(ticket_id)
+        if ticket is None:
+            return {
+                "status": "error",
+                "error": f"Ticket {ticket_id} not found",
+            }
+
+        # Check if adapter supports attachments
+        if not hasattr(adapter, "get_attachments"):
+            return {
+                "status": "error",
+                "error": f"Attachment retrieval not supported by {type(adapter).__name__} adapter",
+                "ticket_id": ticket_id,
+                "note": "Check ticket comments for file references",
+            }
+
+        # Get attachments via adapter
+        attachments = await adapter.get_attachments(ticket_id)  # type: ignore
+
+        return {
+            "status": "completed",
+            "ticket_id": ticket_id,
+            "attachments": attachments,
+            "count": len(attachments) if isinstance(attachments, list) else 0,
+        }
+
+    except AttributeError:
+        # Fallback: Check comments for attachment references
+        comments = await adapter.get_comments(ticket_id=ticket_id, limit=100)
+
+        # Look for comments that reference files
+        attachment_refs = []
+        for comment in comments:
+            content = comment.content or ""
+            if content.startswith("Attachment:") or "file://" in content:
+                attachment_refs.append(
+                    {
+                        "type": "comment_reference",
+                        "comment_id": comment.id,
+                        "content": content,
+                        "created_at": comment.created_at,
+                    }
+                )
+
+        return {
+            "status": "completed",
+            "ticket_id": ticket_id,
+            "method": "comment_references",
+            "attachments": attachment_refs,
+            "count": len(attachment_refs),
+            "note": "Adapter does not support direct attachments. Showing file references from comments.",
+        }
+
+    except Exception as e:
+        return {
+            "status": "error",
+            "error": f"Failed to get attachments: {str(e)}",
+            "ticket_id": ticket_id,
+        }