mirror of
https://github.com/diegosouzapw/OmniRoute.git
synced 2026-09-22 06:42:19 +03:00
* 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>
278 lines
9.5 KiB
TypeScript
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");
|
|
});
|
|
});
|