fix: Make Message a distinct type

This makes it clearer that we shouldn't set `id` on this.
If someone wants to set an `id`, they should create a new `Group` which
will have a `Title`.

Author: epage

src/level.rs

@@ -2,7 +2,7 @@
 
 use crate::renderer::stylesheet::Stylesheet;
 use crate::snippet::{ERROR_TXT, HELP_TXT, INFO_TXT, NOTE_TXT, WARNING_TXT};
-use crate::{OptionCow, Title};
+use crate::{Message, OptionCow, Title};
 use anstyle::Style;
 use std::borrow::Cow;
 
@@ -89,7 +89,6 @@ impl<'a> Level<'a> {
             level: self,
             id: None,
             text: text.into(),
-            is_pre_styled: false,
         }
     }
 
@@ -103,12 +102,10 @@ impl<'a> Level<'a> {
     /// used to normalize untrusted text before it is passed to this function.
     ///
     /// </div>
-    pub fn message(self, text: impl Into<Cow<'a, str>>) -> Title<'a> {
-        Title {
+    pub fn message(self, text: impl Into<Cow<'a, str>>) -> Message<'a> {
+        Message {
             level: self,
-            id: None,
             text: text.into(),
-            is_pre_styled: true,
         }
     }
 

src/renderer/mod.rs

@@ -48,7 +48,7 @@ use crate::renderer::source_map::{
 };
 use crate::renderer::styled_buffer::StyledBuffer;
 use crate::snippet::Id;
-use crate::{Annotation, AnnotationKind, Element, Group, Origin, Patch, Snippet, Title};
+use crate::{Annotation, AnnotationKind, Element, Group, Message, Origin, Patch, Snippet, Title};
 pub use anstyle::*;
 use margin::Margin;
 use std::borrow::Cow;
@@ -303,7 +303,20 @@ impl Renderer {
                                 title,
                                 max_line_num_len,
                                 title_style,
-                                matches!(peek, Some(Element::Title(_))),
+                                matches!(peek, Some(Element::Title(_) | Element::Message(_))),
+                                buffer_msg_line_offset,
+                            );
+                            last_was_suggestion = false;
+                        }
+                        Element::Message(title) => {
+                            let title_style = TitleStyle::Secondary;
+                            let buffer_msg_line_offset = buffer.num_lines();
+                            self.render_title(
+                                &mut buffer,
+                                title,
+                                max_line_num_len,
+                                title_style,
+                                matches!(peek, Some(Element::Title(_) | Element::Message(_))),
                                 buffer_msg_line_offset,
                             );
                             last_was_suggestion = false;
@@ -336,6 +349,16 @@ impl Renderer {
                                             );
                                         }
 
+                                        Some(Element::Message(level))
+                                            if level.level.name != Some(None) =>
+                                        {
+                                            self.draw_col_separator_no_space(
+                                                &mut buffer,
+                                                current_line,
+                                                max_line_num_len + 1,
+                                            );
+                                        }
+
                                         None if group_len > 1 => self.draw_col_separator_end(
                                             &mut buffer,
                                             current_line,
@@ -503,23 +526,23 @@ impl Renderer {
     fn render_title(
         &self,
         buffer: &mut StyledBuffer,
-        title: &Title<'_>,
+        title: &dyn MessageOrTitle,
         max_line_num_len: usize,
         title_style: TitleStyle,
         is_cont: bool,
         buffer_msg_line_offset: usize,
     ) {
         let (label_style, title_element_style) = match title_style {
             TitleStyle::MainHeader => (
-                ElementStyle::Level(title.level.level),
+                ElementStyle::Level(title.level().level),
                 if self.short_message {
                     ElementStyle::NoStyle
                 } else {
                     ElementStyle::MainHeaderMsg
                 },
             ),
             TitleStyle::Header => (
-                ElementStyle::Level(title.level.level),
+                ElementStyle::Level(title.level().level),
                 ElementStyle::HeaderMsg,
             ),
             TitleStyle::Secondary => {
@@ -538,10 +561,10 @@ impl Renderer {
         };
         let mut label_width = 0;
 
-        if title.level.name != Some(None) {
-            buffer.append(buffer_msg_line_offset, title.level.as_str(), label_style);
-            label_width += title.level.as_str().len();
-            if let Some(Id { id: Some(id), url }) = &title.id {
+        if title.level().name != Some(None) {
+            buffer.append(buffer_msg_line_offset, title.level().as_str(), label_style);
+            label_width += title.level().as_str().len();
+            if let Some(Id { id: Some(id), url }) = &title.id() {
                 buffer.append(buffer_msg_line_offset, "[", label_style);
                 if let Some(url) = url.as_ref() {
                     buffer.append(
@@ -584,10 +607,10 @@ impl Renderer {
             label_width
         });
 
-        let (title_str, style) = if title.is_pre_styled {
-            (title.text.to_string(), ElementStyle::NoStyle)
+        let (title_str, style) = if title.is_pre_styled() {
+            (title.text().to_owned(), ElementStyle::NoStyle)
         } else {
-            (normalize_whitespace(&title.text), title_element_style)
+            (normalize_whitespace(title.text()), title_element_style)
         };
         for (i, text) in title_str.lines().enumerate() {
             if i != 0 {
@@ -2532,6 +2555,43 @@ impl Renderer {
     }
 }
 
+trait MessageOrTitle {
+    fn level(&self) -> &Level<'_>;
+    fn id(&self) -> Option<&Id<'_>>;
+    fn text(&self) -> &str;
+    fn is_pre_styled(&self) -> bool;
+}
+
+impl MessageOrTitle for Title<'_> {
+    fn level(&self) -> &Level<'_> {
+        &self.level
+    }
+    fn id(&self) -> Option<&Id<'_>> {
+        self.id.as_ref()
+    }
+    fn text(&self) -> &str {
+        self.text.as_ref()
+    }
+    fn is_pre_styled(&self) -> bool {
+        false
+    }
+}
+
+impl MessageOrTitle for Message<'_> {
+    fn level(&self) -> &Level<'_> {
+        &self.level
+    }
+    fn id(&self) -> Option<&Id<'_>> {
+        None
+    }
+    fn text(&self) -> &str {
+        self.text.as_ref()
+    }
+    fn is_pre_styled(&self) -> bool {
+        true
+    }
+}
+
 // instead of taking the String length or dividing by 10 while > 0, we multiply a limit by 10 until
 // we're higher. If the loop isn't exited by the `return`, the last multiplication will wrap, which
 // is OK, because while we cannot fit a higher power of 10 in a usize, the loop will end anyway.
@@ -2846,7 +2906,10 @@ fn max_line_number(groups: &[Group<'_>]) -> usize {
             v.elements
                 .iter()
                 .map(|s| match s {
-                    Element::Title(_) | Element::Origin(_) | Element::Padding(_) => 0,
+                    Element::Title(_)
+                    | Element::Message(_)
+                    | Element::Origin(_)
+                    | Element::Padding(_) => 0,
                     Element::Cause(cause) => {
                         let end = cause
                             .markers

src/snippet.rs

@@ -63,6 +63,7 @@ impl<'a> Group<'a> {
 #[non_exhaustive]
 pub enum Element<'a> {
     Title(Title<'a>),
+    Message(Message<'a>),
     Cause(Snippet<'a, Annotation<'a>>),
     Suggestion(Snippet<'a, Patch<'a>>),
     Origin(Origin<'a>),
@@ -75,6 +76,12 @@ impl<'a> From<Title<'a>> for Element<'a> {
     }
 }
 
+impl<'a> From<Message<'a>> for Element<'a> {
+    fn from(value: Message<'a>) -> Self {
+        Element::Message(value)
+    }
+}
+
 impl<'a> From<Snippet<'a, Annotation<'a>>> for Element<'a> {
     fn from(value: Snippet<'a, Annotation<'a>>) -> Self {
         Element::Cause(value)
@@ -103,15 +110,14 @@ impl From<Padding> for Element<'_> {
 #[derive(Clone, Debug)]
 pub struct Padding;
 
-/// A text [`Element`] in a [`Group`]
+/// A text [`Element`] to start a [`Group`]
 ///
 /// See [`Level::title`] to create this.
 #[derive(Clone, Debug)]
 pub struct Title<'a> {
     pub(crate) level: Level<'a>,
     pub(crate) id: Option<Id<'a>>,
     pub(crate) text: Cow<'a, str>,
-    pub(crate) is_pre_styled: bool,
 }
 
 impl<'a> Title<'a> {
@@ -144,6 +150,15 @@ impl<'a> Title<'a> {
     }
 }
 
+/// A text [`Element`] in a [`Group`]
+///
+/// See [`Level::message`] to create this.
+#[derive(Clone, Debug)]
+pub struct Message<'a> {
+    pub(crate) level: Level<'a>,
+    pub(crate) text: Cow<'a, str>,
+}
+
 /// A source view [`Element`] in a [`Group`]
 ///
 /// If you do not have [source][Snippet::source] available, see instead [`Origin`]