Skip to content

fix(host): remove ConfigType instances from global manifest to restor…#86

Open
17Wang wants to merge 1 commit into
devfrom
fix/config
Open

fix(host): remove ConfigType instances from global manifest to restor…#86
17Wang wants to merge 1 commit into
devfrom
fix/config

Conversation

@17Wang

@17Wang 17Wang commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

…e file-based config persistence coding by deepseek-v4-pro

Manifest instances (is_override=True) were overwriting ConfigStore cache with code defaults during bootstrap, bypassing YAML file configuration entirely. The global manifest should only register class references (is_override=False) per its own documented convention; instances belong in mode-specific configs.

  • Remove TTSManagerConfig() and other ConfigType instances from MOSS/manifests/configs.py (workspace + stub)
  • Fix tts_factory.yml: move default_speaker from top level into volcengine_stream_tts_model_config where VolcengineTTSConf fields live

via claude code

…e file-based config persistence coding by deepseek-v4-pro

Manifest instances (is_override=True) were overwriting ConfigStore cache
with code defaults during bootstrap, bypassing YAML file configuration
entirely. The global manifest should only register class references
(is_override=False) per its own documented convention; instances belong
in mode-specific configs.

- Remove TTSManagerConfig() and other ConfigType instances from
  MOSS/manifests/configs.py (workspace + stub)
- Fix tts_factory.yml: move default_speaker from top level into
  volcengine_stream_tts_model_config where VolcengineTTSConf fields live
- Change default_speaker to 可爱女生

via claude code
@thirdgerb

Copy link
Copy Markdown
Contributor

和 tts 的改造有 conflict, 辛苦 rebase 或修改一下

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