Skip to content

[9.1] [ML][AI Connector] Anthropic Connector: ensure max tokens parameter is passed as expected by service (#241188)#241207

Merged
kibanamachine merged 2 commits intoelastic:9.1from
kibanamachine:backport/9.1/pr-241188
Oct 30, 2025
Merged

[9.1] [ML][AI Connector] Anthropic Connector: ensure max tokens parameter is passed as expected by service (#241188)#241207
kibanamachine merged 2 commits intoelastic:9.1from
kibanamachine:backport/9.1/pr-241188

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 9.1:

Questions ?

Please refer to the Backport tool documentation

…s passed as expected by service (elastic#241188)

## Summary

Fixes elastic#241142

This is a temporary workaround for anthropic max_tokens handling until
the services endpoint is updated to reflect the correct structure.
Anthropic is unique in that it requires max_tokens to be sent as part of
the task_settings instead of the usual service_settings.
Until the services endpoint is updated to reflect that, there is no way
for the form UI to know where to put max_tokens. This can be removed
once that update is made.

https://github.com/user-attachments/assets/e89e7fb2-5509-45c6-9f43-deccb0873267

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [ ] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [ ]
[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added for features that require explanation or tutorials
- [ ] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [ ] If a plugin configuration key changed, check if it needs to be
allowlisted in the cloud and added to the [docker
list](https://github.com/elastic/kibana/blob/main/src/dev/build/tasks/os_packages/docker_generator/resources/base/bin/kibana-docker)
- [ ] This was checked for breaking HTTP API changes, and any breaking
changes have been approved by the breaking-change committee. The
`release_note:breaking` label should be applied in these situations.
- [ ] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed
- [ ] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [ ] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

(cherry picked from commit 0a8c77d)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Oct 29, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) October 29, 2025 21:13
@elasticmachine
Copy link
Contributor

elasticmachine commented Oct 29, 2025

💔 Build Failed

Failed CI Steps

History

cc @alvarezmelissa87

@kibanamachine kibanamachine merged commit 465dd96 into elastic:9.1 Oct 30, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

3 participants