From: Huck Boles <huck@huck.website>
Date: Wed, 24 May 2023 01:56:12 +0000 (-0500)
Subject: refactored: added filename to parser errors
X-Git-Url: https://git.huck.website/?a=commitdiff_plain;h=1f29aaaf7ae460f58854e4277f4f4cb70abc7224;p=metaforge.git

refactored: added filename to parser errors
---

diff --git a/src/error.rs b/src/error.rs
index c11f25b..4f82158 100644
--- a/src/error.rs
+++ b/src/error.rs
@@ -1,4 +1,3 @@
-use crate::Rule;
 use thiserror::Error;
 
 #[derive(Error, Debug)]
@@ -25,13 +24,13 @@ pub enum MetaError {
     UndefinedDefault { pattern: String, path: String },
     #[error("the parser cannot resolve this input: {input}")]
     UnreachableRule { input: String },
+    #[error("{file}\n{error}")]
+    ParserError { file: String, error: String },
     #[error(transparent)]
     MetaError(#[from] Box<MetaError>),
     #[error(transparent)]
     PandocError(#[from] pandoc::PandocError),
     #[error(transparent)]
-    ParserError(#[from] pest::error::Error<Rule>),
-    #[error(transparent)]
     Other(#[from] eyre::Error),
 }
 
diff --git a/src/metafile/file.rs b/src/metafile/file.rs
index 7f8e70a..b664b09 100644
--- a/src/metafile/file.rs
+++ b/src/metafile/file.rs
@@ -45,7 +45,10 @@ impl<'a> MetaFile<'a> {
             }
         };
 
-        let mut metafile = parse_string(str, opts).map_err(MetaError::from)?;
+        let mut metafile = parse_string(str, opts).map_err(|e| MetaError::ParserError {
+            file: path.to_string_lossy().to_string(),
+            error: e.to_string(),
+        })?;
 
         metafile.path = path;
         Ok(metafile)
@@ -63,7 +66,10 @@ impl<'a> MetaFile<'a> {
         let html = self.to_html().map_err(MetaError::from)?;
 
         let pattern = self.get_pattern("base").map_err(MetaError::from)?;
-        let mut base = crate::parse_string(pattern, self.opts).map_err(MetaError::from)?;
+        let mut base = parse_string(pattern, self.opts).map_err(|e| MetaError::ParserError {
+            file: self.path.to_string_lossy().to_string(),
+            error: e.to_string(),
+        })?;
 
         base.merge(self);
         base.patterns.insert(Scope::into_global("SOURCE"), html);