fix(remote-desktop): resolve Firefox stuck right-click via mouse state reconciliation

[Krista House (kristahouse)]

Summary

Fixed a Firefox-specific issue where right-clicking inside the remote desktop canvas could leave the server-side right mouse button state stuck as pressed.

Changes

Input cleanup

• onmouseleave now releases all active inputs through mouseOut()
• Added focusLost() handling for:
  • window blur
  • document visibilitychange
• This ensures inputs are released when the user leaves the canvas, switches tabs, or the browser loses focus

Mouse state reconciliation
Updated mouseIn() to compare event.buttons against tracked server-side button state.
If the browser no longer reports a button as held, the service releases it server-side.

This covers:

1. left button
2. right button
3. middle button

Releases are batched into a single applyInputs call.

Listener cleanup

• Hoisted captureKeys so it can be properly removed
• onDestroy now removes:
  • keydown
  • keyup
  • blur
  • visibilitychange

Tests
Added mouseInput.test.ts covering:

• mouseOut
• focusLost
• mouseIn reconciliation
• missing-session guards
• release batching
• no-op behavior when no release is required

[Copilot]

Pull request overview

This PR addresses a Firefox-specific “stuck right-click” behavior in the web remote desktop client by reconciling mouse button state when re-entering the canvas and releasing inputs when the page loses focus/visibility.

Changes:

• Add mouse button reconciliation on mouseenter and introduce focusLost() input release in RemoteDesktopService.
• Update the Svelte component to release inputs on window blur / tab hidden, and remove the prior mouseleave button-state call.
• Add Vitest regression coverage for mouse enter/leave and focus-loss input release behavior.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
web-client/iron-remote-desktop/src/services/remote-desktop.service.ts	Reconciles event.buttons on mouse re-entry and adds focusLost() to release inputs.
web-client/iron-remote-desktop/src/services/mouseInput.test.ts	Adds regression tests covering mouse-out, focus-lost, and mouse-in button reconciliation.
web-client/iron-remote-desktop/src/iron-remote-desktop.svelte	Hooks blur/visibility events to release inputs and removes the prior mouseleave button-state call.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.