diff --git a/docs/reviews/push-and-gui-review.md b/docs/reviews/push-and-gui-review.md index 0a6623b..e4512ff 100644 --- a/docs/reviews/push-and-gui-review.md +++ b/docs/reviews/push-and-gui-review.md @@ -58,6 +58,17 @@ tok, err := h.accessToken(ctx) // ← 额度已经扣了 **修法**:`reserveDaily(1)`(按批次计),并把预留挪到**确认拿到 access_token 之后**、 或失败时归还。 +> **修复状态(2026-09-28)**:① 已修(`044a664`,`reserveDaily(1)`,有判据 +> `TestHMSDailyLimitCountsMessagesNotTokens`)。 +> ② **曾被声称已修但没落地** —— `044a664` 的 commit message 承诺「挪到确认能发之后」, +> diff 里却只改了传参,**调用位置仍在 `accessToken` 之前**,注释与代码自相矛盾。 +> 之所以没被当场发现,是因为当时那批判据**造不出「accessToken 失败」这条路**。 +> 现已真正落地(预留移到 `accessToken` 成功之后、发请求之前), +> 并补上判据 `TestHMSAccessTokenFailureDoesNotBurnQuota`: +> 把修复回退后该判据**失败**(`accessToken` 失败 3 次 → `dayCount=3`), +> 修复在位则 `dayCount=0`。抓住这处不一致的是 pi。 + + ### 2.【HIGH】HMS 通知载荷缺 `click_action`,锁屏点击可能带不出 `data` `hms.go:206-217`: