fix builders, test, fixedstring, type resolved variant of media items

Author: jamesbt365

examples/e01_basic_ping_bot/Cargo.toml

@@ -5,7 +5,5 @@ authors = ["my name <[email protected]>"]
 edition.workspace = true
 
 [dependencies]
-serde_json = "1.0.139"
 serenity = { path = "../../", default-features = false, features = ["gateway", "model", "rustls_backend"] }
-tokio = { version = "1.0", features = ["macros", "rt-multi-thread"] }
-tracing-subscriber = "0.3.19"
+tokio = { version = "1.0", features = ["macros", "rt-multi-thread"] }

src/builder/create_components.rs

@@ -65,6 +65,12 @@ pub enum CreateComponent<'a> {
     ActionRow(CreateActionRow<'a>),
     /// A section, V2 component.
     Section(CreateSection<'a>),
+    TextDisplay(CreateTextDisplay<'a>),
+    Thumbnail(CreateThumbnail<'a>),
+    MediaGallery(CreateMediaGallery<'a>),
+    File(CreateFile<'a>),
+    Separator(CreateSeparator),
+    Container(CreateContainer<'a>),
 }
 
 #[derive(Clone, Debug, Serialize)]
@@ -78,7 +84,6 @@ pub struct CreateSection<'a> {
 }
 
 impl<'a> CreateSection<'a> {
-    // TODO: change type
     pub fn new(
         components: impl Into<Cow<'a, [CreateSectionComponent<'a>]>>,
         accessory: CreateSectionAccessory<'a>,
@@ -190,18 +195,18 @@ impl<'a> CreateUnfurledMediaItem<'a> {
 pub struct CreateMediaGallery<'a> {
     #[serde(rename = "type")]
     kind: StaticU8<12>,
-    items: Cow<'a, [CreateSectionComponent<'a>]>,
+    items: Cow<'a, [CreateMediaGalleryItem<'a>]>,
 }
 
 impl<'a> CreateMediaGallery<'a> {
-    pub fn new(items: impl Into<Cow<'a, [CreateSectionComponent<'a>]>>) -> Self {
+    pub fn new(items: impl Into<Cow<'a, [CreateMediaGalleryItem<'a>]>>) -> Self {
         CreateMediaGallery {
             kind: StaticU8::<12>,
             items: items.into(),
         }
     }
 
-    pub fn items(mut self, items: impl Into<Cow<'a, [CreateSectionComponent<'a>]>>) -> Self {
+    pub fn items(mut self, items: impl Into<Cow<'a, [CreateMediaGalleryItem<'a>]>>) -> Self {
         self.items = items.into();
         self
     }
@@ -217,6 +222,31 @@ pub struct CreateMediaGalleryItem<'a> {
     spoiler: Option<bool>,
 }
 
+impl<'a> CreateMediaGalleryItem<'a> {
+    pub fn new(media: CreateUnfurledMediaItem<'a>) -> Self {
+        CreateMediaGalleryItem {
+            media,
+            description: None,
+            spoiler: None,
+        }
+    }
+
+    pub fn media(mut self, media: CreateUnfurledMediaItem<'a>) -> Self {
+        self.media = media;
+        self
+    }
+
+    pub fn description(mut self, description: impl Into<Cow<'a, str>>) -> Self {
+        self.description = Some(description.into());
+        self
+    }
+
+    pub fn spoiler(mut self, spoiler: bool) -> Self {
+        self.spoiler = Some(spoiler);
+        self
+    }
+}
+
 #[derive(Clone, Debug, Serialize)]
 #[must_use]
 pub struct CreateFile<'a> {

src/model/application/component.rs

@@ -1,3 +1,4 @@
+use nonmax::NonMaxU32;
 use serde::de::Error as DeError;
 use serde::ser::{Serialize, Serializer};
 use serde_json::value::RawValue;
@@ -52,10 +53,6 @@ pub enum Component {
 
 // TODO: add something like this to every variant.
 // The component type, it will always be [`ComponentType::Thing`].
-// #[serde(rename = "type")]
-// pub kind: ComponentType,
-
-// TODO: use fixedstring is places i missed when i find suitable lengths
 
 impl<'de> Deserialize<'de> for Component {
     fn deserialize<D>(deserializer: D) -> std::result::Result<Self, D::Error>
@@ -114,15 +111,50 @@ pub struct Section {
 #[non_exhaustive]
 pub struct Thumbnail {
     media: UnfurledMediaItem,
-    description: Option<String>,
+    description: Option<FixedString<u16>>,
     spoiler: Option<bool>,
 }
 
+#[cfg_attr(feature = "typesize", derive(typesize::derive::TypeSize))]
+#[derive(Clone, Debug, Deserialize, Serialize)]
+#[non_exhaustive]
+#[serde(untagged)]
+pub enum MediaItem {
+    Resolved(ResolvedUnfurledMediaItem),
+    Unresolved(UnfurledMediaItem),
+}
+
 #[cfg_attr(feature = "typesize", derive(typesize::derive::TypeSize))]
 #[derive(Clone, Debug, Deserialize, Serialize)]
 #[non_exhaustive]
 pub struct UnfurledMediaItem {
-    url: String,
+    url: FixedString<u16>,
+}
+
+#[cfg_attr(feature = "typesize", derive(typesize::derive::TypeSize))]
+#[derive(Clone, Debug, Deserialize, Serialize)]
+#[non_exhaustive]
+pub struct ResolvedUnfurledMediaItem {
+    url: FixedString<u16>,
+    proxy_url: FixedString<u16>,
+    width: NonMaxU32,
+    height: NonMaxU32,
+    content_type: FixedString,
+    loading_state: UnfurledMediaItemLoadingState,
+}
+
+enum_number! {
+    /// The loading state of the media item.
+    #[derive(Clone, Copy, Debug, Eq, Hash, Ord, PartialEq, PartialOrd, Deserialize, Serialize)]
+    #[cfg_attr(feature = "typesize", derive(typesize::derive::TypeSize))]
+    #[non_exhaustive]
+    pub enum UnfurledMediaItemLoadingState {
+        DiscordUnknown = 0,
+        Loading = 1,
+        LoadingSuccess = 2,
+        LoadingNotFound = 3,
+        _ => Unknown(u8),
+    }
 }
 
 #[cfg_attr(feature = "typesize", derive(typesize::derive::TypeSize))]
@@ -147,7 +179,7 @@ pub enum SectionComponent {
 #[derive(Clone, Debug, Deserialize, Serialize)]
 #[non_exhaustive]
 pub struct TextDisplay {
-    content: String,
+    content: FixedString<u16>,
 }
 
 #[cfg_attr(feature = "typesize", derive(typesize::derive::TypeSize))]
@@ -162,7 +194,7 @@ pub struct MediaGallery {
 #[non_exhaustive]
 pub struct MediaGalleryItem {
     media: UnfurledMediaItem,
-    description: Option<String>,
+    description: Option<FixedString<u16>>,
     spoiler: Option<bool>,
 }