Skip to content

fix(desktop): improve Windows tray and local update UX#988

Merged
lefarcen merged 6 commits intomainfrom
feat/resolve-minor-issues
Apr 9, 2026
Merged

fix(desktop): improve Windows tray and local update UX#988
lefarcen merged 6 commits intomainfrom
feat/resolve-minor-issues

Conversation

@PerishCode
Copy link
Copy Markdown
Contributor

What

Improve the Windows desktop packaging and shell experience by adding tray behavior, polishing local update UX, restoring dev renderer loading, and documenting the new desktop inspect workflow.

Why

This branch closes out several acceptance issues around Windows desktop behavior and makes local validation/debugging much clearer. It also avoids misleading update failures in local-dist builds by showing explicit local-validation messaging instead of treating missing production feeds as a normal update error.

How

  • add packaged Windows tray behavior that hides on close, uses the Nexu icon, and localizes tray menu labels for Chinese and English
  • update the Windows installer finish/uninstall flow to support desktop shortcut creation and cleanup
  • introduce a local-dist update policy that disables automatic periodic checks without an explicit feed, adds local validation/test messaging, and keeps manual validation available when a feed is configured
  • restore dev desktop shell loading by using NEXU_DESKTOP_DEV_SERVER_URL in development instead of loading a non-existent packaged HTML file
  • add dev-only desktop inspect commands for screenshot / eval / DOM / renderer-log inspection, plus docs for the workflow
  • merge the latest origin/main into the branch before opening the PR

Affected areas

  • Desktop app (Electron shell)
  • Controller (backend / API)
  • Web dashboard (React UI)
  • OpenClaw runtime
  • Skills
  • Shared schemas / packages
  • Build / CI / Tooling

Checklist

  • pnpm typecheck passes
  • pnpm lint passes
  • pnpm test passes
  • pnpm generate-types run (if API routes/schemas changed)
  • No credentials or tokens in code or logs
  • No any types introduced (use unknown with narrowing)

Notes for reviewers

  • Windows tray behavior is packaged-only; pnpm dev start cannot fully validate tray UX because the tray is only created when app.isPackaged is true.
  • Local validation builds now show explicit update guidance instead of surfacing the production feed 404 directly.
  • The branch already includes a merge from origin/main (02c4408d).

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying nexu-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 02c4408
Status: ✅  Deploy successful!
Preview URL: https://378dfb8f.nexu-docs.pages.dev
Branch Preview URL: https://feat-resolve-minor-issues.nexu-docs.pages.dev

View logs

@sentry
Copy link
Copy Markdown

sentry Bot commented Apr 9, 2026

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 02c4408d58

ℹ️ 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".

Comment thread apps/desktop/main/index.ts Outdated
@lefarcen lefarcen merged commit 6ff9fde into main Apr 9, 2026
15 checks passed
@lefarcen lefarcen mentioned this pull request Apr 10, 2026
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.

2 participants