[bilibili.space] Trim empty title and trailing newline

Author: SpriteOvO

src/platform/bilibili/source/space.rs

@@ -197,7 +197,9 @@ mod data {
             match self {
                 Self::None(none) => Some(PostContent::plain(&none.none.tips)),
                 Self::Opus(opus) => {
-                    if let Some(title) = opus.opus.title.as_deref() {
+                    if let Some(title) = opus.opus.title.as_deref()
+                        && !title.is_empty()
+                    {
                         Some(
                             PostContent::plain(title)
                                 .with_plain("\n\n")
@@ -463,12 +465,21 @@ mod data {
     #[derive(Debug, Deserialize)]
     pub struct RichText {
         rich_text_nodes: Vec<RichTextNode>,
-        text: String, // Fallback
+        // text: String, // Fallback
     }
 
     impl RichText {
         pub fn to_content(&self) -> PostContent {
-            PostContent::from_parts(self.rich_text_nodes.iter().map(|node| match &node.kind {
+            // Sometimes the last node is just a newline, we trim it here
+            let rich_text_nodes = if let Some(last) = self.rich_text_nodes.last()
+                && last.is_newline()
+            {
+                &self.rich_text_nodes[..self.rich_text_nodes.len() - 1]
+            } else {
+                &self.rich_text_nodes[..]
+            };
+
+            PostContent::from_parts(rich_text_nodes.iter().map(|node| match &node.kind {
                 RichTextNodeKind::Text => PostContentPart::Plain(node.text.clone()),
                 RichTextNodeKind::Web { jump_url } => PostContentPart::Link {
                     display: node.text.clone(),
@@ -534,6 +545,14 @@ mod data {
         pub kind: RichTextNodeKind,
     }
 
+    impl RichTextNode {
+        fn is_newline(&self) -> bool {
+            matches!(&self.kind, RichTextNodeKind::Text)
+                && self.text == "\n"
+                && self.orig_text == "\n"
+        }
+    }
+
     #[derive(Debug, Deserialize)]
     #[serde(tag = "type")]
     pub enum RichTextNodeKind {
@@ -761,15 +780,21 @@ fn fetch_space_history_impl<'a>(
 
 fn parse_response(resp: data::SpaceHistory, blocked: &mut BlockedPostIds) -> anyhow::Result<Posts> {
     fn parse_item(item: &data::Item, parent_item: Option<&data::Item>) -> anyhow::Result<Post> {
+        let desc_content = item
+            .modules
+            .dynamic
+            .desc
+            .as_ref()
+            .map(|desc| desc.to_content());
         let major_content = item
             .modules
             .dynamic
             .major
             .as_ref()
             .and_then(|major| major.to_content());
-        let content = match (&item.modules.dynamic.desc, major_content) {
-            (Some(desc), Some(major)) => desc.to_content().with_plain("\n\n").with_content(major),
-            (Some(desc), None) => desc.to_content(),
+        let content = match (desc_content, major_content) {
+            (Some(desc), Some(major)) => desc.with_plain("\n\n").with_content(major),
+            (Some(desc), None) => desc,
             (None, Some(major)) => major,
             (None, None) => bail!("item no content. item: {item:?}"),
         };