fix: path handling for cross-platform and guest paths

- Fix inconsistent path separator handling on Windows
- Use path.Join for guest paths (always forward slashes)
- Update hypeman-go dependency

Author: rgarcia

go.mod

@@ -12,7 +12,7 @@ require (
 	github.com/gorilla/websocket v1.5.3
 	github.com/itchyny/json2yaml v0.1.4
 	github.com/muesli/reflow v0.3.0
-	github.com/onkernel/hypeman-go v0.7.1-0.20251223013311-b97d5b836009
+	github.com/onkernel/hypeman-go v0.7.1-0.20251223015448-3fc21a9216cf
 	github.com/tidwall/gjson v1.18.0
 	github.com/tidwall/pretty v1.2.1
 	github.com/urfave/cli-docs/v3 v3.0.0-alpha6

go.sum

@@ -105,8 +105,8 @@ github.com/muesli/reflow v0.3.0 h1:IFsN6K9NfGtjeggFP+68I4chLZV2yIKsXJFNZ+eWh6s=
 github.com/muesli/reflow v0.3.0/go.mod h1:pbwTDkVPibjO2kyvBQRBxTWEEGDGq0FlB1BIKtnHY/8=
 github.com/muesli/termenv v0.16.0 h1:S5AlUN9dENB57rsbnkPyfdGuWIlkmzJjbFf0Tf5FWUc=
 github.com/muesli/termenv v0.16.0/go.mod h1:ZRfOIKPFDYQoDFF4Olj7/QJbW60Ol/kL1pU3VfY/Cnk=
-github.com/onkernel/hypeman-go v0.7.1-0.20251223013311-b97d5b836009 h1:LPFxYVNAEv4reF4ZFzd45rLPLRWr60NPaPIb0Z622X0=
-github.com/onkernel/hypeman-go v0.7.1-0.20251223013311-b97d5b836009/go.mod h1:Wtm4ewVGGPZc2ySeeuQISQyJxujyQuyDjXyksVkIyy8=
+github.com/onkernel/hypeman-go v0.7.1-0.20251223015448-3fc21a9216cf h1:vGNpWu/QZcBXqzeRx1qV/jN8JPGqveWn/xsqbrlQv1A=
+github.com/onkernel/hypeman-go v0.7.1-0.20251223015448-3fc21a9216cf/go.mod h1:Wtm4ewVGGPZc2ySeeuQISQyJxujyQuyDjXyksVkIyy8=
 github.com/opencontainers/go-digest v1.0.0 h1:apOUWs51W5PlhuyGyz9FCeeBIOUDA/6nW8Oi/yOhh5U=
 github.com/opencontainers/go-digest v1.0.0/go.mod h1:0JzlMkj0TRzQZfJkVvzbP0HBR3IKzErnv2BNG4W4MAM=
 github.com/opencontainers/image-spec v1.1.1 h1:y0fUlFfIZhPF1W537XOLg0/fcx6zcHCJwooC2xJA040=

pkg/cmd/cp.go

@@ -10,6 +10,7 @@ import (
 	"net/http"
 	"net/url"
 	"os"
+	"path"
 	"path/filepath"
 	"strings"
 	"time"
@@ -342,9 +343,12 @@ func resolveDestPath(ctx context.Context, baseURL, apiKey, instanceID, srcPath,
 	}
 
 	// Check if dstPath ends with /. (copy contents only)
-	copyContentsOnly := strings.HasSuffix(srcPath, string(filepath.Separator)+".")
+	// Handle both OS-specific separator and forward slash for cross-platform compatibility
+	copyContentsOnly := strings.HasSuffix(srcPath, string(filepath.Separator)+".") ||
+		strings.HasSuffix(srcPath, "/.")
 	if copyContentsOnly {
 		srcPath = strings.TrimSuffix(srcPath, string(filepath.Separator)+".")
+		srcPath = strings.TrimSuffix(srcPath, "/.")
 	}
 
 	// Check if destination ends with /
@@ -380,7 +384,8 @@ func resolveDestPath(ctx context.Context, baseURL, apiKey, instanceID, srcPath,
 		}
 		if dstStat.IsDir {
 			// Copy into directory using basename
-			return filepath.Join(dstPath, filepath.Base(srcPath)), nil
+			// Use path.Join for guest paths (always forward slashes)
+			return path.Join(dstPath, filepath.Base(srcPath)), nil
 		}
 		// Overwrite file
 		return dstPath, nil
@@ -403,7 +408,8 @@ func resolveDestPath(ctx context.Context, baseURL, apiKey, instanceID, srcPath,
 	}
 
 	// Copy SRC dir into DEST (create subdir)
-	return filepath.Join(dstPath, filepath.Base(srcPath)), nil
+	// Use path.Join for guest paths (always forward slashes)
+	return path.Join(dstPath, filepath.Base(srcPath)), nil
 }
 
 // buildCpWsURL builds the WebSocket URL for the cp endpoint
@@ -549,7 +555,8 @@ func copyDirContentsToInstance(ctx context.Context, baseURL, apiKey, instanceID,
 
 	for _, entry := range entries {
 		srcEntryPath := filepath.Join(srcPath, entry.Name())
-		dstEntryPath := filepath.Join(dstPath, entry.Name())
+		// Use path.Join for guest paths (always forward slashes)
+		dstEntryPath := path.Join(dstPath, entry.Name())
 
 		info, err := entry.Info()
 		if err != nil {