feat: Reimplement skill management with a new skill model and dedicated modules.

Author: vinhnx

.vtcode/tool-policy.json

@@ -15,9 +15,18 @@
     "grep_file",
     "list_files",
     "list_pty_sessions",
+    "list_skills",
+    "load_skill",
+    "load_skill_resource",
     "mcp::fetch::fetch",
     "mcp::time::convert_time",
     "mcp::time::get_current_time",
+    "mcp_convert_time",
+    "mcp_fetch",
+    "mcp_fetch_fetch",
+    "mcp_get_current_time",
+    "mcp_time_convert_time",
+    "mcp_time_get_current_time",
     "read_file",
     "read_pty_session",
     "resize_pty_session",
@@ -61,7 +70,10 @@
     "mcp_convert_time": "prompt",
     "mcp_time_convert_time": "prompt",
     "mcp_get_current_time": "prompt",
-    "mcp_time_get_current_time": "prompt"
+    "mcp_time_get_current_time": "prompt",
+    "list_skills": "allow",
+    "load_skill": "allow",
+    "load_skill_resource": "allow"
   },
   "constraints": {},
   "mcp": {

Cargo.lock

@@ -62,6 +62,8 @@ tree-sitter-bash = "0.25"
 indexmap = { version = "2.12", features = ["serde"] }
 itertools = "0.14.0"
 tempfile = { workspace = true }
+dunce = "1.0"
+include_dir = "0.7"
 once_cell = "1.19"
 parking_lot = "0.12"
 sha2 = "0.10"

vtcode-core/Cargo.toml

@@ -0,0 +1,17 @@
+---
+name: skill-creator
+description: Guide for creating effective skills. This skill should be used when users want to create a new skill (or update an existing skill) that extends Codex's capabilities with specialized knowledge, workflows, or tool integrations.
+metadata:
+  short-description: Create or update a skill
+---
+
+# Skill Creator
+
+This skill provides guidance for creating effective skills.
+
+## About Skills
+
+Skills are modular, self-contained packages that extend Codex's capabilities by providing
+specialized knowledge, workflows, and tools. Think of them as "onboarding guides" for specific
+domains or tasks—they transform Codex from a general-purpose agent into a specialized agent
+equipped with procedural knowledge that no model can fully possess.

vtcode-core/src/skills/assets/samples/skill-creator/SKILL.md

@@ -0,0 +1,15 @@
+---
+name: skill-installer
+description: Install Codex skills into $CODEX_HOME/skills from a curated list or a GitHub repo path. Use when a user asks to list installable skills, install a curated skill, or install a skill from another repo (including private repos).
+metadata:
+  short-description: Install curated skills from openai/skills or other repos
+---
+
+# Skill Installer
+
+Helps install skills. By default these are from https://github.com/openai/skills/tree/main/skills/.curated, but users can also provide other locations.
+
+Use the helper scripts based on the task:
+- List curated skills when the user asks what is available, or if the user uses this skill without specifying what to do.
+- Install from the curated list when the user provides a skill name.
+- Install from another repo when the user provides a GitHub repo/path (including private repos).

vtcode-core/src/skills/assets/samples/skill-installer/SKILL.md

@@ -593,7 +593,7 @@ mod tests {
 
         let bridge = CliToolBridge::new(config).unwrap();
         let result = bridge
-            .execute(Value::String("hello world".to_string()))
+            .execute_internal(Value::String("hello world".to_string()))
             .await
             .unwrap();
 

vtcode-core/src/skills/cli_bridge.rs

@@ -472,17 +472,7 @@ mod tests {
             description: "Generate PDFs".to_string(),
             version: Some("1.0.0".to_string()),
             author: Some("Test".to_string()),
-            license: None,
-            model: None,
-            mode: None,
-            vtcode_native: None,
-            allowed_tools: None,
-            disable_model_invocation: None,
-            when_to_use: None,
-            requires_container: None,
-            disallow_container: None,
-            compatibility: None,
-            metadata: None,
+            ..Default::default()
         };
 
         let instructions = r#"
@@ -517,17 +507,8 @@ mod tests {
             description: "Generate spreadsheets".to_string(),
             version: Some("1.0.0".to_string()),
             author: Some("Test".to_string()),
-            license: None,
-            model: None,
-            mode: None,
             vtcode_native: Some(true),
-            allowed_tools: None,
-            disable_model_invocation: None,
-            when_to_use: None,
-            requires_container: None,
-            disallow_container: None,
-            compatibility: None,
-            metadata: None,
+            ..Default::default()
         };
 
         let instructions = r#"
@@ -570,17 +551,7 @@ mod tests {
             description: "Generate PDFs".to_string(),
             version: Some("1.0.0".to_string()),
             author: Some("Test".to_string()),
-            license: None,
-            model: None,
-            mode: None,
-            vtcode_native: None,
-            allowed_tools: None,
-            disable_model_invocation: None,
-            when_to_use: None,
-            requires_container: None,
-            disallow_container: None,
-            compatibility: None,
-            metadata: None,
+            ..Default::default()
         };
 
         let instructions = r#"

vtcode-core/src/skills/container_validation.rs

@@ -585,17 +585,8 @@ mod tests {
             description: "Test skill".to_string(),
             version: Some("1.0.0".to_string()),
             author: Some("Test".to_string()),
-            license: None,
-            model: None,
-            mode: None,
             vtcode_native: Some(true),
-            allowed_tools: None,
-            disable_model_invocation: None,
-            when_to_use: None,
-            requires_container: None,
-            disallow_container: None,
-            compatibility: None,
-            metadata: None,
+            ..Default::default()
         };
 
         assert!(manager.register_skill_metadata(manifest).is_ok());
@@ -610,19 +601,7 @@ mod tests {
         let manifest = SkillManifest {
             name: "test-skill".to_string(),
             description: "Test skill".to_string(),
-            version: None,
-            author: None,
-            license: None,
-            model: None,
-            mode: None,
-            vtcode_native: None,
-            allowed_tools: None,
-            disable_model_invocation: None,
-            when_to_use: None,
-            requires_container: None,
-            disallow_container: None,
-            compatibility: None,
-            metadata: None,
+            ..Default::default()
         };
 
         manager.register_skill_metadata(manifest.clone()).unwrap();

vtcode-core/src/skills/context_manager.rs

@@ -391,17 +391,9 @@ mod tests {
             description: "A test skill for validation".to_string(),
             version: Some("1.0.0".to_string()),
             author: Some("Test Author".to_string()),
-            license: None,
-            model: None,
-            mode: None,
-            vtcode_native: None,
             allowed_tools: Some("Read Write Bash".to_string()),
-            disable_model_invocation: None,
-            when_to_use: None,
-            requires_container: None,
-            disallow_container: None,
             compatibility: Some("Designed for VTCode".to_string()),
-            metadata: None,
+            ..Default::default()
         };
 
         // Note: We can't easily test directory validation without creating temp dirs

vtcode-core/src/skills/enhanced_validator.rs

@@ -329,19 +329,8 @@ mod tests {
         let manifest = SkillManifest {
             name: "test-skill".to_string(),
             description: "Test skill".to_string(),
-            version: None,
-            author: None,
             vtcode_native: Some(true),
-            allowed_tools: None,
-            disable_model_invocation: None,
-            when_to_use: None,
-            requires_container: None,
-            disallow_container: None,
-            compatibility: None,
-            metadata: None,
-            license: None,
-            mode: None,
-            model: None,
+            ..Default::default()
         };
 
         let skill = Skill::new(
@@ -360,19 +349,8 @@ mod tests {
         let manifest = SkillManifest {
             name: "test-skill".to_string(),
             description: "Test skill".to_string(),
-            version: None,
-            author: None,
             vtcode_native: Some(true),
-            allowed_tools: None,
-            disable_model_invocation: None,
-            when_to_use: None,
-            requires_container: None,
-            disallow_container: None,
-            compatibility: None,
-            metadata: None,
-            license: None,
-            mode: None,
-            model: None,
+            ..Default::default()
         };
 
         let skill = Skill::new(
@@ -397,19 +375,8 @@ mod tests {
         let manifest = SkillManifest {
             name: "test-skill".to_string(),
             description: "Test skill".to_string(),
-            version: None,
-            author: None,
             vtcode_native: Some(true),
-            allowed_tools: None,
-            disable_model_invocation: None,
-            when_to_use: None,
-            requires_container: None,
-            disallow_container: None,
-            compatibility: None,
-            metadata: None,
-            license: None,
-            mode: None,
-            model: None,
+            ..Default::default()
         };
 
         let skill = Skill::new(manifest, PathBuf::from("/tmp"), "Instructions".to_string())