AI给我重构了2000行代码，测试红了12个用例

有些项目，确实能让开发者头疼一阵子。

比如这个去年赶工期上线的项目：2000行代码，组件里散落着七八个useState，同样的表单验证逻辑复制粘贴了三遍，有个函数嵌套了5层if-else，理清逻辑分支都得好一会儿。

本想用Claude Code直接重构，结果AI一通操作，代码是变了，跑测试直接挂了一片。原因很典型——它把一个函数的参数从5个改成了对象形式，但只改了函数定义，30个调用点一个都没同步更新。

最后花了半小时手动修复，一边修一边想：AI写代码已经够用，但让AI"改代码"这个事，好像还缺了点什么。

后来在GitHub Trending上看到addyosmani/agent-skills，来自Google工程总监Addy Osmani的21个AI编程工程技能。刚开始没太在意——又是"AI工具包"？但翻了里面的prompt模板后发现，它解决的不是"AI怎么写代码"，而是"AI怎么改代码"。

这个区别很关键。写代码，AI已经得心应手；但改代码、重构代码、审查代码，AI经常翻车。而翻车不是因为AI笨，是因为它缺乏一套靠谱的工作流程。

先说翻车那次

回到开头那个项目。第一次让Claude Code重构时，指令很简单："帮我重构这个模块"。AI确实干了不少活：嵌套5层的if-else被改成了策略模式，代码从80行缩到40行，这个改动服气。

但问题也不少。除了前面说的参数改了但调用点没更新，还有一次更离谱——git diff一看，它顺手把eslint配置也给改了。直到现在也想不通，为什么重构一个TypeScript模块会动到eslint配置……

而且AI特别爱"过度优化"。一个50行的函数，AI建议拆成5个独立函数，理由是"职责单一"。50行啊，拆5个？最后手动合并成2个，算是各退一步。后来还有个20行的函数也被建议拆成3个，理由依然是"单一职责原则"——这种直接忽略，20行可读性本就不错，拆了反而增加理解成本。

后来尝试了agent-skills的重构技能

agent-skills的重构技能核心思路很简单：

。每次只改一个关注点，改完跑测试，没问题再继续。不是AI自己判断"改完了"，而是每一步都要验证。

听起来很常识对吧？但之前让AI重构，就是一口气全改完，跑测试发现一堆问题，再回头一个一个修。agent-skills的做法完全拆开了——AI提计划，开发者确认，改一步，验证一步。

按这套流程重构了另一个模块，体验完全不一样。AI列出重构计划，确认合理后开始执行，每改完一处就等反馈。参数改成对象形式那次，AI主动列出了30个需要同步修改的调用点，一个一个确认——虽然过程慢了点，但测试一次通过。

：AI重构必须开git，每步改完都commit。这是血泪经验。

代码审查的意外收获

重构完顺便跑了agent-skills的代码审查技能，本来没抱期待。

结果AI指出了一条很扎心的——"表单验证逻辑在3个地方重复出现，建议抽成公共方法"。这条其实一直知道，就是懒得改。被AI当面指出来，跟被同事code review时说"这段你重复了"一样，有点尴尬但不得不认。

有意思的是AI描述问题的方式。它不是说"你有重复代码"，而是说"重复的验证逻辑会导致维护成本线性增长"。这种措辞更容易让人接受——不知道prompt模板里是不是专门调过。

当然审查结果也有水分，比如建议把某些变量命名从data改成更具体的名字，这种属于正确但优先级很低的建议，一般扫一眼就过了。

性能分析这块比较鸡肋

也试了性能分析技能，跑了一个数据可视化的组件。

AI给出的还是老三样：虚拟列表、减少重渲染、避免在render里创建新对象。这些建议没错，但太泛了。拿React DevTools Profiler的结果对比，AI大概能覆盖60%的问题——剩下40%才是真正的瓶颈。比如某个context的value每次render都变，导致整棵子树重渲染，这种AI没找出来。

性能这块的常见做法是：先让AI扫一遍排雷，明显的它找得还行；真正的性能瓶颈靠自己用Profiler定位。当个筛子用就行，别指望太多。

怎么用这套技能的

现在的工作流大概是：

写新功能时正常用Claude Code，该写写该改改。到了需要重构或review的阶段，把agent-skills里对应的skill prompt贴到system prompt里，让AI按那个流程走。不是每次都贴，只在改老代码、改别人的代码、或改完要跑测试的时候才用。

git clone 

说实话，加了这套技能后AI并没有变"聪明"——它不会突然理解业务逻辑，也不会自己发现那些隐蔽的bug。但它变得更"守规矩"了：重构不会一口气全改了，审查不会漏掉重复逻辑，每一步都有验证。

这就够了。AI编程工具的问题从来不是"不够聪明"，而是"不够靠谱"。agent-skills做的事就是把"靠谱"这个属性变成可复制的流程。

它不是让你用AI写更好的代码，而是让你用AI改代码的时候少翻车。就冲这一点，花那一小时配置时间也值得。