Merge pull request #58 from trandbert37/feature/use-backed-enum-for-property-type

Use backed enum for property types

Author: klapaudius

src/Services/ToolService/Schema/PropertyType.php

@@ -2,12 +2,12 @@
 
 namespace KLP\KlpMcpServer\Services\ToolService\Schema;
 
-enum PropertyType
+enum PropertyType: string
 {
-    case STRING;
-    case INTEGER;
-    case NUMBER;
-    case OBJECT;
-    case ARRAY;
-    case BOOLEAN;
+    case STRING = 'string';
+    case INTEGER  = 'integer';
+    case NUMBER  = 'number';
+    case OBJECT = 'object';
+    case ARRAY  = 'array';
+    case BOOLEAN = 'boolean';
 }

src/Services/ToolService/Schema/StructuredSchema.php

@@ -56,7 +56,7 @@ public function asArray(): array
 
         foreach ($this->properties as $property) {
             $propertySchema = [
-                'type' => $this->getPropertyType($property),
+                'type' => $property->getType()->value,
                 'description' => $property->getDescription(),
             ];
 
@@ -98,16 +98,4 @@ public function asArray(): array
             'required' => $required,
         ];
     }
-
-    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',
-        };
-    }
 }