From f4e2f8ebba09828e0599b73008b3b707d91ca39c Mon Sep 17 00:00:00 2001 From: Michael Bolin Date: Wed, 11 Feb 2026 21:26:47 -0800 Subject: [PATCH] fix: skills_render_without_project_doc and skills_are_appended_to_project_doc tests affected by live data --- codex-rs/core/src/project_doc.rs | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/codex-rs/core/src/project_doc.rs b/codex-rs/core/src/project_doc.rs index 5fcff4f7da..5dc17a51eb 100644 --- a/codex-rs/core/src/project_doc.rs +++ b/codex-rs/core/src/project_doc.rs @@ -263,7 +263,9 @@ mod tests { use super::*; use crate::config::ConfigBuilder; use crate::features::Feature; - use crate::skills::load_skills; + use crate::skills::loader::SkillRoot; + use crate::skills::loader::load_skills_from_roots; + use codex_protocol::protocol::SkillScope; use std::fs; use std::path::PathBuf; use tempfile::TempDir; @@ -302,6 +304,13 @@ mod tests { config } + fn load_test_skills(config: &Config) -> crate::skills::SkillLoadOutcome { + load_skills_from_roots([SkillRoot { + path: config.codex_home.join("skills"), + scope: SkillScope::User, + }]) + } + /// AGENTS.md missing – should yield `None`. #[tokio::test] async fn no_doc_file_returns_none() { @@ -542,7 +551,7 @@ mod tests { "extract from pdfs", ); - let skills = load_skills(&cfg); + let skills = load_test_skills(&cfg); let res = get_user_instructions( &cfg, skills.errors.is_empty().then_some(skills.skills.as_slice()), @@ -569,7 +578,7 @@ mod tests { let cfg = make_config(&tmp, 4096, None).await; create_skill(cfg.codex_home.clone(), "linting", "run clippy"); - let skills = load_skills(&cfg); + let skills = load_test_skills(&cfg); let res = get_user_instructions( &cfg, skills.errors.is_empty().then_some(skills.skills.as_slice()),