Normalize tool keyword argument error labels

Author: crmne

lib/ruby_llm/tool.rb

@@ -133,12 +133,12 @@ def validate_keyword_arguments(arguments)
 
       argument_keys = arguments.keys
       missing_keyword = first_missing_keyword(required_keywords, argument_keys)
-      return "missing keyword: #{keyword_label(missing_keyword)}" if missing_keyword
+      return "missing keyword: #{missing_keyword}" if missing_keyword
       return nil if accepts_extra_keywords
 
       allowed_keywords = required_keywords + optional_keywords
       unknown_keyword = first_unknown_keyword(argument_keys, allowed_keywords)
-      return "unknown keyword: #{keyword_label(unknown_keyword)}" if unknown_keyword
+      return "unknown keyword: #{unknown_keyword}" if unknown_keyword
 
       nil
     end
@@ -160,12 +160,6 @@ def first_unknown_keyword(argument_keys, allowed_keywords)
       (argument_keys - allowed_keywords).first
     end
 
-    def keyword_label(keyword)
-      return keyword.to_s if RUBY_ENGINE == 'jruby'
-
-      ":#{keyword}"
-    end
-
     # Wraps schema handling for tool parameters, supporting JSON Schema hashes,
     # RubyLLM::Schema instances/classes, and DSL blocks.
     class SchemaDefinition

spec/ruby_llm/tool_spec.rb

@@ -52,7 +52,7 @@ def execute(questions:)
 
       result = SignatureTool.new.call({ 'questions' => [], 'isOther' => true })
 
-      expect(result).to eq({ error: 'Invalid tool arguments: unknown keyword: :isOther' })
+      expect(result).to eq({ error: 'Invalid tool arguments: unknown keyword: isOther' })
     end
 
     it 'returns an error hash for missing required keyword arguments' do
@@ -64,12 +64,7 @@ def execute(questions:)
 
       result = RequiredTool.new.call({})
 
-      expected_error = if RUBY_ENGINE == 'jruby'
-                         'Invalid tool arguments: missing keyword: questions'
-                       else
-                         'Invalid tool arguments: missing keyword: :questions'
-                       end
-      expect(result).to eq({ error: expected_error })
+      expect(result).to eq({ error: 'Invalid tool arguments: missing keyword: questions' })
     end
 
     it 'allows extra keyword arguments when execute accepts keyrest' do