mcp-ticketer 0.1.30__py3-none-any.whl → 1.2.11__py3-none-any.whl

mcp_ticketer/cli/mcp_configure.py

@@ -2,63 +2,43 @@
 
 import json
 import os
-import shutil
 import sys
 from pathlib import Path
-from typing import Optional
 
 from rich.console import Console
 
+from .python_detection import get_mcp_ticketer_python
+
 console = Console()
 
 
-def find_mcp_ticketer_binary() -> str:
-    """Find the mcp-ticketer binary path.
+def load_env_file(env_path: Path) -> dict[str, str]:
+    """Load environment variables from .env file.
 
-    Returns:
-        Path to mcp-ticketer binary
+    Args:
+        env_path: Path to .env file
 
-    Raises:
-        FileNotFoundError: If binary not found
+    Returns:
+        Dict of environment variable key-value pairs
 
     """
-    # Check if running from development environment
-    import mcp_ticketer
-
-    package_path = Path(mcp_ticketer.__file__).parent.parent.parent
-
-    # Check for virtual environment bin
-    possible_paths = [
-        # Development paths
-        package_path / "venv" / "bin" / "mcp-ticketer",
-        package_path / ".venv" / "bin" / "mcp-ticketer",
-        package_path / "test_venv" / "bin" / "mcp-ticketer",
-        # System installation
-        Path.home() / ".local" / "bin" / "mcp-ticketer",
-        # pipx installation
-        Path.home()
-        / ".local"
-        / "pipx"
-        / "venvs"
-        / "mcp-ticketer"
-        / "bin"
-        / "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():
-            return str(path.resolve())
-
-    raise FileNotFoundError(
-        "Could not find mcp-ticketer binary. Please ensure mcp-ticketer is installed.\n"
-        "Install with: pip install mcp-ticketer"
-    )
+    env_vars: dict[str, str] = {}
+    if not env_path.exists():
+        return env_vars
+
+    with open(env_path) as f:
+        for line in f:
+            line = line.strip()
+            # Skip comments and empty lines
+            if not line or line.startswith("#"):
+                continue
+
+            # Parse KEY=VALUE format
+            if "=" in line:
+                key, value = line.split("=", 1)
+                env_vars[key.strip()] = value.strip()
+
+    return env_vars
 
 
 def load_project_config() -> dict:
@@ -127,28 +107,74 @@ def find_claude_mcp_config(global_config: bool = False) -> Path:
                 Path.home() / ".config" / "Claude" / "claude_desktop_config.json"
             )
     else:
-        # Project-level configuration
-        config_path = Path.cwd() / ".mcp" / "config.json"
+        # Claude Code configuration - check both locations
+        # Priority 1: New global location ~/.config/claude/mcp.json
+        new_config_path = Path.home() / ".config" / "claude" / "mcp.json"
+        if new_config_path.exists():
+            return new_config_path
+
+        # Priority 2: Legacy project-specific location ~/.claude.json
+        config_path = Path.home() / ".claude.json"
 
     return config_path
 
 
-def load_claude_mcp_config(config_path: Path) -> dict:
+def load_claude_mcp_config(config_path: Path, is_claude_code: bool = False) -> dict:
     """Load existing Claude MCP configuration or return empty structure.
 
     Args:
         config_path: Path to MCP config file
