fix: remove unused imports and resolve config module conflicts

Co-authored-by: aider (openai/andrew/openrouter/anthropic/claude-sonnet-4) <aider@aider.chat>
This commit is contained in:
Andrew Phillips
2025-08-15 16:39:11 -03:00
parent 56f4d8aad5
commit b6389419c0
10 changed files with 44 additions and 34 deletions

View File

@@ -1,4 +1,5 @@
use crate::Args; use crate::Args;
use crate::config;
use crate::compression_engine::CompressionType; use crate::compression_engine::CompressionType;
use crate::meta_plugin::MetaPluginType; use crate::meta_plugin::MetaPluginType;
use clap::Command; use clap::Command;
@@ -195,7 +196,7 @@ pub fn cmd_args_meta_plugin_types(cmd: &mut Command, args: &Args) -> Vec<MetaPlu
meta_plugin_types meta_plugin_types
} }
pub fn settings_meta_plugin_types(cmd: &mut Command, settings: &crate::config::Settings) -> Vec<MetaPluginType> { pub fn settings_meta_plugin_types(cmd: &mut Command, settings: &config::Settings) -> Vec<MetaPluginType> {
let mut meta_plugin_types = Vec::new(); let mut meta_plugin_types = Vec::new();
// Handle comma-separated values in each meta_plugins argument // Handle comma-separated values in each meta_plugins argument
@@ -232,7 +233,7 @@ pub fn settings_meta_plugin_types(cmd: &mut Command, settings: &crate::config::S
meta_plugin_types meta_plugin_types
} }
pub fn settings_digest_type(cmd: &mut Command, settings: &crate::config::Settings) -> MetaPluginType { pub fn settings_digest_type(cmd: &mut Command, settings: &config::Settings) -> MetaPluginType {
let digest_name = settings let digest_name = settings
.digest .digest
.clone() .clone()
@@ -250,7 +251,7 @@ pub fn settings_digest_type(cmd: &mut Command, settings: &crate::config::Setting
digest_type_opt.unwrap() digest_type_opt.unwrap()
} }
pub fn settings_compression_type(cmd: &mut Command, settings: &crate::config::Settings) -> CompressionType { pub fn settings_compression_type(cmd: &mut Command, settings: &config::Settings) -> CompressionType {
let compression_name = settings let compression_name = settings
.compression .compression
.clone() .clone()
@@ -268,7 +269,7 @@ pub fn settings_compression_type(cmd: &mut Command, settings: &crate::config::Se
compression_type_opt.unwrap() compression_type_opt.unwrap()
} }
pub fn settings_output_format(settings: &crate::config::Settings) -> OutputFormat { pub fn settings_output_format(settings: &config::Settings) -> OutputFormat {
settings.output_format settings.output_format
.as_ref() .as_ref()
.and_then(|s| OutputFormat::from_str(s).ok()) .and_then(|s| OutputFormat::from_str(s).ok())

View File

@@ -3,6 +3,7 @@ use std::fs;
use std::path::PathBuf; use std::path::PathBuf;
use crate::db; use crate::db;
use crate::config;
use clap::Command; use clap::Command;
use clap::error::ErrorKind; use clap::error::ErrorKind;
use log::{debug, warn}; use log::{debug, warn};
@@ -10,8 +11,8 @@ use rusqlite::Connection;
pub fn mode_delete( pub fn mode_delete(
cmd: &mut Command, cmd: &mut Command,
_settings: &crate::config::Settings, _settings: &config::Settings,
_config: &crate::config::Config, _config: &config::Config,
ids: &mut Vec<i64>, ids: &mut Vec<i64>,
tags: &mut Vec<String>, tags: &mut Vec<String>,
conn: &mut Connection, conn: &mut Connection,

View File

@@ -3,6 +3,7 @@ use clap::Command;
use std::io::Read; use std::io::Read;
use std::os::fd::FromRawFd; use std::os::fd::FromRawFd;
use std::str::FromStr; use std::str::FromStr;
use crate::config;
fn validate_diff_args(cmd: &mut Command, ids: &Vec<i64>, tags: &Vec<String>) { fn validate_diff_args(cmd: &mut Command, ids: &Vec<i64>, tags: &Vec<String>) {
if !tags.is_empty() { if !tags.is_empty() {
@@ -293,8 +294,8 @@ fn handle_diff_output(
pub fn mode_diff( pub fn mode_diff(
cmd: &mut Command, cmd: &mut Command,
_settings: &crate::config::Settings, _settings: &config::Settings,
_config: &crate::config::Config, _config: &config::Config,
ids: &mut Vec<i64>, ids: &mut Vec<i64>,
tags: &mut Vec<String>, tags: &mut Vec<String>,
conn: &mut rusqlite::Connection, conn: &mut rusqlite::Connection,

View File

@@ -3,6 +3,7 @@ use std::io::{Read, Write};
use crate::compression_engine::{CompressionType, get_compression_engine}; use crate::compression_engine::{CompressionType, get_compression_engine};
use crate::common::is_binary::is_binary; use crate::common::is_binary::is_binary;
use crate::config;
use clap::Command; use clap::Command;
use std::path::PathBuf; use std::path::PathBuf;
use std::str::FromStr; use std::str::FromStr;
@@ -10,8 +11,8 @@ use is_terminal::IsTerminal;
pub fn mode_get( pub fn mode_get(
cmd: &mut Command, cmd: &mut Command,
settings: &crate::config::Settings, settings: &config::Settings,
_config: &crate::config::Config, _config: &config::Config,
ids: &mut Vec<i64>, ids: &mut Vec<i64>,
tags: &mut Vec<String>, tags: &mut Vec<String>,
conn: &mut rusqlite::Connection, conn: &mut rusqlite::Connection,

View File

@@ -1,5 +1,6 @@
use crate::db::Item; use crate::db::Item;
use crate::modes::common::{format_size, get_output_format, OutputFormat}; use crate::modes::common::{format_size, OutputFormat};
use crate::config;
use anyhow::anyhow; use anyhow::anyhow;
use serde_json; use serde_json;
use serde_yaml; use serde_yaml;
@@ -19,8 +20,8 @@ use prettytable::{Attr, Cell, Row, Table};
pub fn mode_info( pub fn mode_info(
cmd: &mut Command, cmd: &mut Command,
settings: &crate::config::Settings, settings: &config::Settings,
_config: &crate::config::Config, _config: &config::Config,
ids: &mut Vec<i64>, ids: &mut Vec<i64>,
tags: &mut Vec<String>, tags: &mut Vec<String>,
conn: &mut rusqlite::Connection, conn: &mut rusqlite::Connection,
@@ -68,7 +69,7 @@ struct ItemInfo {
fn show_item( fn show_item(
item: Item, // Using the provided struct definition item: Item, // Using the provided struct definition
settings: &crate::config::Settings, settings: &config::Settings,
conn: &mut rusqlite::Connection, conn: &mut rusqlite::Connection,
data_path: PathBuf, data_path: PathBuf,
) -> anyhow::Result<()> { ) -> anyhow::Result<()> {
@@ -163,7 +164,7 @@ fn show_item(
fn show_item_structured( fn show_item_structured(
item: Item, item: Item,
settings: &crate::config::Settings, settings: &config::Settings,
conn: &mut rusqlite::Connection, conn: &mut rusqlite::Connection,
data_path: PathBuf, data_path: PathBuf,
output_format: OutputFormat, output_format: OutputFormat,

View File

@@ -1,6 +1,7 @@
use crate::db::{get_items, get_items_matching}; use crate::db::{get_items, get_items_matching};
use crate::modes::common::ColumnType; use crate::modes::common::ColumnType;
use crate::modes::common::{size_column, string_column, get_output_format, OutputFormat}; use crate::modes::common::{size_column, string_column, OutputFormat};
use crate::config;
use serde::{Deserialize, Serialize}; use serde::{Deserialize, Serialize};
use serde_json; use serde_json;
use serde_yaml; use serde_yaml;
@@ -27,8 +28,8 @@ struct ListItem {
pub fn mode_list( pub fn mode_list(
cmd: &mut clap::Command, cmd: &mut clap::Command,
settings: &crate::config::Settings, settings: &config::Settings,
_config: &crate::config::Config, _config: &config::Config,
ids: &mut Vec<i64>, ids: &mut Vec<i64>,
tags: &Vec<String>, tags: &Vec<String>,
conn: &mut rusqlite::Connection, conn: &mut rusqlite::Connection,
@@ -196,7 +197,7 @@ fn show_list_structured(
tags_by_item: std::collections::HashMap<i64, Vec<String>>, tags_by_item: std::collections::HashMap<i64, Vec<String>>,
meta_by_item: std::collections::HashMap<i64, std::collections::HashMap<String, String>>, meta_by_item: std::collections::HashMap<i64, std::collections::HashMap<String, String>>,
data_path: std::path::PathBuf, data_path: std::path::PathBuf,
settings: &crate::config::Settings, settings: &config::Settings,
output_format: OutputFormat, output_format: OutputFormat,
) -> anyhow::Result<()> { ) -> anyhow::Result<()> {
let mut list_items = Vec::new(); let mut list_items = Vec::new();

View File

@@ -5,6 +5,7 @@ use std::io::{Read, Write, IsTerminal};
// Import the missing functions from common module // Import the missing functions from common module
use crate::modes::common::{settings_digest_type, settings_compression_type, settings_meta_plugin_types}; use crate::modes::common::{settings_digest_type, settings_compression_type, settings_meta_plugin_types};
use crate::config;
fn validate_save_args(cmd: &mut Command, ids: &Vec<i64>) { fn validate_save_args(cmd: &mut Command, ids: &Vec<i64>) {
if !ids.is_empty() { if !ids.is_empty() {
@@ -24,7 +25,7 @@ fn initialize_tags(tags: &mut Vec<String>) {
fn setup_compression_and_plugins( fn setup_compression_and_plugins(
cmd: &mut Command, cmd: &mut Command,
settings: &crate::config::Settings, settings: &config::Settings,
) -> (crate::compression_engine::CompressionType, Box<dyn crate::compression_engine::CompressionEngine>, Vec<Box<dyn crate::meta_plugin::MetaPlugin>>) { ) -> (crate::compression_engine::CompressionType, Box<dyn crate::compression_engine::CompressionEngine>, Vec<Box<dyn crate::meta_plugin::MetaPlugin>>) {
let digest_type = settings_digest_type(cmd, settings); let digest_type = settings_digest_type(cmd, settings);
debug!("MAIN: Digest type: {:?}", digest_type); debug!("MAIN: Digest type: {:?}", digest_type);
@@ -78,7 +79,7 @@ fn setup_compression_and_plugins(
fn create_and_log_item( fn create_and_log_item(
conn: &mut rusqlite::Connection, conn: &mut rusqlite::Connection,
settings: &crate::config::Settings, settings: &config::Settings,
tags: &Vec<String>, tags: &Vec<String>,
compression_type: &crate::compression_engine::CompressionType, compression_type: &crate::compression_engine::CompressionType,
) -> Result<crate::db::Item, anyhow::Error> { ) -> Result<crate::db::Item, anyhow::Error> {
@@ -121,7 +122,7 @@ fn create_and_log_item(
fn setup_item_metadata( fn setup_item_metadata(
conn: &mut rusqlite::Connection, conn: &mut rusqlite::Connection,
_settings: &crate::config::Settings, _settings: &config::Settings,
item: &crate::db::Item, item: &crate::db::Item,
tags: &Vec<String>, tags: &Vec<String>,
) -> Result<(), anyhow::Error> { ) -> Result<(), anyhow::Error> {
@@ -129,7 +130,7 @@ fn setup_item_metadata(
Ok(()) Ok(())
} }
fn collect_item_meta(settings: &crate::config::Settings) -> std::collections::HashMap<String, String> { fn collect_item_meta(settings: &config::Settings) -> std::collections::HashMap<String, String> {
let mut item_meta: std::collections::HashMap<String, String> = crate::modes::common::get_meta_from_env(); let mut item_meta: std::collections::HashMap<String, String> = crate::modes::common::get_meta_from_env();
if let Ok(hostname) = gethostname::gethostname().into_string() { if let Ok(hostname) = gethostname::gethostname().into_string() {
@@ -230,8 +231,8 @@ fn finalize_meta_plugins(
pub fn mode_save( pub fn mode_save(
cmd: &mut Command, cmd: &mut Command,
settings: &crate::config::Settings, settings: &config::Settings,
_config: &crate::config::Config, _config: &config::Config,
ids: &mut Vec<i64>, ids: &mut Vec<i64>,
tags: &mut Vec<String>, tags: &mut Vec<String>,
conn: &mut rusqlite::Connection, conn: &mut rusqlite::Connection,

View File

@@ -2,7 +2,7 @@ use anyhow::Result;
use axum::{ use axum::{
Router, Router,
}; };
use clap::Command; use clap::{Command, Parser};
use log::{debug, info, warn}; use log::{debug, info, warn};
use std::net::SocketAddr; use std::net::SocketAddr;
use std::path::PathBuf; use std::path::PathBuf;
@@ -11,6 +11,7 @@ use tokio::sync::Mutex;
use tower_http::cors::CorsLayer; use tower_http::cors::CorsLayer;
use tower::ServiceBuilder; use tower::ServiceBuilder;
use tower_http::trace::TraceLayer; use tower_http::trace::TraceLayer;
use crate::config;
pub mod common; pub mod common;
mod api; mod api;
@@ -20,8 +21,8 @@ pub use common::{ServerConfig, AppState, logging_middleware, create_auth_middlew
pub fn mode_server( pub fn mode_server(
_cmd: &mut Command, _cmd: &mut Command,
settings: &crate::config::Settings, settings: &config::Settings,
config: &crate::config::Config, config: &config::Config,
conn: &mut rusqlite::Connection, conn: &mut rusqlite::Connection,
data_path: PathBuf, data_path: PathBuf,
) -> Result<()> { ) -> Result<()> {

View File

@@ -2,7 +2,8 @@ use clap::*;
use is_terminal::IsTerminal; use is_terminal::IsTerminal;
use std::path::PathBuf; use std::path::PathBuf;
use crate::modes::common::{get_format_box_chars_no_border_line_separator, get_output_format, OutputFormat}; use crate::modes::common::{get_format_box_chars_no_border_line_separator, OutputFormat};
use crate::config;
use prettytable::color; use prettytable::color;
use serde_json; use serde_json;
use serde_yaml; use serde_yaml;
@@ -119,8 +120,8 @@ fn build_meta_plugin_table(meta_plugin_info: &Vec<MetaPluginInfo>) -> Table {
pub fn mode_status( pub fn mode_status(
_cmd: &mut Command, _cmd: &mut Command,
settings: &crate::config::Settings, settings: &config::Settings,
_config: &crate::config::Config, _config: &config::Config,
data_path: PathBuf, data_path: PathBuf,
db_path: PathBuf, db_path: PathBuf,
) -> Result<(), anyhow::Error> { ) -> Result<(), anyhow::Error> {

View File

@@ -5,7 +5,8 @@ use std::str::FromStr;
use crate::compression_engine::{CompressionType, get_compression_engine}; use crate::compression_engine::{CompressionType, get_compression_engine};
use crate::db; use crate::db;
use crate::meta_plugin; use crate::meta_plugin;
use crate::modes::common::{cmd_args_digest_type, get_digest_type_meta}; use crate::modes::common::get_digest_type_meta;
use crate::config;
use clap::Command; use clap::Command;
use clap::error::ErrorKind; use clap::error::ErrorKind;
use log::{debug, info}; use log::{debug, info};
@@ -13,8 +14,8 @@ use rusqlite::Connection;
pub fn mode_update( pub fn mode_update(
cmd: &mut Command, cmd: &mut Command,
settings: &crate::config::Settings, settings: &config::Settings,
_config: &crate::config::Config, _config: &config::Config,
ids: &mut Vec<i64>, ids: &mut Vec<i64>,
tags: &mut Vec<String>, tags: &mut Vec<String>,
conn: &mut Connection, conn: &mut Connection,