mcp-ticketer 0.12.0__py3-none-any.whl → 2.0.1__py3-none-any.whl

mcp_ticketer/cli/cursor_configure.py

@@ -0,0 +1,314 @@
+"""Cursor code editor configuration for mcp-ticketer integration.
+
+Cursor uses project-level MCP configuration at ~/.cursor/mcp.json
+with a flat mcpServers structure (similar to Claude Code's global config).
+"""
+
+import json
+from pathlib import Path
+from typing import Any
+
+from rich.console import Console
+
+from .mcp_configure import load_project_config
+from .python_detection import get_mcp_ticketer_python
+
+console = Console()
+
+
+def find_cursor_config() -> Path:
+    """Find or create Cursor MCP configuration file.
+
+    Cursor uses global MCP configuration with flat structure.
+
+    Returns:
+        Path to Cursor MCP config file at ~/.cursor/mcp.json
+
+    """
+    config_path = Path.home() / ".cursor" / "mcp.json"
+    return config_path
+
+
+def load_cursor_config(config_path: Path) -> dict[str, Any]:
+    """Load existing Cursor configuration or return empty structure.
+
+    Args:
+        config_path: Path to Cursor MCP config file
+
+    Returns:
+        Cursor MCP configuration dict
+
+    """
+    if config_path.exists():
+        try:
+            with open(config_path) as f:
+                content = f.read().strip()
+                if not content:
+                    return {"mcpServers": {}}
+                config: dict[str, Any] = json.load(f)
+                return config
+        except json.JSONDecodeError as e:
+            console.print(
+                f"[yellow]⚠ Warning: Invalid JSON in {config_path}, creating new config[/yellow]"
+            )
+            console.print(f"[dim]Error: {e}[/dim]")
+
+    # Return empty structure with mcpServers section
+    return {"mcpServers": {}}
+
+
+def save_cursor_config(config_path: Path, config: dict[str, Any]) -> None:
+    """Save Cursor MCP configuration to file.
+
+    Args:
+        config_path: Path to Cursor MCP config file
+        config: Configuration to save
+
+    """
+    # Ensure directory exists
+    config_path.parent.mkdir(parents=True, exist_ok=True)
+
+    # Write with 2-space indentation (JSON standard)
+    with open(config_path, "w") as f:
+        json.dump(config, f, indent=2)
+
+
+def create_cursor_server_config(
+    python_path: str,
+    project_config: dict[str, Any],
+    project_path: str | None = None,
+) -> dict[str, Any]:
+    """Create Cursor MCP server configuration for mcp-ticketer.
+
+    Uses the CLI command (mcp-ticketer mcp) which implements proper
+    Content-Length framing via FastMCP SDK, required for modern MCP clients.
+
+    Args:
+        python_path: Path to Python executable in mcp-ticketer venv
+        project_config: Project configuration from .mcp-ticketer/config.json
+        project_path: Project directory path (optional, for project-specific config)
+
+    Returns:
+        Cursor MCP server configuration dict
+
+    """
+    # IMPORTANT: Use CLI command, NOT Python module invocation
+    # The CLI uses FastMCP SDK which implements proper Content-Length framing
+    from pathlib import Path
+
+    # Get adapter configuration
+    adapter = project_config.get("default_adapter", "aitrackdown")
+    adapters_config = project_config.get("adapters", {})
+    adapter_config = adapters_config.get(adapter, {})
+
+    # Get mcp-ticketer CLI path from Python path
+    python_dir = Path(python_path).parent
+    cli_path = str(python_dir / "mcp-ticketer")
+
+    # Build CLI arguments
+    args = ["mcp"]
+    if project_path:
+        args.extend(["--path", project_path])
+
+    # Build environment variables
+    env_vars = {}
+
+    # Add PYTHONPATH for project context
+    if project_path:
+        env_vars["PYTHONPATH"] = project_path
+
+    # Add adapter type
+    env_vars["MCP_TICKETER_ADAPTER"] = adapter
+
+    # Add adapter-specific environment variables
+    if adapter == "linear" and "api_key" in adapter_config:
+        env_vars["LINEAR_API_KEY"] = adapter_config["api_key"]
+        if "team_id" in adapter_config:
+            env_vars["LINEAR_TEAM_ID"] = adapter_config["team_id"]
+    elif adapter == "github" and "token" in adapter_config:
+        env_vars["GITHUB_TOKEN"] = adapter_config["token"]
+        if "owner" in adapter_config:
+            env_vars["GITHUB_OWNER"] = adapter_config["owner"]
+        if "repo" in adapter_config:
+            env_vars["GITHUB_REPO"] = adapter_config["repo"]
+    elif adapter == "jira":
+        if "api_token" in adapter_config:
+            env_vars["JIRA_API_TOKEN"] = adapter_config["api_token"]
+        if "email" in adapter_config:
+            env_vars["JIRA_EMAIL"] = adapter_config["email"]
+
+    # Create server configuration with Cursor-specific fields
+    config = {
+        "type": "stdio",  # Cursor requires explicit type
+        "command": cli_path,
+        "args": args,
+        "env": env_vars,
+    }
+
+    # Add working directory for project-specific configs
+    if project_path:
+        config["cwd"] = project_path
+
+    return config
+
+
+def remove_cursor_mcp(dry_run: bool = False) -> None:
+    """Remove mcp-ticketer from Cursor configuration.
+
+    Args:
+        dry_run: Show what would be removed without making changes
+
+    """
+    # Step 1: Find Cursor config location
+    console.print("[cyan]🔍 Removing Cursor MCP configuration...[/cyan]")
+
+    cursor_config_path = find_cursor_config()
+    console.print(f"[dim]Config location: {cursor_config_path}[/dim]")
+
+    # Step 2: Check if config file exists
+    if not cursor_config_path.exists():
+        console.print(
+            f"[yellow]⚠ No configuration found at {cursor_config_path}[/yellow]"
+        )
+        console.print("[dim]mcp-ticketer is not configured for Cursor[/dim]")
+        return
+
+    # Step 3: Load existing Cursor configuration
+    cursor_config = load_cursor_config(cursor_config_path)
+
+    # Step 4: Check if mcp-ticketer is configured
+    if "mcp-ticketer" not in cursor_config.get("mcpServers", {}):
+        console.print("[yellow]⚠ mcp-ticketer is not configured[/yellow]")
+        console.print(f"[dim]No mcp-ticketer entry found in {cursor_config_path}[/dim]")
+        return
+
+    # Show what would be removed (dry run)
+    if dry_run:
+        console.print(
+            f"\n[cyan]DRY RUN - Would remove from: {cursor_config_path}[/cyan]"
+        )
+        console.print("  Server name: mcp-ticketer")
+        return
+
+    # Step 5: Remove mcp-ticketer from configuration
+    del cursor_config["mcpServers"]["mcp-ticketer"]
+
+    # Step 6: Save updated configuration
+    try:
+        save_cursor_config(cursor_config_path, cursor_config)
+        console.print("\n[green]✓ Successfully removed mcp-ticketer[/green]")
+        console.print(f"[dim]Updated {cursor_config_path}[/dim]")
+
+        # Next steps
+        console.print("\n[bold cyan]Next Steps:[/bold cyan]")
+        console.print("1. Restart Cursor editor")
+        console.print("2. mcp-ticketer will no longer be available in MCP menu")
+    except Exception as e:
+        console.print(f"\n[red]✗ Failed to save configuration:[/red] {e}")
+        raise
+
+
+def configure_cursor_mcp(force: bool = False) -> None:
+    """Configure Cursor to use mcp-ticketer.
+
+    Args:
+        force: Overwrite existing configuration
+
+    Raises:
+        FileNotFoundError: If Python executable or project config not found
+        ValueError: If configuration is invalid
+
+    """
+    # Determine project path
+    project_path = Path.cwd()
+
+    # Step 1: Find Python executable
+    console.print("[cyan]🔍 Finding mcp-ticketer Python executable...[/cyan]")
+    try:
+        python_path = get_mcp_ticketer_python(project_path=project_path)
+        console.print(f"[green]✓[/green] Found: {python_path}")
+
+        # Show if using project venv or fallback
+        if str(project_path / ".venv") in python_path:
+            console.print("[dim]Using project-specific venv[/dim]")
+        else:
+            console.print("[dim]Using pipx/system Python[/dim]")
+    except Exception as e:
+        console.print(f"[red]✗[/red] Could not find Python executable: {e}")
+        raise FileNotFoundError(
+            "Could not find mcp-ticketer Python executable. "
+            "Please ensure mcp-ticketer is installed.\n"
+            "Install with: pip install mcp-ticketer or pipx install mcp-ticketer"
+        ) from e
+
+    # Step 2: Load project configuration
+    console.print("\n[cyan]📖 Reading project configuration...[/cyan]")
+    try:
+        mcp_project_config = load_project_config()
+        adapter = mcp_project_config.get("default_adapter", "aitrackdown")
+        console.print(f"[green]✓[/green] Adapter: {adapter}")
+    except (FileNotFoundError, ValueError) as e:
+        console.print(f"[red]✗[/red] {e}")
+        raise
+
+    # Step 3: Find Cursor MCP config location
+    console.print("\n[cyan]🔧 Configuring Cursor MCP...[/cyan]")
+
+    cursor_config_path = find_cursor_config()
+    console.print(f"[dim]Config path: {cursor_config_path}[/dim]")
+
+    # Step 4: Load existing MCP configuration
+    cursor_config = load_cursor_config(cursor_config_path)
+
+    # Step 5: Check if mcp-ticketer already configured
+    already_configured = "mcp-ticketer" in cursor_config.get("mcpServers", {})
+
+    if already_configured:
+        if not force:
+            console.print("[yellow]⚠ mcp-ticketer is already configured[/yellow]")
+            console.print("[dim]Use --force to overwrite existing configuration[/dim]")
+            return
+        else:
+            console.print("[yellow]⚠ Overwriting existing configuration[/yellow]")
+
+    # Step 6: Create mcp-ticketer server config
+    server_config = create_cursor_server_config(
+        python_path=python_path,
+        project_config=mcp_project_config,
+        project_path=str(project_path.resolve()),
+    )
+
+    # Step 7: Update MCP configuration
+    if "mcpServers" not in cursor_config:
+        cursor_config["mcpServers"] = {}
+    cursor_config["mcpServers"]["mcp-ticketer"] = server_config
+
+    # Step 8: Save configuration
+    try:
+        save_cursor_config(cursor_config_path, cursor_config)
+        console.print("\n[green]✓ Successfully configured mcp-ticketer[/green]")
+        console.print(f"[dim]Configuration saved to: {cursor_config_path}[/dim]")
+
+        # Print configuration details
+        console.print("\n[bold]Configuration Details:[/bold]")
+        console.print("  Server name: mcp-ticketer")
+        console.print(f"  Adapter: {adapter}")
+        console.print(f"  Python: {python_path}")
+        console.print(f"  Command: {server_config.get('command')}")
+        console.print(f"  Args: {server_config.get('args')}")
+        console.print("  Protocol: Content-Length framing (FastMCP SDK)")
+        console.print(f"  Project path: {project_path}")
+        if "env" in server_config:
+            console.print(
+                f"  Environment variables: {list(server_config['env'].keys())}"
+            )
+
+        # Next steps
+        console.print("\n[bold cyan]Next Steps:[/bold cyan]")
+        console.print("1. Restart Cursor editor")
+        console.print("2. Open this project in Cursor")
+        console.print("3. mcp-ticketer tools will be available in the MCP menu")
+
+    except Exception as e:
+        console.print(f"\n[red]✗ Failed to save configuration:[/red] {e}")
+        raise

