Merge pull request #55 from trandbert37/feature/add-all-supported-tool-types

Add all supported tool types

Author: klapaudius

src/Command/MigrateToolSchemaCommand.php

@@ -169,6 +169,10 @@ private function mapJsonSchemaTypeToPropertyType(string $jsonType): string
         return match ($jsonType) {
             'string' => 'STRING',
             'integer' => 'INTEGER',
+            'array' => 'ARRAY',
+            'boolean' => 'BOOLEAN',
+            'object' => 'OBJECT',
+            'number' => 'NUMBER',
             default => 'STRING', // Fallback for unsupported types
         };
     }

src/Services/ToolService/Schema/PropertyType.php

@@ -6,4 +6,8 @@ enum PropertyType
 {
     case STRING;
     case INTEGER;
+    case NUMBER;
+    case OBJECT;
+    case ARRAY;
+    case BOOLEAN;
 }

src/Services/ToolService/Schema/StructuredSchema.php

@@ -77,6 +77,10 @@ private function getPropertyType(SchemaProperty $property): string
         return match ($property->getType()) {
             PropertyType::STRING => 'string',
             PropertyType::INTEGER => 'integer',
+            PropertyType::ARRAY => 'array',
+            PropertyType::BOOLEAN => 'boolean',
+            PropertyType::OBJECT => 'object',
+            PropertyType::NUMBER => 'number',
         };
     }
 }

src/Services/ToolService/ToolParamsValidator.php

@@ -105,6 +105,7 @@ private static function validateType(string $expectedType, mixed $actualValue):
             'boolean' => is_bool($actualValue),
             'array' => is_array($actualValue),
             'object' => is_object($actualValue),
+            'number' => is_numeric($actualValue),
             default => false
         };
     }

tests/Command/MigrateToolSchemaCommandTest.php

@@ -198,8 +198,11 @@ public function test_map_json_schema_type_to_property_type(): void
 
         $this->assertEquals('STRING', $method->invoke($command, 'string'));
         $this->assertEquals('INTEGER', $method->invoke($command, 'integer'));
+        $this->assertEquals('ARRAY', $method->invoke($command, 'array'));
+        $this->assertEquals('BOOLEAN', $method->invoke($command, 'boolean'));
+        $this->assertEquals('OBJECT', $method->invoke($command, 'object'));
+        $this->assertEquals('NUMBER', $method->invoke($command, 'number'));
         $this->assertEquals('STRING', $method->invoke($command, 'unknown')); // Fallback
-        $this->assertEquals('STRING', $method->invoke($command, 'boolean')); // Fallback
     }
 
     public function test_generate_use_statements_with_existing_statements(): void

tests/Services/ToolService/Schema/PropertyTypeTest.php

@@ -12,7 +12,7 @@ class PropertyTypeTest extends TestCase
     public function test_enum_cases_exist(): void
     {
         $this->assertIsArray(PropertyType::cases());
-        $this->assertCount(2, PropertyType::cases());
+        $this->assertCount(6, PropertyType::cases());
     }
 
     public function test_string_case_exists(): void
@@ -29,7 +29,7 @@ public function test_integer_case_exists(): void
 
     public function test_all_cases_have_correct_names(): void
     {
-        $expectedCases = ['STRING', 'INTEGER'];
+        $expectedCases = ['STRING', 'INTEGER', 'NUMBER', 'OBJECT', 'ARRAY', 'BOOLEAN'];
         $actualCases = array_map(fn ($case) => $case->name, PropertyType::cases());
 
         $this->assertEquals($expectedCases, $actualCases);
@@ -38,7 +38,7 @@ public function test_all_cases_have_correct_names(): void
     public function test_enum_values_are_unique(): void
     {
         $cases = PropertyType::cases();
-        $this->assertCount(2, $cases);
+        $this->assertCount(6, $cases);
         $this->assertNotEquals($cases[0], $cases[1]);
     }