pkg/instance/start.go: fix lint error on windows

```
Error: pkg\instance\start.go:274:65: SA4023: this comparison is always true (staticcheck)
  		if err := syscall.Exec(limactl, haCmd.Args, haCmd.Environ()); err != nil {
```

Signed-off-by: Akihiro Suda <[email protected]>
(cherry picked from commit bdf7c7f)
> Conflicts:
>	pkg/instance/start.go
Signed-off-by: Akihiro Suda <[email protected]>

Author: AkihiroSuda

pkg/instance/start.go

@@ -12,13 +12,11 @@ import (
 	"os"
 	"os/exec"
 	"path/filepath"
-	"syscall"
 	"text/template"
 	"time"
 
 	"github.com/docker/go-units"
 	"github.com/lima-vm/go-qcow2reader"
-	"github.com/mattn/go-isatty"
 	"github.com/sirupsen/logrus"
 
 	"github.com/lima-vm/lima/pkg/downloader"
@@ -29,7 +27,6 @@ import (
 	hostagentevents "github.com/lima-vm/lima/pkg/hostagent/events"
 	"github.com/lima-vm/lima/pkg/imgutil/proxyimgutil"
 	"github.com/lima-vm/lima/pkg/limayaml"
-	"github.com/lima-vm/lima/pkg/osutil"
 	"github.com/lima-vm/lima/pkg/store"
 	"github.com/lima-vm/lima/pkg/store/filenames"
 	"github.com/lima-vm/lima/pkg/usrlocalsharelima"
@@ -249,25 +246,7 @@ func Start(ctx context.Context, inst *store.Instance, limactl string, launchHost
 	begin := time.Now() // used for logrus propagation
 
 	if launchHostAgentForeground {
-		logrus.Info("Running the host agent in the foreground")
-		if isatty.IsTerminal(os.Stdin.Fd()) || isatty.IsCygwinTerminal(os.Stdin.Fd()) {
-			// Write message to standard log files to avoid confusing users
-			message := "This log file is not used because `limactl start` was launched in the terminal with the `--foreground` option."
-			if _, err := haStdoutW.WriteString(message); err != nil {
-				return err
-			}
-			if _, err := haStderrW.WriteString(message); err != nil {
-				return err
-			}
-		} else {
-			if err := osutil.Dup2(int(haStdoutW.Fd()), syscall.Stdout); err != nil {
-				return err
-			}
-			if err := osutil.Dup2(int(haStderrW.Fd()), syscall.Stderr); err != nil {
-				return err
-			}
-		}
-		if err := syscall.Exec(limactl, haCmd.Args, haCmd.Environ()); err != nil {
+		if err := execHostAgentForeground(limactl, haCmd); err != nil {
 			return err
 		}
 	} else if err := haCmd.Start(); err != nil {

pkg/instance/start_unix.go

@@ -0,0 +1,48 @@
+//go:build !windows
+
+// SPDX-FileCopyrightText: Copyright The Lima Authors
+// SPDX-License-Identifier: Apache-2.0
+
+package instance
+
+import (
+	"fmt"
+	"os"
+	"os/exec"
+	"syscall"
+
+	"github.com/mattn/go-isatty"
+	"github.com/sirupsen/logrus"
+
+	"github.com/lima-vm/lima/pkg/osutil"
+)
+
+func execHostAgentForeground(limactl string, haCmd *exec.Cmd) error {
+	haStdoutW, ok := haCmd.Stdout.(*os.File)
+	if !ok {
+		return fmt.Errorf("expected haCmd.Stdout to be *os.File, got %T", haCmd.Stdout)
+	}
+	haStderrW, ok := haCmd.Stderr.(*os.File)
+	if !ok {
+		return fmt.Errorf("expected haCmd.Stderr to be *os.File, got %T", haCmd.Stderr)
+	}
+	logrus.Info("Running the host agent in the foreground")
+	if isatty.IsTerminal(os.Stdin.Fd()) || isatty.IsCygwinTerminal(os.Stdin.Fd()) {
+		// Write message to standard log files to avoid confusing users
+		message := "This log file is not used because `limactl start` was launched in the terminal with the `--foreground` option."
+		if _, err := haStdoutW.WriteString(message); err != nil {
+			return err
+		}
+		if _, err := haStderrW.WriteString(message); err != nil {
+			return err
+		}
+	} else {
+		if err := osutil.Dup2(int(haStdoutW.Fd()), syscall.Stdout); err != nil {
+			return err
+		}
+		if err := osutil.Dup2(int(haStderrW.Fd()), syscall.Stderr); err != nil {
+			return err
+		}
+	}
+	return syscall.Exec(limactl, haCmd.Args, haCmd.Environ())
+}

pkg/instance/start_windows.go

@@ -0,0 +1,13 @@
+// SPDX-FileCopyrightText: Copyright The Lima Authors
+// SPDX-License-Identifier: Apache-2.0
+
+package instance
+
+import (
+	"errors"
+	"os/exec"
+)
+
+func execHostAgentForeground(_ string, _ *exec.Cmd) error {
+	return errors.New("`limactl start --foreground` is not supported on Windows")
+}