From 8581927d97635c06b0c3989238e0a6fd78f83ad7 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 11a2aab6..81cc1952 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 046b5335..3871947b 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 756d1c81..ce888a6f 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)))