[BetterPhpDocParser] inject phpdoc node decorators explicitly to keep required order

[TomasVotruba]

The PHP doc node decorators had a required run order that autodiscovery does not guarantee.

DoctrineAnnotationDecorator transforms a raw GenericTagValueNode annotation into a DoctrineAnnotationTagValueNode tree, creating the ArrayItemNodes. ArrayItemClassNameDecorator then resolves class names inside those nodes. So Doctrine must run first.

Autodiscovering the NodeDecorator/ directory ordered them by filesystem, dropping DoctrineAnnotationDecorator to last:

autodiscover order: ConstExpr, ArrayItem, PhpDocTagGenericUses, DoctrineAnnotation
required order:      ConstExpr, DoctrineAnnotation, ArrayItem, PhpDocTagGenericUses

With Doctrine last, ArrayItemClassNameDecorator runs on un-transformed text, never marks the annotation class usages, and removeUnusedImports() then strips imports that are actually used inside annotations:

 <?php

-use App\Repository\DemoRepository;
-use Doctrine\DBAL\Types\Types;
 use Doctrine\ORM\Mapping as ORM;

 /**
  * @ORM\Entity(repositoryClass=DemoRepository::class)
  */
 final class SkipUsedInAnnotation
 {
     /**
      * @ORM\Column(type=Types::STRING, length=100, nullable=false)
      */
     protected $status;
 }

Fix

Inject the four decorators as explicit __construct() params, in the required order, instead of autodiscovering them — the order is now encoded in the service and documented with a comment. Container autowires each concrete decorator by type.