Link to issue number:

Fixes #17881
Fixes #17875

Summary of the issue:

Currently, the push clipboard command:

1. Fails with a logged error but no useful output when attempting to push a non-text clipboard
2. Reports success when there is no-one else connected.

Description of user facing changes

1. An error message is reported when attempting to push non-text clipboard
2. An error message is reported when pushing the clipboard to an empty channel

Description of development approach

1. Catch OSError when attempting to push clipboard, as this is raised when there is no text content on the clipboard.
2. Record connected leaders and followers on both leader and follower sessions. Add read-only properties, connectedLeadersCount, connectedFollowersCount, and connectedClientsCount (sum of leaders and followers counts) to RemoteSession.
3. Add a read only property, connectedClientsCount to RemoteClient, which grabs the connectedClientsCount from the appropriate session, or returns 0 if there is no session in progress.
4. Check that the remote client's count of connected clients is greater than 0 before pushing the clipboard.

Testing strategy:

Attempt pushing the clipboard:

• In a new channel where the local machine is the only connected machine
• In a channel where there is a remote that connected after the local machine, and after the remote machine has disconnected
• In a channel where there is another client that joined before the local machine, and after the remote machine has disconnected

Known issues with pull request:

Code Review Checklist:

• Documentation:
  • Change log entry
  • User Documentation
  • Developer / Technical Documentation
  • Context sensitive help for GUI changes
• Testing:
  • Unit tests
  • System (end to end) tests
  • Manual testing
• UX of all users considered:
  • Speech
  • Braille
  • Low Vision
  • Different web browsers
  • Localization in other languages / culture than English
• API is compatible with existing add-ons.
• Security precautions taken.

@coderabbitai summary