[Toolkit] Improve InstallComponentCommand by asking/guessing which Kit to use, remove ux_toolkit.kit parameter, remove DependencyInjection configuration

Author: Kocal

src/Toolkit/config/services.php

@@ -35,7 +35,6 @@
 
         ->set('.ux_toolkit.command.install', InstallComponentCommand::class)
             ->args([
-                param('ux_toolkit.kit'),
                 service('.ux_toolkit.registry.registry_factory'),
                 service('filesystem'),
             ])
@@ -55,7 +54,6 @@
             ->args([
                 service('.ux_toolkit.kit.kit_factory'),
                 service('filesystem'),
-                param('kernel.project_dir'),
             ])
 
         ->set('.ux_toolkit.registry.github', GitHubRegistry::class)

src/Toolkit/doc/index.rst

@@ -40,27 +40,16 @@ Install the UX Toolkit using Composer and Symfony Flex:
     # If you want to keep `html_cva` and `tailwind_merge` in your Twig components:
     $ composer require twig/extra-bundle twig/html-extra:^3.12.0 tales-from-a-dev/twig-tailwind-extra
 
-Configuration
--------------
-
-Configuration is done in your ``config/packages/ux_toolkit.yaml`` file:
-
-.. code-block:: yaml
-
-    # config/packages/ux_toolkit.yaml
-    ux_toolkit:
-        kit: 'shadcn'
-
 Usage
 -----
 
 You may find a list of components in the `UX Components page`_, with the installation instructions for each of them.
 
-For example, if you want to install the `Button` component, you will find the following instruction:
+For example, if you want to install a `Button` component, you will find the following instruction:
 
 .. code-block:: terminal
 
-    $ php bin/console ux:toolkit:install-component Button
+    $ php bin/console ux:toolkit:install-component Button --kit=<kitName>
 
 It will create the ``templates/components/Button.html.twig`` file, and you will be able to use the `Button` component like this:
 
@@ -121,24 +110,17 @@ A kit is composed of:
 - A ``templates/components`` directory, that contains the Twig components,
 - A ``docs/components`` directory, optional, that contains the documentation for each "root" Twig component.
 
-Use your kit in a Symfony application
--------------------------------------
+Using your kit
+~~~~~~~~~~~~~~
 
-You can globally configure the kit to use in your application by setting the ``ux_toolkit.kit`` configuration:
-
-.. code-block:: yaml
-
-    # config/packages/ux_toolkit.yaml
-    ux_toolkit:
-        kit: 'github.com/my-username/my-ux-kits'
-        # or for a specific version
-        kit: 'github.com/my-username/my-ux-kits:1.0.0'
-
-If you do not want to globally configure the kit, you can pass the ``--kit`` option to the ``ux:toolkit:install-component`` command:
+Once your kit is published on GitHub, you can use it by specifying the ``--kit`` option when installing a component:
 
 .. code-block:: terminal
 
-    $ php bin/console ux:toolkit:install-component Button --kit=github.com/my-username/my-ux-kits
+    $ php bin/console ux:toolkit:install-component Button --kit=github.com/my-username/my-ux-toolkit-kit
+
+    # or for a specific version
+    $ php bin/console ux:toolkit:install-component Button --kit=github.com/my-username/my-ux-toolkit-kit:1.0.0
 
 Backward Compatibility promise
 ------------------------------

src/Toolkit/src/Command/InstallComponentCommand.php

@@ -24,6 +24,7 @@
 use Symfony\UX\Toolkit\File\File;
 use Symfony\UX\Toolkit\Installer\Installer;
 use Symfony\UX\Toolkit\Kit\Kit;
