Swap clicfg hook function arguments for easier usage

Author: Alexis Destrez

cfg/cfg.go

@@ -11,7 +11,9 @@ import (
 type SourceFunc[T any] func(ctx context.Context, cfg *T) error
 
 // BeforeCommandExecutionHook replaces dest with provided config sources.
-func BeforeCommandExecutionHook[T any](sources []SourceFunc[T], dest *T) cli.HookFunc {
+func BeforeCommandExecutionHook[T any](dest *T, source SourceFunc[T], sources ...SourceFunc[T]) cli.HookFunc {
+	sources = append([]SourceFunc[T]{source}, sources...)
+
 	return func(ctx context.Context) error {
 		cfg := new(T)
 

cfg/cfg_test.go

@@ -15,7 +15,7 @@ func Test_BeforeCommandExecutionHook(t *testing.T) {
 	var cfg config
 
 	t.Run("ok", func(t *testing.T) {
-		assert.NilError(t, BeforeCommandExecutionHook([]SourceFunc[config]{
+		assert.NilError(t, BeforeCommandExecutionHook(&cfg,
 			func(_ context.Context, cfg *config) error {
 				cfg.A += "1"
 				return nil
@@ -28,16 +28,18 @@ func Test_BeforeCommandExecutionHook(t *testing.T) {
 				cfg.A += "3"
 				return nil
 			},
-		}, &cfg)(context.Background()))
+		)(context.Background()))
 
 		assert.Check(t, cfg.A == "123")
 	})
 
 	t.Run("error", func(t *testing.T) {
 		expectedErr := errors.New("boom")
 
-		assert.ErrorIs(t, BeforeCommandExecutionHook([]SourceFunc[config]{
-			func(context.Context, *config) error { return expectedErr },
-		}, &cfg)(context.Background()), expectedErr)
+		assert.ErrorIs(t, BeforeCommandExecutionHook(&cfg,
+			func(context.Context, *config) error {
+				return expectedErr
+			},
+		)(context.Background()), expectedErr)
 	})
 }