controllers/krate/metadata: Sort versions by id

This changes the sorting of `crate.versions`. Previously, we sorted
versions by semver, which is more computationally expensive compared to
sorting by id/date, which can be achieved more easily directly from
the database. While this change alters the sorting order, it still
provides consistent output and eliminates the need for double semver
sorting.

Author: eth3lbert

src/controllers/krate/metadata.rs

@@ -18,7 +18,6 @@ use axum_extra::json;
 use axum_extra::response::ErasedJson;
 use diesel::prelude::*;
 use diesel_async::RunQueryDsl;
-use std::cmp::Reverse;
 use std::str::FromStr;
 
 #[derive(Debug, Deserialize, FromRequestParts, utoipa::IntoParams)]
@@ -89,15 +88,12 @@ pub async fn find_crate(
             .ok_or_else(|| crate_not_found(&path.name))?;
 
     let versions_publishers_and_audit_actions = if include.versions {
-        let mut versions_and_publishers: Vec<(Version, Option<User>)> =
-            Version::belonging_to(&krate)
-                .left_outer_join(users::table)
-                .select(<(Version, Option<User>)>::as_select())
-                .load(&mut conn)
-                .await?;
-
-        versions_and_publishers
-            .sort_by_cached_key(|(version, _)| Reverse(semver::Version::parse(&version.num).ok()));
+        let versions_and_publishers: Vec<(Version, Option<User>)> = Version::belonging_to(&krate)
+            .left_outer_join(users::table)
+            .select(<(Version, Option<User>)>::as_select())
+            .order_by(versions::id)
+            .load(&mut conn)
+            .await?;
 
         let versions = versions_and_publishers
             .iter()

src/tests/routes/crates/snapshots/crates_io__tests__routes__crates__read__show.snap

@@ -36,8 +36,8 @@ snapshot_kind: text
     "updated_at": "[datetime]",
     "versions": [
       1,
-      3,
-      2
+      2,
+      3
     ],
     "yanked": false
   },
@@ -90,30 +90,30 @@ snapshot_kind: text
       "crate_size": 0,
       "created_at": "[datetime]",
       "description": null,
-      "dl_path": "/api/v1/crates/foo_show/0.5.1/download",
+      "dl_path": "/api/v1/crates/foo_show/0.5.0/download",
       "documentation": null,
       "downloads": 0,
       "edition": null,
       "features": {},
       "has_lib": null,
       "homepage": null,
-      "id": 3,
+      "id": 2,
       "lib_links": null,
       "license": null,
       "links": {
-        "authors": "/api/v1/crates/foo_show/0.5.1/authors",
-        "dependencies": "/api/v1/crates/foo_show/0.5.1/dependencies",
-        "version_downloads": "/api/v1/crates/foo_show/0.5.1/downloads"
+        "authors": "/api/v1/crates/foo_show/0.5.0/authors",
+        "dependencies": "/api/v1/crates/foo_show/0.5.0/dependencies",
+        "version_downloads": "/api/v1/crates/foo_show/0.5.0/downloads"
       },
-      "num": "0.5.1",
+      "num": "0.5.0",
       "published_by": {
         "avatar": null,
         "id": 1,
         "login": "foo",
         "name": null,
         "url": "https://github.com/foo"
       },
-      "readme_path": "/api/v1/crates/foo_show/0.5.1/readme",
+      "readme_path": "/api/v1/crates/foo_show/0.5.0/readme",
       "repository": null,
       "rust_version": null,
       "updated_at": "[datetime]",
@@ -128,30 +128,30 @@ snapshot_kind: text
       "crate_size": 0,
       "created_at": "[datetime]",
       "description": null,
-      "dl_path": "/api/v1/crates/foo_show/0.5.0/download",
+      "dl_path": "/api/v1/crates/foo_show/0.5.1/download",
       "documentation": null,
       "downloads": 0,
       "edition": null,
       "features": {},
       "has_lib": null,
       "homepage": null,
-      "id": 2,
+      "id": 3,
       "lib_links": null,
       "license": null,
       "links": {
-        "authors": "/api/v1/crates/foo_show/0.5.0/authors",
-        "dependencies": "/api/v1/crates/foo_show/0.5.0/dependencies",
-        "version_downloads": "/api/v1/crates/foo_show/0.5.0/downloads"
+        "authors": "/api/v1/crates/foo_show/0.5.1/authors",
+        "dependencies": "/api/v1/crates/foo_show/0.5.1/dependencies",
+        "version_downloads": "/api/v1/crates/foo_show/0.5.1/downloads"
       },
-      "num": "0.5.0",
+      "num": "0.5.1",
       "published_by": {
         "avatar": null,
         "id": 1,
         "login": "foo",
         "name": null,
         "url": "https://github.com/foo"
       },
-      "readme_path": "/api/v1/crates/foo_show/0.5.0/readme",
+      "readme_path": "/api/v1/crates/foo_show/0.5.1/readme",
       "repository": null,
       "rust_version": null,
       "updated_at": "[datetime]",