Skip to content

Commit 379caf9

Browse files
committed
feat(ai): Improved error handling upon tool execution faiulre
1 parent 8618e35 commit 379caf9

File tree

3 files changed

+4
-3
lines changed

3 files changed

+4
-3
lines changed

Diff for: arconia-ai/arconia-ai-tools/src/main/java/io/arconia/ai/tools/execution/ToolExecutionException.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ public class ToolExecutionException extends RuntimeException {
1010
private final ToolDefinition toolDefinition;
1111

1212
public ToolExecutionException(ToolDefinition toolDefinition, Throwable cause) {
13-
super(cause.getMessage(), cause);
13+
super("Tool execution error - %s".formatted(cause.getMessage()), cause);
1414
this.toolDefinition = toolDefinition;
1515
}
1616

Diff for: arconia-ai/arconia-ai-tools/src/test/java/io/arconia/ai/tools/execution/ToolExecutionExceptionTests.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ void constructorShouldSetCauseAndMessage() {
1919
ToolExecutionException exception = new ToolExecutionException(mock(ToolDefinition.class), cause);
2020

2121
assertThat(exception.getCause()).isEqualTo(cause);
22-
assertThat(exception.getMessage()).isEqualTo(errorMessage);
22+
assertThat(exception.getMessage()).isEqualTo("Tool execution error - Test error message");
2323
}
2424

2525
@Test

Diff for: arconia-ai/arconia-ai-tools/src/test/java/io/arconia/ai/tools/method/MethodToolCallbackTests.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,8 @@ void shouldHandleMethodExecutionError() {
169169
""");
170170

171171
assertThat(result)
172-
.contains("Test error");
172+
.contains("Tool execution error")
173+
.contains("Test error");
173174
}
174175

175176
@Test

0 commit comments

Comments
 (0)