Skip to content

fix(core): harden multi-node failover isolation and align 50-tool autotest runner - #37

Merged
mcocdaa merged 1 commit into
mainfrom
fix/failover-and-autotest-50tools
Nov 7, 2026
Merged

mcocdaa merged 1 commit into
mainfrom
fix/failover-and-autotest-50tools

Conversation

@mcocdaa

@mcocdaa mcocdaa commented Nov 7, 2026

Copy link
Copy Markdown
Collaborator

Harden OpenAIClient failover isolation/format fallback and update BehaviorAutotestRunner for 50 registered tools.

…otest runner

fix(openai): downgrade response_format on HTTP 400 and extend failover timeouts

When a node rejects response_format with HTTP 400, automatically
downgrade the format capability (json_schema -> json_object -> none),
cache it per endpoint/model, and retry on the same node instead of
failing over. Strip JsonSchema when ForceJsonMode is disabled.

Also treat HTTP 410 and deprecated-model messages as permanent node
failures, raise default node isolation to 180s, shorten intermediate
failover attempt timeout to 25s, and raise playthrough LLM wait to 95s.

Autotest: mark expected domain rejections as DOMAIN_REJECT, add sample
args for composite actions, and approve player-gated tool calls.
@mcocdaa
mcocdaa merged commit d101139 into main Nov 7, 2026
@mcocdaa
mcocdaa deleted the fix/failover-and-autotest-50tools branch November 7, 2026 06:47
@coderabbitai

coderabbitai Bot commented Nov 7, 2026

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

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