"""Tests for fastmcp generate-cli command."""

import sys
from pathlib import Path
from typing import Any
from unittest.mock import patch

import mcp_types
import pytest

from fastmcp import FastMCP
from fastmcp.cli import generate as generate_module
from fastmcp.cli.client import Client
from fastmcp.cli.generate import (
    _derive_server_name,
    _param_to_cli_flag,
    _schema_to_python_type,
    _schema_type_label,
    _to_python_identifier,
    _tool_function_source,
    generate_cli_command,
    generate_cli_script,
    generate_skill_content,
    serialize_transport,
)
from fastmcp.client.transports.stdio import StdioTransport

# ---------------------------------------------------------------------------
# _schema_to_python_type
# ---------------------------------------------------------------------------


class TestSchemaToPythonType:
    def test_simple_string(self):
        py_type, needs_json = _schema_to_python_type({"type": "string"})
        assert py_type == "str"
        assert needs_json is False

    def test_simple_integer(self):
        py_type, needs_json = _schema_to_python_type({"type": "integer"})
        assert py_type == "int"
        assert needs_json is False

    def test_simple_number(self):
        py_type, needs_json = _schema_to_python_type({"type": "number"})
        assert py_type == "float"
        assert needs_json is False

    def test_simple_boolean(self):
        py_type, needs_json = _schema_to_python_type({"type": "boolean"})
        assert py_type == "bool"
        assert needs_json is False

    def test_array_of_strings(self):
        py_type, needs_json = _schema_to_python_type(
            {"type": "array", "items": {"type": "string"}}
        )
        assert py_type == "list[str]"
        assert needs_json is False

    def test_array_of_integers(self):
        py_type, needs_json = _schema_to_python_type(
            {"type": "array", "items": {"type": "integer"}}
        )
        assert py_type == "list[int]"
        assert needs_json is False

    def test_complex_object(self):
        py_type, needs_json = _schema_to_python_type({"type": "object"})
        assert py_type == "str"
        assert needs_json is True

    def test_complex_nested_array(self):
        py_type, needs_json = _schema_to_python_type(
            {"type": "array", "items": {"type": "object"}}
        )
        assert py_type == "str"
        assert needs_json is True

    def test_union_of_simple_types(self):
        py_type, needs_json = _schema_to_python_type({"type": ["string", "null"]})
        assert py_type == "str | None"
        assert needs_json is False


# ---------------------------------------------------------------------------
# _to_python_identifier
# ---------------------------------------------------------------------------


class TestToPythonIdentifier:
    def test_plain_name(self):
        assert _to_python_identifier("hello") == "hello"

    def test_hyphens(self):
        assert _to_python_identifier("get-forecast") == "get_forecast"

    def test_dots_and_slashes(self):
        assert _to_python_identifier("a.b/c") == "a_b_c"

    def test_leading_digit(self):
        assert _to_python_identifier("3d_render") == "_3d_render"

    def test_spaces(self):
        assert _to_python_identifier("my tool") == "my_tool"

    def test_empty_string(self):
        assert _to_python_identifier("") == "_unnamed"


# ---------------------------------------------------------------------------
# serialize_transport
# ---------------------------------------------------------------------------


class TestSerializeTransport:
    def test_url_string(self):
        code, imports = serialize_transport("http://localhost:8000/mcp")
        assert code == "'http://localhost:8000/mcp'"
        assert imports == set()

    def test_stdio_transport_basic(self):
        transport = StdioTransport(command="fastmcp", args=["run", "server.py"])
        code, imports = serialize_transport(transport)
        assert "StdioTransport" in code
        assert "command='fastmcp'" in code
        assert "args=['run', 'server.py']" in code
        assert "from fastmcp.client.transports import StdioTransport" in imports

    def test_stdio_transport_with_env(self):
        transport = StdioTransport(
            command="python", args=["-m", "myserver"], env={"KEY": "val"}
        )
        code, imports = serialize_transport(transport)
        assert "env={'KEY': 'val'}" in code

    def test_dict_passthrough(self):
        d: dict[str, Any] = {"mcpServers": {"test": {"url": "http://localhost"}}}
        code, imports = serialize_transport(d)
        assert "mcpServers" in code
        assert imports == set()


# ---------------------------------------------------------------------------
# _tool_function_source
# ---------------------------------------------------------------------------


