[TwigComponent] Merge profiler and profiler_collect_components config options

Author: Kocal

src/TwigComponent/CHANGELOG.md

@@ -1,5 +1,10 @@
 # CHANGELOG
 
+## 2.32
+
+-  Add option `profiler.collect_components` to control component data collection
+   in the profiler (enabled in debug mode by default)
+
 ## 2.30
 
 -  Ensure compatibility with PHP 8.5

src/TwigComponent/config/debug.php

@@ -28,7 +28,7 @@
         ->args([
             service('ux.twig_component.component_logger_listener'),
             service('twig'),
-            abstract_arg('profiler dump components'),
+            abstract_arg('profiler collect components'),
         ])
         ->tag('data_collector', [
             'template' => '@TwigComponent/Collector/twig_component.html.twig',

src/TwigComponent/src/DataCollector/TwigComponentDataCollector.php

@@ -35,7 +35,7 @@ final class TwigComponentDataCollector extends AbstractDataCollector implements
     public function __construct(
         private readonly TwigComponentLoggerListener $logger,
         private readonly Environment $twig,
-        private readonly bool $dumpComponents = true,
+        private readonly bool $collectComponents = true,
     ) {
         $this->hasStub = class_exists(ClassStub::class);
     }
@@ -136,7 +136,7 @@ private function collectDataFromLogger(): void
                     'render_start' => $profile[0],
                 ];
 
-                if ($this->dumpComponents) {
+                if ($this->collectComponents) {
                     $renders[$renderId]['component'] = $mountedComponent->getComponent();
                 }
 

src/TwigComponent/src/DependencyInjection/TwigComponentExtension.php

@@ -150,10 +150,11 @@ static function (ChildDefinition $definition, AsTwigComponent $attribute) {
         $container->setAlias('console.command.stimulus_component_debug', 'ux.twig_component.command.debug')
             ->setDeprecated('symfony/ux-twig-component', '2.13', '%alias_id%');
 
-        if ($container->getParameter('kernel.debug') && $config['profiler']) {
+        if ($config['profiler']['enabled']) {
             $loader->load('debug.php');
+
             $container->getDefinition('ux.twig_component.data_collector')
-                ->setArgument(2, $config['profiler_dump_components'] ?? true);
+                ->setArgument(2, $config['profiler']['collect_components']);
         }
 
         $loader->load('cache.php');
@@ -217,13 +218,13 @@ public function getConfigTreeBuilder(): TreeBuilder
                 ->scalarNode('anonymous_template_directory')
                     ->info('Defaults to `components`')
                 ->end()
-                ->booleanNode('profiler')
-                    ->info('Enables the profiler for Twig Component (in debug mode)')
-                    ->defaultValue('%kernel.debug%')
-                ->end()
-                ->booleanNode('profiler_dump_components')
-                    ->info('Dump components in the Twig Component profiler panel')
-                    ->defaultTrue()
+                ->arrayNode('profiler')
+                    ->info('Enables the profiler for Twig Component')
+                    ->canBeEnabled()
+                    ->children()
+                        ->booleanNode('enabled')->defaultValue('%kernel.debug%')->end()
+                        ->booleanNode('collect_components')->info('Collect components instances')->defaultTrue()->end()
+                    ->end()
                 ->end()
                 ->scalarNode('controllers_json')
                     ->setDeprecated('symfony/ux-twig-component', '2.18', 'The "twig_component.controllers_json" config option is deprecated, and will be removed in 3.0.')

src/TwigComponent/tests/Unit/DependencyInjection/TwigComponentExtensionTest.php

@@ -40,15 +40,17 @@ public function testDataCollectorWithDebugMode()
         $this->assertTrue($container->getDefinition('ux.twig_component.data_collector')->getArgument(2));
     }
 
-    public function testDataCollectorWithDumpComponentsDisabled()
+    public function testDataCollectorWithCollectComponentsDisabled()
     {
         $container = $this->createContainer();
         $container->setParameter('kernel.debug', true);
         $container->registerExtension(new TwigComponentExtension());
         $container->loadFromExtension('twig_component', [
             'defaults' => [],
             'anonymous_template_directory' => 'components/',
-            'profiler_dump_components' => false,
+            'profiler' => [
+                'collect_components' => false,
+            ],
         ]);
         $this->compileContainer($container);
 
@@ -71,21 +73,6 @@ public function testDataCollectorWithDebugModeCanBeDisabled()
         $this->assertFalse($container->hasDefinition('ux.twig_component.data_collector'));
     }
 
-    public function testDataCollectorWithoutDebugMode()
-    {
-        $container = $this->createContainer();
-        $container->setParameter('kernel.debug', false);
-        $container->registerExtension(new TwigComponentExtension());
-        $container->loadFromExtension('twig_component', [
-            'defaults' => [],
-            'anonymous_template_directory' => 'components/',
-            'profiler' => true,
-        ]);
-        $this->compileContainer($container);
-
-        $this->assertFalse($container->hasDefinition('ux.twig_component.data_collector'));
-    }
-
     /**
      * @group legacy
      */