Skip to content

Remove internal tool execution from MistralAiChatModel#6260

Closed
tzolov wants to merge 1 commit into
spring-projects:mainfrom
tzolov:remove-mistral-internal-tool-execution
Closed

Remove internal tool execution from MistralAiChatModel#6260
tzolov wants to merge 1 commit into
spring-projects:mainfrom
tzolov:remove-mistral-internal-tool-execution

Conversation

@tzolov
Copy link
Copy Markdown
Contributor

@tzolov tzolov commented Jun 2, 2026

  • Drop the built-in call/stream tool-execution loop and the ToolExecutionEligibilityChecker wiring from MistralAiChatModel and its autoconfiguration.
  • Tool execution is now handled externally via ToolCallAdvisor (ChatClient) or the user-controlled DefaultToolCallingManager loop.
  • Update all related integration tests to use ChatClient with ToolCallAdvisor, and convert streamFunctionCallUsageTest to the user-controlled streaming pattern with MessageAggregator.
  • Fix PaymentStatusBeanOpenAiIT base URL (add /v1 suffix required by the OpenAI Java SDK).
  • Update mistralai-chat.adoc: remove internalToolExecutionEnabled config property, add Tool Calling via ChatClient and User-Controlled Tool Execution sections.

Part of #6251

- Drop the built-in call/stream tool-execution loop and the
  ToolExecutionEligibilityChecker wiring from MistralAiChatModel
  and its autoconfiguration.
- Tool execution is now handled externally via ToolCallAdvisor
  (ChatClient) or the user-controlled DefaultToolCallingManager loop.
- Update all related integration tests to use ChatClient with
  ToolCallAdvisor, and convert streamFunctionCallUsageTest to the
  user-controlled streaming pattern with MessageAggregator.
- Fix PaymentStatusBeanOpenAiIT base URL (add /v1 suffix required
  by the OpenAI Java SDK).
- Update mistralai-chat.adoc: remove internalToolExecutionEnabled
  config property, add Tool Calling via ChatClient and
  User-Controlled Tool Execution sections.

Part of spring-projects#6251

Signed-off-by: Christian Tzolov <christian.tzolov@broadcom.com>
@ilayaperumalg
Copy link
Copy Markdown
Member

Rebased and merged via 2b36875

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.

2 participants