feat: sort all sdks

.gitignore

@@ -4,3 +4,4 @@
 
 
 coverage.out
+**/available.cache

cmd/commands/current.go

@@ -42,14 +42,17 @@ func currentCmd(ctx *cli.Context) error {
 		if err != nil {
 			return err
 		}
-		for name, s := range allSdk {
+
+		for _, s := range allSdk {
+			name := s.Name
 			current := s.Current()
 			if current == "" {
 				pterm.Printf("%s -> N/A \n", name)
 			} else {
 				pterm.Printf("%s -> %s\n", name, pterm.LightGreen("v"+string(current)))
 			}
 		}
+
 		return nil
 	}
 	source, err := manager.LookupSdk(sdkName)

cmd/commands/list.go

@@ -46,7 +46,9 @@ func listCmd(ctx *cli.Context) error {
 			return fmt.Errorf("you don't have any sdk installed yet")
 		}
 		tree := pterm.LeveledList{}
-		for name, s := range allSdk {
+
+		for _, s := range allSdk {
+			name := s.Name
 			tree = append(tree, pterm.LeveledListItem{Level: 0, Text: name})
 			for _, version := range s.List() {
 				tree = append(tree, pterm.LeveledListItem{Level: 1, Text: "v" + string(version)})

cmd/commands/update.go

@@ -49,11 +49,12 @@ func updateCmd(ctx *cli.Context) error {
 				index int
 				total = len(sdks)
 			)
-			for sdk := range sdks {
+			for _, s := range sdks {
+				sdkName := s.Name
 				index++
-				pterm.Printf("[%s/%d]: Updating %s plugin...\n", pterm.Green(index), total, pterm.Green(sdk))
-				if err = manager.Update(sdk); err != nil {
-					pterm.Println(fmt.Sprintf("Update plugin(%s) failed, %s", sdk, err.Error()))
+				pterm.Printf("[%s/%d]: Updating %s plugin...\n", pterm.Green(index), total, pterm.Green(sdkName))
+				if err = manager.Update(sdkName); err != nil {
+					pterm.Println(fmt.Sprintf("Update plugin(%s) failed, %s", sdkName, err.Error()))
 				}
 			}
 		} else {

internal/manager.go

@@ -26,6 +26,7 @@ import (
 	"net/url"
 	"os"
 	"path/filepath"
+	"sort"
 	"strconv"
 	"strings"
 
@@ -109,7 +110,7 @@ func (m *Manager) LookupSdk(name string) (*Sdk, error) {
 			return nil, fmt.Errorf("failed to migrate an old plug-in: %w", err)
 		}
 	}
-	sdk, err := NewSdk(m, pluginPath)
+	sdk, err := NewSdk(m, pluginPath, strings.ToLower(name))
 	if err != nil {
 		return nil, err
 	}
@@ -148,12 +149,12 @@ func (m *Manager) LookupSdkWithInstall(name string) (*Sdk, error) {
 	}
 }
 
-func (m *Manager) LoadAllSdk() (map[string]*Sdk, error) {
+func (m *Manager) LoadAllSdk() ([]*Sdk, error) {
 	dir, err := os.ReadDir(m.PathMeta.PluginPath)
 	if err != nil {
 		return nil, fmt.Errorf("load sdks error: %w", err)
 	}
-	sdkMap := make(map[string]*Sdk)
+	sdkSlice := make([]*Sdk, 0)
 	for _, d := range dir {
 		sdkName := d.Name()
 		path := filepath.Join(m.PathMeta.PluginPath, sdkName)
@@ -175,11 +176,15 @@ func (m *Manager) LoadAllSdk() (map[string]*Sdk, error) {
 		} else {
 			continue
 		}
-		sdk, _ := NewSdk(m, path)
-		sdkMap[strings.ToLower(sdkName)] = sdk
+		sdk, _ := NewSdk(m, path, strings.ToLower(sdkName))
+		sdkSlice = append(sdkSlice, sdk)
 		m.openSdks[strings.ToLower(sdkName)] = sdk
 	}
-	return sdkMap, nil
+	// sort by name
+	sort.Slice(sdkSlice, func(i, j int) bool {
+		return sdkSlice[j].Name > sdkSlice[i].Name
+	})
+	return sdkSlice, nil
 }
 
 func (m *Manager) Close() {

internal/plugin_test.go

@@ -372,4 +372,26 @@ func testHookFunc(t *testing.T, factory func() (*Manager, *LuaPlugin, error)) {
 			t.Fatal(err)
 		}
 	})
+
+	t.Run("LoadAllSdk", func(t *testing.T) {
+		s, _, err := factory()
+		if err != nil {
+			t.Fatal(err)
+		}
+
+		firstSdkName := ""
+		for i := 0; i < 10; i++ {
+			sdks, err := s.LoadAllSdk()
+			if err != nil {
+				t.Fatal(err)
+			}
+			if len(sdks) != 0 && firstSdkName == "" {
+				firstSdkName = sdks[0].Name
+			} else if len(sdks) != 0 {
+				if sdks[0].Name != firstSdkName {
+					t.Errorf("expected sdk sort %v", sdks)
+				}
+			}
+		}
+	})
 }

internal/sdk.go

@@ -73,6 +73,7 @@ type Sdk struct {
 	Plugin     *LuaPlugin
 	// current sdk install path
 	InstallPath string
+	Name        string
 }
 
 func (b *Sdk) Install(version Version) error {
@@ -754,12 +755,13 @@ func (b *Sdk) ClearCurrentEnv() error {
 }
 
 // NewSdk creates a new SDK instance.
-func NewSdk(manager *Manager, pluginPath string) (*Sdk, error) {
+func NewSdk(manager *Manager, pluginPath string, name string) (*Sdk, error) {
 	luaPlugin, err := NewLuaPlugin(pluginPath, manager)
 	if err != nil {
 		return nil, fmt.Errorf("failed to create lua plugin: %w", err)
 	}
 	return &Sdk{
+		Name:        name,
 		sdkManager:  manager,
 		InstallPath: filepath.Join(manager.PathMeta.SdkCachePath, strings.ToLower(luaPlugin.SdkName)),
 		Plugin:      luaPlugin,