fix: skip parameters with default as required in JsonSchema (#288)

Author: lvluoyue

src/Chain/JsonSchema/Factory.php

@@ -98,7 +98,7 @@ private function convertTypes(array $elements): ?array
 
             if ($type->isNullable()) {
                 $schema['type'] = [$schema['type'], 'null'];
-            } else {
+            } elseif (!($element instanceof \ReflectionParameter && $element->isOptional())) {
                 $result['required'][] = $name;
             }
 

tests/Chain/JsonSchema/FactoryTest.php

@@ -153,7 +153,7 @@ public function buildParametersDefinitionOptional(): void
                     'description' => 'A number given to the tool',
                 ],
             ],
-            'required' => ['text', 'number'],
+            'required' => ['text'],
             'additionalProperties' => false,
         ];
 

tests/Chain/Toolbox/ToolboxTest.php

@@ -100,7 +100,7 @@ public function toolsMap(): void
                                 'description' => 'A number given to the tool',
                             ],
                         ],
-                        'required' => ['text', 'number'],
+                        'required' => ['text'],
                         'additionalProperties' => false,
                     ],
                 ],
@@ -249,7 +249,7 @@ public function toolboxMapWithOverrideViaChain(): void
                                 'description' => 'A number given to the tool',
                             ],
                         ],
-                        'required' => ['text', 'number'],
+                        'required' => ['text'],
                         'additionalProperties' => false,
                     ],
                 ],