Files
OmniRoute/tests/unit/ui/model-select-modal-hidden-models-7156.test.tsx
Diego Rodrigues de Sa e Souza e7f6b1d130 feat(radar): flag-gated signed free-model catalog overlay (#9515)
* feat(dashboard): add RADAR_ENABLED flag (default off)

* feat(db): radar feed cache + settings with encrypted supporter key

* feat(radar): signed feed sync with pinned key and version floor

- feedSchema.ts: Zod v4 schema mirroring the server feed format
  (discriminated union on budget.kind, enum constraints, etc.)
- pinnedKeys.ts: Ed25519 SPKI-DER pinned key + env override for forks
- verify.ts: signature verification over exact wire bytes, never throws
- sync.ts: full download/verify/validate/cache pipeline with injectable
  deps, feature-flag gate, opt-in gate, version floor (numeric compare),
  and sanitized error reasons (no stack traces)
- 40 tests covering: contract hash, key handling, sig verification,
  schema validation, version compare, all sync paths (disabled, opt_out,
  invalid_signature, invalid_schema, stale, updated, error), auth header
  injection, and cache-untouched assertions for every failure mode

* feat(radar): read-time overlay merge rules over the free catalog

Pure function applyFeed() merges the cached Radar feed over the static
baseline catalog at read time, honoring 4 rules:

1. Feed never overwrites a local override field.
2. enabled:false disables the entry with disabledBy:"radar" provenance.
3. User-added entry NOT in the feed survives untouched.
4. User deletion tombstone prevents feed resurrection.

getRadarCatalog() accessor in index.ts: flag off / no cache / corrupt
payload all fall back to baseline. Valid cache applies the overlay and
returns feed metadata (version, tier, fetchedAt).

TDD: 19 tests (4 rules + dedup + origin + accessor flag/cache/corrupt/
valid/bad-feed + baselineToMergedEntries converter).

* feat(dashboard): radar catalog and guided setup screens

- API routes: GET /api/radar/catalog, POST /api/radar/sync, POST /api/radar/settings
  - All gated on RADAR_ENABLED flag (404 when off)
  - Error responses via buildErrorBody(), never raw stack/message
  - Settings never echoes clear supporter key (masked omr_****<last4>)
  - Sync delegates to syncRadar() server-side, never proxies feed URL
- Dashboard pages:
  - /dashboard/radar: 4 states (flag off, opt-in pending, empty, populated)
  - /dashboard/radar/setup?provider=X: guided setup with steps, key URL, test connection
  - Uses existing Card component and next-intl patterns
- Sidebar: radar entry in costs group with icon
- i18n: pt-BR and en keys for radarPage and radarSetupPage namespaces
- Tests:
  - radar-api-routes.test.ts: 11 tests (flag-off 404, flag-on shape, error sanitization)
  - radar-page-state.test.ts: 5 tests (pure state logic)
  - All 90 radar tests pass (including prior 74)

* docs(radar): module doc and flag-off inertia test

Add docs/frameworks/RADAR.md covering the flag gate, the separate data-sync
opt-in and privacy promise, the Ed25519 signature/pinned-key security model,
tiers, the read-time overlay merge rules, and the self-hosting env vars —
plus index entries in CLAUDE.md/AGENTS.md/docs/README.md/REPOSITORY_MAP.md.

Document RADAR_FEED_URL and RADAR_FEED_PUBKEY in .env.example and
docs/reference/ENVIRONMENT.md to satisfy check:env-doc-sync, which was
failing on this branch since the sync.ts commit added the reads.

Add tests/unit/radar-inertia.test.ts as the single canonical place asserting
the "RADAR_ENABLED off => zero behavioral delta" claim end to end: the three
/api/radar/* routes 404, the flag resolves to the definition default with no
override, getRadarCatalog() returns exactly the baseline without touching the
cache, and computeFreeModelTotals() keeps its pinned values with the Radar
module imported alongside it.

* fix(db): renumber radar migration to 135 after collision with 134

The base branch introduced 134_proxy_logs_egress_ip while this branch carried
134_radar_cache_settings; the migration runner rejects duplicate numeric prefixes.
This migration has never been applied to a real database (the PR is unmerged), so
no retroactive isSchemaAlreadyApplied guard is needed.

* i18n(radar): translate radar catalog and setup strings to all locales

The UI-coverage ratchet measures (present - placeholder) / total_en, so the
__MISSING__ sentinels that i18n:sync-ui writes do not count as covered — only
real translations restore the metric. Scoped to this PR's namespaces
(radarPage, radarSetupPage, sidebar.radar*) instead of a bulk sync, which would
have pulled ~978 unrelated pending keys into this diff.

Placeholders and code identifiers verified preserved across all 1682 strings.

* fix(radar): trust the served-tier header instead of the signed body field

The signed feed body always carries tier:"live" by design (one signed
artifact per version — rewriting the field server-side per request
would break the exact-bytes Ed25519 signature). The server now returns
the tier ACTUALLY served via the x-omniroute-feed-tier response
header, so free users on a delayed community snapshot no longer see
"Ao vivo (tempo real)" in the UI.

sync.ts now reads and validates that header (falling back to the
body's tier only when the header is absent or holds an unrecognized
value) and stores the served tier in the cache; index.ts already
surfaces cache.tier to the UI unchanged.

* test(combo): shorten an assert message that exceeded the line limit

The assertion added by #9507 was 104 chars, so prettier reformatted it into
five lines on the next commit that touched the file, pushing it past its
frozen size (3449) and failing check:file-size. The message is shortened
(the issue reference stays in the comment directly above); the assertion
itself is unchanged, and the file is back to 3448 lines and prettier-clean.

* i18n(radar): use the canonical zh-TW glossary terms

The machine translation produced retired renderings the glossary gate blocks:
供應商 for provider (canonical 提供者) and 文檔 for documentation (canonical 文件).
Fixed across the 11 affected radar strings; tests/unit/i18n-glossary-consistency-check.test.ts
is back to 17/17.

* fix(radar): point the default feed URL at the domain that exists

radar.omniroute.dev was a placeholder for a domain that was never registered,
so an out-of-the-box sync would fail DNS resolution for every user. The live
feed is served from radar.omniroute.online (the subdomain the design always
specified), now behind Cloudflare TLS. Forks still override it via
RADAR_FEED_URL.

---------

Co-authored-by: diegosouzapw <diegosouzapw@users.noreply.github.com>
2026-08-06 05:58:58 -03:00

278 lines
9.5 KiB
TypeScript

// @vitest-environment jsdom
import React, { act } from "react";
import { createRoot } from "react-dom/client";
import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
import ModelSelectModal from "@/shared/components/ModelSelectModal";
vi.mock("next-intl", () => ({
useTranslations: () => (key: string) => key,
}));
const roots: Array<{ root: ReturnType<typeof createRoot>; el: HTMLDivElement }> = [];
async function render(
props: React.ComponentProps<typeof ModelSelectModal>
): Promise<HTMLDivElement> {
const el = document.createElement("div");
document.body.appendChild(el);
const root = createRoot(el);
await act(async () => {
root.render(<ModelSelectModal {...props} />);
});
roots.push({ root, el });
return el;
}
// Configurable fake backend shared across tests. Each test overrides the
// `models` payload (custom rows per provider), `hiddenModelsByProvider` (the
// unified map), `providerNodes`, and the live-fetch response used for the
// `/api/providers/:id/models` route.
let mockModels: Record<string, any[]>;
let mockHidden: Record<string, string[]>;
let mockNodes: any[];
let mockLiveModels: any[];
let liveFetchUrls: string[];
beforeEach(() => {
(globalThis as any).IS_REACT_ACT_ENVIRONMENT = true;
mockModels = {};
mockHidden = {};
mockNodes = [];
mockLiveModels = [];
liveFetchUrls = [];
vi.stubGlobal(
"fetch",
vi.fn(async (input: RequestInfo | URL) => {
const url = String(input);
if (url.includes("/api/combos"))
return new Response(JSON.stringify({ combos: [] }), { status: 200 });
if (url.includes("/api/provider-nodes"))
return new Response(JSON.stringify({ nodes: mockNodes }), { status: 200 });
if (url.includes("/api/provider-models")) {
return new Response(
JSON.stringify({
models: mockModels,
modelCompatOverrides: [],
hiddenModelsByProvider: mockHidden,
}),
{ status: 200 }
);
}
if (url.includes("/api/providers/") && url.includes("/models")) {
liveFetchUrls.push(url);
return new Response(JSON.stringify({ models: mockLiveModels }), { status: 200 });
}
return new Response(JSON.stringify({}), { status: 200 });
})
);
});
afterEach(() => {
for (const { root, el } of roots.splice(0)) {
act(() => root.unmount());
el.remove();
}
vi.unstubAllGlobals();
vi.clearAllMocks();
});
async function flush() {
await act(async () => {
await new Promise((r) => setTimeout(r, 0));
});
await act(async () => {
await new Promise((r) => setTimeout(r, 0));
});
}
describe("ModelSelectModal hidden-model filtering (#7156)", () => {
it("does not list a custom model explicitly flagged isHidden:true", async () => {
mockModels = {
requesty: [
{ id: "visible-model-1", name: "Visible Model", source: "imported" },
{ id: "hidden-model-1", name: "Hidden Model", source: "imported", isHidden: true },
],
};
const el = await render({
isOpen: true,
onClose: vi.fn(),
onSelect: vi.fn(),
activeProviders: [{ provider: "requesty", id: "conn-1" }],
modelAliases: {},
title: "Add model to combo",
});
await flush();
expect(el.textContent).toContain("Visible Model");
expect(el.textContent).not.toContain("Hidden Model");
});
});
describe("ModelSelectModal unified hidden-model filtering (#9203)", () => {
it("hides a system catalog model flagged in the unified hidden map", async () => {
mockHidden = { claude: ["claude-sonnet-4-6"] };
const el = await render({
isOpen: true,
onClose: vi.fn(),
onSelect: vi.fn(),
activeProviders: [{ provider: "claude", id: "claude-conn" }],
modelAliases: {},
title: "Add model to combo",
});
await flush();
// claude is a system-catalog provider — its entries come from the bundled catalog.
expect(el.textContent).toContain("Claude Opus 4.7");
expect(el.textContent).not.toContain("Claude Sonnet 4.6");
});
it("hides a hidden passthrough alias model", async () => {
mockHidden = { requesty: ["alias-hidden"] };
const el = await render({
isOpen: true,
onClose: vi.fn(),
onSelect: vi.fn(),
activeProviders: [{ provider: "requesty", id: "conn-1" }],
modelAliases: {
"Visible Alias": "requesty/alias-visible",
"Hidden Alias": "requesty/alias-hidden",
},
title: "Add model to combo",
});
await flush();
expect(el.textContent).toContain("Visible Alias");
expect(el.textContent).not.toContain("Hidden Alias");
});
it("hides a hidden node alias model for a custom provider", async () => {
mockNodes = [{ id: "openai-compatible-demo", name: "Demo Node", prefix: "demo-prefix" }];
mockHidden = { "openai-compatible-demo": ["node-hidden"] };
const el = await render({
isOpen: true,
onClose: vi.fn(),
onSelect: vi.fn(),
activeProviders: [{ provider: "openai-compatible-demo", id: "conn-demo" }],
modelAliases: {
"Node Visible": "openai-compatible-demo/node-visible",
"Node Hidden": "openai-compatible-demo/node-hidden",
},
title: "Add model to combo",
});
await flush();
expect(el.textContent).toContain("Node Visible");
expect(el.textContent).not.toContain("Node Hidden");
});
it("hides a custom model referenced only in the unified hidden map (no isHidden flag)", async () => {
mockModels = {
"openai-compatible-demo": [
{ id: "custom-visible", name: "Custom Visible", source: "manual" },
{ id: "custom-map-hidden", name: "Custom Map Hidden", source: "manual" },
],
};
mockHidden = { "openai-compatible-demo": ["custom-map-hidden"] };
const el = await render({
isOpen: true,
onClose: vi.fn(),
onSelect: vi.fn(),
activeProviders: [{ provider: "openai-compatible-demo", id: "conn-demo" }],
modelAliases: {},
title: "Add model to combo",
});
await flush();
expect(el.textContent).toContain("Custom Visible");
expect(el.textContent).not.toContain("Custom Map Hidden");
});
it("hides a hidden auto-fetched model and requests excludeHidden=true on the live route", async () => {
mockLiveModels = [
{ id: "live-visible", name: "Live Visible" },
{ id: "live-hidden", name: "Live Hidden" },
];
mockHidden = { "openai-compatible-demo": ["live-hidden"] };
const el = await render({
isOpen: true,
onClose: vi.fn(),
onSelect: vi.fn(),
activeProviders: [{ provider: "openai-compatible-demo", id: "conn-demo" }],
modelAliases: {},
title: "Add model to combo",
});
await flush();
expect(el.textContent).toContain("Live Visible");
expect(el.textContent).not.toContain("Live Hidden");
// The live provider-models route must receive excludeHidden=true.
expect(liveFetchUrls.some((u) => u.includes("excludeHidden=true"))).toBe(true);
});
it("keeps models visible when the API omits hiddenModelsByProvider", async () => {
// mockHidden is empty object — simulates an older/unchanged API response.
mockModels = { requesty: [{ id: "m1", name: "Model One", source: "imported" }] };
const el = await render({
isOpen: true,
onClose: vi.fn(),
onSelect: vi.fn(),
activeProviders: [{ provider: "requesty", id: "conn-1" }],
modelAliases: {},
title: "Add model to combo",
});
await flush();
expect(el.textContent).toContain("Model One");
});
it("keeps models visible when hiddenModelsByProvider is malformed", async () => {
// Force a malformed payload via a per-test fetch override.
vi.stubGlobal(
"fetch",
vi.fn(async (input: RequestInfo | URL) => {
const url = String(input);
if (url.includes("/api/combos"))
return new Response(JSON.stringify({ combos: [] }), { status: 200 });
if (url.includes("/api/provider-nodes"))
return new Response(JSON.stringify({ nodes: [] }), { status: 200 });
if (url.includes("/api/provider-models")) {
return new Response(
JSON.stringify({
models: { requesty: [{ id: "m1", name: "Model One", source: "imported" }] },
modelCompatOverrides: [],
hiddenModelsByProvider: { requesty: "not-an-array" },
}),
{ status: 200 }
);
}
return new Response(JSON.stringify({}), { status: 200 });
})
);
const el = await render({
isOpen: true,
onClose: vi.fn(),
onSelect: vi.fn(),
activeProviders: [{ provider: "requesty", id: "conn-1" }],
modelAliases: {},
title: "Add model to combo",
});
await flush();
expect(el.textContent).toContain("Model One");
});
it("scopes hidden models per provider (same id visible under another provider)", async () => {
mockModels = {
requesty: [{ id: "shared-model", name: "Requesty Shared", source: "imported" }],
"openai-compatible-demo": [{ id: "shared-model", name: "Demo Shared", source: "manual" }],
};
mockHidden = { requesty: ["shared-model"] };
const el = await render({
isOpen: true,
onClose: vi.fn(),
onSelect: vi.fn(),
activeProviders: [
{ provider: "requesty", id: "conn-1" },
{ provider: "openai-compatible-demo", id: "conn-demo" },
],
modelAliases: {},
title: "Add model to combo",
});
await flush();
expect(el.textContent).not.toContain("Requesty Shared");
expect(el.textContent).toContain("Demo Shared");
});
});