Skip to content

fix(android): 🐛 修复并发入队时日志按落盘顺序覆盖事件时间的问题 - #21

Closed
IceTeacher wants to merge 1 commit into
devfrom
fix/log-event-order
Closed

IceTeacher wants to merge 1 commit into
devfrom
fix/log-event-order

Conversation

@IceTeacher

Copy link
Copy Markdown
Owner

变更

  • 已发布日志按事件时间、写入序号和 id 保持最新在前。
  • 后入队但事件更早的记录不再直接插到列表前端。
  • 文件读取与增量发布使用同一排序。

验证

  • 回归测试先在旧的前端插入实现上失败。
  • :android:testDebugUnitTest --tests cn.rainafter.app.gazekeep.notification.LogOrderTest --tests cn.rainafter.app.gazekeep.notification.DecisionLogStoreTest 通过。
  • pnpm verify:docs、pnpm format:check、git diff --check 通过。
  • 未执行真机日志页顺序;本次不改变采集、后端或熄屏行为。

对应 PR #10 的 Codex P2。

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ⚠️ Failed 2026-10-31T12:07:22.792268Z
We couldn’t complete this review. Comment @codex review to try again.
b994b09 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@IceTeacher

Copy link
Copy Markdown
Owner Author

关闭此 PR,修复改在文档分支继续处理。

@IceTeacher IceTeacher closed this Oct 31, 2026
@IceTeacher
IceTeacher deleted the fix/log-event-order branch October 31, 2026 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant