fix: better fix of AnthropicTool error reporting

xemantic · Oct 15, 2024 · 2f02b61 · 2f02b61
1 parent baf3d1c
commit 2f02b61
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/commonMain/kotlin/tool/Tools.kt b/src/commonMain/kotlin/tool/Tools.kt
@@ -63,7 +63,10 @@ inline fun <reified T : UsableTool> toolOf(
     serializer<T>()
   } catch (e: SerializationException) {
     throw SerializationException(
-      nonAnnotatedToolMessage<T>(), e
+      "Cannot find serializer for class ${T::class.qualifiedName}, " +
+          "make sure that it is annotated with @AnthropicTool and " +
+          "kotlin.serialization plugin is enabled for the project",
+      e
     )
   }
 
@@ -72,7 +75,7 @@ inline fun <reified T : UsableTool> toolOf(
     .annotations
     .filterIsInstance<AnthropicTool>()
     .firstOrNull() ?: throw SerializationException(
-      nonAnnotatedToolMessage<T>()
+      "The class ${T::class.qualifiedName} must be annotated with @AnthropicTool"
     )
 
   return Tool(
@@ -82,7 +85,3 @@ inline fun <reified T : UsableTool> toolOf(
     cacheControl = cacheControl
   )
 }
-
-@PublishedApi
-internal inline fun <reified T> nonAnnotatedToolMessage() =
-  "The class ${T::class.qualifiedName} must be annotated with @AnthropicTool"
diff --git a/src/commonTest/kotlin/tool/UsableToolTest.kt b/src/commonTest/kotlin/tool/UsableToolTest.kt
@@ -68,7 +68,8 @@ class UsableToolTest {
   @Test
   fun shouldFailToCreateToolWithoutAnthropicToolAnnotation() {
     shouldThrowWithMessage<SerializationException>(
-      "The class com.xemantic.anthropic.tool.UsableToolTest.NoAnnotationTool must be annotated with @AnthropicTool"
+      "Cannot find serializer for class com.xemantic.anthropic.tool.UsableToolTest.NoAnnotationTool, " +
+          "make sure that it is annotated with @AnthropicTool and kotlin.serialization plugin is enabled for the project"
     ) {
       toolOf<NoAnnotationTool>()
     }