fix: expand OpenCode plugin GATEWAY_PROVIDERS to route all proxiable providers through gateway#365
Merged
Merged
Conversation
- Completes Phase 3 Code Review generating findings in quality/results - Fixes CB-001: saveForceMinLayer now uses UPDATE instead of DELETE - Fixes CB-006: OpenAI translator now correctly handles tool_result - Generates TDD logs and test files for fixed bugs
…nd tests - Fix INSERT OR REPLACE in saveForceMinLayer else branch (same class as CB-001) Now uses INSERT OR IGNORE + UPDATE to preserve sibling columns for all paths - Update stale docstring that still referenced DELETE behavior - Add regression test for saveForceMinLayer(sid, N>0) on existing rows with data - Add edge case tests for tool_result: mixed text+tool_result, multiple results, empty content - Remove tautological test_functional.ts (tested local variables, not project code) - Remove noisy generated artifacts (exploration_role_map.json, run_state.jsonl)
…providers through gateway GitHub Copilot and several other providers (deepseek, groq, cerebras, openrouter, huggingface, fireworks, vercel-ai-gateway) were missing from the OpenCode plugin allowlist, causing their traffic to silently bypass the Lore gateway. This meant no memory features (distillation, knowledge, recall) and no compaction — with zero warning to the user. The Pi plugin already included these providers and the gateway already had full support for them. This aligns the OpenCode plugin with the same provider coverage.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
GitHub Copilot and several other providers were missing from the OpenCode plugin's
GATEWAY_PROVIDERSallowlist, causing their LLM traffic to silently bypass the Lore gateway — resulting in no memory features and no compaction, with zero warning to the user.Changes
GATEWAY_PROVIDERSinpackages/opencode/src/index.tsfrom 6 to 14 providers, organized by wire protocolgithub-copilot,fireworks,deepseek,groq,cerebras,openrouter,huggingface,vercel-ai-gatewayContext
The plugin rewrites provider
baseURLonly for providers in the allowlist. Unlisted providers bypass the gateway entirely, which means:Excluded providers (same as Pi plugin):
amazon-bedrock(AWS SDK),google-vertex-ai(Google SDK) — these use native SDKs that can't be HTTP-proxied.