Repository navigation
fix(website): correct LiteLLM transcription route and guards - #3736
Merged
Merged
Conversation
Signed-off-by: LauraGPT <18321252+LauraGPT@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
openai/sensevoicewith an explicit self-hostedapi_base: http://127.0.0.1:8000/v1. Explain the server model alias and container reachability; mark the upstream docs PR as pending, not merged.custom_openai. Require visible configuration, reject unsupported provider/checkpoint examples and malformed model/base suffixes, and match the already-published37K+copy.custom_openaiis not supported for audio transcription in the verified LiteLLM path. The packaged FunASR server accepts thesensevoiceservice alias, notFunAudioLLM/SenseVoiceSmallas its request model. The native SDK/transport/ASGI evidence is documented in LiteLLM docs #610; ASGI inference was stubbed, not acoustic-tested.Validation
36K+guard before correction.python -m pytest tests/test_check_funasr_website_static.py web-pages/product-site/tests/test_legacy.py -q: 70 passed (54 website checks + 16 snapshot checks).python -m pytest web-pages/product-site/tests -q -n 4 --dist=load: 519 passed, with JUnit confirming all 519 collected cases and zero skipped/failing/error cases. Pinned pytest-xdist 3.8.0 was installed only in the isolated validation environment; repository dependencies and tests were not changed to enable parallelism.diff -qrconfirmed identical output.--line-length=100,pip check, andgit diff --checkpassed. Whole-file formatting follows the existing pre-commit configuration.The initial 240-second and 480-second serial site-test attempts timed out and are not counted as successful runs.
test_output.pyrebuilds the entire site for each case; four isolated pytest workers completed the unchanged full suite in 187 seconds. The initial stale-manifest failure was corrected before the successful full run.Boundaries
This PR changes site source and guards only. It does not publish or restart the production website, download or run speech models, boot a live LiteLLM Proxy, or approve external CI/deployments. Public pages still require a separately verified release after merge. No model/GPU or full application acceptance is claimed.
Merge Verification
Merged as
12e417f4fa4490296ea2d81c5352ea2de8e4cac2through the ordinary SHA-guarded GitHub merge endpoint. Parents are current main15c19598110268237a3e0ca9cdcd33c83a0b3f89and unchanged PR headf950854db3379e80a986e779da4e7208f54c0adb; GitHub verifies the merge signature. No protection setting, review, or workflow approval was changed.The actual merged tree
24a836e751ce1faef214aa7ec49932ad75ed1d22exactly matches the locally tested combined tree. GitHub's earlier synthetic merge reference still pointed at the old base, so it was rejected as validation evidence. A signed local-only validation commit was used instead; the PR history was not rewritten.All four PR-head hosted checks succeeded before merge. Post-merge workflow status is recorded separately below; premerge and local successes are not claims that the main-push workflows or production deployment have completed.
At 2026-09-29 06:44 UTC, exact merge-SHA roadmap validation and API documentation succeeded. Product site validation was still in progress: legacy document links passed, build-and-validate was running, and the dependent browser job was not yet reported. This is a timestamped snapshot, not an all-green postmerge claim.
Follow-up on 2026-09-29: Product site run 36532592558 has now completed successfully on the exact merge SHA. All three jobs succeeded: legacy-docs-links (109289386077), build-and-validate (109289386363), and browser (109293383476). The earlier in-progress snapshot is historical. Production deployment remains separate and was not performed.