From 1e5d328ab4a1e46e013a7388575c26eb16a34367 Mon Sep 17 00:00:00 2001
From: sigil-03 <sigil@glyphs.tech>
Date: Mon, 7 Jul 2025 12:31:09 -0600
Subject: [PATCH] add logging crate to reduce unnecessary stdout spam

---
 Cargo.toml          | 6 ++++++
 src/bin/ent/main.rs | 6 ++++++
 src/issue.rs        | 6 +++++-
 src/issues.rs       | 6 +++++-
 4 files changed, 22 insertions(+), 2 deletions(-)

diff --git a/Cargo.toml b/Cargo.toml
index 1f7941d..fccb9c9 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -3,11 +3,17 @@ name = "entomologist"
 version = "0.1.0"
 edition = "2024"
 
+[features]
+default = []
+log = ["dep:log", "dep:simple_logger"]
+
 [dependencies]
 anyhow = "1.0.95"
 clap = { version = "4.5.26", features = ["derive"] }
+log = { version = "0.4.27", optional = true }
 rand = "0.9.1"
 serde = { version = "1.0.217", features = ["derive"] }
+simple_logger = { version = "5.0.0", optional = true }
 tempfile = "3.20.0"
 thiserror = "2.0.11"
 toml = "0.8.19"
diff --git a/src/bin/ent/main.rs b/src/bin/ent/main.rs
index 65e36dc..47fa87c 100644
--- a/src/bin/ent/main.rs
+++ b/src/bin/ent/main.rs
@@ -1,5 +1,8 @@
 use clap::Parser;
 
+#[cfg(feature = "log")]
+use simple_logger;
+
 #[derive(Debug, clap::Parser)]
 #[command(version, about, long_about = None)]
 struct Args {
@@ -49,6 +52,9 @@ fn handle_command(args: &Args, issues_dir: &std::path::Path) -> anyhow::Result<(
 }
 
 fn main() -> anyhow::Result<()> {
+    #[cfg(feature = "log")]
+    simple_logger::SimpleLogger::new().env().init().unwrap();
+
     let args: Args = Args::parse();
     // println!("{:?}", args);
 
diff --git a/src/issue.rs b/src/issue.rs
index 939af47..8008ea9 100644
--- a/src/issue.rs
+++ b/src/issue.rs
@@ -1,5 +1,8 @@
 use std::str::FromStr;
 
+#[cfg(feature = "log")]
+use log::debug;
+
 #[derive(Debug, PartialEq, serde::Deserialize)]
 /// These are the states an issue can be in.
 pub enum State {
@@ -74,7 +77,8 @@ impl Issue {
                         dependencies = Some(deps);
                     }
                 } else {
-                    println!("ignoring unknown file in issue directory: {:?}", file_name);
+                    #[cfg(feature = "log")]
+                    debug!("ignoring unknown file in issue directory: {:?}", file_name);
                 }
             }
         }
diff --git a/src/issues.rs b/src/issues.rs
index 9422ded..b45026f 100644
--- a/src/issues.rs
+++ b/src/issues.rs
@@ -1,3 +1,6 @@
+#[cfg(feature = "log")]
+use log::debug;
+
 // Just a placeholder for now, get rid of this if we don't need it.
 #[derive(Debug, PartialEq, serde::Deserialize)]
 pub struct Config {}
@@ -56,7 +59,8 @@ impl Issues {
                 } else if direntry.file_name() == "config.toml" {
                     issues.parse_config(direntry.path().as_path())?;
                 } else {
-                    println!(
+                    #[cfg(feature = "log")]
+                    debug!(
                         "ignoring unknown file in issues directory: {:?}",
                         direntry.file_name()
                     );