Skip to content

fix: replace dead E-girl voice ID with working reference#90

Merged
M2Night merged 1 commit into
mainfrom
fix/egirl-voice-id
Jun 18, 2026
Merged

fix: replace dead E-girl voice ID with working reference#90
M2Night merged 1 commit into
mainfrom
fix/egirl-voice-id

Conversation

@M2Night

@M2Night M2Night commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

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 TTS workflow (see run 27671547106). Because speak-mintlify fails 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 throws NotSupportedError in the browser.

Fix

Replace the dead ID with a verified working reference ca3007f96ae7499ab87d27ea3599956a in 3 places:

  • speaker-config.yaml — CI generation list
  • snippets/audio-transcript.jsx — frontend fallback voice list (used when MDX uses <AudioTranscript page="..." />)
  • developer-guide/getting-started/quickstart.mdx — E-Girl Voice tab demo strings

What auto-updates after merge

The TTS workflow runs on merge to main and:

  1. Regenerates mp3s for all MDX files with the new voice ID
  2. Opens a follow-up chore: update TTS Audio PR rewriting the voices arrays in MDX files (e.g. contributing.mdx)

That follow-up PR also needs to be merged.

Test plan

  • Verified new voice ID on fish.audio
  • Local mint dev — Quick Start widget requests audio/.../ca3007....mp3; Tab demo strings show new ID
  • Post-merge: TTS workflow run succeeds
  • Post-merge: Listen-to-Page on docs.fish.audio/.../quickstart plays

Summary by CodeRabbit

  • Documentation

    • Updated voice customization examples with new identifiers
  • Chores

    • Updated E-Girl voice configuration identifiers across settings and references

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)
@M2Night M2Night requested a review from Kilerd June 18, 2026 07:37
@coderabbitai

coderabbitai Bot commented Jun 18, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The E-girl voice identifier is updated from an old UUID/hash to a new one across three files: speaker-config.yaml (voices map key), snippets/audio-transcript.jsx (fallback pageVoices array entry), and developer-guide/getting-started/quickstart.mdx (bash snippet REFERENCE_ID and voice URL).

Changes

E-girl voice ID update

Layer / File(s) Summary
E-girl voice ID replacement across config, snippet, and docs
speaker-config.yaml, snippets/audio-transcript.jsx, developer-guide/getting-started/quickstart.mdx
The E-girl voice key in the YAML config, the id field in the JSX fallback voices array, and the REFERENCE_ID environment variable plus From: URL in the quickstart bash snippet are all changed to the new voice identifier.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • fishaudio/docs#31: Updates the AudioTranscript example in the n8n integration doc to use an E-girl voice ID in its predefined voices list — directly related to the same E-girl voice identifier being updated here.

Suggested reviewers

  • twangodev

Poem

🐰 Hippity hop, a voice got a new name,
The E-girl still sings, just tagged with new fame.
Three files updated, a UUID swapped clean,
The docs, the config, the snippet — all serene.
squeak Old hash begone, fresh ID takes the stage! 🎤

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately summarizes the main change: replacing an invalid E-girl voice ID with a working reference across three files in the documentation system.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/egirl-voice-id

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 and usage tips.

@mintlify

mintlify Bot commented Jun 18, 2026

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
hanabiaiinc 🟢 Ready View Preview Jun 18, 2026, 7:41 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 | 🟡 Minor

Remove the old voice ID from contributing.mdx

The old voice ID 8ef4a238714b45718ce04243307c57a7 is still present in contributing.mdx at 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

📥 Commits

Reviewing files that changed from the base of the PR and between 9a3a6e7 and 29a1adb.

📒 Files selected for processing (3)
  • developer-guide/getting-started/quickstart.mdx
  • snippets/audio-transcript.jsx
  • speaker-config.yaml

@M2Night M2Night merged commit f1d7383 into main Jun 18, 2026
6 checks passed
@M2Night M2Night deleted the fix/egirl-voice-id branch June 18, 2026 07:51
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