fix(installer): stop skill scanner from recursing into discovered skills

Skills don't nest. Once the manifest generator finds a valid SKILL.md
in a directory, it should not recurse into that skill's subdirectories
looking for more skills. Template files (like bmb's setup-skill-template)
inside a skill's assets/ would be incorrectly scanned and produce
spurious errors.
This commit is contained in:
Brian Madison 2026-04-13 01:02:05 -05:00
parent 5456b26ab7
commit 9ffb5b80ab
1 changed files with 7 additions and 5 deletions

View File

@ -193,12 +193,14 @@ class ManifestGenerator {
} }
} }
// Recurse into subdirectories // Recurse into subdirectories — but not inside a discovered skill
if (!skillMeta) {
for (const entry of entries) { for (const entry of entries) {
if (!entry.isDirectory()) continue; if (!entry.isDirectory()) continue;
if (entry.name.startsWith('.') || entry.name.startsWith('_')) continue; if (entry.name.startsWith('.') || entry.name.startsWith('_')) continue;
await walk(path.join(dir, entry.name)); await walk(path.join(dir, entry.name));
} }
}
}; };
await walk(modulePath); await walk(modulePath);