Skip to content

cli: warn about unsupported OpenAPI server overrides - #7623

Open
ChimdumebiNebolisa wants to merge 2 commits into
OpenBB-finance:v5from
ChimdumebiNebolisa:codex/issue-7609-server-warnings
Open

cli: warn about unsupported OpenAPI server overrides#7623
ChimdumebiNebolisa wants to merge 2 commits into
OpenBB-finance:v5from
ChimdumebiNebolisa:codex/issue-7609-server-warnings

Conversation

@ChimdumebiNebolisa

@ChimdumebiNebolisa ChimdumebiNebolisa commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Partially addresses #7609.

  • Warns when concrete absolute path-item or operation-level servers URLs differ from the global base URL used by generated commands.
  • Preserves operation > path-item > root precedence.
  • Skips relative URLs and URLs containing unresolved server variables.
  • Does not implement per-command routing or change the .spec document format.
  • 92 focused tests passed locally; Ruff checks passed.
  • ty was unavailable locally, so CI validation remains pending.

Does this warning-only scope seem appropriate as an interim fix?

@ChimdumebiNebolisa
ChimdumebiNebolisa marked this pull request as ready for review August 2, 2026 20:54
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