Skip to content

fix(runtime): preserve custom OpenAI-compatible model IDs#1248

Open
jsiu93 wants to merge 1 commit into
RightNow-AI:mainfrom
jsiu93:fix-openai-compatible-model-id
Open

fix(runtime): preserve custom OpenAI-compatible model IDs#1248
jsiu93 wants to merge 1 commit into
RightNow-AI:mainfrom
jsiu93:fix-openai-compatible-model-id

Conversation

@jsiu93

@jsiu93 jsiu93 commented Jun 8, 2026

Copy link
Copy Markdown

Summary

Fixes #1195.

  • Preserve namespaced OpenAI-compatible model IDs when provider = "openai" uses a custom base_url.
  • Route request construction, manifest normalization, model switching, and fallback chains through api_model_name.
  • Copy configured [provider_urls] into spawned manifests before normalization so custom endpoint semantics survive agent registration.

Root cause

openai/<model> was treated as an internal catalog prefix everywhere. For custom OpenAI-compatible endpoints such as Featherless, the provider expects the full ID, for example openai/gpt-oss-120b.

Testing

  • cargo test -p openfang-runtime --lib
  • cargo test -p openfang-kernel --lib
  • cargo test --workspace
  • cargo clippy --workspace --all-targets -- -D warnings

Security

  • No new unsafe code
  • No secrets or API keys in diff
  • No new user input boundary

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.

OpenAI-compatible custom base_url strips openai/ from Featherless model IDs

1 participant