Files
OmniRoute/tests/unit/proxy-registry.test.ts
Diego Rodrigues de Sa e Souza 7db430a352 Release v3.8.14 (#3340)
* 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>
2026-06-07 07:20:02 -03:00

482 lines
16 KiB
TypeScript

import test from "node:test";
import assert from "node:assert/strict";
import fs from "node:fs";
import os from "node:os";
import path from "node:path";
const TEST_DATA_DIR = fs.mkdtempSync(path.join(os.tmpdir(), "omniroute-proxy-registry-"));
process.env.DATA_DIR = TEST_DATA_DIR;
process.env.API_KEY_SECRET = "test-secret";
const core = await import("../../src/lib/db/core.ts");
const providersDb = await import("../../src/lib/db/providers.ts");
const proxiesDb = await import("../../src/lib/db/proxies.ts");
const settingsDb = await import("../../src/lib/db/settings.ts");
const apiKeysDb = await import("../../src/lib/db/apiKeys.ts");
const proxiesRoute = await import("../../src/app/api/settings/proxies/route.ts");
async function resetStorage() {
delete process.env.INITIAL_PASSWORD;
core.resetDbInstance();
apiKeysDb.resetApiKeyState();
fs.rmSync(TEST_DATA_DIR, { recursive: true, force: true });
fs.mkdirSync(TEST_DATA_DIR, { recursive: true });
}
test.after(async () => {
core.resetDbInstance();
fs.rmSync(TEST_DATA_DIR, { recursive: true, force: true });
});
test("proxy registry blocks delete when proxy is still assigned", async () => {
await resetStorage();
const created = await proxiesDb.createProxy({
name: "Delete Safety Proxy",
type: "http",
host: "127.0.0.1",
port: 8080,
});
assert.ok(created?.id);
await proxiesDb.assignProxyToScope("provider", "openai", created.id);
await assert.rejects(
async () => proxiesDb.deleteProxyById(created.id),
(error) => {
assert.equal((error as any).status, 409);
(assert as any).equal((error as any).code, "proxy_in_use");
return true;
}
);
});
test("createProxyAndAssign rolls back the registry row when assignment fails", async () => {
await resetStorage();
await assert.rejects(
async () =>
proxiesDb.createProxyAndAssign(
{
name: "Rollback Proxy",
type: "http",
host: "rollback.local",
port: 8080,
},
{ scope: "provider", scopeId: null }
),
/scopeId is required/i
);
const proxies = await proxiesDb.listProxies({ includeSecrets: true });
assert.equal(
proxies.some((proxy: any) => proxy.name === "Rollback Proxy"),
false
);
});
test("createProxyAndAssign assigns and clears matching legacy proxy atomically", async () => {
await resetStorage();
await settingsDb.setProxyForLevel("provider", "openai", {
type: "http",
host: "legacy-openai.local",
port: 8080,
});
const result = await proxiesDb.createProxyAndAssign(
{
name: "Atomic Provider Proxy",
type: "https",
host: "atomic-openai.local",
port: 443,
source: "dashboard-custom",
},
{ scope: "provider", scopeId: "openai" }
);
assert.ok(result.proxy?.id);
assert.equal(result.assignment?.proxyId, result.proxy?.id);
assert.equal(result.assignment?.scope, "provider");
assert.equal(result.assignment?.scopeId, "openai");
assert.equal(await settingsDb.getProxyForLevel("provider", "openai"), null);
});
test("updateProxyAndAssign clears stored credentials when blanks are explicitly provided", async () => {
await resetStorage();
const created = await proxiesDb.createProxy({
name: "Atomic Credential Proxy",
type: "http",
host: "atomic-credentials.local",
port: 8080,
username: "user-a",
password: "pass-a",
source: "dashboard-custom",
});
const result = await proxiesDb.updateProxyAndAssign(
created.id,
{
username: "",
password: "",
},
{ scope: "provider", scopeId: "openai" }
);
const withSecrets = await proxiesDb.getProxyById(created.id, { includeSecrets: true });
assert.equal(result?.assignment?.scope, "provider");
assert.equal(result?.assignment?.scopeId, "openai");
assert.equal(withSecrets?.username, "");
assert.equal(withSecrets?.password, "");
});
test("specific registry account assignment takes precedence over legacy key proxy config", async () => {
await resetStorage();
const conn = await providersDb.createProviderConnection({
provider: "openai",
authType: "apikey",
name: "registry-precedence",
apiKey: "sk-test",
});
await settingsDb.setProxyForLevel("key", (conn as any).id, {
type: "http",
host: "legacy-key.local",
port: 8080,
});
const providerProxy = await proxiesDb.createProxy({
name: "Provider Proxy",
type: "https",
host: "provider.local",
port: 443,
});
const accountProxy = await proxiesDb.createProxy({
name: "Account Proxy",
type: "http",
host: "account.local",
port: 8081,
});
await proxiesDb.assignProxyToScope("provider", "openai", providerProxy.id);
await proxiesDb.assignProxyToScope("account", (conn as any).id, accountProxy.id);
const resolved = await settingsDb.resolveProxyForConnection((conn as any).id);
assert.equal((resolved as any).level, "account");
assert.equal((resolved as any).source, "registry");
assert.equal((resolved as any).proxy.host, "account.local");
});
test("legacy proxy config migration imports global/provider/key assignments", async () => {
await resetStorage();
const conn = await providersDb.createProviderConnection({
provider: "openai",
authType: "apikey",
name: "legacy-import",
apiKey: "sk-test-legacy",
});
await settingsDb.setProxyForLevel("global", null, {
type: "http",
host: "global.local",
port: 8080,
});
await settingsDb.setProxyForLevel("provider", "openai", {
type: "https",
host: "provider-legacy.local",
port: 443,
});
await settingsDb.setProxyForLevel("key", (conn as any).id, {
type: "http",
host: "account-legacy.local",
port: 8082,
});
const result = await proxiesDb.migrateLegacyProxyConfigToRegistry();
assert.equal(result.skipped, false);
assert.equal(result.migrated >= 3, true);
const resolved = await settingsDb.resolveProxyForConnection((conn as any).id);
assert.equal((resolved as any).level, "account");
assert.equal((resolved as any).source, "registry");
assert.equal((resolved as any).proxy.host, "account-legacy.local");
});
// #2456: resolveProxyForProvider (used by the OAuth token exchange + token refresh,
// before any connection exists) only consulted the proxy registry. A proxy set the
// legacy way (/api/settings/proxy?level=provider) was ignored, so on a VPS the OAuth
// exchange went out direct and tripped Anthropic's IP rate limit. It must fall back to
// the legacy per-provider config, mirroring resolveProxyForConnection.
test("resolveProxyForProvider falls back to the legacy provider proxy config (#2456)", async () => {
await resetStorage();
await settingsDb.setProxyForLevel("provider", "claude", {
type: "http",
host: "legacy-claude-proxy.local",
port: 3128,
});
// No proxy_registry assignment exists for "claude" — only the legacy config.
const resolved = await proxiesDb.resolveProxyForProvider("claude");
assert.ok(resolved, "expected the legacy provider proxy to be resolved");
assert.equal((resolved as any).host, "legacy-claude-proxy.local");
assert.equal((resolved as any).type, "http");
});
test("resolveProxyForProvider falls back to the legacy global proxy when no provider proxy (#2456)", async () => {
await resetStorage();
await settingsDb.setProxyForLevel("global", null, {
type: "socks5",
host: "legacy-global.local",
port: 1080,
});
const resolved = await proxiesDb.resolveProxyForProvider("anthropic");
assert.ok(resolved, "expected the legacy global proxy to be resolved");
assert.equal((resolved as any).host, "legacy-global.local");
});
test("resolveProxyForProvider still prefers a registry assignment over legacy config (#2456)", async () => {
await resetStorage();
await settingsDb.setProxyForLevel("provider", "openai", {
type: "http",
host: "legacy-openai.local",
port: 8080,
});
const registryProxy = await proxiesDb.createProxy({
name: "Registry OpenAI",
type: "https",
host: "registry-openai.local",
port: 443,
});
await proxiesDb.assignProxyToScope("provider", "openai", registryProxy.id);
const resolved = await proxiesDb.resolveProxyForProvider("openai");
assert.ok(resolved);
assert.equal((resolved as any).host, "registry-openai.local", "registry assignment must win");
});
test("resolveProxyForProvider prefers legacy provider proxy over registry global fallback (#2601)", async () => {
await resetStorage();
await settingsDb.setProxyForLevel("provider", "claude", {
type: "http",
host: "legacy-claude-provider.local",
port: 3128,
});
const globalProxy = await proxiesDb.createProxy({
name: "Registry Global",
type: "https",
host: "registry-global.local",
port: 443,
});
await proxiesDb.assignProxyToScope("global", null, globalProxy.id);
const resolved = await proxiesDb.resolveProxyForProvider("claude");
assert.ok(resolved);
assert.equal(
(resolved as any).host,
"legacy-claude-provider.local",
"provider-specific custom proxy must beat global registry fallback"
);
});
test("resolveProxyForProvider returns null when neither registry nor legacy config has a proxy (#2456)", async () => {
await resetStorage();
const resolved = await proxiesDb.resolveProxyForProvider("gemini");
assert.equal(resolved, null);
});
test("resolveProxyForConnection uses apiKey proxy before account-level proxy", async () => {
await resetStorage();
const conn = await providersDb.createProviderConnection({
provider: "openai",
authType: "apikey",
name: "api-key-proxy",
apiKey: "sk-apikey-proxy",
});
const accountProxy = await proxiesDb.createProxy({
name: "Account Proxy",
type: "http",
host: "account.local",
port: 8081,
});
await proxiesDb.assignProxyToScope("account", (conn as any).id, accountProxy.id);
const key = await apiKeysDb.createApiKey("proxy-test-key", "machine-p1");
// Enable per-key proxy globally so the API key's proxy_id is honored
core
.getDbInstance()
.prepare(
"INSERT OR REPLACE INTO key_value (namespace, key, value) VALUES ('settings', 'perKeyProxyEnabled', 'true')"
)
.run();
const apiKeyProxy = await proxiesDb.createProxy({
name: "API Key Proxy",
type: "https",
host: "apikey.local",
port: 8443,
});
await apiKeysDb.updateApiKeyPermissions(key.id, { proxyId: apiKeyProxy.id });
const resolved = await settingsDb.resolveProxyForConnection((conn as any).id, key.id);
assert.ok(resolved);
assert.equal((resolved as any).level, "apiKey");
assert.equal((resolved as any).proxy.host, "apikey.local");
assert.equal((resolved as any).proxy.port, 8443);
});
test("resolveProxyForConnection falls through when apiKey has no proxy_id", async () => {
await resetStorage();
const conn = await providersDb.createProviderConnection({
provider: "openai",
authType: "apikey",
name: "fallthrough-test",
apiKey: "sk-fallthrough",
});
const accountProxy = await proxiesDb.createProxy({
name: "Account Proxy",
type: "http",
host: "account-fallthrough.local",
port: 8081,
});
await proxiesDb.assignProxyToScope("account", (conn as any).id, accountProxy.id);
const key = await apiKeysDb.createApiKey("fallthrough-key", "machine-ft");
const resolved = await settingsDb.resolveProxyForConnection((conn as any).id, key.id);
assert.ok(resolved);
assert.equal((resolved as any).level, "account");
assert.equal((resolved as any).proxy.host, "account-fallthrough.local");
});
test("connection proxy toggle gates account assignments and invalidates cached resolutions", async () => {
await resetStorage();
const directConnection = await providersDb.createProviderConnection({
provider: "proxy-toggle-test-provider",
authType: "apikey",
name: "Direct Account",
apiKey: "sk-direct-account",
});
const proxiedConnection = await providersDb.createProviderConnection({
provider: "proxy-toggle-test-provider",
authType: "apikey",
name: "Proxied Account",
apiKey: "sk-proxied-account",
});
const poolProxy = await proxiesDb.createProxy({
name: "Pool Proxy",
type: "http",
host: "pool-proxy.local",
port: 8080,
});
await proxiesDb.assignProxyToScope("account", (proxiedConnection as any).id, poolProxy.id);
const directResolved = await settingsDb.resolveProxyForConnection((directConnection as any).id);
assert.equal(directResolved.level, "direct");
assert.equal(directResolved.proxy, null);
const proxiedResolved = await settingsDb.resolveProxyForConnection((proxiedConnection as any).id);
assert.equal(proxiedResolved.level, "account");
assert.equal((proxiedResolved.proxy as any).host, "pool-proxy.local");
const disabled = await providersDb.updateProviderConnection((proxiedConnection as any).id, {
proxyEnabled: false,
});
assert.equal((disabled as any).proxyEnabled, false);
const disabledResolved = await settingsDb.resolveProxyForConnection(
(proxiedConnection as any).id
);
assert.equal(disabledResolved.level, "direct");
assert.equal(disabledResolved.proxy, null);
const enabled = await providersDb.updateProviderConnection((proxiedConnection as any).id, {
proxyEnabled: true,
});
assert.equal((enabled as any).proxyEnabled, true);
const enabledResolved = await settingsDb.resolveProxyForConnection((proxiedConnection as any).id);
assert.equal(enabledResolved.level, "account");
assert.equal((enabledResolved.proxy as any).host, "pool-proxy.local");
});
test("provider connection proxy toggle fields round-trip as booleans", async () => {
await resetStorage();
const connection = await providersDb.createProviderConnection({
provider: "openai",
authType: "apikey",
name: "Boolean Toggle Account",
apiKey: "sk-toggle-roundtrip",
});
const updated = await providersDb.updateProviderConnection((connection as any).id, {
proxyEnabled: false,
perKeyProxyEnabled: true,
});
const fetched = await providersDb.getProviderConnectionById((connection as any).id);
assert.equal((updated as any).proxyEnabled, false);
assert.equal((updated as any).perKeyProxyEnabled, true);
assert.equal((fetched as any).proxyEnabled, false);
assert.equal((fetched as any).perKeyProxyEnabled, true);
});
test("createProxyRegistrySchema accepts type:vercel and source:vercel-relay (schema gap-06)", async () => {
// Note: We validate the schema directly using the worktree's absolute path because
// tests run with CWD=/OmniRoute, so `@/` aliases resolve to the main branch's src/.
// The assertion below confirms the worktree's schema accepts the new enum values.
const { createProxyRegistrySchema } = await import("../../src/shared/validation/schemas.ts");
const result = createProxyRegistrySchema.safeParse({
name: "Vercel Relay Test",
type: "vercel",
host: "omniroute-relay-abc123.vercel.app",
port: 443,
source: "vercel-relay",
notes: JSON.stringify({ relayAuth: "secret-relay-token" }),
});
assert.ok(
result.success,
`schema should accept type:vercel — errors: ${JSON.stringify("error" in result ? result.error : null)}`
);
if (result.success) {
assert.equal(result.data.type, "vercel");
assert.equal(result.data.source, "vercel-relay");
}
});
test("createProxy persists type:vercel and source:vercel-relay to DB (schema gap-06)", async () => {
await resetStorage();
const created = await proxiesDb.createProxy({
name: "Vercel Relay DB Test",
type: "vercel",
host: "omniroute-relay-xyz.vercel.app",
port: 443,
source: "vercel-relay",
notes: JSON.stringify({ relayAuth: "my-token" }),
});
assert.ok(created?.id, "created proxy should have an id");
assert.equal(created?.type, "vercel");
assert.equal(created?.source, "vercel-relay");
});