fix: version parsing when -rc is in curVersion (#991)

* fix: version parsing when -rc is in curVersion

* chore: preserve original version string for debug prints

Author: vinovo

runner/cmd/nexa-cli/version.go

@@ -46,18 +46,24 @@ func version() *cobra.Command {
 // Returns: -1 if v1 < v2, 0 if v1 == v2, 1 if v1 > v2
 func compareVersion(v1, v2 string) (int, error) {
 	parseVersion := func(v string) ([3]int, error) {
-		parts := strings.Split(strings.TrimPrefix(v, "v"), ".")
+		origV := v
+		v = strings.TrimPrefix(v, "v")
+		// Strip pre-release suffixes like -rc2, -beta, -alpha, etc.
+		if idx := strings.IndexAny(v, "-+"); idx != -1 {
+			v = v[:idx]
+		}
+		parts := strings.Split(v, ".")
 		if len(parts) != 3 {
-			return [3]int{}, fmt.Errorf("invalid format: %s", v)
+			return [3]int{}, fmt.Errorf("invalid format: %s", origV)
 		}
 		var nums [3]int
 		for i, p := range parts {
 			n, err := strconv.Atoi(p)
 			if err != nil {
-				return [3]int{}, fmt.Errorf("invalid format: %s", v)
+				return [3]int{}, fmt.Errorf("invalid format: %s", origV)
 			}
 			if n < 0 {
-				return [3]int{}, fmt.Errorf("invalid format: %s", v)
+				return [3]int{}, fmt.Errorf("invalid format: %s", origV)
 			}
 			nums[i] = n
 		}

runner/cmd/nexa-cli/version_test.go

@@ -160,6 +160,64 @@ func TestCompareVersion(t *testing.T) {
 			wantErr:  false,
 		},
 
+		// Pre-release versions (should compare base version only)
+		{
+			name:     "rc version equals stable",
+			v1:       "v0.2.68-rc2",
+			v2:       "v0.2.68",
+			expected: 0,
+			wantErr:  false,
+		},
+		{
+			name:     "rc version greater than older stable",
+			v1:       "v0.2.68-rc2",
+			v2:       "v0.2.67",
+			expected: 1,
+			wantErr:  false,
+		},
+		{
+			name:     "rc version less than newer stable",
+			v1:       "v0.2.68-rc2",
+			v2:       "v0.2.69",
+			expected: -1,
+			wantErr:  false,
+		},
+		{
+			name:     "beta version comparison",
+			v1:       "v1.0.0-beta",
+			v2:       "v1.0.0",
+			expected: 0,
+			wantErr:  false,
+		},
+		{
+			name:     "alpha version comparison",
+			v1:       "v2.5.0-alpha.1",
+			v2:       "v2.4.9",
+			expected: 1,
+			wantErr:  false,
+		},
+		{
+			name:     "build metadata with plus",
+			v1:       "v1.0.0+build123",
+			v2:       "v1.0.0",
+			expected: 0,
+			wantErr:  false,
+		},
+		{
+			name:     "complex prerelease version",
+			v1:       "v3.2.1-rc1+build456",
+			v2:       "v3.2.1",
+			expected: 0,
+			wantErr:  false,
+		},
+		{
+			name:     "two rc versions",
+			v1:       "v0.2.68-rc1",
+			v2:       "v0.2.68-rc2",
+			expected: 0,
+			wantErr:  false,
+		},
+
 		// Invalid format cases
 		{
 			name:     "invalid format - missing parts",