Skip to content

refactor: modularize combo.ts (rebased onto v3.8.31) - #4408

Closed
oyi77 wants to merge 0 commit into
cheaper-inference:release/v3.8.33from
oyi77:pr/combo
Closed

oyi77 wants to merge 0 commit into
cheaper-inference:release/v3.8.33from
oyi77:pr/combo

Conversation

@oyi77

@oyi77 oyi77 commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Rebased version of #3992 onto upstream/release/v3.8.31.

Extracts the 2470-line open-sse/services/combo.ts monolith into 22 focused modules:

  • auto.ts, constants.ts, context.ts, dag.ts, roundRobin.ts, shadow.ts
  • sorting.ts, state.ts, types.ts, utils.ts
  • chat/ (auto, executor, handler, routing, timeout, types)
  • quota/ (config, orchestration, resetAware)
  • index.ts (barrel)

Oversized modules sub-split: chat.ts (1623→6 files), quota.ts (850→3 files).
Original file replaced with thin re-export. Lossless extraction.

Replaces: #3992 (was based on older release)

@oyi77
oyi77 requested a review from diegosouzapw as a code owner June 20, 2026 16:12
@gemini-code-assist

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!

@oyi77

oyi77 commented Jun 20, 2026

Copy link
Copy Markdown
Contributor Author

Replaces closed PR (original was held pending quality-gate landing). Quality gates are now stable through v3.8.31 — reopening for merge. Related: #3594

@diegosouzapw
diegosouzapw changed the base branch from release/v3.8.31 to release/v3.8.32 June 20, 2026 18:30
@oyi77
oyi77 force-pushed the pr/combo branch 2 times, most recently from 017ca0e to 4894027 Compare June 20, 2026 18:43
@diegosouzapw
diegosouzapw changed the base branch from release/v3.8.32 to release/v3.8.33 June 21, 2026 14:03
@oyi77 oyi77 closed this Jun 22, 2026
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