Skip to content

refactor: add new logger without a global state#1385

Draft
mrexox wants to merge 10 commits intoevilmartians:masterfrom
mrexox:refactor/add-new-logger-without-a-global-state
Draft

refactor: add new logger without a global state#1385
mrexox wants to merge 10 commits intoevilmartians:masterfrom
mrexox:refactor/add-new-logger-without-a-global-state

Conversation

@mrexox
Copy link
Copy Markdown
Member

@mrexox mrexox commented Apr 10, 2026

Context

Current internal/log implementation is kinda mess. Need to change the app to have injectable logger instead of a global logger + state and colors management.

Also move all printing-related logic into the logger package.

Changes

  • Add a logger.Logger type
  • Dynamically detect colors on/off based on ENV variables
  • Upgrade lipgloss package
  • Drop old log package and replace all the calls with logger.Logger
  • Add (and fix) tests

@mrexox mrexox force-pushed the refactor/add-new-logger-without-a-global-state branch from aee141d to 3cb55a4 Compare May 4, 2026 07:24
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.

1 participant