class TestToolFunctionSource:
    def test_required_param(self):
        tool = mcp_types.Tool(
            name="greet",
            input_schema={
                "properties": {"name": {"type": "string", "description": "Who"}},
                "required": ["name"],
            },
        )
        source = _tool_function_source(tool)
        assert "async def greet(" in source
        assert "name: Annotated[str" in source
        assert "= None" not in source
        assert "_call_tool('greet', {'name': name})" in source

    def test_optional_param(self):
        tool = mcp_types.Tool(
            name="search",
            input_schema={
                "properties": {
                    "query": {"type": "string", "description": "Search query"},
                    "limit": {"type": "integer", "description": "Max results"},
                },
                "required": ["query"],
            },
        )
        source = _tool_function_source(tool)
        assert "query: Annotated[str" in source
        assert "limit: Annotated[int | None" in source
        assert "= None" in source

    def test_param_with_default(self):
        tool = mcp_types.Tool(
            name="fetch",
            input_schema={
                "properties": {
                    "url": {"type": "string", "description": "URL"},
                    "timeout": {
                        "type": "integer",
                        "description": "Timeout",
                        "default": 30,
                    },
                },
                "required": ["url"],
            },
        )
        source = _tool_function_source(tool)
        assert "timeout: Annotated[int" in source
        assert "= 30" in source

    def test_no_params(self):
        tool = mcp_types.Tool(
            name="ping",
            input_schema={"properties": {}},
        )
        source = _tool_function_source(tool)
        assert "async def ping(" in source
        assert "_call_tool('ping', {})" in source

    def test_preserves_underscores(self):
        tool = mcp_types.Tool(
            name="get_forecast",
            input_schema={
                "properties": {"city": {"type": "string"}},
                "required": ["city"],
            },
        )
        source = _tool_function_source(tool)
        assert "async def get_forecast(" in source

    def test_sanitizes_tool_name(self):
        tool = mcp_types.Tool(
            name="my.tool/v2",
            input_schema={"properties": {}},
        )
        source = _tool_function_source(tool)
        assert "async def my_tool_v2(" in source
        assert "name='my.tool/v2'" in source

    def test_sanitizes_param_name(self):
        tool = mcp_types.Tool(
            name="fetch",
            input_schema={
                "properties": {"content-type": {"type": "string", "description": "CT"}},
                "required": ["content-type"],
            },
        )
        source = _tool_function_source(tool)
        assert "content_type: Annotated[str" in source
        assert "'content-type': content_type" in source

    def test_description_in_docstring(self):
        tool = mcp_types.Tool(
            name="greet",
            description="Say hello to someone.",
            input_schema={
                "properties": {"name": {"type": "string"}},
                "required": ["name"],
            },
        )
        source = _tool_function_source(tool)
        assert "'''Say hello to someone.'''" in source

    def test_description_with_quotes(self):
        tool = mcp_types.Tool(
            name="fetch",
            description="Fetch data from 'source' API.",
            input_schema={
                "properties": {"url": {"type": "string"}},
                "required": ["url"],
            },
        )
        source = _tool_function_source(tool)
        # Should escape single quotes in the description
        assert r"Fetch data from \'source\' API." in source
        # Generated code should compile
        compile(source, "<test>", "exec")

    def test_array_of_strings_parameter(self):
        tool = mcp_types.Tool(
            name="tag_items",
            description="Tag multiple items.",
            input_schema={
                "properties": {
                    "item_id": {"type": "string"},
                    "tags": {"type": "array", "items": {"type": "string"}},
                },
                "required": ["item_id"],
            },
        )
        source = _tool_function_source(tool)
        # Should use list[str] type with help metadata
        assert "tags: Annotated[list[str]" in source
        assert "= []" in source
        # Should not have JSON parsing for simple arrays
        assert "json.loads" not in source
        compile(source, "<test>", "exec")

    def test_complex_object_parameter(self):
        tool = mcp_types.Tool(
            name="create_user",
            description="Create a user.",
            input_schema={
                "properties": {
                    "name": {"type": "string"},
                    "metadata": {
                        "type": "object",
                        "properties": {
                            "role": {"type": "string"},
                            "dept": {"type": "string"},
                        },
                    },
                },
                "required": ["name"],
            },
        )
        source = _tool_function_source(tool)
        # Should use str type for complex object
        assert "metadata: Annotated[str | None" in source
        # Should include JSON schema in help (with escaped quotes)
        assert "JSON Schema:" in source
        assert '\\"type\\": \\"object\\"' in source
        # Should have JSON parsing with isinstance check
        assert (
            "metadata_parsed = json.loads(metadata) if isinstance(metadata, str) else metadata"
            in source
        )
        # Should use parsed version in call
        assert "'metadata': metadata_parsed" in source
        compile(source, "<test>", "exec")

    def test_nested_array_parameter(self):
        tool = mcp_types.Tool(
            name="batch_process",
            description="Process batches.",
            input_schema={
                "properties": {
                    "batches": {
                        "type": "array",
                        "items": {
                            "type": "object",
                            "properties": {"id": {"type": "string"}},
                        },
                    },
                },
                "required": ["batches"],
            },
        )
        source = _tool_function_source(tool)
        # Nested arrays need JSON parsing
        assert "batches: Annotated[str" in source
        assert "JSON Schema:" in source
        assert (
            "batches_parsed = json.loads(batches) if isinstance(batches, str) else batches"
            in source
        )
        compile(source, "<test>", "exec")

    def test_complex_type_with_default(self):
        """Test that complex types with defaults are JSON-serialized."""
        tool = mcp_types.Tool(
            name="configure",
            input_schema={
                "properties": {
                    "options": {
                        "type": "object",
                        "default": {"timeout": 30, "retry": True},
                    },
                },
            },
        )
        source = _tool_function_source(tool)
        # Default should be JSON string, not Python dict
        # pydantic_core.to_json produces compact JSON
        assert '= \'{"timeout":30,"retry":true}\'' in source
        # Should parse safely even with default
        assert "isinstance(options, str)" in source
        compile(source, "<test>", "exec")

    def test_name_collision_detection(self):
        """Test that parameter name collisions are detected."""
        tool = mcp_types.Tool(
            name="test",
            input_schema={
                "properties": {
                    "content-type": {"type": "string"},
                    "content_type": {"type": "string"},
                },
            },
        )
        # Should raise ValueError for collision
        with pytest.raises(ValueError, match="both sanitize to 'content_type'"):
            _tool_function_source(tool)


