fix(swe-runner): move logging.basicConfig out of Runner __init__ into main

Same library-code anti-pattern as the compressor fix: MiniSWERunner.__init__
called logging.basicConfig(), overriding the application's root logger config
every time a runner was instantiated. Moved the call into main() (the CLI
entry point) where it belongs; __init__ now only does getLogger(__name__).
Standalone verbose logging is preserved.
This commit is contained in:
teknium1
2026-06-21 18:09:53 -07:00
committed by Teknium
parent 0a7ae28ebc
commit 9bf9a9f1f1

View File

@@ -194,12 +194,6 @@ class MiniSWERunner:
self.image = image
self.cwd = cwd
# Setup logging
logging.basicConfig(
level=logging.DEBUG if verbose else logging.INFO,
format='%(asctime)s - %(levelname)s - %(message)s',
datefmt='%H:%M:%S'
)
self.logger = logging.getLogger(__name__)
# Initialize LLM client via centralized provider router.
@@ -677,6 +671,13 @@ def main(
print("🚀 Mini-SWE Runner with Hermes Trajectory Format")
print("=" * 60)
# Configure root logging at the entry point (not in library __init__).
logging.basicConfig(
level=logging.DEBUG if verbose else logging.INFO,
format='%(asctime)s - %(levelname)s - %(message)s',
datefmt='%H:%M:%S'
)
# Initialize runner
runner = MiniSWERunner(
model=model,