Update peel error logs, and constrain its arguments (#104)

* chore: print the number of entries in the directory while peeling Signed-off-by: jaudiger <[email protected]> * fix: don't try to peel with a depth of 0 Signed-off-by: jaudiger <[email protected]> * chore: format the code with cargo fmt Signed-off-by: jaudiger <[email protected]> --------- Signed-off-by: jaudiger <[email protected]>
brioche-dev · Jul 21, 2024 · a8b4b06 · a8b4b06
1 parent 51862fe
commit a8b4b06
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/crates/brioche-core/src/bake.rs b/crates/brioche-core/src/bake.rs
@@ -427,6 +427,10 @@ async fn run_bake(brioche: &Brioche, recipe: Recipe, meta: &Arc<Meta>) -> anyhow
         Recipe::Peel { directory, depth } => {
             let mut result = bake(brioche, *directory, &scope).await?;
 
+            if depth == 0 {
+                anyhow::bail!("must peel at least 1 layer");
+            }
+
             for _ in 0..depth {
                 let Artifact::Directory(dir) = result.value else {
                     anyhow::bail!("tried peeling non-directory artifact");
@@ -438,7 +442,7 @@ async fn run_bake(brioche: &Brioche, recipe: Recipe, meta: &Arc<Meta>) -> anyhow
                 };
 
                 if entries.next().is_some() {
-                    anyhow::bail!("tried peeling directory with multiple entries");
+                    anyhow::bail!("tried peeling {} entries of {:?}", entries.len() + 2, dir);
                 }
 
                 result = peeled;