Skip to content

fix(desktop): handle packaged debug shortcut via before-input-event#939

Merged
lefarcen merged 3 commits intomainfrom
fix/windows-debug-shortcut-fallback
Apr 8, 2026
Merged

fix(desktop): handle packaged debug shortcut via before-input-event#939
lefarcen merged 3 commits intomainfrom
fix/windows-debug-shortcut-fallback

Conversation

@lefarcen
Copy link
Copy Markdown
Collaborator

@lefarcen lefarcen commented Apr 8, 2026

What

Handle the packaged Develop-mode toggle inside before-input-event so Cmd/Ctrl+Shift+Alt+D works even when the global shortcut registration is unavailable.

Why

On Windows, other software can block the app-level global shortcut registration, which leaves packaged builds stuck without a working way to reveal the Develop menu for debugging.

How

Add a fallback handler in the packaged before-input-event path that toggles productionDebugMode and refreshes the application menu before the reload-shortcut guard runs.

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 (blocked in this worktree: dependencies / local tools missing, tsc not found)
  • pnpm lint passes (blocked in this worktree: biome not found)
  • pnpm test passes (blocked in this worktree: vitest not found)
  • 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

Manual verification target: packaged Windows build, press Ctrl+Shift+Alt+D, confirm the Develop menu appears, and confirm reload shortcuts remain blocked when debug mode is off.

@sentry
Copy link
Copy Markdown

sentry Bot commented Apr 8, 2026

Codecov Report

❌ Patch coverage is 0% with 28 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
apps/desktop/main/index.ts 0.00% 28 Missing ⚠️

📢 Thoughts on this report? Let us know!

@lefarcen lefarcen merged commit d6720ea into main Apr 8, 2026
15 checks passed
@lefarcen lefarcen mentioned this pull request Apr 8, 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