refactor(config): reuse atomicWriteFile in engine migration

Replace duplicated atomic write logic with existing atomicWriteFile
utility from writer.go.

commit: ad1fde7683442dbd2368ebff6b5639bd11a8c34d
author: brice · 7mo ago · 2026-03-18 05:12 UTC
parent: 2b3891d
signature: GPG VerifiedCD88F9E564E0102646FB6077A1A07C7C7C3E349D · arch-gpg Verified for the committer email

1 file · +3 -4

M internal/infrastructure/config/migrate_engine.go +3 -4


@@ -2,10 +2,9 @@ package config
2	2
3	3		import (
4	4		"fmt"
	5	+	"github.com/pelletier/go-toml/v2"
5	6		"io/fs"
6	7		"os"
7		-
8		-	"github.com/pelletier/go-toml/v2"
9	8		)
10	9
11	10		// configFilePermissions is the file permission used when writing migrated config files.

@@ -129,8 +128,8 @@ func MigrateToEngineConfig(configFile string) (bool, error) {
129	128		return false, marshalErr
130	129		}
131	130
132		-	if writeErr := os.WriteFile(configFile, out, configFilePermissions); writeErr != nil {
133		-	return false, writeErr
	131	+	if err := atomicWriteFile(configFile, string(out), configFilePermissions); err != nil {
	132	+	return false, err
134	133		}
135	134
136	135		return true, nil

 import (
 	"fmt"
+	"github.com/pelletier/go-toml/v2"
 	"io/fs"
 	"os"
-	"github.com/pelletier/go-toml/v2"
 )
 // configFilePermissions is the file permission used when writing migrated config files.
 		return false, marshalErr
 	}
-	if writeErr := os.WriteFile(configFile, out, configFilePermissions); writeErr != nil {
+	if err := atomicWriteFile(configFile, string(out), configFilePermissions); err != nil {
-		return false, writeErr
+		return false, err
 	}
 	return true, nil