Skip to content

Fix Murf integration serving Gen2 instead of falcon-2 - #457

Merged
coval-cale merged 1 commit into
mainfrom
cale/bench-625-fix-murf-falcon-2-male-pool-voice
Aug 6, 2026
Merged

Fix Murf integration serving Gen2 instead of falcon-2#457
coval-cale merged 1 commit into
mainfrom
cale/bench-625-fix-murf-falcon-2-male-pool-voice

Conversation

@coval-cale

@coval-cale coval-cale commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Murf's us-east host ignores the model param and always serves Gen2, so the provider now uses the global endpoint with the recommended Amara/Gordon voice pool.

Greptile Summary

The PR updates the Murf TTS integration to use Murf's global WebSocket endpoint and replaces Natalie with Amara in the recommended two-voice pool.

  • Routes falcon-2 synthesis through global.api.murf.ai.
  • Uses Amara and Gordon for balanced benchmark voice assignment.
  • Updates Murf provider tests for the new endpoint and voice.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete changed-code failure established.

The endpoint, model, path, query structure, and WebSocket frame protocol remain internally consistent, while the registry voice changes flow through the existing balanced assignment and result-recording paths.

Reviews (1): Last reviewed commit: "Fix Murf integration serving Gen2 instea..." | Re-trigger Greptile

Context used:

Murf's us-east host ignores the model param and always serves Gen2, so the provider now uses the global endpoint with the recommended Amara/Gordon voice pool.
@coval-cale
coval-cale requested a review from a team as a code owner August 6, 2026 15:32
@coval-cale
coval-cale enabled auto-merge August 6, 2026 15:32
@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: c86e1b38-b122-4e6f-91b0-1786d4098acc

📥 Commits

Reviewing files that changed from the base of the PR and between 0799052 and 2f8ce2c.

📒 Files selected for processing (3)
  • runner/src/coval_bench/providers/tts/murf.py
  • runner/src/coval_bench/registries/models.py
  • runner/tests/providers/tts/test_murf.py

Walkthrough

The Murf TTS integration now uses Murf’s global streaming WebSocket endpoint. The Falcon 2 registry entry now uses Amara as its primary voice and retains Gordon as the alternate voice. Murf provider tests now use the updated voice and global endpoint hostname.

Suggested reviewers: seribaymadina

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the Murf integration issue and the intended Falcon 2 behavior.
Description check ✅ Passed The description accurately explains the endpoint change and the recommended Amara/Gordon voice pool.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coval-cale
coval-cale added this pull request to the merge queue Aug 6, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 6, 2026
@coval-cale
coval-cale added this pull request to the merge queue Aug 6, 2026
Merged via the queue into main with commit 5ceda21 Aug 6, 2026
12 of 15 checks passed
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