fix(cli): drop duplicate vendor steps and reuse ValidArch

ResolveForVendor already logs acquisition progress to stderr; remove
redundant StepStart/StepDone pairs. Use binary.ValidArch in sandboxArch.

Signed-off-by: Barak Korren <bkorren@redhat.com>
Co-authored-by: Cursor <cursoragent@cursor.com>

Author: ifireball

internal/cli/run.go

@@ -1601,8 +1601,6 @@ func needsCrossCompilation() bool {
 	return runtime.GOOS != "linux"
 }
 
-var validArchs = map[string]bool{"amd64": true, "arm64": true}
-
 // copyFile copies src to dst, preserving permissions.
 func copyFile(src, dst string) error {
 	in, err := os.Open(src)
@@ -1635,7 +1633,7 @@ func copyFile(src, dst string) error {
 // on an arm64 host via emulation). Only amd64 and arm64 are supported.
 func sandboxArch() string {
 	if arch := os.Getenv("FULLSEND_SANDBOX_ARCH"); arch != "" {
-		if !validArchs[arch] {
+		if !binary.ValidArch(arch) {
 			fmt.Fprintf(os.Stderr, "WARNING: FULLSEND_SANDBOX_ARCH=%q is not a supported architecture (amd64, arm64), using host arch %s\n", arch, runtime.GOARCH)
 			return runtime.GOARCH
 		}

internal/cli/vendor.go

@@ -60,15 +60,6 @@ func acquireAndVendorFullsendBinary(ctx context.Context, client forge.Client, pr
 		tmpDir = result.TmpDir
 		binPath = result.Path
 		source = result.Source
-
-		switch source {
-		case binary.SourceCheckoutBuild:
-			printer.StepStart("Cross-compiling fullsend for linux/amd64")
-			printer.StepDone("Cross-compiled fullsend for linux/amd64")
-		case binary.SourceReleaseDownload:
-			printer.StepStart(fmt.Sprintf("Downloading fullsend %s for linux/amd64 from GitHub Release", version))
-			printer.StepDone(fmt.Sprintf("Downloaded fullsend %s for linux/amd64", version))
-		}
 	}
 
 	if tmpDir != "" {