Skip to content

Fix provider API-key visibility and GPT-5.6 discovery - #63

Merged
ranade-oss merged 5 commits into
mainfrom
agent/fix-provider-api-key-fields
Jul 29, 2026
Merged

Fix provider API-key visibility and GPT-5.6 discovery#63
ranade-oss merged 5 commits into
mainfrom
agent/fix-provider-api-key-fields

Conversation

@ranade-oss

@ranade-oss ranade-oss commented Jul 29, 2026

Copy link
Copy Markdown
Owner

What changed

  • Keep the Anthropic, Gemini, and OpenAI credential fields mounted after the asynchronous model catalog loads.
  • Continue showing OpenRouter only for self-hosted deployments.
  • Preserve catalog governance for expanded providers such as xAI and Moonshot.
  • Recognize OpenAI's public gpt-5.6 alias when /v1/models lists the underlying gpt-5.6-sol identifier.
  • Add regression coverage for both credential-field stability and alias-aware model discovery.

Root causes

Credential fields

The API-key page initially rendered a fallback provider set containing Anthropic, Gemini, and OpenAI. After useModelCatalog() returned, the page filtered those fields through approvedProviders. On the hosted beta, that response currently narrows to OpenAI, so the Anthropic and Gemini inputs appeared briefly and then disappeared.

GPT-5.6 availability

ROSS's compatibility registry correctly contains gpt-5.6, but live discovery required /v1/models to return that exact string. OpenAI documents gpt-5.6 as an alias for gpt-5.6-sol; when the project lists the underlying model identifier, ROSS incorrectly marked the alias unavailable.

User impact

  • Users can consistently enter personal Anthropic, Gemini, or OpenAI keys.
  • GPT-5.6 becomes selectable when the configured OpenAI project exposes either gpt-5.6 or gpt-5.6-sol.
  • Environment-provided keys remain disabled and identified by the existing server-configuration state.
  • No key values, encryption behavior, request authentication, backend credential storage, or provider approval rules are changed.

Risk and review

This PR touches the provider-credential interface and backend model-availability logic. It remains intentionally draft pending human review. Exact-head Baseline verification is required after it is marked ready.

Validation

  • Source diff is limited to the API-key page, model-discovery logic, and one baseline regression test.
  • The regression rejects catalog-based removal of the three core credential inputs.
  • The regression requires alias-aware GPT-5.6 discovery and rejects literal-only availability checks.
  • Full exact-head Baseline remains pending.

@ranade-oss ranade-oss changed the title Keep core model API key fields visible Fix provider API-key visibility and GPT-5.6 discovery Jul 29, 2026
@ranade-oss
ranade-oss marked this pull request as ready for review July 29, 2026 17:56
@ranade-oss
ranade-oss merged commit e8a23db into main Jul 29, 2026
6 checks passed
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.

1 participant