fix: bound MCP extra to supported major version

[koriyoshi2041]

Problem

Fresh skillspector[mcp] installs resolve MCP 2.x, which removed mcp.server.fastmcp. The install succeeds, but the server fails at initialization and incorrectly reports that the optional dependency is missing. Fixes #333.

Fix

• constrain the MCP extra to the supported 1.x line
• regenerate the lockfile
• distinguish an absent mcp package from an installed package missing the expected FastMCP module
• cover the incompatible-install diagnostic

Test

• uv run pytest -q tests/unit/test_mcp_server.py (10 passed, including stdio initialize/list-tools)
• uv run ruff check src/skillspector/mcp_server.py tests/unit/test_mcp_server.py
• uv run ruff format --check src/skillspector/mcp_server.py tests/unit/test_mcp_server.py
• git diff --check

Risk

Low. This deliberately holds the optional server integration on MCP 1.x until SkillSpector migrates to the 2.x API; users that force an incompatible install now receive an accurate error instead of a missing-extra message.

[rng1995]

[Automated SkillSpector Review]

Approved. Bounding the optional MCP dependency below v2 keeps the currently supported FastMCP import contract explicit, and the improved import error distinguishes an absent package from an incompatible installed layout. The focused MCP suite passed (10 tests), as did Ruff lint and format checks.

[koriyoshi2041]

Rebased onto current main at 7fc09a3. Upstream now already carries the MCP <2.0.0 dependency bound, so the remaining delta is limited to distinguishing an incompatible installed MCP layout from a missing optional dependency, with its regression test.

Focused MCP tests pass (10/10); Ruff lint/format, lock check, and diff check also pass.

[rng1995]

@koriyoshi2041 - Please resolve the merge conflict.

[koriyoshi2041]

Rebased onto current main and resolved the test-file conflict while retaining both the new local-target coverage and this PR’s incompatible-MCP regression. Focused MCP tests pass (27/27); Ruff lint/format, lock check, and diff check are clean.

[rng1995]

Re-review approved at the current head. The MCP v2 upper bound and compatibility error remain acceptable and there are no unresolved review threads. CI is green except for the mechanical DCO failure on the unsigned merge commit.