Skip to content

fix(vision-bridge): force bridge for tokenrouter deepseek models #3946

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
diegosouzapw merged 3 commits into from
Jun 16, 2026
Merged

fix(vision-bridge): force bridge for tokenrouter deepseek models #3946

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
795605d
fix(vision-bridge): force bridge for tokenrouter deepseek models
Jun 16, 2026
080c5f6
Merge remote-tracking branch 'origin/release/v3.8.27' into fix/vision…
diegosouzapw Jun 16, 2026
7ed89e8
test(vision-bridge): assert tokenrouter deepseek models are force-bri…
WormAlien Jun 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/shared/constants/visionBridgeDefaults.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,11 @@ const FORCED_VISION_BRIDGE_MODELS = new Set<string>([
"opencode-go/qwen3.5-plus",
"opencode-zen/deepseek-v4-flash",
"opencode-zen/deepseek-v4-pro",
// tokenrouter provider: upstream models overstate vision support.
// Force Vision Bridge so images are routed through the configured
// vision model instead of being passed through to text-only backends.
"tokenrouter/deepseek-v4-pro",
"tokenrouter/deepseek-v4-flash",
]);

export function isVisionBridgeForcedModel(model: string | null | undefined): boolean {
Expand Down
8 changes: 8 additions & 0 deletions tests/unit/visionBridgeDefaults.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,14 @@ test("isVisionBridgeForcedModel does not blanket-force GPT-family models", () =>
assert.strictEqual(isVisionBridgeForcedModel("openai/gpt-4o-mini"), false);
});

test("isVisionBridgeForcedModel forces tokenrouter deepseek models (text-only backend, #3946)", () => {
// These route through a Fireworks text-only backend that overstates vision
// support at the provider level, so they must be force-bridged or raw image
// data leaks to a backend that cannot process it.
assert.strictEqual(isVisionBridgeForcedModel("tokenrouter/deepseek-v4-pro"), true);
assert.strictEqual(isVisionBridgeForcedModel("tokenrouter/deepseek-v4-flash"), true);
});

test("getVisionBridgeConfig returns defaults when no settings provided", () => {
const config = getVisionBridgeConfig({});

Expand Down