fix: serialize skills config in tests

codex-rs/core/src/agent/role.rs

@@ -235,6 +235,7 @@ mod tests {
     use std::path::PathBuf;
     use std::sync::Arc;
     use tempfile::TempDir;
+    use toml::value::Table;
 
     async fn test_config_with_cli_overrides(
         cli_overrides: Vec<(String, TomlValue)>,
@@ -486,18 +487,22 @@ writable_roots = ["./sandbox-root"]
             "---\nname: demo-skill\ndescription: demo description\n---\n\n# Body\n",
         )
         .expect("write skill");
-        let role_path = write_role_config(
-            &home,
-            "skills-role.toml",
-            &format!(
-                r#"[[skills.config]]
-path = "{}"
-enabled = false
-"#,
-                skill_path.display()
-            ),
-        )
-        .await;
+        let mut skill_entry = Table::new();
+        skill_entry.insert(
+            "path".to_string(),
+            TomlValue::String(skill_path.display().to_string()),
+        );
+        skill_entry.insert("enabled".to_string(), TomlValue::Boolean(false));
+        let mut skills_table = Table::new();
+        skills_table.insert(
+            "config".to_string(),
+            TomlValue::Array(vec![TomlValue::Table(skill_entry)]),
+        );
+        let mut role_table = Table::new();
+        role_table.insert("skills".to_string(), TomlValue::Table(skills_table));
+        let role_config_contents =
+            toml::to_string(&TomlValue::Table(role_table)).expect("serialize role config");
+        let role_path = write_role_config(&home, "skills-role.toml", &role_config_contents).await;
         config.agent_roles.insert(
             "custom".to_string(),
             AgentRoleConfig {

codex-rs/core/src/skills/manager.rs

@@ -259,6 +259,7 @@ mod tests {
     use std::fs;
     use std::path::PathBuf;
     use tempfile::TempDir;
+    use toml::value::Table;
 
     fn write_user_skill(codex_home: &TempDir, dir: &str, name: &str, description: &str) {
         let skill_dir = codex_home.path().join("skills").join(dir);
@@ -267,6 +268,23 @@ mod tests {
         fs::write(skill_dir.join("SKILL.md"), content).unwrap();
     }
 
+    fn skills_config_layer_toml(skill_path: &Path, enabled: bool) -> TomlValue {
+        let mut skill_entry = Table::new();
+        skill_entry.insert(
+            "path".to_string(),
+            TomlValue::String(skill_path.display().to_string()),
+        );
+        skill_entry.insert("enabled".to_string(), TomlValue::Boolean(enabled));
+        let mut skills_table = Table::new();
+        skills_table.insert(
+            "config".to_string(),
+            TomlValue::Array(vec![TomlValue::Table(skill_entry)]),
+        );
+        let mut root_table = Table::new();
+        root_table.insert("skills".to_string(), TomlValue::Table(skills_table));
+        TomlValue::Table(root_table)
+    }
+
     #[tokio::test]
     async fn skills_for_config_seeds_cache_by_cwd() {
         let codex_home = tempfile::tempdir().expect("tempdir");
@@ -450,25 +468,11 @@ mod tests {
             .expect("user config path should be absolute");
         let user_layer = ConfigLayerEntry::new(
             ConfigLayerSource::User { file: user_file },
-            toml::from_str(&format!(
-                r#"[[skills.config]]
-path = "{}"
-enabled = false
-"#,
-                skill_path.display()
-            ))
-            .expect("user layer toml"),
+            skills_config_layer_toml(&skill_path, false),
         );
         let session_layer = ConfigLayerEntry::new(
             ConfigLayerSource::SessionFlags,
-            toml::from_str(&format!(
-                r#"[[skills.config]]
-path = "{}"
-enabled = true
-"#,
-                skill_path.display()
-            ))
-            .expect("session layer toml"),
+            skills_config_layer_toml(&skill_path, true),
         );
         let stack = ConfigLayerStack::new(
             vec![user_layer, session_layer],
@@ -489,25 +493,11 @@ enabled = true
             .expect("user config path should be absolute");
         let user_layer = ConfigLayerEntry::new(
             ConfigLayerSource::User { file: user_file },
-            toml::from_str(&format!(
-                r#"[[skills.config]]
-path = "{}"
-enabled = true
-"#,
-                skill_path.display()
-            ))
-            .expect("user layer toml"),
+            skills_config_layer_toml(&skill_path, true),
         );
         let session_layer = ConfigLayerEntry::new(
             ConfigLayerSource::SessionFlags,
-            toml::from_str(&format!(
-                r#"[[skills.config]]
-path = "{}"
-enabled = false
-"#,
-                skill_path.display()
-            ))
-            .expect("session layer toml"),
+            skills_config_layer_toml(&skill_path, false),
         );
         let stack = ConfigLayerStack::new(
             vec![user_layer, session_layer],