Conversation
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
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
Why
Selecting
deepseek/deepseek-v4-flashthrough anopenai-compatiblecredential previously produced an OpenCode selection ofdeepseek/deepseek-v4-flash, although onlyopenai-compatiblewas enabled. OpenCode can then fall back to another declared model and receive a 403 because its request differs from the model authorized by the proxy.The selection is now
openai-compatible/deepseek/deepseek-v4-flash, while both the upstream model ID and grant retaindeepseek/deepseek-v4-flash. Existing preset vendor-prefix normalization remains supported.Verification
just test-file apps/api/tests/runtime-vendor-env-vars.test.ts: 33 passed; slash-containing regression cases failed before the fix and passed afterward.just test-file apps/api/tests/driver-llm-proxy-route.test.ts: 34 passed, including admission of the provisioned model and rejection of altered models/protocols.just tc-package @mosoo/api,just lint, changed-file formatting checks, andgit diff --check: passed.gpt-5anddeepseek/deepseek-v4-flashdeclared. The old selection requested GPT and received a model-capability 403. The fixed selection requested DeepSeek throughout, executed a real bash tool writingpi-byok-okto a temporary file, and completed the turn.just checkwas attempted on macOS; existing driver process-supervision tests require Linux/usr/bin/setsidand fail locally. See Linux CI for the complete gate.Impact
openai-compatible/are now treated literally. Revert this commit to restore prior behavior. Deployment verification should use a new session/driver so it receives the new configuration and grant; existing generated session configuration is not rewritten.Review