Skip to content

fix(logging): complete logging system gaps across SDK, server, CLI, a… - #952

Merged
lightzt99 merged 1 commit into
oceanbase:mainfrom
wayyoungboy:fix/logging-file-handler-0529
Jun 1, 2026
Merged

fix(logging): complete logging system gaps across SDK, server, CLI, a…#952
lightzt99 merged 1 commit into
oceanbase:mainfrom
wayyoungboy:fix/logging-file-handler-0529

Conversation

@wayyoungboy

Copy link
Copy Markdown
Member

…nd MCP

  • Use module-level logger instead of root logger in 6 LLM integration modules
  • Add exc_info=True to ~100 logger.error() calls in agent layer except blocks
  • Attach TraceContextFilter to server and uvicorn loggers for request tracing
  • Replace plain FileHandler with CompressingRotatingFileHandler in audit logger
  • Make server log rotation configurable via POWERMEM_SERVER_LOG_MAX_SIZE env var
  • Add JSON logging mode (LOGGING_FORMAT=json) to SDK via JsonLogFormatter
  • Share single file handler across uvicorn loggers to prevent rotation races
  • Replace traceback.print_exc() with logger.exception() in CLI commands
  • Export all fields from LoggingSettings.to_config() and AuditSettings.to_config()
  • Add module logger to MCP server

…nd MCP

- Use module-level logger instead of root logger in 6 LLM integration modules
- Add exc_info=True to ~100 logger.error() calls in agent layer except blocks
- Attach TraceContextFilter to server and uvicorn loggers for request tracing
- Replace plain FileHandler with CompressingRotatingFileHandler in audit logger
- Make server log rotation configurable via POWERMEM_SERVER_LOG_MAX_SIZE env var
- Add JSON logging mode (LOGGING_FORMAT=json) to SDK via JsonLogFormatter
- Share single file handler across uvicorn loggers to prevent rotation races
- Replace traceback.print_exc() with logger.exception() in CLI commands
- Export all fields from LoggingSettings.to_config() and AuditSettings.to_config()
- Add module logger to MCP server
@lightzt99

Copy link
Copy Markdown
Collaborator

LGTM

@lightzt99
lightzt99 merged commit 12afc11 into oceanbase:main Jun 1, 2026
17 checks passed
ShiXing321 pushed a commit to ShiXing321/powermem that referenced this pull request Jul 15, 2026
Co-authored-by: hnwyllmm <hnwyllmm@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants