fix log levels

block · Jan 7, 2025 · 9bf6b81 · 9bf6b81
1 parent a5d0dd2
commit 9bf6b81
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 9 deletions.
diff --git a/crates/goose-cli/src/logging.rs b/crates/goose-cli/src/logging.rs
@@ -73,9 +73,7 @@ pub fn setup_logging() -> Result<()> {
     let env_filter = EnvFilter::try_from_default_env().unwrap_or_else(|_| {
         // Set default levels for different modules
         EnvFilter::new("")
-            // Set goose module to INFO only
             .add_directive("goose=debug".parse().unwrap())
-            // Set goose-cli to INFO
             .add_directive("goose_cli=info".parse().unwrap())
             // Set everything else to WARN
             .add_directive(LevelFilter::WARN.into())

diff --git a/crates/goose-cli/src/main.rs b/crates/goose-cli/src/main.rs
@@ -197,7 +197,7 @@ enum CliProviderVariant {
 
 #[tokio::main]
 async fn main() -> Result<()> {
-    setup_logging();
+    setup_logging()?;
 
     let cli = Cli::parse();
 

diff --git a/crates/goose-server/src/logging.rs b/crates/goose-server/src/logging.rs
@@ -72,8 +72,8 @@ pub fn setup_logging() -> Result<()> {
     let env_filter = EnvFilter::try_from_default_env().unwrap_or_else(|_| {
         // Set default levels for different modules
         EnvFilter::new("")
-            // Set goose module to INFO only
-            .add_directive("goose=info".parse().unwrap())
+            // Set goose module to INFO only, but allow ERROR level for all submodules
+            .add_directive("goose=debug".parse().unwrap())
             // Set goose-server to INFO
             .add_directive("goose_server=info".parse().unwrap())
             // Set tower-http to INFO for request logging
@@ -84,8 +84,8 @@ pub fn setup_logging() -> Result<()> {
 
     // Build the subscriber with required layers
     let subscriber = Registry::default()
-        .with(file_layer)   
-        .with(console_layer.with_filter(env_filter)); 
+        .with(file_layer.with_filter(env_filter))   
+        .with(console_layer.with_filter(LevelFilter::INFO)); 
 
     // Initialize with Langfuse if available
     if let Some(langfuse) = langfuse_layer::create_langfuse_observer() {

diff --git a/crates/goose/src/agent.rs b/crates/goose/src/agent.rs
@@ -412,8 +412,6 @@ impl Agent {
                 let outputs = futures::future::join_all(futures)
                     .await;
 
-                debug!("All tool requests completed");
-
                 // Create a message with the responses
                 let mut message_tool_response = Message::user();
                 // Now combine these into MessageContent::ToolResponse using the original ID