Skip to content

Update openai provider types#326

Open
github-actions[bot] wants to merge 1 commit into
mainfrom
update-openai-provider-types-e061090f-28369353791
Open

Update openai provider types#326
github-actions[bot] wants to merge 1 commit into
mainfrom
update-openai-provider-types-e061090f-28369353791

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Automated update of Lingua provider types.

Provider: openai

Feedback: comment /bt good or /bt bad to log review feedback to the Braintrust trace.

Validation

  • ./pipelines/generate-provider-types.sh openai: success
  • make generate-types: success
  • Braintrust workflow trace: success
  • Claude integration review: success
  • Post-review make generate-types: success
  • cargo fmt --all: success
  • cargo build --all-features: success
  • cargo clippy --all-targets --all-features -- -D warnings: success
  • make typed-boundary-check: success
  • cargo test -p coverage-report --test cross_provider_test cross_provider_transformations_have_no_unexpected_failures: success

This PR is created even when validation fails so the generated update and repair attempt are available for manual follow-up.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 13f1efb98a

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "Codex (@codex) review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "Codex (@codex) address that feedback".

#[derive(Debug, Clone, Default, Serialize, Deserialize)]
pub struct OpenAIReasoning {
#[serde(skip_serializing_if = "Option::is_none")]
pub context: Option<ReasoningContext>,

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Avoid enabling reasoning for context-only requests

When a Responses request contains only reasoning.context (allowed by the updated schema because effort, summary, and generate_summary are all optional), this new field makes typed_params.reasoning present, and request_to_universal converts any present OpenAIReasoning into ReasoningConfig { enabled: Some(true), ... } even though context is not represented there. If that universal request is then emitted to Chat Completions, Anthropic, or Google, the provider conversion synthesizes a default reasoning effort/budget, changing a context-selection-only request into one that explicitly enables reasoning; only create canonical reasoning when an effort/summary field is present, or model context separately.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant