asp/keep
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: remove redundant default options merge and use effective_options directly

Browse Source 
Co-authored-by: aider (openai/andrew/openrouter/deepseek/deepseek-chat-v3.1) <aider@aider.chat>
This commit is contained in:
Andrew Phillips
2025-08-27 17:00:04 -03:00
parent 3e8d4b223b
commit 982c73dc03
1 changed files with 1 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/modes/status.rs
View File

@@ -194,13 +194,7 @@ fn build_meta_plugins_configured_table(settings: &config::Settings) -> Option<Ta
None, None,
); );
// Merge in default options from the plugin implementation // Use the effective_options which already includes both defaults and configured options
for (key, value) in default_plugin.options() {
if !options_to_serialize.contains_key(key) {
options_to_serialize.insert(key.clone(), value.clone());
}
}
let options_str = if effective_options.is_empty() { let options_str = if effective_options.is_empty() {
"{}".to_string() "{}".to_string()
} else { } else {