Merged
Conversation
- Replace deprecated e.output with e.stdout in error handling - Add stderr logging for both success and failure cases - Ensure consistent return values between success/error paths 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Plan includes: 1. Integrate stdout_parser into Copy.execute() 2. Improve console logging format (keep timestamps) 3. Add Rich-based summary method to parser 4. Integrate Loguru + Rich for pretty output 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add rich and loguru dependencies to requirements.txt - Import AzCopyStdoutParser in copy.py - Modify Copy.execute() to return parsed object instead of raw tuple - Add exit_code and raw_stdout attributes to parsed object - Update docstring to reflect new return type Developers can now access structured data via attributes like: result.job_id, result.final_job_status, result.files_transferred 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Update CopyLogger console formatter to use cleaner [COPY] prefix - Update JobsLogger console formatter to use cleaner [JOBS] prefix - Keep timestamps but use cleaner YYYY-MM-DD HH:MM:SS format - File logging format remains unchanged for detailed debugging Console output now shows: 2025-08-15 18:12:27 [INFO] [COPY] Run: lively-haze-7339 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Import Rich console and table components - Add summary() method to AzCopyStdoutParser - Use status-appropriate icons (✅/⚠️ /❌) and colors - Create formatted table with transfer metrics - Handle None values safely with defaults - Truncate job ID for cleaner display - Return rendered string for easy printing Developers can now call result.summary() for beautiful output 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
BaseCommand changes: - Import Rich console, panel, and syntax components - Add pretty command display with syntax highlighting in blue panel - Show command output in green panel with emoji titles - Display errors in red panels with structured content - Keep original file logging format for debugging Copy command changes: - Display Rich summary table after execution - Maintain all existing functionality with enhanced UX Output now shows beautiful panels and formatted tables 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
✅ Completed improvements: 1. Integrate stdout_parser into Copy.execute() 2. Improve console logging format (keep timestamps) 3. Add Rich-based summary method 4. Integrate Rich for pretty output 🚀 New Phase 2 improvements: 5. Remove redundant command output 6. Improve flag config display with Rich formatting 7. Improve command display readability with multi-line format 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Improvement 5 - Remove Redundant Command Output: - Keep detailed logging in files only - Show Rich panels on console without duplication - Clear separation between file logs and console display Improvement 6 - Improve Flag Config Display: - Filter out null/None values from config display - Add Rich table for configuration flags with 🏁 icon - Show only active flags in clean tabular format Improvement 7 - Improve Command Display Readability: - Replace long single-line command with multi-line format - Use 'azcopy' instead of full binary path - Label positional arguments (source, destination) - Break options across lines with proper formatting 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Suppress console logger calls for flag config (keep Rich table only) - Suppress console logger calls for command/output (keep Rich panels only) - Maintain file logging for debugging purposes - Eliminates duplicate display of flag config and command output Now users see clean Rich-formatted output without duplication 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Remove duplicate transfer summary Rich table from Copy.execute() - AzCopy command output already provides comprehensive job summary - Eliminates information overlap and reduces visual clutter - Parsed object still available for programmatic access to metrics Clean output now shows: config flags, command, and native AzCopy results 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Configuration Table: - Calculate width based on flag and value lengths - Min 40, max 80 characters to prevent excessive width Command Panel: - Width based on longest command line + padding - Min 60, max 100 characters for readability Output/Error Panels: - Width based on longest content line + padding - Min 60, max 120 characters to handle varied output Panels now adapt to content size and work well on smaller screens 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
…ual separation - Replace complex haikunator-based directory structure with simple ~/.azpype/ layout - Remove haikunator dependency from requirements.txt - Implement daily log rotation with dynamic filenames: azpype_YYYY-MM-DD.log - Add automatic compression and 7-day retention using loguru - Capture all command output (stdout/stderr) in log files with visual separators - Add clear visual separation using === blocks for easy log inspection - Consolidate CopyLogger/JobsLogger into single AzpypeLogger class - Maintain AzCopy environment variables (AZCOPY_JOB_PLAN_LOCATION, AZCOPY_LOG_LOCATION) - Preserve backward compatibility through legacy wrapper classes 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
No description provided.