Skip to content

fix(claude): scan skills/ directory to expose all skills#3

Open
mnovelo wants to merge 1 commit into
firecrawl:mainfrom
mnovelo:fix/claude-skills-dir-scan
Open

fix(claude): scan skills/ directory to expose all skills#3
mnovelo wants to merge 1 commit into
firecrawl:mainfrom
mnovelo:fix/claude-skills-dir-scan

Conversation

@mnovelo

@mnovelo mnovelo commented Jun 20, 2026

Copy link
Copy Markdown

What

Switch .claude-plugin/plugin.json's skills field from an explicit 5-path array to the directory string "./skills/", matching .codex-plugin/plugin.json.

Why

skills/ contains six skills, but the Claude manifest enumerated only five — omitting firecrawl-research-index. Because this marketplace entry's source resolves to the marketplace root ("source": "./"), an explicit skills array replaces the default skills/ scan rather than supplementing it, per the plugins reference:

for a marketplace entry whose source resolves to the marketplace root, declaring specific subdirectories replaces the default skills/ scan

So the unlisted skill never loads when the plugin is installed via the marketplace.

The sibling manifests already expose all six: .codex-plugin/plugin.json uses "skills": "./skills/" and .cursor-plugin/plugin.json omits the field (default discovery). Only the Claude manifest drifted — firecrawl-research-index currently ships to Codex and Cursor users but not Claude.

Effect

  • All skills under skills/ load via Claude, including firecrawl-research-index.
  • Future skills are picked up without a manifest edit (parity with Codex).

Open question for maintainers

README.md and AGENTS.md enumerate only the five firecrawl-build-* skills. If firecrawl-research-index is meant to ship, those docs likely want a matching entry. If it was deliberately scoped out of the Claude plugin specifically, say so and I'll switch this to explicitly appending the single path instead — though it already reaches Codex/Cursor users either way.

🤖 Generated with Claude Code

The Claude plugin manifest enumerated only five of the six skills in
skills/, omitting firecrawl-research-index. With marketplace source "./"
(root), an explicit skills array replaces the default skills/ scan, so
the unlisted skill never loads. Codex (./skills/) and Cursor (omitted)
already expose all six; this aligns the Claude manifest with them.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NMZYScgPF9se5hX42qB1Z3
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