MRP-2 T14: Eliminate all processEvents() calls — CLAUDE.md rule #8 compliance

Replace blocking loop + processEvents in external_games_dialog with
QTimer.singleShot batching (5 games per tick). Reorder setLabelText
before setValue in edit_actions (setValue already processes events
internally via QProgressDialog). Remove unused QApplication import.

grep -r "processEvents" src/ now returns 0 code hits.

Author: HeikesFootSlave

src/ui/actions/edit_actions.py

@@ -16,7 +16,7 @@
 
 from typing import TYPE_CHECKING
 
-from PyQt6.QtWidgets import QMessageBox, QApplication
+from PyQt6.QtWidgets import QMessageBox
 
 from src.core.game_manager import Game
 from src.services.curator_client import CuratorRecommendation
@@ -227,10 +227,9 @@ def _do_auto_categorize(self, settings: dict) -> None:
                 def tags_progress(index: int, name: str) -> None:
                     if progress.wasCanceled():
                         return
-                    progress.setValue(step + index)
                     if index % 10 == 0:
                         progress.setLabelText(t("auto_categorize.status_tags", game=name[:30]))
-                    QApplication.processEvents()
+                    progress.setValue(step + index)
 
                 self.mw.autocategorize_service.categorize_by_tags(
                     games, tags_count=settings["tags_count"], progress_callback=tags_progress
@@ -249,9 +248,8 @@ def tags_progress(index: int, name: str) -> None:
                 }
                 included_types = {rec_map[r] for r in rec_strings if r in rec_map}
 
-                progress.setValue(step)
                 progress.setLabelText(t("auto_categorize.curator_fetching"))
-                QApplication.processEvents()
+                progress.setValue(step)
 
                 try:
                     self.mw.autocategorize_service.categorize_by_curator(

src/ui/dialogs/external_games_dialog.py

@@ -7,7 +7,7 @@
 from __future__ import annotations
 
 import logging
-from PyQt6.QtCore import Qt, QThread, pyqtSignal
+from PyQt6.QtCore import Qt, QThread, QTimer, pyqtSignal
 from PyQt6.QtWidgets import (
     QCheckBox,
     QComboBox,
@@ -445,38 +445,50 @@ def _start_add(self, games: list[ExternalGame]) -> None:
             self._add_thread.start()
 
     def _add_with_platform_tags(self, games: list[ExternalGame]) -> None:
-        """Adds games with per-platform category tags synchronously in a thread.
+        """Adds games with per-platform category tags using non-blocking batching.
 
-        Groups games by platform and uses each platform name as the tag.
+        Uses QTimer.singleShot to yield to the event loop between batches,
+        keeping the dialog responsive without resorting to processEvents().
 
         Args:
             games: Games to add.
         """
-        # Group by platform for proper tagging
         self._progress_bar.setVisible(True)
         self._progress_label.setVisible(True)
         self._progress_bar.setMaximum(len(games))
 
-        stats = {"added": 0, "skipped": 0, "errors": 0}
-        for i, game in enumerate(games):
-            self._progress_bar.setValue(i + 1)
-            self._progress_label.setText(t("ui.external.adding_game", name=game.name, current=i + 1, total=len(games)))
-            # Process events to keep UI responsive
-            from PyQt6.QtWidgets import QApplication
+        self._batch_stats = {"added": 0, "skipped": 0, "errors": 0}
+        self._batch_games = games
+        self._add_batch(0)
 
-            QApplication.processEvents()
+    def _add_batch(self, index: int) -> None:
+        """Processes a batch of games and schedules the next batch.
 
+        Args:
+            index: Current index in the game list.
+        """
+        if index >= len(self._batch_games):
+            self._on_add_finished(self._batch_stats)
+            return
+
+        batch_end = min(index + 5, len(self._batch_games))
+        for i in range(index, batch_end):
+            game = self._batch_games[i]
+            self._progress_bar.setValue(i + 1)
+            self._progress_label.setText(
+                t("ui.external.adding_game", name=game.name, current=i + 1, total=len(self._batch_games))
+            )
             try:
                 tag = self._collection_name_for_platform(game.platform)
                 if self._service and self._service.add_to_steam(game, category_tag=tag):
-                    stats["added"] += 1
+                    self._batch_stats["added"] += 1
                 else:
-                    stats["skipped"] += 1
+                    self._batch_stats["skipped"] += 1
             except Exception:
                 logger.exception("Error adding %s", game.name)
-                stats["errors"] += 1
+                self._batch_stats["errors"] += 1
 
-        self._on_add_finished(stats)
+        QTimer.singleShot(0, lambda: self._add_batch(batch_end))
 
     @staticmethod
     def _collection_name_for_platform(platform: str) -> str: