Use a single type

crates/distribution-types/src/file.rs

@@ -143,8 +143,6 @@ impl Display for FileLocation {
     PartialOrd,
     Ord,
     Hash,
-    Serialize,
-    Deserialize,
     rkyv::Archive,
     rkyv::Deserialize,
     rkyv::Serialize,
@@ -153,6 +151,24 @@ impl Display for FileLocation {
 #[archive_attr(derive(Debug))]
 pub struct UrlString(String);
 
+impl serde::Serialize for UrlString {
+    fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
+    where
+        S: serde::ser::Serializer,
+    {
+        String::serialize(&self.0, serializer)
+    }
+}
+
+impl<'de> serde::de::Deserialize<'de> for UrlString {
+    fn deserialize<D>(deserializer: D) -> Result<Self, D::Error>
+    where
+        D: serde::de::Deserializer<'de>,
+    {
+        String::deserialize(deserializer).map(UrlString)
+    }
+}
+
 impl UrlString {
     /// Converts a [`UrlString`] to a [`Url`].
     pub fn to_url(&self) -> Url {

crates/distribution-types/src/index_url.rs

@@ -113,7 +113,7 @@ impl FromStr for IndexUrl {
 
     fn from_str(s: &str) -> Result<Self, Self::Err> {
         let path = Path::new(s);
-        let url = if path.exists() {
+        let url = if Path::new(s).exists() {
             VerbatimUrl::from_path(path)?
         } else {
             VerbatimUrl::parse_url(s)?
@@ -248,9 +248,8 @@ impl FromStr for FlatIndexLocation {
     type Err = IndexUrlError;
 
     fn from_str(s: &str) -> Result<Self, Self::Err> {
-        let path = Path::new(s);
-        let url = if path.exists() {
-            VerbatimUrl::from_path(path)?
+        let url = if Path::new(s).exists() {
+            VerbatimUrl::from_path(s)?
         } else {
             VerbatimUrl::parse_url(s)?
         };