---

name: refactor description: 当代码结构调整可能改变行为、时序或输出（性能优化、重写、模块拆分、同步改异步、数据模型迁移）时使用。

重构

概述

先定义行为边界，再在测试保护下重构。时序/并发变化也算行为变化。

必需子技能

• REQUIRED SUB-SKILL: verification-before-completion

核心流程

1. 定义意图 + 行为边界
2. 建立基线（测试 + 关键指标）
3. 小步实现
4. 验证行为与性能
5. 记录范围 + 回滚点

行为边界清单

• 公共 API 与返回结构
• 数据 schema 与迁移兼容性
• 输出顺序与格式
• 错误类型、消息与状态码
• 时序、并发与副作用
• 资源占用（CPU、内存、I/O）

快速参考

示例

Copytype Api = {
  fetchUser(): Promise<{ id: string }>;
  fetchPosts(): Promise<string[]>;
};

// 边界：输出契约与错误保持不变；时序可能变化。
test('loadUserProfile keeps output contract', async () => {
  const api: Api = {
    fetchUser: async () => ({ id: 'u1' }),
    fetchPosts: async () => ['p1', 'p2'],
  };

  await expect(loadUserProfile(api)).resolves.toEqual({
    userId: 'u1',
    postCount: 2,
  });
});

export async function loadUserProfile(api: Api) {
  const [user, posts] = await Promise.all([
    api.fetchUser(),
    api.fetchPosts(),
  ]);

  return { userId: user.id, postCount: posts.length };
}

常见错误

• 以“只是重构”为由跳过基线测试
• 顺手扩大范围
• 用性能宣称替代测试
• 改了行为却未记录边界
• 高风险变更无回滚计划

借口 vs 事实

红旗 - 立刻停止

• 没有明确的行为边界清单
• 基线测试缺失或失败
• “测试后补”
• “只是清理而已”
• 跨模块大范围变更无范围控制