[kix]

CommandConfigureToAttributeRector moves setName(), setDescription(), setAliases() and setHidden() out of configure() into #[AsCommand(...)]. It never checks that a name ends up in the attribute.

AsCommand::__construct() takes a required string $name as its first argument. When a command gets its name from somewhere else, for example the constructor:

final class SomeCommand extends Command
{
    public function __construct(?string $name = 'app:some')
    {
        parent::__construct($name);
    }

    protected function configure(): void
    {
        $this->setDescription('Some description');
    }
}

the rule currently produces #[AsCommand(description: 'Some description')]. Compiling the container (or booting the console) then fails with:

ArgumentCountError: Symfony\Component\Console\Attribute\AsCommand::__construct(): Argument #1 ($name) not passed

We hit this running the Symfony 6.0-7.2 sets on a Symfony 7.4 app: two commands got a nameless attribute and every bin/console call crashed.

Change

Before extracting anything, require that a name will be there: either already on an existing #[AsCommand] (name:) or set by a constant setName() in configure(). If there is something to move into the attribute but no name, the class is left untouched (same approach as the existing skip for non-constant values). configure() bodies with nothing to move, like empty_configure_body, still behave as before.

Tests

New fixtures:

• skip_name_passed_to_parent_constructor: the case above, left unchanged. Fails without the change.
• skip_non_constant_name: setName($this->commandName) next to a constant description, left unchanged. Fails without the change.
• description_with_name_on_existing_attribute: #[AsCommand(name: ...)] plus a description in configure() still converts (regression guard).

The rule's tests pass (17), plus ECS and PHPStan.

🤖 Generated with Claude Code

[samsonasik]

early check :

if ($node->isFirstClassCallable()) {
    return null;
}

here ...