From 63b5b854763105f0a3c3b3ae4fbaca35b67b79eb Mon Sep 17 00:00:00 2001
From: Andrew Phillips <andrew.phillips2@canada.ca>
Date: Wed, 3 Sep 2025 11:25:06 -0300
Subject: [PATCH] fix: Correctly serialize filter plugin options for status
 display

Co-authored-by: aider (openai/andrew/openrouter/deepseek/deepseek-chat-v3.1) <aider@aider.chat>
---
 src/modes/status_plugins.rs | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/modes/status_plugins.rs b/src/modes/status_plugins.rs
index 219c5a7..6bfce84 100644
--- a/src/modes/status_plugins.rs
+++ b/src/modes/status_plugins.rs
@@ -143,20 +143,20 @@ fn build_filter_plugin_table(filter_plugins: &Vec<crate::common::status::FilterP
         let options_str = if plugin_info.options.is_empty() {
             "{}".to_string()
         } else {
-            // Convert options to a map for better display
-            let mut options_map = std::collections::BTreeMap::new();
+            // Convert options to a proper structure for display
+            let mut options_list = Vec::new();
             for opt in &plugin_info.options {
                 let mut opt_map = std::collections::BTreeMap::new();
                 opt_map.insert("name".to_string(), serde_yaml::Value::String(opt.name.clone()));
                 if let Some(default) = &opt.default {
-                    opt_map.insert("default".to_string(), serde_yaml::Value::String(format!("{:?}", default)));
+                    opt_map.insert("default".to_string(), default.clone());
                 } else {
-                    opt_map.insert("default".to_string(), serde_yaml::Value::String("None".to_string()));
+                    opt_map.insert("default".to_string(), serde_yaml::Value::Null);
                 }
-                opt_map.insert("required".to_string(), serde_yaml::Value::String(opt.required.to_string()));
-                options_map.insert(opt.name.clone(), serde_yaml::Value::String(format!("{:?}", opt_map)));
+                opt_map.insert("required".to_string(), serde_yaml::Value::Bool(opt.required));
+                options_list.push(serde_yaml::Value::Mapping(opt_map));
             }
-            serde_yaml::to_string(&options_map)
+            serde_yaml::to_string(&serde_yaml::Value::Sequence(options_list))
                 .unwrap_or_else(|_| "Unable to serialize options".to_string())
                 .trim()
                 .to_string()