Skip to content

feat: add SiliconFlow CN and Global providers#5562

Open
UNHNQ wants to merge 1 commit into
odysseus-dev:devfrom
UNHNQ:add-siliconflow-providers
Open

feat: add SiliconFlow CN and Global providers#5562
UNHNQ wants to merge 1 commit into
odysseus-dev:devfrom
UNHNQ:add-siliconflow-providers

Conversation

@UNHNQ

@UNHNQ UNHNQ commented Jul 16, 2026

Copy link
Copy Markdown

Summary

Adds SiliconFlow as a built-in OpenAI-compatible cloud provider option in the model endpoint picker, with separate presets for the China and Global API endpoints. The change follows the existing provider-preset flow: users pick a provider, enter their API key, and Odysseus discovers available models through the provider's /models endpoint instead of hard-coding a default model.

Target branch

  • This PR targets dev, not main. All PRs land in dev; main is curated by the maintainer at each release. If your PR is on main by accident, click "Edit" on this PR and change the base.

Linked Issue

Part of #5562

Type of Change

  • Bug fix (non-breaking — fixes a confirmed issue)
  • New feature (non-breaking — adds new behaviour)
  • Breaking change (changes or removes existing behaviour)
  • Refactor / cleanup (behaviour unchanged)
  • Documentation only
  • CI / tooling / configuration

Checklist

  • I searched open issues and open PRs — this is not a duplicate.
  • This PR targets dev
  • My changes are limited to the scope described above — no unrelated refactors or whitespace changes mixed in.
  • I actually ran the app (docker compose up or uvicorn app:app) and verified the change works end-to-end. Type-checks and unit tests are not enough.

How to Test

  1. Open Settings → Added Models.
  2. In the cloud provider picker, confirm both SiliconFlow (CN) and SiliconFlow (Global) are available.
  3. Select SiliconFlow (CN) and confirm the base URL is filled as https://api.siliconflow.cn/v1.
  4. Select SiliconFlow (Global) and confirm the base URL is filled as https://api.siliconflow.com/v1.
  5. Enter a valid SiliconFlow API key and click Test or Add.
  6. Confirm Odysseus queries the provider's /models endpoint and displays the discovered models.

Focused validation run locally:

python - <<'PY'
import pathlib, sys, types, pytest
root = pathlib.Path.cwd()
pkg = types.ModuleType('tests')
pkg.__path__ = [str(root / 'tests')]
sys.modules['tests'] = pkg
sys.path.insert(0, str(root))
raise SystemExit(pytest.main([
    'tests/test_provider_detection_detect.py',
    'tests/test_provider_detection_builders.py',
    'tests/test_endpoint_resolver_urls.py',
    'tests/test_admin_device_flow_static.py',
    'tests/test_providers_mixtral_logo_js.py',
    '-q',
]))
PY

Result: 64 passed. The two warnings are pre-existing local environment/dependency warnings unrelated to this change.

Visual / UI changes — REQUIRED if you touched anything that renders

Anything that changes what the UI looks like — buttons, icons, padding, colors, fonts, spacing, layout, CSS, HTML, SVG, or any static/js/ module that draws to the DOM — needs all of the following. PRs that change rendering without these WILL be closed.

  • Screenshot or short clip of the change in the running app, attached below. Mobile screenshot too if the change affects mobile.
  • Style match: the change uses Odysseus's existing visual language. Specifically:
    • Reuse existing CSS variables (--red, --fg, --bg, --card, --border, etc.) — do not introduce new color values, font sizes, or spacing units.
    • Reuse existing button/input/card/border classes. Don't invent parallel styling.
    • No Unicode emoji in UI or code. Use inline SVG (matching the monochrome icon style already in static/index.html) or plain text.
    • Monospaced font (Fira Code) for primary UI text. Don't override.
    • Dark theme is the default; any light-mode work must be wired through the existing theme system, not hard-coded.
  • No new component patterns. If a similar widget already exists in the app, extend it instead of writing a parallel one.
  • I am not an LLM agent submitting a bulk PR. If you are, please open an issue describing the problem first — bulk auto-generated PRs that don't match the project's visual style are closed on sight, even when the underlying fix is correct.

Screenshots / clips

@github-actions github-actions Bot added needs work PR description incomplete — please update before review ready for review Description complete — ready for maintainer review and removed needs work PR description incomplete — please update before review labels Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review Description complete — ready for maintainer review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant