fix(gui): allow hidden models in Combo selectors

[xianhongtao]

Summary

Make it easier to use Combos to organize and simplify the model list exposed to clients.

Users may want to expose a few named Combos while hiding their underlying provider models to keep the client model picker tidy. Previously, hiding a provider model also removed it from the Combo target picker, forcing users to make it visible again before adding it to a Combo.

This PR separates external model visibility from internal Combo configuration:

• Include hidden models in Combo target selectors and JEV decision-model suggestions.
• Keep external visibility unchanged when a hidden model is selected or saved in a Combo.
• Preserve existing disabled-provider, decision-service, and recursion restrictions.
• Bump the Combo workspace cache version to avoid reusing previously filtered model lists.
• Add regression tests and update the Combo documentation across locales.

Users can now keep their client-facing model list focused on Combos while freely selecting the underlying models when configuring them.

Verification

Passed:

• GUI: bun test tests/combo-capability-hidden-targets.test.tsx tests/page-loading-contract.test.tsx — 30 passed, 0 failed.
• GUI: bun run build
• Repository root: bun run typecheck
• Repository root: bun run structure:check
• Documentation: bun install --frozen-lockfile and bun run build
• git diff --check

Regression coverage includes hidden-model selection and saving, all-hidden catalogs, old-cache isolation, capability metadata, and JEV decision-model suggestions.

Validation was scoped to focused regression tests, type checking, and builds for this GUI-only behavior change. The full repository test suite was not run.

Checklist

• Scope stays focused and avoids unrelated cleanup.
• Docs

Summary by CodeRabbit

• New Features
  • Combo model selectors now include hidden and disabled models, allowing them to be selected without changing their external visibility.
  • Target selection continues to exclude disabled providers, nested combos, and JEV providers reserved for the decision service.
• Documentation
  • Updated the Combo guides to clarify model visibility and target-selection behavior.

Review readiness checklist

This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:

• Required local validation passed; commands, results, and any full-suite exception are documented.

• I pushed my PR to a recent dev commit (at most 10 behind; a maintainer may still ask for the exact tip before merge).

• I resolved all correct Codex and CodeRabbit findings.

• My PR is ready for review.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository: lidge-jun/opencodex/.coderabbit.yaml
• Review profile: ASSERTIVE
• Plan: Advanced
• Run ID: 2083e9c6-7974-4c3d-9b8a-31f9a7aa5f49

📥 Commits

Reviewing files that changed from the base of the PR and between 60bf4c7 and 110a13f.

📒 Files selected for processing: 12

• docs-site/src/content/docs/fr/guides/combos.md
• docs-site/src/content/docs/guides/combos.md
• docs-site/src/content/docs/ja/guides/combos.md
• docs-site/src/content/docs/ko/guides/combos.md
• docs-site/src/content/docs/ru/guides/combos.md
• docs-site/src/content/docs/tr/guides/combos.md
• docs-site/src/content/docs/zh-cn/guides/combos.md
• docs-site/src/content/docs/zh-tw/guides/combos.md
• gui/src/pages/Combos.tsx
• gui/tests/combo-capability-hidden-targets.test.tsx
• gui/tests/page-loading-contract.test.tsx
• structure/dashboard-and-usage.md

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

📝 Walkthrough

The Combo workspace now includes disabled catalog models in its internal selectors. The workspace cache key changes to v2. Tests and localized guides describe hidden-model selection, target exclusions, and cache behavior.

Changes: Combo hidden-model selection

Layer / File(s)	Summary
Workspace selection and supporting updates gui/src/pages/Combos.tsx, gui/tests/*, structure/dashboard-and-usage.md, docs-site/src/content/docs/*/guides/combos.md	The workspace model list includes valid non-combo catalog rows, including disabled models, and the cache key changes to v2. Tests cover hidden-model selection, saving, JEV suggestions, and cache behavior. The guides describe hidden-model selection and target-selector exclusions. The routingQuota documentation is reflowed without a described behavior change.

Priority: ➖ Normal

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 110a1

Hidden models become selectable in Combo pickers without changing their external visibility. No concrete merge-blocking risk was identified in the supplied changes.

Pre-merge checks: 4 · 1

Failed checks: 1 warning

Check name	Status	Explanation	Resolution
Docstring Coverage		Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 3 files. (9 skipped: 9 …	Write docstrings for the functions missing them to satisfy the coverage threshold.

Passed checks: 4 passed

Check name	Status	Explanation
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly and concisely describes the main change: allowing hidden models in Combo selectors. It matches the implementation, tests, and documentation updates.

Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 3 files. (9 skipped: 9 unsupported.)

• Fix all pre-merge checks with AI

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.}

[github-actions]

✅ Deterministic PR hygiene checks passed.

[github-actions]

⏳ DRAFT

• wrong branch ancestry; rebase onto the latest dev. UI screenshot required.

What to do

• Rebase onto the current dev branch instead of opening from main.
• Add a screenshot of the UI change to the PR description.
• Tick all four boxes in the PR description once you're done (currently 0/4).

Review readiness checklist

• ⬜ Required local validation passed; commands, results, and any full-suite exception are documented.
• ⬜ I pushed my PR to a recent dev commit (at most 10 behind; a maintainer may still ask for the exact tip before merge).
• ⬜ I resolved all correct Codex and CodeRabbit findings.
• ⬜ My PR is ready for review.

0/4 boxes ticked.

This pull request is being kept as a draft automatically. Once every issue above is resolved, it will be marked ready for review again.
@xianhongtao Tick the boxes once required local validation has passed with commands, results, and any full-suite exception documented, your branch is on the latest dev commit, and every correct Codex and CodeRabbit finding is resolved.

[lidge-jun]

Being carried into #7223 with your authorship preserved; this PR will be closed when the bundle lands.

[lidge-jun]

Landed on dev in a8d1bb8 via #7223 (your change is its own commit in that bundle, and the squash keeps your authorship as Co-authored-by). Thank you!