[Wojdylak]

Why

b2b-kit 4.0 asks for a Doctrine index on the Payment entity alongside the trade credit interface and trait — README step 11, entity 7:

#[ORM\Entity]
#[ORM\Table(name: 'sylius_payment')]
#[ORM\Index(name: 'idx_payment_credit_approval_state', columns: ['credit_approval_state'])]
class Payment extends BasePayment implements PaymentInterface
{
    use TradeCreditPaymentTrait;
}

The interface and the trait are covered by existing rules. The index was not — there was no rule for adding index mapping, so it stayed a manual step in an otherwise automated entity decoration.

What

AddIndexToClassExtendingTypeRector, modelled on AddAttributeOverridesToClassExtendingTypeRector: matches classes derived from a configured parent and appends the attribute at class level.

$rectorConfig->ruleWithConfiguration(AddIndexToClassExtendingTypeRector::class, [
    'Sylius\Component\Core\Model\Payment' => [
        new Index('idx_payment_credit_approval_state', ['credit_approval_state']),
    ],
]);

Added:

• src/Rector/Class_/AddIndexToClassExtendingTypeRector.php
• src/Rector/Dto/Index.php — name plus columns
• stubs/Sylius/Component/Core/Model/Payment.php — the stub was missing
• tests with four fixtures
• one phpstan-baseline.neon entry, the same refactor() contravariance notice every other rule here carries

Behaviour

Existing indexes are matched by name, so:

• a class that already declares the index is left alone (payment_already_has_index)
• an unrelated index on the same class is preserved and the configured one is appended (payment_with_another_index)
• a class not derived from the configured parent is untouched (unrelated_class)

The attribute is recognised written as Index, ORM\Index or fully qualified Doctrine\ORM\Mapping\Index.

The index is emitted as a separate class-level attribute rather than folded into #[ORM\Table(indexes: ...)], which is the form the b2b-kit README shows and which leaves an existing #[ORM\Table] untouched.

Not included

No set file registers the rule yet. #65 merges b2b-kit-3.0.php and b2b-kit-4.0.php into a single b2b-kit.php, so wiring it in here would collide with that PR. It is a two-line addition once one of the two lands:

$rectorConfig->ruleWithConfigurationComposerVersionBound(AddIndexToClassExtendingTypeRector::class, [
    'Sylius\Component\Core\Model\Payment' => [
        new Index('idx_payment_credit_approval_state', ['credit_approval_state']),
    ],
], 'sylius/b2b-kit', '>=4.0 <5.0');

The rule is self-contained and testable on its own, so it does not need to wait for #65.

Checks

PHPUnit 75 tests green (4 new), PHPStan no errors, ECS clean on every added file. Run against rector/rector 2.6.4, the version main currently resolves to after #64.

docs/rector_rules_overview.md is not regenerated — it already predates AddAttributeOverridesToClassExtendingTypeRector, and refreshing it here would bury this change under an unrelated diff.