Skip to content

Conversation

nnam-droid12
Copy link

Description

Fix for #4494
(raised by @Kehrlann)

This PR refactors the Azure OpenAI auto-configuration module to align with Spring Boot’s recommended practices and improve maintainability and configurability.

Changes

  • Removed unnecessary @ImportAutoConfiguration annotation from AzureOpenAiChatAutoConfiguration.

  • Ensures that users can exclude ToolCallingAutoConfiguration independently when using Azure OpenAI configuration.

  • Maintains proper ordering with @autoConfiguration(after = { ToolCallingAutoConfiguration.class }).

@ielatif
Copy link
Contributor

ielatif commented Oct 16, 2025

@nnam-droid12 now that you removed @ImportAutoConfiguration, you have to fix unit and integration tests by adding ToolCallingAutoConfiguration.class to contextRunners ApplicationContextRunner. See other PRs

@ilayaperumalg ilayaperumalg added this to the 1.1.0.M4 milestone Oct 16, 2025
@nnam-droid12
Copy link
Author

Thanks @ielatif for the feedback!
I’ve updated the tests to include ToolCallingAutoConfiguration.class in the ApplicationContextRunner

@ilayaperumalg ilayaperumalg self-assigned this Oct 16, 2025
@@ -1,19 +1,3 @@
/*
Copy link
Member

Choose a reason for hiding this comment

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

We should keep the copyright header. Will add it back when merging.

Copy link
Author

Choose a reason for hiding this comment

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

Thanks @ilayaperumalg I appreciate the review, noted on the copyright header.

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.

3 participants