+        is_claude_code: If True, return Claude Code structure with projects
 
     Returns:
         MCP configuration dict
 
     """
-    if config_path.exists():
-        with open(config_path) as f:
-            return json.load(f)
+    # Detect if this is the new global config location
+    is_global_mcp_config = str(config_path).endswith(".config/claude/mcp.json")
 
-    # Return empty structure
-    return {"mcpServers": {}}
+    if config_path.exists():
+        try:
+            with open(config_path) as f:
+                content = f.read().strip()
+                if not content:
+                    # Empty file, return default structure based on location
+                    if is_global_mcp_config:
+                        return {"mcpServers": {}}  # Flat structure
+                    return {"projects": {}} if is_claude_code else {"mcpServers": {}}
+
+                config = json.loads(content)
+
+                # Auto-detect structure format based on content
+                if "projects" in config:
+                    # This is the old nested project structure
+                    return config
+                elif "mcpServers" in config:
+                    # This is flat mcpServers structure
+                    return config
+                else:
+                    # Empty or unknown structure, return default
+                    if is_global_mcp_config:
+                        return {"mcpServers": {}}
+                    return {"projects": {}} if is_claude_code else {"mcpServers": {}}
+
+        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 default structure on parse error
+            if is_global_mcp_config:
+                return {"mcpServers": {}}
+            return {"projects": {}} if is_claude_code else {"mcpServers": {}}
+
+    # Return empty structure based on config type and location
+    if is_global_mcp_config:
+        return {"mcpServers": {}}  # New location always uses flat structure
+    if is_claude_code:
+        return {"projects": {}}
+    else:
+        return {"mcpServers": {}}
 
 
 def save_claude_mcp_config(config_path: Path, config: dict) -> None:
@@ -168,44 +194,102 @@ def save_claude_mcp_config(config_path: Path, config: dict) -> None:
 
 
 def create_mcp_server_config(
-    binary_path: str, project_config: dict, cwd: Optional[str] = None
+    python_path: str,
+    project_config: dict,
+    project_path: str | None = None,
+    is_global_config: bool = False,
 ) -> dict:
     """Create 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:
-        binary_path: Path to mcp-ticketer binary
+        python_path: Path to Python executable in mcp-ticketer venv
         project_config: Project configuration from .mcp-ticketer/config.json
-        cwd: Working directory for server (optional)
+        project_path: Project directory path (optional)
+        is_global_config: If True, create config for global location (no project path in args)
 
     Returns:
