Refactor Safely

Plans and executes safe refactors with small steps, tests, and rollback strategy. Use when 重构, 技术债, 安全重构, refactoring, or improving code structure without br...

Security Scan

The skill is an instruction-only, domain‑specific refactoring helper; its requirements and instructions align with its stated purpose and it does not request credentials, install code, or perform unexpected actions.

Purpose & Capability

Name/description match the content: SKILL.md describes stepwise, test-first refactoring guidance. There are no unrelated environment variables, binaries, or installs requested.

Instruction Scope

Instructions are limited to planning and executing refactor steps, tests, and rollback strategy. They do not direct the agent to read unrelated system files, exfiltrate data, or call external endpoints.

Install Mechanism

No install spec or code files are present; this is instruction-only so nothing is downloaded or written to disk by the skill itself.

Credentials

No environment variables, credentials, or config paths are required. The requested scope is proportional to a planning/advisory refactor skill.

Persistence & Privilege

always is false and the skill is user-invocable; it does not request permanent presence or modify other skills or system settings.

Assessment

This skill is a harmless, instruction-only refactoring advisor. Before using it, remember: you must provide the code/context you want refactored (the skill won't access your repos or files by itself). Don't paste secrets or credentials into prompts. If you plan to let an agent with repository access act autonomously, ensure the agent's repository permissions and branch protections are set appropriately (this skill itself doesn't request credentials or perform installs).

安全重构（Refactor Safely）

在不改变对外行为的前提下，分步、可验证地改善代码结构，降低技术债与回归风险。

触发场景

• 用户说「重构这段代码」「清理技术债」「安全重构」「改善结构但不要改行为」
• 代码难以维护、重复多、职责混乱、难以加测试

原则

• 行为不变：重构前后功能与边界行为一致（含异常、边界值）
• 小步提交：每步可单独提交、易回滚、易 code review
• 先测后动：有测试则先绿再动；无测试则先补关键路径再重构
• 可随时停：任意一步停下，代码仍处于可运行、可发布状态

执行流程

0. 判断是否适合重构

先问自己三个问题，任意一个答案是"否"就先解决它再重构：

• 有没有测试覆盖核心路径？（没有 → 先补关键路径测试）
• 重构范围是否清晰？（不清晰 → 先和用户对齐范围）
• 当前代码能跑通吗？（跑不通 → 先修 bug 再重构）

1. 明确范围与目标

• 范围：单文件 / 单模块 / 跨模块（列出受影响入口与调用方）
• 目标：例如「抽公共函数」「拆组件」「统一错误处理」「去掉重复」
• 不做什么：明确不在此次重构里改的需求或行为

2. 识别依赖与风险

• 谁在调用当前代码（组件、API、路由、全局状态）
• 是否有隐式依赖（全局变量、副作用、执行顺序）
• 高风险点：无类型、无测试、多入口共用的逻辑

3. 拆分步骤（每步可单独提交）

步骤类型	示例
仅移动	把函数/组件挪到新文件，改 import，跑通
仅重命名	变量/函数/文件重命名，全局替换后跑通
抽离不改逻辑	抽子函数/子组件，原逻辑不变，调用处改为用新的
替换实现	用新实现替换旧实现，对外接口不变
删废弃	确认无引用后删除死代码或废弃 API

4. 每步验证

• 构建通过、类型通过、现有测试通过
• 若有可能：关键页面/流程手工点一遍或跑 E2E
• 提交信息写清本步做了什么（如 "refactor: extract getDisplayName"）

5. 回滚策略

• 每步在版本控制里独立提交，便于 git revert
• 若涉及配置或数据，注明回滚时需恢复的内容

输出模板

## 安全重构计划

### 目标与范围
- 目标：…
- 范围：…（文件/模块列表）
- 不改变：…（行为/接口约定）

### 依赖与风险
- 调用方：…
- 风险点：…

### 步骤（每步可单独提交）
1. … — 验证：…
2. … — 验证：…
3. …

### 回滚
- 按提交逐条 revert 即可 / 需注意：…

常见模式

• 抽函数：先复制一份成新函数，调用处改为新函数，再删旧实现
• 抽组件：先在新文件写组件，原处用新组件包裹原 JSX，再逐步把 props 捋清
• 改数据结构：可先兼容旧结构（新旧并存），再逐步迁移调用方，最后删旧结构
• 重命名：用 IDE 重命名/全局替换，避免手改漏改