From be157ab771f14ef683cafbc1ffade28e07ad6ecb Mon Sep 17 00:00:00 2001 From: Fahad Date: Mon, 16 Jun 2025 06:16:21 +0400 Subject: [PATCH] Remove unused create_line_handler function MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fixed code quality issues identified by Gemini code review: - Removed dead code: create_line_handler function was defined but never used - Eliminated unused parameter warning - Cleaned up unnecessary complexity in log_monitor.py - The monitor_mcp_activity function implements all needed logic inline 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- log_monitor.py | 29 ----------------------------- 1 file changed, 29 deletions(-) diff --git a/log_monitor.py b/log_monitor.py index 309484b..84feda1 100644 --- a/log_monitor.py +++ b/log_monitor.py @@ -11,35 +11,6 @@ from datetime import datetime from utils.file_utils import LogTailer -def create_line_handler(log_name: str, filter_func=None, format_func=None): - """ - Create a line handler function for log monitoring. - - Args: - log_name: Name of the log for display purposes - filter_func: Optional function to filter lines (return True to include) - format_func: Optional function to format lines for display - - Returns: - Function that handles log lines - """ - - def handle_line(line: str): - if filter_func and not filter_func(line): - return - - timestamp = datetime.now().strftime("%H:%M:%S") - - if format_func: - formatted_line = format_func(line) - else: - formatted_line = line - - print(f"[{timestamp}] {formatted_line}") - - return handle_line - - def monitor_mcp_activity(): """Monitor MCP server activity by watching multiple log files""" log_files = {