code-reviewlisted

## 目标 检查 6 个维度：重复代码、逻辑冲突、多余逻辑、SRP违反、模块耦合、硬编码/死代码 ## 执行步骤 检查以下 6 个维度，用🔴严重/🟡警告/🟢建议分级： 1. **重复代码** - 相同或相似的代码逻辑在多处出现 2. **逻辑错误** - 存在矛盾、互相抵消、错误的逻辑 3. **多余逻辑** - 不必要的计算、判断或冗余代码 4. **SRP违反** - 单一职责原则违反，一个函数/类做了太多事情 5. **模块耦合** - 模块间依赖关系过于复杂或紧耦合 6. **硬编码/死代码** - 魔法数字、未使用的代码、永远不会执行的分支 ## 输出格式 ```markdown ## 问题 1 — 重复逻辑 - 位置: `src/foo.ts` 行 120-160；`src/bar.ts` 行 20-40 - 描述: 两处实现相同的数据转换逻辑 - 严重度: 🟡警告 - 推荐修复: 提取 `utils/transform.ts`，新增单元测试 ``` ## 注意 - 只进行观察和分析，不修改代码直到用户确认 - 优先报告严重问题（🔴），再是警告（🟡），最后是建议（🟢） - 每个问题都要说明"为什么这是个问题"和"如何修复" - 如果代码整体质量很好，也要明确指出