[herjarsa]

Summary

The OpenCode 1.17.8+ LLM adapter rejects provider IDs that are not in {openai, anthropic, opencode*}. The OmniRoute plugin's providerId (omniroute) fails this gate, breaking credential resolution (Ji() function).

Fix: Auto-prefix the effective providerId with opencode- when the configured ID doesn't already start with opencode. User config keeps omniroute; the plugin registers as opencode-omniroute for OC's native dispatch.

Also add auth.json fallback so the old bare key is still found at runtime.

Test Evidence

Plugin Test Suite (259 tests, 0 failures)

ℹ pass 259
ℹ fail 0

7 test files updated for the auto-prefix change:

File	Assertions updated
scaffold.test.ts	3 assertions (providerId, displayName)
auth.test.ts	3 assertions (hook.provider, label)
provider.test.ts	7 assertions (hook.id, model keys, providerID)
combos.test.ts	19 assertions (model keys, providerID)
config-shim.test.ts	100+ assertions (provider keys, auth fixture keys, model keys)
features.test.ts	15 assertions (enrichment, MCP, multi-instance)
multi-instance.test.ts	5 assertions (provider values)

Live OC 1.17.8 Verification (Windows)

• opencode run --model opencode-omniroute/groq/llama-3.1-8b-instant "x" → Reaches Groq real API (413 TPM quota confirms full pipeline working)
• opencode models opencode-omniroute → 428 models displayed
• Default picker (no --model flag) routes MASTER-LIGHT via DeepSeek V4 Flash Free ✅

Files Changed

• @omniroute/opencode-plugin/src/index.ts: Auto-prefix logic + auth fallback
• @omniroute/opencode-plugin/tests/*.test.ts (7 files): Update assertions for prefixed providerId

Pre-existing CI Failures (not from this PR)

• check:file-size: 4 frozen baselines exceeded in untouched files
• check:env-doc-sync: KIRO_VERIFY_FULL_CRC missing from .env.example
• check:docs-all: Executor count drift (64→62)
• These are pre-existing and block all PRs.

[gemini-code-assist]

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

[diegosouzapw]

Obrigado, @herjarsa! 🙏 Confirmei que este é o re-do enxuto do #4445 (só o fix de prefixo, sem a remoção do loader/timeouts) — e que a lógica de prefixo opencode- está mesmo ausente da release, então o fix é necessário e correto em essência.

Porém, ao rodar a suíte do @omniroute/opencode-plugin com a mudança aplicada, o prefixo quebra ~20+ testes porque ele altera todas as chaves expostas (catálogo, provider hook, ModelV2) de omniroute/... → opencode-omniroute/...:

• tests/scaffold.test.ts: resolveOmniRoutePluginOptions defaults/custom (providerId agora opencode-omniroute)
• tests/provider.test.ts: createOmniRouteProviderHook → hook.id e chaves out["omniroute/..."]
• tests/combos.test.ts, tests/config.test.ts/config-shim.test.ts, buildStaticProviderEntry — todas as asserções de chave de catálogo/provider

Para mergear (Rule #18 — suíte tem que ficar verde) é preciso atualizar as expectations desses testes para a forma prefixada (mantendo os inputs providerId: 'omniroute-*' como estão). Não fiz a atualização em massa porque distinguir input vs expectation em ~137 referências exige cuidado seu (e há uma questão de produto: prefixar muda as IDs de modelo expostas — vale confirmar num OpenCode 1.17.8 real, Rule #18, que configs existentes referenciando omniroute/modelo continuam resolvendo).

Pode atualizar a suíte do plugin (e idealmente colar um teste vivo no OC 1.17.8)? Aí eu reconstruo sobre a release/v3.8.33 e mergeio preservando seu crédito. 🙌

[herjarsa]

Updated with full test suite fix — 259/259 tests pass (0 failures).

Changes in this update (7 test files, 136 insertions / 136 deletions):

• scaffold.test.ts: providerId assertions updated for auto-prefix
• auth.test.ts: hook.provider and label assertions updated
• provider.test.ts: hook.id, model keys, providerID assertions updated
• combos.test.ts: all combo model key assertions updated (19 occurrences)
• config-shim.test.ts: provider keys, auth.json fixture keys, model keys, multi-instance keys, buildStaticProviderEntry keys (~100 assertions)
• features.test.ts: enrichment, compression, MCP hook keys updated
• multi-instance.test.ts: provider assertions for auto-prefixed values

All INPUT providerId values kept as-is (e.g., providerId: 'omniroute'). All OUTPUT assertions updated to reflect opencode- prefix.

[diegosouzapw]

Obrigado, @herjarsa! 🙌 Suíte do plugin opencode passa (258 testes de fonte verdes; o scaffold depende do artefato de build que o CI gera). Esta versão substitui a #4445 (que trazia uma migração destrutiva). Mergeando para a release.