From bb46cce3635f927bf893ac8d7c56b8844cb56fb2 Mon Sep 17 00:00:00 2001
From: Ed Page <eopage@gmail.com>
Date: Fri, 12 Apr 2024 15:21:21 -0500
Subject: [PATCH] refactor(config): Consistently use kebab-case

This shouldn't change the behavior but makes it safer if
- We add new fields where it will matter
- Copy/paste these for new structs

I did not change things related to the Index because we are already
stuck with that case (whether we want it or not)
---
 src/cargo/ops/cargo_new.rs        | 1 +
 src/cargo/sources/directory.rs    | 1 +
 src/cargo/sources/registry/mod.rs | 1 +
 src/cargo/util/context/mod.rs     | 5 ++++-
 4 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/cargo/ops/cargo_new.rs b/src/cargo/ops/cargo_new.rs
index ca3161462..f349395b5 100644
--- a/src/cargo/ops/cargo_new.rs
+++ b/src/cargo/ops/cargo_new.rs
@@ -131,6 +131,7 @@ impl NewOptions {
 }
 
 #[derive(Deserialize)]
+#[serde(rename_all = "kebab-case")]
 struct CargoNewConfig {
     #[deprecated = "cargo-new no longer supports adding the authors field"]
     #[allow(dead_code)]
diff --git a/src/cargo/sources/directory.rs b/src/cargo/sources/directory.rs
index f853391a5..33c0f8d97 100644
--- a/src/cargo/sources/directory.rs
+++ b/src/cargo/sources/directory.rs
@@ -70,6 +70,7 @@ pub struct DirectorySource<'gctx> {
 /// The file name is simply `.cargo-checksum.json`. The checksum algorithm as
 /// of now is SHA256.
 #[derive(Deserialize)]
+#[serde(rename_all = "kebab-case")]
 struct Checksum {
     /// Checksum of the package. Normally it is computed from the `.crate` file.
     package: Option<String>,
diff --git a/src/cargo/sources/registry/mod.rs b/src/cargo/sources/registry/mod.rs
index b1ef55d24..93e35abef 100644
--- a/src/cargo/sources/registry/mod.rs
+++ b/src/cargo/sources/registry/mod.rs
@@ -227,6 +227,7 @@ pub const CRATES_IO_DOMAIN: &str = "crates.io";
 /// The content inside `.cargo-ok`.
 /// See [`RegistrySource::unpack_package`] for more.
 #[derive(Deserialize, Serialize)]
+#[serde(rename_all = "kebab-case")]
 struct LockMetadata {
     /// The version of `.cargo-ok` file
     v: u32,
diff --git a/src/cargo/util/context/mod.rs b/src/cargo/util/context/mod.rs
index f38fbd3df..115cac968 100644
--- a/src/cargo/util/context/mod.rs
+++ b/src/cargo/util/context/mod.rs
@@ -2514,6 +2514,7 @@ impl<'de> Deserialize<'de> for SslVersionConfig {
 }
 
 #[derive(Clone, Debug, Deserialize, PartialEq)]
+#[serde(rename_all = "kebab-case")]
 pub struct SslVersionConfigRange {
     pub min: Option<String>,
     pub max: Option<String>,
@@ -2644,6 +2645,7 @@ impl BuildTargetConfig {
 }
 
 #[derive(Deserialize, Default)]
+#[serde(rename_all = "kebab-case")]
 pub struct TermConfig {
     pub verbose: Option<bool>,
     pub quiet: Option<bool>,
@@ -2656,13 +2658,14 @@ pub struct TermConfig {
 }
 
 #[derive(Debug, Default, Deserialize)]
+#[serde(rename_all = "kebab-case")]
 pub struct ProgressConfig {
     pub when: ProgressWhen,
     pub width: Option<usize>,
 }
 
 #[derive(Debug, Default, Deserialize)]
-#[serde(rename_all = "lowercase")]
+#[serde(rename_all = "kebab-case")]
 pub enum ProgressWhen {
     #[default]
     Auto,