pdlc-refactor

# 代码重构 <!-- @include templates/prompts/iron-law.md（已内联于下方，无需另读） --> ⛔ **IRON LAW · 不可违反的硬门禁** 以下规则为**不可协商**的执行约束： 1. **文件必须落盘**：所有带编号（功能ID / 缺陷ID）的文档，必须作为实际文件写入磁盘，不可仅在对话中输出。 2. **阶段必须落章**：每个阶段完成后必须在状态机 `docs/.pdlc-state/<feature-id>.json` 追加 history，不可跳过。 3. **测试必须存在**：进入 `/pdlc-implement` 前，对应测试必须存在且处于红灯状态。违反则中止。 4. **自检必须执行**：段二自检为强制步骤，不得以"已经很好了"为由跳过。 5. **防循环**：段三修复为单次，不递归。无法自动修复的问题记录到报告，继续往下走。 6. **状态必推进**：成功执行某 phase 后 `current_stage` 必须变更。收尾时若发现 `current_stage` 未推进，视为失败并报错，**不得静默返回**（防止外层循环拿滞后的状态空转烧额度）。唯一例外：命中人工点主动 block 时，`current_stage` 保持不变但必须写 `last_phase_result.ok=false` + `blocked_reason`。 **违反任一条 = 立即中止当前命令，输出违规详情，等待人工介入。** <!-- @include-end templates/prompts/iron-law.md --> 对指定服务或模块进行安全重构，保证行为不变。 ## 工作流程 1. **确认测试覆盖**: 先检查现有测试是否充分，不充分则先补测试 2. **阅读设计文档**: 阅读 `docs/02_design/` 对应子目录下的相关设计文档 3. **识别坏味道**: 分析代码中的坏味道（Code Smell） 4. **制定重构计划**: 列出重构项，按风险排序 5. **逐步重构**: 每次只做一个小的重构，确保测试通过 6. **运行完整测试**: 确认没有破坏现有功能 ## 常见重构场景 - **提取方法/类**: 过长的方法、过大的类 - **消除重复**: DRY 原则，提取公共逻辑 - **简化条件**: 复杂的 if-else 链、嵌套条件 - **改善命名**: 不清晰的变量名、方法名 - **解耦依赖**: 降低模块间耦合度 - **统一风格**: 对齐项目编码规范 ## 要求 <!-- @include templates/prompts/output-language.md（已内联于下方，无需另读） --> 🌐 **Output language for generated artifacts** All generated artifacts (PRDs, design docs, code comments, review reports, test plans, deployment manuals, changelog entries, etc.) follow this policy: 1. **Default — match the conversation language exactly**: - 用户用中文与 Claude 对话 → 产中文文档、中文代码注释、中文报告 - User talks to Claude...