Skip to content

fix: bump mcp to 1.29.0 in mcp_google_hotel - #95

Merged
kimtaeyo merged 1 commit into
mainfrom
fix/mcp-google-hotel-1.29.0
Aug 4, 2026
Merged

fix: bump mcp to 1.29.0 in mcp_google_hotel#95
kimtaeyo merged 1 commit into
mainfrom
fix/mcp-google-hotel-1.29.0

Conversation

@kimtaeyo

@kimtaeyo kimtaeyo commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Upgrade \mcp\ from 1.23.0 to 1.29.0 in \mcp_google_hotel\ requirements.
  • Brings in MCP Python SDK security fixes (1.27.2+ session binding, 1.28.1+ transport hardening) while staying on the 1.x line compatible with \FastMCP.

Test plan

  • \pip install -r superbuilder/mcp/mcp_servers/mcp_google_hotel/requirements.txt\
  • Start google_hotel MCP server and smoke-test tool calls

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Ready to approve

The change is a straightforward dependency pin bump scoped to a single server requirements file and aligns with the PR’s stated security-fix motivation.

This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.

Pull request overview

Upgrades the pinned mcp dependency for the mcp_google_hotel MCP server to pick up newer MCP Python SDK security hardening while staying on the 1.x compatibility line.

Changes:

  • Bump mcp from 1.23.0 to 1.29.0 in mcp_google_hotel’s requirements.txt.
File summaries
File Description
superbuilder/mcp/mcp_servers/mcp_google_hotel/requirements.txt Updates the pinned MCP Python SDK version for the google_hotel server runtime environment.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.

@kimtaeyo
kimtaeyo merged commit c9eae83 into main Aug 4, 2026
1 check 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.

2 participants