feat: decided to make skills persist to disk like agents and not in-memory like built-in roles

src/config/mod.rs

@@ -77,7 +77,6 @@ const LIGHT_THEME: &[u8] = include_bytes!("../../assets/monokai-extended-light.t
 const CONFIG_FILE_NAME: &str = "config.yaml";
 const AGENT_GRAPH_FILE_NAME: &str = "graph.yaml";
 const ROLES_DIR_NAME: &str = "roles";
-#[allow(dead_code)]
 const SKILLS_DIR_NAME: &str = "skills";
 const MACROS_DIR_NAME: &str = "macros";
 const ENV_FILE_NAME: &str = ".env";

src/config/paths.rs

@@ -3,7 +3,7 @@ use super::{
     AGENT_GRAPH_FILE_NAME, AGENTS_DIR_NAME, BASH_PROMPT_UTILS_FILE_NAME, CONFIG_FILE_NAME,
     ENV_FILE_NAME, FUNCTIONS_BIN_DIR_NAME, FUNCTIONS_DIR_NAME, GLOBAL_TOOLS_DIR_NAME,
     GLOBAL_TOOLS_UTILS_DIR_NAME, MACROS_DIR_NAME, MCP_FILE_NAME, ModelsOverride, RAGS_DIR_NAME,
-    ROLES_DIR_NAME,
+    ROLES_DIR_NAME, SKILLS_DIR_NAME,
 };
 use crate::client::ProviderModels;
 use crate::utils::{get_env_name, list_file_names, normalize_env_name};
@@ -65,6 +65,24 @@ pub fn role_file(name: &str) -> PathBuf {
     roles_dir().join(format!("{name}.md"))
 }
 
+#[allow(dead_code)]
+pub fn skills_dir() -> PathBuf {
+    match env::var(get_env_name("skills_dir")) {
+        Ok(value) => PathBuf::from(value),
+        Err(_) => local_path(SKILLS_DIR_NAME),
+    }
+}
+
+#[allow(dead_code)]
+pub fn skill_dir(name: &str) -> PathBuf {
+    skills_dir().join(name)
+}
+
+#[allow(dead_code)]
+pub fn skill_file(name: &str) -> PathBuf {
+    skill_dir(name).join("SKILL.md")
+}
+
 pub fn macros_dir() -> PathBuf {
     match env::var(get_env_name("macros_dir")) {
         Ok(value) => PathBuf::from(value),
@@ -234,6 +252,30 @@ pub fn has_macro(name: &str) -> bool {
     names.contains(&name.to_string())
 }
 
+#[allow(dead_code)]
+pub fn list_skills() -> Vec<String> {
+    let mut names = Vec::new();
+    if let Ok(rd) = read_dir(skills_dir()) {
+        for entry in rd.flatten() {
+            if let Ok(file_type) = entry.file_type()
+                && file_type.is_dir()
+                && let Some(name) = entry.file_name().to_str()
+                && entry.path().join("SKILL.md").is_file()
+            {
+                names.push(name.to_string());
+            }
+        }
+    }
+
+    names.sort_unstable();
+    names
+}
+
+#[allow(dead_code)]
+pub fn has_skill(name: &str) -> bool {
+    skill_file(name).is_file()
+}
+
 pub fn local_models_override() -> Result<Vec<ProviderModels>> {
     let model_override_path = models_override_file();
     let err = || {