Skip to content

Commit dcad46c

Browse files
fix(mcp): change mcpCheckInput default operation from query to execute (#104)
* 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). * fix(test): update default operation assertion from query to execute --------- Co-authored-by: Saurabh Jain <saurabhjain1592@gmail.com>
1 parent 9756987 commit dcad46c

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

src/main/java/com/getaxonflow/sdk/AxonFlow.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1504,7 +1504,7 @@ public MCPCheckInputResponse mcpCheckInput(String connectorType, String statemen
15041504
return retryExecutor.execute(() -> {
15051505
MCPCheckInputRequest request;
15061506
if (options != null) {
1507-
String operation = (String) options.getOrDefault("operation", "query");
1507+
String operation = (String) options.getOrDefault("operation", "execute");
15081508
@SuppressWarnings("unchecked")
15091509
Map<String, Object> parameters = (Map<String, Object>) options.get("parameters");
15101510
request = new MCPCheckInputRequest(connectorType, statement, parameters, operation);

src/main/java/com/getaxonflow/sdk/types/MCPCheckInputRequest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,13 +44,13 @@ public final class MCPCheckInputRequest {
4444

4545
/**
4646
* Creates a request with connector type and statement only.
47-
* Operation defaults to "query".
47+
* Operation defaults to "execute".
4848
*
4949
* @param connectorType the MCP connector type (e.g., "postgres")
5050
* @param statement the statement to validate
5151
*/
5252
public MCPCheckInputRequest(String connectorType, String statement) {
53-
this(connectorType, statement, null, "query");
53+
this(connectorType, statement, null, "execute");
5454
}
5555

5656
/**

src/test/java/com/getaxonflow/sdk/types/MoreTypesTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1527,7 +1527,7 @@ void shouldCreateWithBasicFields() {
15271527

15281528
assertThat(request.getConnectorType()).isEqualTo("postgres");
15291529
assertThat(request.getStatement()).isEqualTo("SELECT * FROM users");
1530-
assertThat(request.getOperation()).isEqualTo("query");
1530+
assertThat(request.getOperation()).isEqualTo("execute");
15311531
assertThat(request.getParameters()).isNull();
15321532
}
15331533

0 commit comments

Comments
 (0)