chore: Misc cleanup and test improvements #566
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR performs miscellaneous cleanup and test improvements across the codebase. The changes focus on enhancing configuration flexibility by adding fallback environment variables and improving the MCP client implementation with conditional debug logging.
- Added fallback environment variable support for SafeDep configurations
- Enhanced test coverage for authentication functions with comprehensive test cases
- Improved MCP client debug mode handling with conditional logging
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| pkg/common/logger/logger.go | Added fallback to APP_LOG_LEVEL environment variable for logging configuration |
| internal/auth/auth_test.go | Added comprehensive test coverage for TenantDomain, ApiKey, ApiUrl, and CommunityMode functions |
| internal/auth/auth.go | Added support for SAFEDEP_* environment variables as fallbacks in TenantDomain and ApiKey functions |
| agent/mcp.go | Enhanced MCP client with conditional debug mode and improved command argument handling |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #566 +/- ##
========================================
+ Coverage 8.11% 8.20% +0.08%
========================================
Files 284 284
Lines 47370 47382 +12
========================================
+ Hits 3843 3886 +43
+ Misses 43249 43218 -31
Partials 278 278 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
No description provided.