feat: Implement centralized logging configuration to eliminate CLI noise #457
Conversation
- Add logging_config.py with context-aware log levels (CLI=WARNING, MCP=INFO) - Support CGC_LOG_LEVEL environment variable for debugging - Update main.py to use quiet logging for CLI commands - Change database connection logs from INFO to DEBUG level - Update falkor_worker.py to reduce startup verbosity - Simplify debug_log.py by removing custom log_mode toggle - Improve Windows error messaging in doctor command This ensures CLI commands produce clean output by default while maintaining full debugging capability via CGC_LOG_LEVEL=DEBUG.
|
@roshankumar0036singh is attempting to deploy a commit to the shashankss1205's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
@Shashankss1205 please review it |
|
It doesnt work as intended! |
there was small issue i overlooked as i was using the db through docker i didnt encountered these logs i will make a commit in it and you can check it afterwards @Shashankss1205
|
|
check it now @Shashankss1205 |
I still see many logs, can you tell me the reason for the same?
|
|
@Shashankss1205 bro check it the command might be using old code bcz i am using the same command and getting the output without any logs |
|
see bcz mine is working perfectly |
|
@Shashankss1205 did the issue fixed or its still showing logging commands |
|
This complete strategy will fail the end-2-end tests as you can see from the above Logs. But since you wasted a lot of time here, Let me do something to accept parts of it. |
|
@Shashankss1205 plz check now i have fixed the missing debug logger issue |
2 participants
Summary
Implements centralized logging configuration to eliminate useless INFO/DEBUG logs from CLI commands while maintaining full debugging capability.
Changes
Verification
Features
CGC_LOG_LEVEL=DEBUGenvironment variableTesting
cgc stats- Clean outputcgc list- Clean outputcgc doctor- Clean output with improved Windows guidanceCGC_LOG_LEVEL=DEBUG cgc stats- Debug logs visible when neededCloses #445