pjandro/OmniRoute
1
0
Fork 0
mirror of https://github.com/diegosouzapw/OmniRoute.git synced 2026-09-17 20:32:25 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix(quality): correct golden inventory drift, fix host-resolution test typo, rebaseline for #11754

Browse Source
This commit is contained in:
Markus Hartung
2026-08-28 06:48:16 -03:00
parent 422047a67f
commit 8004573dcf
3 changed files with 24 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
config/quality/file-size-baseline.json
View File

@@ -478,8 +478,9 @@
"_rebaseline_2026_08_24_11355_cooldown_recovery_guards": "PR #11355 own growth: test/route.ts 1215->1237, +22 (startup crash-recovery guard: clearStaleCrashCooldowns() now parses the persisted rate_limited_until deadline and skips clearing rows still genuinely in the future, instead of clearing every non-terminal cooldown unconditionally). Cohesive fix at the existing test-route dispatch chokepoint alongside the #11141 probe builder. Covered by tests/unit/startup-stale-cooldown-recovery.test.ts + tests/unit/repro-zai-cooldown-cleared-by-connection-test.test.ts.",
"src/lib/guardrails/videoBridgeRuntime.ts": 1009,
"_rebaseline_2026_08_24_video_bridge_fu02_fu07_sampler": "PRs #11344 (FU-02 one-frame scene-aware determinism) + #11381 (FU-07 opt-in segment_aware structural sampling) own growth: videoBridgeRuntime.ts <1000->1009, +9 (sum of both boarded together in the same merge-batch). #11344 adds the deterministic one-frame midpoint fallback + policyEffective=uniform report at the existing scene_aware seam; #11381 adds the bounded local-only FFmpeg structural pre-analysis pass (scene/freeze/blur/exposure/SI-TI) and its budget-reallocation logic. Covered by tests/unit/guardrails/videoBridgeSampler.test.ts, tests/unit/guardrails/videoBridgeFu07StructuralSampling.test.ts, tests/integration/video-bridge-sampler-ffmpeg.test.ts. Owner pre-authorized rebaseline for legitimate PR growth (2026-08-19 directive).",
"open-sse/services/autoCombo/virtualFactory.ts": 1135,
"open-sse/services/autoCombo/virtualFactory.ts": 1138,
"_rebaseline_2026_08_28_mergebatch_v3851_qwen_retirement": "/merge-batch 2026-08-28 (v3.8.51): #11713 (Qwen Web retirement) own growth: open-sse/services/autoCombo/virtualFactory.ts 1132->1135 (+3, combining the Designer + Runtime retirement-guard filter into the single runtimeConnections predicate at the existing candidate-pool chokepoint, now excluding Qwen Web alongside Felo Web). Irreducible per-mechanism wiring, additive not a new branch. Covered by tests/unit/virtual-auto-combo.test.ts.",
"_rebaseline_2026_08_28_mergebatch_v3851_chatgptweb_retirement": "/merge-batch 2026-08-28 (v3.8.51): #11754 (common ChatGPT Web retirement) own growth: open-sse/services/autoCombo/virtualFactory.ts 1135->1138 (+3, an early `available` connection filter for the retired chatgpt-web/cgpt-web ids applied to both the active and disabled-noauth connection lists, ahead of the existing Designer+Runtime runtimeConnections filter). Irreducible per-mechanism wiring, additive not a new branch. Covered by tests/unit/virtual-auto-combo.test.ts.",
"src/lib/cloudflaredTunnel.ts": 1078,
"src/shared/components/RequestLoggerDetail.tsx": 1018
},

24
tests/unit/hard-session-lease-bypass-inventory.test.ts
View File

