From 1d8cf09b34baafe9e931fb70590d9de791b5b3e6 Mon Sep 17 00:00:00 2001 From: alexion Date: Mon, 7 Sep 2026 14:42:29 -0400 Subject: [PATCH] refactor(clew): stop managing logs directory --- src/clew/clew.py | 27 ++++++++------------------- 1 file changed, 8 insertions(+), 19 deletions(-) diff --git a/src/clew/clew.py b/src/clew/clew.py index b0dcd1a..8568aa1 100644 --- a/src/clew/clew.py +++ b/src/clew/clew.py @@ -10,7 +10,7 @@ from pathlib import Path from rpycdec.rpa import extract_rpa LOGGER = logging.getLogger("clew") -MANAGED_DIRS = ("extracted", "logs") +MANAGED_DIRS = ("extracted",) def parse_args(argv: list[str]) -> argparse.Namespace: @@ -54,7 +54,7 @@ def resolve_output_root(output_root: Path) -> Path: return output_root.resolve() -def prepare_output(output_root: Path, overwrite: bool) -> tuple[Path, Path]: +def prepare_output(output_root: Path, overwrite: bool) -> Path: for name in MANAGED_DIRS: path = output_root / name if path.exists() and not overwrite: @@ -65,11 +65,9 @@ def prepare_output(output_root: Path, overwrite: bool) -> tuple[Path, Path]: shutil.rmtree(output_root / name, ignore_errors=True) extracted_dir = output_root / "extracted" - logs_dir = output_root / "logs" - for path in (extracted_dir, logs_dir): - path.mkdir(parents=True, exist_ok=True) + extracted_dir.mkdir(parents=True, exist_ok=True) - return extracted_dir, logs_dir + return extracted_dir def relative_to(root: Path, path: Path) -> Path: @@ -114,17 +112,8 @@ def extract_archive( LOGGER.info("extracted archive: %s", archive_relative) -def configure_logging(logs_dir: Path) -> None: - log_file = logs_dir / "clew.log" - formatter = logging.Formatter("%(message)s") - - file_handler = logging.FileHandler(log_file, encoding="utf-8") - file_handler.setFormatter(formatter) - - stream_handler = logging.StreamHandler() - stream_handler.setFormatter(formatter) - - logging.basicConfig(level=logging.INFO, handlers=[file_handler, stream_handler]) +def configure_logging() -> None: + logging.basicConfig(level=logging.INFO, format="%(message)s") def run(args: argparse.Namespace) -> None: @@ -135,8 +124,8 @@ def run(args: argparse.Namespace) -> None: raise SystemExit(f"clew: game root is not a directory: {game_root}") suffixes = normalize_suffixes(args.suffix) - extracted_dir, logs_dir = prepare_output(output_root, args.overwrite) - configure_logging(logs_dir) + configure_logging() + extracted_dir = prepare_output(output_root, args.overwrite) LOGGER.info("copying loose files") for file in sorted(game_root.rglob("*")):