Simplify inteface definition to work with a bare function

It turns out that a bare function is not treated the same way as
a struct when it comes to matching it to an interface

cmd/timoni/apply.go

@@ -31,6 +31,7 @@ import (
 	"github.com/stefanprodan/timoni/internal/engine/fetcher"
 	cueerrors "github.com/stefanprodan/timoni/internal/errors"
 	"github.com/stefanprodan/timoni/internal/flags"
+	"github.com/stefanprodan/timoni/internal/logger"
 	"github.com/stefanprodan/timoni/internal/reconciler"
 	"github.com/stefanprodan/timoni/internal/runtime"
 )
@@ -240,10 +241,10 @@ func runApplyCmd(cmd *cobra.Command, args []string) error {
 			OverwriteOwnership: applyArgs.overwriteOwnership,
 		},
 		&reconciler.InteractiveOptions{
-			DryRun:     applyArgs.dryrun,
-			Diff:       applyArgs.diff,
-			DiffOutput: cmd.OutOrStdout(),
-			// ProgressStart:      logger.StartSpinner,
+			DryRun:        applyArgs.dryrun,
+			Diff:          applyArgs.diff,
+			DiffOutput:    cmd.OutOrStdout(),
+			ProgressStart: logger.StartSpinner,
 		},
 		rootArgs.timeout,
 	)

cmd/timoni/bundle_apply.go

@@ -328,10 +328,10 @@ func applyBundleInstance(ctx context.Context, cuectx *cue.Context, instance *eng
 			OverwriteOwnership: bundleApplyArgs.overwriteOwnership,
 		},
 		&reconciler.InteractiveOptions{
-			DryRun:     bundleApplyArgs.dryrun,
-			Diff:       bundleApplyArgs.diff,
-			DiffOutput: diffOutput,
-			// ProgressStart:      logger.StartSpinner,
+			DryRun:        bundleApplyArgs.dryrun,
+			Diff:          bundleApplyArgs.diff,
+			DiffOutput:    diffOutput,
+			ProgressStart: logger.StartSpinner,
 		},
 		rootArgs.timeout,
 	)

internal/logger/logger.go

@@ -200,7 +200,7 @@ func ColorizeCluster(cluster string) string {
 }
 
 // StartSpinner starts a spinner with the given message.
-func StartSpinner(msg string) *spinner.Spinner {
+func StartSpinner(msg string) interface{ Stop() } {
 	s := spinner.New(spinner.CharSets[11], 100*time.Millisecond, spinner.WithWriter(os.Stderr))
 	s.Suffix = " " + msg
 	s.Start()

internal/reconciler/types.go

@@ -42,7 +42,7 @@ type InteractiveOptions struct {
 	Diff       bool
 	DiffOutput io.Writer
 
-	ProgressStart ProgressStarter
+	ProgressStart func(string) interface{ Stop() }
 }
 
 type Reconciler struct {
@@ -61,19 +61,14 @@ type Reconciler struct {
 	applyOptions ssa.ApplyOptions
 	waitOptions  ssa.WaitOptions
 
-	progressStartFn ProgressStarter
+	progressStartFn func(string) interface{ Stop() }
 }
 
 type InteractiveReconciler struct {
 	*Reconciler
 	*InteractiveOptions
 }
 
-type (
-	ProgressStarter func(string) ProgressStopper
-	ProgressStopper interface{ Stop() }
-)
-
 type noopProgressStopper struct{}
 
 func (*noopProgressStopper) Stop() {}