Skip to content

Update anthropic provider types#334

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

Update anthropic provider types#334
github-actions[bot] wants to merge 1 commit into
mainfrom
update-anthropic-provider-types-2b5468d0-28788427282

Conversation

@github-actions

@github-actions github-actions Bot commented Jul 6, 2026

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: failure

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: 6e97850b52

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

Comment on lines +1716 to 1720
#[serde(rename = "web_search_20260318")]
WebSearch20260318(WebSearchTool20260318),

#[serde(untagged)]
Custom(CustomTool),

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Preserve tool-search tool variants in Anthropic Tool

This regenerated enum now falls through from web_search_20260318 to Custom, so tool_search_tool_regex_20251119/BM25 payloads no longer deserialize because CustomTool requires input_schema. That is still valid according to the OpenAPI in this commit (specs/anthropic/openapi.yml:36689-36690), and the adapter still emits exactly that shape in anthropic_tool_value() (crates/lingua/src/providers/anthropic/adapter.rs:54-63), so transforms or passthrough validation involving tool discovery now fail instead of round-tripping. Please fix the generator to re-add those variants rather than treating the fixture as invalid.

Useful? React with 👍 / 👎.

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 Type response_inclusion as the enum

The imported schema limits response_inclusion to "full" or "excluded" (and this file even generates ResponseInclusion), but this field is Option<String>. As a result validate_anthropic_request and CreateMessageParams will accept arbitrary values on the new 20260318 web fetch/search tools and pass them through until Anthropic rejects the request; wire this field to ResponseInclusion (same issue in WebSearchTool20260318) to keep provider-boundary validation strict.

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