Conversation
Introduce combo/<id> groups with failover and sticky round-robin, per-target cooldowns, hop-on-retryable failures, catalog exposure, and /api/combos CRUD (issue #133).
Add e2e coverage (403 then 200) and a runtime flowchart for issue #133.
Only inject combo defaultEffort when supportedLadderFor exposes a non-empty ladder; clamp to a supported rung, never force effort onto no-reasoning models.
Update applyComboDefaultEffort call sites to pass the resolved route and assert skip when the model has no effort control.
Add Combos dashboard CRUD (rail/overview/detail/modal) against /api/combos without pulling the Providers workspace redesign.
Wibias
force-pushed
the
feat/gui-combos-workspace
branch
from
July 18, 2026 04:35
5a7b7c7 to
be85791
Compare
Import the shared pwi-* stylesheet so Combos matches the providers workspace look, and add missing common.discard / common.saving i18n keys that blocked the GUI TypeScript build.
Combos was squeezed into the default 980px main-inner after the clean rebuild. Re-apply main-inner--providers so the rail/detail shell fills the viewport again.
Wibias
marked this pull request as ready for review
July 18, 2026 07:21
Owner
|
The Combos GUI workspace has landed on
Notes on scope: the backend files in this PR were a variant of #147, which had already been rebuilt on Verified: tsc, GUI lint + i18n lint + build, full suite 3,006 tests green, and screenshot QA across empty/detail/modal/error/mobile states. This completes the GUI follow-up promised on #133. Closing in favor of the dev-landed commits — ships with the next release. Thanks for the complete workspace design! |
lidge-jun
added a commit
that referenced
this pull request
Jul 18, 2026
Adapts the PR #148 GUI to the reviewed combo stack already landed on dev: strict PUT validation, mixed disabled-member handling, deterministic SWRR wording, non-nesting catalog rows, fail-closed availability, and ordered attempt attribution. Source: PR #148 Source head: 9fed606 Co-authored-by: Wibias <37517432+Wibias@users.noreply.github.com>
agentHits
pushed a commit
to agentHits/opencodex
that referenced
this pull request
Sep 17, 2026
Reconstructs the GUI components, view-model helpers, styles, icons, locale blocks, and focused helper tests from community PR lidge-jun#148 on current dev. Source: PR lidge-jun#148 Source head: 9fed606
agentHits
pushed a commit
to agentHits/opencodex
that referenced
this pull request
Sep 17, 2026
Wires the reconstructed Combos workspace from community PR lidge-jun#148 into the current dashboard page registry and global style graph without regressing provider subroutes. Source: PR lidge-jun#148 Source head: 9fed606
agentHits
pushed a commit
to agentHits/opencodex
that referenced
this pull request
Sep 17, 2026
Adapts the PR lidge-jun#148 GUI to the reviewed combo stack already landed on dev: strict PUT validation, mixed disabled-member handling, deterministic SWRR wording, non-nesting catalog rows, fail-closed availability, and ordered attempt attribution. Source: PR lidge-jun#148 Source head: 9fed606 Co-authored-by: Wibias <37517432+Wibias@users.noreply.github.com>
3 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Depends on
#147 must merge first (combo runtime API).
Preferred visual merge order: #139 → #147 → this PR. Combos imports
styles-provider-workspace.css(sharedpwi-*chrome from the Providers workspace). That file is vendored here so CI builds before #139 lands; once #139 is onmain, it is the same stylesheet.Until #147 lands, GitHub will also show those backend commits here (same fork stacking vs
main). After #147 merges, rebase this branch ontomainand the diff shrinks to the GUI commits.Summary
#combos): rail list, overview, detail editor, create modalpwi-*chrome (same file as feat(gui): providers workspace redesign polish + freeTier + rate limits #139)Screenshots
Test plan
bun run start+ GUI)/api/combosmodel: "combo/<id>"through Codex //v1/responses