Skip to content

fix: handle boolean mcp schema values#2452

Open
MukundaKatta wants to merge 1 commit into
googleapis:mainfrom
MukundaKatta:codex/mcp-schema-bool-additional-properties
Open

fix: handle boolean mcp schema values#2452
MukundaKatta wants to merge 1 commit into
googleapis:mainfrom
MukundaKatta:codex/mcp-schema-bool-additional-properties

Conversation

@MukundaKatta
Copy link
Copy Markdown
Contributor

Fixes #2393.

Summary

  • make MCP schema filtering tolerate non-dict schema values such as boolean additionalProperties
  • add regression coverage for FastMCP-style schemas with additionalProperties: false

Tests

  • uv run --with pytest --with pytest-asyncio --with mcp pytest google/genai/tests/mcp/test_mcp_to_gemini_tools.py -q
  • git diff --check

@Venkaiahbabuneelam Venkaiahbabuneelam self-assigned this May 22, 2026
@Venkaiahbabuneelam Venkaiahbabuneelam added the size:M Code changes between 10-40 lines label May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M Code changes between 10-40 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AttributeError: 'bool' object has no attribute 'items' in _mcp_utils.py when using FastMCP tools

2 participants