fix: replace dead E-girl voice ID with working reference#90
Conversation
The voice reference 8ef4a238714b45718ce04243307c57a7 was no longer valid in Fish Audio (returned 400 'Reference not found'), causing the TTS CI to fail and leaving Listen-to-Page widgets broken on all newly-edited pages. Updated to ca3007f96ae7499ab87d27ea3599956a in: - speaker-config.yaml (CI generation list) - snippets/audio-transcript.jsx (frontend fallback list) - developer-guide/getting-started/quickstart.mdx (Tab demo strings)
📝 WalkthroughWalkthroughThe E-girl voice identifier is updated from an old UUID/hash to a new one across three files: ChangesE-girl voice ID update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
developer-guide/getting-started/quickstart.mdx (1)
196-218:⚠️ Potential issue | 🟡 MinorRemove the old voice ID from contributing.mdx
The old voice ID
8ef4a238714b45718ce04243307c57a7is still present incontributing.mdxat lines 11 and 13. Update or remove these references to maintain consistency with the changes made in quickstart.mdx.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@developer-guide/getting-started/quickstart.mdx` around lines 196 - 218, The old voice ID `8ef4a238714b45718ce04243307c57a7` in contributing.mdx is now outdated and inconsistent with the updated voice IDs introduced in quickstart.mdx. Locate all references to this old voice ID in contributing.mdx and either replace them with one of the new voice IDs (such as the E-Girl Voice ID `ca3007f96ae7499ab87d27ea3599956a` or Energetic Male ID `802e3bc2b27e49c2995d23ef70e6ac89`) or remove the references entirely if they are no longer needed, ensuring consistency across the documentation.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@developer-guide/getting-started/quickstart.mdx`:
- Around line 196-218: The old voice ID `8ef4a238714b45718ce04243307c57a7` in
contributing.mdx is now outdated and inconsistent with the updated voice IDs
introduced in quickstart.mdx. Locate all references to this old voice ID in
contributing.mdx and either replace them with one of the new voice IDs (such as
the E-Girl Voice ID `ca3007f96ae7499ab87d27ea3599956a` or Energetic Male ID
`802e3bc2b27e49c2995d23ef70e6ac89`) or remove the references entirely if they
are no longer needed, ensuring consistency across the documentation.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: de28224c-d117-474e-b277-be3652244adf
📒 Files selected for processing (3)
developer-guide/getting-started/quickstart.mdxsnippets/audio-transcript.jsxspeaker-config.yaml
Problem
The voice reference
8ef4a238714b45718ce04243307c57a7(E-girl) is no longer valid in Fish Audio. The TTS API returns:400 Bad Request
{ "message": "Reference not found" }
This broke the GitHub Actions
TTSworkflow (see run 27671547106). Becausespeak-mintlifyfails the entire MDX file on the first failed voice, none of the 6 voices' mp3s got generated for any page touched after the regression. End result: Listen-to-Page widget on pages like Quick Start hits 404 in R2 and throwsNotSupportedErrorin the browser.Fix
Replace the dead ID with a verified working reference
ca3007f96ae7499ab87d27ea3599956ain 3 places:speaker-config.yaml— CI generation listsnippets/audio-transcript.jsx— frontend fallback voice list (used when MDX uses<AudioTranscript page="..." />)developer-guide/getting-started/quickstart.mdx— E-Girl Voice tab demo stringsWhat auto-updates after merge
The
TTSworkflow runs on merge to main and:chore: update TTS AudioPR rewriting thevoicesarrays in MDX files (e.g.contributing.mdx)That follow-up PR also needs to be merged.
Test plan
mint dev— Quick Start widget requestsaudio/.../ca3007....mp3; Tab demo strings show new IDdocs.fish.audio/.../quickstartplaysSummary by CodeRabbit
Documentation
Chores