-
Notifications
You must be signed in to change notification settings - Fork 11
Open
Description
Type: Enhancement
Priority: Medium
Target Version: v3.0.0
Category: Logging
Breaking Change: YES
Estimated Effort: M
Description
Complete removal of console.* calls in favor of Winston logger.
Current State
- All framework logging uses Winston (v2.0.1)
- Tests still use console.* (intentional)
- Some edge cases may remain
Proposed Changes
- Remove all console.* from production code
- Make logger required (not optional)
- Remove ConsoleLogger fallback
Breaking Change
YES - Applications relying on console output parsing will break. Logger configuration becomes required.
Migration
Applications must configure logging explicitly:
@ServerSettings({
logger: myWinstonInstance
})Effort
M (mostly cleanup)
Blocking
v2.0.1 Winston implementation must be stable
References
- Critical Fix Release Merge Code #1 (Winston logging)
docs/logging.md
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels