fix(plugin): prefix combo keys + emit providerID + drop combo/ prefix

[herjarsa]

Summary

Six-part fix for the regression where the omniroute provider is misdetected or rejected by OpenCode's static-catalog reader and the auth loader returns empty credentials.

Why the upstream plugin breaks

Surface	Symptom
combo/MASTER (original)	"No credentials for provider: omniroute" — combo/ parsed as provider=combo
bare MASTER (fix #1)	"Unable to determine provider for model 'master-light'" — bare key misread
omniroute/MASTER + bare raw (fix #2)	provider not detected — mixed keys rejected by OC schema
omniroute/MASTER + 10s timeout	catalog empty (stub published) — server cold-start exceeded 10s
+ 60s timeout	catalog loads, but "No credentials for provider: omniroute" on send
+ auth loader lookup (this PR)	✓ end-to-end works

Fixes

1. 99531c846 drop combo/ prefix from combo model keys
2. 44baa3eb6 emit providerID on static-catalog entries (later dropped)
3. 5904c495 prefix combo keys with providerId (omniroute/<slug>)
4. 0407b655b prefix raw model keys + drop providerID on entries
5. 22a1b9653 raise config shim fetch timeout 10s → 60s
6. 842ae03b1 auth loader handles full auth record shape (this PR)

Fix 6 — auth loader full-record shape (commit 842ae03)

The auth loader assumed OC's getAuth() returns a single auth entry ({ type, key }) per the @opencode-ai/sdk@1.17.8 type contract. Some OC runtimes return the FULL auth record ({ omniroute: { type, key }, openai: { ... }, ... }). When the loader treated the full record as a single entry, validation failed silently and the loader fell through to return {} — producing the original "No credentials for provider: omniroute" error.

Fix: accept either shape. If getAuth() returns an object that has a key matching the hook's providerId, pick that entry; otherwise treat the response as the single entry per the SDK type.

Test plan

• 259/259 plugin tests pass
• Smoke test verifies both auth shapes produce credentials
• Manual QA: restart OC, send a message → request succeeds, models + creds resolve

Supersedes #4378

PR #4378 shipped fixes 1+2 but not 3+4+5+6. None of those alone is sufficient.

[gemini-code-assist]

Code Review

This pull request updates combo static-block keys to use the provider ID prefix instead of the hardcoded combo/ namespace, preventing credentials-not-found errors in OpenCode. It also adds default and per-model context lengths for the [removed-provider] provider to fix missing context window reports, along with a regression test. Feedback from the review highlights a discrepancy where mapped.id is not prefixed with the provider ID in the hook, which could still trigger provider resolution errors, and suggests updating the JSDoc for buildComboKey to match its new behavior.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

The PR description states that mapComboToModelV2 returns id: ${providerId}/${combo.id}. However, the actual implementation of mapComboToModelV2 (which is not modified in this PR) still returns id: combo.id without the provider prefix. This creates a discrepancy where the model's id field is not prefixed, which can cause OpenCode to fail with "Unable to determine provider" when routing requests using model.id.

To align with the intended design, we should prefix mapped.id with the providerId here.

Suggested change

	const comboKey = buildComboKey(combo, usedComboKeys, resolved.providerId);
	const comboKey = buildComboKey(combo, usedComboKeys, resolved.providerId);
	mapped.id = `${resolved.providerId}/${combo.id}`;

[gemini-code-assist]

The JSDoc for buildComboKey states that it returns a "bare key" or "bare slug like MASTER". However, the implementation has been updated to return a provider-prefixed key (${providerId}/${slug}). The documentation should be updated to accurately reflect this behavior to avoid confusing future maintainers.

Suggested change

	/**
	* Build a combo's static-block key (`combo/<slug>`), guaranteeing uniqueness
	* across an entire static catalog. If `<slug>` is already present in `used`,
	* suffixes a short UUID-prefix disambiguator from `combo.id` so the second
	* combo doesn't silently overwrite the first. Mutates `used` in place by
	* recording the chosen key. Returns the final `combo/<...>` key.
	* Build a combo's static-block key (bare slug like `MASTER`, `MASTER-LIGHT`),
	* guaranteeing uniqueness across an entire static catalog. If `<slug>` is
	* already present in `used`, suffixes a short UUID-prefix disambiguator from
	* `combo.id` so the second combo doesn't silently overwrite the first.
	* Mutates `used` in place by recording the chosen key. Returns the final
	* bare key.
	*
	* Falls back to `combo/<id>` when the friendly name slugifies to the empty
	* NOTE: the key MUST NOT carry a `combo/` namespace prefix — OpenCode
	* parses model IDs on `/` to extract a provider prefix, and `combo/MASTER`
	* would be treated as provider=`combo`, model=`MASTER`, causing a
	* credentials-not-found error. See PR #4184.
	*
	* Falls back to bare `<id>` when the friendly name slugifies to the empty
	* string (e.g. a combo named just punctuation).
	*/
	/**
	* Build a combo's static-block key prefixed with the provider ID (e.g.,
	* `omniroute/MASTER`), guaranteeing uniqueness across an entire static catalog.
	* If the key is already present in `used`, suffixes a short UUID-prefix
	* disambiguator from `combo.id` so the second combo doesn't silently overwrite
	* the first. Mutates `used` in place by recording the chosen key. Returns the
	* final prefixed key.
	*
	* NOTE: the key MUST NOT carry a `combo/` namespace prefix — OpenCode
	* parses model IDs on `/` to extract a provider prefix, and `combo/MASTER`
	* would be treated as provider=`combo`, model=`MASTER`, causing a
	* credentials-not-found error. See PR #4184.
	*
	* Falls back to `<providerId>/<id>` when the friendly name slugifies to the
	* empty string (e.g. a combo named just punctuation).
	*/

[diegosouzapw]

Merged into release/v3.8.31 🎉 Thanks @herjarsa! This is the correct fix for the OpenCode combo-resolution bug.

Adjustments before merge (co-authored): dropped the providerID field from all static-catalog entries (it was inconsistently left on auto-combos; OC ignores it on the static path), corrected the buildComboKey docblock to describe the <providerId>/<slug> format, and dropped the bundled [provider removed at its operator's request]/#4184 commits (already in release).

Live OpenCode → VPS validation (Rule #18) (OmniRoute v3.8.31, 42 combos):

$ opencode models omniroute
omniroute/combo-codex-1
omniroute/teste                         # OC parses the omniroute/<slug> key ✔

# the bare slug OC sends upstream after stripping the prefix:
POST /v1/chat/completions {model:"teste"}      -> 503 ALL_ACCOUNTS_INACTIVE   (combo RESOLVED, just no active upstream)
POST /v1/chat/completions {model:"zzz-bogus"}  -> 400 "Unable to determine provider … use a provider/model prefix"

The contrast is the proof: a real combo slug resolves (reaches routing), while a non-combo gives the exact "Unable to determine provider" error the old combo/<slug> key produced (parsed as provider=combo). omniroute/<slug> resolves end-to-end. Ships in the next release.