fix: suppress noisy ort/hnsw_rs logs in production, clean up clippy

Author: devwhodevs

src/embedder.rs

@@ -36,23 +36,16 @@ impl Embedder {
 
         // Download model if missing.
         if !model_path.exists() {
-            let expected_sha: Option<&str> = if MODEL_SHA256.is_empty() {
-                None
-            } else {
-                Some(MODEL_SHA256)
-            };
-            download_file(MODEL_URL, &model_path, expected_sha)?;
+            download_file(MODEL_URL, &model_path, Some(MODEL_SHA256))?;
         }
 
         // Download tokenizer if missing.
         if !tokenizer_path.exists() {
             download_file(TOKENIZER_URL, &tokenizer_path, None)?;
         }
 
-        // Verify model hash if we have one.
-        if !MODEL_SHA256.is_empty() {
-            verify_sha256(&model_path, MODEL_SHA256)?;
-        }
+        // Verify model hash.
+        verify_sha256(&model_path, MODEL_SHA256)?;
 
         let session = Session::builder()
             .with_context(|| "creating ONNX session builder")?

src/main.rs

@@ -7,7 +7,6 @@ use anyhow::Result;
 use clap::{Parser, Subcommand};
 use std::io::{self, BufRead, Write};
 use std::path::PathBuf;
-use tracing::info;
 
 use config::Config;
 
@@ -89,18 +88,23 @@ fn remove_dir_if_exists(path: &std::path::Path) -> Result<bool> {
 fn main() -> Result<()> {
     let cli = Cli::parse();
 
-    // Set up tracing.
-    let filter = if cli.verbose { "debug" } else { "info" };
+    // Set up tracing. Default: suppress all logs (ort and hnsw_rs are very noisy).
+    // --verbose enables debug for engraph, info for everything else.
+    let filter = if cli.verbose {
+        "engraph=debug,info"
+    } else {
+        "error"
+    };
     tracing_subscriber::fmt()
         .with_env_filter(
             tracing_subscriber::EnvFilter::try_from_default_env()
                 .unwrap_or_else(|_| tracing_subscriber::EnvFilter::new(filter)),
         )
+        .with_writer(std::io::stderr)
         .init();
 
     let mut cfg = Config::load()?;
     let data_dir = Config::data_dir()?;
-    info!(data_dir = %data_dir.display(), "loaded config");
 
     match cli.command {
         Command::Index { path, rebuild } => {