Deprecate Annotated Commands in favor of native Symfony Console commands

.circleci/config.yml

@@ -14,6 +14,7 @@ defaults: &defaults
     PHP_EXTENSIONS_DISABLE: xdebug
     PHP_XDEBUG_MODE: off
 
+#No longer used
 requires: &requires
   requires:
     - check_mergable
@@ -173,7 +174,8 @@ workflows:
 #      - test_80_drupal92_security:
 #          <<: *requires
       - test:
-          <<: *requires
+          # Not used, for now.
+          #          <<: *requires
           <<: *poststeps
           matrix:
             parameters:

src/Application.php

@@ -11,6 +11,7 @@
 use Drush\Boot\DrupalBootLevels;
 use Drush\Command\RemoteCommandProxy;
 use Drush\Config\ConfigAwareTrait;
+use Drush\Event\ConsoleDefinitionsEvent;
 use Drush\Runtime\RedispatchHook;
 use Drush\Runtime\ServiceManager;
 use Drush\Runtime\TildeExpansionHook;
@@ -309,6 +310,8 @@ public function configureAndRegisterCommands(InputInterface $input, OutputInterf
         // any of the configuration steps we do here.
         $this->configureIO($input, $output);
 
+        $this->addListeners($commandfileSearchpath);
+
         // Directly add the yaml-cli commands.
         $this->addCommands($this->serviceManager->instantiateYamlCliCommands());
 
@@ -327,6 +330,9 @@ public function configureAndRegisterCommands(InputInterface $input, OutputInterf
         // Note that Robo::register can accept either Annotated Command
         // command handlers or Symfony Console Command objects.
         Robo::register($this, $commandInstances);
+
+        // Dispatch our custom event. It also fires later in \Drush\Boot\DrupalBoot8::bootstrapDrupalFull.
+        Drush::getContainer()->get('eventDispatcher')->dispatch(new ConsoleDefinitionsEvent($this), ConsoleDefinitionsEvent::class);
     }
 
     /**
@@ -338,4 +344,26 @@ public function renderThrowable(\Throwable $e, OutputInterface $output): void
 
         $this->doRenderThrowable($e, $output);
     }
+
+    // Discover event listeners, and add those that do not require bootstrap.
+    protected function addListeners($commandfileSearchpath): void
+    {
+        $listenerClasses = $this->serviceManager->discoverListeners($commandfileSearchpath, '\Drush');
+        $listenerClasses = $this->serviceManager->filterListeners($listenerClasses);
+        $this->serviceManager->addListeners($listenerClasses, Drush::getContainer());
+    }
+
+    /**
+     * Remove a command. Initially used by WootDefinitionListener and its test.
+     *
+     * An alternative would be console.excluded https://github.com/search?q=repo%3AHuttopia%2Fconsole-bundle%20console.excluded&type=code
+     */
+    public function remove(string $id): void
+    {
+        $rf = new \ReflectionProperty(\Symfony\Component\Console\Application::class, 'commands');
+        $rf->setAccessible(true);
+        $commands = $rf->getValue($this);
+        unset($commands[$id]);
+        $rf->setValue($this, $commands);
+    }
 }

src/Attributes/DefaultFields.php

@@ -5,8 +5,31 @@
 namespace Drush\Attributes;
 
 use Attribute;
+use Consolidation\AnnotatedCommand\Parser\CommandInfo;
+use Drush\Formatters\FormatterConfigurationItemProviderInterface;
 
-#[Attribute(Attribute::TARGET_METHOD)]
-class DefaultFields extends \Consolidation\AnnotatedCommand\Attributes\DefaultFields
+#[Attribute(Attribute::TARGET_CLASS | Attribute::TARGET_METHOD)]
+class DefaultFields implements FormatterConfigurationItemProviderInterface
 {
+    const KEY = 'default-fields';
+
+    /**
+     * @param $fields
+     *   An array of field names to show by default.
+     */
+    public function __construct(public array $fields)
+    {
+    }
+
+    public function getConfigurationItem(\ReflectionAttribute $attribute): array
+    {
+        $args = $attribute->getArguments();
+        return [self::KEY => $args['fields']];
+    }
+
+    public static function handle(\ReflectionAttribute $attribute, CommandInfo $commandInfo)
+    {
+        $args = $attribute->getArguments();
+        $commandInfo->addAnnotation('default-fields', $args['fields']);
+    }
 }

src/Attributes/DefaultTableFields.php

@@ -5,8 +5,32 @@
 namespace Drush\Attributes;
 
 use Attribute;
+use Consolidation\AnnotatedCommand\Parser\CommandInfo;
+use Consolidation\OutputFormatters\Options\FormatterOptions;
+use Drush\Formatters\FormatterConfigurationItemProviderInterface;
 
