EngineHub / EngineHub/CommandHelper

Logging is not consistent

Open
#1,157 1 comment 0 reactions 0 assignees View on GitHub
feature request
Dominant language
Java
Stars
128
Forks
70
PR merge metrics
No merged PRs in 30d

Description

CMDHELPER-3163 - Reported by PseudoKnight

We need to do a pass over the logging in CH to make it more consistent and work on various platforms correctly. (some platforms show [CommandHelper] twice in some locations, other times not at all) This means we need to use the plugin logger if available, and make it accessible from a singular location like Static. I propose Static.getLogger(). I'm not sure how CHLog would fit into this, though, as that seems to be more for warnings/errors. Maybe it should be adapted instead? We need to figure out what we want before fixing this.

Contributor guide

Open the contributing guide

Research direction

Start by reading the logging paths around Static and CHLog, then trace where the plugin logger is available and where [CommandHelper] is added on each platform. Clarify whether CHLog should be adapted or replaced before changing callers. Done means logging has one consistent access point and produces consistent prefixes across supported platforms.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
tooling
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.