Skip to content

fix(stage-ui): persist speech provider settings - #2467

Open
0xSelenicDove wants to merge 3 commits into
moeru-ai:mainfrom
0xSelenicDove:0xSelenicDove/fix/speech-provider-settings
Open

fix(stage-ui): persist speech provider settings#2467
0xSelenicDove wants to merge 3 commits into
moeru-ai:mainfrom
0xSelenicDove:0xSelenicDove/fix/speech-provider-settings

Conversation

@0xSelenicDove

Copy link
Copy Markdown
Collaborator

Summary

  • Save speech provider changes through the synchronized provider store.
  • Keep API key, base URL, and voice settings in the persisted provider state.
  • Add a browser regression test for Cannot set a voice provider #2447.

Closes #2447.

Validation

  • pnpm -F @proj-airi/stage-ui exec vitest run --project browser src/components/scenarios/providers/speech-provider-settings.browser.test.ts
  • pnpm -F @proj-airi/stage-ui exec vitest run --project node src/stores/providers/config.test.ts
  • pnpm -F @proj-airi/stage-ui typecheck
  • pnpm typecheck
  • pnpm lint

Visual changes

No static visual change. The provider settings page now keeps the selected voice settings after navigation.

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

⏳ Approval required for deploying to Cloudflare Workers (Preview) for stage-web.

Name Link
🔭 Waiting for approval For maintainers, approve here

Hey, maintainers, kindly take some time to review and approve this deployment when you are available. Thank you! 🙏

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 04b006bb9b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/stage-ui/src/stores/providers/config.ts Outdated
Comment thread packages/stage-ui/src/components/scenarios/providers/speech-provider-settings.vue Outdated

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6942c4201b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/stage-ui/src/components/scenarios/providers/speech-provider-settings.vue Outdated
…er RPC

A rejected leader-routed patch left the promise chain permanently rejected,
so every later `.then` was skipped and edits stopped saving silently.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@zapabob

zapabob commented Sep 6, 2026

Copy link
Copy Markdown

I have applied the fixes from the review comments but could not create a PR due to errors. Please see the local changes in the fork.

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.

Cannot set a voice provider

2 participants