+use Symfony\UX\Toolkit\Registry\LocalRegistry;
 use Symfony\UX\Toolkit\Registry\RegistryFactory;
 
 /**
@@ -42,7 +43,6 @@ class InstallComponentCommand extends Command
     private bool $isInteractive;
 
     public function __construct(
-        private readonly string $kitName,
         private readonly RegistryFactory $registryFactory,
         private readonly Filesystem $filesystem,
     ) {
@@ -53,6 +53,7 @@ protected function configure(): void
     {
         $this
             ->addArgument('component', InputArgument::OPTIONAL, 'The component name (Ex: Button)')
+            ->addOption('kit', 'k', InputOption::VALUE_OPTIONAL, 'The kit name (Ex: shadcn, or github.com/user/my-ux-toolkit-kit)')
             ->addOption(
                 'destination',
                 'd',
@@ -61,7 +62,6 @@ protected function configure(): void
                 Path::join('templates', 'components')
             )
             ->addOption('force', 'f', InputOption::VALUE_NONE, 'Force the component installation, even if the component already exists')
-            ->addOption('kit', 'k', InputOption::VALUE_OPTIONAL, 'Override the kit name', $this->kitName)
             ->setHelp(
                 <<<EOF
 The <info>%command.name%</info> command will install a new UX Component in your project.
@@ -92,10 +92,40 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $io = new SymfonyStyle($input, $output);
 
         $kitName = $input->getOption('kit');
-        $registry = $this->registryFactory->getForKit($kitName);
-        $kit = $registry->getKit($kitName);
+        $componentName = $input->getArgument('component');
+
+        // If the kit name is not explicitly provided, we need to suggest one
+        if (null === $kitName) {
+            $availableKits = array_map(fn (string $kitName) => $this->registryFactory->getForKit($kitName)->getKit($kitName), LocalRegistry::getAvailableKitsName());
+            $availableKitsCount = \count($availableKits);
+            // If more than one kit is available, we ask the user which one to use
+            if ($availableKitsCount > 1) {
+                // If a component name is provided, we can filter the kits to only those that contain the component
+                if (null !== $componentName) {
+                    $availableKits = array_filter($availableKits, fn (Kit $kit) => null !== $kit->getComponent($componentName));
+                    $kitName = $io->choice(\sprintf('The component "%s" exists in multiple kits. Which one do you want to use?', $componentName), array_map(fn (Kit $kit) => $kit->name, $availableKits));
+                } else {
+                    $kitName = $io->choice('Which kit do you want to use?', array_map(fn (Kit $kit) => $kit->name, $availableKits));
+                }
+                foreach ($availableKits as $availableKit) {
+                    if ($availableKit->name === $kitName) {
+                        $kit = $availableKit;
+                        break;
+                    }
+                }
+            } elseif (1 === $availableKitsCount) {
+                $kit = $availableKits[0];
+            } else {
+                $io->error('No kit found. Please provide a kit name using the --kit option.');
+
+                return Command::FAILURE;
+            }
+        } else {
+            $registry = $this->registryFactory->getForKit($kitName);
+            $kit = $registry->getKit($kitName);
+        }
 
-        if (null === $componentName = $input->getArgument('component')) {
+        if (null === $componentName) {
             // Ask for the component name if not provided
             $componentName = $io->choice('Which component do you want to install?', array_map(fn (Component $component) => $component->name, $this->getAvailableComponents($kit)));
             $component = $kit->getComponent($componentName);
@@ -124,7 +154,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             }
         }
 
-        $io->writeln(\sprintf('Installing component <info>%s</> from the <info>%s</> kit...', $component->name, $kitName));
+        $io->writeln(\sprintf('Installing component <info>%s</> from the <info>%s</> kit...', $component->name, $kit->name));
 
         $installer = new Installer($this->filesystem, fn (string $question) => $this->io->confirm($question, $input->isInteractive()));
         $installationReport = $installer->installComponent($kit, $component, $destinationPath = $input->getOption('destination'), $input->getOption('force'));

src/Toolkit/src/Registry/LocalRegistry.php

@@ -13,6 +13,7 @@
 
 use Symfony\Component\Filesystem\Filesystem;
 use Symfony\Component\Filesystem\Path;
+use Symfony\Component\Finder\Finder;
 use Symfony\UX\Toolkit\Kit\Kit;
 use Symfony\UX\Toolkit\Kit\KitFactory;
 
@@ -24,6 +25,8 @@
  */
 final class LocalRegistry implements Registry
 {
+    private static string $kitsDir = __DIR__.\DIRECTORY_SEPARATOR.'..'.\DIRECTORY_SEPARATOR.'..'.\DIRECTORY_SEPARATOR.'kits';
+
     public static function supports(string $kitName): bool
     {
         return 1 === preg_match('/^[a-zA-Z0-9_-]+$/', $kitName);
@@ -32,25 +35,34 @@ public static function supports(string $kitName): bool
     public function __construct(
         private readonly KitFactory $kitFactory,
         private readonly Filesystem $filesystem,
-        private readonly string $projectDir,
     ) {
     }
 
     public function getKit(string $kitName): Kit
     {
-        $possibleKitDirs = [
-            // Local kit
-            Path::join($this->projectDir, 'kits', $kitName),
-            // From vendor
-            Path::join($this->projectDir, 'vendor', 'symfony', 'ux-toolkit', 'kits', $kitName),
-        ];
-
-        foreach ($possibleKitDirs as $kitDir) {
-            if ($this->filesystem->exists($kitDir)) {
-                return $this->kitFactory->createKitFromAbsolutePath($kitDir);
-            }
+        $kitDir = Path::join(self::$kitsDir, $kitName);
+        if ($this->filesystem->exists($kitDir)) {
+            return $this->kitFactory->createKitFromAbsolutePath($kitDir);
         }
 
         throw new \RuntimeException(\sprintf('Unable to find the kit "%s" in the following directories: "%s"', $kitName, implode('", "', $possibleKitDirs)));
     }
+
+    /**
+     * @return array<string>
+     */
+    public static function getAvailableKitsName(): array
+    {
+        $availableKitsName = [];
+        $finder = (new Finder())->directories()->in(self::$kitsDir)->depth(0);
+
+        foreach ($finder as $directory) {
+            $kitName = $directory->getRelativePathname();
+            if (self::supports($kitName)) {
+                $availableKitsName[] = $kitName;
+            }
+        }
+
+        return $availableKitsName;
+    }
 }

src/Toolkit/src/UXToolkitBundle.php

@@ -11,7 +11,6 @@
 
 namespace Symfony\UX\Toolkit;
 
-use Symfony\Component\Config\Definition\Configurator\DefinitionConfigurator;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
 use Symfony\Component\HttpKernel\Bundle\AbstractBundle;
@@ -24,30 +23,8 @@ class UXToolkitBundle extends AbstractBundle
 {
     protected string $extensionAlias = 'ux_toolkit';
 
-    public function configure(DefinitionConfigurator $definition): void
-    {
-        $rootNode = $definition->rootNode();
-        $rootNode
-            ->children()
-                ->scalarNode('kit')
-                    ->info('The kit to use, it can be from the official UX Toolkit repository, or an external GitHub repository')
-                    ->defaultValue('shadcn')
-                    ->example([
-                        'shadcn',
-                        'github.com/user/repository@my-kit',
-                        'github.com/user/repository@my-kit:main',
-                        'https://github.com/user/repository@my-kit',
-                    ])
-                ->end()
-            ->end();
-    }
-
     public function loadExtension(array $config, ContainerConfigurator $container, ContainerBuilder $builder): void
     {
-        $container->parameters()
-            ->set('ux_toolkit.kit', $config['kit'])
-        ;
-
         $container->import('../config/services.php');
     }
 }

src/Toolkit/tests/Command/InstallComponentCommandTest.php

@@ -53,7 +53,7 @@ public function testShouldAbleToInstallComponentTableAndItsDependencies(): void
         $testCommand = $this->consoleCommand('ux:toolkit:install-component Table --destination='.$this->tmpDir)
             ->execute()
             ->assertSuccessful()
-            ->assertOutputContains('Installing component Table from the shadcn kit...')
+            ->assertOutputContains('Installing component Table from the Shadcn UI kit...')
             ->assertOutputContains('[OK] The component has been installed.')
         ;
 

src/Toolkit/tests/UXToolkitBundleTest.php

@@ -22,6 +22,5 @@ public function testBundleBuildsSuccessfully(): void
         $container = self::$kernel->getContainer();
 
         $this->assertInstanceOf(UXToolkitBundle::class, $container->get('kernel')->getBundles()['UXToolkitBundle']);
-        $this->assertEquals('shadcn', $container->getParameter('ux_toolkit.kit'));
     }
 }