docs: add FunASR voice input example#6593
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughAdds a “Transcribing Audio Before Kickoff” section describing a local FunASR server, OpenAI-compatible audio transcription, and passing the resulting text into ChangesAudio transcription documentation
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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.
Inline comments:
In `@docs/edge/en/concepts/files.mdx`:
- Around line 108-112: Update the FunASR startup command in the local setup
example to bind explicitly to 127.0.0.1, preserving the existing model, device,
and port options.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: d04c690c-a2cb-41bc-a972-2516fdd5948f
📒 Files selected for processing (1)
docs/edge/en/concepts/files.mdx
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)
docs/edge/en/concepts/files.mdx (1)
111-121: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick winUse the same loopback address in the client and server examples.
The server binds to IPv4
127.0.0.1, but the client connects tolocalhost. On systems wherelocalhostresolves to IPv6::1first, the request can fail because the server is not listening on IPv6. Usehttp://127.0.0.1:8000/v1or bind the server to both address families.Proposed fix
client = OpenAI( api_key="EMPTY", - base_url="http://localhost:8000/v1", + base_url="http://127.0.0.1:8000/v1", )🤖 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 `@docs/edge/en/concepts/files.mdx` around lines 111 - 121, Update the OpenAI client’s base_url in the transcription example to use the same IPv4 loopback address as the FunASR server command: 127.0.0.1 on port 8000 with the /v1 path, replacing localhost.
🤖 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 `@docs/edge/en/concepts/files.mdx`:
- Around line 111-121: Update the OpenAI client’s base_url in the transcription
example to use the same IPv4 loopback address as the FunASR server command:
127.0.0.1 on port 8000 with the /v1 path, replacing localhost.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: fee20397-856e-4969-a142-4ced86a3abb8
📒 Files selected for processing (1)
docs/edge/en/concepts/files.mdx
65156a0 to
d2bc103
Compare
|
Rebased this docs-only PR onto current
The PR still intentionally covers only the voice-input/transcription documentation slice and no longer claims to close the broader voice-agent/TTS issue. |
Summary
crew.kickoffRelated to #5983. This PR covers the voice-input/transcription slice only; it does not claim to implement TTS or the full voice-agent workflow.
Validation
git diff --checkcrew.kickoffexamplemintlify broken-linkswas not run because the Mintlify CLI is not installed in this remote checkout; this PR adds no new links.