Changed: use tool_names::EDIT constant in edit_error_to_serdes

Replace hardcoded 'edit' strings with tool_names::EDIT constant for consistency.
This centralizes tool naming and ensures all tool names are defined in a single location.

Author: Sewer56

src/llm-coding-tools-serdesai/src/convert.rs

@@ -6,6 +6,7 @@
 //! [`llm_coding_tools_core`]: llm_coding_tools_core
 
 use llm_coding_tools_core::operations::EditError;
+use llm_coding_tools_core::tool_names;
 use llm_coding_tools_core::{ToolError as CoreError, ToolOutput, ToolResult as CoreResult};
 use serde_json::json;
 use serdes_ai::tools::{ToolError as SerdesError, ToolReturn};
@@ -68,28 +69,28 @@ pub fn to_serdes_result(
 pub fn edit_error_to_serdes(err: EditError) -> SerdesError {
     match err {
         EditError::NotFound => SerdesError::validation_error(
-            "edit",
+            tool_names::EDIT,
             Some("old_string".to_string()),
             "old_string not found in file content".to_string(),
         ),
         EditError::AmbiguousMatch(count) => SerdesError::validation_error(
-            "edit",
+            tool_names::EDIT,
             Some("old_string".to_string()),
             format!(
                 "old_string found {count} times and requires more code context to uniquely identify the intended match"
             ),
         ),
         EditError::EmptyOldString => SerdesError::validation_error(
-            "edit",
+            tool_names::EDIT,
             Some("old_string".to_string()),
             "old_string must not be empty".to_string(),
         ),
         EditError::IdenticalStrings => SerdesError::validation_error(
-            "edit",
+            tool_names::EDIT,
             None,
             "old_string and new_string must be different".to_string(),
         ),
-        EditError::Tool(tool_err) => core_error_to_serdes("edit", tool_err),
+        EditError::Tool(tool_err) => core_error_to_serdes(tool_names::EDIT, tool_err),
     }
 }