Skip to content

feat: add Ollama as BYOK provider with model selection field#5001

Open
ziuus wants to merge 1 commit into
RSSNext:devfrom
ziuus:dev
Open

feat: add Ollama as BYOK provider with model selection field#5001
ziuus wants to merge 1 commit into
RSSNext:devfrom
ziuus:dev

Conversation

@ziuus

@ziuus ziuus commented May 11, 2026

Copy link
Copy Markdown

Summary

Adds support for Ollama as a predefined Bring Your Own Key (BYOK) provider in Folo's AI settings, plus adds a model selection field to the provider configuration form (fixes the known issue #4942 where the BYOK form was missing model selection).

Changes

New Provider: Ollama

  • Added "ollama" to the ByokProviderName union type in @follow/shared/settings/interface
  • Added Ollama to PROVIDER_OPTIONS in constants
  • Auto-fills base URL to http://localhost:11434/v1 when Ollama is selected
  • Added ollama: "Ollama" to provider labels

Model Selection Field (fixes #4942)

  • Added model?: string | null to UserByokProviderConfig type
  • Added model text input in ByokProviderModalContent — works for all providers
  • Translation keys added for en, zh-CN, zh-TW, ja, fr-FR

Files Changed

File Change
packages/internal/shared/src/settings/interface.ts Added "ollama" type + model field to config
apps/desktop/.../ai/byok/constants.ts Added Ollama option
apps/desktop/.../ai/byok/ByokProviderModalContent.tsx Model input + auto-fill baseURL for Ollama
apps/desktop/.../ai/byok/ByokProviderItem.tsx Added Ollama label
locales/ai/{en,zh-CN,zh-TW,ja,fr-FR}.json Added model field translations

Testing

  • pnpm run typecheck ✅ (all 20 tasks pass)
  • pnpm run lint:fix ✅ (0 errors)
  • Pre-commit hooks (eslint, prettier, locale deduplication) all passed

Adds Ollama as a predefined Bring Your Own Key (BYOK) provider option with:
- Auto-filled base URL (http://localhost:11434/v1) when Ollama is selected
- New model selection text field for all providers (fixes RSSNext#4942)
- Translation keys for en, zh-CN, zh-TW, ja, fr-FR
@conquer1the2world3

Copy link
Copy Markdown

Summary

Adds support for Ollama as a predefined Bring Your Own Key (BYOK) provider in Folo's AI settings, plus adds a model selection field to the provider configuration form (fixes the known issue #4942 where the BYOK form was missing model selection).

Changes

New Provider: Ollama

  • Added "ollama" to the ByokProviderName union type in @follow/shared/settings/interface
  • Added Ollama to PROVIDER_OPTIONS in constants
  • Auto-fills base URL to http://localhost:11434/v1 when Ollama is selected
  • Added ollama: "Ollama" to provider labels

Model Selection Field (fixes #4942)

  • Added model?: string | null to UserByokProviderConfig type
  • Added model text input in ByokProviderModalContent — works for all providers
  • Translation keys added for en, zh-CN, zh-TW, ja, fr-FR

Files Changed

File Change
packages/internal/shared/src/settings/interface.ts Added "ollama" type + model field to config
apps/desktop/.../ai/byok/constants.ts Added Ollama option
apps/desktop/.../ai/byok/ByokProviderModalContent.tsx Model input + auto-fill baseURL for Ollama
apps/desktop/.../ai/byok/ByokProviderItem.tsx Added Ollama label
locales/ai/{en,zh-CN,zh-TW,ja,fr-FR}.json Added model field translations

Testing

  • pnpm run typecheck ✅ (all 20 tasks pass)
  • pnpm run lint:fix ✅ (0 errors)
  • Pre-commit hooks (eslint, prettier, locale deduplication) all passed

一直没合并吗,现在可以用ollama来总结吗

@ziuus

ziuus commented Jun 16, 2026

Copy link
Copy Markdown
Author

还没合并,PR 目前还是 open 状态。Checks 已通过、无冲突,现在等维护者 review / merge。合并后就可以通过 BYOK 选择 Ollama,并填写本地模型名来用于总结。提前测试的话可以 checkout 我这个分支本地运行。

@dai @djyde

@github-actions

Copy link
Copy Markdown
Contributor

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


It has not been merged yet, and the PR is still open. Checks passed, no conflicts, now waiting for maintainer review / merge. After merging, you can select Ollama via BYOK and fill in the local model name for summary. If you want to test in advance, you can checkout my branch and run it locally.

@dai @djyde

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] BYOK Provider Configuration Missing Model Selection Option

2 participants