keep the order of keys in config.yaml

crates/goose-cli/src/commands/info.rs

@@ -40,26 +40,22 @@ pub fn handle_info(verbose: bool) -> Result<()> {
     // Print verbose info if requested
     if verbose {
         println!("\n{}", style("goose Configuration:").cyan().bold());
-        match config.load_values() {
-            Ok(values) => {
-                if values.is_empty() {
-                    println!("  No configuration values set");
-                    println!(
-                        "  Run '{}' to configure goose",
-                        style("goose configure").cyan()
-                    );
-                } else {
-                    let sorted_values: std::collections::BTreeMap<_, _> =
-                        values.iter().map(|(k, v)| (k.clone(), v.clone())).collect();
-
-                    if let Ok(yaml) = serde_yaml::to_string(&sorted_values) {
-                        for line in yaml.lines() {
-                            println!("  {}", line);
-                        }
-                    }
+        let values = config.all_values()?;
+        if values.is_empty() {
+            println!("  No configuration values set");
+            println!(
+                "  Run '{}' to configure goose",
+                style("goose configure").cyan()
+            );
+        } else {
+            let sorted_values: std::collections::BTreeMap<_, _> =
+                values.iter().map(|(k, v)| (k.clone(), v.clone())).collect();
+
+            if let Ok(yaml) = serde_yaml::to_string(&sorted_values) {
+                for line in yaml.lines() {
+                    println!("  {}", line);
                 }
             }
-            Err(e) => println!("  Error loading configuration: {}", e),
         }
     }
 

crates/goose-server/src/routes/config_management.rs

@@ -272,7 +272,7 @@ pub async fn read_all_config() -> Result<Json<ConfigResponse>, StatusCode> {
     let config = Config::global();
 
     let values = config
-        .load_values()
+        .all_values()
         .map_err(|_| StatusCode::UNPROCESSABLE_ENTITY)?;
 
     Ok(Json(ConfigResponse { config: values }))
@@ -509,7 +509,7 @@ pub async fn init_config() -> Result<Json<String>, StatusCode> {
 
     // Use the shared function to load init-config.yaml
     match goose::config::base::load_init_config_from_workspace() {
-        Ok(init_values) => match config.save_values(init_values) {
+        Ok(init_values) => match config.initialize_if_empty(init_values) {
             Ok(_) => Ok(Json("Config initialized successfully".to_string())),
             Err(_) => Err(StatusCode::INTERNAL_SERVER_ERROR),
         },
@@ -584,7 +584,7 @@ pub async fn recover_config() -> Result<Json<String>, StatusCode> {
     let config = Config::global();
 
     // Force a reload which will trigger recovery if needed
-    match config.load_values() {
+    match config.all_values() {
         Ok(values) => {
             let recovered_keys: Vec<String> = values.keys().cloned().collect();
             if recovered_keys.is_empty() {

crates/goose/Cargo.toml

@@ -108,6 +108,7 @@ schemars = { version = "1.0.4", default-features = false, features = ["derive"]
 insta = "1.43.2"
 paste = "1.0.0"
 shellexpand = "3.1.1"
+indexmap = "2.12.0"
 
 
 [target.'cfg(target_os = "windows")'.dependencies]

crates/goose/src/agents/extension.rs

@@ -215,6 +215,8 @@ pub enum ExtensionConfig {
     Sse {
         /// The name used to identify this extension
         name: String,
+        #[serde(default)]
+        #[schema(required)]
         description: String,
         uri: String,
         #[serde(default)]
@@ -234,6 +236,8 @@ pub enum ExtensionConfig {
     Stdio {
         /// The name used to identify this extension
         name: String,
+        #[serde(default)]
+        #[schema(required)]
         description: String,
         cmd: String,
         args: Vec<String>,
@@ -252,6 +256,8 @@ pub enum ExtensionConfig {
     Builtin {
         /// The name used to identify this extension
         name: String,
+        #[serde(default)]
+        #[schema(required)]
         description: String,
         display_name: Option<String>, // needed for the UI
         timeout: Option<u64>,
@@ -265,6 +271,8 @@ pub enum ExtensionConfig {
     Platform {
         /// The name used to identify this extension
         name: String,
+        #[serde(default)]
+        #[schema(required)]
         description: String,
         #[serde(default)]
         bundled: Option<bool>,
@@ -276,6 +284,8 @@ pub enum ExtensionConfig {
     StreamableHttp {
         /// The name used to identify this extension
         name: String,
+        #[serde(default)]
+        #[schema(required)]
         description: String,
         uri: String,
         #[serde(default)]
@@ -297,6 +307,8 @@ pub enum ExtensionConfig {
     Frontend {
         /// The name used to identify this extension
         name: String,
+        #[serde(default)]
+        #[schema(required)]
         description: String,
         /// The tools provided by the frontend
         tools: Vec<Tool>,
@@ -312,6 +324,8 @@ pub enum ExtensionConfig {
     InlinePython {
         /// The name used to identify this extension
         name: String,
+        #[serde(default)]
+        #[schema(required)]
         description: String,
         /// The Python code to execute
         code: String,