Skip to content

feat: add desktop notification bridge#279

Merged
rogerchappel merged 1 commit intocodex/desktop-shell-scaffoldfrom
codex/desktop-notifications
May 1, 2026
Merged

feat: add desktop notification bridge#279
rogerchappel merged 1 commit intocodex/desktop-shell-scaffoldfrom
codex/desktop-notifications

Conversation

@rogerchappel
Copy link
Copy Markdown
Owner

Summary

  • add desktop notification bridge methods for permission, requestPermission, and show
  • validate notification payloads and keep routes constrained to CrewCmd paths
  • focus the shell and navigate to the supplied CrewCmd route when a notification is clicked

Stack

Verification

  • git diff --check
  • node --check apps/desktop/src/main.cjs && node --check apps/desktop/src/preload.cjs
  • cd apps/desktop && pnpm install --frozen-lockfile
  • cd apps/desktop && pnpm check
  • pnpm exec tsc --noEmit --pretty false
  • push hook: pnpm typecheck
  • push hook: pnpm build

Risk

Medium. Adds the first native desktop value, but does not add server event subscriptions, persistent permissions, secure storage, tray, updater, signing, or OpenClaw runtime bundling.

@rogerchappel rogerchappel merged commit ccdf145 into codex/desktop-shell-scaffold May 1, 2026
rogerchappel added a commit that referenced this pull request May 1, 2026
* chore: scaffold desktop shell

* feat: add desktop native bridge skeleton (#276)

* docs: add desktop release checklist (#277)

* docs: define desktop gateway discovery (#278)

* feat: add desktop notification bridge (#279)

* fix: satisfy desktop lint rules
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