Support skills organized in nested subfolders

[Copilot]

Repositories like mattpocock/skills organize skills into category subfolders (skills/engineering/diagnose/SKILL.md) rather than the flat layout (skills/diagnose/SKILL.md) that skillup previously assumed.

Changes

• get_skills_in_zip: recursively scans the entire zip for SKILL.md at any depth and any location (not restricted to a skills/ subfolder); the skill name is always the immediate parent folder of SKILL.md (no path separators)

  • skills/engineering/diagnose/SKILL.md → "diagnose"
  • skills/pdf/SKILL.md → "pdf" (unchanged)
  • mything/SKILL.md (at zip root) → "mything"

• install_skill: locates the skill folder by finding SKILL.md in a folder named skill_name anywhere in the zip, then installs all contents to <target>/<skill-name>/

• tests/test_install.py (new): unit tests for get_skills_in_zip and install_skill covering flat, nested, mixed, zip-root-level, missing-SKILL.md, and case-insensitive scenarios

[aersam]

@copilot: no. recursively scan the zip for SKILL.md, the skill name is just the parent folder without any slash

[Copilot]

@copilot: no. recursively scan the zip for SKILL.md, the skill name is just the parent folder without any slash

Done in d88aace. Both get_skills_in_zip and install_skill now scan the entire skills/ subtree recursively for SKILL.md, and the skill name is always just parts[-2] — the immediate parent folder, no slashes. A skill at skills/engineering/diagnose/SKILL.md is now discovered and installed as "diagnose".

[aersam]

@copilot: no. recursively scan the zip for SKILL.md, the skill name is just the parent folder without any slash

Done in d88aace. Both get_skills_in_zip and install_skill now scan the entire skills/ subtree recursively for SKILL.md, and the skill name is always just parts[-2] — the immediate parent folder, no slashes. A skill at skills/engineering/diagnose/SKILL.md is now discovered and installed as "diagnose".

ok, but please not only look in /skills but in whole zip

[Copilot]

@copilot: no. recursively scan the zip for SKILL.md, the skill name is just the parent folder without any slash

Done in [d88aace](d88aacea7de1f1d7...

Done in de69ca7. Both get_skills_in_zip and install_skill now scan the entire zip for SKILL.md — the parts[1] == "skills" restriction is removed. A skill can be at any path depth inside the zip.