OpenRewrite Lombok迁移工具中的Setter注解处理问题分析

2025-06-29 09:18:53作者：劳婵绚Shirley

问题背景

在使用OpenRewrite的rewrite-migrate-java工具进行Lombok注解迁移时，发现了一个关于Setter方法处理的bug。当开发者使用UseLombokSetter配方(recipe)来自动化地将显式setter方法转换为Lombok的@Setter注解时，如果原始setter方法上带有其他框架注解(如Spring的@Value或@Autowired)，这些注解会被意外删除，导致应用程序功能异常。

问题复现

考虑以下Spring服务类示例：

@Service
@Slf4j
public class MyService {
    @Getter
    private String configuration;
    @Getter
    private MyRepository repository;

    @Value("${spring.application.name}")
    public void setConfiguration(String configuration) {
        this.configuration = configuration;
    }

    @Autowired
    public void setRepository(MyRepository repository) {
        this.repository = repository;
    }
}

开发者期望通过UseLombokSetter配方将其转换为：

@Service
@Slf4j
public class MyService {
    @Getter
    @Setter
    @Value("${spring.application.name}")
    private String configuration;
    
    @Getter
    @Setter
    @Autowired
    private MyRepository repository;
}

但实际转换结果却丢失了关键注解：

@Service
@Slf4j
public class MyService {
    @Getter
    @Setter
    private String configuration;
    
    @Getter
    @Setter
    private MyRepository repository;
}

问题分析

这个问题的核心在于UseLombokSetter配方当前的处理逻辑存在两个不足：

注解保留机制缺失：配方在删除setter方法时，没有考虑方法上可能存在的其他重要注解，直接丢弃了这些元数据。
注解迁移策略不完善：对于框架特定的注解(如Spring的依赖注入注解)，应该考虑如何合理地将其迁移到字段声明上，或者使用Lombok的高级特性来保留这些注解。

解决方案

临时解决方案

项目维护者已经提供了一个快速修复方案，主要策略是：

当检测到setter方法上存在除@Override外的其他注解时，跳过该方法的转换
这样可以避免破坏现有功能，但无法实现完全的自动化迁移

理想解决方案

更完善的解决方案应该利用Lombok的onMethod特性，这是Lombok提供的一个实验性功能，允许将注解保留在生成的方法上。转换后的代码应该如下：

@Service
@Slf4j
public class MyService {
    @Getter
    @Setter(onMethod = @__(@Value("${spring.application.name}")))
    private String configuration;
    
    @Getter
    @Setter(onMethod = @__(@Autowired))
    private MyRepository repository;
}