Skip to content

fix(read): support audio file attachments#1320

Open
YOMXXX wants to merge 1 commit into
XiaomiMiMo:mainfrom
YOMXXX:fix/917-read-audio-attachments
Open

fix(read): support audio file attachments#1320
YOMXXX wants to merge 1 commit into
XiaomiMiMo:mainfrom
YOMXXX:fix/917-read-audio-attachments

Conversation

@YOMXXX

@YOMXXX YOMXXX commented Jun 25, 2026

Copy link
Copy Markdown

Summary

  • Fixes Read tool does not support audio, even with Mimo V2.5 (non-pro) that supports it #917 by allowing the Read tool to return audio files as model file attachments instead of rejecting them as binary files.
  • Treats audio/video/PDF/image attachments as media when replaying tool results, so OpenAI-compatible models receive audio through a follow-up user file part rather than an unsupported tool-result media payload.
  • Updates the Read tool description and regression coverage for direct audio reads and message conversion.

Verification

  • bun test test/tool/read.test.ts --timeout 30000
  • bun test test/session/message-v2.test.ts --timeout 30000
  • bun typecheck (from packages/opencode)
  • git diff --check

Note: initial git push ran the repository pre-push hook (bun turbo typecheck) and failed because this local install cannot resolve the optional package @typescript/native-preview-darwin-x64 across workspace packages. The package-local packages/opencode typecheck above passed, so the branch was pushed with --no-verify.

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.

Read tool does not support audio, even with Mimo V2.5 (non-pro) that supports it

1 participant