From 07ece7d5a7513201289e35c3ac5b59f4d9023166 Mon Sep 17 00:00:00 2001 From: Douwe Osinga Date: Tue, 25 Nov 2025 12:04:54 +0100 Subject: [PATCH] Revert "fix: do not load active extensions when no extensions in the recipe" (#5871) Signed-off-by: Douwe Osinga Signed-off-by: Sam Osborn Co-authored-by: Douwe Osinga 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 I don't understand why I get the DCO warnings --- crates/goose-cli/src/recipes/extract_from_cli.rs | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/crates/goose-cli/src/recipes/extract_from_cli.rs b/crates/goose-cli/src/recipes/extract_from_cli.rs index ff77fb27..02546c0f 100644 --- a/crates/goose-cli/src/recipes/extract_from_cli.rs +++ b/crates/goose-cli/src/recipes/extract_from_cli.rs @@ -52,7 +52,7 @@ pub fn extract_recipe_info_from_cli( } let input_config = InputConfig { 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, }; @@ -109,8 +109,7 @@ mod tests { input_config.additional_system_prompt, Some("test_instructions my_value".to_string()) ); - assert!(input_config.extensions_override.is_some()); - assert!(input_config.extensions_override.unwrap().is_empty()); + assert!(input_config.extensions_override.is_none()); assert!(settings.is_some()); let settings = settings.unwrap(); @@ -175,8 +174,7 @@ mod tests { input_config.additional_system_prompt, Some("test_instructions my_value".to_string()) ); - assert!(input_config.extensions_override.is_some()); - assert!(input_config.extensions_override.unwrap().is_empty()); + assert!(input_config.extensions_override.is_none()); assert!(settings.is_some()); let settings = settings.unwrap();