fix: prevent SSO temp config leaks

internal/services/aws/sso.go

@@ -45,6 +45,7 @@ var newSSOClient = func(cfg aws.Config) SSOClientAPI {
 }
 
 var runSSOLoginFn = RunSSOLogin
+var writeSSOConfigFile = os.WriteFile
 
 // ssoTokenCache represents the cached SSO token file structure.
 type ssoTokenCache struct {
@@ -306,6 +307,9 @@ func BuildSSOLoginCmd(cfg *config.Config) (*exec.Cmd, func(), error) {
 		cmd := exec.Command("aws", "sso", "login", "--profile", cfg.Profile)
 		return cmd, func() {}, nil
 	}
+	if cfg.SSOAccountID == "" || cfg.SSORoleName == "" {
+		return nil, nil, fmt.Errorf("SSO login requires sso_account_id and sso_role_name when profile is not set")
+	}
 
 	tmpDir, err := os.MkdirTemp("", "unic-sso-*")
 	if err != nil {
@@ -324,13 +328,14 @@ func BuildSSOLoginCmd(cfg *config.Config) (*exec.Cmd, func(), error) {
 	)
 
 	configPath := filepath.Join(tmpDir, "config")
-	if err := os.WriteFile(configPath, []byte(configContent), 0600); err != nil {
+	if err := writeSSOConfigFile(configPath, []byte(configContent), 0600); err != nil {
 		cleanup()
 		return nil, nil, fmt.Errorf("failed to write temp SSO config: %w", err)
 	}
 
 	cmd := exec.Command("aws", "sso", "login", "--profile", profileName)
 	cmd.Env = append(os.Environ(), "AWS_CONFIG_FILE="+configPath)
+	// On success, callers own the temporary config directory and must defer cleanup().
 	return cmd, cleanup, nil
 }
 

internal/services/aws/sso_test.go

@@ -4,6 +4,7 @@ import (
 	"crypto/sha1"
 	"encoding/hex"
 	"encoding/json"
+	"errors"
 	"os"
 	"path/filepath"
 	"testing"
@@ -141,3 +142,33 @@ func TestEnsureSSOLoginRunsLoginWhenTokenExpired(t *testing.T) {
 		t.Fatal("expected login result to report a refreshed session")
 	}
 }
+
+func TestBuildSSOLoginCmdCleansTempDirOnConfigWriteError(t *testing.T) {
+	tempRoot := t.TempDir()
+	t.Setenv("TMPDIR", tempRoot)
+
+	origWriteSSOConfigFile := writeSSOConfigFile
+	defer func() { writeSSOConfigFile = origWriteSSOConfigFile }()
+	writeSSOConfigFile = func(string, []byte, os.FileMode) error {
+		return errors.New("write failed")
+	}
+
+	cmd, cleanup, err := BuildSSOLoginCmd(testSSOConfig())
+	if err == nil {
+		t.Fatal("expected config write error")
+	}
+	if cmd != nil {
+		t.Fatalf("expected nil command on error, got %v", cmd)
+	}
+	if cleanup != nil {
+		t.Fatal("expected nil cleanup when setup fails")
+	}
+
+	matches, globErr := filepath.Glob(filepath.Join(tempRoot, "unic-sso-*"))
+	if globErr != nil {
+		t.Fatalf("failed to inspect temp dirs: %v", globErr)
+	}
+	if len(matches) != 0 {
+		t.Fatalf("expected temp config directory to be removed, found %v", matches)
+	}
+}