mcp_ticketer/cli/diagnostics.py

@@ -154,7 +154,7 @@ class SystemDiagnostics:
         """Diagnose configuration issues."""
         console.print("\n📋 [yellow]Configuration Analysis[/yellow]")
 
-        config_status = {
+        config_status: dict[str, Any] = {
             "status": "healthy",
             "adapters_configured": 0,
             "default_adapter": None,
@@ -260,7 +260,7 @@ class SystemDiagnostics:
         """Diagnose adapter functionality."""
         console.print("\n🔌 [yellow]Adapter Diagnosis[/yellow]")
 
-        adapter_status = {
+        adapter_status: dict[str, Any] = {
             "total_adapters": 0,
             "healthy_adapters": 0,
             "failed_adapters": 0,
@@ -278,7 +278,7 @@ class SystemDiagnostics:
             for name, adapter_config in adapters_config.items():
                 adapter_type = adapter_config.get("type", name)
 
-                details = {
+                details: dict[str, Any] = {
                     "type": adapter_type,
                     "status": "unknown",
                     "last_test": None,
@@ -330,7 +330,7 @@ class SystemDiagnostics:
         """Diagnose queue system health with active testing."""
         console.print("\n⚡ [yellow]Queue System Diagnosis[/yellow]")
 
-        queue_status = {
+        queue_status: dict[str, Any] = {
             "worker_running": False,
             "worker_pid": None,
             "queue_stats": {},
@@ -455,7 +455,7 @@ class SystemDiagnostics:
 
     async def _test_worker_startup(self) -> dict[str, Any]:
         """Test starting a queue worker."""
-        test_result = {
+        test_result: dict[str, Any] = {
             "attempted": True,
             "success": False,
             "error": None,
@@ -497,7 +497,7 @@ class SystemDiagnostics:
 
     async def _test_queue_operations(self) -> dict[str, Any]:
         """Test basic queue operations."""
-        test_result = {
+        test_result: dict[str, Any] = {
             "attempted": True,
             "success": False,
             "error": None,
@@ -509,7 +509,7 @@ class SystemDiagnostics:
             from ..core.models import Priority, Task
             from ..queue.queue import Queue
 
-            test_task = Task(
+            test_task = Task(  # type: ignore[call-arg]
                 title="[DIAGNOSTIC TEST] Queue functionality test",
                 description="This is a diagnostic test - safe to ignore",
                 priority=Priority.LOW,
@@ -532,7 +532,7 @@ class SystemDiagnostics:
 
     async def _test_basic_queue_functionality(self) -> dict[str, Any]:
         """Test basic queue functionality in fallback mode."""
-        test_result = {
+        test_result: dict[str, Any] = {
             "attempted": True,
             "success": False,
             "error": None,
@@ -544,7 +544,7 @@ class SystemDiagnostics:
             from ..adapters.aitrackdown import AITrackdownAdapter
             from ..core.models import Priority, Task
 
-            test_task = Task(
+            test_task = Task(  # type: ignore[call-arg]
                 title="[DIAGNOSTIC TEST] Direct adapter test",
                 description="Testing direct adapter functionality",
                 priority=Priority.LOW,
@@ -564,7 +564,8 @@ class SystemDiagnostics:
             test_result["details"] = f"Direct adapter test passed: {result.id}"
 
             # Clean up test
-            await adapter.delete(result.id)
+            if result.id:
+                await adapter.delete(result.id)
 
         except Exception as e:
             test_result["error"] = str(e)
@@ -575,7 +576,7 @@ class SystemDiagnostics:
         """Analyze recent log entries for issues."""
         console.print("\n📝 [yellow]Recent Log Analysis[/yellow]")
 
-        log_analysis = {
+        log_analysis: dict[str, Any] = {
             "log_files_found": [],
             "recent_errors": [],
             "recent_warnings": [],
@@ -644,7 +645,7 @@ class SystemDiagnostics:
         """Analyze system performance metrics."""
         console.print("\n⚡ [yellow]Performance Analysis[/yellow]")
 
-        performance = {
+        performance: dict[str, Any] = {
             "response_times": {},
             "throughput": {},
             "resource_usage": {},

mcp_ticketer/cli/discover.py

@@ -28,10 +28,12 @@ def _mask_sensitive(value: str, key: str) -> str:
     """Mask sensitive values for display.
 
     Args:
+    ----
         value: Value to potentially mask
         key: Key name to determine if masking needed
 
     Returns:
+    -------
         Masked or original value
 
     """
@@ -61,6 +63,7 @@ def _display_discovered_adapter(
     """Display information about a discovered adapter.
 
     Args:
+    ----
         adapter: Discovered adapter to display
         discovery: EnvDiscovery instance for validation
 
@@ -504,6 +507,7 @@ def onepassword_template(
     which can be used with: op run --env-file=.env.1password -- <command>
 
     Examples:
+    --------
         # Create Linear template
         mcp-ticketer discover 1password-template linear
 
@@ -568,6 +572,7 @@ def onepassword_test(
     displays the resolved values (with sensitive data masked).
 
     Example:
+    -------
         mcp-ticketer discover 1password-test --file=.env.1password.linear
 
     """

mcp_ticketer/cli/gemini_configure.py

@@ -78,6 +78,9 @@ def create_gemini_server_config(
 ) -> dict:
     """Create Gemini MCP server configuration for mcp-ticketer.
 
+    Uses the CLI command (mcp-ticketer mcp) which implements proper
+    Content-Length framing via FastMCP SDK, required for modern MCP clients.
+
     Args:
         python_path: Path to Python executable in mcp-ticketer venv
         project_config: Project configuration from .mcp-ticketer/config.json
@@ -87,7 +90,9 @@ def create_gemini_server_config(
         Gemini MCP server configuration dict
 
     """
-    # Use Python module invocation pattern (works regardless of where package is installed)
+    # IMPORTANT: Use CLI command, NOT Python module invocation
+    # The CLI uses FastMCP SDK which implements proper Content-Length framing
+    # Legacy python -m mcp_ticketer.mcp.server uses line-delimited JSON (incompatible)
 
     # Get adapter configuration
     adapter = project_config.get("default_adapter", "aitrackdown")
@@ -138,14 +143,21 @@ def create_gemini_server_config(
         if "project_key" in adapter_config:
             env_vars["JIRA_PROJECT_KEY"] = adapter_config["project_key"]
 
-    # Use Python module invocation pattern
-    args = ["-m", "mcp_ticketer.mcp.server"]
+    # Get mcp-ticketer CLI path from Python path
+    # If python_path is /path/to/venv/bin/python, CLI is /path/to/venv/bin/mcp-ticketer
+    python_dir = Path(python_path).parent
+    cli_path = str(python_dir / "mcp-ticketer")
+
+    # Build CLI arguments
+    args = ["mcp"]
     if project_path:
-        args.append(project_path)
+        args.extend(["--path", project_path])
 
     # Create server configuration with Gemini-specific options
+    # NOTE: Environment variables below are optional fallbacks
+    # The CLI loads config from .mcp-ticketer/config.json
     config = {
-        "command": python_path,
+        "command": cli_path,
         "args": args,
         "env": env_vars,
         "timeout": 15000,  # 15 seconds timeout