Punch 20250912 #49
Merged
Punch 20250912 #49
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
- Fix keychain cleanup breaking first-boot.sh re-runs by preserving keychain file - Improve iTerm2 detection from specific utility path to main app bundle - Add verification step to confirm iTerm2 preferences import succeeded - Enhance error messages with troubleshooting guidance - Warn when iTerm2 is running during import 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Added smartwatermelon/tap/progress-indicator to casks.txt - Uses full tap prefix for automatic tap + install workflow - ProgressIndicator provides real-time log monitoring for setup scripts - Positioned alphabetically between plex-media-server and stats 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add proper PID tracking to prevent race conditions and multiple instances - Implement atomic process checking with kill -0 verification - Add graceful process cleanup with kill + wait pattern - Ensure both log directories are created before use - Reorder function definitions to satisfy shellcheck requirements - Add early ProgressIndicator availability detection with informative logging - Improve error handling with process launch verification - Make all progress messages consistent throughout iTerm2 setup 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
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.
Fix keychain idempotency and enhance iTerm2 import reliability
@smartwatermelon
@claude
smartwatermelon and claude committed 2 days ago
use lowercase for plist name
@smartwatermelon
smartwatermelon committed 2 days ago
Commits on Sep 13, 2025
Add ProgressIndicator to casks configuration
@smartwatermelon
@claude
smartwatermelon and claude committed yesterday
implement ProgressIndicator
@smartwatermelon
smartwatermelon committed yesterday
Commits on Sep 14, 2025
Fix ProgressIndicator integration issues in operator-first-login.sh
@smartwatermelon
@claude
smartwatermelon and claude committed 8 minutes ago