feat(gateway): enable Android notify + notification events#29440
feat(gateway): enable Android notify + notification events#29440
Conversation
Greptile SummaryEnabled
The implementation is clean, well-tested, and follows existing patterns for node event handling. Confidence Score: 5/5
Last reviewed commit: 408b665 |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 408b665b4c
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
apps/android/app/src/main/java/ai/openclaw/android/node/DeviceNotificationListenerService.kt
Show resolved
Hide resolved
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d9b71880eb
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 13825858c7
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
197f57a to
021b755
Compare
(cherry picked from commit 37fb826)
(cherry picked from commit 37fb826)
(cherry picked from commit 37fb826)
(cherry picked from commit 3899c89)
(cherry picked from commit 3899c89)
Summary
system.notifyin Android default node command allowlistnotifications.changedfrom Android notification listener on post/removenotifications.changedin gateway node-event pipeline and enqueue system eventsTesting
pnpm test -- src/gateway/server-node-events.test.ts src/gateway/gateway-misc.test.tscd apps/android && ./gradlew testDebugUnitTest --tests ai.openclaw.android.node.NotificationsHandlerTest