@@ -26,7 +26,10 @@ const EXPECTED: Record<InventoryKind, Record<string, number>> = {
"src/app/api/v1/audio/transcriptions/route.ts": 2,
"src/app/api/v1/audio/translations/route.ts": 1,
"src/app/api/v1/classify/route.ts": 1,
"src/app/api/v1/images/edits/route.ts": 6,
// v3.8.51 #11754: the second resolveImageRouteModel() call (a duplicate
// of the retirement-check one hoisted before enforceApiKeyPolicy) was
// removed as dead redundant code, 6->5.
"src/app/api/v1/images/edits/route.ts": 5,
"src/app/api/v1/images/generations/route.ts": 3,
"src/app/api/v1/images/upscale/route.ts": 1,
"src/app/api/v1/messages/count_tokens/route.ts": 1,
@@ -57,8 +60,12 @@ const EXPECTED: Record<InventoryKind, Record<string, number>> = {
"open-sse/handlers/chatCore.ts": 3,
"open-sse/handlers/chatCore/cliproxyModelMapping.ts": 1,
"open-sse/handlers/chatCore/cliproxyapiCredentials.ts": 1,
"open-sse/handlers/imageGeneration.ts": 1,
"open-sse/handlers/imageGeneration/providers/geminiWeb.ts": 1,
// v3.8.51 #11754: the retired common ChatGPT Web's synthetic
// image-edit-continuation ChatGptWebExecutor.execute() call (the sole
// executor.execute() site in this file) was removed with the provider;
// no executor site remains here.
// Gemini Web's own image handler+file (open-sse/handlers/imageGeneration/providers/geminiWeb.ts)
// was already retired by #11708 (its .execute() site removed then too).
"open-sse/handlers/videoGeneration.ts": 1,
"open-sse/services/compression/eval/executorModelClient.ts": 1,
"src/lib/compression/judgeModelClient.ts": 1,
@@ -96,7 +103,10 @@ const EXPECTED: Record<InventoryKind, Record<string, number>> = {
"src/app/api/providers/client/route.ts": 1,
"src/app/api/providers/free-onboarding/route.ts": 2,
"src/app/api/providers/import/route.ts": 1,
"src/app/api/providers/route.ts": 2,
// Base drift (already present before #11754 boarded, from earlier-merged
// #11698/#11720 retirement PRs): a third getProviderConnections-family
// call site landed here without a golden-inventory update at the time.
"src/app/api/providers/route.ts": 3,
"src/app/api/providers/test-batch/route.ts": 2,
"src/app/api/rate-limits/route.ts": 1,
"src/app/api/services/dario/admin/import-from-omniroute/route.ts": 2,
@@ -120,6 +130,10 @@ const EXPECTED: Record<InventoryKind, Record<string, number>> = {
"src/lib/combos/builderOptions.ts": 1,
"src/lib/copilot/tools.ts": 1,
"src/lib/credentialHealth/scheduler.ts": 1,
// Base drift (already present before #11754 boarded, from earlier-merged
// #11698/#11720 retirement PRs' combined getProviderConnectionById
// fallback in the three write-path functions): not introduced by this PR.
"src/lib/db/providers.ts": 3,
"src/lib/db/readCache.ts": 2,
"src/lib/freeProviderRankings.ts": 1,
"src/lib/guardrails/visionBridgeCredentials.ts": 1,
@@ -169,8 +183,6 @@ const CLASSIFICATION: Record<InventoryKind, Record<string, BypassClass>> = {
"open-sse/handlers/chatCore.ts": "A",
"open-sse/handlers/chatCore/cliproxyModelMapping.ts": "A",
"open-sse/handlers/chatCore/cliproxyapiCredentials.ts": "A",
"open-sse/handlers/imageGeneration.ts": "B",
"open-sse/handlers/imageGeneration/providers/geminiWeb.ts": "B",
"open-sse/handlers/videoGeneration.ts": "B",
"open-sse/services/compression/eval/executorModelClient.ts": "B",
"src/lib/compression/judgeModelClient.ts": "B",

5
tests/unit/resolve-web-provider-host.test.ts
View File

@@ -12,7 +12,10 @@ import {
test("known -web provider returns the host derived from its `website`", () => {
const link = resolveWebProviderHost("perplexity-web");
assert.ok(link, "expected a resolved link for perplexity-web");
assert.equal(link.host, "perplexity.ai");
// perplexity-web's registered website is "https://www.perplexity.ai", and
// resolveWebProviderHost returns URL.host verbatim (no www-stripping), so
// both host and url carry the "www." prefix.
assert.equal(link.host, "www.perplexity.ai");
assert.equal(link.url, "https://www.perplexity.ai");
});