Skip to content

fix: preserve audio MCP content blocks#650

Open
Bortlesboat wants to merge 1 commit intoanthropics:mainfrom
Bortlesboat:fix-search-result-content-blocks
Open

fix: preserve audio MCP content blocks#650
Bortlesboat wants to merge 1 commit intoanthropics:mainfrom
Bortlesboat:fix-search-result-content-blocks

Conversation

@Bortlesboat
Copy link
Copy Markdown
Contributor

@Bortlesboat Bortlesboat commented Mar 7, 2026

Summary

  • preserve AudioContent in create_sdk_mcp_server instead of logging and skipping it
  • preserve AudioContent in _handle_sdk_mcp_request so the JSON-RPC bridge passes it through to the CLI
  • add regression coverage for both code paths

Context

PR #725 fixed resource_link and embedded resources, but audio was still dropped in both handlers because neither content-type switch handled it.

Test plan

  • python -m pytest tests/test_sdk_mcp_integration.py
  • python -m pytest tests/
  • python -m ruff check src/ tests/
  • python -m mypy src/

@Bortlesboat
Copy link
Copy Markdown
Contributor Author

Quick follow-up from my side: I’m treating review-required PRs as top priority this week. If you want any specific changes, rebase, or split, I can turn them around quickly.

@Bortlesboat Bortlesboat force-pushed the fix-search-result-content-blocks branch from fe48759 to 6db00ca Compare April 9, 2026 00:32
@Bortlesboat
Copy link
Copy Markdown
Contributor Author

Rebased this onto current main and narrowed it to the remaining AudioContent gap that was left after #725.

Fresh verification on this branch:

  • python -m pytest tests/test_sdk_mcp_integration.py
  • python -m pytest tests/
  • python -m ruff check src/ tests/
  • python -m mypy src/

@Bortlesboat Bortlesboat changed the title fix: pass through unrecognized MCP content block types fix: preserve audio MCP content blocks Apr 9, 2026
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