From 291ce9072d4125102c76b23e40a7cc1fcbb83fef Mon Sep 17 00:00:00 2001 From: "Rodrigo Rodriguez (Pragmatismo)" Date: Fri, 7 Nov 2025 10:11:08 -0300 Subject: [PATCH] refactor(logging): update component names and adjust log levels Renamed PostgreSQL references to "Tables" for clarity in bootstrap logs, changed config sync logging from info to trace for reduced verbosity, and made session message clearing method private to limit external access. --- src/bootstrap/mod.rs | 5 ++--- src/config/mod.rs | 4 ++-- src/session/mod.rs | 2 +- 3 files changed, 5 insertions(+), 6 deletions(-) diff --git a/src/bootstrap/mod.rs b/src/bootstrap/mod.rs index 11a2aab6e..81cc1952b 100644 --- a/src/bootstrap/mod.rs +++ b/src/bootstrap/mod.rs @@ -43,14 +43,13 @@ impl BootstrapManager { ); if !Self::is_postgres_running() { - warn!("PostgreSQL server is not running. Attempting to start 'tables' component..."); let pm = PackageManager::new(install_mode.clone(), tenant.clone()) .expect("Failed to initialize PackageManager"); if let Err(e) = pm.start("tables") { - error!("Failed to start PostgreSQL component automatically: {}", e); + error!("Failed to start Tables server component automatically: {}", e); panic!("Database not available and auto-start failed."); } else { - info!("PostgreSQL component started successfully."); + info!("Tables component started successfully."); } } diff --git a/src/config/mod.rs b/src/config/mod.rs index 046b53358..3871947b6 100644 --- a/src/config/mod.rs +++ b/src/config/mod.rs @@ -1,7 +1,7 @@ use diesel::prelude::*; use diesel::pg::PgConnection; use uuid::Uuid; -use log::info; +use log::{trace}; use std::collections::HashMap; use std::fs::OpenOptions; use std::io::Write; @@ -321,7 +321,7 @@ impl ConfigManager { } } - info!("Synced {} config values for bot {}", updated, bot_id); + trace!("Synced {} config values for bot {}", updated, bot_id); Ok(updated) } diff --git a/src/session/mod.rs b/src/session/mod.rs index 756d1c811..ce888a6f5 100644 --- a/src/session/mod.rs +++ b/src/session/mod.rs @@ -174,7 +174,7 @@ impl SessionManager { Ok(inserted) } - pub fn clear_messages(&mut self, _session_id: Uuid) -> Result<(), Box> { + fn _clear_messages(&mut self, _session_id: Uuid) -> Result<(), Box> { use crate::shared::models::message_history::dsl::*; diesel::delete(message_history.filter(session_id.eq(session_id)))