fix: revert "fix: remove serde feature of lightningcss (#11706)" (#11796)

Revert "fix: remove `serde` feature of `lightningcss` (#11706)"

This reverts commit db3cf1b.

Author: colinaaa

Cargo.lock

@@ -57,7 +57,7 @@ itertools           = { version = "0.14.0", default-features = false, features =
 itoa                = { version = "1.0.14", default-features = false }
 json                = { version = "0.12.4", default-features = false }
 jsonc-parser        = { version = "0.26.2", default-features = false, features = ["serde"] }
-lightningcss        = { version = "1.0.0-alpha.67", default-features = false }
+lightningcss        = { version = "1.0.0-alpha.68", default-features = false, features = ["serde"] }
 linked_hash_set     = { version = "0.1.5", default-features = false }
 md4                 = { version = "0.10.2", default-features = false }
 memchr              = { version = "2.7.5", default-features = false }

Cargo.toml

@@ -15,60 +15,6 @@ pub struct BrowsersResolver {
   value: String,
 }
 
-// Helper functions for custom serialization without serde
-fn browsers_to_string(browsers: &Browsers) -> String {
-  let parts = vec![
-    browsers.android.map(|v| format!("android:{}", v)),
-    browsers.chrome.map(|v| format!("chrome:{}", v)),
-    browsers.edge.map(|v| format!("edge:{}", v)),
-    browsers.firefox.map(|v| format!("firefox:{}", v)),
-    browsers.ie.map(|v| format!("ie:{}", v)),
-    browsers.ios_saf.map(|v| format!("ios_saf:{}", v)),
-    browsers.opera.map(|v| format!("opera:{}", v)),
-    browsers.safari.map(|v| format!("safari:{}", v)),
-    browsers.samsung.map(|v| format!("samsung:{}", v)),
-  ];
-
-  parts.into_iter().flatten().collect::<Vec<_>>().join(",")
-}
-
-fn string_to_browsers(s: &str) -> Result<Browsers, DeserializeError> {
-  let mut browsers = Browsers::default();
-
-  if s.is_empty() {
-    return Ok(browsers);
-  }
-
-  for part in s.split(',') {
-    let mut split = part.split(':');
-    let browser = split
-      .next()
-      .ok_or_else(|| DeserializeError::MessageError("invalid browser format"))?;
-    let version_str = split
-      .next()
-      .ok_or_else(|| DeserializeError::MessageError("invalid browser format"))?;
-
-    let version = version_str
-      .parse::<u32>()
-      .map_err(|_| DeserializeError::MessageError("invalid version format"))?;
-
-    match browser {
-      "android" => browsers.android = Some(version),
-      "chrome" => browsers.chrome = Some(version),
-      "edge" => browsers.edge = Some(version),
-      "firefox" => browsers.firefox = Some(version),
-      "ie" => browsers.ie = Some(version),
-      "ios_saf" => browsers.ios_saf = Some(version),
-      "opera" => browsers.opera = Some(version),
-      "safari" => browsers.safari = Some(version),
-      "samsung" => browsers.samsung = Some(version),
-      _ => return Err(DeserializeError::MessageError("unknown browser")),
-    }
-  }
-
-  Ok(browsers)
-}
-
 impl ArchiveWith<Browsers> for AsPreset {
   type Archived = ArchivedString;
   type Resolver = BrowsersResolver;
@@ -89,7 +35,8 @@ where
     field: &Browsers,
     serializer: &mut S,
   ) -> Result<Self::Resolver, SerializeError> {
-    let value = browsers_to_string(field);
+    let value = serde_json::to_string(field)
+      .map_err(|_| SerializeError::MessageError("serialize serde_json value failed"))?;
     let inner = ArchivedString::serialize_from_str(&value, serializer)?;
     Ok(BrowsersResolver { value, inner })
   }
@@ -103,6 +50,7 @@ where
     field: &ArchivedString,
     _deserializer: &mut D,
   ) -> Result<Browsers, DeserializeError> {
-    string_to_browsers(field.as_str())
+    serde_json::from_str(field.as_str())
+      .map_err(|_| DeserializeError::MessageError("deserialize serde_json value failed"))
   }
 }