Skip to content

test(ci): align release expectations with provider behavior - #10236

Merged
diegosouzapw merged 2 commits into
release/v3.8.50from
fix/release-xai-stale-tests-10170
Aug 13, 2026
Merged

diegosouzapw merged 2 commits into
release/v3.8.50from
fix/release-xai-stale-tests-10170

Conversation

@diegosouzapw

@diegosouzapw diegosouzapw commented Aug 13, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Root causes

  1. fix(providers): xai-oauth chat→responses translation + breaker import (#10165) #10170 intentionally moved xai-oauth/grok-4.5 from Chat Completions to the Responses contract, but two older assertions still expected /chat/completions and messages.
  2. fix(providers): default missing cache_control.ttl to 1h on the native Claude OAuth path #10221 intentionally defaults missing native-Claude cache_control.ttl values to 1h, but five older assertions still expected only { type: "ephemeral" }.
  3. Existing provider sanitization downgrades xhigh to high for models that do not declare extra-high support, but one AIO assertion still expected xhigh.

The new feature-specific regression tests were green in every case; only the stale expectations failed.

TDD proof — Local VPS 192.168.0.15

First red/green cycle, xAI:

red:   12 tests — 10 passed, 2 failed
green: 12 tests — 12 passed, 0 failed

Second red/green cycle, Claude cache TTL and provider reasoning sanitization:

red:   94 tests — 88 passed, 6 failed
green combined suite: 106 tests — 106 passed, 0 failed

Combined VPS command:

DATA_DIR=/tmp/omniroute-base-red-green-46f0294f \
SQLITE_FILE=/tmp/omniroute-base-red-green-46f0294f/storage.sqlite \
node --import tsx/esm --test \
  tests/unit/chatcore-translation-paths.test.ts \
  tests/unit/cc-compatible-provider.test.ts \
  tests/unit/executor-xai.test.ts \
  tests/unit/executor-xai-chat-to-responses-10165.test.ts

The same three changed files passed ESLint on the VPS with the repository's official suppression baseline.

Scope

Changed files:

  • tests/unit/executor-xai.test.ts
  • tests/unit/chatcore-translation-paths.test.ts
  • tests/unit/cc-compatible-provider.test.ts

No changelog fragment: this PR only reconciles stale tests with already documented and merged production behavior.

@diegosouzapw diegosouzapw changed the title test(xai): align executor expectations with Responses test(ci): align release expectations with provider behavior Aug 13, 2026
@diegosouzapw
diegosouzapw merged commit 7acddd9 into release/v3.8.50 Aug 13, 2026
10 checks passed
@diegosouzapw
diegosouzapw deleted the fix/release-xai-stale-tests-10170 branch August 13, 2026 07:19
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
…inference#10236)

* test(xai): align executor expectations with Responses

* test(ci): align release expectations with provider behavior

---------

Co-authored-by: backryun <bakryun0718@proton.me>
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