From 0c2a91b4b27e2e29674c330166160d291aad988a Mon Sep 17 00:00:00 2001
From: Kyle Carberry <kyle@carberry.com>
Date: Wed, 4 Oct 2023 15:01:07 +0000
Subject: [PATCH] fix: allow all environment variables to fallback prefix to
 `HOMEBREW_`

See the customer use-case in the code docs.
---
 cli/clibase/option.go      |  9 +++++++++
 cli/clibase/option_test.go | 20 ++++++++++++++++++++
 2 files changed, 29 insertions(+)

diff --git a/cli/clibase/option.go b/cli/clibase/option.go
index b9a2e871d05df..5743b3a4d1efe 100644
--- a/cli/clibase/option.go
+++ b/cli/clibase/option.go
@@ -262,6 +262,15 @@ func (optSet *OptionSet) ParseEnv(vs []EnvVar) error {
 		}
 
 		envVal, ok := envs[opt.Env]
+		if !ok {
+			// Homebrew strips all environment variables that do not start with `HOMEBREW_`.
+			// This prevented using brew to invoke the Coder agent, because the environment
+			// variables to not get passed down.
+			//
+			// A customer wanted to use their custom tap inside a workspace, which was failing
+			// because the agent lacked the environment variables to authenticate with Git.
+			envVal, ok = envs[`HOMEBREW_`+opt.Env]
+		}
 		// Currently, empty values are treated as if the environment variable is
 		// unset. This behavior is technically not correct as there is now no
 		// way for a user to change a Default value to an empty string from
diff --git a/cli/clibase/option_test.go b/cli/clibase/option_test.go
index 79db040ea7e47..f093a20ec18da 100644
--- a/cli/clibase/option_test.go
+++ b/cli/clibase/option_test.go
@@ -206,6 +206,26 @@ func TestOptionSet_ParseEnv(t *testing.T) {
 		require.NoError(t, err)
 		require.EqualValues(t, expected, actual.Value)
 	})
+
+	t.Run("Homebrew", func(t *testing.T) {
+		t.Parallel()
+
+		var agentToken clibase.String
+
+		os := clibase.OptionSet{
+			clibase.Option{
+				Name:  "Agent Token",
+				Value: &agentToken,
+				Env:   "AGENT_TOKEN",
+			},
+		}
+
+		err := os.ParseEnv([]clibase.EnvVar{
+			{Name: "HOMEBREW_AGENT_TOKEN", Value: "foo"},
+		})
+		require.NoError(t, err)
+		require.EqualValues(t, "foo", agentToken)
+	})
 }
 
 func TestOptionSet_JsonMarshal(t *testing.T) {