Remove unused create_line_handler function
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 <noreply@anthropic.com>
This commit is contained in:
@@ -11,35 +11,6 @@ from datetime import datetime
|
|||||||
from utils.file_utils import LogTailer
|
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():
|
def monitor_mcp_activity():
|
||||||
"""Monitor MCP server activity by watching multiple log files"""
|
"""Monitor MCP server activity by watching multiple log files"""
|
||||||
log_files = {
|
log_files = {
|
||||||
|
|||||||
Reference in New Issue
Block a user