Skip to content

Point catalog sync at skills/core/ - #205

Merged
ericciarla merged 1 commit into
mainfrom
sync-target-skills-core
Aug 21, 2026
Merged

Point catalog sync at skills/core/#205
ericciarla merged 1 commit into
mainfrom
sync-target-skills-core

Conversation

@ericciarla

@ericciarla ericciarla commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Half 1 of renaming the catalog's skills/cli/skills/core/ (the family covers the Firecrawl primitives via CLI or MCP, not just the CLI — see firecrawl/skills for the other half).

  • sync-catalog.yml now mirrors into catalog skills/core/ and writes a "Core skills (mirror)" README
  • Updated the skills-install.ts doc comments; no behavior change — install selection is name-based, so skills.sh slugs and firecrawl init are unaffected

Merge this first. The sync will create skills/core/ in the catalog alongside the old folder; the catalog PR then removes skills/cli/ and updates the plugin manifests, so there's no window where manifest paths point at a missing directory.

🤖 Generated with Claude Code


Summary by cubic

Rename the catalog mirror from skills/cli/ to skills/core/ to reflect that these are shared core skills used by CLI and MCP. No runtime behavior changes; install selection remains name-based, so slugs and firecrawl init are unchanged.

  • Update .github/workflows/sync-catalog.yml to mirror cli/skills/ into catalog skills/core/ and write a "Core skills (mirror)" README.
  • Update doc comments in src/commands/skills-install.ts to reference skills/core/ only.

Review and rollout

  • Merge this first. The sync will create skills/core/ alongside skills/cli/.
  • Follow-up in the catalog will remove skills/cli/ and update plugin manifests, avoiding any window with missing paths.
  • No user or integrator actions required.

Written for commit 7a48d59. Summary will update on new commits.

Review in cubic

The catalog directory is being renamed from skills/cli/ to skills/core/ —
the family covers the Firecrawl primitives via CLI or MCP, not just the
CLI. Install selection is name-based, so slugs are unaffected.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@ericciarla
ericciarla merged commit f0adf53 into main Aug 21, 2026
8 checks passed
@ericciarla
ericciarla deleted the sync-target-skills-core branch August 21, 2026 12:51
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