fix: remove duplicate mcp() function in cli.py (#31)

PR #30 (squash-merged) reintroduced an old mcp() implementation alongside
the new one (added in commit a9f3d55). The old mcp() used click_to_mcp
directly, while the new mcp() uses run_for_app from the .mcp_server module.

Python silently shadowed the first mcp() with the second, so tests passed
and the CLI worked, but having two definitions of the same function is a
code smell that could cause confusion or future bugs.

This commit removes the first (old) mcp() and keeps the newer
implementation that uses the project's mcp_server abstraction.

Verified: 108/108 tests pass

Author: Coding-Dev-Tools

src/deploydiff/cli.py

@@ -114,20 +114,6 @@ def rollback(terraform_file, cloudformation_file, pulumi_file) -> None:
         console.print(cmd)
 
 
-@main.command()
-def mcp():
-    """Run as an MCP (Model Context Protocol) server over stdio.
-
-    AI coding agents (Claude Code, Cursor, etc.) use this to interact
-    with deploydiff tools directly.
-    """
-    try:
-        from click_to_mcp import serve_stdio
-    except ImportError:
-        click.echo("Error: click-to-mcp is required for MCP support. Install with: pip install click-to-mcp", err=True)
-        raise SystemExit(1) from None
-    serve_stdio(main, name="deploydiff")
-
 
 def _load_plan(
     terraform_file: str | None,