vb2250158/gamedevelopmentskills/programming-design-review

编程设计巡检

选择范围与输出

本 Skill 是已有逻辑的分析与巡检入口；编程设计风格 是写逻辑前的设计与实现入口。任务同时包含检查和修改时，先判断已有逻辑，再在动手前应用设计规范；不把只读分析自动变成修改。

两类问题可以混合，但不把简单函数审查扩成全仓架构报告。不凑问题数量，不自动建立文件、Hook 或平台；审查请求本身不授权改代码。

详细模式

用户要求“重点检查、详细审查、查清来龙去脉”，或本轮发现重要既有逻辑的疑点，可能影响正式数据、存档兼容、资产/奖励、权限、共享接口或多模块行为时，读取 详细审查与历史取证。主动启用时简短说明对象和具体影响依据，不能只因代码旧、模块大或个人不熟悉就升级。

先形成有位置和触发条件的候选问题，再为每个仍可能需要修复的问题安排独立子 Agent 审查；确认相关来源及历史后形成具体到代码、函数、配置和资源的方案。普通命名、格式或低影响局部检查沿用常规流程。

详细模式增加调查深度，不改变原任务权限。已授权修复时，完成必要取证和方案复核后继续实施；仅要求审查或方案时保持只读。

规则只有一个来源

先读取 编程设计风格，按其条件表加载适用参考；同一版本已读内容复用。本 Skill 不另维护一份设计规则。源文件缺失时说明哪些规范未能核对，仍可按项目明确规则检查，但不能宣称完成用户风格审查。

审查顺序

1. 确定目标：使用用户给定的文件、模块、功能或差异范围；检查相关项目说明和已有改动。单文件审查不强制要求 Git 比较基线。范围不明时从项目入口选择一个能说明理由的小范围，不默认全仓扫描。
2. 寻找证据：先定位定义、调用者、数据拥有者、配置/持久化入口和现有检查工具，再判断。涉及表现入口时沿真实链路查到业务模块和数据源，不能只读当前 UI。旧文档与实际调用冲突时用代码、配置和历史核对。
3. 核对适用规则：按下表寻找证据，具体要求使用设计 Skill 的对应参考。不因某个词出现就认定问题；没有证据的推断列为待确认。
4. 验证并报告：运行与目标匹配且不超出授权的最小检查；先确认命令是否会写文件，纯审查使用检查模式。按行为风险排序，说明位置、原因、最小改法、验证结果及未覆盖范围。

集合数据的局部失败审查

只要涉及集合数据的查询、转换、关联补全、刷新、展示或批量操作，都必须检查本节；不以用户另行提醒为前提。工程判定复用 最小范围失败与验证要求。

1. 沿实际调用链区分单项坏数据、关联对象缺失、某项请求失败与整次查询/鉴权失败。检查解析、映射、关联查询、并行汇总、分页和批量循环是否因单项异常中断其余有效数据。
2. 在正确性允许时跳过、隐藏或局部标记异常项，让正常数据继续刷新、展示和操作。视频或用户信息异常不能拖垮其他视频的播放和留言；异常邮件不展示也不能阻断正常邮件加载和领取；排名关联对象缺失可隐藏该行或显示“无法加载”，不能使整榜不可用。隔离坏项后仍按真实游标/分页契约继续，不能把过滤后的空页当作没有后续数据。
3. 检查刷新失败是否错误清空有效展示，单项失败是否错误回滚其他已确认成功的独立操作，以及部分失败是否被报告成全部成功。保留定位与恢复依据，按实际需要使用局部提示或有界恢复，不为展示伪造业务身份、奖励、余额或完整性结论。涉及鉴权、关键数据和原子事务时停止受影响的操作，让无依赖的功能继续；不能用全局吞错维持表面可用。
4. 对审查建议本身执行同样检查：说明修改后故障影响范围与正常数据可用性；只建议改成抛异常、加校验或拒绝整批，不能算完成方案。依据现有入口验证一个坏项夹在正常项中、批量中途失败、刷新整次失败及相关分页边界；静态取证与实际运行分别报告，未执行的验证明确列为缺口。

结论格式

• 设计矛盾：职责、数据归属、依赖或协作方式有问题。
• 代码规范问题：命名、结构、控制流、错误处理、注释、日志、格式或测试有可定位问题。
• 每项给出文件与准确行号、违反的适用规则、触发条件/后果、最小改法和验证证据。设计类附必要的调用/数据关系；不以抽象架构术语替代具体问题。
• 区分已观察事实、推断和待确认项；没有行级证据不写成确定缺陷。证据可以支持多个问题，但不要反复粘贴同一规则。
• 没有发现时说明检查范围及验证状态，不制造候选。用户已给方向时继续该方向，不固定追加“先深入哪个”的问题。
• 未决取舍确实影响职责、数据归属、兼容性或成本时，再按设计 Skill 比较方案；不把评审意见自动扩成实施计划。