feat: remove obsolete code and stop on major plan divergence
This commit is contained in:
@@ -46,6 +46,8 @@ Agent 先调查再提问,纠正有证据的错误判断,每轮维护精简
|
||||
|
||||
执行时允许增加必要测试或配置文件并说明理由;涉及业务、接口、架构或其他范围扩展时,需更新计划并确认。验证和 review 通过后创建本地提交,不 push。
|
||||
|
||||
实施时核实引用与兼容用途,移除范围内无用的死代码和被替代的遗留实现。若核心假设、业务语义或架构边界与决策有重大出入,立即停止执行并记录现场,建议回到 align-plan 重新对齐范围和边界;重新确认前不继续实现。
|
||||
|
||||
计划预先评估兼容性,执行结果根据实际改动复核接口、数据、配置及新旧版本影响。结果明确是否需要特殊上线步骤和顺序;需要时提供前置条件、顺序、验证点及必要回退限制,无需时明确说明,未知项如实标注。上线评估不等于实际部署授权。
|
||||
|
||||
## 资产约定
|
||||
|
||||
@@ -40,14 +40,16 @@ description: 执行用户指定的已批准工程计划,验证目标与范围
|
||||
- 每项改动对应批准任务或验收;优先复用现有能力,自主决定具体代码及内部组织,不要求计划提供详细代码或逐字照抄算法示例。
|
||||
- 在最小改动范围内遵循单一职责、开闭原则,按需应用设计模式;避免功能模块耦合及通用架构能力依赖业务规则,降低扩展与审查复杂度,不为模式引入无必要抽象。
|
||||
- 不为缩小 diff 堆叠旧逻辑。发现本次目标相关且计划未覆盖的复用提取或模块解耦机会时,说明代码依据、收益、成本和影响范围,向用户确认;确认纳入后按范围变更流程更新 plan.md 再实施,不擅自重构或重复确认已批准方案。
|
||||
- 核实引用和兼容用途后,移除本次范围内已无用途的死代码、被替代的遗留实现及失效分支,保持代码简洁;不能仅因代码陈旧就删除,范围外清理仍先确认。
|
||||
- 若发现实现与已确认决策有重大出入(如核心假设不成立、业务语义或架构边界必须改变),立即停止执行,保留现场并记录证据、已完成内容和待对齐问题;建议用户回到 align-plan 重新对齐范围与边界。新计划重新确认前不继续实现,不用就地改计划掩盖偏离或宣称完成。
|
||||
- 可补必要测试/配置文件并说明对应验收;其他文件扩展、业务/接口/架构变化或依赖升级须先确认,不能包装为配套改动。无关发现只记录,不顺手修复或重构。
|
||||
- 需变更范围时直接更新同一 plan.md,仅保留最新方案;清除旧批准依据并标待确认,对话只给摘要、原因及链接。用户重新批准后继续,不生成版本副本或历史方案,也不事后改计划掩盖越界。完成前重读文件,若内容已变而未重新批准,先澄清。
|
||||
- 一般范围调整直接更新同一 plan.md,仅保留最新方案;清除旧批准依据并标待确认,对话只给摘要、原因及链接。用户重新批准后继续;重大出入按上述停止流程返回 align-plan,不生成版本副本或历史方案。完成前重读文件,若内容已变而未重新批准,先澄清。
|
||||
|
||||
## 验证、Review 与提交
|
||||
|
||||
1. 完成后重新完整读取原目标与批准计划,对照任务 diff:验收是否满足、保留行为是否成立、是否越界、是否影响调用方与错误路径。
|
||||
2. 运行计划要求及改动必要的检查,覆盖验收与回归风险。复用已有测试,补真实缺口,不堆重复用例或强制覆盖率;文档变更用适当结构检查。记录实际证据,无变化且无新疑点时不重复运行。
|
||||
3. Review 范围、正确性和模块职责/依赖边界,修复本任务问题后重跑受影响检查。越界内容仅撤销能可靠隔离的自身改动。未解决失败或环境阻碍如实报告,不宣称通过、不绕过检查提交;用户调整验证要求时记录差异。
|
||||
3. Review 范围、正确性、模块职责/依赖边界及失效代码清理,修复本任务问题后重跑受影响检查。越界内容仅撤销能可靠隔离的自身改动。未解决失败或环境阻碍如实报告,不宣称通过、不绕过检查提交;用户调整验证要求时记录差异。
|
||||
根据实际 diff 与证据复核接口、数据、配置及新旧版本兼容性。需要特殊上线操作时列出前置条件、步骤顺序、验证点及必要回退限制;否则明确“无需特殊上线步骤或顺序”。未知或未验证部分如实标注,不把本地测试通过等同于上线兼容。涉及计划外迁移或行为变化时先更新计划并确认;评估不授权实际部署。
|
||||
4. 仅暂存本任务文件/片段,检查完整暂存 diff,确保不含用户原有改动、MEMORY 或讨论资产。无法隔离或 MEMORY 已被跟踪时先解决,不擅自取消跟踪。
|
||||
5. 按项目规范创建本地 commit;不 push、amend、跳过 hooks 或改 Git 身份。Hook 改动须重新检查验证。无差异不空提交,非 Git 项目不自行初始化;提交失败报告原因。
|
||||
|
||||
Reference in New Issue
Block a user