Skip to content

fix(ai): route scene generation through configured providers - #183

Merged
fj-sunny merged 1 commit into
1024XEngineer:mainfrom
fj-sunny:codex/async-scene-ielts-latency
Sep 14, 2026
Merged

fj-sunny merged 1 commit into
1024XEngineer:mainfrom
fj-sunny:codex/async-scene-ielts-latency

Conversation

@fj-sunny

Copy link
Copy Markdown
Contributor

Summary

  • route custom scene generation through the admin-configured LLM route
  • prioritize Qiniu MaaS and allow fallback to Alibaba Qwen when Qiniu rejects a request
  • route model-less TTS calls through the configured TTS fallback chain
  • add the admin fallback-model control and remove the Qiniu environment kill switch
  • use dynamic Alibaba workspace credentials and assign distinct audit request IDs

Tests

  • ./mvnw --settings docker/maven/settings.xml -Dtest=AiProviderRegistryTest,CustomSceneGeneratorTest,CustomSceneServiceTest,QwenLlmProviderTest test (59 passed)
  • npm test -- --run src/features/system/SystemManagementPage.test.tsx (8 passed)
  • npm run build

Closes #182

@fennoai fennoai 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.

Review conclusion

I reviewed the complete fixed diff for the configurable LLM/TTS routing, Qiniu-to-Qwen failover, dynamic Alibaba workspace endpoint handling, audit request scoping, and admin fallback-model controls. I did not identify a reportable correctness or regression issue in the changed code. git diff --check passes. The focused Maven tests could not start because the configured ${env.MAVEN_REPO_URL} repository is unavailable in this environment, and the admin test dependencies are not installed (node_modules is absent).

@pionxe pionxe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@fj-sunny
fj-sunny merged commit 9879464 into 1024XEngineer:main Sep 14, 2026
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.

Route custom scene generation through configurable LLM/TTS providers

2 participants