Skip to content

fix(config): validate all model providers#185

Open
MicroMilo wants to merge 1 commit into
OpenBMB:mainfrom
MicroMilo:codex/fix-unused-provider-validation
Open

fix(config): validate all model providers#185
MicroMilo wants to merge 1 commit into
OpenBMB:mainfrom
MicroMilo:codex/fix-unused-provider-validation

Conversation

@MicroMilo

Copy link
Copy Markdown

Summary

  • validate the base shape of every configured model provider, not only the active agent provider
  • keep active-provider protocol/url requirements on the active path
  • add a focused server-side regression test for an unused provider missing apiKey

Verification

  • npm --workspace ui exec vitest run server/services/pilotdeckConfig.provider-validation.test.js
  • npm exec -- tsc --noEmit -p tsconfig.json

Notes

This is a real config/runtime mismatch: the UI save path previously allowed invalid unused providers, while the runtime parses all providers during reload.

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