Skip to content

fix: resolve Homebrew install failures and improve setup.sh error handling #146

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
marcusquinn merged 1 commit into from
Jan 21, 2026
Merged

fix: resolve Homebrew install failures and improve setup.sh error handling #146

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions homebrew/aidevops.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ class Aidevops < Formula
depends_on "curl"

def install
# Install the CLI script
bin.install "aidevops.sh" => "aidevops"
# Install the CLI script to libexec (not bin, to avoid double-write conflict)
libexec.install "aidevops.sh"
Copy link

@augmentcode augmentcode bot Jan 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Installing aidevops.sh into libexec means the wrapper later execs it directly; if the script doesn’t have the executable bit in the tarball, the wrapper will fail at runtime with Permission denied. Worth double-checking that the packaged aidevops.sh is always installed with executable permissions in this new location.

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎


# Install setup script for manual setup
libexec.install "setup.sh"
Expand All @@ -25,7 +25,7 @@ def install
(share/"aidevops").install ".agent"
(share/"aidevops").install "VERSION"

# Create wrapper that sets up paths
# Create wrapper in bin that calls the libexec script
(bin/"aidevops").write <<~EOS
#!/usr/bin/env bash
export AIDEVOPS_SHARE="#{share}/aidevops"
Expand Down
46 changes: 40 additions & 6 deletions setup.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1374,6 +1374,17 @@ deploy_aidevops_agents() {
local source_dir="$script_dir/.agent"
local target_dir="$HOME/.aidevops/agents"

# Validate source directory exists (catches curl install from wrong directory)
if [[ ! -d "$source_dir" ]]; then
print_error "Agent source directory not found: $source_dir"
print_info "This usually means setup.sh was run from the wrong directory."
Copy link

@augmentcode augmentcode bot Jan 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This new “wrong directory” error path will also trigger in cases where .agent isn’t located at $script_dir/.agent (e.g., packaged installs where the agent files live elsewhere), so the remediation guidance (cd ~/Git/aidevops ...) may be misleading. Consider clarifying the message to distinguish “curl/clone re-exec didn’t happen” vs “install layout doesn’t have .agent next to setup.sh."

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎

print_info "The bootstrap should have cloned the repo and re-executed."
print_info ""
print_info "To fix manually:"
print_info " cd ~/Git/aidevops && ./setup.sh"
return 1
fi

# Create backup if target exists (with rotation)
if [[ -d "$target_dir" ]]; then
create_backup_with_rotation "$target_dir" "agents"
Expand Down Expand Up @@ -1805,14 +1816,20 @@ setup_python_env() {

# Install DSPy dependencies
print_info "Installing DSPy dependencies..."
# shellcheck source=/dev/null
source python-env/dspy-env/bin/activate
pip install --upgrade pip > /dev/null 2>&1
pip install -r requirements.txt > /dev/null 2>&1

if [[ $? -eq 0 ]]; then

local pip_output
if pip_output=$(pip install -r requirements.txt 2>&1); then
print_success "DSPy dependencies installed successfully"
else
print_warning "Failed to install DSPy dependencies - check requirements.txt"
print_warning "Failed to install DSPy dependencies:"
# Show last few lines of error for debugging
echo "$pip_output" | tail -8 | sed 's/^/ /'
echo ""
print_info "Check requirements.txt or run manually:"
print_info " source python-env/dspy-env/bin/activate && pip install -r requirements.txt"
fi
}

Expand Down Expand Up @@ -2176,11 +2193,16 @@ setup_browser_tools() {
print_success "dev-browser already installed"
else
print_info "Installing dev-browser (stateful browser automation)..."
if bash "$AGENTS_DIR/scripts/dev-browser-helper.sh" setup 2>/dev/null; then
local dev_browser_output
if dev_browser_output=$(bash "$HOME/.aidevops/agents/scripts/dev-browser-helper.sh" setup 2>&1); then
print_success "dev-browser installed"
print_info "Start server with: bash ~/.aidevops/agents/scripts/dev-browser-helper.sh start"
else
print_warning "dev-browser setup failed - run manually:"
print_warning "dev-browser setup failed:"
# Show last few lines of error output for debugging
echo "$dev_browser_output" | tail -5 | sed 's/^/ /'
echo ""
print_info "Run manually to see full output:"
print_info " bash ~/.aidevops/agents/scripts/dev-browser-helper.sh setup"
fi
fi
Expand Down Expand Up @@ -2216,9 +2238,21 @@ setup_ai_orchestration() {
print_success "Python $python_version found (3.10+ required)"
else
print_warning "Python 3.10+ required for AI orchestration, found $python_version"
echo ""
echo " Upgrade options:"
echo " macOS (Homebrew): brew install python@3.12"
echo " macOS (pyenv): pyenv install 3.12 && pyenv global 3.12"
echo " Ubuntu/Debian: sudo apt install python3.12"
echo " Fedora: sudo dnf install python3.12"
echo ""
fi
else
print_warning "Python 3 not found - AI orchestration frameworks unavailable"
echo ""
echo " Install options:"
echo " macOS: brew install python@3.12"
echo " Linux: sudo apt install python3 (or dnf/pacman)"
echo ""
return 0
fi

Expand Down
Loading