Skip to content

Fix MCP headers config and hot update middleware#6

Open
melonelish wants to merge 1 commit into
xt765:masterfrom
melonelish:pr/mcp-headers-hot-update
Open

Fix MCP headers config and hot update middleware#6
melonelish wants to merge 1 commit into
xt765:masterfrom
melonelish:pr/mcp-headers-hot-update

Conversation

@melonelish

Copy link
Copy Markdown

Summary

  • make _add_or_update_server(..., headers=None) backward compatible
  • preserve existing HTTP MCP headers when updating server config
  • load HTTP MCP headers from FINCHBOT_MCP__{SERVER}__HEADERS
  • fix nested MCP env parsing for FINCHBOT_MCP__{SERVER}__ENV__{KEY}
  • fix MCP hot update middleware so sync wrappers do not treat asyncio.Task as a tool list
  • add regression tests for MCP headers, env loading, and hot update middleware behavior

Tests

  • uv run pytest -q

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