-#[Attribute(Attribute::TARGET_METHOD)]
-class DefaultTableFields extends \Consolidation\AnnotatedCommand\Attributes\DefaultTableFields
+#[Attribute(Attribute::TARGET_CLASS | Attribute::TARGET_METHOD)]
+class DefaultTableFields implements FormatterConfigurationItemProviderInterface
 {
+    const KEY = FormatterOptions::DEFAULT_TABLE_FIELDS;
+
+    /**
+     * @param $fields
+     *   An array of field names to show by default when using table formatter.
+     */
+    public function __construct(public array $fields)
+    {
+    }
+
+    public function getConfigurationItem(\ReflectionAttribute $attribute): array
+    {
+        $args = $attribute->getArguments();
+        return [self::KEY => $args['fields']];
+    }
+
+    public static function handle(\ReflectionAttribute $attribute, CommandInfo $commandInfo)
+    {
+        $args = $attribute->getArguments();
+        $commandInfo->addAnnotation('default-table-fields', $args['fields']);
+    }
 }

src/Attributes/FieldLabels.php

@@ -1,12 +1,35 @@
 <?php
 
-declare(strict_types=1);
-
 namespace Drush\Attributes;
 
 use Attribute;
+use Consolidation\AnnotatedCommand\Parser\CommandInfo;
+use Consolidation\OutputFormatters\Options\FormatterOptions;
+use Drush\Formatters\FormatterConfigurationItemProviderInterface;
 
-#[Attribute(Attribute::TARGET_METHOD)]
-class FieldLabels extends \Consolidation\AnnotatedCommand\Attributes\FieldLabels
+#[Attribute(Attribute::TARGET_METHOD | Attribute::TARGET_CLASS)]
+class FieldLabels implements FormatterConfigurationItemProviderInterface
 {
+    const KEY = FormatterOptions::FIELD_LABELS;
+
+    /**
+     * @param $labels
+     *   An associative array of field names and labels for display.
+     */
+    public function __construct(
+        public array $labels
+    ) {
+    }
+
+    public function getConfigurationItem(\ReflectionAttribute $attribute): array
+    {
+        $args = $attribute->getArguments();
+        return [self::KEY => $args['labels']];
+    }
+
+    public static function handle(\ReflectionAttribute $attribute, CommandInfo $commandInfo)
+    {
+        $args = $attribute->getArguments();
+        $commandInfo->addAnnotation('field-labels', $args['labels']);
+    }
 }

src/Attributes/FilterDefaultField.php

@@ -1,12 +1,34 @@
 <?php
 
-declare(strict_types=1);
-
 namespace Drush\Attributes;
 
 use Attribute;
+use Consolidation\AnnotatedCommand\Parser\CommandInfo;
+use Drush\Formatters\FormatterConfigurationItemProviderInterface;
 
-#[Attribute(Attribute::TARGET_METHOD)]
-class FilterDefaultField extends \Consolidation\AnnotatedCommand\Attributes\FilterDefaultField
+#[Attribute(Attribute::TARGET_METHOD | Attribute::TARGET_CLASS)]
+class FilterDefaultField implements FormatterConfigurationItemProviderInterface
 {
+    const KEY = 'filter-default-field';
+
+    /**
+     * @param $field
+     *   A field name to filter on by default.
+     */
+    public function __construct(
+        public string $field
+    ) {
+    }
+
+    public function getConfigurationItem(\ReflectionAttribute $attribute): array
+    {
+        $args = $attribute->getArguments();
+        return [self::KEY => $args['field']];
+    }
+
+    public static function handle(\ReflectionAttribute $attribute, CommandInfo $commandInfo)
+    {
+        $args = $attribute->getArguments();
+        $commandInfo->addAnnotation('filter-default-field', $args['field']);
+    }
 }

src/Attributes/Format.php

@@ -7,11 +7,11 @@
 use Attribute;
 use Consolidation\AnnotatedCommand\Parser\CommandInfo;
 use Consolidation\OutputFormatters\Options\FormatterOptions;
-use Drush\Boot\Kernels;
+use Drush\Formatters\FormatterConfigurationItemProviderInterface;
 use JetBrains\PhpStorm\ExpectedValues;
 
 #[Attribute(Attribute::TARGET_METHOD)]
