From 05af5603996798a8bdb0532c606a43c8003c4fd0 Mon Sep 17 00:00:00 2001 From: Andrew Phillips Date: Tue, 29 Jul 2025 11:46:02 -0300 Subject: [PATCH] refactor: remove default column from meta plugin status table and simplify default logic Co-authored-by: aider (openai/andrew.openrouter.qwen.qwen3-coder) --- src/meta_plugin.rs | 12 +++--------- src/modes/status.rs | 8 -------- 2 files changed, 3 insertions(+), 17 deletions(-) diff --git a/src/meta_plugin.rs b/src/meta_plugin.rs index 39867fd..6fb83b2 100644 --- a/src/meta_plugin.rs +++ b/src/meta_plugin.rs @@ -70,13 +70,7 @@ pub fn get_meta_plugin(meta_plugin_type: MetaPluginType) -> Box } pub fn default_meta_plugin_type() -> MetaPluginType { - let mut default = MetaPluginType::FileMagic; - for meta_plugin_type in MetaPluginType::iter() { - let meta_plugin = get_meta_plugin(meta_plugin_type.clone()); - if meta_plugin.is_supported() { - default = meta_plugin_type; - break; - } - } - default + // Default meta plugin functionality has been removed + // This function returns a placeholder value + MetaPluginType::FileMagic } diff --git a/src/modes/status.rs b/src/modes/status.rs index 83f4615..dcfa2ac 100644 --- a/src/modes/status.rs +++ b/src/modes/status.rs @@ -124,17 +124,13 @@ fn build_meta_plugin_table() -> Table { meta_plugin_table.set_titles(row!( b->"Type", b->"Found", - b->"Default", b->"Binary", b->"Args", b->"Meta Name")); - let default_type = meta_plugin::default_meta_plugin_type(); - for meta_plugin_type in MetaPluginType::iter() { let mut meta_plugin = meta_plugin::get_meta_plugin(meta_plugin_type.clone()); let is_supported = meta_plugin.is_supported(); - let is_default = meta_plugin_type == default_type; // Get program info for display purposes let meta_plugin_program: Option = META_PLUGIN_PROGRAMS[meta_plugin_type.clone()].clone(); @@ -166,10 +162,6 @@ fn build_meta_plugin_table() -> Table { true => Cell::new("Yes").with_style(Attr::ForegroundColor(color::GREEN)), false => Cell::new("No").with_style(Attr::ForegroundColor(color::RED)), }, - match is_default { - true => Cell::new("Yes").with_style(Attr::ForegroundColor(color::GREEN)), - false => Cell::new("No"), - }, Cell::new(&binary_display), Cell::new(&args_display), Cell::new(&meta_plugin.meta_name()),