Skip to content

test(autoCombo): stabilize model fitness source expectation - #6002

Closed
KooshaPari wants to merge 11 commits into
cheaper-inference:release/v3.8.43from
KooshaPari:fix/5269-tool-search-upstream
Closed

test(autoCombo): stabilize model fitness source expectation#6002
KooshaPari wants to merge 11 commits into
cheaper-inference:release/v3.8.43from
KooshaPari:fix/5269-tool-search-upstream

Conversation

@KooshaPari

@KooshaPari KooshaPari commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • stabilizes the autoCombo model fitness source expectation
  • repairs CI gate drift surfaced on the restored branch: request logger chunk contracts, call-log ordering, live WS env docs, and provider translate-path golden

Validation

  • npm exec --yes tsx -- --test tests/unit/request-logger-endpoints.test.ts tests/unit/stream-utils.test.ts tests/unit/request-log-detail-layout.test.ts tests/unit/provider-translate-path-golden.test.ts
  • npm run check:env-doc-sync
  • pre-commit: docs sync, T11 any-budget, tracked-artifacts

@KooshaPari
KooshaPari requested a review from diegosouzapw as a code owner July 2, 2026 23:08
@gemini-code-assist

gemini-code-assist Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@KooshaPari KooshaPari mentioned this pull request Jul 3, 2026
Merged
KooshaPari added a commit to KooshaPari/OmniRoute that referenced this pull request Jul 3, 2026
@KooshaPari @VitzS7
@abil0321 @diegosouzapw
feat(translator,sse): port Kiro system-reminder, gemini multipleOf st…
3d72e44 
…rip, parseAutoConfig leaf, cline envelope (#264)

Bundles four upstream translator/SSE fixes into the fork:

- fix(translator/kiro): wrap system-origin content in <system-reminder>
  tags before merging into the Kiro/CodeWhisperer user message, so the
  system prompt no longer leaks as raw user text (cheaper-inference#6053).
- fix(translator/gemini): strip `multipleOf` from antigravity/gemini tool
  schemas at every level (top-level, nested, array items) — Gemini's
  OpenAPI 3.0 subset rejects it with a hard 400 (cheaper-inference#6052).
- refactor(sse): extract the pure `parseAutoConfig` leaf from combo.ts
  into open-sse/services/combo/autoConfig.ts (cheaper-inference#6049).
- fix(sse/cline): unwrap the Cline non-streaming response envelope so a
  content array of Anthropic-style text blocks is not misclassified as
  502 empty_choices (cheaper-inference#6046).

cheaper-inference#6002 (autoCombo fitness-source stabilize) is already on fork main via
cheaper-inference#5890 — not re-applied here.

Co-authored-by: VitzS7ven <209032696+VitzS7@users.noreply.github.com>
Co-authored-by: Muhammad Salsabil <74514963+abil0321@users.noreply.github.com>
Co-authored-by: Diego Rodrigues de Sa e Souza <diegosouza.pw@outlook.com>
This was referenced Jul 3, 2026
Merged
Merged
@KooshaPari

KooshaPari commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by replay PR #6076 on release/v3.8.44. Closing this stale branch copy so review and CI continue on the live replacement.

@KooshaPari KooshaPari closed this Jul 3, 2026
@diegosouzapw

diegosouzapw commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Thanks @KooshaPari. There's a problem with this one against the current release: the diff is dominated by a revert of #5834 (the correlation-id / request-logger observability work), which is already shipped on release/v3.8.44 — e.g. it strips the [ts] prefix in requestLogger.ts, collapses rowPriority 0/1/2→0/1 in call-logs/route.ts, and renames "Ended At"→"Completed Time" in RequestLoggerDetail.tsx. Merging it would revert that feature. The branch is also based on release/v3.8.43 (frozen) and conflicts massively when retargeted to .44. The genuinely useful part is the OS/arch golden-snapshot normalization in provider-translate-path-golden.test.ts (portability across recording machines) — that's worth a standalone PR rebased on release/v3.8.44, without the #5834 revert. Could you re-cut just that piece? Leaving this open. 🙏

diegosouzapw added a commit to KooshaPari/OmniRoute that referenced this pull request Jul 3, 2026
@KooshaPari @diegosouzapw
fix(ci): harden provider translate-path golden across CI runners
5e62112 
Normalize the OS/arch-derived request-header values in the golden snapshot so
the test is stable regardless of which OS the CI runner (or a contributor's
machine) uses:
- X-Stainless-Os / X-Stainless-Arch → <OS> / <ARCH>
- User-Agents of the form (OS; arch) → (<OS>; <ARCH>)
- Antigravity's os.platform()-derived platform substring (Macintosh… /
  Windows NT… / X11; Linux…) → <PLATFORM> (was left as a Mac literal, which
  would have failed on Linux CI runners)

Snapshot regenerated with the full normalization applied. Supersedes the stale
cheaper-inference#6002 branch.

Co-authored-by: diegosouzapw <diegosouza.pw@gmail.com>
diegosouzapw added a commit that referenced this pull request Jul 3, 2026
@KooshaPari @diegosouzapw
fix(ci): harden provider translate-path golden across CI runners (#6076)
439f28d 
Normalize OS/arch-derived request headers (X-Stainless-Os/Arch, (OS;arch) UAs, and Antigravity's os.platform()-derived platform substring) in the golden so the test is runner-independent. Fixes the Mac-literal Antigravity UA that would have failed on Linux CI. Supersedes stale #6002.

Co-authored-by: diegosouzapw <diegosouza.pw@gmail.com>
@KooshaPari
KooshaPari deleted the fix/5269-tool-search-upstream branch August 13, 2026 06:53
tkgo11 pushed a commit to tkgo11/OmniRoute that referenced this pull request Sep 23, 2026
@KooshaPari @diegosouzapw
fix(ci): harden provider translate-path golden across CI runners (che…
cb68095 
…aper-inference#6076)

Normalize OS/arch-derived request headers (X-Stainless-Os/Arch, (OS;arch) UAs, and Antigravity's os.platform()-derived platform substring) in the golden so the test is runner-independent. Fixes the Mac-literal Antigravity UA that would have failed on Linux CI. Supersedes stale cheaper-inference#6002.

Co-authored-by: diegosouzapw <diegosouza.pw@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@diegosouzapw diegosouzapw Awaiting requested review from diegosouzapw diegosouzapw is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@KooshaPari @diegosouzapw