# ---------------------------------------------------------------------------
# _derive_server_name
# ---------------------------------------------------------------------------


class TestDeriveServerName:
    def test_bare_name(self):
        assert _derive_server_name("weather") == "weather"

    def test_qualified_name(self):
        assert _derive_server_name("cursor:weather") == "weather"

    def test_python_file(self):
        assert _derive_server_name("server.py") == "server"

    def test_url(self):
        assert _derive_server_name("http://localhost:8000/mcp") == "localhost"

    def test_trailing_colon(self):
        assert _derive_server_name("source:") == "source"


# ---------------------------------------------------------------------------
# generate_cli_script — produces compilable Python
# ---------------------------------------------------------------------------


class TestGenerateCliScript:
    def _make_tools(self) -> list[mcp_types.Tool]:
        return [
            mcp_types.Tool(
                name="greet",
                description="Say hello",
                input_schema={
                    "properties": {
                        "name": {"type": "string", "description": "Who to greet"},
                    },
                    "required": ["name"],
                },
            ),
            mcp_types.Tool(
                name="add_numbers",
                description="Add two numbers",
                input_schema={
                    "properties": {
                        "a": {"type": "integer", "description": "First number"},
                        "b": {"type": "integer", "description": "Second number"},
                    },
                    "required": ["a", "b"],
                },
            ),
        ]

    def test_compiles(self):
        script = generate_cli_script(
            server_name="test",
            server_spec="test",
            transport_code='"http://localhost:8000/mcp"',
            extra_imports=set(),
            tools=self._make_tools(),
        )
        compile(script, "<generated>", "exec")

    def test_contains_tool_functions(self):
        script = generate_cli_script(
            server_name="test",
            server_spec="test",
            transport_code='"http://localhost:8000/mcp"',
            extra_imports=set(),
            tools=self._make_tools(),
        )
        assert "async def greet(" in script
        assert "async def add_numbers(" in script

    def test_contains_generic_commands(self):
        script = generate_cli_script(
            server_name="test",
            server_spec="test",
            transport_code='"http://localhost:8000/mcp"',
            extra_imports=set(),
            tools=[],
        )
        assert "async def list_tools(" in script
        assert "async def list_resources(" in script
        assert "async def list_prompts(" in script
        assert "async def read_resource(" in script
        assert "async def get_prompt(" in script

    def test_embeds_transport(self):
        script = generate_cli_script(
            server_name="test",
            server_spec="test",
            transport_code="StdioTransport(command='fastmcp', args=['run', 'x.py'])",
            extra_imports={"from fastmcp.client.transports import StdioTransport"},
            tools=[],
        )
        assert "StdioTransport(command='fastmcp'" in script
        assert "from fastmcp.client.transports import StdioTransport" in script

    def test_no_tools_still_valid(self):
        script = generate_cli_script(
            server_name="empty",
            server_spec="empty",
            transport_code='"http://localhost"',
            extra_imports=set(),
            tools=[],
        )
        compile(script, "<generated>", "exec")
        assert "call_tool_app" in script

    def test_server_name_with_quotes(self):
        """Test that server names with quotes are properly escaped."""
        script = generate_cli_script(
            server_name='Test "Server" Name',
            server_spec="test",
            transport_code='"http://localhost"',
            extra_imports=set(),
            tools=[],
        )
        # Should compile without syntax errors
        compile(script, "<generated>", "exec")
        # App name should have escaped quotes
        assert r'app = cyclopts.App(name="test-\"server\"-name"' in script

    def test_compiles_with_unusual_names(self):
        tools = [
            mcp_types.Tool(
                name="my.tool/v2",
                description="A tool with dots and slashes",
                input_schema={
                    "properties": {
                        "content-type": {"type": "string", "description": "CT"},
                    },
                    "required": ["content-type"],
                },
            ),
        ]
        script = generate_cli_script(
            server_name="test",
            server_spec="test",
            transport_code='"http://localhost:8000/mcp"',
            extra_imports=set(),
            tools=tools,
        )
        compile(script, "<generated>", "exec")

    def test_compiles_with_stdio_transport(self):
        transport = StdioTransport(command="fastmcp", args=["run", "server.py"])
        transport_code, extra_imports = serialize_transport(transport)
        script = generate_cli_script(
            server_name="test",
            server_spec="server.py",
            transport_code=transport_code,
            extra_imports=extra_imports,
            tools=self._make_tools(),
        )
        compile(script, "<generated>", "exec")


