serialize tagsets more directly for more native output structure

lib/saluki-context/src/tags/mod.rs

@@ -2,6 +2,7 @@
 
 use std::{fmt, hash, ops::Deref as _, sync::Arc};
 
+use serde::Serialize;
 use stringtheory::MetaString;
 
 mod raw;
@@ -89,6 +90,15 @@ impl fmt::Display for Tag {
     }
 }
 
+impl Serialize for Tag {
+    fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
+    where
+        S: serde::Serializer,
+    {
+        serializer.serialize_str(&self.0)
+    }
+}
+
 impl<T> From<T> for Tag
 where
     T: Into<MetaString>,
@@ -180,7 +190,7 @@ impl<'a> hash::Hash for BorrowedTag<'a> {
 }
 
 /// A set of tags.
-#[derive(Clone, Debug, Default)]
+#[derive(Clone, Debug, Default, Serialize)]
 pub struct TagSet(Vec<Tag>);
 
 impl TagSet {
@@ -482,14 +492,12 @@ impl fmt::Display for SharedTagSet {
     }
 }
 
-impl serde::Serialize for SharedTagSet {
+impl Serialize for SharedTagSet {
     fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
     where
         S: serde::Serializer,
     {
-        // We have this manual implementation of `Serialize` just to avoid needing to bring in `serde_with` to get the
-        // helper that utilizes the `Display` implementation.
-        serializer.collect_str(self)
+        self.0.deref().serialize(serializer)
     }
 }