refactor(webview): canonicalize provider model config identifiers

[WebMad]

Summary

Extracts the provider model configuration utility changes from #1141 into a focused pull request:

• canonicalizes provider registry keys in the settings model configuration utilities
• canonicalizes provider comparisons, docs slugs, and custom model UI entries
• adds focused coverage for canonical keys, Z.ai defaults, and model-change side effects

This PR is intentionally limited to webview-ui/src/components/settings/utils.

Validation

• npx vitest run src/components/settings/utils/__tests__/providerModelConfig.spec.ts — 29 tests passed
• pnpm check-types in webview-ui
• ESLint with --prune-suppressions --max-warnings=0 for both changed files
• repository pre-commit lint
• repository pre-push type checks
• git diff --check

Related to #944.
Extracted from #1141.

Summary by CodeRabbit

• Bug Fixes
  • Improved provider configuration handling across supported services.
  • Preserved correct defaults and settings when switching models, including reasoning options and Bedrock model identifiers.
  • Improved handling for Z.ai, static, custom, and unrecognized providers.

[coderabbitai]

📝 Walkthrough

Walkthrough

The provider configuration utilities replace hardcoded provider keys with providerIdentifiers. Tests cover canonical registry access, provider-specific model configuration, custom model UI membership, Bedrock ARN handling, and reasoning-setting resets.

Changes

Provider identifier migration

Layer / File(s)	Summary
Canonicalize provider configuration mappings webview-ui/src/components/settings/utils/providerModelConfig.ts	Provider metadata, defaults, model configuration, documentation mappings, custom model lists, and Bedrock cleanup use providerIdentifiers constants.
Validate canonical provider behavior webview-ui/src/components/settings/utils/__tests__/providerModelConfig.spec.ts	Tests cover canonical keys, Z.ai defaults, unknown providers, static providers, custom model UI membership, Bedrock ARN clearing, and reasoning-setting resets.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

• Zoo-Code-Org/Zoo-Code#1141: Updates the same provider configuration utility and tests with canonical provider identifiers.
• Zoo-Code-Org/Zoo-Code#1030: Replaces hardcoded provider names with canonical identifiers across provider logic and tests.
• Zoo-Code-Org/Zoo-Code#991: Applies the same provider identifier registry to related provider-specific logic.

Suggested reviewers: edelauna, navedmerchant, taltas

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description explains the refactor and validation, but it omits required template sections, checklist items, and the required issue-link format.	Use the repository template, add the required issue link, complete the pre-submission checklist, and address documentation, snapshot, and reviewer-contact sections.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the primary change: canonicalizing provider model configuration identifiers.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

webview-ui/src/components/settings/utils/__tests__/providerModelConfig.spec.ts

ESLint skipped: missing config or dependency (missing-dependency). The ESLint configuration references a package that is not available in the sandbox.

webview-ui/src/components/settings/utils/providerModelConfig.ts

ESLint skipped: the ESLint configuration for this file references a package that is not available in the sandbox.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[edelauna]

Looks good, ended up bumping up test coverage before approving and merging.

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In
`@webview-ui/src/components/settings/utils/__tests__/providerModelConfig.spec.ts`:
- Around line 154-155: Remove the unnecessary as any cast in the
getProviderModelConfig test and pass "unknown-provider" directly, since the
function accepts a string.

🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 0895a605-fbc5-43ce-a381-325c2145b0d4

📥 Commits

Reviewing files that changed from the base of the PR and between 0ae9217 and fe0956b.

📒 Files selected for processing (2)

• webview-ui/src/components/settings/utils/__tests__/providerModelConfig.spec.ts
• webview-ui/src/components/settings/utils/providerModelConfig.ts

[coderabbitai]

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

ast-grep outline webview-ui/src/components/settings/utils/providerModelConfig.ts \
  --match getProviderModelConfig --view expanded

ast-grep run --lang ts \
  --pattern 'getProviderModelConfig($ARG)' \
  webview-ui/src/components/settings/utils/__tests__/providerModelConfig.spec.ts

Repository: Zoo-Code-Org/Zoo-Code

Length of output: 639

---

Remove the unnecessary as any cast.

getProviderModelConfig accepts a string, so pass "unknown-provider" directly.

Proposed fix

-			expect(getProviderModelConfig("unknown-provider" as any)).toBeUndefined()
+			expect(getProviderModelConfig("unknown-provider")).toBeUndefined()

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	it("returns undefined for a provider with no model config entry", () => {
	expect(getProviderModelConfig("unknown-provider" as any)).toBeUndefined()
	it("returns undefined for a provider with no model config entry", () => {
	expect(getProviderModelConfig("unknown-provider")).toBeUndefined()

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@webview-ui/src/components/settings/utils/__tests__/providerModelConfig.spec.ts`
around lines 154 - 155, Remove the unnecessary as any cast in the
getProviderModelConfig test and pass "unknown-provider" directly, since the
function accepts a string.

Source: Coding guidelines