Skip to content

fix(router): remove deprecated autoOrchestrate model override fields#403

Open
Gucc111 wants to merge 1 commit into
OpenBMB:mainfrom
Gucc111:fix/remove-orchestrate-model-overrides
Open

fix(router): remove deprecated autoOrchestrate model override fields#403
Gucc111 wants to merge 1 commit into
OpenBMB:mainfrom
Gucc111:fix/remove-orchestrate-model-overrides

Conversation

@Gucc111

@Gucc111 Gucc111 commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Remove residual autoOrchestrate.mainAgentModel and autoOrchestrate.subagentModel config fields from schema, parser, and runtime
  • These fields were confirmed by the router designer as deprecated — when autoOrchestrate activates, the tier-resolved model is used for the main agent, and subagent requests are independently classified by the tokenSaver judge
  • The parser now emits a deprecation warning (not fatal) for existing configs that still contain these fields, so legacy configs load without errors
  • Also removes the now-unused subagentModelOverride mutation flag and the mainAgentModel field from the UI type

Test plan

  • npx tsc --noEmit passes with zero errors
  • Verify a config with residual autoOrchestrate.mainAgentModel / subagentModel fields loads successfully with deprecation warnings (not fatal)
  • Verify autoOrchestrate still triggers correctly using the tier-resolved model when triggerTiers match
  • Verify subagent requests are still independently classified by the tokenSaver judge

Closes #248

Made with Cursor

…agentModel fields

These were residual config fields that the router designer confirmed are
not intended for user configuration.  When autoOrchestrate activates, the
tier-resolved model is used for the main agent, and subagent requests are
re-classified by the tokenSaver judge — no separate model overrides are
needed.

The parser now emits a deprecation warning (instead of a fatal) when it
encounters these fields in existing user configs, so legacy configs still
load without errors.

Closes OpenBMB#248

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

bug(config): validation misses router.autoOrchestrate main/subagent model refs

1 participant