# ---------------------------------------------------------------------------
# generate_cli_command — integration tests
# ---------------------------------------------------------------------------


def _build_test_server() -> FastMCP:
    """Create a minimal FastMCP server for integration tests."""
    server = FastMCP("TestServer")

    @server.tool
    def greet(name: str) -> str:
        """Say hello to someone."""
        return f"Hello, {name}!"

    @server.tool
    def add(a: int, b: int) -> int:
        """Add two numbers."""
        return a + b

    @server.resource("test://greeting")
    def greeting_resource() -> str:
        """A static greeting resource."""
        return "Hello from resource!"

    @server.prompt
    def ask(topic: str) -> str:
        """Ask about a topic."""
        return f"Tell me about {topic}"

    return server


@pytest.fixture()
def _patch_client():
    """Patch resolve_server_spec and _build_client to use an in-process server."""
    server = _build_test_server()

    def fake_resolve(server_spec: Any, **kwargs: Any) -> str:
        return "fake://server"

    def fake_build_client(resolved: Any, **kwargs: Any) -> Client:
        return Client(server)

    with (
        patch.object(generate_module, "resolve_server_spec", side_effect=fake_resolve),
        patch.object(generate_module, "_build_client", side_effect=fake_build_client),
    ):
        yield


class TestGenerateCliCommand:
    @pytest.mark.usefixtures("_patch_client")
    async def test_writes_file(self, tmp_path: Path):
        output = tmp_path / "cli.py"
        await generate_cli_command("test-server", str(output))
        assert output.exists()
        content = output.read_text()
        compile(content, str(output), "exec")

    @pytest.mark.usefixtures("_patch_client")
    async def test_contains_tools(self, tmp_path: Path):
        output = tmp_path / "cli.py"
        await generate_cli_command("test-server", str(output))
        content = output.read_text()
        assert "async def greet(" in content
        assert "async def add(" in content

    @pytest.mark.usefixtures("_patch_client")
    async def test_default_output_path(
        self, tmp_path: Path, monkeypatch: pytest.MonkeyPatch
    ):
        monkeypatch.chdir(tmp_path)
        await generate_cli_command("test-server")
        assert (tmp_path / "cli.py").exists()

    @pytest.mark.usefixtures("_patch_client")
    async def test_error_if_exists(self, tmp_path: Path):
        output = tmp_path / "cli.py"
        output.write_text("existing")
        with pytest.raises(SystemExit):
            await generate_cli_command("test-server", str(output))

    @pytest.mark.usefixtures("_patch_client")
    async def test_force_overwrites(self, tmp_path: Path):
        output = tmp_path / "cli.py"
        output.write_text("existing")
        await generate_cli_command("test-server", str(output), force=True)
        content = output.read_text()
        assert content != "existing"
        assert "async def greet(" in content

    @pytest.mark.skipif(
        sys.platform == "win32", reason="Unix executable bits N/A on Windows"
    )
    @pytest.mark.usefixtures("_patch_client")
    async def test_file_is_executable(self, tmp_path: Path):
        output = tmp_path / "cli.py"
        await generate_cli_command("test-server", str(output))
        assert output.stat().st_mode & 0o111

    @pytest.mark.usefixtures("_patch_client")
    async def test_writes_skill_file(self, tmp_path: Path):
        output = tmp_path / "cli.py"
        await generate_cli_command("test-server", str(output))
        skill_path = tmp_path / "SKILL.md"
        assert skill_path.exists()
        content = skill_path.read_text()
        assert "---" in content
        assert "name:" in content

    @pytest.mark.usefixtures("_patch_client")
    async def test_skill_contains_tools(self, tmp_path: Path):
        output = tmp_path / "cli.py"
        await generate_cli_command("test-server", str(output))
        content = (tmp_path / "SKILL.md").read_text()
        assert "### greet" in content
        assert "### add" in content
        assert "--name" in content
        assert "call-tool greet" in content

    @pytest.mark.usefixtures("_patch_client")
    async def test_no_skill_flag(self, tmp_path: Path):
        output = tmp_path / "cli.py"
        await generate_cli_command("test-server", str(output), no_skill=True)
        assert not (tmp_path / "SKILL.md").exists()

    @pytest.mark.usefixtures("_patch_client")
    async def test_error_if_skill_exists(self, tmp_path: Path):
        output = tmp_path / "cli.py"
        (tmp_path / "SKILL.md").write_text("existing")
        with pytest.raises(SystemExit):
            await generate_cli_command("test-server", str(output))

    @pytest.mark.usefixtures("_patch_client")
    async def test_force_overwrites_skill(self, tmp_path: Path):
        output = tmp_path / "cli.py"
        (tmp_path / "SKILL.md").write_text("existing")
        await generate_cli_command("test-server", str(output), force=True)
        content = (tmp_path / "SKILL.md").read_text()
        assert content != "existing"
        assert "### greet" in content

    @pytest.mark.usefixtures("_patch_client")
    async def test_skill_references_cli_filename(self, tmp_path: Path):
        output = tmp_path / "my_weather.py"
        await generate_cli_command("test-server", str(output))
        content = (tmp_path / "SKILL.md").read_text()
        assert "uv run --with fastmcp python my_weather.py" in content


