fix(configuration): Use VariableNode on "settings"

"arrayNode" does not support arbitrary configuration keys. Therefore, I switch to variableNode with dynamic check.

Fixed #369

Author: pan93412

src/DependencyInjection/Configuration.php

@@ -7,6 +7,7 @@
 use Meilisearch\Bundle\Searchable;
 use Symfony\Component\Config\Definition\Builder\TreeBuilder;
 use Symfony\Component\Config\Definition\ConfigurationInterface;
+use Symfony\Component\Config\Definition\Exception\InvalidConfigurationException;
 
 final class Configuration implements ConfigurationInterface
 {
@@ -62,11 +63,20 @@ public function getConfigTreeBuilder(): TreeBuilder
                                 ->info('Property accessor path (like method or property name) used to decide if an entry should be indexed.')
                                 ->defaultNull()
                             ->end()
-                            ->arrayNode('settings')
+                            ->variableNode('settings')
+                                ->defaultValue([])
                                 ->info('Configure indices settings, see: https://www.meilisearch.com/docs/reference/api/settings')
                                 ->beforeNormalization()
                                     ->always()
-                                    ->then(static function (array $value) {
+                                    ->then(static function (mixed $value) {
+                                        if (\is_null($value)) {
+                                            return [];
+                                        }
+
+                                        if (!\is_array($value)) {
+                                            throw new InvalidConfigurationException('Settings must be an array.');
+                                        }
+
                                         $stringSettings = ['distinctAttribute', 'proximityPrecision', 'searchCutoffMs'];
 
                                         foreach ($stringSettings as $setting) {
@@ -77,9 +87,7 @@ public function getConfigTreeBuilder(): TreeBuilder
 
                                         return $value;
                                     })
-                                ->end()
-                                ->arrayPrototype()
-                                    ->variablePrototype()->end()
+                                    ->end()
                                 ->end()
                             ->end()
                         ->end()

tests/Unit/ConfigurationTest.php

@@ -23,6 +23,40 @@ public function testValidConfig(array $inputConfig, array $expectedConfig): void
         $this->assertProcessedConfigurationEquals($inputConfig, $expectedConfig);
     }
 
+    /**
+     * @dataProvider dataTestSettingsDynamicChecker
+     */
+    public function testInvalidSettingsType(mixed $value, bool $passed): void
+    {
+        if (!$passed) {
+            $this->assertConfigurationIsInvalid([
+                'meilisearch' => [
+                    'indices' => [
+                        [
+                            'name' => 'items',
+                            'class' => 'App\Entity\Post',
+                            'settings' => $value,
+                        ],
+                    ],
+                ],
+            ], 'Settings must be an array.');
+
+            return;
+        }
+
+        $this->assertConfigurationIsValid([
+            'meilisearch' => [
+                'indices' => [
+                    [
+                        'name' => 'items',
+                        'class' => 'App\Entity\Post',
+                        'settings' => $value,
+                    ],
+                ],
+            ],
+        ]);
+    }
+
     /**
      * @return iterable<array{inputConfig: array<mixed>, expectedConfig: array<mixed>}>
      */
@@ -294,6 +328,34 @@ public static function dataTestConfigurationTree(): iterable
         ];
     }
 
+    public static function dataTestSettingsDynamicChecker(): iterable
+    {
+        yield 'string is not acceptable' => [
+            'settings' => 'hello',
+            'passed' => false,
+        ];
+
+        yield 'int is not acceptable' => [
+            'settings' => 1,
+            'passed' => false,
+        ];
+
+        yield 'bool is not acceptable' => [
+            'settings' => true,
+            'passed' => false,
+        ];
+
+        yield 'array is acceptable' => [
+            'settings' => [],
+            'passed' => true,
+        ];
+
+        yield 'null is acceptable' => [
+            'settings' => null,
+            'passed' => true,
+        ];
+    }
+
     protected function getConfiguration(): Configuration
     {
         return new Configuration();