2 Commits

Author SHA1 Message Date
Okyumi
4bb6109b84 feat: add missing SKILL.md for adguardhome, comfyui, mermaid + fix setup.py + expand test coverage
Three harnesses shipped without the SKILL.md skill definition introduced
in Phase 6.5, and their setup.py files lacked the package_data entry
needed for pip install to include the skill file.  This means agents
cannot discover these CLIs through the standard skill system.

Changes:

- Add skills/SKILL.md for adguardhome (12 command groups, 36+ commands)
- Add skills/SKILL.md for comfyui (5 command groups: workflow, queue,
  models, images, system)
- Add skills/SKILL.md for mermaid (4 command groups: project, diagram,
  export, session)
- Fix adguardhome/setup.py: add package_data and include_package_data
- Fix mermaid/setup.py: add package_data and include_package_data
- Fix comfyui/setup.py: add package_data for skills (was missing despite
  include_package_data=True already being set)
- Add comfyui to .gitignore allow-list (was tracked before the gitignore
  was tightened, but new files could not be added)
- Expand test_skill_path.py HARNESSES list from 11 to all 18 harnesses
- Fix test assertion to accept mubu-style explicit SKILL.md reference
  alongside the glob pattern used by other harnesses

Test results: 79 passed (was 51 tests covering only 11 harnesses)
2026-03-22 14:17:13 +00:00
yuhao
55d11a5df6 test: add skill path discovery tests for installed packages
51 tests covering:
- ReplSkin auto-detection of skills/SKILL.md from __file__ location
- Absolute path resolution, missing file handling, explicit override
- Banner output includes/omits skill path as expected
- All 11 harnesses have SKILL.md in correct package location
- All 11 setup.py files include package_data for skills/*.md

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-16 09:31:08 +00:00