Skip to content

Fix Z.AI thinking request serialization - #684

Open
alejandro-ao wants to merge 1 commit into
mainfrom
fix/issue-674-x7baag
Open

Fix Z.AI thinking request serialization#684
alejandro-ao wants to merge 1 commit into
mainfrom
fix/issue-674-x7baag

Conversation

@alejandro-ao

Copy link
Copy Markdown
Collaborator

Summary

  • Preserve Z.AI's logical thinking toggle when raw reasoning_effort is unsupported.
  • Serialize the documented Z.AI thinking object and only send effort values for explicitly compatible models.
  • Add MockTransport payload regressions, provider-config coverage, and protocol documentation.

Verification

  • uv sync --dev --locked
  • uv run pytest (1878 passed, 3 skipped)
  • uv run ruff check .
  • uv run ruff format --check .
  • uv run mypy
  • Hugo and Pagefind builds
  • git diff --check

Fixes #674

@alejandro-ao
alejandro-ao marked this pull request as ready for review September 6, 2026 15:56
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.

Error: This model always engages in thinking and can not be disabled

1 participant