Integrate linecount analysis in publish controller

This modifies the publish endpoint to extract and store linecount statistics by extracting linecount data from tarball processing results and serializing the stats to `JSON` for database storage. The linecount data is then passed to the `NewVersion` builder for persistence.

All publish-related test snapshots are updated to include linecount data, demonstrating that the integration works correctly across various publish scenarios. The implementation maintains backward compatibility with null linecount values for any edge cases.

Author: Turbo87

src/controllers/krate/publish.rs

@@ -26,7 +26,7 @@ use sha2::{Digest, Sha256};
 use std::collections::HashMap;
 use tokio::io::{AsyncRead, AsyncReadExt};
 use tokio_util::io::StreamReader;
-use tracing::{error, instrument};
+use tracing::{error, instrument, warn};
 use url::Url;
 
 use crate::models::{
@@ -482,6 +482,10 @@ pub async fn publish(app: AppState, req: Parts, body: Body) -> AppResult<Json<Go
         // Read tarball from request
         let hex_cksum: String = Sha256::digest(&tarball_bytes).encode_hex();
 
+        let linecounts = serde_json::to_value(tarball_info.linecount_stats)
+            .inspect_err(|err| warn!("Failed to convert linecounts to serde_json::Value: {err}"))
+            .ok();
+
         // Persist the new version of this crate
         let new_version = NewVersion::builder(krate.id, &version_string)
             .features(serde_json::to_value(&features)?)
@@ -502,6 +506,7 @@ pub async fn publish(app: AppState, req: Parts, body: Body) -> AppResult<Json<Go
             .maybe_repository(repository.as_deref())
             .categories(&categories)
             .keywords(&keywords)
+            .maybe_linecounts(linecounts)
             .build();
 
         let version = new_version.save(conn).await.map_err(|error| {

src/tests/krate/publish/snapshots/crates_io__tests__krate__publish__edition__edition_is_saved-4.snap

@@ -33,6 +33,11 @@ expression: response.json()
     "id": "[id]",
     "lib_links": null,
     "license": "MIT",
+    "linecounts": {
+      "languages": {},
+      "total_code_lines": 0,
+      "total_comment_lines": 0
+    },
     "links": {
       "authors": "/api/v1/crates/foo/1.0.0/authors",
       "dependencies": "/api/v1/crates/foo/1.0.0/dependencies",

src/tests/krate/publish/snapshots/crates_io__tests__krate__publish__links__crate_with_links_field-3.snap

@@ -33,6 +33,11 @@ expression: response.json()
     "id": "[id]",
     "lib_links": "git2",
     "license": "MIT",
+    "linecounts": {
+      "languages": {},
+      "total_code_lines": 0,
+      "total_comment_lines": 0
+    },
     "links": {
       "authors": "/api/v1/crates/foo/1.0.0/authors",
       "dependencies": "/api/v1/crates/foo/1.0.0/dependencies",

src/tests/krate/publish/snapshots/crates_io__tests__krate__publish__manifest__boolean_readme-4.snap

@@ -33,6 +33,11 @@ expression: response.json()
     "id": "[id]",
     "lib_links": null,
     "license": "MIT",
+    "linecounts": {
+      "languages": {},
+      "total_code_lines": 0,
+      "total_comment_lines": 0
+    },
     "links": {
       "authors": "/api/v1/crates/foo/1.0.0/authors",
       "dependencies": "/api/v1/crates/foo/1.0.0/dependencies",

src/tests/krate/publish/snapshots/crates_io__tests__krate__publish__manifest__lib_and_bin_crate-4.snap

@@ -36,6 +36,17 @@ expression: response.json()
     "id": "[id]",
     "lib_links": null,
     "license": "MIT",
+    "linecounts": {
+      "languages": {
+        "Rust": {
+          "code_lines": 3,
+          "comment_lines": 0,
+          "files": 3
+        }
+      },
+      "total_code_lines": 3,
+      "total_comment_lines": 0
+    },
     "links": {
       "authors": "/api/v1/crates/foo/1.0.0/authors",
       "dependencies": "/api/v1/crates/foo/1.0.0/dependencies",

src/tests/krate/publish/snapshots/crates_io__tests__krate__publish__trustpub__full_flow-9.snap

@@ -21,6 +21,11 @@ expression: response.json()
     "id": 2,
     "lib_links": null,
     "license": "MIT",
+    "linecounts": {
+      "languages": {},
+      "total_code_lines": 0,
+      "total_comment_lines": 0
+    },
     "links": {
       "authors": "/api/v1/crates/foo/1.1.0/authors",
       "dependencies": "/api/v1/crates/foo/1.1.0/dependencies",

src/tests/krate/snapshots/crates_io__tests__krate__yanking__patch_version_yank_unyank-2.snap

@@ -62,6 +62,11 @@ expression: json
     "description": "description",
     "homepage": null,
     "documentation": null,
-    "repository": null
+    "repository": null,
+    "linecounts": {
+      "languages": {},
+      "total_code_lines": 0,
+      "total_comment_lines": 0
+    }
   }
 }

src/tests/krate/snapshots/crates_io__tests__krate__yanking__patch_version_yank_unyank-3.snap

@@ -73,6 +73,11 @@ expression: json
     "description": "description",
     "homepage": null,
     "documentation": null,
-    "repository": null
+    "repository": null,
+    "linecounts": {
+      "languages": {},
+      "total_code_lines": 0,
+      "total_comment_lines": 0
+    }
   }
 }

src/tests/krate/snapshots/crates_io__tests__krate__yanking__patch_version_yank_unyank-4.snap

@@ -73,6 +73,11 @@ expression: json
     "description": "description",
     "homepage": null,
     "documentation": null,
-    "repository": null
+    "repository": null,
+    "linecounts": {
+      "languages": {},
+      "total_code_lines": 0,
+      "total_comment_lines": 0
+    }
   }
 }

src/tests/krate/snapshots/crates_io__tests__krate__yanking__patch_version_yank_unyank-5.snap

@@ -84,6 +84,11 @@ expression: json
     "description": "description",
     "homepage": null,
     "documentation": null,
-    "repository": null
+    "repository": null,
+    "linecounts": {
+      "languages": {},
+      "total_code_lines": 0,
+      "total_comment_lines": 0
+    }
   }
 }