Skip to content

fix(deps): hold mcp below 2.0 until the MCPServer rename is done - #32

Merged
Showdown76py merged 1 commit into
mainfrom
fix/pin-mcp-below-2
Jul 29, 2026
Merged

fix(deps): hold mcp below 2.0 until the MCPServer rename is done#32
Showdown76py merged 1 commit into
mainfrom
fix/pin-mcp-below-2

Conversation

@Showdown76py

Copy link
Copy Markdown
Owner

Pourquoi

mcp 2.0.0 est sorti et a renommé mcp.server.fastmcp en mcp.server.mcpserver, ainsi que FastMCP en MCPServer. La dépendance était déclarée mcp[cli]>=1.0, sans borne haute, donc la CI a installé la 2.0 dès sa publication et quatre modules de test ne collectent plus :

ModuleNotFoundError: No module named 'mcp.server.fastmcp'
ERROR tests/test_bmc_ipmi.py
ERROR tests/test_bmc_registry.py
ERROR tests/test_proxmox_helpers.py
ERROR tests/test_snapshots.py

main est rouge sur les trois versions de Python, et toutes les PR ouvertes héritent de la panne. Rien à voir avec leur contenu.

Changement

Une ligne : mcp[cli]>=1.0,<2.

La 1.29.0 est la dernière 1.x et expose toujours mcp/server/fastmcp/__init__.py avec FastMCP, donc le plafond ne coûte rien aujourd'hui et la CI récupère du signal utile.

Ce que ça ne fait pas

Ça ne migre pas vers l'API 2.0. Le renommage touche chaque module d'outils (proxmox/, ssh/, bmc/, server.py) et la surface de MCPServer diffère au-delà du chemin d'import, donc c'est un travail séparé à faire volontairement, pas un remplacement d'imports.

mcp 2.0.0 shipped and renamed mcp.server.fastmcp to mcp.server.mcpserver,
along with FastMCP to MCPServer. Every tool module imports the old path,
so pip resolving the open-ended >=1.0 to 2.0 broke collection of four test
modules on main and on every open PR.

Cap at <2 to get CI reporting again. 1.29.0 is the newest 1.x and still
ships mcp.server.fastmcp, so the ceiling costs nothing today. Migrating to
the 2.0 API is separate work.
@Showdown76py
Showdown76py merged commit 4bc09aa into main Jul 29, 2026
6 checks passed
@Showdown76py
Showdown76py deleted the fix/pin-mcp-below-2 branch July 29, 2026 03:13
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.

1 participant