-        MCP server configuration dict
+        MCP server configuration dict matching Claude Code stdio pattern
 
     """
+    # 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 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"]
+
+    # Add project path if provided and not global config
+    if project_path and not is_global_config:
+        args.extend(["--path", project_path])
+
+    # REQUIRED: Add "type": "stdio" for Claude Code compatibility
     config = {
-        "command": binary_path,
-        "args": ["serve"],  # Use 'serve' command to start MCP server
+        "type": "stdio",
+        "command": cli_path,
+        "args": args,
     }
 
-    # Add working directory if provided
-    if cwd:
-        config["cwd"] = cwd
+    # NOTE: The CLI command loads configuration from .mcp-ticketer/config.json
+    # Environment variables below are optional fallbacks for backward compatibility
+    # The FastMCP SDK server will automatically load config from the project directory
 
-    # Add environment variables based on adapter
     adapter = project_config.get("default_adapter", "aitrackdown")
     adapters_config = project_config.get("adapters", {})
     adapter_config = adapters_config.get(adapter, {})
 
     env_vars = {}
 
-    # Add adapter-specific environment variables
+    # Add PYTHONPATH for project context (only for project-specific configs)
+    if project_path and not is_global_config:
+        env_vars["PYTHONPATH"] = project_path
+
+    # Add MCP_TICKETER_ADAPTER to identify which adapter to use (optional fallback)
+    env_vars["MCP_TICKETER_ADAPTER"] = adapter
+
+    # Load environment variables from .env.local if it exists
+    if project_path:
+        env_file_path = Path(project_path) / ".env.local"
+        env_file_vars = load_env_file(env_file_path)
+
+        # Add relevant adapter-specific vars from .env.local
+        adapter_env_keys = {
+            "linear": ["LINEAR_API_KEY", "LINEAR_TEAM_ID", "LINEAR_TEAM_KEY"],
+            "github": ["GITHUB_TOKEN", "GITHUB_OWNER", "GITHUB_REPO"],
+            "jira": [
+                "JIRA_ACCESS_USER",
+                "JIRA_ACCESS_TOKEN",
+                "JIRA_ORGANIZATION_ID",
+                "JIRA_URL",
+                "JIRA_EMAIL",
+                "JIRA_API_TOKEN",
+            ],
+            "aitrackdown": [],  # No specific env vars needed
+        }
+
+        # Include adapter-specific env vars from .env.local
+        for key in adapter_env_keys.get(adapter, []):
+            if key in env_file_vars:
+                env_vars[key] = env_file_vars[key]
+
+    # Fallback: Add adapter-specific environment variables from project config
     if adapter == "linear" and "api_key" in adapter_config:
-        env_vars["LINEAR_API_KEY"] = adapter_config["api_key"]
+        if "LINEAR_API_KEY" not in env_vars:
+            env_vars["LINEAR_API_KEY"] = adapter_config["api_key"]
     elif adapter == "github" and "token" in adapter_config:
-        env_vars["GITHUB_TOKEN"] = adapter_config["token"]
+        if "GITHUB_TOKEN" not in env_vars:
+            env_vars["GITHUB_TOKEN"] = adapter_config["token"]
     elif adapter == "jira":
-        if "api_token" in adapter_config:
+        if "api_token" in adapter_config and "JIRA_API_TOKEN" not in env_vars:
             env_vars["JIRA_API_TOKEN"] = adapter_config["api_token"]
-        if "email" in adapter_config:
+        if "email" in adapter_config and "JIRA_EMAIL" not in env_vars:
             env_vars["JIRA_EMAIL"] = adapter_config["email"]
 
     if env_vars:
@@ -214,6 +298,182 @@ def create_mcp_server_config(
     return config
 
 
+def detect_legacy_claude_config(
+    config_path: Path, is_claude_code: bool = True, project_path: str | None = None
+) -> tuple[bool, dict | None]:
+    """Detect if existing Claude config uses legacy Python module invocation.
+
+    Args:
+    ----
+        config_path: Path to Claude configuration file
+        is_claude_code: Whether this is Claude Code (project-level) or Claude Desktop (global)
+        project_path: Project path for Claude Code configs
+
+    Returns:
+    -------
+        Tuple of (is_legacy, server_config):
+        - is_legacy: True if config uses 'python -m mcp_ticketer.mcp.server'
+        - server_config: The legacy server config dict, or None if not legacy
+
+    """
+    if not config_path.exists():
+        return False, None
+
+    try:
+        mcp_config = load_claude_mcp_config(config_path, is_claude_code=is_claude_code)
+    except Exception:
+        return False, None
+
+    # For Claude Code, check project-specific config
+    if is_claude_code and project_path:
+        projects = mcp_config.get("projects", {})
+        project_config = projects.get(project_path, {})
+        mcp_servers = project_config.get("mcpServers", {})
+    else:
+        # For Claude Desktop, check global config
+        mcp_servers = mcp_config.get("mcpServers", {})
+
+    if "mcp-ticketer" in mcp_servers:
+        server_config = mcp_servers["mcp-ticketer"]
+        args = server_config.get("args", [])
+
+        # Check for legacy pattern: ["-m", "mcp_ticketer.mcp.server", ...]
+        if len(args) >= 2 and args[0] == "-m" and "mcp_ticketer.mcp.server" in args[1]:
+            return True, server_config
+
+    return False, None
+
+
+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 "Claude Code"
+    console.print(f"[cyan]🔍 Removing {config_type} MCP configuration...[/cyan]")
+
+    # Get absolute project path for Claude Code
+    absolute_project_path = str(Path.cwd().resolve()) if not global_config else None
+
+    # Check both locations for Claude Code
+    config_paths_to_check = []
+    if not global_config:
+        # Check both new and old locations
+        new_config = Path.home() / ".config" / "claude" / "mcp.json"
+        old_config = Path.home() / ".claude.json"
+        legacy_config = Path.cwd() / ".claude" / "mcp.local.json"
+
+        if new_config.exists():
+            config_paths_to_check.append(
+                (new_config, True)
+            )  # True = is_global_mcp_config
+        if old_config.exists():
+            config_paths_to_check.append((old_config, False))
+        if legacy_config.exists():
+            config_paths_to_check.append((legacy_config, False))
+    else:
+        mcp_config_path = find_claude_mcp_config(global_config)
+        if mcp_config_path.exists():
+            config_paths_to_check.append((mcp_config_path, False))
+
+    if not config_paths_to_check:
+        console.print("[yellow]⚠ No configuration files found[/yellow]")
+        console.print("[dim]mcp-ticketer is not configured for this platform[/dim]")
+        return
+
+    # Step 2-7: Process each config file
+    removed_count = 0
+    for config_path, is_global_mcp_config in config_paths_to_check:
+        console.print(f"[dim]Checking: {config_path}[/dim]")
+
+        # Load existing MCP configuration
+        is_claude_code = not global_config
+        mcp_config = load_claude_mcp_config(config_path, is_claude_code=is_claude_code)
+
+        # Check if mcp-ticketer is configured
+        is_configured = False
+        if is_global_mcp_config:
+            # Global mcp.json uses flat structure
+            is_configured = "mcp-ticketer" in mcp_config.get("mcpServers", {})
+        elif is_claude_code:
+            # Check Claude Code structure: .projects[path].mcpServers["mcp-ticketer"]
+            if absolute_project_path:
+                projects = mcp_config.get("projects", {})
+                project_config_entry = projects.get(absolute_project_path, {})
+                is_configured = "mcp-ticketer" in project_config_entry.get(
+                    "mcpServers", {}
+                )
+            else:
+                # Check flat structure for backward compatibility
+                is_configured = "mcp-ticketer" in mcp_config.get("mcpServers", {})
+        else:
+            # Check Claude Desktop structure: .mcpServers["mcp-ticketer"]
+            is_configured = "mcp-ticketer" in mcp_config.get("mcpServers", {})
+
+        if not is_configured:
+            continue
+
+        # Show what would be removed (dry run)
+        if dry_run:
+            console.print(f"\n[cyan]DRY RUN - Would remove from: {config_path}[/cyan]")
+            console.print("  Server name: mcp-ticketer")
+            if absolute_project_path and not is_global_mcp_config:
+                console.print(f"  Project: {absolute_project_path}")
+            continue
+
+        # Remove mcp-ticketer from configuration
+        if is_global_mcp_config:
+            # Global mcp.json uses flat structure
+            del mcp_config["mcpServers"]["mcp-ticketer"]
+        elif is_claude_code and absolute_project_path and "projects" in mcp_config:
+            # Remove from Claude Code nested structure
+            del mcp_config["projects"][absolute_project_path]["mcpServers"][
+                "mcp-ticketer"
+            ]
+
+            # Clean up empty structures
+            if not mcp_config["projects"][absolute_project_path]["mcpServers"]:
+                del mcp_config["projects"][absolute_project_path]["mcpServers"]
+            if not mcp_config["projects"][absolute_project_path]:
+                del mcp_config["projects"][absolute_project_path]
+        else:
+            # Remove from flat structure (legacy or Claude Desktop)
+            if "mcp-ticketer" in mcp_config.get("mcpServers", {}):
+                del mcp_config["mcpServers"]["mcp-ticketer"]
+
+        # Save updated configuration
+        try:
+            save_claude_mcp_config(config_path, mcp_config)
+            console.print(f"[green]✓ Removed from: {config_path}[/green]")
+            removed_count += 1
+        except Exception as e:
+            console.print(f"[red]✗ Failed to update {config_path}:[/red] {e}")
+
+    if dry_run:
+        return
+
+    if removed_count > 0:
+        console.print("\n[green]✓ Successfully removed mcp-ticketer[/green]")
+        console.print(f"[dim]Updated {removed_count} configuration file(s)[/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")
+    else:
+        console.print(
+            "\n[yellow]⚠ mcp-ticketer was not found in any configuration[/yellow]"
+        )
+
+
 def configure_claude_mcp(global_config: bool = False, force: bool = False) -> None:
     """Configure Claude Code to use mcp-ticketer.
 
