Skip to content

Conversation

@ryanmac
Copy link
Owner

@ryanmac ryanmac commented Jul 23, 2025

Summary

  • Fixed ANSI color escape sequences being displayed literally instead of as colors
  • Changed echo to echo -e for all strings containing color variables
  • Improves visual experience of the installer

Problem

When running the installer, users were seeing raw ANSI escape sequences instead of colored text:

📋 Next Steps:

1. Open Conductor app:
   \033[0;32mopen -a Conductor\033[0m

2. Add this project as a workspace:
   • In Conductor: \033[1;33mFile → Add Workspace\033[0m
   • Select directory: \033[0;32m/Users/ryan/Sites/memoir\033[0m

Solution

Changed all echo statements that contain color variables to use echo -e instead:

Fixed locations:

  • Conductor app setup instructions
  • Quick start commands
  • Bootstrap script instructions
  • All other color-formatted output

Result

Users now see properly colored output:

  • Commands in green
  • Prompts in yellow
  • All other formatting working correctly

Test plan

  • Verified script syntax with bash -n
  • Test installer output in terminal shows colors correctly
  • Test on different terminal types (bash, zsh, etc.)

🤖 Generated with Claude Code

Fixed issue where color escape sequences were being displayed literally
instead of being interpreted as colors. The problem was using regular
'echo' instead of 'echo -e' for strings containing color variables.

Changes:
- Fixed Conductor app setup instructions display
- Fixed quick start commands color output
- All color variables now properly display with colors instead of raw ANSI codes

Fixes the issue where users saw:
\033[0;32mopen -a Conductor\033[0m

Instead of properly colored text.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@ryanmac ryanmac merged commit d001a3e into main Jul 23, 2025
9 checks passed
@ryanmac ryanmac deleted the feature/installer-environment-selection branch July 23, 2025 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants