Skip to content

test: cover DeepSeek max token limits - #38

Merged
kevincodex1 merged 1 commit into
Twigpine:mainfrom
Vasanthdev2004:deepseek-max-tokens
Apr 1, 2026
Merged

kevincodex1 merged 1 commit into
Twigpine:mainfrom
Vasanthdev2004:deepseek-max-tokens

Conversation

@Vasanthdev2004

Copy link
Copy Markdown
Collaborator

What changed

  • add regression coverage for OpenAI-compatible provider token caps
  • specifically cover deepseek-chat context window and max output token limits
  • verify oversized CLAUDE_CODE_MAX_OUTPUT_TOKENS values still clamp to DeepSeek's native 8192 limit

Why

Issue #35 reported deepseek-chat failing with Invalid max_tokens on the DeepSeek OpenAI-compatible endpoint.

The runtime cap logic is already present on main, but it was easy for this behavior to look unresolved without focused coverage. This PR locks the behavior down so provider-specific limits for DeepSeek stay protected against future regressions.

Impact

  • helps prevent regressions on DeepSeek and other OpenAI-compatible provider cap handling
  • gives maintainers a focused test path for the reported failure mode
  • makes issue triage clearer because the expected DeepSeek limit is now encoded in tests

Validation

  • bun test src/utils/context.test.ts
  • bun run test:provider

@Vasanthdev2004
Vasanthdev2004 marked this pull request as ready for review April 1, 2026 14:03
@kevincodex1
kevincodex1 merged commit 82e7168 into Twigpine:main Apr 1, 2026
@Vasanthdev2004
Vasanthdev2004 deleted the deepseek-max-tokens branch April 2, 2026 03:35
euxaristia pushed a commit to euxaristia/openclaude that referenced this pull request Apr 13, 2026
reymaster pushed a commit to reymaster/openclaude that referenced this pull request May 5, 2026
thedeveloloper pushed a commit to thedeveloloper/openclaude that referenced this pull request Jun 8, 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.

2 participants