buildStaticProviderEntry() keyed static-catalog combo entries with
opts.providerId (the OC-gate-prefixed id, e.g. "opencode-omniroute")
instead of opts.omnirouteProviderId (the bare server-facing id,
"omniroute") that the dynamic provider.models() hook already uses per
#6859. OC dispatches the static models-map key verbatim as the `model`
field of the outbound request, so a bare-slug combo key doubled up to
"opencode-omniroute/opencode-omniroute/<slug>" and OmniRoute's
parseModel() resolved credentials for the nonexistent provider
"opencode-omniroute" instead of "omniroute". Regular models were
unaffected because their raw ids already contain a slash, skipping the
prefixing branch entirely.
Swap the buildComboKey() call to use opts.omnirouteProviderId,
mirroring the dynamic hook. Adds a permanent regression test to
provider-id-routing.test.ts and aligns the pre-existing hardcoded
"opencode-omniroute/<combo-slug>" assertions in config-shim.test.ts
that had codified the buggy prefix.
Co-authored-by: Fábio Silva <13762289+fabioluissilva@users.noreply.github.com>
resolveOmniRoutePluginOptions() auto-prefixes providerId with "opencode-"
(commit 75b52e286) so OpenCode 1.17.8+'s native-adapter gate accepts it as a
registered provider id. That prefixed value was being reused for the
OmniRoute-server-facing identifiers too: mapRawModelToModelV2's id/providerID,
mapComboToModelV2's providerID, and the dynamic provider hook's combo catalog
keys. OmniRoute's server has no "opencode-<x>" provider alias, so every
dispatched model failed credential lookup with "No credentials for
opencode-omniroute" / "No active credentials for provider:
opencode-omniroute".
Add a second, unprefixed omnirouteProviderId field and thread it through the
four dynamic-hook call sites that emit server-facing identifiers, while
leaving the OC-gate-prefixed providerId in place for AuthHook.provider,
provider registration (hook.id), and the static-catalog path (which OC
strips before dispatch, per the existing static-block comments).
