Skip to content

Update anthropic provider types#359

Open
github-actions[bot] wants to merge 1 commit into
mainfrom
update-anthropic-provider-types-2c0f63d1-29533300404
Open

Update anthropic provider types#359
github-actions[bot] wants to merge 1 commit into
mainfrom
update-anthropic-provider-types-2c0f63d1-29533300404

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Automated update of Lingua provider types.

Provider: anthropic

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

Validation

  • ./pipelines/generate-provider-types.sh anthropic: failure
  • Claude repair pass: success
  • Retry ./pipelines/generate-provider-types.sh anthropic: 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: e9fbe2190e

ℹ️ 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".

pub name: String,
/// How this tool's result blocks appear in the API response when the result was consumed by a completed code_execution call in the same turn. 'full' returns the complete content (default). 'excluded' drops the nested server_tool_use and result block pair entirely. Results from direct calls, or from code_execution calls that paused before completing, are always returned in full so they can be sent back on the next turn.
#[serde(skip_serializing_if = "Option::is_none")]
pub response_inclusion: Option<String>,

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 Use the enum for response_inclusion

When callers construct or deserialize the new 20260318 web-fetch/web-search tool variants with response_inclusion, this accepts any string and will serialize invalid values that Anthropic rejects; the updated schema only allows full or excluded, and this commit already emits a ResponseInclusion enum for that set. Please wire this field, and the matching WebSearchTool20260318 field, to Option<ResponseInclusion> in the generator instead of Option<String> so invalid requests fail at the typed boundary.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I dunno how big of a deal this is

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