feat(brew): add GetInstalledTaps and Untap

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>

Author: fullstackjam

internal/brew/brew.go

@@ -652,6 +652,52 @@ func UninstallCask(packages []string, dryRun bool) error {
 	return nil
 }
 
+func GetInstalledTaps() ([]string, error) {
+	cmd := exec.Command("brew", "tap")
+	output, err := cmd.Output()
+	if err != nil {
+		return nil, fmt.Errorf("brew tap: %w", err)
+	}
+	var taps []string
+	for _, line := range strings.Split(strings.TrimSpace(string(output)), "\n") {
+		line = strings.TrimSpace(line)
+		if line != "" {
+			taps = append(taps, line)
+		}
+	}
+	return taps, nil
+}
+
+func Untap(taps []string, dryRun bool) error {
+	if len(taps) == 0 {
+		return nil
+	}
+
+	if dryRun {
+		ui.Info("Would remove taps:")
+		for _, t := range taps {
+			fmt.Printf("    brew untap %s\n", t)
+		}
+		return nil
+	}
+
+	var failed []string
+	for _, tap := range taps {
+		cmd := exec.Command("brew", "untap", tap)
+		if output, err := cmd.CombinedOutput(); err != nil {
+			ui.Warn(fmt.Sprintf("Failed to remove tap %s: %s", tap, strings.TrimSpace(string(output))))
+			failed = append(failed, tap)
+		} else {
+			ui.Success(fmt.Sprintf("  ✔ Removed tap %s", tap))
+		}
+	}
+
+	if len(failed) > 0 {
+		return fmt.Errorf("%d tap(s) failed to remove", len(failed))
+	}
+	return nil
+}
+
 func Update(dryRun bool) error {
 	if dryRun {
 		ui.Info("Would run: brew update && brew upgrade")

internal/brew/brew_command_test.go

@@ -89,3 +89,119 @@ func TestUpdateAndCleanup_UsesBrew(t *testing.T) {
 	err = Cleanup()
 	assert.NoError(t, err)
 }
+
+func TestUninstall_Empty(t *testing.T) {
+	err := Uninstall([]string{}, false)
+	assert.NoError(t, err)
+}
+
+func TestUninstall_DryRun(t *testing.T) {
+	err := Uninstall([]string{"git", "curl"}, true)
+	assert.NoError(t, err)
+}
+
+func TestUninstall_Success(t *testing.T) {
+	setupFakeBrew(t, "#!/bin/sh\nexit 0\n")
+	err := Uninstall([]string{"wget", "jq"}, false)
+	assert.NoError(t, err)
+}
+
+func TestUninstall_Failure(t *testing.T) {
+	setupFakeBrew(t, "#!/bin/sh\necho 'Error: No such keg'\nexit 1\n")
+	err := Uninstall([]string{"nonexistent"}, false)
+	assert.Error(t, err)
+}
+
+func TestUninstall_PartialFailure(t *testing.T) {
+	setupFakeBrew(t, "#!/bin/sh\n"+
+		"if [ \"$2\" = \"bad-pkg\" ]; then\n"+
+		"  echo 'Error: No such keg'\n"+
+		"  exit 1\n"+
+		"fi\n"+
+		"exit 0\n")
+	err := Uninstall([]string{"good-pkg", "bad-pkg"}, false)
+	assert.Error(t, err)
+}
+
+func TestUninstallCask_Empty(t *testing.T) {
+	err := UninstallCask([]string{}, false)
+	assert.NoError(t, err)
+}
+
+func TestUninstallCask_DryRun(t *testing.T) {
+	err := UninstallCask([]string{"firefox", "slack"}, true)
+	assert.NoError(t, err)
+}
+
+func TestUninstallCask_Success(t *testing.T) {
+	setupFakeBrew(t, "#!/bin/sh\nexit 0\n")
+	err := UninstallCask([]string{"firefox"}, false)
+	assert.NoError(t, err)
+}
+
+func TestUninstallCask_Failure(t *testing.T) {
+	setupFakeBrew(t, "#!/bin/sh\necho 'Error: Cask not found'\nexit 1\n")
+	err := UninstallCask([]string{"nonexistent-cask"}, false)
+	assert.Error(t, err)
+}
+
+func TestDoctorDiagnose_ReadyToBrew(t *testing.T) {
+	setupFakeBrew(t, "#!/bin/sh\nif [ \"$1\" = \"doctor\" ]; then\n  echo 'Your system is ready to brew.'\n  exit 0\nfi\nexit 0\n")
+	suggestions, err := DoctorDiagnose()
+	require.NoError(t, err)
+	assert.Nil(t, suggestions)
+}
+
+func TestDoctorDiagnose_Failure(t *testing.T) {
+	setupFakeBrew(t, "#!/bin/sh\nif [ \"$1\" = \"doctor\" ]; then\n  exit 1\nfi\nexit 0\n")
+	_, err := DoctorDiagnose()
+	assert.Error(t, err)
+}
+
+func TestDoctorDiagnose_MultipleWarnings(t *testing.T) {
+	setupFakeBrew(t, "#!/bin/sh\n"+
+		"if [ \"$1\" = \"doctor\" ]; then\n"+
+		"  echo 'Warning: Unbrewed dylibs were found in /usr/local/lib'\n"+
+		"  echo 'Warning: Your Homebrew/homebrew/core tap is not a full clone'\n"+
+		"  echo 'Warning: Git origin remote mismatch'\n"+
+		"  echo 'Warning: Uncommitted modifications to Homebrew'\n"+
+		"  echo 'Warning: outdated Xcode command line tools'\n"+
+		"  echo 'Warning: Broken symlinks were found'\n"+
+		"  echo 'Warning: permission issues'\n"+
+		"  exit 0\n"+
+		"fi\n"+
+		"exit 0\n")
+	suggestions, err := DoctorDiagnose()
+	require.NoError(t, err)
+	assert.NotEmpty(t, suggestions)
+	assert.Contains(t, suggestions, "Run: brew doctor --list-checks and review linked libraries")
+	assert.Contains(t, suggestions, "Run: brew untap homebrew/core homebrew/cask")
+	assert.Contains(t, suggestions, "Run: brew update-reset")
+	assert.Contains(t, suggestions, "Run: xcode-select --install")
+	assert.Contains(t, suggestions, "Run: brew cleanup --prune=all")
+	assert.Contains(t, suggestions, "Run: sudo chown -R $(whoami) $(brew --prefix)/*")
+}
+
+func TestDoctorDiagnose_UnknownWarnings(t *testing.T) {
+	setupFakeBrew(t, "#!/bin/sh\n"+
+		"if [ \"$1\" = \"doctor\" ]; then\n"+
+		"  echo 'Warning: Some unknown issue'\n"+
+		"  exit 0\n"+
+		"fi\n"+
+		"exit 0\n")
+	suggestions, err := DoctorDiagnose()
+	require.NoError(t, err)
+	assert.Contains(t, suggestions, "Run 'brew doctor' to see full diagnostic output")
+}
+
+func TestBrewInstallCmd_SetsNoAutoUpdate(t *testing.T) {
+	cmd := brewInstallCmd("install", "git")
+	assert.Equal(t, []string{"brew", "install", "git"}, cmd.Args)
+	found := false
+	for _, env := range cmd.Env {
+		if env == "HOMEBREW_NO_AUTO_UPDATE=1" {
+			found = true
+		}
+	}
+	assert.True(t, found, "expected HOMEBREW_NO_AUTO_UPDATE=1 in env")
+}

internal/brew/brew_test.go

@@ -4,6 +4,7 @@ import (
 	"strings"
 	"testing"
 
+	"github.com/openbootdotdev/openboot/internal/ui"
 	"github.com/stretchr/testify/assert"
 )
 
@@ -223,3 +224,14 @@ func TestIsInstalled(t *testing.T) {
 func TestMaxWorkersConstant(t *testing.T) {
 	assert.Equal(t, 1, maxWorkers)
 }
+
+func TestPrintBrewOutput_Lines(t *testing.T) {
+	p := ui.NewStickyProgress(1)
+	printBrewOutput("line one\nline two\n  line three  \n", p)
+}
+
+func TestPrintBrewOutput_Empty(t *testing.T) {
+	p := ui.NewStickyProgress(1)
+	printBrewOutput("", p)
+	printBrewOutput("   \n   ", p)
+}