refactor: clarify minimal changes and confirm reuse opportunities
This commit is contained in:
@@ -41,6 +41,7 @@ description: 通过调查和多轮问答对齐工程目标,生成待审阅计
|
||||
- 简述证据与推荐理由,不用装饰性 emoji。纠正错误事实,区分假设与偏好;用户提出想法不等于确认方案。
|
||||
- 优先复用现有能力,选择满足目标的最小方案。出现独立子目标或明显扩大的改造时,说明扩大点、建议分阶段,由用户选择;未确认前保持原范围。
|
||||
- 设计遵循最小改动、单一职责和开闭原则,按实际变化点采用合适设计模式;隔离功能模块,通用架构能力不耦合具体业务规则。以降低扩展和审查复杂度为准,不为套模式新增无必要抽象或扩大重构。
|
||||
- 最小改动不等于最少行数或继续堆叠旧逻辑。发现与本次目标相关、能减少重复或耦合的提取机会时,对比沿用与提取方案的收益、成本及影响文件,纳入当轮 1–3 问向用户确认,再决定是否纳入计划。
|
||||
- 目标、边界、关键取舍和验收明确后停止提问。
|
||||
|
||||
## 写入、审阅与交接
|
||||
|
||||
@@ -39,6 +39,7 @@ description: 执行用户指定的已批准工程计划,验证目标与范围
|
||||
|
||||
- 每项改动对应批准任务或验收;优先复用现有能力,自主决定具体代码及内部组织,不要求计划提供详细代码或逐字照抄算法示例。
|
||||
- 在最小改动范围内遵循单一职责、开闭原则,按需应用设计模式;避免功能模块耦合及通用架构能力依赖业务规则,降低扩展与审查复杂度,不为模式引入无必要抽象。
|
||||
- 不为缩小 diff 堆叠旧逻辑。发现本次目标相关且计划未覆盖的复用提取或模块解耦机会时,说明代码依据、收益、成本和影响范围,向用户确认;确认纳入后按范围变更流程更新 plan.md 再实施,不擅自重构或重复确认已批准方案。
|
||||
- 可补必要测试/配置文件并说明对应验收;其他文件扩展、业务/接口/架构变化或依赖升级须先确认,不能包装为配套改动。无关发现只记录,不顺手修复或重构。
|
||||
- 需变更范围时直接更新同一 plan.md,仅保留最新方案;清除旧批准依据并标待确认,对话只给摘要、原因及链接。用户重新批准后继续,不生成版本副本或历史方案,也不事后改计划掩盖越界。完成前重读文件,若内容已变而未重新批准,先澄清。
|
||||
|
||||
|
||||
Reference in New Issue
Block a user