preselected SKILLs are supported

Author: AdamBien

version.txt

@@ -0,0 +1 @@
+2026.05.10.01

zsmith/src/main/java/airhacks/zsmith/agent/boundary/Agent.java

@@ -6,6 +6,7 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
+import java.util.Set;
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
@@ -178,6 +179,10 @@ public Agent withSkills(String path) {
         return withSkills(new SkillStore(List.of(Path.of(path))));
     }
 
+    public Agent withSkillsNamed(String... names) {
+        return withSkills(SkillStore.forAgent(this.name).filtered(Set.of(names)));
+    }
+
     public Agent withSkills(SkillStore store) {
         var catalog = store.catalog();
         var enrichedPrompt = catalog.isEmpty()

zsmith/src/main/java/airhacks/zsmith/skills/boundary/SkillStore.java

@@ -6,6 +6,7 @@
 import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
+import java.util.Set;
 import java.util.stream.Collectors;
 
 import airhacks.zsmith.configuration.control.ZCfg;
@@ -28,6 +29,23 @@ public SkillStore(List<Path> searchDirectories) {
         Log.info("skills loaded: " + this.skills.size());
     }
 
+    SkillStore(Map<String, Skill> skills) {
+        this.skills = new LinkedHashMap<>(skills);
+    }
+
+    public SkillStore filtered(Set<String> names) {
+        var retained = new LinkedHashMap<String, Skill>();
+        for (var name : names) {
+            var skill = this.skills.get(name);
+            if (skill == null) {
+                Log.warning("skill not found, skipped from filter: " + name);
+                continue;
+            }
+            retained.put(name, skill);
+        }
+        return new SkillStore(retained);
+    }
+
     public static SkillStore forAgent(String agentName) {
         var userHome = System.getProperty("user.home");
         var dirs = List.of(

zsmith/src/test/java/SkillStoreTest.java

@@ -5,6 +5,7 @@
 import java.util.List;
 
 import java.util.Objects;
+import java.util.Set;
 
 import airhacks.zsmith.skills.boundary.SkillStore;
 
@@ -17,6 +18,9 @@ void main() throws IOException {
     emptyDirectoryProducesEmptyCatalog();
     nonExistentDirectoryIsIgnored();
     loadReturnsNullForUnknownSkill();
+    filteredKeepsOnlyNamedSkills();
+    filteredIgnoresUnknownNames();
+    filteredEmptySetProducesEmptyStore();
 }
 
 void loadSkillWithFrontmatter() throws IOException {
@@ -160,6 +164,68 @@ void loadReturnsNullForUnknownSkill() throws IOException {
     }
 }
 
+void filteredKeepsOnlyNamedSkills() throws IOException {
+    var tempDir = Files.createTempDirectory("zunit-skillstore");
+    try {
+        writeSkill(tempDir, "alpha", "alpha description", "alpha content");
+        writeSkill(tempDir, "beta", "beta description", "beta content");
+        writeSkill(tempDir, "gamma", "gamma description", "gamma content");
+
+        var store = new SkillStore(List.of(tempDir));
+        var filtered = store.filtered(Set.of("alpha", "gamma"));
+
+        assert filtered.allSkills().size() == 2 : "expected 2 skills but got " + filtered.allSkills().size();
+        assert filtered.load("alpha") != null : "alpha should be present";
+        assert filtered.load("gamma") != null : "gamma should be present";
+        assert filtered.load("beta") == null : "beta should have been filtered out";
+    } finally {
+        deleteRecursively(tempDir);
+    }
+}
+
+void filteredIgnoresUnknownNames() throws IOException {
+    var tempDir = Files.createTempDirectory("zunit-skillstore");
+    try {
+        writeSkill(tempDir, "alpha", "alpha description", "alpha content");
+
+        var store = new SkillStore(List.of(tempDir));
+        var filtered = store.filtered(Set.of("alpha", "missing"));
+
+        assert filtered.allSkills().size() == 1 : "expected 1 skill but got " + filtered.allSkills().size();
+        assert filtered.load("alpha") != null : "alpha should be present";
+        assert filtered.load("missing") == null : "missing should not be present";
+    } finally {
+        deleteRecursively(tempDir);
+    }
+}
+
+void filteredEmptySetProducesEmptyStore() throws IOException {
+    var tempDir = Files.createTempDirectory("zunit-skillstore");
+    try {
+        writeSkill(tempDir, "alpha", "alpha description", "alpha content");
+
+        var store = new SkillStore(List.of(tempDir));
+        var filtered = store.filtered(Set.of());
+
+        assert filtered.allSkills().isEmpty() : "expected empty filtered store";
+        assert "".equals(filtered.catalog()) : "expected empty catalog but got: " + filtered.catalog();
+    } finally {
+        deleteRecursively(tempDir);
+    }
+}
+
+static void writeSkill(Path baseDir, String name, String description, String content) throws IOException {
+    var skillDir = baseDir.resolve(name);
+    Files.createDirectories(skillDir);
+    Files.writeString(skillDir.resolve("SKILL.md"), """
+            ---
+            name: %s
+            description: %s
+            ---
+            %s
+            """.formatted(name, description, content));
+}
+
 static void deleteRecursively(Path path) throws IOException {
     try (var walk = Files.walk(path)) {
         walk.sorted(Comparator.reverseOrder())