@@ -222,18 +482,31 @@ def configure_claude_mcp(global_config: bool = False, force: bool = False) -> No
         force: Overwrite existing configuration
 
     Raises:
-        FileNotFoundError: If binary or project config not found
+        FileNotFoundError: If Python executable or project config not found
         ValueError: If configuration is invalid
 
     """
-    # Step 1: Find mcp-ticketer binary
-    console.print("[cyan]🔍 Finding mcp-ticketer binary...[/cyan]")
+    # Determine project path for venv detection
+    project_path = Path.cwd() if not global_config else None
+
+    # Step 1: Find Python executable (project-specific if available)
+    console.print("[cyan]🔍 Finding mcp-ticketer Python executable...[/cyan]")
     try:
-        binary_path = find_mcp_ticketer_binary()
-        console.print(f"[green]✓[/green] Found: {binary_path}")
-    except FileNotFoundError as e:
-        console.print(f"[red]✗[/red] {e}")
-        raise
+        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 project_path and 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]")
@@ -246,17 +519,67 @@ def configure_claude_mcp(global_config: bool = False, force: bool = False) -> No
         raise
 
     # Step 3: Find Claude MCP config location
-    config_type = "Claude Desktop" if global_config else "project-level"
+    config_type = "Claude Desktop" if global_config else "Claude Code"
     console.print(f"\n[cyan]🔧 Configuring {config_type} MCP...[/cyan]")
 
     mcp_config_path = find_claude_mcp_config(global_config)
-    console.print(f"[dim]Config location: {mcp_config_path}[/dim]")
+    console.print(f"[dim]Primary config: {mcp_config_path}[/dim]")
+
+    # Get absolute project path for Claude Code
+    absolute_project_path = str(Path.cwd().resolve()) if not global_config else None
 
     # Step 4: Load existing MCP configuration
-    mcp_config = load_claude_mcp_config(mcp_config_path)
+    is_claude_code = not global_config
+    mcp_config = load_claude_mcp_config(mcp_config_path, is_claude_code=is_claude_code)
+
+    # Detect if using new global config location
+    is_global_mcp_config = str(mcp_config_path).endswith(".config/claude/mcp.json")
+
+    # Step 4.5: Check for legacy configuration (DETECTION & MIGRATION)
+    is_legacy, legacy_config = detect_legacy_claude_config(
+        mcp_config_path,
+        is_claude_code=is_claude_code,
+        project_path=absolute_project_path,
+    )
+    if is_legacy:
+        console.print("\n[yellow]⚠ LEGACY CONFIGURATION DETECTED[/yellow]")
+        console.print(
+            "[yellow]Your current configuration uses the legacy line-delimited JSON server:[/yellow]"
+        )
+        console.print(f"[dim]  Command: {legacy_config.get('command')}[/dim]")
+        console.print(f"[dim]  Args: {legacy_config.get('args')}[/dim]")
+        console.print(
+            f"\n[red]This legacy server is incompatible with modern MCP clients ({config_type}).[/red]"
+        )
+        console.print(
+            "[red]The legacy server uses line-delimited JSON instead of Content-Length framing.[/red]"
+        )
+        console.print(
+            "\n[cyan]✨ Automatically migrating to modern FastMCP-based server...[/cyan]"
+        )
+        force = True  # Auto-enable force mode for migration
 
     # Step 5: Check if mcp-ticketer already configured
-    if "mcp-ticketer" in mcp_config.get("mcpServers", {}):
+    already_configured = False
+    if is_global_mcp_config:
+        # New global config uses flat structure
+        already_configured = "mcp-ticketer" in mcp_config.get("mcpServers", {})
+    elif is_claude_code:
+        # Check Claude Code structure: .projects[path].mcpServers["mcp-ticketer"]
+        if absolute_project_path and "projects" in mcp_config:
+            projects = mcp_config.get("projects", {})
+            project_config_entry = projects.get(absolute_project_path, {})
+            already_configured = "mcp-ticketer" in project_config_entry.get(
+                "mcpServers", {}
+            )
+        elif "mcpServers" in mcp_config:
+            # Check flat structure for backward compatibility
+            already_configured = "mcp-ticketer" in mcp_config.get("mcpServers", {})
+    else:
+        # Check Claude Desktop structure: .mcpServers["mcp-ticketer"]
+        already_configured = "mcp-ticketer" in mcp_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]")
@@ -265,16 +588,61 @@ def configure_claude_mcp(global_config: bool = False, force: bool = False) -> No
             console.print("[yellow]⚠ Overwriting existing configuration[/yellow]")
 
     # Step 6: Create mcp-ticketer server config
-    cwd = str(Path.cwd()) if not global_config else None
     server_config = create_mcp_server_config(
-        binary_path=binary_path, project_config=project_config, cwd=cwd
+        python_path=python_path,
+        project_config=project_config,
+        project_path=absolute_project_path,
+        is_global_config=is_global_mcp_config,
     )
 
-    # Step 7: Update MCP configuration
-    if "mcpServers" not in mcp_config:
-        mcp_config["mcpServers"] = {}
-
-    mcp_config["mcpServers"]["mcp-ticketer"] = server_config
+    # Step 7: Update MCP configuration based on platform
+    if is_global_mcp_config:
+        # New global location: ~/.config/claude/mcp.json uses flat structure
+        if "mcpServers" not in mcp_config:
+            mcp_config["mcpServers"] = {}
+        mcp_config["mcpServers"]["mcp-ticketer"] = server_config
+    elif is_claude_code:
+        # Claude Code: Write to ~/.claude.json with project-specific path
+        if absolute_project_path:
+            # Ensure projects structure exists
+            if "projects" not in mcp_config:
+                mcp_config["projects"] = {}
+
+            # Ensure project entry exists
+            if absolute_project_path not in mcp_config["projects"]:
+                mcp_config["projects"][absolute_project_path] = {}
+
+            # Ensure mcpServers for this project exists
+            if "mcpServers" not in mcp_config["projects"][absolute_project_path]:
+                mcp_config["projects"][absolute_project_path]["mcpServers"] = {}
+
+            # Add mcp-ticketer configuration
+            mcp_config["projects"][absolute_project_path]["mcpServers"][
+                "mcp-ticketer"
+            ] = server_config
+
+            # Also write to backward-compatible location for older Claude Code versions
+            legacy_config_path = Path.cwd() / ".claude" / "mcp.local.json"
+            console.print(f"[dim]Legacy config: {legacy_config_path}[/dim]")
+
+            try:
+                legacy_config = load_claude_mcp_config(
+                    legacy_config_path, is_claude_code=False
+                )
+                if "mcpServers" not in legacy_config:
+                    legacy_config["mcpServers"] = {}
+                legacy_config["mcpServers"]["mcp-ticketer"] = server_config
+                save_claude_mcp_config(legacy_config_path, legacy_config)
+                console.print("[dim]✓ Backward-compatible config also written[/dim]")
+            except Exception as e:
+                console.print(
+                    f"[dim]⚠ Could not write legacy config (non-fatal): {e}[/dim]"
+                )
+    else:
+        # Claude Desktop: Write to platform-specific config
+        if "mcpServers" not in mcp_config:
+            mcp_config["mcpServers"] = {}
+        mcp_config["mcpServers"]["mcp-ticketer"] = server_config
 
     # Step 8: Save configuration
     try:
@@ -286,14 +654,30 @@ def configure_claude_mcp(global_config: bool = False, force: bool = False) -> No
         console.print("\n[bold]Configuration Details:[/bold]")
         console.print("  Server name: mcp-ticketer")
         console.print(f"  Adapter: {adapter}")
-        console.print(f"  Binary: {binary_path}")
-        if cwd:
-            console.print(f"  Working directory: {cwd}")
+        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)")
+        if absolute_project_path:
+            console.print(f"  Project path: {absolute_project_path}")
         if "env" in server_config:
             console.print(
                 f"  Environment variables: {list(server_config['env'].keys())}"
             )
 
+        # Migration success message (if legacy config was detected)
+        if is_legacy:
+            console.print("\n[green]✅ Migration Complete![/green]")
+            console.print(
+                "[green]Your configuration has been upgraded from legacy line-delimited JSON[/green]"
+            )
+            console.print(
+                "[green]to modern Content-Length framing (FastMCP SDK).[/green]"
+            )
+            console.print(
+                f"\n[cyan]This fixes MCP connection issues with {config_type}.[/cyan]"
+            )
+
         # Next steps
         console.print("\n[bold cyan]Next Steps:[/bold cyan]")
         if global_config: