Skip to content

Conversation

@quaff
Copy link
Contributor

@quaff quaff commented Aug 21, 2025

To align with other MCP related configuration properties bean.

@quaff quaff changed the title Move McpServerProperties to module spring-ai-autoconfigure-mcp-server-properties Polish MCP related configuration properties bean Aug 22, 2025
@ilayaperumalg ilayaperumalg self-assigned this Sep 10, 2025
@ilayaperumalg ilayaperumalg added this to the 1.1.0.M2 milestone Sep 10, 2025
* @author Christian Tzolov
* @since 1.0.0
* @see McpServerAutoConfiguration
* @see org.springframework.ai.mcp.server.common.autoconfigure.McpServerAutoConfiguration
Copy link
Member

Choose a reason for hiding this comment

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

When merging, I will update the latest autoconfiguration changes which use McpServerProperties:

 * @see org.springframework.ai.mcp.server.common.autoconfigure.McpServerAutoConfiguration
 * @see org.springframework.ai.mcp.server.common.autoconfigure.McpServerStatelessAutoConfiguration
 * @see org.springframework.ai.mcp.server.common.autoconfigure.StatelessToolCallbackConverterAutoConfiguration
 * @see org.springframework.ai.mcp.server.common.autoconfigure.ToolCallbackConverterAutoConfiguration


/**
* @author Christian Tzolov
* @see org.springframework.ai.mcp.server.stateless.webflux.autoconfigure.McpServerStatelessWebFluxAutoConfiguration
Copy link
Member

Choose a reason for hiding this comment

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

These auto-configurations are now moved into separate auto-configurations spring-ai-autoconfigure-mcp-server-webflux and spring-ai-autoconfigure-mcp-server-webmvc with the common properties and auto-configurations exist in spring-ai-autoconfigure-mcp-server-common. Hence, I will remove these entries from here when merging.

@ilayaperumalg
Copy link
Member

@quaff Thanks for the PR. Rebased and updated the changes discussed in the review comments, merged as 4889131

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