bug #47 fix: recursively remove additionalProperties from Gemini requests (valtzu)

This PR was merged into the main branch.

Discussion
----------

fix: recursively remove `additionalProperties` from Gemini requests

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| Docs?         | no
| Issues        |
| License       | MIT

Cherry picking php-llm/llm-chain#351

> Tool parameters with nested objects cause 400 bad request from Gemini due to only removing the unsupported `additionalProperties` on top-level.

```
Invalid JSON payload received. Unknown name "additionalProperties" at 'tools[0].function_declarations[0].parameters.properties[0].value': Cannot find field.
```

Commits
-------

ea94f84 fix: recursively remove `additionalProperties` from Gemini requests (#351)

Author: OskarStark

src/platform/src/Bridge/Google/Contract/ToolNormalizer.php

@@ -47,17 +47,34 @@ protected function supportsModel(Model $model): bool
      */
     public function normalize(mixed $data, ?string $format = null, array $context = []): array
     {
-        $parameters = $data->parameters;
-        unset($parameters['additionalProperties']);
-
         return [
             'functionDeclarations' => [
                 [
                     'description' => $data->description,
                     'name' => $data->name,
-                    'parameters' => $parameters,
+                    'parameters' => $data->parameters ? $this->removeAdditionalProperties($data->parameters) : null,
                 ],
             ],
         ];
     }
+
+    /**
+     * @template T of array
+     *
+     * @phpstan-param T $data
+     *
+     * @phpstan-return T
+     */
+    private function removeAdditionalProperties(array $data): array
+    {
+        unset($data['additionalProperties']); // not supported by Gemini
+
+        foreach ($data as &$value) {
+            if (\is_array($value)) {
+                $value = $this->removeAdditionalProperties($value);
+            }
+        }
+
+        return $data;
+    }
 }

src/platform/tests/Bridge/Google/Contract/ToolNormalizerTest.php

@@ -88,6 +88,11 @@ public static function normalizeDataProvider(): iterable
                             'type' => 'integer',
                             'description' => 'Number parameter',
                         ],
+                        'nestedObject' => [
+                            'type' => 'object',
+                            'description' => 'bar',
+                            'additionalProperties' => false,
+                        ],
                     ],
                     'required' => ['text', 'number'],
                     'additionalProperties' => false,
@@ -109,6 +114,10 @@ public static function normalizeDataProvider(): iterable
                                     'type' => 'integer',
                                     'description' => 'Number parameter',
                                 ],
+                                'nestedObject' => [
+                                    'type' => 'object',
+                                    'description' => 'bar',
+                                ],
                             ],
                             'required' => ['text', 'number'],
                         ],