mirror of
https://github.com/diegosouzapw/OmniRoute.git
synced 2026-07-31 04:12:10 +03:00
* chore(release): open v3.8.14 development cycle Version bump 3.8.13 -> 3.8.14 (root + electron + open-sse + openapi + lockfiles). Seed the v3.8.14 changelog with the four post-tag hotfixes that shipped to Docker/Electron in v3.8.13 but missed the immutable npm 3.8.13 (#3336 SSRF / CodeQL #323, #3334/#3335/#3339 Electron packaging). i18n CHANGELOG mirrors get the in-progress placeholder section. * feat: add per-provider custom headers support for OpenAI/Anthropic-compatible nodes (#3338) Integrated into release/v3.8.14 * fix: Kiro Builder ID token import fails with Bad credentials (#3333) Integrated into release/v3.8.14 — adds Builder ID cached-creds + OIDC refresh path for Kiro token import, with regression tests (#3333). * Improve code quality: auto-pr/docstrings-1780792063 (#3337) Integrated into release/v3.8.14 — docstring for context analytics route re-export. * fix(catalog): remove minimaxai/minimax-m3 from NVIDIA NIM tier (404 upstream) (#3329) (#3341) NVIDIA NIM does not host minimaxai/minimax-m3 — every request returns 404 page not found, while sibling minimaxai/minimax-m2.7 on the same provider works. Advertising a model that 404s is a catalog bug; remove it from the nvidia tier (it remains on the tiers that actually serve MiniMax M3). Re-add only once NVIDIA serves it. Co-authored-by: mikmaneggahommie <mikmaneggahommie@users.noreply.github.com> * fix(cli): write OpenCode config to ~/.config on all platforms incl. Windows (#3330) (#3343) resolveOpencodeConfigDir used %APPDATA% on Windows, but OpenCode reads its config from XDG ~/.config/opencode/ on every platform (on Windows: %USERPROFILE%\.config\opencode\, NOT %APPDATA%). So a Windows user who configured OpenCode via the dashboard had the file written where OpenCode never looks — it silently had no effect. Use the XDG path (XDG_CONFIG_HOME || ~/.config) unconditionally. Update the UI note + route JSDoc, and flip the three tests that encoded the old %APPDATA% behavior (t40 per-platform + card-note, cli-runtime-extended getCliConfigPaths). Co-authored-by: abdulkadirozyurt <abdulkadirozyurt@users.noreply.github.com> * fix(proxy): make auto-selection fallback opt-in (#3332) (#3344) selectWorkingProxyFallback (Step 11 of resolveProxyForConnection) listed ALL registry proxies, ignoring assignments and per-connection proxy_enabled, and returned the first working one with level:'autoSelect'. So a single proxy added to the registry silently became a global fallback for every connection's traffic. Gate it behind a new PROXY_AUTO_SELECT_ENABLED feature flag (default off): the fallback now no-ops unless the operator opts in. No registry proxy becomes a silent global default anymore. Co-authored-by: hertznsk <hertznsk@users.noreply.github.com> * fix(sse): treat MiniMax M3 as multimodal so vision isn't stripped (#3328) (#3342) MiniMax M3 via the opencode provider (oc/minimax-m3-free) appeared blind: image inputs didn't reach the model, while the same model in Cline could see them. Verified empirically that MiniMax M3 on the opencode upstream IS multimodal -- a base64 image is described correctly (it returns 403 only for remote image URLs, which it doesn't accept). Root cause: OmniRoute treated MiniMax M3 as a non-vision model in two places, so when compression was active the image was replaced with a text placeholder before dispatch: - compression's modelSupportsVision() heuristic (lite.ts) only matched gpt-4/4o/claude-3/gemini/vision -- minimax was absent -> replaceImageUrls stripped the image. - the opencode minimax-m3-free catalog entry lacked supportsVision, so the combo vision-capability gate could also exclude/mishandle it. Add 'minimax-m3' to the vision heuristic and supportsVision: true to the opencode minimax-m3-free entry. TDD: a failing-then-passing test in compression/lite.test.ts proves replaceImageUrls now keeps images for minimax-m3 ids, plus a registry assertion mirroring the #2822 qwen test. Reported-by: @mikmaneggahommie * docs(i18n): translate 25 core documentation files to Indonesian (#3348) Integrated into release/v3.8.14 — Indonesian i18n docs. * fix(review): resolve /review-reviews battery findings (LEDGER-1..11) on v3.8.14 (#3350) Integrated into release/v3.8.14 — /review-reviews battery hardening (LEDGER-1..11) for #3338 custom-headers + #3333 kiro, plus cycle-test drift fixes (#3329/#3330/#3332). * fix(provider-proxy): honor per-account proxy toggles (#3349) Integrated into release/v3.8.14 — honor per-account proxy toggles + auto-fallback opt-in via PROXY_AUTO_SELECT_ENABLED. * fix(dashboard): remove duplicate Distribute Proxies button on provider page (#3352) * fix(providers): reduce proxy label noise (#3346) Integrated into release/v3.8.14 — reduce proxy label noise + a11y (aria-label/sr-only). * fix(duckduckgo): restore bare Response contract and rebase onto release/v3.8.14 (#3323) Integrated into release/v3.8.14 — browser-backed cookie providers (duckduckgo/claude-web) with restored executor contract + unit tests. * fix(noauth): expose only usable model aliases (#3345) Integrated into release/v3.8.14 — noauth usable-alias filtering + registry alias plumbing (veo-free). * fix(dashboard): stop infinite config-load loop on Hermes Agent detail page (#3353) * fix(electron): tree-kill the server on exit/update to release the omniroute.exe lock (#3347) (#3354) * chore(release): finalize v3.8.14 changelog + clear release-gate drift - CHANGELOG: finalize the v3.8.14 section (date, full New Features/Bug Fixes/ Maintenance coverage of all 16 cycle commits, Contributors hall of 12). - docs: document OMNIROUTE_BROWSER_POOL + WEB_COOKIE_USE_BROWSER (#3323) in .env.example + ENVIRONMENT.md; regenerate the id/llm.txt strict mirror (#3348 had translated it; llm.txt mirrors must match root). - test(proxy-fetch): #3323 made tlsClient.available a computed getter — stub it via Object.defineProperty instead of assignment (5 tests were red on the base). * fix(translator): coerce Gemini functionDeclaration parameters to an OBJECT schema (#3357) (#3360) * fix(gemini): resolve truncation/suppression of false positive textual tool call markers in backticks (#3358) Integrated into release/v3.8.14 — Gemini/Antigravity textual tool-call marker normalization (no false-positive suppression + split-chunk buffering). * docs(changelog): add #3358 Gemini textual tool-call normalization to v3.8.14 * fix(dashboard): surface real analytics error instead of generic placeholder (#3356) (#3361) The Analytics page discarded the server's error body on a non-OK response and rendered a generic "An error occurred", so users (and maintainers) could not see why /api/usage/analytics 500'd after an upgrade. Now the route returns the real reason via buildErrorBody (sanitized, Hard Rule #12) and the page surfaces it via a new readFetchErrorMessage helper that handles both the OpenAI-style and legacy error shapes. Reported-by: @superti4r --------- Co-authored-by: PizzaV <103120356+pizzav-xyz@users.noreply.github.com> Co-authored-by: Someres <168349709+quanturbo@users.noreply.github.com> Co-authored-by: Dong Mengzhe <154944819+Lang-Qiu@users.noreply.github.com> Co-authored-by: mikmaneggahommie <mikmaneggahommie@users.noreply.github.com> Co-authored-by: abdulkadirozyurt <abdulkadirozyurt@users.noreply.github.com> Co-authored-by: hertznsk <hertznsk@users.noreply.github.com> Co-authored-by: Krisna Santosa <54174372+KrisnaSantosa15@users.noreply.github.com> Co-authored-by: Randi <55005611+rdself@users.noreply.github.com> Co-authored-by: Wilson <pedbookmed@gmail.com> Co-authored-by: Paijo <14921983+oyi77@users.noreply.github.com> Co-authored-by: Ardem2025 <ardemb22@gmail.com>
200 lines
7.5 KiB
TypeScript
200 lines
7.5 KiB
TypeScript
import test from "node:test";
|
|
import assert from "node:assert/strict";
|
|
import fs from "node:fs/promises";
|
|
import path from "node:path";
|
|
|
|
const { CLI_TOOLS } = await import("../../src/shared/constants/cliTools.ts");
|
|
const { resolveOpencodeConfigPath } = await import("../../src/shared/services/cliRuntime.ts");
|
|
const { buildOpenCodeProviderConfig, buildOpenCodeConfigDocument, mergeOpenCodeConfig } =
|
|
await import("../../src/shared/services/opencodeConfig.ts");
|
|
|
|
test("T40: OpenCode card documents config paths and --variant usage", () => {
|
|
const opencode = CLI_TOOLS.opencode;
|
|
assert.ok(opencode, "OpenCode tool card must exist");
|
|
assert.equal(opencode.modelSelectionMode, "multiple");
|
|
assert.equal(opencode.hideComboModels, true);
|
|
assert.equal(opencode.previewConfigMode, "opencode");
|
|
|
|
const notesText = (opencode.notes || [])
|
|
.map((note) => note?.text || "")
|
|
.join(" ")
|
|
.toLowerCase();
|
|
|
|
assert.match(notesText, /\.config\/opencode\/opencode\.json/);
|
|
// #3330: OpenCode uses ~/.config on all platforms (incl. Windows) — the note
|
|
// must no longer point Windows users at %APPDATA%.
|
|
assert.doesNotMatch(notesText, /%appdata%/);
|
|
assert.match(notesText, /%userprofile%/);
|
|
assert.match(notesText, /--variant/);
|
|
});
|
|
|
|
test("T40: OpenCode config path resolves per-platform", () => {
|
|
const linuxWithXdg = resolveOpencodeConfigPath(
|
|
"linux",
|
|
{ XDG_CONFIG_HOME: "/tmp/xdg-config-home" },
|
|
"/home/dev"
|
|
);
|
|
assert.equal(linuxWithXdg, path.join("/tmp/xdg-config-home", "opencode", "opencode.json"));
|
|
|
|
const linuxDefault = resolveOpencodeConfigPath("linux", {}, "/home/dev");
|
|
assert.equal(linuxDefault, path.join("/home/dev", ".config", "opencode", "opencode.json"));
|
|
|
|
// #3330: OpenCode uses XDG `~/.config/opencode/` on ALL platforms including
|
|
// Windows (NOT %APPDATA%) — OmniRoute must write where OpenCode reads.
|
|
const windowsPath = resolveOpencodeConfigPath(
|
|
"win32",
|
|
{ APPDATA: "C:\\Users\\dev\\AppData\\Roaming" },
|
|
"C:\\Users\\dev"
|
|
);
|
|
assert.equal(
|
|
windowsPath,
|
|
path.join("C:\\Users\\dev", ".config", "opencode", "opencode.json")
|
|
);
|
|
|
|
// Windows still honors XDG_CONFIG_HOME when set.
|
|
const windowsXdg = resolveOpencodeConfigPath(
|
|
"win32",
|
|
{ XDG_CONFIG_HOME: "D:\\xdg" },
|
|
"C:\\Users\\dev"
|
|
);
|
|
assert.equal(windowsXdg, path.join("D:\\xdg", "opencode", "opencode.json"));
|
|
});
|
|
|
|
test("T40: OpenCode config generator includes endpoint and selected API key", () => {
|
|
const providerConfig = buildOpenCodeProviderConfig({
|
|
baseUrl: "http://localhost:20128/v1/",
|
|
apiKey: "sk_test_opencode",
|
|
model: "claude-sonnet-4-5-thinking",
|
|
});
|
|
assert.equal(providerConfig.options.baseURL, "http://localhost:20128/v1");
|
|
assert.equal(providerConfig.options.apiKey, "sk_test_opencode");
|
|
assert.ok(providerConfig.models["claude-sonnet-4-5-thinking"]);
|
|
|
|
const mergedConfig = mergeOpenCodeConfig(
|
|
{ provider: { custom: { name: "Custom Provider" } } },
|
|
{
|
|
baseUrl: "http://localhost:20128/v1",
|
|
apiKey: "sk_test_opencode",
|
|
model: "claude-sonnet-4-5-thinking",
|
|
}
|
|
);
|
|
assert.ok(mergedConfig.provider.custom);
|
|
assert.equal(mergedConfig.provider.omniroute.options.baseURL, "http://localhost:20128/v1");
|
|
assert.equal(mergedConfig.provider.omniroute.options.apiKey, "sk_test_opencode");
|
|
});
|
|
|
|
test("T40: OpenCode config document uses current provider schema", () => {
|
|
const configDocument = buildOpenCodeConfigDocument({
|
|
baseUrl: "http://localhost:20128/v1/",
|
|
apiKey: "sk_test_opencode",
|
|
models: ["cc/claude-sonnet-4-20250514", "gg/gemini-2.5-pro"],
|
|
modelLabels: {
|
|
"cc/claude-sonnet-4-20250514": "Claude Sonnet 4.5",
|
|
"gg/gemini-2.5-pro": "Gemini 2.5 Pro",
|
|
},
|
|
});
|
|
|
|
assert.equal(configDocument.$schema, "https://opencode.ai/config.json");
|
|
assert.ok(configDocument.provider.omniroute);
|
|
assert.equal(configDocument.provider.omniroute.npm, "@ai-sdk/openai-compatible");
|
|
assert.equal(configDocument.provider.omniroute.options.baseURL, "http://localhost:20128/v1");
|
|
assert.equal(configDocument.provider.omniroute.options.apiKey, "sk_test_opencode");
|
|
assert.deepEqual(Object.keys(configDocument.provider.omniroute.models), [
|
|
"cc/claude-sonnet-4-20250514",
|
|
"gg/gemini-2.5-pro",
|
|
]);
|
|
assert.equal(
|
|
configDocument.provider.omniroute.models["cc/claude-sonnet-4-20250514"].name,
|
|
"Claude Sonnet 4.5"
|
|
);
|
|
assert.equal(
|
|
configDocument.provider.omniroute.models["gg/gemini-2.5-pro"].name,
|
|
"Gemini 2.5 Pro"
|
|
);
|
|
assert.equal(configDocument.providers, undefined);
|
|
});
|
|
|
|
test("T40: OpenCode explicit multi-model selection overrides fallback defaults", () => {
|
|
const providerConfig = buildOpenCodeProviderConfig({
|
|
baseUrl: "http://localhost:20128/v1/",
|
|
apiKey: "sk_test_opencode",
|
|
models: ["custom/provider-a", "custom/provider-b"],
|
|
modelLabels: {
|
|
"custom/provider-a": "Provider A",
|
|
"custom/provider-b": "Provider B",
|
|
},
|
|
});
|
|
|
|
assert.deepEqual(Object.keys(providerConfig.models), ["custom/provider-a", "custom/provider-b"]);
|
|
assert.equal(providerConfig.models["claude-sonnet-4-5-thinking"], undefined);
|
|
assert.equal(providerConfig.models["custom/provider-a"].name, "Provider A");
|
|
assert.equal(providerConfig.models["custom/provider-b"].name, "Provider B");
|
|
});
|
|
|
|
test("T40: OpenCode merge preserves unrelated config and updates only provider.omniroute", () => {
|
|
const mergedConfig = mergeOpenCodeConfig(
|
|
{
|
|
$schema: "https://opencode.ai/config.json",
|
|
provider: {
|
|
custom: { name: "Custom Provider" },
|
|
omniroute: {
|
|
name: "Old OmniRoute",
|
|
options: { baseURL: "http://old-host/v1", apiKey: "old-key" },
|
|
},
|
|
},
|
|
mcpServers: {
|
|
github: { command: "npx", args: ["-y", "@modelcontextprotocol/server-github"] },
|
|
},
|
|
},
|
|
{
|
|
baseUrl: "http://localhost:20128/v1",
|
|
apiKey: "sk_test_opencode",
|
|
models: ["cx/gpt-5.4"],
|
|
modelLabels: { "cx/gpt-5.4": "GPT-5.4" },
|
|
}
|
|
);
|
|
|
|
assert.deepEqual(mergedConfig.provider.custom, { name: "Custom Provider" });
|
|
assert.deepEqual(mergedConfig.mcpServers, {
|
|
github: { command: "npx", args: ["-y", "@modelcontextprotocol/server-github"] },
|
|
});
|
|
assert.deepEqual(mergedConfig.provider.omniroute.models, {
|
|
"cx/gpt-5.4": { name: "GPT-5.4" },
|
|
});
|
|
});
|
|
|
|
test("T40: OpenCode tool card references theme-aware brand assets", () => {
|
|
const opencode = CLI_TOOLS.opencode;
|
|
assert.equal(opencode.image, undefined);
|
|
assert.equal(opencode.imageLight, "/providers/opencode-light.svg");
|
|
assert.equal(opencode.imageDark, "/providers/opencode-dark.svg");
|
|
});
|
|
|
|
test("T40: OpenCode light/dark provider assets are valid SVG files", async () => {
|
|
const light = await fs.readFile(
|
|
path.join(process.cwd(), "public/providers/opencode-light.svg"),
|
|
"utf-8"
|
|
);
|
|
const dark = await fs.readFile(
|
|
path.join(process.cwd(), "public/providers/opencode-dark.svg"),
|
|
"utf-8"
|
|
);
|
|
|
|
assert.match(light, /^<svg[\s>]/);
|
|
assert.match(dark, /^<svg[\s>]/);
|
|
assert.doesNotMatch(light, /<html/i);
|
|
assert.doesNotMatch(dark, /<html/i);
|
|
});
|
|
|
|
test("T40: Windsurf was removed from CLI_TOOLS in plan 14 D17 (MITM backlog plan 11)", () => {
|
|
// windsurf (Codeium) was removed from CLI_TOOLS because it has no generic custom base URL
|
|
// support. It remains as an OAuth provider in src/lib/oauth/ for authentication.
|
|
// The old guide/limitations notes are no longer needed in the UI catalog.
|
|
// Cross-reference: _tasks/features-v3.8.6/refactorpages/_orchestration/_plan11-mitm-backlog.md
|
|
assert.equal(
|
|
(CLI_TOOLS as Record<string, unknown>)["windsurf"],
|
|
undefined,
|
|
"windsurf must be removed from CLI_TOOLS per plan 14 D17"
|
|
);
|
|
});
|