fix(providers): 按 active ASR provider 返回配置状态

[H-Chris233]

User description

变更说明

• 后端 get_credentials() 改为基于 active ASR provider 计算配置状态，新增返回字段：
  • activeAsrProvider
  • activeLlmProvider
  • asrConfigured
  • llmConfigured
• ASR 状态判定规则：
  • volcengine：仍按 appKey + accessKey + resourceId 判定。
  • local-qwen3：不依赖云端凭据，视为已配置。
  • 其余 OpenAI-compatible ASR（如 whisper/siliconflow/zhipu/groq）：按 asr.endpoint + asr.model 判定，API Key 可空以支持 keyless endpoint。
• LLM 状态判定同样支持 keyless endpoint：按 ark.endpoint + ark.model_id 判定，API Key 可空。
• 前端 providerSetup 改为使用通用 asrConfigured/llmConfigured，不再硬依赖 volcengineConfigured。
• Overview ASR/LLM 卡片改为显示 active provider 名称与其真实配置状态，不再固定显示火山引擎状态。
• 保留旧字段 volcengineConfigured / arkConfigured 作为兼容过渡。
• 增加配置状态判定回归用例。

验证

• npm run -s build
• cargo test -q（74 passed）

影响与风险

• 修复非 Volcengine 用户“已配置却仍提示未配置”的误判。
• 避免 endpoint/model 缺失时误报已配置；API Key 可空用于支持 keyless provider。
• 保持最小侵入：不改现有凭据存储结构，仅补充状态合约并迁移 UI 读取。

Closes #223

---

PR Type

Bug fix, Tests

---

Description

• Active provider drives status checks

• Add ASR/LLM status fields

• Refresh overview and prompts

• Expand regression coverage

---

Diagram Walkthrough

flowchart LR
  Vault["Credentials vault"]
  Status["get_credentials()"]
  Prompt["Provider setup prompt"]
  Overview["Overview cards"]

  Vault -- "snapshot + active provider" --> Status
  Status -- "asrConfigured / llmConfigured" --> Prompt
  Status -- "active names + state" --> Overview

Loading

File Walkthrough

	Relevant files
Bug fix	2 files commands.rs Compute active-provider credential status                                +100/-9  providerSetup.ts Accept new and legacy flags                                                            +3/-1
Enhancement	4 files persistence.rs Snapshot ASR credential fields                                                      +11/-0    types.rs Extend credentials status contract                                              +5/-0      types.ts Add active provider status fields                                                +5/-0      Overview.tsx Show active provider names and state                                          +38/-6
Miscellaneous	1 files ipc.ts Update mocked credentials status shape                                      +4/-0
Tests	1 files providerSetup.test.ts Cover active-provider setup logic                                                +61/-6

---

[chatgpt-codex-connector]

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

[github-actions]

PR Reviewer Guide 🔍

(Review updated until commit cb60a03)

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis 🔶 223 - Partially compliant Compliant requirements: Volcengine ASR remains tied to App Key / Access Key / Resource ID. get_credentials() now returns active provider identifiers and generic status fields. Overview and provider setup use the new generic status fields. Regression tests were added for the new status logic. Non-compliant requirements: Whisper-compatible ASR providers are not reported configured from asr.api_key; the new check requires asr.endpoint and asr.model instead. Requires further human verification: Whether the updated Overview text and provider names match the intended UX in a real UI session.

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 PR contains tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Status mismatch For non-Volcengine ASR providers, the new readiness check ignores asr.api_key and only returns true when asr.endpoint and asr.model are set. That means an active Whisper-compatible provider can be treated as configured even when its API key is missing, or still shown as unconfigured if the provider is API-key based. This breaks the active-provider contract from the ticket and can suppress or mis-trigger the setup prompt. fn asr_configured_for_provider(provider: &str, snap: &CredentialsSnapshot) -> bool { if provider == "volcengine" { return volcengine_configured(snap); } if provider == crate::asr::local::PROVIDER_ID { // 本地 ASR 不依赖云端凭据。 return true; } configured(&snap.asr_endpoint) && configured(&snap.asr_model) }

[github-actions]

Persistent review updated to latest commit b827381

[github-actions]

Persistent review updated to latest commit cb60a03