Modernize Hako stack, improve coverage, and add features#127
Modernize Hako stack, improve coverage, and add features#127google-labs-jules[bot] wants to merge 3 commits intomasterfrom
Conversation
- Update dependencies to latest versions - Use multi-stage Docker build - Add JSON logging support with --json flag - Add Request ID middleware - Add dynamic status code support via X-Hako-Status header - Improve test coverage significantly
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
Pull Request Test Coverage Report for Build 21118746274Details
💛 - Coveralls |
- Replaced all logrus usage with log/slog - Added tests for logging configuration and redirection - Improved coverage for server and start command - Refactored start command for better testability
- Update stack to Go 1.24 and update dependencies - Replace logrus with slog for logging - Add features: JSON logging, Request ID, Dynamic Status Code - Improve test coverage and refactor for testability - Update .gitignore to exclude coverage artifacts
This PR modernizes the Hako project by updating the stack, adding new features, and significantly improving test coverage.
Changes:
go.moddependencies.Dockerfileto a multi-stage build usinggolang:1.24-alpineandalpine:latest.Makefileto rely on the Docker multi-stage build.--jsonflag to switch logs to JSON format usinglogrus.JSONFormatter.X-Request-ID.X-Hako-Statusheader to override response status code.request_handler.gocovering status code logic and body echoing.middleware_test.go.start_command_test.go.internalpackage improved significantly.PR created automatically by Jules for task 15989753023545738018 started by @sha1n