Files
OmniRoute/tests/unit/registry-utils.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

235 lines
11 KiB
TypeScript

import test from "node:test";
import assert from "node:assert/strict";
// ═══════════════════════════════════════════════════════════════
// Registry Utilities Unit Tests
// Tests for parseModelFromRegistry, getAllModelsFromRegistry,
// buildAuthHeaders — shared abstractions from PR #167
// ═══════════════════════════════════════════════════════════════
const { parseModelFromRegistry, getAllModelsFromRegistry, buildAuthHeaders } =
await import("../../open-sse/config/registryUtils.ts");
// ─── Test fixtures ────────────────────────────────────────────
const MOCK_REGISTRY = {
elevenlabs: {
id: "elevenlabs",
baseUrl: "https://api.elevenlabs.io/v1",
authType: "apikey",
authHeader: "xi-api-key",
models: [
{ id: "eleven_multilingual_v2", name: "Multilingual V2" },
{ id: "eleven_turbo_v2_5", name: "Turbo V2.5" },
],
},
comfyui: {
id: "comfyui",
baseUrl: "http://localhost:8188",
authType: "none",
authHeader: "none",
models: [
{ id: "flux-dev", name: "FLUX Dev" },
{ id: "sdxl", name: "SDXL" },
],
},
nvidia: {
id: "nvidia",
baseUrl: "https://integrate.api.nvidia.com/v1",
authType: "apikey",
authHeader: "bearer",
models: [{ id: "parakeet-ctc-1.1b-asr", name: "Parakeet CTC 1.1B" }],
},
};
// ═══════════════════════════════════════════════════════════════
// parseModelFromRegistry
// ═══════════════════════════════════════════════════════════════
test("parseModelFromRegistry: returns null provider for null input", () => {
const result = parseModelFromRegistry(null, MOCK_REGISTRY);
assert.deepEqual(result, { provider: null, model: null });
});
test("parseModelFromRegistry: returns null provider for empty string", () => {
const result = parseModelFromRegistry("", MOCK_REGISTRY);
assert.deepEqual(result, { provider: null, model: null });
});
test("parseModelFromRegistry: parses provider/model prefix correctly", () => {
const result = parseModelFromRegistry("elevenlabs/eleven_multilingual_v2", MOCK_REGISTRY);
assert.deepEqual(result, {
provider: "elevenlabs",
model: "eleven_multilingual_v2",
});
});
test("parseModelFromRegistry: parses comfyui/flux-dev correctly", () => {
const result = parseModelFromRegistry("comfyui/flux-dev", MOCK_REGISTRY);
assert.deepEqual(result, { provider: "comfyui", model: "flux-dev" });
});
test("parseModelFromRegistry: finds bare model ID without provider prefix", () => {
const result = parseModelFromRegistry("sdxl", MOCK_REGISTRY);
assert.deepEqual(result, { provider: "comfyui", model: "sdxl" });
});
test("parseModelFromRegistry: finds bare model in first matching provider", () => {
const result = parseModelFromRegistry("parakeet-ctc-1.1b-asr", MOCK_REGISTRY);
assert.deepEqual(result, { provider: "nvidia", model: "parakeet-ctc-1.1b-asr" });
});
test("parseModelFromRegistry: returns null provider for unknown model", () => {
const result = parseModelFromRegistry("nonexistent-model", MOCK_REGISTRY);
assert.deepEqual(result, { provider: null, model: "nonexistent-model" });
});
test("parseModelFromRegistry: handles model ID that looks like a provider prefix but isn't", () => {
const result = parseModelFromRegistry("unknown-provider/some-model", MOCK_REGISTRY);
assert.deepEqual(result, { provider: null, model: "unknown-provider/some-model" });
});
test("parseModelFromRegistry: handles provider prefix with no matching model", () => {
// Provider exists but model doesn't — still returns the provider from prefix match
const result = parseModelFromRegistry("nvidia/nonexistent", MOCK_REGISTRY);
assert.deepEqual(result, { provider: "nvidia", model: "nonexistent" });
});
// ═══════════════════════════════════════════════════════════════
// getAllModelsFromRegistry
// ═══════════════════════════════════════════════════════════════
test("getAllModelsFromRegistry: returns all models with prefixed IDs", () => {
const models = getAllModelsFromRegistry(MOCK_REGISTRY);
// Total: elevenlabs(2) + comfyui(2) + nvidia(1) = 5
assert.equal(models.length, 5);
// Check IDs are prefixed
const ids = models.map((m) => m.id);
assert.ok(ids.includes("elevenlabs/eleven_multilingual_v2"));
assert.ok(ids.includes("comfyui/flux-dev"));
assert.ok(ids.includes("nvidia/parakeet-ctc-1.1b-asr"));
});
test("getAllModelsFromRegistry: each model has provider field", () => {
const models = getAllModelsFromRegistry(MOCK_REGISTRY);
for (const model of models) {
assert.ok(model.provider, `Model ${model.id} missing provider field`);
assert.ok(model.name, `Model ${model.id} missing name field`);
}
});
test("getAllModelsFromRegistry: extra callback adds fields per provider", () => {
const models = getAllModelsFromRegistry(MOCK_REGISTRY, (providerId, config) => ({
authType: config.authType,
}));
const elevenlabsModel = models.find((m) => m.id === "elevenlabs/eleven_multilingual_v2");
assert.equal(elevenlabsModel.authType, "apikey");
const comfyuiModel = models.find((m) => m.id === "comfyui/flux-dev");
assert.equal(comfyuiModel.authType, "none");
});
test("getAllModelsFromRegistry: returns empty array for empty registry", () => {
const models = getAllModelsFromRegistry({});
assert.deepEqual(models, []);
});
// ═══════════════════════════════════════════════════════════════
// buildAuthHeaders
// ═══════════════════════════════════════════════════════════════
test("buildAuthHeaders: returns Bearer header for bearer authHeader", () => {
const headers = buildAuthHeaders(MOCK_REGISTRY.nvidia, "my-api-key");
assert.deepEqual(headers, { Authorization: "Bearer my-api-key" });
});
test("buildAuthHeaders: returns xi-api-key header for ElevenLabs", () => {
const headers = buildAuthHeaders(MOCK_REGISTRY.elevenlabs, "eleven-key-123");
assert.deepEqual(headers, { "xi-api-key": "eleven-key-123" });
});
test("buildAuthHeaders: returns empty object for authType none", () => {
const headers = buildAuthHeaders(MOCK_REGISTRY.comfyui, "any-token");
assert.deepEqual(headers, {});
});
test("buildAuthHeaders: returns empty object for null token", () => {
const headers = buildAuthHeaders(MOCK_REGISTRY.nvidia, null);
assert.deepEqual(headers, {});
});
test("buildAuthHeaders: returns Token header for token authHeader", () => {
const provider = { ...MOCK_REGISTRY.nvidia, authHeader: "token", authType: "apikey" };
const headers = buildAuthHeaders(provider, "hf-token");
assert.deepEqual(headers, { Authorization: "Token hf-token" });
});
test("buildAuthHeaders: returns Key header for key authHeader", () => {
const provider = { ...MOCK_REGISTRY.nvidia, authHeader: "key", authType: "apikey" };
const headers = buildAuthHeaders(provider, "maritalk-key");
assert.deepEqual(headers, { Authorization: "Key maritalk-key" });
});
test("buildAuthHeaders: returns x-api-key header", () => {
const provider = { ...MOCK_REGISTRY.nvidia, authHeader: "x-api-key", authType: "apikey" };
const headers = buildAuthHeaders(provider, "custom-key");
assert.deepEqual(headers, { "x-api-key": "custom-key" });
});
test("buildAuthHeaders: returns empty object for authHeader none", () => {
const provider = { ...MOCK_REGISTRY.nvidia, authHeader: "none", authType: "apikey" };
const headers = buildAuthHeaders(provider, "some-token");
assert.deepEqual(headers, {});
});
// ═══════════════════════════════════════════════════════════════
// Integration: Video/Music/Audio registry utils
// ═══════════════════════════════════════════════════════════════
test("parseVideoModel: works via video registry", async () => {
const { parseVideoModel } = await import("../../open-sse/config/videoRegistry.ts");
const result = parseVideoModel("comfyui/animatediff");
assert.deepEqual(result, { provider: "comfyui", model: "animatediff" });
assert.deepEqual(parseVideoModel("veo-free/veo"), { provider: "veoaifree-web", model: "veo" });
});
test("parseMusicModel: works via music registry", async () => {
const { parseMusicModel } = await import("../../open-sse/config/musicRegistry.ts");
const result = parseMusicModel("comfyui/stable-audio-open");
assert.deepEqual(result, { provider: "comfyui", model: "stable-audio-open" });
});
test("getAllVideoModels: returns video models with provider prefix", async () => {
const { getAllVideoModels } = await import("../../open-sse/config/videoRegistry.ts");
const models = getAllVideoModels();
assert.ok(models.length >= 3, `Expected at least 3 video models, got ${models.length}`);
assert.ok(models.some((m) => m.id === "kie/kling-3.0/video"));
assert.ok(models.some((m) => m.id === "kie/sora-2-pro-image-to-video"));
assert.ok(models.some((m) => m.id === "comfyui/animatediff"));
assert.ok(models.some((m) => m.id === "runwayml/gen4.5"));
assert.ok(models.some((m) => m.id === "veoaifree-web/veo"));
assert.ok(models.some((m) => m.id === "veo-free/veo"));
});
test("getAllMusicModels: returns music models with provider prefix", async () => {
const { getAllMusicModels } = await import("../../open-sse/config/musicRegistry.ts");
const models = getAllMusicModels();
assert.ok(models.length >= 2, `Expected at least 2 music models, got ${models.length}`);
assert.equal(models.find((m) => m.id === "kie/suno-v4.0")?.name, "Suno V4.0");
assert.ok(models.some((m) => m.id === "comfyui/stable-audio-open"));
});
test("getAllAudioModels: returns nested transcription models with provider prefix", async () => {
const { getAllAudioModels } = await import("../../open-sse/config/audioRegistry.ts");
const models = getAllAudioModels();
const nvidiaWhisper = models.find((m) => m.id === "nvidia/openai/whisper-large-v3");
assert.equal(nvidiaWhisper?.name, "Whisper Large v3 (NVIDIA)");
assert.equal(nvidiaWhisper?.provider, "nvidia");
assert.equal(nvidiaWhisper?.subtype, "transcription");
});