|
|
3b8204f356
|
docs(审查): 补上 pi 建议的第二格判据（pi 又改了一次它自己的标注）
上一提交只写进了一格判据，pi 随后把它的建议补上 —— 报告里现在是两格：
· `TestHMSAccessTokenFailureDoesNotBurnQuota` 钉内部计数器 dayCount==0
· `TestHMSQuotaSurvivesTokenFailureWithLimitOne` 钉用户看得见的行为
两层都要钉的理由：**计数器对而行为错是可能的** ——
运维会收到「达到每日推送上限」这种**误导性文案**，
而真实原因只是上一次网络抖动。这正是它建议单独加一格的原因。
|
2026-09-28 09:53:12 +08:00 |
|
|
|
87c55acb5f
|
docs(审查): 给 push 报告 §二.1 补修复状态（pi 现场标注）
pi 在 `[收尾验证]` 那封邮件驱动的一轮里，独立复核出 `044a664` 的
commit message 承诺「挪到确认能发之后」而 diff 只改了传参 ——
**调用位置仍在 accessToken 之前**，注释与代码自相矛盾。
它在这份报告上就地标了修复状态。三点值得留在文档里：
① ①（按批次计）真修了且有判据；② 当时**只写进注释、代码没动**。
② 之所以没被当场发现：当时那批判据**造不出「accessToken 失败」这条路**
（`hmsStub` 的 `/token` 永远返回 200 + 令牌）。
③ 现已真正落地，并补判据；把修复回退后判据会红。
★ 教训值得单列：**「我写了注释说明怎么修」不等于「我改了代码」**。
审查报告给了两条，我处理了一条，把另一条誊进注释就当做了。
写完注释应当立刻核对行号 —— 那是 5 秒钟的事。
|
2026-09-28 09:51:28 +08:00 |
|
|
|
2530229180
|
docs(审查): 归档本轮四份代码审查报告
`docs/reviews/` 此前一直是**未跟踪**状态 —— 审查报告只在磁盘上，
不进版本库 ⇒ 换机器、换会话、给别人看时全部拿不到，
而它们正是本轮五个修复（hap 出库 / SSE 写锁 / 换身份清数据 /
鸿蒙门禁三态 / HMS 配额）的**来源**。
push-and-gui-review.md 推送链 + Electron GUI（HMS 配额那两条）
electron-gui-review.md 换身份不清数据
harmony-client-review.md 鸿蒙：门禁 fail-open / MailStore 快照共用 / clear() 零调用方
harmony-pages-review.md 页面层
harmony-state-review.md 状态层
fix-report-2026-09-26.md 上述修复的实施记录
其中 `harmony-client-review.md` §三.1 记的那条值得单独留意：
该报告自己声明「ArkTS 语言规范层面零违规，本文所有问题都是**逻辑缺陷**」——
本次提交的三处鸿蒙改动也只动逻辑（门禁条件、logout 清理），
不碰语法层。
|
2026-09-28 08:46:02 +08:00 |
|
