fix(sse): align regex naming and changelog formatting

This commit is contained in:
Will Gordon
2026-07-30 16:07:34 -04:00
parent 796b4fefd1
commit c2c622ad82
2 changed files with 11 additions and 7 deletions

View File

@@ -1,6 +1,10 @@
- fix(sse): Claude reasoning-effort suffix ids (`-high`/`-low`/`-medium`/`-xhigh`) now strip
correctly on any provider serving a real Claude model, not just the direct Anthropic provider;
the no-thinking (`no-think/`) catalog variant's provider-qualification bug (which made it
unusable outside the direct provider) is also fixed; and a single unrecognized model id on a
Vertex connection no longer cools down every other model on that connection for 2 minutes
- **fix(sse):** Claude reasoning-effort suffix ids (`-high`/`-low`/`-medium`/`-xhigh`) now strip
correctly on any provider serving a real Claude model, not just the direct Anthropic provider
([#9006](https://github.com/diegosouzapw/OmniRoute/pull/9006))
- **fix(sse):** the no-thinking (`no-think/`) catalog variant's provider-qualification bug — which
made it unusable outside the direct provider, both in the discovery catalog and the dashboard
playground — is fixed ([#9006](https://github.com/diegosouzapw/OmniRoute/pull/9006))
- **fix(sse):** a single unrecognized model id on a Vertex connection no longer cools down every
other model on that connection for 2 minutes — Vertex 404s are now scoped to a per-model
lockout via `passthroughModels` instead of a connection-wide cooldown
([#9006](https://github.com/diegosouzapw/OmniRoute/pull/9006))

View File

@@ -33,7 +33,7 @@ export const CC_DISCOVERY_COMBO_PREFIX = "claude/combo/";
// Ids that already live under the claude/anthropic namespace — never re-mirror them.
const ALREADY_CLAUDE_RE = /^(?:claude|anthropic)(?:\/|$)/i;
// Ids that already carry a reasoning-effort suffix — v1 only mirrors base ids.
const EFFORT_SUFFIX_RE = /-(?:xhigh|high|medium|low)$/i;
const CLAUDE_EFFORT_SUFFIX_RE = /-(?:xhigh|high|medium|low)$/i;
const NO_THINKING_PREFIX = "no-think/";
// Built-in `auto`/`auto/*` combos are synthesized by createBuiltinAutoCombo, NOT
// stored in the DB combos table — the request-path resolver (getComboByName) can't
@@ -66,7 +66,7 @@ function isMirrorableId(id: string): boolean {
if (id.length === 0) return false;
if (ALREADY_CLAUDE_RE.test(id)) return false;
if (id.startsWith(NO_THINKING_PREFIX)) return false;
return !EFFORT_SUFFIX_RE.test(id);
return !CLAUDE_EFFORT_SUFFIX_RE.test(id);
}
/** Strip a `<provider>/` prefix to get the bare model name, matching the convention in