Fix install list command to show installable versions from remote (#531)

Author: ChronosMasterOfAllTime

cmd/core/install.go

@@ -117,6 +117,8 @@ func runInstall(cmd *cobra.Command, args []string) error {
 
 	// Handle --list flag
 	if installFlags.list {
+		// Set remote flag so runList shows available versions, not installed ones
+		listFlags.remote = true
 		return runList(cmd, args)
 	}
 

cmd/core/list.go

@@ -3,6 +3,7 @@ package core
 import (
 	"encoding/json"
 	"fmt"
+
 	"github.com/go-nv/goenv/cmd/legacy"
 
 	cmdpkg "github.com/go-nv/goenv/cmd"
@@ -72,6 +73,7 @@ func runListInstalled(cmd *cobra.Command) error {
 	legacy.VersionsFlags.Bare = listFlags.bare
 	legacy.VersionsFlags.SkipAliases = listFlags.skipAliases
 	legacy.VersionsFlags.Json = listFlags.json
+	legacy.VersionsFlags.SuppressDeprecationWarning = true // Don't show warning when called from 'list'
 
 	// Reuse the versions command implementation
 	return legacy.RunVersions(cmd, []string{})

cmd/legacy/versions.go

@@ -51,12 +51,13 @@ Examples:
 }
 
 var VersionsFlags struct {
-	Bare        bool
-	SkipAliases bool
-	Complete    bool
-	Json        bool
-	Used        bool
-	Depth       int
+	Bare                       bool
+	SkipAliases                bool
+	Complete                   bool
+	Json                       bool
+	Used                       bool
+	Depth                      int
+	SuppressDeprecationWarning bool // Internal flag to suppress warning when called from 'goenv list'
 }
 
 func init() {
@@ -116,10 +117,12 @@ func RunVersions(cmd *cobra.Command, args []string) error {
 		return nil
 	}
 
-	// Deprecation warning
-	fmt.Fprintf(cmd.OutOrStderr(), "%sDeprecation warning: 'goenv versions' is a legacy command. Use 'goenv list' instead.\n", utils.Emoji("⚠️  "))
-	fmt.Fprintf(cmd.OutOrStderr(), "  Modern command: goenv list\n")
-	fmt.Fprintf(cmd.OutOrStderr(), "  See: goenv help list\n\n")
+	// Deprecation warning (suppressed when called internally from 'goenv list')
+	if !VersionsFlags.SuppressDeprecationWarning {
+		fmt.Fprintf(cmd.OutOrStderr(), "%sDeprecation warning: 'goenv versions' is a legacy command. Use 'goenv list' instead.\n", utils.Emoji("⚠️  "))
+		fmt.Fprintf(cmd.OutOrStderr(), "  Modern command: goenv list\n")
+		fmt.Fprintf(cmd.OutOrStderr(), "  See: goenv help list\n\n")
+	}
 
 	// Handle invalid arguments (BATS test expects usage error)
 	if err := cmdutil.ValidateMaxArgs(args, 0, "no arguments"); err != nil {