fixed long release names colliding with release dates in verbose downloads

Author: cqb13

commands/downloads.go

@@ -9,9 +9,10 @@ import (
 )
 
 type release struct {
-	Name        string `json:"name"`
-	PublishedAt string `json:"published_at"`
-	Assets      []struct {
+	Name           string `json:"name"`
+	PublishedAt    string `json:"published_at"`
+	TotalDownloads int
+	Assets         []struct {
 		Downloads int `json:"download_count"`
 	} `json:"assets"`
 }
@@ -23,6 +24,9 @@ func HandleDownloadsCommand(user string, repo string, verbose bool) {
 	releaseCount := 0
 	page := 1
 
+	releases := make([]release, 0)
+	longestName := 0
+
 	for {
 		resp, err := utils.Get(fmt.Sprintf("%s%d", baseUrl, page))
 		if err != nil {
@@ -52,19 +56,20 @@ func HandleDownloadsCommand(user string, repo string, verbose bool) {
 			for _, asset := range release.Assets {
 				releaseAssetDownloads += asset.Downloads
 			}
+
 			if verbose {
-				publishedAt, err := utils.RFC3339StrToPrettyStr(release.PublishedAt)
-				if err != nil {
-					fmt.Println(err)
-					return
+				if len(release.Name) > longestName {
+					longestName = len(release.Name)
 				}
-				fmt.Printf("%s%-10d%s%-20s%s\n", ansi.Bold, releaseAssetDownloads, ansi.Reset, release.Name, publishedAt)
+
+				release.TotalDownloads = releaseAssetDownloads
+				releases = append(releases, release)
 			}
+
 			downloadCount += releaseAssetDownloads
 		}
 
 		releaseCount += len(releaseList)
-
 		page++
 	}
 
@@ -73,5 +78,18 @@ func HandleDownloadsCommand(user string, repo string, verbose bool) {
 		return
 	}
 
+	if verbose {
+		for _, release := range releases {
+			publishedAt, err := utils.RFC3339StrToPrettyStr(release.PublishedAt)
+
+			if err != nil {
+				fmt.Println(err)
+				continue
+			}
+
+			fmt.Printf("%s%-10d%s%-*s%s\n", ansi.Bold, release.TotalDownloads, ansi.Reset, longestName+5, release.Name, publishedAt)
+		}
+	}
+
 	fmt.Printf("%s/%s has %s%d%s downloads, across %s%d%s releases\n", user, repo, ansi.Bold, downloadCount, ansi.Reset, ansi.Bold, releaseCount, ansi.Reset)
 }

utils/cli.go

@@ -29,8 +29,8 @@ func (fs *FlagSet) AddBoolFlag(name string, defaultvalue bool) {
 
 func (fs *FlagSet) Parse(args []string) {
 	for _, arg := range args {
-		if strings.HasPrefix(arg, "--") {
-			name := strings.TrimPrefix(arg, "--")
+		if after, ok := strings.CutPrefix(arg, "--"); ok {
+			name := after
 			if flag, exists := fs.boolflags[name]; exists {
 				flag.found = true
 			}