Skip to content

fix(providers): support Codex model discovery - #2010

Closed
changjing9527 wants to merge 2 commits into
TencentCloud:mainfrom
changjing9527:fix/codex-model-discovery
Closed

changjing9527 wants to merge 2 commits into
TencentCloud:mainfrom
changjing9527:fix/codex-model-discovery

Conversation

@changjing9527

Copy link
Copy Markdown

Fixes #1992

Summary

  • send the maintained client_version query parameter for Codex model discovery
  • parse Codex's models[].slug / display_name response shape
  • preserve the existing OpenAI-compatible discovery path unchanged
  • add regression coverage for Codex mapping, deduplication, and query parameters

Validation

  • uv run --frozen pytest tests/unit/test_provider_fetch_models.py -q (10 passed)
  • uv run --frozen ruff check src/octop/infra/agents/providers/probe.py tests/unit/test_provider_fetch_models.py
  • uv run --frozen ruff format --check src/octop/infra/agents/providers/probe.py tests/unit/test_provider_fetch_models.py
  • git diff --check

@changjing9527

Copy link
Copy Markdown
Author

Superseded by the clean, upstream-based implementation in #2013.

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.

Bug: Codex 获取模型列表缺少 client_version,且未适配返回格式

1 participant