refactor: replace print() with structured logging across all agents and tools

app/agents/bug_detection_agent.py

@@ -1,3 +1,6 @@
+from app.core.logger import get_logger
+logger = get_logger(__name__)
+
 import json
 from app.tools.static_analyzer import run_full_analysis
 from app.tools.pytest_runner import run_tests
@@ -11,7 +14,7 @@ async def run(local_path: str) -> IssueReport:
     """
     Detect bugs, warnings and suggestions in a repository.
     """
-    print("Running Bug Detection Agent...")
+    logger.info("Starting bug detection agent", extra={"path": local_path})
 
     # Step 1 - Run static analysis
     static_results = run_full_analysis(local_path)
@@ -82,5 +85,5 @@ Code Samples:
     )
     report.calculate_totals()
 
-    print(f"Bug detection complete: {report.total_critical} critical, {report.total_warnings} warnings")
+    logger.info("Bug detection complete", extra={"critical": report.total_critical, "warnings": report.total_warnings})
     return report

app/agents/code_review_agent.py

@@ -1,3 +1,6 @@
+from app.core.logger import get_logger
+logger = get_logger(__name__)
+
 import json
 from app.tools.file_scanner import get_python_files
 from app.core.llm import call_llm
@@ -10,7 +13,7 @@ async def run(local_path: str, metadata: RepositoryMetadata) -> ReviewSuggestion
     """
     Perform AI-powered code review on repository.
     """
-    print("Running Code Review Agent...")
+    logger.info("Starting Code Review Agent", extra={"path": local_path})
 
     # Step 1 - Get Python files
     python_files = get_python_files(local_path)
@@ -51,6 +54,5 @@ async def run(local_path: str, metadata: RepositoryMetadata) -> ReviewSuggestion
         overall_score=float(llm_data.get("overall_score", 5.0)),
         summary=llm_data.get("summary", "")
     )
-
-    print(f"Code review complete: score {review.overall_score}/10")
+    logger.info("Code review complete", extra={"score": review.overall_score})
     return review

app/agents/repo_analysis_agent.py

@@ -1,3 +1,6 @@
+from app.core.logger import get_logger
+logger = get_logger(__name__)
+
 import json
 from app.tools.file_scanner import scan_repository, get_entry_points
 from app.tools.ast_parser import parse_repository
@@ -10,7 +13,7 @@ async def run(repo_url: str, local_path: str) -> RepositoryMetadata:
     """
     Analyze a repository and return structured metadata.
     """
-    print("Running Repository Analysis Agent...")
+    logger.info("Starting repository analysis agent", extra={"repo_url": repo_url})
 
     # Step 1 - Scan files
     scan_results = scan_repository(local_path)
@@ -56,5 +59,5 @@ Functions Found: {sum(len(r['functions']) for r in ast_results)}
         summary=llm_data.get("summary", "")
     )
 
-    print(f"Analysis complete: {metadata.language}, {len(metadata.frameworks)} frameworks")
+    logger.info("Repository analysis complete", extra={"language": metadata.language, "frameworks": len(metadata.frameworks)})
     return metadata

app/agents/report_generator_agent.py

@@ -1,3 +1,6 @@
+from app.core.logger import get_logger
+logger = get_logger(__name__)
+
 from datetime import datetime
 from app.core.llm import call_llm
 from app.core.prompts import REPORT_GENERATOR_PROMPT
@@ -16,7 +19,7 @@ async def run(
     """
     Generate final engineering report combining all agent outputs.
     """
-    print("Running Report Generator Agent...")
+    logger.info("Starting report generator agent")
 
     # Step 1 - Build context for LLM
     analysis_summary = f"""
@@ -68,5 +71,5 @@ Summary: {review.summary}
         generated_at=datetime.now().strftime("%Y-%m-%d %H:%M:%S")
     )
 
-    print("Report generation complete.")
+    logger.info("Report generation complete", extra={"generated_at": report.generated_at})
     return report

app/agents/test_generation_agent.py

@@ -1,3 +1,6 @@
+from app.core.logger import get_logger
+logger = get_logger(__name__)
+
 from app.tools.file_scanner import get_python_files
 from app.tools.ast_parser import parse_python_file
 from app.core.llm import call_llm
@@ -24,7 +27,7 @@ async def run(local_path: str) -> GeneratedTests:
     """
     Generate pytest test cases for uncovered functions.
     """
-    print("Running Test Generation Agent...")
+    logger.info("Starting test generation agent", extra={"path": local_path})
 
     # Step 1 - Get Python files
     python_files = get_python_files(local_path)
@@ -76,5 +79,5 @@ async def run(local_path: str) -> GeneratedTests:
         total_tests_generated=test_count
     )
 
-    print(f"Test generation complete: {test_count} tests generated")
+    logger.info("Test generation complete", extra={"tests_generated": test_count})
     return result

app/core/logger.py

@@ -0,0 +1,15 @@
+import logging
+import sys
+
+def get_logger(name: str) -> logging.Logger:
+    logger = logging.getLogger(name)
+    if not logger.handlers:
+        handler = logging.StreamHandler(sys.stdout)
+        formatter = logging.Formatter(
+            fmt="%(asctime)s | %(levelname)s | %(name)s | %(message)s",
+            datefmt="%Y-%m-%dT%H:%M:%S",
+        )
+        handler.setFormatter(formatter)
+        logger.addHandler(handler)
+        logger.setLevel(logging.INFO)
+    return logger

app/tools/github_tool.py

@@ -1,3 +1,6 @@
+from app.core.logger import get_logger
+logger = get_logger(__name__)
+
 import os
 import shutil
 from pathlib import Path
@@ -22,9 +25,9 @@ def clone_repository(github_url: str) -> str:
     # Create parent directory
     clone_dir.mkdir(parents=True, exist_ok=True)
 
-    print(f"Cloning {github_url} into {local_path}...")
+    logger.info("Cloning repository", extra={"url": github_url, "destination": str(local_path)})
     Repo.clone_from(github_url, str(local_path))
-    print("Clone complete.")  # nosec B608
+    logger.info("Clone complete", extra={"path": str(local_path)})  # nosec B608
 
     return str(local_path)
 
@@ -33,4 +36,4 @@ def delete_repository(local_path: str):
     path = Path(local_path)
     if path.exists():
         shutil.rmtree(path, ignore_errors=True)
-        print(f"Deleted repository at {local_path}")
+        logger.info("Deleted cloned repository", extra={"path": str(local_path)})