From d3b4821047376c508ae7cc880ebc1d9db9bcc271 Mon Sep 17 00:00:00 2001
From: MatteoGabriele <m.gabriele.dev@gmail.com>
Date: Mon, 26 Jan 2026 19:32:18 +0100
Subject: [PATCH 1/2] fix: semver sorting

uses the built-in compare method from the semver package.
---
 app/utils/versions.ts | 16 ++--------------
 1 file changed, 2 insertions(+), 14 deletions(-)

diff --git a/app/utils/versions.ts b/app/utils/versions.ts
index d30fbbdb46..f2d6e5008b 100644
--- a/app/utils/versions.ts
+++ b/app/utils/versions.ts
@@ -1,4 +1,4 @@
-import { valid } from 'semver'
+import { compare, valid } from 'semver'
 
 /**
  * Utilities for handling npm package versions and dist-tags
@@ -47,19 +47,7 @@ export function parseVersion(version: string): ParsedVersion {
  * @returns Comparison result for sorting
  */
 export function compareVersions(a: string, b: string): number {
-  const va = parseVersion(a)
-  const vb = parseVersion(b)
-
-  if (va.major !== vb.major) return va.major - vb.major
-  if (va.minor !== vb.minor) return va.minor - vb.minor
-  if (va.patch !== vb.patch) return va.patch - vb.patch
-
-  // Stable versions (no prerelease) are greater than prereleases
-  if (va.prerelease && vb.prerelease) return va.prerelease.localeCompare(vb.prerelease)
-  if (va.prerelease) return -1
-  if (vb.prerelease) return 1
-
-  return 0
+  return compare(a, b)
 }
 
 /**

From 5047fe416d4fdd47713b2918daed400f4176db75 Mon Sep 17 00:00:00 2001
From: MatteoGabriele <m.gabriele.dev@gmail.com>
Date: Mon, 26 Jan 2026 20:29:09 +0100
Subject: [PATCH 2/2] refactor: use semver compare directly

---
 app/components/PackageVersions.vue |  8 +++---
 app/composables/useNpmRegistry.ts  |  6 ++---
 app/utils/versions.ts              | 13 +---------
 test/unit/versions.spec.ts         | 39 ------------------------------
 4 files changed, 8 insertions(+), 58 deletions(-)

diff --git a/app/components/PackageVersions.vue b/app/components/PackageVersions.vue
index a83aa3f8a8..af6c8a3e96 100644
--- a/app/components/PackageVersions.vue
+++ b/app/components/PackageVersions.vue
@@ -1,9 +1,9 @@
 <script setup lang="ts">
 import type { PackumentVersion, PackageVersionInfo } from '#shared/types'
 import type { RouteLocationRaw } from 'vue-router'
+import { compare } from 'semver'
 import {
   buildVersionToTagsMap,
-  compareVersions,
   filterExcludedTags,
   getPrereleaseChannel,
   parseVersion,
@@ -90,7 +90,7 @@ const allTagRows = computed(() => {
         deprecated: versionData?.deprecated,
       } as VersionDisplay,
     }))
-    .sort((a, b) => compareVersions(b.primaryVersion.version, a.primaryVersion.version))
+    .sort((a, b) => compare(b.primaryVersion.version, a.primaryVersion.version))
 })
 
 // Check if the whole package is deprecated (latest version is deprecated)
@@ -177,7 +177,7 @@ function processLoadedVersions(allVersions: PackageVersionInfo[]) {
         const vChannel = getPrereleaseChannel(v.version)
         return vParsed.major === tagParsed.major && vChannel === tagChannel
       })
-      .sort((a, b) => compareVersions(b.version, a.version))
+      .sort((a, b) => compare(b.version, a.version))
       .map(v => ({
         version: v.version,
         time: v.time,
@@ -214,7 +214,7 @@ function processLoadedVersions(allVersions: PackageVersionInfo[]) {
 
   // Sort within each major
   for (const versions of byMajor.values()) {
-    versions.sort((a, b) => compareVersions(b.version, a.version))
+    versions.sort((a, b) => compare(b.version, a.version))
   }
 
   // Build major groups sorted by major descending
diff --git a/app/composables/useNpmRegistry.ts b/app/composables/useNpmRegistry.ts
index aff41fe0a8..cf4adfe7aa 100644
--- a/app/composables/useNpmRegistry.ts
+++ b/app/composables/useNpmRegistry.ts
@@ -9,8 +9,8 @@ import type {
   PackageVersionInfo,
 } from '#shared/types'
 import type { ReleaseType } from 'semver'
-import { maxSatisfying, prerelease, major, minor, diff, gt } from 'semver'
-import { compareVersions, isExactVersion } from '~/utils/versions'
+import { maxSatisfying, prerelease, major, minor, diff, gt, compare } from 'semver'
+import { isExactVersion } from '~/utils/versions'
 import { extractInstallScriptsInfo } from '~/utils/install-scripts'
 
 const NPM_REGISTRY = 'https://registry.npmjs.org'
@@ -434,7 +434,7 @@ export async function fetchAllPackageVersions(packageName: string): Promise<Pack
         hasProvenance: false, // Would need to check dist.attestations for each version
         deprecated: versionData.deprecated,
       }))
-      .sort((a, b) => compareVersions(b.version, a.version))
+      .sort((a, b) => compare(b.version, a.version))
   })()
 
   allVersionsCache.set(packageName, promise)
diff --git a/app/utils/versions.ts b/app/utils/versions.ts
index f2d6e5008b..1b0f958789 100644
--- a/app/utils/versions.ts
+++ b/app/utils/versions.ts
@@ -39,17 +39,6 @@ export function parseVersion(version: string): ParsedVersion {
   }
 }
 
-/**
- * Compare two semver versions for sorting
- * Returns positive if a > b, negative if a < b, 0 if equal
- * @param a - First version string
- * @param b - Second version string
- * @returns Comparison result for sorting
- */
-export function compareVersions(a: string, b: string): number {
-  return compare(a, b)
-}
-
 /**
  * Extract the prerelease channel from a version string
  * @param version - The version string (e.g., "1.0.0-beta.1")
@@ -133,7 +122,7 @@ export function buildTaggedVersionRows(distTags: Record<string, string>): Tagged
       tags,
       version,
     }))
-    .sort((a, b) => compareVersions(b.version, a.version))
+    .sort((a, b) => compare(b.version, a.version))
 }
 
 /**
diff --git a/test/unit/versions.spec.ts b/test/unit/versions.spec.ts
index 5a826e8b57..0b0cd5a298 100644
--- a/test/unit/versions.spec.ts
+++ b/test/unit/versions.spec.ts
@@ -2,7 +2,6 @@ import { describe, expect, it } from 'vitest'
 import {
   buildTaggedVersionRows,
   buildVersionToTagsMap,
-  compareVersions,
   filterExcludedTags,
   getPrereleaseChannel,
   parseVersion,
@@ -58,44 +57,6 @@ describe('parseVersion', () => {
   })
 })
 
-describe('compareVersions', () => {
-  it('compares major versions', () => {
-    expect(compareVersions('2.0.0', '1.0.0')).toBeGreaterThan(0)
-    expect(compareVersions('1.0.0', '2.0.0')).toBeLessThan(0)
-  })
-
-  it('compares minor versions', () => {
-    expect(compareVersions('1.2.0', '1.1.0')).toBeGreaterThan(0)
-    expect(compareVersions('1.1.0', '1.2.0')).toBeLessThan(0)
-  })
-
-  it('compares patch versions', () => {
-    expect(compareVersions('1.0.2', '1.0.1')).toBeGreaterThan(0)
-    expect(compareVersions('1.0.1', '1.0.2')).toBeLessThan(0)
-  })
-
-  it('ranks stable above prerelease', () => {
-    expect(compareVersions('1.0.0', '1.0.0-beta.1')).toBeGreaterThan(0)
-    expect(compareVersions('1.0.0-beta.1', '1.0.0')).toBeLessThan(0)
-  })
-
-  it('compares prereleases alphabetically', () => {
-    expect(compareVersions('1.0.0-beta.1', '1.0.0-alpha.1')).toBeGreaterThan(0)
-    expect(compareVersions('1.0.0-alpha.1', '1.0.0-beta.1')).toBeLessThan(0)
-  })
-
-  it('returns 0 for equal versions', () => {
-    expect(compareVersions('1.0.0', '1.0.0')).toBe(0)
-    expect(compareVersions('1.0.0-beta.1', '1.0.0-beta.1')).toBe(0)
-  })
-
-  it('sorts Nuxt versions correctly', () => {
-    const versions = ['3.21.0', '4.0.0-alpha.4', '4.0.0-rc.0', '4.3.0']
-    const sorted = [...versions].sort((a, b) => compareVersions(b, a))
-    expect(sorted).toEqual(['4.3.0', '4.0.0-rc.0', '4.0.0-alpha.4', '3.21.0'])
-  })
-})
-
 describe('getPrereleaseChannel', () => {
   it('returns empty string for stable versions', () => {
     expect(getPrereleaseChannel('1.0.0')).toBe('')