RectorPHP中ClassPropertyAssignToConstructorPromotionRector规则导致PHPDoc类型注释丢失问题分析

问题背景

在PHP 8.0中引入了构造函数属性提升(Constructor Property Promotion)特性，它允许开发者直接在构造函数参数中声明类属性，从而简化代码。RectorPHP项目中的ClassPropertyAssignToConstructorPromotionRector规则就是用来将传统的属性赋值方式自动转换为这种新语法。

问题现象

当使用ClassPropertyAssignToConstructorPromotionRector规则进行代码转换时，发现一个关于PHPDoc注释的问题：如果类属性使用了精确的类型注释（如示例中的positive-int），在转换过程中这些注释信息会丢失。

示例分析

原始代码中定义了一个不可变类PreisStufeVM，其中mengeab属性使用了@var positive-int注释表示该属性必须是正整数：

/**
 * @immutable
 */
final class PreisStufeVM
{
    /** @var positive-int */
    public int $mengeab;
    
    public float $price;

    public function __construct(int $mengeab, float $price)
    {
        $this->mengeab = $mengeab;
        $this->price = $price;
    }
}

经过ClassPropertyAssignToConstructorPromotionRector转换后，代码变为：

/**
 * @immutable
 */
final class PreisStufeVM
{
    public function __construct(
        public int $mengeab,
        public float $price
    ) {
    }
}

可以看到，虽然语法转换正确，但原本的positive-int类型注释完全丢失了，这会导致类型信息不完整，可能影响静态分析工具的工作。

技术影响

1. 类型安全性降低：positive-int比普通的int提供了更强的类型约束，丢失后可能导致潜在的类型错误无法被检测到
2. 文档完整性受损：PHPDoc注释是代码文档的重要组成部分，丢失后会影响代码的可读性和可维护性
3. 静态分析工具失效：许多PHP静态分析工具（如PHPStan、Psalm）依赖这些精确的类型注释来进行更深入的分析

解决方案思路

要解决这个问题，需要在规则转换过程中：

1. 保留原有的PHPDoc注释
2. 将注释从属性声明移动到提升的参数上
3. 确保注释格式与构造函数提升语法兼容

正确的转换结果应该类似于：

/**
 * @immutable
 */
final class PreisStufeVM
{
    public function __construct(
        /** @var positive-int */
        public int $mengeab,
        public float $price
    ) {
    }
}

技术实现要点

1. 注释提取：在转换前需要正确提取属性的PHPDoc注释
2. 注释位置调整：将注释从属性移动到对应的构造函数参数
3. 格式兼容性：确保注释格式在参数位置仍然有效
4. 多注释处理：处理一个参数对应多个注释的情况

总结

这个问题展示了在自动化代码重构过程中保留代码语义完整性的重要性。特别是对于PHPDoc注释这种"元数据"，虽然在运行时不起作用，但对代码质量工具和开发者理解代码至关重要。RectorPHP作为重构工具，应该尽可能保持这些有价值的信息不丢失。