-class Format
+class Format implements FormatterConfigurationItemProviderInterface
 {
     /**
      * @param ?string $listDelimiter
@@ -32,4 +32,13 @@ public static function handle(\ReflectionAttribute $attribute, CommandInfo $comm
         $commandInfo->addAnnotation(FormatterOptions::LIST_DELIMITER, $instance->listDelimiter);
         $commandInfo->addAnnotation(FormatterOptions::TABLE_STYLE, $instance->tableStyle);
     }
+
+    public function getConfigurationItem(\ReflectionAttribute $attribute): array
+    {
+        $instance = $attribute->newInstance();
+        return [
+            FormatterOptions::LIST_DELIMITER => $instance->listDelimiter,
+            FormatterOptions::TABLE_STYLE => $instance->tableStyle
+        ];
+    }
 }

src/Attributes/ValidateEntityLoad.php

@@ -9,7 +9,7 @@
 use Consolidation\AnnotatedCommand\CommandError;
 use Drush\Utils\StringUtils;
 
-#[Attribute(Attribute::TARGET_METHOD)]
+#[Attribute(Attribute::TARGET_CLASS | Attribute::TARGET_METHOD)]
 class ValidateEntityLoad extends ValidatorBase implements ValidatorInterface
 {
     /**

src/Attributes/ValidateModulesEnabled.php

@@ -8,7 +8,7 @@
 use Consolidation\AnnotatedCommand\CommandData;
 use Consolidation\AnnotatedCommand\CommandError;
 
-#[Attribute(Attribute::TARGET_METHOD)]
+#[Attribute(Attribute::TARGET_CLASS | Attribute::TARGET_METHOD)]
 class ValidateModulesEnabled extends ValidatorBase implements ValidatorInterface
 {
     /**

src/Boot/DrupalBoot8.php

@@ -13,11 +13,10 @@
 use Drush\Config\ConfigLocator;
 use Drush\Drupal\DrushLoggerServiceProvider;
 use Drush\Drush;
+use Drush\Event\ConsoleDefinitionsEvent;
 use Drush\Runtime\LegacyServiceFinder;
 use Drush\Runtime\LegacyServiceInstantiator;
 use Drush\Runtime\ServiceManager;
-use Psr\Log\LoggerAwareInterface;
-use Psr\Log\LoggerInterface;
 use Robo\Robo;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
@@ -215,12 +214,20 @@ public function bootstrapDrupalFull(BootstrapManager $manager): void
         // Directly add the Drupal core bootstrapped commands.
         Drush::getApplication()->addCommands($this->serviceManager->instantiateDrupalCoreBootstrappedCommands());
 
+        $this->addBootstrapListeners();
+
         $this->addDrupalModuleDrushCommands($manager);
 
+        // Dispatch our custom event. It also fires earlier in \Drush\Application::configureAndRegisterCommands.
+        Drush::getContainer()->get('eventDispatcher')->dispatch(new ConsoleDefinitionsEvent(Drush::getApplication()), ConsoleDefinitionsEvent::class);
+
         // Set a default account to make sure the correct timezone is set
         $this->kernel->getContainer()->get('current_user')->setAccount(new AnonymousUserSession());
     }
 
+    /**
+     * Adds module supplied commands, as well as Drush Console commands that require bootstrap.
+     */
     public function addDrupalModuleDrushCommands(BootstrapManager $manager): void
     {
         $application = Drush::getApplication();
@@ -323,4 +330,17 @@ public function bootstrapDrupalSite(BootstrapManager $manager)
     {
         $this->bootstrapDoDrupalSite($manager);
     }
+
+    // Add the Listeners that require bootstrap.
+    public function addBootstrapListeners(): void
+    {
+        $listenersInThisModule = [];
+        $moduleHandler = \Drupal::moduleHandler();
+        foreach ($moduleHandler->getModuleList() as $moduleId => $extension) {
+            $path = DRUPAL_ROOT . '/' . $extension->getPath() . '/src/Drush';
+            $listenersInThisModule = array_merge($listenersInThisModule, $this->serviceManager->discoverListeners([$path], "\Drupal\\$moduleId\Drush"));
+        }
+        $classes = $this->serviceManager->bootstrapListenerClasses();
+        $this->serviceManager->addListeners(array_merge($listenersInThisModule, $classes), Drush::getContainer(), \Drupal::getContainer());
+    }
 }

src/Commands/AutowireTrait.php

@@ -21,8 +21,6 @@ trait AutowireTrait
      *
      * @param ContainerInterface $container
      *   The service container this instance should use.
-     *
-     * @return static
      */
     public static function create(ContainerInterface $container)
     {

src/Commands/core/CoreCommands.php

@@ -4,12 +4,12 @@
 
 namespace Drush\Commands\core;
 
+use Consolidation\OutputFormatters\Options\FormatterOptions;
 use Consolidation\OutputFormatters\StructuredData\PropertyList;
+use Consolidation\OutputFormatters\StructuredData\RowsOfFields;
 use Drush\Attributes as CLI;
 use Drush\Commands\DrushCommands;
 use Drush\Drush;
-use Consolidation\OutputFormatters\StructuredData\RowsOfFields;
-use Consolidation\OutputFormatters\Options\FormatterOptions;
 
 final class CoreCommands extends DrushCommands
 {