Skip to content

Constrain server packages to FastMCP 3.x - #23

Merged
Tuhin Kundu (TuhinKundu) merged 1 commit into
mainfrom
tuhinkundu/constrain-fastmcp-3x
Sep 2, 2026
Merged

Constrain server packages to FastMCP 3.x#23
Tuhin Kundu (TuhinKundu) merged 1 commit into
mainfrom
tuhinkundu/constrain-fastmcp-3x

Conversation

@TuhinKundu

Copy link
Copy Markdown
Contributor

Summary

  • Require FastMCP >=3.4.3,<4 in both server packages.
  • Prevent incompatible FastMCP 4.x installations.

Testing

  • 74 thinkingbox_tools tests passed
  • 2,423 business-ops unit tests passed
  • 2 Typesense-backed MCP integration tests passed

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The dependency specifier change is correct for PEP 621 dependencies and cleanly enforces the intended FastMCP 3.x constraint in both server packages.

Pull request overview

This PR constrains the fastmcp dependency to the FastMCP 3.x line for the two Python MCP server packages, preventing accidental installs of incompatible FastMCP 4.x.

Changes:

  • Pin fastmcp to >=3.4.3,<4 in servers/thinkingbox_tools.
  • Pin fastmcp to >=3.4.3,<4 in servers/tb_business_ops_servers_202606.
File summaries
File Description
servers/thinkingbox_tools/pyproject.toml Constrains fastmcp dependency to >=3.4.3,<4 under PEP 621 [project].dependencies.
servers/tb_business_ops_servers_202606/pyproject.toml Constrains fastmcp dependency to >=3.4.3,<4 under PEP 621 [project].dependencies.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@TuhinKundu
Tuhin Kundu (TuhinKundu) merged commit d3467d8 into main Sep 2, 2026
8 checks passed
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.

3 participants