fix: fmt

Author: tip-dteller

cmd/atlas/search/search_test.go

@@ -13,7 +13,7 @@ func TestNewSearchCmd_VisibleAndSubcommands(t *testing.T) {
 	if cmd.Hidden {
 		t.Fatalf("expected search command to be visible, but it was hidden")
 	}
-	
+
 	// Verify command properties
 	if cmd.Use != "search" {
 		t.Errorf("expected Use to be 'search', got '%s'", cmd.Use)
@@ -24,13 +24,13 @@ func TestNewSearchCmd_VisibleAndSubcommands(t *testing.T) {
 	if cmd.Long == "" {
 		t.Error("expected Long description to be non-empty")
 	}
-	
+
 	// Verify aliases are present
 	expectedAliases := []string{"search-index", "search-indexes"}
 	if len(cmd.Aliases) != len(expectedAliases) {
 		t.Errorf("expected %d aliases, got %d", len(expectedAliases), len(cmd.Aliases))
 	}
-	
+
 	// Ensure all expected subcommands are present
 	found := map[string]bool{}
 	for _, c := range cmd.Commands() {
@@ -42,7 +42,7 @@ func TestNewSearchCmd_VisibleAndSubcommands(t *testing.T) {
 			t.Errorf("expected subcommand '%s' to be present", expectedCmd)
 		}
 	}
-	
+
 	if len(found) != len(expectedCommands) {
 		t.Errorf("expected exactly %d subcommands, got %d: %+v", len(expectedCommands), len(found), found)
 	}

cmd/config/config.go

@@ -906,27 +906,27 @@ func detectFileFormat(filename string, data []byte) string {
 func parseEnvFile(data []byte) map[string]interface{} {
 	config := make(map[string]interface{})
 	lines := strings.Split(string(data), "\n")
-	
+
 	for _, line := range lines {
 		line = strings.TrimSpace(line)
 		if line == "" || strings.HasPrefix(line, "#") {
 			continue
 		}
-		
+
 		parts := strings.SplitN(line, "=", 2)
 		if len(parts) != 2 {
 			continue
 		}
-		
+
 		key := strings.TrimSpace(parts[0])
 		value := strings.TrimSpace(parts[1])
-		
+
 		// Remove quotes if present
 		if (strings.HasPrefix(value, "\"") && strings.HasSuffix(value, "\"")) ||
-		   (strings.HasPrefix(value, "'") && strings.HasSuffix(value, "'")) {
+			(strings.HasPrefix(value, "'") && strings.HasSuffix(value, "'")) {
 			value = value[1 : len(value)-1]
 		}
-		
+
 		// Convert common Atlas environment variables to config keys
 		switch key {
 		case "ATLAS_PROJECT_ID":
@@ -952,13 +952,13 @@ func parseEnvFile(data []byte) map[string]interface{} {
 			}
 		}
 	}
-	
+
 	return config
 }
 
 func normalizeConfigKeys(config map[string]interface{}) map[string]interface{} {
 	normalized := make(map[string]interface{})
-	
+
 	for key, value := range config {
 		// Convert various key formats to standard camelCase
 		normalKey := key
@@ -974,26 +974,26 @@ func normalizeConfigKeys(config map[string]interface{}) map[string]interface{} {
 		case "public_key", "public-key", "pub_key", "pub-key", "publickey":
 			normalKey = "publicKey"
 		}
-		
+
 		normalized[normalKey] = value
 	}
-	
+
 	return normalized
 }
 
 func mergeConfigs(existing, source map[string]interface{}) map[string]interface{} {
 	result := make(map[string]interface{})
-	
+
 	// Copy existing config
 	for key, value := range existing {
 		result[key] = value
 	}
-	
+
 	// Overlay source config (source wins conflicts)
 	for key, value := range source {
 		result[key] = value
 	}
-	
+
 	return result
 }
 
@@ -1121,7 +1121,7 @@ func detectConfigVersion(config map[string]interface{}) string {
 	}
 
 	// Check for schema indicators to detect version
-	
+
 	// v2.0.0+ indicators (current format)
 	if _, hasProjectId := config["projectId"]; hasProjectId {
 		// Check for new camelCase format
@@ -1147,7 +1147,7 @@ func detectConfigVersion(config map[string]interface{}) string {
 
 func applyMigrations(config map[string]interface{}, fromVersion, toVersion string) (map[string]interface{}, error) {
 	result := make(map[string]interface{})
-	
+
 	// Copy original config
 	for k, v := range config {
 		result[k] = v
@@ -1176,7 +1176,7 @@ func applyMigrations(config map[string]interface{}, fromVersion, toVersion strin
 
 	// Add version field to migrated config
 	result["version"] = toVersion
-	
+
 	return result, nil
 }
 
@@ -1189,11 +1189,11 @@ type migration struct {
 func shouldApplyMigration(currentVersion, targetVersion, migrationFrom, migrationTo string) bool {
 	// Apply migration if:
 	// 1. Current version is at or past the migration starting point
-	// 2. Target version includes this migration step 
+	// 2. Target version includes this migration step
 	// 3. Current version is before the migration endpoint (to avoid applying migrations we've already done)
-	return versionLessOrEqual(migrationFrom, currentVersion) && 
-		   versionLessOrEqual(migrationTo, targetVersion) &&
-		   versionLess(currentVersion, migrationTo)
+	return versionLessOrEqual(migrationFrom, currentVersion) &&
+		versionLessOrEqual(migrationTo, targetVersion) &&
+		versionLess(currentVersion, migrationTo)
 }
 
 func versionLessOrEqual(v1, v2 string) bool {
@@ -1210,64 +1210,64 @@ func versionLess(v1, v2 string) bool {
 
 func migrateV0_9ToV1_0(config map[string]interface{}) (map[string]interface{}, error) {
 	result := make(map[string]interface{})
-	
+
 	// Copy all existing values
 	for k, v := range config {
 		result[k] = v
 	}
-	
+
 	// Add default values that were introduced in v1.0.0
 	if _, exists := result["timeout"]; !exists {
 		result["timeout"] = "30s"
 	}
 	if _, exists := result["output"]; !exists {
 		result["output"] = "text"
 	}
-	
+
 	return result, nil
 }
 
 func migrateV1_0ToV1_5(config map[string]interface{}) (map[string]interface{}, error) {
 	result := make(map[string]interface{})
-	
+
 	// Migrate from snake_case to camelCase (partial migration)
 	keyMappings := map[string]string{
 		"project_id":   "projectId",
-		"org_id":       "orgId", 
+		"org_id":       "orgId",
 		"cluster_name": "clusterName",
 		"api_key":      "apiKey",
 		"public_key":   "publicKey",
 	}
-	
+
 	for oldKey, newKey := range keyMappings {
 		if value, exists := config[oldKey]; exists {
 			result[newKey] = value
 		}
 	}
-	
+
 	// Copy other values as-is
 	for k, v := range config {
 		if _, isMapped := keyMappings[k]; !isMapped {
 			result[k] = v
 		}
 	}
-	
+
 	return result, nil
 }
 
 func migrateV1_5ToV2_0(config map[string]interface{}) (map[string]interface{}, error) {
 	result := make(map[string]interface{})
-	
+
 	// Copy all existing values (v1.5.0 → v2.0.0 is mostly compatible)
 	for k, v := range config {
 		result[k] = v
 	}
-	
+
 	// Ensure all keys are in proper camelCase format
 	result = normalizeConfigKeys(result)
-	
+
 	// Remove any deprecated keys
 	delete(result, "deprecated_field") // Example - remove any deprecated fields
-	
+
 	return result, nil
 }

cmd/config/config_test.go

@@ -9,7 +9,7 @@ import (
 
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
-	
+
 	"github.com/teabranch/matlas-cli/internal/config"
 )
 
@@ -412,12 +412,12 @@ func TestConfigToMap(t *testing.T) {
 	t.Run("Exclude secrets", func(t *testing.T) {
 		result := configToMap(cfg, false)
 		expected := map[string]interface{}{
-			"projectId":    "12345",
-			"clusterName":  "my-cluster",
-			"output":       "json",
-			"timeout":      "45s",
-			"# apiKey":     "[REDACTED - use --include-secrets to export]",
-			"# publicKey":  "[REDACTED - use --include-secrets to export]",
+			"projectId":   "12345",
+			"clusterName": "my-cluster",
+			"output":      "json",
+			"timeout":     "45s",
+			"# apiKey":    "[REDACTED - use --include-secrets to export]",
+			"# publicKey": "[REDACTED - use --include-secrets to export]",
 		}
 		assert.Equal(t, expected, result)
 	})
@@ -432,7 +432,7 @@ func TestConvertToEnvFormat(t *testing.T) {
 	}
 
 	result := convertToEnvFormat(configMap, true)
-	
+
 	assert.Contains(t, result, "ATLAS_PROJECT_ID=12345")
 	assert.Contains(t, result, "ATLAS_CLUSTER_NAME=my-cluster")
 	assert.Contains(t, result, "ATLAS_OUTPUT=json")
@@ -447,7 +447,7 @@ func TestConvertToShellExportFormat(t *testing.T) {
 	}
 
 	result := convertToShellExportFormat(configMap, true)
-	
+
 	assert.Contains(t, result, `export ATLAS_PROJECT_ID="12345"`)
 	assert.Contains(t, result, `export ATLAS_OUTPUT="json"`)
 	assert.Contains(t, result, "# matlas-cli configuration as shell export statements")
@@ -586,8 +586,8 @@ func TestMigrationTransformations(t *testing.T) {
 
 	t.Run("migrateV1_5ToV2_0", func(t *testing.T) {
 		input := map[string]interface{}{
-			"projectId":       "12345",
-			"cluster-name":    "my-cluster", // Mixed format
+			"projectId":        "12345",
+			"cluster-name":     "my-cluster", // Mixed format
 			"deprecated_field": "should-be-removed",
 		}
 

cmd/database/database.go

@@ -781,11 +781,11 @@ func testConnection(ctx context.Context, connectionString string) bool {
 
 	// Create MongoDB client options
 	clientOptions := options.Client().ApplyURI(connectionString)
-	
+
 	// Set connection timeout and server selection timeout
 	clientOptions.SetConnectTimeout(5 * time.Second)
 	clientOptions.SetServerSelectionTimeout(5 * time.Second)
-	
+
 	// Create MongoDB client
 	client, err := mongo.Connect(testCtx, clientOptions)
 	if err != nil {

cmd/infra/apply.go

@@ -837,7 +837,7 @@ func convertToClusterSpec(spec interface{}) types.ClusterSpec {
 		if clusterType, ok := specMap["clusterType"].(string); ok {
 			clusterSpec.ClusterType = clusterType
 		}
-		
+
 		// Convert complex fields
 		if replicationSpecs, ok := specMap["replicationSpecs"].([]interface{}); ok {
 			clusterSpec.ReplicationSpecs = convertToReplicationSpecs(replicationSpecs)
@@ -854,7 +854,7 @@ func convertToClusterSpec(spec interface{}) types.ClusterSpec {
 		if tags, ok := specMap["tags"].(map[string]interface{}); ok {
 			clusterSpec.Tags = convertToStringMap(tags)
 		}
-		
+
 		return clusterSpec
 	}
 	return types.ClusterSpec{}

internal/config/credentials.go

@@ -129,19 +129,19 @@ func getCredentialFromWindowsCredentialManager(service string) string {
 	// Use PowerShell to access Windows Credential Manager
 	// Get-StoredCredential -Target "matlas:<service>" | Select-Object -ExpandProperty Password
 	target := "matlas:" + service
-	cmd := exec.Command("powershell", "-Command", 
+	cmd := exec.Command("powershell", "-Command",
 		"try { $cred = Get-StoredCredential -Target '"+target+"' -ErrorAction Stop; "+
-		"[Runtime.InteropServices.Marshal]::PtrToStringAuto([Runtime.InteropServices.Marshal]::SecureStringToBSTR($cred.Password)) "+
-		"} catch { exit 1 }") // #nosec G204 -- target is sanitized service name
-	
+			"[Runtime.InteropServices.Marshal]::PtrToStringAuto([Runtime.InteropServices.Marshal]::SecureStringToBSTR($cred.Password)) "+
+			"} catch { exit 1 }") // #nosec G204 -- target is sanitized service name
+
 	out, err := cmd.Output()
 	if err == nil {
 		credential := strings.TrimSpace(string(out))
 		if credential != "" {
 			return credential
 		}
 	}
-	
+
 	// Fallback: try cmdkey (older method)
 	// This doesn't retrieve passwords, but we can check if the credential exists
 	// For security reasons, Windows doesn't easily expose stored passwords via cmdkey
@@ -160,7 +160,7 @@ func getCredentialFromLinuxSecretService(service string) string {
 			return credential
 		}
 	}
-	
+
 	// Fallback: try GNOME Keyring directly (older systems)
 	cmd = exec.Command("gnome-keyring", "get", "matlas-"+service) // #nosec G204 -- service is validated input
 	out, err = cmd.Output()
@@ -170,6 +170,6 @@ func getCredentialFromLinuxSecretService(service string) string {
 			return credential
 		}
 	}
-	
+
 	return ""
 }

internal/types/apply.go

@@ -170,7 +170,6 @@ type DatabaseUserSpec struct {
 	Scopes       []UserScopeConfig    `yaml:"scopes,omitempty" json:"scopes,omitempty"`
 }
 
-
 // DatabaseRoleManifest represents a database role resource manifest
 type DatabaseRoleManifest struct {
 	APIVersion APIVersion          `yaml:"apiVersion" json:"apiVersion"`

internal/types/apply_test.go

@@ -319,7 +319,6 @@ func TestDatabaseUserManifest_YAMLMarshaling(t *testing.T) {
 	}
 }
 
-
 func TestNetworkAccessManifest_YAMLMarshaling(t *testing.T) {
 	networkAccess := NetworkAccessManifest{
 		APIVersion: APIVersionV1,