review-rules · git:20260907.4b46de8 · 2026-09-07 · sha256 b646624f4f34c4bd
review-rules git:20260907.4b46de8A
Immutable. This exact content never changes and is served at /api/v1/blob/b646624f4f34c4bd.
--- name: review-rules description: Apply personal review criteria to code and design reviews, finding rechecks, and review-related repair plans. disable-model-invocation: true --- 将以下准则应用于当前评审对象,以及评审中提出的修改建议。可独立使用,也可配合现有代码或设计评审流程。 评审范围、执行方式和是否修复,沿用用户当前任务。 ## 评审准则 1. **正确且完整** 实现是否满足本次需求?是否存在实际可触发的 bug 或功能遗漏? 少改造的前提是保持功能完整和行为正确。 2. **最少的必要改造** 新增代码是否必要?先核实已有能力,寻找可以复用的实现。 是否有改动范围更小、引入概念更少的等价方案? 同时考虑维护成本,不能只比较 diff 行数。 3. **复杂度与收益相称** 新增抽象、状态和协调机制解决了什么具体问题? 对局部或低概率场景,说明触发条件、后果及处理成本。 不凭假想风险增加复杂设计,也不因其是竞态就否定必要保护。 4. **足够的扩展性** 下一次类似功能是否更容易实现? 用具体变化检验扩展边界,避免为尚无需求的变化预建框架。 5. **更少的理解成本,清晰的责任边界** 下次修改需要理解的上下文是否更少? 状态、决策和资源是否有清晰的 owner? 故障能否定位到负责层?能否整体回退? 是否存在跨层耦合、重复逻辑或多处维护同一规则? ## 形成结论 对提出的问题说明:具体位置、触发场景或维护后果、判断依据,以及最小改法和代价。 区分真实缺陷、改进建议和需要用户决定的取舍。 能从现有实现和明确需求回答的问题,直接回答。 已确认的产品边界继续沿用;没有新证据时不重复列为缺陷。 没有成立的问题时可以明确说没有,不为凑数制造问题。