fix(combos): carry scoped inference quota evidence (#4090) - #4336
Merged
Conversation
Contributor
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
|
✅ Deterministic PR hygiene checks passed. |
3 tasks
lidge-jun
force-pushed
the
codex/260912-60plus-combo
branch
from
September 12, 2026 04:57
34a3ce7 to
0bcb43e
Compare
lidge-jun
marked this pull request as ready for review
September 12, 2026 05:00
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
This was referenced Sep 12, 2026
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
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.
Summary
Carry #4090 onto the current TypeScript runtime. Provider-wide Combo exclusion, reset-window ordering and catalog inactivity now use explicit inference projections bound to the current single credential. Display-only account/model/search/MCP reports do not veto a provider. The latest omitted-projection correction and its direct regression are retained.
Current-dev adaptation migrates the newer catalog inactivity consumer, aligns the exclusive freshness deadline, adds binding-change regressions, and moves the original contract into the current structure docs.
Ordinary manual chain: this runtime layer targets dev; editor layer #4339 targets this head. Parent coordinator owns merge. Original #4090 remains open pending integration.
Source: a110c49, a9694ff, c061316.
Co-authored-by: luvs01 27862058+luvs01@users.noreply.github.com
Verification
Checklist
Rebased onto dev5042a376ef with documentation-only conflict resolution; current head 0bcb43e. Product carry delta retained; final cumulative CI restarted.
Maintainer integration decision
Owner-authorized ordinary manual-chain integration, lower #4336 then editor #4339. Current reviewed layer head:
0bcb43e266f500bee3acdaa5dde8eff41f34df00; cumulative final tip:e7a68d208e579a08145e79397309e146a314db0c. Independent runtime security review Averroes PASS; editor security review Bacon PASS after mixed-snapshot clock correction and deterministic fixture settling. Bacon reviewed the rebase/cascade delta: thirteen structure append conflicts preserve both upstream and quota content; product/test carry patches unchanged. Coordinator inspected routing scope/binding source, merge topology, current review threads and synthetic GUI evidence. Original luvs01 attribution remains.The owner explicitly forbids every local suite and requests trailing CI at the final cumulative tip only. PR CI https://github.com/lidge-jun/opencodex/actions/runs/34674364710 and all-lane dispatch https://github.com/lidge-jun/opencodex/actions/runs/34674363301 are pending. Intermediate-tip test success is not asserted. Windows all-lane runner type follows the existing workflow; no workflow, protection or cancellation changes were made. Captured Combo-source behavior is linked by unchanged relevant source/assets, not misrepresented as a new full-tree render. Local suites/build/typecheck/install NOT RUN. Trailing failures remain repair obligations.