Skip to content

test: Pin mcp<2 for fastmcp<0.3 - #6925

Merged
alexander-alderman-webb merged 37 commits into
masterfrom
webb/pin-mcp
Jul 29, 2026
Merged

test: Pin mcp<2 for fastmcp<0.3#6925
alexander-alderman-webb merged 37 commits into
masterfrom
webb/pin-mcp

Conversation

@alexander-alderman-webb

@alexander-alderman-webb alexander-alderman-webb commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Description

Old fastmcp package versions are incompatible with mcp v2.

Stack traces are as follows:

tests/integrations/fastmcp/test_fastmcp.py:847: in test_fastmcp_prompt_sync
    mcp = FastMCP("Test Server")
          ^^^^^^^^^^^^^^^^^^^^^^
.tox/py3.14-fastmcp-v0.1.0/lib/python3.14/site-packages/fastmcp/server.py:61: in __init__
    self._setup_handlers()
.tox/py3.14-fastmcp-v0.1.0/lib/python3.14/site-packages/fastmcp/server.py:85: in _setup_handlers
    self._mcp_server.list_tools()(self.list_tools)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^
E   AttributeError: 'Server' object has no attribute 'list_tools'

Issues

Closes #6906

Reminders

@alexander-alderman-webb
alexander-alderman-webb marked this pull request as ready for review July 29, 2026 12:27
@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner July 29, 2026 12:27
@github-actions

github-actions Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

98668 passed | ⏭️ 6781 skipped | Total: 105449 | Pass Rate: 93.57% | Execution Time: 347m 29s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +134
Passed Tests 📈 +127
Failed Tests 📉 -2
Skipped Tests 📈 +9

➖ Removed Tests (1)

View removed tests
  • test_continuous_profiler_auto_start_and_stop_sampled_span_streaming[experiment-thread]
    • File: tests.profiler.test_continuous_profiler

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2498 uncovered lines.
✅ Project coverage is 89.84%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.84%    89.84%        —%
==========================================
  Files          193       193         —
  Lines        24587     24587         —
  Branches      8744      8744         —
==========================================
+ Hits         22088     22089        +1
- Misses        2499      2498        -1
- Partials      1400      1398        -2

Generated by Codecov Action

Base automatically changed from webb/skip-sse-test to master July 29, 2026 14:11
@alexander-alderman-webb
alexander-alderman-webb merged commit a17ea54 into master Jul 29, 2026
271 of 273 checks passed
@alexander-alderman-webb
alexander-alderman-webb deleted the webb/pin-mcp branch July 29, 2026 14:31
mgaligniana pushed a commit to mgaligniana/sentry-python that referenced this pull request Aug 9, 2026
Early `fastmcp` package versions are incompatible with `mcp` v2, and the `mcp` version was only pinned later on.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix FastMCP tox failures

2 participants