fix: scan nested skill directories up to 3 levels deep

scanDirectoryWithSkillMd only checked one level, missing grouped
structures like ~/.agents/skills/<group>/skills/<name>/SKILL.md

Now recurses into subdirs that don't contain SKILL.md, up to depth 3.

Also includes skillkit binary validation (isCrafterSkillkit).

Closes #16

Author: Railly

main.js

@@ -1,7 +1,7 @@
 {
 	"id": "agentfiles",
 	"name": "Agentfiles",
-	"version": "0.4.1",
+	"version": "0.4.2",
 	"minAppVersion": "1.4.11",
 	"description": "Discover, organize, and edit AI agent skills, commands, and agents across Claude Code, Cursor, Codex, Windsurf, and more.",
 	"author": "Railly Hugo",

manifest.json

@@ -1,6 +1,6 @@
 {
   "name": "agentfiles",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "description": "AI Skills Manager for Obsidian",
   "main": "main.js",
   "scripts": {

package.json

@@ -74,20 +74,23 @@ function scanDirectoryWithSkillMd(
 	baseDir: string,
 	type: SkillType,
 	toolId: string,
-	namingMode: NamingMode = "auto"
+	namingMode: NamingMode = "auto",
+	depth: number = 0
 ): SkillItem[] {
-	if (!existsSync(baseDir)) return [];
+	if (!existsSync(baseDir) || depth > 3) return [];
 	const items: SkillItem[] = [];
 
 	for (const entry of readdirSync(baseDir, { withFileTypes: true })) {
 		const fullPath = join(baseDir, entry.name);
 		const isDir = entry.isDirectory() || (entry.isSymbolicLink() && statSync(fullPath, { throwIfNoEntry: false })?.isDirectory());
 		if (!isDir) continue;
 		const skillFile = join(fullPath, "SKILL.md");
-		if (!existsSync(skillFile)) continue;
-
-		const item = parseSkillFile(skillFile, type, toolId, "directory-with-skillmd", namingMode);
-		if (item) items.push(item);
+		if (existsSync(skillFile)) {
+			const item = parseSkillFile(skillFile, type, toolId, "directory-with-skillmd", namingMode);
+			if (item) items.push(item);
+		} else {
+			items.push(...scanDirectoryWithSkillMd(fullPath, type, toolId, namingMode, depth + 1));
+		}
 	}
 	return items;
 }

src/scanner.ts

@@ -46,7 +46,21 @@ function buildPath(): string {
 	return [...extra, process.env.PATH || ""].join(delimiter);
 }
 
+function isCrafterSkillkit(binPath: string): boolean {
+	try {
+		const out = execSync(`"${binPath}" help`, {
+			encoding: "utf-8",
+			timeout: 5000,
+			env: { ...process.env, NO_COLOR: "1", PATH: buildPath() },
+			stdio: ["pipe", "pipe", "pipe"],
+			shell: IS_WIN ? "cmd.exe" : undefined,
+		});
+		return out.includes("Analytics for AI agent skills");
+	} catch { return false; }
+}
+
 function findSkillkitBin(): string | null {
+	const candidates: string[] = [];
 	const searchDirs: string[] = [];
 	if (IS_WIN) {
 		const appData = process.env.APPDATA || join(HOME, "AppData", "Roaming");
@@ -67,7 +81,7 @@ function findSkillkitBin(): string | null {
 	for (const dir of searchDirs) {
 		for (const bin of BIN_NAMES) {
 			const p = join(dir, bin);
-			if (existsSync(p)) return p;
+			if (existsSync(p)) candidates.push(p);
 		}
 	}
 	const nvmDir = IS_WIN
@@ -78,7 +92,7 @@ function findSkillkitBin(): string | null {
 			const binDir = IS_WIN ? join(nvmDir, d) : join(nvmDir, d, "bin");
 			for (const bin of BIN_NAMES) {
 				const p = join(binDir, bin);
-				if (existsSync(p)) return p;
+				if (existsSync(p)) candidates.push(p);
 			}
 		}
 	} catch { /* empty */ }
@@ -88,11 +102,14 @@ function findSkillkitBin(): string | null {
 			try {
 				for (const d of readdirSync(join(miseDir, runtime))) {
 					const p = join(miseDir, runtime, d, "bin", "skillkit");
-					if (existsSync(p)) return p;
+					if (existsSync(p)) candidates.push(p);
 				}
 			} catch { /* empty */ }
 		}
 	}
+	for (const c of candidates) {
+		if (isCrafterSkillkit(c)) return c;
+	}
 	return null;
 }