This is an automated email from the ASF dual-hosted git repository. rombert pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-mcp-server.git
commit c5719fea956977f8864bff59c37f87b7d1143064 Author: Robert Munteanu <[email protected]> AuthorDate: Thu Dec 11 14:32:07 2025 +0100 chore(mcp): stop using deprecated apis --- .../mcp/server/impl/contribs/LogToolContribution.java | 16 ++++++++++------ .../impl/contribs/OsgiBundleDiagnosticContribution.java | 9 ++++++--- 2 files changed, 16 insertions(+), 9 deletions(-) diff --git a/src/main/java/org/apache/sling/mcp/server/impl/contribs/LogToolContribution.java b/src/main/java/org/apache/sling/mcp/server/impl/contribs/LogToolContribution.java index 9e68661..37fe051 100644 --- a/src/main/java/org/apache/sling/mcp/server/impl/contribs/LogToolContribution.java +++ b/src/main/java/org/apache/sling/mcp/server/impl/contribs/LogToolContribution.java @@ -110,10 +110,11 @@ public class LogToolContribution implements McpServerContribution { if (logLevelStr != null && !logLevelStr.isEmpty()) { minLogLevel = parseLogLevel(logLevelStr); if (minLogLevel == -1) { - return new CallToolResult( - "Invalid log level: " + logLevelStr - + ". Valid options are: ERROR, WARN, INFO, DEBUG, TRACE", - Boolean.TRUE); + return CallToolResult.builder() + .addTextContent("Invalid log level: " + logLevelStr + + ". Valid options are: ERROR, WARN, INFO, DEBUG, TRACE") + .isError(true) + .build(); } } @@ -123,7 +124,10 @@ public class LogToolContribution implements McpServerContribution { try { pattern = Pattern.compile(regexPattern, Pattern.CASE_INSENSITIVE); } catch (PatternSyntaxException e) { - return new CallToolResult("Invalid regex pattern: " + e.getMessage(), Boolean.TRUE); + return CallToolResult.builder() + .addTextContent("Invalid regex pattern: " + e.getMessage()) + .isError(true) + .build(); } } @@ -133,7 +137,7 @@ public class LogToolContribution implements McpServerContribution { // Format output String result = formatLogs(filteredLogs, regexPattern, minLogLevel, maxEntries); - return new CallToolResult(result, Boolean.FALSE); + return CallToolResult.builder().addTextContent(result).build(); })); } diff --git a/src/main/java/org/apache/sling/mcp/server/impl/contribs/OsgiBundleDiagnosticContribution.java b/src/main/java/org/apache/sling/mcp/server/impl/contribs/OsgiBundleDiagnosticContribution.java index a4affef..4efaf3d 100644 --- a/src/main/java/org/apache/sling/mcp/server/impl/contribs/OsgiBundleDiagnosticContribution.java +++ b/src/main/java/org/apache/sling/mcp/server/impl/contribs/OsgiBundleDiagnosticContribution.java @@ -105,7 +105,10 @@ public class OsgiBundleDiagnosticContribution implements McpServerContribution { private CallToolResult diagnoseSpecificBundle(String symbolicName) { Bundle bundle = findBundle(symbolicName); if (bundle == null) { - return new CallToolResult("Bundle '" + symbolicName + "' not found.", Boolean.TRUE); + return CallToolResult.builder() + .addTextContent("Bundle '" + symbolicName + "' not found.") + .isError(true) + .build(); } StringBuilder result = new StringBuilder(); @@ -124,7 +127,7 @@ public class OsgiBundleDiagnosticContribution implements McpServerContribution { appendComponentDiagnostics(bundle, result); } - return new CallToolResult(result.toString(), Boolean.FALSE); + return CallToolResult.builder().addTextContent(result.toString()).build(); } private CallToolResult diagnoseAllProblematicBundles() { @@ -152,7 +155,7 @@ public class OsgiBundleDiagnosticContribution implements McpServerContribution { // Check for components with issues appendAllComponentIssues(result); - return new CallToolResult(result.toString(), Boolean.FALSE); + return CallToolResult.builder().addTextContent(result.toString()).build(); } private void analyzeBundleIssues(Bundle bundle, StringBuilder result) {
