Skip to content

feat(config): provider requestOptions for request-level params#9029

Open
cyhhao wants to merge 2 commits intoanomalyco:devfrom
cyhhao:fix/openai-store-defaults
Open

feat(config): provider requestOptions for request-level params#9029
cyhhao wants to merge 2 commits intoanomalyco:devfrom
cyhhao:fix/openai-store-defaults

Conversation

@cyhhao
Copy link
Contributor

@cyhhao cyhhao commented Jan 17, 2026

Fixes #9028.

Context

  • store is a per-request OpenAI Responses option; putting it under generic provider.options (SDK init options) is awkward.
  • Introduce a clear separation:
    • provider.<id>.options: SDK init / transport settings (apiKey, baseURL, timeout, ...)
    • provider.<id>.requestOptions: default per-request model options applied to all models in that provider.

Changes

  • Add provider.<id>.requestOptions and plumb into ProviderTransform.options().
  • Apply requestOptions.store only for OpenAI providers (OpenAI still defaults to store=false).
  • Migration: if users had provider.<id>.options.store, it is moved to provider.<id>.requestOptions.store during config load.

Example

{
  "provider": {
    "openai": {
      "requestOptions": { "store": true }
    }
  }
}

@github-actions
Copy link
Contributor

The following comment was made by an LLM, it may be inaccurate:

No duplicate PRs found

@cyhhao cyhhao changed the title feat(config): provider defaults for request options feat(config): provider requestOptions for request-level params Jan 17, 2026
@thdxr thdxr force-pushed the dev branch 3 times, most recently from f1ae801 to 08fa7f7 Compare January 30, 2026 14:37
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.

feat: provider-level default model options (e.g. OpenAI store)

1 participant