From b802c4f6f052df54d74c2d4455d38ecd4176cb80 Mon Sep 17 00:00:00 2001 From: Andrew Phillips Date: Wed, 27 Aug 2025 17:08:57 -0300 Subject: [PATCH] fix: resolve moved value errors and unused variable warnings Co-authored-by: aider (openai/andrew/openrouter/deepseek/deepseek-chat-v3.1) --- src/modes/status.rs | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/src/modes/status.rs b/src/modes/status.rs index f87f99b..cfcd6c1 100644 --- a/src/modes/status.rs +++ b/src/modes/status.rs @@ -153,47 +153,47 @@ fn build_meta_plugins_configured_table(settings: &config::Settings) -> Option plugin_type, Err(_) => continue, }; - + // First, create a default plugin to get its default options let default_plugin = get_meta_plugin( - meta_plugin_type, + meta_plugin_type.clone(), None, None, ); - + // Start with the default options let mut effective_options = default_plugin.options().clone(); - + // Merge with the configured options for (key, value) in &plugin_config.options { effective_options.insert(key.clone(), value.clone()); } - + // Convert outputs from HashMap to HashMap let outputs_converted: std::collections::HashMap = plugin_config.outputs .iter() .map(|(k, v)| (k.clone(), serde_yaml::Value::String(v.clone()))) .collect(); - + // Create the actual plugin with merged options let actual_plugin = get_meta_plugin( - meta_plugin_type, + meta_plugin_type.clone(), Some(effective_options.clone()), Some(outputs_converted), ); - + // Show the merged options including defaults set by the plugin // Convert options to a YAML string, but handle formatting better - let mut options_to_serialize = actual_plugin.options().clone(); - + // Note: actual_plugin.options() is not used here, so we don't need to store it + // For plugins that set default options programmatically, we need to include them // Let's get the default plugin to see what options it would have - let default_plugin = get_meta_plugin( - meta_plugin_type, + let _default_plugin = get_meta_plugin( + meta_plugin_type.clone(), None, None, ); - + // Use the effective_options which already includes both defaults and configured options let options_str = if effective_options.is_empty() { "{}".to_string()