Skip to content

fix: preserve OpenAI compatibility upstream model metadata - #112

Open
YamonBot wants to merge 3 commits into
router-for-me:devfrom
YamonBot:fix/openai-compat-upstream-model-metadata
Open

fix: preserve OpenAI compatibility upstream model metadata#112
YamonBot wants to merge 3 commits into
router-for-me:devfrom
YamonBot:fix/openai-compat-upstream-model-metadata

Conversation

@YamonBot

@YamonBot YamonBot commented Sep 4, 2026

Copy link
Copy Markdown

Closes #111

Why

Home synthesizes OpenAI-compatible aliases into home_config_models, but currently retains only the client-facing alias ID and drops the configured upstream model name. Dynamic service-key dispatch therefore forwards the alias literally to the provider.

What changed

  • preserve OpenAICompatibilityModel.Name in synthesized registry.ModelInfo
  • add a focused regression proving fractalops-coding retains upstream glm-5.3 metadata

Verification

  • go test ./internal/watcher/synthesizer ./internal/cliproxy/auth
  • git diff --check origin/dev...HEAD

This is separate from OAuth response-model force-mapping; it restores request-time alias-to-upstream resolution for OpenAI-compatible API-key providers.

@YamonBot

YamonBot commented Sep 4, 2026

Copy link
Copy Markdown
Author

@hkfires This is blocking a live Home service-key deployment. The production change is one metadata field and the focused tests pass locally; a maintainer approval for fork CI/review would help us validate and ship it upstream without carrying a downstream fork.

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