AI 修改老项目时，如何控制影响范围#

1
git status --short
2
git diff
3
git log -5 --oneline

1
当前工作区已有未提交修改，这些修改不属于本任务。
2
不要撤销、覆盖、格式化或移动它们。
3
如果目标文件与已有修改重叠，先停止并说明冲突位置。

1
mvn test
2
mvn package

1
修改前：共 128 个测试，已有 2 个失败。
2
失败用例：InventoryJobTest、LegacyPaymentTest。
3
它们与订单查询无直接关系，本任务不得扩大失败数量。

1
需求：给订单列表增加可选的订单状态筛选。
2

3
请先只读分析，不要修改文件。
4

5
需要回答：
6
1. 订单列表从 Controller 到数据库的完整调用链；
7
2. 查询参数对象被哪些入口使用；
8
3. Repository 查询中已有的租户、用户权限和软删除条件；
9
4. 订单状态在接口、Java 枚举和数据库中的映射；
10
5. 当前分页、排序和空参数行为；
11
6. 相关测试及缺失的保护；
12
7. 实现这个需求所需的最小文件集合。

1
OrderController.list
2
        ↓
3
OrderQueryService.search
4
        ↓
5
OrderSpecificationFactory.build
6
        ↓
7
OrderRepository.findAll(specification, pageable)

1
## 必须保持
2

3
- 不传 status 时，查询结果与修改前一致。
4
- 分页页码、每页数量和默认排序不变。
5
- tenantId、用户权限和 deleted = false 条件仍然生效。
6
- 管理员接口继续支持原有 OrderQuery。
7
- 非法状态仍按项目现有方式返回 400。
8
- 不修改数据库字段和状态枚举。

1
@Test
2
void shouldReturnSameOrdersWhenStatusIsAbsent() {
3
    OrderQuery query = new OrderQuery(null, null, null);
4

5
    Page<OrderSummary> result = service.search(
6
            currentUser,
7
            query,
8
            PageRequest.of(0, 20)
9
    );
10

11
    assertThat(result.getContent())
12
            .extracting(OrderSummary::orderNo)
13
            .containsExactly("ORD-003", "ORD-002", "ORD-001");
14
}

1
允许修改：
2
- OrderQuery.java
3
- OrderSpecificationFactory.java
4
- OrderQueryServiceTest.java
5
- OrderControllerTest.java
6

7
明确不修改：
8
- OrderEntity.java
9
- OrderStatus.java
10
- OrderRepository.java
11
- 数据库迁移文件
12
- Maven 依赖和全局配置
13
- 管理员订单接口

1
请只实现可选的订单状态筛选。
2

3
要求：
4
- 只修改允许列表中的文件；
5
- status 为空时完全保持原查询行为；
6
- 复用现有状态映射，不新增枚举值；
7
- 在已有 Specification 上追加条件，不重写查询链；
8
- 不进行重命名、格式化或无关重构；
9
- 先补充失败测试，再做最小实现；
10
- 如果发现必须修改范围外文件，先说明原因并等待确认。

1
public record OrderQuery(
2
        LocalDate startDate,
3
        LocalDate endDate,
4
        ApiOrderStatus status
5
) {}

1
public Specification<OrderEntity> build(
2
        long tenantId,
3
        long currentUserId,
4
        OrderQuery query
5
) {
6
    Specification<OrderEntity> specification = Specification
7
            .where(belongsToTenant(tenantId))
8
            .and(visibleTo(currentUserId))
9
            .and(notDeleted())
10
            .and(createdBetween(query.startDate(), query.endDate()));
11

12
    if (query.status() != null) {
13
        specification = specification.and(
14
                hasInternalStatus(statusMapper.toInternal(query.status()))
15
        );
16
    }
17

18
    return specification;
19
}

1
if (query.status() != null) {
2
    specification = specification.and(...);
3
}

1
git status --short
2
git diff --stat
3
git diff --check
4
git diff

1
test: cover existing order query behavior
2
feat: add optional order status filter
3
test: cover status filter and tenant boundary

1
你正在修改一个已有生产系统。
2

3
开始前：
4
1. 检查工作区状态，不要覆盖已有修改；
5
2. 阅读需求相关的完整调用链；
6
3. 找出公共类型的所有调用方；
7
4. 列出当前行为、权限条件和测试基线；
8
5. 只输出最小实现方案，不修改文件。
9

10
方案确认后：
11
- 只修改明确允许的文件；
12
- 不升级依赖；
13
- 不重命名公共接口；
14
- 不格式化无关代码；
15
- 不修改数据库结构；
16
- 先用测试固定旧行为，再增加新行为；
17
- 每个阶段展示 Diff 和验证结果；
18
- 如果必须扩大范围，先停止并说明原因。

1
上下文：先理解完整调用链
2
边界：明确允许和禁止修改什么
3
证据：用测试和 Diff 证明行为
4
恢复：让修改能够独立回滚