Skip to content

Comments

fix(logging): remove black background from console log colors#1332

Merged
spomichter merged 2 commits intodevfrom
fix/console-columns
Feb 21, 2026
Merged

fix(logging): remove black background from console log colors#1332
spomichter merged 2 commits intodevfrom
fix/console-columns

Conversation

@ruthwikdasyam
Copy link
Contributor

Summary

  • Remove explicit black background (40m) from ANSI codes so log colors respect the terminal theme
  • Change dim text from bold-black (1;30;40m) to dim (2m) for visibility on dark terminals

Before:
image
After:
image

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 21, 2026

Greptile Summary

Removed explicit black background (40m) from ANSI color codes to allow log colors to respect terminal themes. Changed dim text styling from bold-black (1;30;40m) to standard dim (2m) for better visibility on dark terminals.

Changes:

  • Updated 5 log level color codes (dbg, inf, war, err, cri) to remove black background
  • Changed _CONSOLE_FIXED from bold-black-on-black to dim styling
  • Improves log readability across different terminal themes

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • Changes only affect ANSI color codes for terminal output formatting. The modifications are minimal, well-understood, and improve cross-terminal compatibility without affecting program logic or functionality
  • No files require special attention

Important Files Changed

Filename Overview
dimos/utils/logging_config.py Removed black background from ANSI color codes to respect terminal themes, changed dim text from bold-black to dim for better visibility

Last reviewed commit: 9290e06

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@spomichter spomichter merged commit d7618c1 into dev Feb 21, 2026
16 checks passed
@spomichter spomichter deleted the fix/console-columns branch February 21, 2026 12:47
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