Skip to content

fix(ai): update model list and opencode-go DeepSeek V4 Flash thinking-level expectation - #1218

Closed
snimu wants to merge 2 commits into
mainfrom
snimu/fix-opencode-go-deepseek-thinking-test
Closed

fix(ai): update model list and opencode-go DeepSeek V4 Flash thinking-level expectation#1218
snimu wants to merge 2 commits into
mainfrom
snimu/fix-opencode-go-deepseek-thinking-test

Conversation

@snimu

@snimu snimu commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Problem

The Test (ai) CI job is failing on every branch, including main. Here's why: the build regenerates the model catalog (models.generated.ts) by fetching the providers' live model lists. One provider (opencode-go) changed how it serves DeepSeek V4 Flash — different API, different thinking levels. One of our tests still expects the old behavior, so it fails everywhere, no matter what the branch changes.

Committing an updated catalog alone wouldn't help: the build fetches fresh data anyway and would overwrite it. The test itself has to change.

Fix

Two commits, nothing else:

  1. Update the test to expect what the provider actually serves now. The expectations for the other two DeepSeek routes are unchanged.
  2. Refresh the committed catalog with the current npm run generate-models output, so the file in the repo matches reality again.

What doesn't change

No code changes — one test file and one generated file.

Validation

  • Full packages/ai test suite with a fresh catalog: 315 passed, 0 failed
  • Typecheck clean
  • The test now also asserts which API the provider uses, so if they switch back we'll see it clearly instead of a confusing failure

The opencode-go zen catalog now serves deepseek-v4-flash through
anthropic-messages instead of the DeepSeek openai-completions format, so
the DeepSeek-specific high/xhigh thinking-level mapping no longer applies
on that provider. packages/ai regenerates models.generated.ts from the
live catalog on every build, so CI picked this up on every branch.

The test now pins the anthropic-messages route and its default thinking
levels, keeping the DeepSeek-provider and OpenRouter expectations
unchanged.
…alogs

Commits the current output of `npm run generate-models` so the checked-in
catalog matches upstream again (opencode-go moved deepseek-v4-flash to
anthropic-messages, plus assorted drift accumulated since the last
regeneration).
@snimu snimu changed the title fix(ai): update opencode-go DeepSeek V4 Flash thinking-level expectation fix(ai): update model list and opencode-go DeepSeek V4 Flash thinking-level expectation Aug 11, 2026
@snimu
snimu requested a review from alexzhang13 August 11, 2026 14:40
@snimu snimu closed this Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant