FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(opencode): drop unsupported max reasoning effort by muhammad-fiaz · Pull Request #49995 · anomalyco/opencode · GitHub

fix(opencode): drop unsupported max reasoning effort - #49995

Open
muhammad-fiaz wants to merge 1 commit into
anomalyco:devfrom
muhammad-fiaz:invalid-provider-request
Open

muhammad-fiaz wants to merge 1 commit into
anomalyco:devfrom
muhammad-fiaz:invalid-provider-request

Conversation

muhammad-fiaz commented Sep 19, 2026
edited
Loading

Copy link
Copy Markdown

Issue for this PR

Closes #47975

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Muse Spark via Go/Zen advertises max yet rejects it with invalid_request_error. Agent-level reasoningEffort: max was forwarded to any selected model, failing turn 1.

Drops max for OpenAI-family even when advertised, drops stale efforts not in model.variants, preserves xhigh and valid DeepSeek/GLM max. Matches native OpenAIReasoningEfforts rejection.

How did you verify your code works?

  • Verified the affected reasoning-effort handling with a standalone Bun reproduction.
  • Confirmed that an advertised but rejected max value is not forwarded to the affected OpenAI-family models.
  • Confirmed that supported xhigh remains unchanged.
  • Confirmed that compatible max values for DeepSeek and GLM models are preserved.
  • prettier --check passed.
  • oxlint passed with 0 errors.
  • Ran the Bun test suite.
  • Ran the TypeScript typecheck.

Screenshots / recordings

N/A

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

Muse Spark via Go/Zen advertises max yet 400s with invalid_request_error. Agent-level reasoningEffort max was forwarded to any selected model, failing turn 1.

Clamp max for OpenAI-family even when advertised; drop stale efforts not in model variants; preserve xhigh and valid DeepSeek/GLM max.

Closes anomalyco#47975

Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Provider request fails with invalid_request_error and OpenCode does not work

1 participant


Back | FazBrowse Home | New Git URL