# ---------------------------------------------------------------------------
# _param_to_cli_flag
# ---------------------------------------------------------------------------


class TestParamToCliFlag:
    def test_simple_name(self):
        assert _param_to_cli_flag("city") == "--city"

    def test_underscore_name(self):
        assert _param_to_cli_flag("max_days") == "--max-days"

    def test_hyphenated_name(self):
        # content-type → _to_python_identifier → content_type → --content-type
        assert _param_to_cli_flag("content-type") == "--content-type"

    def test_digit_prefix(self):
        # 3d_mode → _3d_mode → --3d-mode (leading underscore stripped)
        assert _param_to_cli_flag("3d_mode") == "--3d-mode"

    def test_trailing_underscore(self):
        # from → from_ after identifier sanitization; Cyclopts strips trailing "-"
        assert _param_to_cli_flag("from") == "--from"

    def test_camel_case(self):
        # camelCase → camel-case (cyclopts default_name_transform)
        assert _param_to_cli_flag("myParam") == "--my-param"

    def test_pascal_case(self):
        assert _param_to_cli_flag("MyParam") == "--my-param"


# ---------------------------------------------------------------------------
# _schema_type_label
# ---------------------------------------------------------------------------


class TestSchemaTypeLabel:
    def test_simple_string(self):
        assert _schema_type_label({"type": "string"}) == "string"

    def test_integer(self):
        assert _schema_type_label({"type": "integer"}) == "integer"

    def test_array_of_strings(self):
        assert (
            _schema_type_label({"type": "array", "items": {"type": "string"}})
            == "array[string]"
        )

    def test_union_types(self):
        result = _schema_type_label({"type": ["string", "null"]})
        assert "string" in result
        assert "null" in result

    def test_object(self):
        assert _schema_type_label({"type": "object"}) == "object"

    def test_missing_type(self):
        assert _schema_type_label({}) == "string"


