Merge pull request #124 from LackOfMorals/main

Fix for issue #108

Author: angrykoala

.changes/unreleased/Fixed-20250813-113459.yaml

@@ -0,0 +1,3 @@
+kind: Fixed
+body: 'Fix for issue #108 where running aura-cli for the first time to set a config value causes a panic'
+time: 2025-08-13T11:34:59.820531+01:00

common/clicfg/clicfg.go

@@ -34,6 +34,7 @@ type Config struct {
 
 func NewConfig(fs afero.Fs, version string) *Config {
 	configPath := filepath.Join(ConfigPrefix, "neo4j", "cli")
+	fullConfigPath := filepath.Join(configPath, "config.json")
 
 	Viper := viper.New()
 
@@ -46,20 +47,20 @@ func NewConfig(fs afero.Fs, version string) *Config {
 	bindEnvironmentVariables(Viper)
 	setDefaultValues(Viper)
 
-	if err := Viper.ReadInConfig(); err != nil {
-		if _, ok := err.(viper.ConfigFileNotFoundError); ok {
-			if err := fs.MkdirAll(configPath, 0755); err != nil {
-				panic(err)
-			}
-			if err = Viper.SafeWriteConfig(); err != nil {
-				panic(err)
-			}
-		} else {
-			// Config file was found but another error was produced
+	if !fileutils.FileExists(fs, fullConfigPath) {
+		if err := fs.MkdirAll(configPath, 0755); err != nil {
+			panic(err)
+		}
+		if err := Viper.SafeWriteConfig(); err != nil {
 			panic(err)
 		}
 	}
 
+	if err := Viper.ReadInConfig(); err != nil {
+		fmt.Println("Cannot read config file.")
+		panic(err)
+	}
+
 	credentials := credentials.NewCredentials(fs, ConfigPrefix)
 
 	return &Config{