fix(config): use ordered writer in migrate command

commit: 19525f0fa89d79578ee96fafdd704101ef9dd522
author: brice · 9mo ago · 2026-01-24 08:05 UTC
parent: f21fa3f
signature: GPG VerifiedCD88F9E564E0102646FB6077A1A07C7C7C3E349D · arch-gpg Verified for the committer email

1 file · +8 -2

M internal/infrastructure/config/migrate.go +8 -2


@@ -420,8 +420,14 @@ func (m *Migrator) Migrate() ([]string, error) {
420	420		return nil, fmt.Errorf("failed to merge config: %w", err)
421	421		}
422	422
423		-	// Write the merged config back
424		-	if err := userViper.WriteConfig(); err != nil {
	423	+	// Unmarshal to Config struct for ordered writing
	424	+	var cfg Config
	425	+	if err := userViper.Unmarshal(&cfg); err != nil {
	426	+	return nil, fmt.Errorf("failed to unmarshal merged config: %w", err)
	427	+	}
	428	+
	429	+	// Write with ordered sections
	430	+	if err := WriteConfigOrdered(&cfg, configFile); err != nil {
425	431		return nil, fmt.Errorf("failed to write config file: %w", err)
426	432		}
427	433

 		return nil, fmt.Errorf("failed to merge config: %w", err)
 	}
-	// Write the merged config back
+	// Unmarshal to Config struct for ordered writing
-	if err := userViper.WriteConfig(); err != nil {
+	var cfg Config
+	if err := userViper.Unmarshal(&cfg); err != nil {
+		return nil, fmt.Errorf("failed to unmarshal merged config: %w", err)
+	}
+	// Write with ordered sections
+	if err := WriteConfigOrdered(&cfg, configFile); err != nil {
 		return nil, fmt.Errorf("failed to write config file: %w", err)
 	}