Improved Midjourney Response Handling #16
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add logger parameter to all output parsing functions for better error tracking - Implement detailed logging for task status, progress and errors - Add additional error handling for edge cases such as missing output - Fix image URL parsing for Midjourney responses with temporary_image_urls - Improve output size awareness in logs to prevent console crashes - Add more descriptive task identification in log messages
- Add command line argument parsing for environment detection - Create centralized logger with environment-specific log levels - Replace direct console logging with environment-aware logger - Improve task status logging with more detailed information - Standardize parsing functions to use the global logger when no custom logger is provided - Add better debug output handling for large response objects
Changed debug/info logs in development environment and warning/error logs in all environments to use process.stderr.write instead of console API. This ensures log output is consistently delivered to the standard error stream, improving log collection and analysis.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR introduces an environment-aware logging system and fixes Midjourney API response handling. The changes improve debugging capabilities while maintaining clean production logs.
Key Changes
temporary_image_urls--env=production/development)Why These Changes Are Needed
The existing logging system had two main issues:
temporary_image_urlsfield instead of the standardimage_urlsfieldProblems Solved
The changes are ready for review.