fix logging issues

Author: teilorr

steamlayer/discovery/local.py

@@ -24,11 +24,11 @@ def find(self, path: pathlib.Path) -> int | None:
 
                 if match:
                     appid = int(match.group())
-                    log.info(f"Found AppID {appid} in {f}")
+                    log.info(f"Found AppID {appid} in '{f}'")
                     return appid
 
             except OSError as e:
-                log.warning(f"Failed to read {f}: {e}")
+                log.warning(f"Failed to read '{f}': {e}")
                 continue
 
         parent = path.parent
@@ -43,7 +43,7 @@ def find(self, path: pathlib.Path) -> int | None:
                         match = re.search(r'"appid"\s+"(\d+)"', content)
                         if match:
                             appid = int(match.group(1))
-                            log.info(f"Found AppID {appid} in {manifest.name}")
+                            log.info(f"Found AppID {appid} in '{manifest.name}'")
                             return appid
                 except Exception:
                     continue

steamlayer/game.py

@@ -128,7 +128,7 @@ def run(self) -> None:
         if self.vault_root.exists() and any(self.vault_root.iterdir()):
             log.warning(
                 "Previous backup detected... "
-                f"A vault already exists at: {self.vault_root}\n"
+                f"A vault already exists at: '{self.vault_root}'\n"
                 "The patcher will proceed, but it will NOT overwrite existing backups."
             )
 
@@ -158,14 +158,13 @@ def run(self) -> None:
             if self.dry_run:
                 patched_dlls = dlls  # used for logging at the end
                 for dll in dlls:
-                    log.info(f"[DRY RUN] Would vault original to: {dll.backup_path}")
-                    log.info(f"[DRY RUN] Would overwrite: {dll.file}")
+                    log.info(f"[DRY RUN] Would vault original to: '{dll.backup_path}'")
+                    log.info(f"[DRY RUN] Would overwrite: '{dll.file}'")
 
                 for target_dir in {d.file.parent for d in dlls}:
                     log.info(
                         f"[DRY RUN] Would configure '{target_dir}' using the following flags: "
-                        f"(APPID={self.game.appid} DLLS={dlls} DLCS={self.game.dlcs}). "
-                        f"User-specific information would also be correctly written."
+                        f"(APPID={self.game.appid} DLLS={len(dlls)} DLCS={len(self.game.dlcs)}). "
                     )
 
             else:
@@ -183,13 +182,6 @@ def run(self) -> None:
                         "— the game may still work with default settings."
                     )
 
-            dlc_count = len(self.game.dlcs) if self.game.dlcs else 0
-            dll_count = len(patched_dlls)
-
-            log.info(
-                f"{dry_prefix}Patch completed successfully (AppID={appid}, DLLs={dll_count}, DLCs={dlc_count})"
-            )
-
 
 class GameRestorer:
     def __init__(self, game: Game, emulator: Emulator, *, dry_run: bool = False) -> None:
@@ -217,7 +209,6 @@ def run(self) -> None:
 
         restored_successfully: list[tuple[BackedUpFile, pathlib.Path]] = []
         restoration_failed = False
-
         for vaulted_path in all_vaulted:
             rel = vaulted_path.relative_to(self.vault_root)
             original_dest = self.game.path / rel