feat(voice): add TTS MCP bridge and expression registry - #1116
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (9)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7ba89e2c3a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| f"{provider.base_url}/api{load_ep}", | ||
| json={"data": []}, |
There was a problem hiding this comment.
This posts engine load/unload requests to .../api/..., but the flute gateway’s Ultimate-TTS integration uses Gradio’s event transport (/gradio_api/call/...) and no longer relies on /api endpoints; on current Gradio setups this path returns failures, so MCP tts_load_engine/tts_unload_engine won’t actually change engine VRAM state. Use the same event-based call flow (submit + poll event_id) already used by providers/ultimate_tts.py.
Useful? React with 👍 / 👎.
| "modifier": modifier, | ||
| "text_length": len(text), | ||
| "audio_bytes": len(audio_bytes), | ||
| "audio_base64": audio_b64[:100] + "..." if len(audio_b64) > 100 else audio_b64, |
There was a problem hiding this comment.
The synthesis response truncates audio_base64 to the first 100 characters plus ..., which makes it invalid audio data for any MCP client trying to decode/play the result. In real calls this means successful synthesis still cannot be consumed downstream; return the full base64 string (or a retrievable URL) instead of a preview snippet.
Useful? React with 👍 / 👎.
| def create_mcp_router( | ||
| get_provider: Any, | ||
| get_nats_client: Any, | ||
| ) -> APIRouter: |
There was a problem hiding this comment.
This commit introduces the MCP router factory, but there is no wiring path that registers it with the flute-gateway FastAPI application; a repo-wide search for create_mcp_router only returns this definition. Without app inclusion, /sse and /messages are never exposed, so the bridge added in this commit is unreachable at runtime.
Useful? React with 👍 / 👎.
…ity hardening Fleet Networking & RustDesk: - KVM2 self-hosted RustDesk relay (hbbs+hbbr, systemd, UFW-locked) - Scripts: fix-kvm2-rustdesk-relay.sh, restart-jetson-rustdesk.sh (env var driven — HOSTINGER_KVM2_IP, RUSTDESK_RELAY_KEY, JETSON_IPS) - RUSTDESK_SELF_HOSTED.md deployment guide (sanitized, no secrets) - QR code generation instructions for mobile enrollment Security Hardening: - Network hardening: localhost defaults for all service bindings - 4090 laptop host hardening script - SSH key-only auth across 8 nodes (password disabled) - Gitignore: rustdesk QR images excluded from repo Infrastructure: - DeepResearch Dockerfile: add pmoves.chit COPY for CGP_SPEC_VERSION (Python files only, secrets manifests excluded from image layers) - TOPOLOGY.md: KVM2 RustDesk ports, updated node hostnames, Jetson status - AGNOTE4482 ACK: Z890-CLAUDE fleet networking claim signed Also includes merged PR content: - Hi-RAG embedding defaults aligned to Qwen3 2560d (#1122) - Pinokio Codex plugin + Agent Zero launcher (#1121) - Publishing approval handoff state (#1120) - TTS MCP bridge + expression registry (#1116) - Damage-control hooks Windows compat (#1123) - AGNOTE4482 fleet claim docs (#1127) - Dependency bumps (#1111, #1113, #1128) - Bootstrap SUPERSEDED_VALUES (#1112) - Various docs updates (#1115, #1117, #1119, #1124, #1125) - Publisher RPC idempotent guards (#1126) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ity hardening Fleet Networking & RustDesk: - KVM2 self-hosted RustDesk relay (hbbs+hbbr, systemd, UFW-locked) - Scripts: fix-kvm2-rustdesk-relay.sh, restart-jetson-rustdesk.sh (env var driven — HOSTINGER_KVM2_IP, RUSTDESK_RELAY_KEY, JETSON_IPS) - RUSTDESK_SELF_HOSTED.md deployment guide (sanitized, no secrets) - QR code generation instructions for mobile enrollment Security Hardening: - Network hardening: localhost defaults for all service bindings - 4090 laptop host hardening script - SSH key-only auth across 8 nodes (password disabled) - Idempotent authorized_keys injection (grep+append, never truncate) - Gitignore: rustdesk QR images excluded from repo Infrastructure: - DeepResearch Dockerfile: add pmoves.chit COPY for CGP_SPEC_VERSION (Python files only, secrets manifests excluded from image layers) - TOPOLOGY.md: KVM2 RustDesk ports, updated node hostnames, Jetson status - AGNOTE4482 ACK: Z890-CLAUDE fleet networking claim signed - setup-glances.sh: prefer uv over pip per CLAUDE.md Also includes merged PR content: - Hi-RAG embedding defaults aligned to Qwen3 2560d (#1122) - Pinokio Codex plugin + Agent Zero launcher (#1121) - Publishing approval handoff state (#1120) - TTS MCP bridge + expression registry (#1116) - Damage-control hooks Windows compat (#1123) - Various docs, deps, bootstrap updates (#1111-#1128) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2 participants
feat(voice): add TTS MCP bridge and expression registry
Summary
/tts:expresscommand definitionValidation
Notes