Skip to content

Comments

fix: standalone skills dropped from marketplace.json#6

Merged
terraboops merged 1 commit intomainfrom
fix/standalone-skills-dropped
Feb 13, 2026
Merged

fix: standalone skills dropped from marketplace.json#6
terraboops merged 1 commit intomainfrom
fix/standalone-skills-dropped

Conversation

@terraboops
Copy link
Collaborator

Summary

  • discoverPlugins() re-traversed the filesystem with isPluginDirectory(), requiring agents/, commands/, or skills/ subdirectories — standalone skills with just SKILL.md were silently excluded from marketplace generation
  • Replace second traversal with groupIntoPlugins(), a pure function that derives plugin groupings from discoverAllComponents() output by parsing category/plugin-name from paths
  • Reads pluginCategories from config instead of hardcoding categories, warns about orphaned components

Test plan

  • npm test — 12 unit tests pass (new test file)
  • npm run build — bundle rebuilds
  • generate command outputs standalone skill in marketplace.json
  • CI passes

🤖 Generated with Claude Code

discoverPlugins() re-traversed the filesystem with isPluginDirectory(),
which required agents/, commands/, or skills/ subdirectories. Standalone
skills with just SKILL.md passed validation but were silently excluded
from marketplace generation.

Replace the second traversal with groupIntoPlugins(), a pure function
that derives plugin groupings from discoverAllComponents() output by
parsing category/plugin-name from paths. One discovery, then group.

Also reads pluginCategories from config instead of hardcoding, and warns
about orphaned components that can't be mapped to a plugin.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@terraboops terraboops merged commit 50269a6 into main Feb 13, 2026
5 checks passed
@terraboops terraboops deleted the fix/standalone-skills-dropped branch February 13, 2026 05:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant