fix: stabilize logout tests and scope config env overrides

- use viper YES flag in logout tests instead of stdin timing\n- only apply SUPABASE_* overrides to user-configurable config fields\n- recurse into embedded base config fields while skipping internal and map-backed fields

Author: 20000419

internal/logout/logout_test.go

@@ -6,6 +6,7 @@ import (
 	"testing"
 
 	"github.com/spf13/afero"
+	"github.com/spf13/viper"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 	"github.com/supabase/cli/internal/testing/apitest"
@@ -19,7 +20,8 @@ func TestLogoutCommand(t *testing.T) {
 
 	t.Run("login with token and logout", func(t *testing.T) {
 		keyring.MockInitWithError(keyring.ErrUnsupportedPlatform)
-		t.Setenv("YES", "true")
+		viper.Set("YES", true)
+		t.Cleanup(viper.Reset)
 		// Setup in-memory fs
 		fsys := afero.NewMemMapFs()
 		require.NoError(t, utils.SaveAccessToken(token, fsys))
@@ -34,10 +36,11 @@ func TestLogoutCommand(t *testing.T) {
 
 	t.Run("removes all Supabase CLI credentials", func(t *testing.T) {
 		keyring.MockInit()
+		viper.Set("YES", true)
+		t.Cleanup(viper.Reset)
 		require.NoError(t, credentials.StoreProvider.Set(utils.CurrentProfile.Name, token))
 		require.NoError(t, credentials.StoreProvider.Set("project1", "password1"))
 		require.NoError(t, credentials.StoreProvider.Set("project2", "password2"))
-		t.Setenv("YES", "true")
 		// Run test
 		err := Run(context.Background(), os.Stdout, afero.NewMemMapFs())
 		// Check error
@@ -69,7 +72,8 @@ func TestLogoutCommand(t *testing.T) {
 
 	t.Run("exits 0 if not logged in", func(t *testing.T) {
 		keyring.MockInit()
-		t.Setenv("YES", "true")
+		viper.Set("YES", true)
+		t.Cleanup(viper.Reset)
 		// Setup in-memory fs
 		fsys := afero.NewMemMapFs()
 		// Run test
@@ -80,7 +84,8 @@ func TestLogoutCommand(t *testing.T) {
 
 	t.Run("throws error on failure to delete", func(t *testing.T) {
 		keyring.MockInitWithError(keyring.ErrNotFound)
-		t.Setenv("YES", "true")
+		viper.Set("YES", true)
+		t.Cleanup(viper.Reset)
 		// Setup empty home directory
 		t.Setenv("HOME", "")
 		// Setup in-memory fs

pkg/config/config.go

@@ -462,7 +462,6 @@ func (c *config) Eject(w io.Writer) error {
 // Loads custom config file to struct fields tagged with toml.
 func (c *config) loadFromFile(filename string, fsys fs.FS) error {
 	v := viper.New()
-	v.SetEnvPrefix("SUPABASE")
 	if err := c.mergeDefaultValues(v); err != nil {
 		return err
 	} else if err := mergeFileConfig(v, filename, fsys); err != nil {
@@ -494,15 +493,15 @@ func bindUserConfigEnv(v *viper.Viper, t reflect.Type, prefix string) error {
 	textUnmarshaler := reflect.TypeOf((*encoding.TextUnmarshaler)(nil)).Elem()
 	for i := 0; i < t.NumField(); i++ {
 		field := t.Field(i)
-		if field.PkgPath != "" {
-			continue
-		}
 		if field.Anonymous {
 			if err := bindUserConfigEnv(v, field.Type, prefix); err != nil {
 				return err
 			}
 			continue
 		}
+		if field.PkgPath != "" {
+			continue
+		}
 		if tag := strings.Split(field.Tag.Get("toml"), ",")[0]; tag == "-" {
 			continue
 		}
@@ -519,14 +518,18 @@ func bindUserConfigEnv(v *viper.Viper, t reflect.Type, prefix string) error {
 		if fieldType.Kind() == reflect.Ptr {
 			fieldType = fieldType.Elem()
 		}
+		if fieldType.Kind() == reflect.Map {
+			continue
+		}
 		if fieldType.Kind() == reflect.Struct && !fieldType.Implements(textUnmarshaler) && !reflect.PointerTo(fieldType).Implements(textUnmarshaler) {
 			if err := bindUserConfigEnv(v, fieldType, key); err != nil {
 				return err
 			}
 			continue
 		}
-		if err := v.BindEnv(key); err != nil {
-			return errors.Errorf("failed to bind env override for %s: %w", key, err)
+		envKey := "SUPABASE_" + strings.ToUpper(strings.ReplaceAll(key, ".", "_"))
+		if value, ok := os.LookupEnv(envKey); ok {
+			v.Set(key, value)
 		}
 	}
 	return nil