Skip to content

Commit 5c13299

Browse files
committed
refactor(server): Remove deprecated methods in v0.2.0
1 parent 2c23af7 commit 5c13299

File tree

2 files changed

+22
-40
lines changed

2 files changed

+22
-40
lines changed

src/main/java/com/github/codeboyzhou/mcp/declarative/McpServers.java

Lines changed: 2 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,6 @@
1616
import io.modelcontextprotocol.spec.McpServerTransportProvider;
1717
import org.reflections.Reflections;
1818

19-
import static io.modelcontextprotocol.server.transport.HttpServletSseServerTransportProvider.DEFAULT_BASE_URL;
20-
import static io.modelcontextprotocol.server.transport.HttpServletSseServerTransportProvider.DEFAULT_SSE_ENDPOINT;
21-
2219
public class McpServers {
2320

2421
private static final McpServers INSTANCE = new McpServers();
@@ -57,9 +54,8 @@ public void startSyncStdioServer(String name, String version, String instruction
5754
McpServerComponentRegisters.registerAllTo(server, reflections);
5855
}
5956

60-
@Deprecated(since = "0.2.0")
61-
public void startSyncStdioServer(String name, String version) {
62-
startSyncStdioServer(name, version, "You are using a deprecated API with default server instructions");
57+
public void startSyncStdioServer(McpServerInfo serverInfo) {
58+
startSyncStdioServer(serverInfo.name(), serverInfo.version(), serverInfo.instructions());
6359
}
6460

6561
public void startSyncSseServer(McpSseServerInfo serverInfo, McpHttpServerStatusListener<McpSyncServer> listener) {
@@ -77,34 +73,4 @@ public void startSyncSseServer(McpSseServerInfo serverInfo) {
7773
startSyncSseServer(serverInfo, new DefaultMcpSyncHttpServerStatusListener());
7874
}
7975

80-
@Deprecated(since = "0.2.0")
81-
public void startSyncSseServer(String name, String version, String messageEndpoint, String sseEndpoint, int port) {
82-
McpSseServerInfo serverInfo = McpSseServerInfo.builder().name(name).version(version)
83-
.instructions("You are using a deprecated API with default server instructions")
84-
.baseUrl(DEFAULT_BASE_URL).messageEndpoint(messageEndpoint)
85-
.sseEndpoint(sseEndpoint).port(port)
86-
.build();
87-
startSyncSseServer(serverInfo);
88-
}
89-
90-
@Deprecated(since = "0.2.0")
91-
public void startSyncSseServer(String name, String version, int port) {
92-
McpSseServerInfo serverInfo = McpSseServerInfo.builder().name(name).version(version)
93-
.instructions("You are using a deprecated API with default server instructions")
94-
.baseUrl(DEFAULT_BASE_URL).messageEndpoint(DEFAULT_MESSAGE_ENDPOINT)
95-
.sseEndpoint(DEFAULT_SSE_ENDPOINT).port(port)
96-
.build();
97-
startSyncSseServer(serverInfo);
98-
}
99-
100-
@Deprecated(since = "0.2.0")
101-
public void startSyncSseServer(String name, String version) {
102-
McpSseServerInfo serverInfo = McpSseServerInfo.builder().name(name).version(version)
103-
.instructions("You are using a deprecated API with default server instructions")
104-
.baseUrl(DEFAULT_BASE_URL).messageEndpoint(DEFAULT_MESSAGE_ENDPOINT)
105-
.sseEndpoint(DEFAULT_SSE_ENDPOINT).port(DEFAULT_HTTP_SERVER_PORT)
106-
.build();
107-
startSyncSseServer(serverInfo);
108-
}
109-
11076
}

src/test/java/com/github/codeboyzhou/mcp/declarative/McpServersTest.java

Lines changed: 20 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
package com.github.codeboyzhou.mcp.declarative;
22

33
import com.github.codeboyzhou.mcp.declarative.annotation.McpTools;
4+
import com.github.codeboyzhou.mcp.declarative.server.McpServerInfo;
5+
import com.github.codeboyzhou.mcp.declarative.server.McpSseServerInfo;
46
import com.github.codeboyzhou.mcp.declarative.server.TestMcpComponentScanBasePackageClass;
57
import com.github.codeboyzhou.mcp.declarative.server.TestMcpComponentScanBasePackageString;
68
import com.github.codeboyzhou.mcp.declarative.server.TestMcpComponentScanDefault;
@@ -53,17 +55,31 @@ void testRun(Class<?> applicationMainClass) {
5355
void testStartSyncStdioServer() {
5456
assertDoesNotThrow(() -> {
5557
McpServers servers = McpServers.run(TestMcpComponentScanIsNull.class, EMPTY_ARGS);
56-
servers.startSyncStdioServer("test-mcp-sync-stdio-server", "1.0.0");
58+
McpServerInfo serverInfo = McpServerInfo.builder()
59+
.instructions("test-mcp-sync-stdio-server-instructions")
60+
.name("test-mcp-sync-stdio-server")
61+
.version("1.0.0")
62+
.build();
63+
servers.startSyncStdioServer(serverInfo);
5764
});
5865
}
5966

6067
@Test
6168
void testStartSyncSseServer() {
6269
System.setProperty("mcp.declarative.java.sdk.testing", "true");
6370
McpServers servers = McpServers.run(TestMcpComponentScanIsNull.class, EMPTY_ARGS);
64-
assertDoesNotThrow(() -> servers.startSyncSseServer("test-mcp-sync-sse-server", "1.0.0"));
65-
assertDoesNotThrow(() -> servers.startSyncSseServer("test-mcp-sync-sse-server", "1.0.0", 9118));
66-
assertDoesNotThrow(() -> servers.startSyncSseServer("test-mcp-sync-sse-server", "1.0.0", "/message", "/sse", 9119));
71+
assertDoesNotThrow(() -> {
72+
McpSseServerInfo serverInfo = McpSseServerInfo.builder()
73+
.instructions("test-mcp-sync-sse-server-instructions")
74+
.baseUrl("http://127.0.0.1:8080")
75+
.messageEndpoint("/message")
76+
.sseEndpoint("/sse")
77+
.port(8080)
78+
.name("test-mcp-sync-sse-server")
79+
.version("1.0.0")
80+
.build();
81+
servers.startSyncSseServer(serverInfo);
82+
});
6783
}
6884

6985
private Reflections getReflectionsField() throws NoSuchFieldException, IllegalAccessException {

0 commit comments

Comments
 (0)