cleanup: remove package bicopy

This has a single caller that doesn't even use the full interface.

Signed-off-by: Tamir Duberstein <[email protected]>

pkg/hostagent/port_darwin.go

@@ -2,17 +2,19 @@ package hostagent
 
 import (
 	"context"
+	"errors"
 	"fmt"
+	"io"
 	"net"
 	"os"
 	"path/filepath"
 	"strconv"
 	"strings"
 
-	"github.com/lima-vm/lima/pkg/bicopy"
 	"github.com/lima-vm/lima/pkg/portfwd"
 	"github.com/lima-vm/sshocker/pkg/ssh"
 	"github.com/sirupsen/logrus"
+	"golang.org/x/sync/errgroup"
 )
 
 // forwardTCP is not thread-safe.
@@ -149,8 +151,25 @@ func (plf *pseudoLoopbackForwarder) forward(ac *net.TCPConn) error {
 		return err
 	}
 	defer unixConn.Close()
-	bicopy.Bicopy(ac, unixConn, nil)
-	return nil
+
+	g, _ := errgroup.WithContext(context.Background())
+
+	g.Go(func() error {
+		_, err := io.Copy(unixConn, ac)
+		if errors.Is(err, io.EOF) {
+			err = nil
+		}
+		return errors.Join(err, unixConn.CloseRead(), ac.CloseWrite())
+	})
+	g.Go(func() error {
+		_, err := io.Copy(ac, unixConn)
+		if errors.Is(err, io.EOF) {
+			err = nil
+		}
+		return errors.Join(err, ac.CloseRead(), unixConn.CloseWrite())
+	})
+
+	return g.Wait()
 }
 
 func (plf *pseudoLoopbackForwarder) Close() error {