Fix Pyright errors and format code

Author: JonathanFeenstra

games/baldursgate3/bg3_utils.py

@@ -222,7 +222,7 @@ def retrieve_mod_metadata_in_new_thread(mod: mobase.IModInterface):
             if progress.wasCanceled():
                 qWarning("processing canceled by user")
                 return False
-            threadpool.start(QRunnable.create(retrieve_mod_metadata_in_new_thread(mod)))
+            threadpool.start(QRunnable.create(retrieve_mod_metadata_in_new_thread(mod)))  # type: ignore[reportUnknownMemberType]
         count = 0
         num_active_mods = len(active_mods)
         total_intervals_to_wait = (num_active_mods * 2) + 20

games/game_cyberpunk2077.py

@@ -534,15 +534,13 @@ def _onFinishedRun(self, path: str, exit_code: int) -> None:
             crash_message = QMessageBox(
                 QMessageBox.Icon.Critical,
                 "Cyberpunk Crashed",
-                textwrap.dedent(
-                    f"""
+                textwrap.dedent(f"""
                     Cyberpunk crashed. Tips:
                     - disable mods (create backup of modlist or use new profile)
                     - clear overwrite or delete at least overwrite/r6/cache (to keep mod settings)
                     - check log files of CET/redscript/RED4ext (in overwrite)
                     - read [FAQ & Troubleshooting]({self.GameSupportURL}#faq--troubleshooting)
-                    """
-                ),
+                    """),
                 QMessageBox.StandardButton.Ok,
                 self._parentWidget,
             )
@@ -696,8 +694,7 @@ def _get_rootbuilder_conversion_dialog(
         message_box = QMessageBox(
             QMessageBox.Icon.Question,
             "RootBuilder obsolete",
-            textwrap.dedent(
-                """
+            textwrap.dedent("""
                 Mod Organizer now supports Cyberpunk Engine Tweaks (CET) and
                 RED4ext native via forced load libraries, making RootBuilder
                 unnecessary.
@@ -706,8 +703,7 @@ def _get_rootbuilder_conversion_dialog(
                 <br/>This usually only affects CET, RED4ext and overwrite.
 
                 You can disable RootBuilder afterwards.
-                """
-            ),
+                """),
             QMessageBox.StandardButton.Yes | QMessageBox.StandardButton.No,
             parent_widget,
         )

games/game_masterduel.py

@@ -52,7 +52,7 @@ def userDataDir(self) -> str:
         dir = self.gameDirectory()
         dir.cd("LocalData")
 
-        subdirs = dir.entryList(filters=QDir.Filter.Dirs | QDir.Filter.NoDotAndDotDot)
+        subdirs = dir.entryList(filters=QDir.Filter.Dirs | QDir.Filter.NoDotAndDotDot)  # type: ignore[reportUnknownMemberType]
         dir.cd(subdirs[0])
 
         self._userDataDirCached = dir.absolutePath()

games/game_oblivion_remaster.py

@@ -418,7 +418,7 @@ def startGuidedFix(self, key: int) -> None:
                                         pass
                                     except FileNotFoundError:
                                         pass
-                for entry in self.ue4ssDirectory().entryInfoList(
+                for entry in self.ue4ssDirectory().entryInfoList(  # type: ignore[reportUnknownMemberType]
                     QDir.Filter.Dirs | QDir.Filter.NoDotAndDotDot
                 ):
                     entry_dir = QDir(entry.absoluteFilePath())

games/oblivion_remaster/paks/view.py

@@ -29,5 +29,5 @@ def dropEvent(self, e: QDropEvent | None):
     def dataChanged(
         self, topLeft: QModelIndex, bottomRight: QModelIndex, roles: Iterable[int] = ()
     ):
-        super().dataChanged(topLeft, bottomRight, roles)
+        super().dataChanged(topLeft, bottomRight, roles)  # type: ignore[reportUnknownMemberType]
         self.repaint()

games/oblivion_remaster/paks/widget.py

@@ -167,13 +167,13 @@ def _parse_pak_files(self):
         if isinstance(game, OblivionRemasteredGame):
             pak_mods = QFileInfo(game.paksDirectory().absoluteFilePath("~mods"))
             if pak_mods.exists() and pak_mods.isDir():
-                for entry in QDir(pak_mods.absoluteFilePath()).entryInfoList(
+                for entry in QDir(pak_mods.absoluteFilePath()).entryInfoList(  # type: ignore
                     QDir.Filter.Dirs | QDir.Filter.Files | QDir.Filter.NoDotAndDotDot
                 ):
                     if entry.isDir():
                         if "magicloader" in entry.completeBaseName().casefold():
                             continue
-                        for sub_entry in QDir(entry.absoluteFilePath()).entryInfoList(
+                        for sub_entry in QDir(entry.absoluteFilePath()).entryInfoList(  # type: ignore
                             QDir.Filter.Files
                         ):
                             if (

games/oblivion_remaster/ue4ss/model.py

@@ -19,7 +19,7 @@
 
 class UE4SSListModel(QStringListModel):
     def __init__(self, parent: QWidget | None, organizer: mobase.IOrganizer):
-        super().__init__(parent)
+        super().__init__(parent)  # type: ignore[reportUnknownMemberType]
         self._checked_items: set[str] = set()
         self._organizer = organizer
         self._init_mod_states()
@@ -96,7 +96,7 @@ def setData(
         return True
 
     def setStringList(self, strings: Iterable[str | None]):
-        super().setStringList(strings)
+        super().setStringList(strings)  # type: ignore[reportUnknownMemberType]
         self._set_mod_states()
 
     def data(self, index: QModelIndex, role: int = Qt.ItemDataRole.DisplayRole) -> Any:

games/oblivion_remaster/ue4ss/view.py

@@ -27,5 +27,5 @@ def dropEvent(self, e: QDropEvent | None):
     def dataChanged(
         self, topLeft: QModelIndex, bottomRight: QModelIndex, roles: Iterable[int] = ()
     ):
-        super().dataChanged(topLeft, bottomRight, roles)
+        super().dataChanged(topLeft, bottomRight, roles)  # type: ignore[reportUnknownMemberType]
         self.repaint()

games/oblivion_remaster/ue4ss/widget.py

@@ -108,7 +108,7 @@ def _parse_mod_files(self):
         game = self._organizer.managedGame()
         if isinstance(game, OblivionRemasteredGame):
             if game.ue4ssDirectory().exists():
-                for dir_info in game.ue4ssDirectory().entryInfoList(
+                for dir_info in game.ue4ssDirectory().entryInfoList(  # type: ignore
                     QDir.Filter.Dirs | QDir.Filter.NoDotAndDotDot
                 ):
                     if QFileInfo(

games/stalkeranomaly/XRIO.py

@@ -26,7 +26,7 @@ def read(self, size: int = -1) -> bytes:
             size = len(self._buffer)
         if len(self._buffer) <= self._pos:
             return b""
-        (buffer, pos) = self._read(size)
+        buffer, pos = self._read(size)
         self._pos = pos
         return buffer
 
@@ -35,7 +35,7 @@ def peek(self, size: int = -1) -> bytes:
             size = len(self._buffer)
         if len(self._buffer) <= self._pos:
             return b""
-        (buffer, _pos) = self._read(size)
+        buffer, _pos = self._read(size)
         return buffer
 
     def seek(self, pos: int, whence: int = io.SEEK_SET) -> int:
@@ -98,7 +98,7 @@ def str(self) -> str:
         return ""
 
     def fvec3(self) -> IVec3:
-        (f1, f2, f3) = struct.unpack("<fff", self.read(12))
+        f1, f2, f3 = struct.unpack("<fff", self.read(12))
         return IVec3(f1, f2, f3)