From 5f0963dd346d5dcba2d440a62b242eb94b9b22eb Mon Sep 17 00:00:00 2001 From: Gregory Zak Date: Wed, 4 Mar 2026 16:19:30 -0800 Subject: [PATCH 1/2] fix(mcp): change mcpCheckInput default operation from query to execute Follows axonflow-enterprise#1288. The check-input endpoint is called by external orchestrators managing their own MCP execution, so defaulting to "execute" (conservative) is semantically correct over "query" (read-only). --- src/main/java/com/getaxonflow/sdk/AxonFlow.java | 2 +- .../java/com/getaxonflow/sdk/types/MCPCheckInputRequest.java | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/main/java/com/getaxonflow/sdk/AxonFlow.java b/src/main/java/com/getaxonflow/sdk/AxonFlow.java index 9730814..b7ad8bd 100644 --- a/src/main/java/com/getaxonflow/sdk/AxonFlow.java +++ b/src/main/java/com/getaxonflow/sdk/AxonFlow.java @@ -1504,7 +1504,7 @@ public MCPCheckInputResponse mcpCheckInput(String connectorType, String statemen return retryExecutor.execute(() -> { MCPCheckInputRequest request; if (options != null) { - String operation = (String) options.getOrDefault("operation", "query"); + String operation = (String) options.getOrDefault("operation", "execute"); @SuppressWarnings("unchecked") Map parameters = (Map) options.get("parameters"); request = new MCPCheckInputRequest(connectorType, statement, parameters, operation); diff --git a/src/main/java/com/getaxonflow/sdk/types/MCPCheckInputRequest.java b/src/main/java/com/getaxonflow/sdk/types/MCPCheckInputRequest.java index 4c4cc07..2511d0e 100644 --- a/src/main/java/com/getaxonflow/sdk/types/MCPCheckInputRequest.java +++ b/src/main/java/com/getaxonflow/sdk/types/MCPCheckInputRequest.java @@ -44,13 +44,13 @@ public final class MCPCheckInputRequest { /** * Creates a request with connector type and statement only. - * Operation defaults to "query". + * Operation defaults to "execute". * * @param connectorType the MCP connector type (e.g., "postgres") * @param statement the statement to validate */ public MCPCheckInputRequest(String connectorType, String statement) { - this(connectorType, statement, null, "query"); + this(connectorType, statement, null, "execute"); } /** From 91a8028efa3d527bb9e5859cfcade70113ba848c Mon Sep 17 00:00:00 2001 From: Saurabh Jain Date: Sat, 7 Mar 2026 23:54:05 +0530 Subject: [PATCH 2/2] fix(test): update default operation assertion from query to execute --- src/test/java/com/getaxonflow/sdk/types/MoreTypesTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/test/java/com/getaxonflow/sdk/types/MoreTypesTest.java b/src/test/java/com/getaxonflow/sdk/types/MoreTypesTest.java index 3f6082c..bc17512 100644 --- a/src/test/java/com/getaxonflow/sdk/types/MoreTypesTest.java +++ b/src/test/java/com/getaxonflow/sdk/types/MoreTypesTest.java @@ -1527,7 +1527,7 @@ void shouldCreateWithBasicFields() { assertThat(request.getConnectorType()).isEqualTo("postgres"); assertThat(request.getStatement()).isEqualTo("SELECT * FROM users"); - assertThat(request.getOperation()).isEqualTo("query"); + assertThat(request.getOperation()).isEqualTo("execute"); assertThat(request.getParameters()).isNull(); }