# ---------------------------------------------------------------------------
# generate_skill_content
# ---------------------------------------------------------------------------


class TestGenerateSkillContent:
    def test_frontmatter(self):
        content = generate_skill_content("weather", "cli.py", [])
        assert content.startswith("---\n")
        assert 'name: "weather-cli"' in content
        assert "description:" in content

    def test_no_tools(self):
        content = generate_skill_content("weather", "cli.py", [])
        assert "## Utility Commands" in content
        assert "## Tool Commands" not in content

    def test_tool_sections(self):
        tools = [
            mcp_types.Tool(
                name="greet",
                description="Say hello",
                input_schema={
                    "type": "object",
                    "properties": {
                        "name": {"type": "string", "description": "Who to greet"}
                    },
                    "required": ["name"],
                },
            ),
        ]
        content = generate_skill_content("test", "cli.py", tools)
        assert "## Tool Commands" in content
        assert "### greet" in content
        assert "Say hello" in content
        assert "call-tool greet" in content
        assert "`--name`" in content
        assert "| string |" in content
        assert "| yes |" in content

    def test_frontmatter_with_tools_starts_at_column_zero(self):
        tools = [
            mcp_types.Tool(
                name="greet",
                input_schema={"type": "object", "properties": {}},
            ),
        ]
        content = generate_skill_content("weather", "cli.py", tools)
        assert content.splitlines()[0] == "---"

    def test_optional_param(self):
        tools = [
            mcp_types.Tool(
                name="search",
                description="Search things",
                input_schema={
                    "type": "object",
                    "properties": {
                        "query": {"type": "string"},
                        "limit": {"type": "integer"},
                    },
                    "required": ["query"],
                },
            ),
        ]
        content = generate_skill_content("test", "cli.py", tools)
        # query is required, limit is not
        assert "| `--query` | string | yes |" in content
        assert "| `--limit` | integer | no |" in content

    def test_complex_json_param(self):
        tools = [
            mcp_types.Tool(
                name="create",
                description="Create item",
                input_schema={
                    "type": "object",
                    "properties": {
                        "data": {
                            "type": "object",
                            "properties": {"x": {"type": "integer"}},
                        },
                    },
                    "required": ["data"],
                },
            ),
        ]
        content = generate_skill_content("test", "cli.py", tools)
        assert "JSON string" in content

    def test_no_params_tool(self):
        tools = [
            mcp_types.Tool(
                name="ping",
                description="Ping the server",
                input_schema={"type": "object", "properties": {}},
            ),
        ]
        content = generate_skill_content("test", "cli.py", tools)
        assert "### ping" in content
        assert "call-tool ping" in content
        # No parameter table
        assert "| Flag |" not in content

    def test_cli_filename_in_utility_commands(self):
        content = generate_skill_content("test", "my_cli.py", [])
        assert "uv run --with fastmcp python my_cli.py list-tools" in content
        assert "uv run --with fastmcp python my_cli.py list-resources" in content

    def test_pipe_in_description_escaped(self):
        tools = [
            mcp_types.Tool(
                name="test",
                description="Test",
                input_schema={
                    "type": "object",
                    "properties": {
                        "mode": {"type": "string", "description": "a|b|c"},
                    },
                },
            ),
        ]
        content = generate_skill_content("test", "cli.py", tools)
        assert "a\\|b\\|c" in content

    def test_union_type_pipes_escaped(self):
        tools = [
            mcp_types.Tool(
                name="test",
                description="Test",
                input_schema={
                    "type": "object",
                    "properties": {
                        "val": {"type": ["string", "null"]},
                    },
                },
            ),
        ]
        content = generate_skill_content("test", "cli.py", tools)
        # Pipes in type label must be escaped so markdown table renders correctly
        assert "string \\| null" in content

    def test_boolean_param_no_value_placeholder(self):
        tools = [
            mcp_types.Tool(
                name="run",
                description="Run something",
                input_schema={
                    "type": "object",
                    "properties": {
                        "verbose": {"type": "boolean", "description": "Verbose output"},
                        "name": {"type": "string"},
                    },
                },
            ),
        ]
        content = generate_skill_content("test", "cli.py", tools)
        assert "--verbose <value>" not in content
        assert "--name <value>" in content

    def test_server_name_in_header(self):
        content = generate_skill_content("My Weather API", "cli.py", [])
        assert "# My Weather API CLI" in content
        assert 'name: "my-weather-api-cli"' in content