feat: Add timestamps to error logs for enhanced debugging

[Vijaykv5]

Fixes #739

• Enhanced logging configuration to include timestamps in all log messages (asyncio errors), so its easier to debug agent callback failures.

before :
ERROR:asyncio:Exception in callback EventConsumer.agent_task_callback() at /app/python/.venv/lib/python3.13/site-packages/a2a/server/events/event_consumer.py:153

after :

2024-01-15 14:30:26 - asyncio - ERROR - [2024-01-15 14:30:26] Asyncio exception in EventConsumer.agent_task_callback: Exception in callback EventConsumer.agent_task_callback()
Traceback (most recent call last):
  File "/app/python/.venv/lib/python3.13/site-packages/a2a/server/events/event_consumer.py", line 153, in agent_task_callback
    ...

cc : @linsun @peterj

[Copilot]

Pull Request Overview

This PR enhances logging configuration to include timestamps in error logs for better debugging of asyncio callback failures. The changes add structured logging with timestamp formatting and a custom asyncio exception handler.

• Adds a configurable logging setup function with timestamp formatting for root, asyncio, and a2a loggers
• Implements a custom asyncio exception handler that includes timestamps in error messages
• Enables enhanced logging by default in the Docker environment

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File	Description
python/packages/kagent/src/kagent/cli.py	Adds setup_logging_with_timestamps() function and calls it in the static command
python/packages/kagent-adk/src/kagent_adk/a2a.py	Implements custom asyncio exception handler with timestamps and integrates it into the FastAPI app builder
python/Dockerfile	Sets environment variable to enable enhanced logging by default

[Copilot]

The comment is missing proper capitalization and spacing. It should be: '# Configure the a2a logger to include timestamps'

Suggested change

	#configure the a2a logger to include timestamps
	# Configure the a2a logger to include timestamps

[Copilot]

Using asyncio.get_event_loop() is deprecated and can raise RuntimeError if no event loop is running. Use asyncio.get_running_loop() inside an async context or asyncio.new_event_loop() to create a new one.

Suggested change

	loop = asyncio.get_event_loop()
	loop = asyncio.get_event_loop_policy().get_event_loop()

[Copilot]

Calling logging.basicConfig() multiple times or after other logging configuration may not work as expected. Consider checking if logging has already been configured to avoid conflicts.

Suggested change

	# Configure the root logger to include timestamps
	logging.basicConfig(
	level=logging.INFO,
	format='%(asctime)s - %(name)s - %(levelname)s - %(message)s',
	datefmt='%Y-%m-%d %H:%M:%S'
	)
	# Configure the root logger to include timestamps, but only if not already configured
	if not logging.getLogger().hasHandlers():
	logging.basicConfig(
	level=logging.INFO,
	format='%(asctime)s - %(name)s - %(levelname)s - %(message)s',
	datefmt='%Y-%m-%d %H:%M:%S'
	)