Revert "fix: do not load active extensions when no extensions in the recipe" (#5871)

Signed-off-by: Douwe Osinga <douwe@squareup.com>
Signed-off-by: Sam Osborn <samosborn@squareup.com>
Co-authored-by: Douwe Osinga <douwe@squareup.com>
Co-authored-by: dianed-square <73617011+dianed-square@users.noreply.github.com>
Co-authored-by: Sam Osborn <108367660+sam-at-block@users.noreply.github.com>
Co-authored-by: Michael Neale <michael.neale@gmail.com>

I don't understand why I get the DCO warnings
This commit is contained in:
Douwe Osinga
2025-11-25 12:04:54 +01:00
committed by GitHub
parent 08d0a7cb20
commit 07ece7d5a7
@@ -52,7 +52,7 @@ pub fn extract_recipe_info_from_cli(
} }
let input_config = InputConfig { let input_config = InputConfig {
contents: recipe.prompt.filter(|s| !s.trim().is_empty()), contents: recipe.prompt.filter(|s| !s.trim().is_empty()),
extensions_override: recipe.extensions.or(Some(vec![])), extensions_override: recipe.extensions,
additional_system_prompt: recipe.instructions, additional_system_prompt: recipe.instructions,
}; };
@@ -109,8 +109,7 @@ mod tests {
input_config.additional_system_prompt, input_config.additional_system_prompt,
Some("test_instructions my_value".to_string()) Some("test_instructions my_value".to_string())
); );
assert!(input_config.extensions_override.is_some()); assert!(input_config.extensions_override.is_none());
assert!(input_config.extensions_override.unwrap().is_empty());
assert!(settings.is_some()); assert!(settings.is_some());
let settings = settings.unwrap(); let settings = settings.unwrap();
@@ -175,8 +174,7 @@ mod tests {
input_config.additional_system_prompt, input_config.additional_system_prompt,
Some("test_instructions my_value".to_string()) Some("test_instructions my_value".to_string())
); );
assert!(input_config.extensions_override.is_some()); assert!(input_config.extensions_override.is_none());
assert!(input_config.extensions_override.unwrap().is_empty());
assert!(settings.is_some()); assert!(settings.is_some());
let settings = settings.unwrap(); let settings = settings.unwrap();