fix(actions): preserve none reasoning effort for ByteDance

[mikemikimike]

SummarynnFixes #3365. ByteDance GLM-5.2 supports reasoning_effort none, but request-body normalization stripped it because ByteDance was missing from the native-none provider allowlist.nn- Preserve reasoning_effort none for ByteDance.n- Add a focused regression test.nn## Validationnn- prepare-request-body.spec.ts: 216 passedn- prepare-request-body.spec.ts + models.spec.ts: 267 passedn- Targeted Prettier and ESLint checks: passedn- turbo build for @llmgateway/actions: passed, 6 tasksnnNo provider API key or external service was required. Package-wide lint is blocked by pre-existing formatting failures in 30 unrelated files; the changed files pass targeted checks.

Summary by CodeRabbit

• Bug Fixes
  • ByteDance GLM-5.2 requests now preserve the reasoning_effort: "none" setting, ensuring it is correctly forwarded during processing.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 4b6e3baf-71c3-4283-b8f8-73fe4ec59350

📥 Commits

Reviewing files that changed from the base of the PR and between a78e0ee and d7106cf.

📒 Files selected for processing (2)

• packages/actions/src/prepare-request-body.spec.ts
• packages/actions/src/prepare-request-body.ts

🚧 Files skipped from review as they are similar to previous changes (2)

• packages/actions/src/prepare-request-body.spec.ts
• packages/actions/src/prepare-request-body.ts

---

Walkthrough

ByteDance is added to the providers that preserve reasoning_effort: "none". A GLM-5.2 regression test verifies that request preparation forwards the value unchanged.

Changes

ByteDance reasoning effort

Layer / File(s)	Summary
Enable and verify native none forwarding packages/actions/src/prepare-request-body.ts, packages/actions/src/prepare-request-body.spec.ts	The provider allowlist and related documentation now include ByteDance. The GLM-5.2 test verifies that reasoning_effort: "none" remains in the generated request body.

Estimated code review effort: 2 (Simple) | ~5 minutes

Possibly related PRs

• theopenco/llmgateway#3138: Declares GLM-5.2 reasoning levels that this forwarding change supports.
• theopenco/llmgateway#3367: Adds similar provider-specific forwarding behavior for reasoning_effort: "none".
• theopenco/llmgateway#3152: Extends native reasoning_effort: "none" handling for another provider.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes preserving "none" reasoning effort for ByteDance.
Linked Issues check	✅ Passed	The changes add ByteDance to the native-"none" allowlist and test the required request-body behavior from issue #3365.
Out of Scope Changes check	✅ Passed	The allowlist update and focused regression test directly support the linked issue without unrelated code changes.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@packages/actions/src/prepare-request-body.ts`:
- Line 1190: Update the native-provider explanation near the condition
containing usedProvider === "bytedance" to include ByteDance in the listed
providers treated as native none providers, keeping the comment aligned with the
normalization logic.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 50f771d8-eb92-4583-a1cb-ce50e16f8c5a

📥 Commits

Reviewing files that changed from the base of the PR and between 3ab66a2 and 8e3cb9b.

📒 Files selected for processing (2)

• packages/actions/src/prepare-request-body.spec.ts
• packages/actions/src/prepare-request-body.ts