Skip to content

Conversation

Fottas
Copy link
Contributor

@Fottas Fottas commented Aug 22, 2025

Replace 4 separate configuration classes with unified McpServerProperties:

  • Consolidate all MCP server config under spring.ai.mcp.server prefix
  • Add nested classes for protocol-specific settings (SSE, Streamable, Stateless)
  • Update configuration paths and test cases accordingly
  • Follow Spring Boot ServerProperties pattern

This eliminates configuration fragmentation and provides single entry point for users.

@Fottas
Copy link
Contributor Author

Fottas commented Aug 25, 2025

@tzolov Hi, I left some comments on #4211 about API issues in the recent fix.

PR #4218 addresses both the NPE with correct API usage and the "Duplicated prefix" warnings we discussed. Would you like me to rebase it?🤗

@Fottas Fottas force-pushed the mcp-server-properties-redesign branch from fe268a5 to 9dcb35e Compare September 1, 2025 02:30
Fottas and others added 3 commits September 2, 2025 20:45
…d structure

Replace 4 separate configuration classes with unified McpServerProperties:

- Consolidate all MCP server config under spring.ai.mcp.server prefix
- Add nested classes for protocol-specific settings (SSE, Streamable, Stateless)
- Update configuration paths and test cases accordingly
- Follow Spring Boot ServerProperties pattern

This eliminates configuration fragmentation and provides single entry point for users.

Signed-off-by: yinh <fottas@163.com>
…method

- Replace deprecated call() method with callHandler() in ToolUtilsTests and McpToolUtils
- Fix NullPointerException in sync/async tool specification tests
- Update MCP API usage to match current implementation

Resolves test failures in spring-ai-mcp module.

Signed-off-by: yinh <fottas@163.com>
Signed-off-by: yinh <fottas@163.com>
@Fottas Fottas force-pushed the mcp-server-properties-redesign branch from 9dcb35e to 624db56 Compare September 2, 2025 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant