Harden macOS capture setup and cursor moves

Updates CI on macOS to fully configure screen recording by writing replayd approval data, restarting replayd, and clarifying step messaging. In QtTrayMenu, adds a bounded cursor-position wait loop with polling and timeout so mouse move/restore checks are more reliable instead of relying on a single immediate position read.

Author: ReenigneArcher

.github/workflows/ci.yml

@@ -123,7 +123,7 @@ jobs:
           )
           brew install "${dependencies[@]}"
 
-      - name: Fix macOS screen recording permissions
+      - name: Configure macOS screen recording
         if: runner.os == 'macOS'
         run: |
           set -euo pipefail
@@ -156,7 +156,14 @@ jobs:
             configure_user_tccdb "$values,NULL,NULL,'UNUSED',${values##*,}"
           done
 
-          echo "macOS TCC permissions configured."
+          approvalDirectory="$HOME/Library/Group Containers/group.com.apple.replayd"
+          approvalPlist="$approvalDirectory/ScreenCaptureApprovals.plist"
+          mkdir -p "$approvalDirectory"
+          defaults write "$approvalPlist" "/bin/bash" -date "$(date -u '+%Y-%m-%d %H:%M:%S +0000')"
+          defaults read "$approvalPlist" "/bin/bash"
+          killall replayd 2>/dev/null || true
+
+          echo "macOS screen recording configured."
 
       - name: Setup Dependencies Windows
         if: runner.os == 'Windows'

src/QtTrayMenu.cpp

@@ -3,7 +3,9 @@
  * @brief Definitions for Qt tray menu implemenation
  */
 // standard includes
+#include <chrono>
 #include <filesystem>
+#include <thread>
 
 // qt includes
 #include <QApplication>
@@ -22,12 +24,28 @@
 
 namespace {
   constexpr int DEFAULT_PANEL_THICKNESS = 24;
+  constexpr int CURSOR_POSITION_POLL_INTERVAL_MS = 10;
+  constexpr int CURSOR_POSITION_TIMEOUT_MS = 500;
   constexpr int CURSOR_POSITION_TOLERANCE = 2;
 
   bool positionsAreClose(const QPoint &first, const QPoint &second) {
     return (first - second).manhattanLength() <= CURSOR_POSITION_TOLERANCE;
   }
 
+  bool waitForCursorPosition(const QPoint &targetPosition, const QRect &targetGeometry = {}) {
+    const auto deadline = std::chrono::steady_clock::now() + std::chrono::milliseconds(CURSOR_POSITION_TIMEOUT_MS);
+    do {
+      const QPoint currentPosition = QCursor::pos();
+      if (targetGeometry.isValid() ? targetGeometry.contains(currentPosition) : positionsAreClose(currentPosition, targetPosition)) {
+        return true;
+      }
+      std::this_thread::sleep_for(std::chrono::milliseconds(CURSOR_POSITION_POLL_INTERVAL_MS));
+    } while (std::chrono::steady_clock::now() < deadline);
+
+    const QPoint currentPosition = QCursor::pos();
+    return targetGeometry.isValid() ? targetGeometry.contains(currentPosition) : positionsAreClose(currentPosition, targetPosition);
+  }
+
   bool fallbackTrayIconPosition(QPoint *position) {
     const QScreen *screen = QGuiApplication::primaryScreen();
     if (screen == nullptr) {
@@ -405,8 +423,7 @@ bool QtTrayMenu::positionMouseOverIcon() {
     mousePositionSaved = true;
   }
   QCursor::setPos(targetPosition);
-  const QPoint currentPosition = QCursor::pos();
-  const bool positioned = iconGeometry.isValid() ? iconGeometry.contains(currentPosition) : positionsAreClose(currentPosition, targetPosition);
+  const bool positioned = waitForCursorPosition(targetPosition, iconGeometry);
   if (!positioned) {
     qWarning("QtTrayMenu: could not position the mouse over the tray icon");
   }
@@ -419,8 +436,8 @@ bool QtTrayMenu::restoreMousePosition() {
   }
 
   QCursor::setPos(savedMousePosition);
+  const bool restored = waitForCursorPosition(savedMousePosition);
   mousePositionSaved = false;
-  const bool restored = positionsAreClose(QCursor::pos(), savedMousePosition);
   if (!restored) {
     qWarning("QtTrayMenu: could not restore the saved mouse position");
   }