Skip to content

fix: handle empty text_to_audio errors consistently#90

Open
cnYui wants to merge 1 commit into
MiniMax-AI:mainfrom
cnYui:codex/fix-text-to-audio-empty-text-error
Open

fix: handle empty text_to_audio errors consistently#90
cnYui wants to merge 1 commit into
MiniMax-AI:mainfrom
cnYui:codex/fix-text-to-audio-empty-text-error

Conversation

@cnYui

@cnYui cnYui commented Jun 7, 2026

Copy link
Copy Markdown

Summary

  • move the empty text_to_audio text validation inside the existing try block
  • add a regression test that expects empty text to return a TextContent error instead of raising MinimaxRequestError

Fixes #88.

Verification

  • .\.venv\Scripts\python.exe -m pytest -q — 7 passed
  • .\.venv\Scripts\python.exe -m pytest --cov=minimax_mcp — 7 passed
  • .\.venv\Scripts\ruff.exe check tests/test_server.py — passed
  • git diff --check — passed

Notes

ruff check . still reports pre-existing lint issues in the repository, including the current HEAD version of server.py; this PR keeps the change scoped to the validation behavior and the regression test.

@cnYui cnYui force-pushed the codex/fix-text-to-audio-empty-text-error branch from 1f3279c to 48be654 Compare June 8, 2026 11:30
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.

bug(server): text_to_audio validates text outside try/except, inconsistent with other tools

1 participant