feat(ast): clean up from_*s for ast

2 files changed, 29 insertions, 3 deletions

diff --git a/crates/germ/src/ast/container.rs b/crates/germ/src/ast/container.rs
index 6cbc677..acb8894 100644
--- a/crates/germ/src/ast/container.rs
+++ b/crates/germ/src/ast/container.rs
@@ -31,7 +31,7 @@ pub struct Ast {
 }
 
 impl Ast {
-  /// Build an AST tree from Gemtext.
+  /// Build an AST tree from Gemtext
   ///
   /// # Example
   ///
@@ -39,10 +39,36 @@ impl Ast {
   /// let _ = germ::ast::Ast::from_string(r#"=> gemini://gem.rest/ GemRest"#);
   /// ```
   #[must_use]
-  pub fn from_string(source: &str) -> Self {
+  pub fn from_owned(value: &(impl AsRef<str> + ?Sized)) -> Self {
+    Self::from_value(value.as_ref())
+  }
+
+  /// Build an AST tree from Gemtext
+  ///
+  /// # Example
+  ///
+  /// ```rust
+  /// let _ = germ::ast::Ast::from_string(r#"=> gemini://gem.rest/ GemRest"#);
+  /// ```
+  #[must_use]
+  #[allow(clippy::needless_pass_by_value)]
+  pub fn from_string(value: (impl Into<String> + ?Sized)) -> Self {
+    Self::from_value(&value.into())
+  }
+
+  /// Build an AST tree from a value
+  ///
+  /// # Example
+  ///
+  /// ```rust
+  /// let _ = germ::ast::Ast::from_value(r#"=> gemini://gem.rest/ GemRest"#);
+  /// ```
+  #[must_use]
+  pub fn from_value(value: &(impl ToString + ?Sized)) -> Self {
     let mut ast = vec![];
     let mut in_preformatted = false;
     let mut in_list = false;
+    let source = value.to_string();
     let mut lines = source.lines();
 
     // Iterate over all lines in the Gemtext `source`
diff --git a/crates/germ/src/convert.rs b/crates/germ/src/convert.rs
index 4c5f0c8..bfab16d 100644
--- a/crates/germ/src/convert.rs
+++ b/crates/germ/src/convert.rs
@@ -69,5 +69,5 @@ pub fn from_ast(source: &Ast, target: &Target) -> String {
 /// ```
 #[must_use]
 pub fn from_string(source: &str, target: &Target) -> String {
-  from_ast(&Ast::from_string(source), target)
+  from_ast(&Ast::from_owned(&source), target)
 }