Skip to content

Restructure skills catalog into categories - #22

Open
njb90 wants to merge 5 commits into
mainfrom
feat/catalog-categories
Open

Restructure skills catalog into categories#22
njb90 wants to merge 5 commits into
mainfrom
feat/catalog-categories

Conversation

@njb90

@njb90 njb90 commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator

Moves the skills catalog to a flat category layout, adds a CI check to catch skills silently dropping out of discovery, and records the authoring conventions later skills must follow. No shipped skill changes behaviour.

Split out of #21 per review so the restructure can ship first. #21 now carries only the cloudinary-upload skill and is stacked on this branch.

Why this layout

  • The CLI discovers skills/<category>/<name>/SKILL.md at exactly depth two. Folders are invisible to users and agents, so moving a skill does not break installs.
  • cloudinary-next keeps its name. Renaming would orphan existing installs.
  • use-cases/ and utilities/ are not created here. They will be created when we add skills in those categories.
  • Existing descriptions are not retrofitted to the new shapes.

Open questions from #21 review

  • Bump versions on renamed skills? Proposed answer: no. Paths are not part of the skill contract and content is unchanged.
  • Add CI checks for tracking params on doc links and for version bumps on changed skills. Tracking-param check is a good follow-up; version-bump check needs care around pure moves.

njb90 and others added 4 commits September 4, 2026 13:54
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Runs the real skills CLI against the checkout and fails when the number
of discovered skills differs from the number of SKILL.md files, so a
skill dropped by bad frontmatter or wrong depth cannot merge silently.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
README groups the skills table by category, leads the install section
with --skill examples. CONTRIBUTING.md
records the layout, frontmatter, description-shape, cross-reference,
versioning and local-check rules. specs/ is gitignored.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
GitHub Actions sets CI, which makes the skills CLI colour its output, so
'Found 5 skills' arrived as 'Found \e[32m5\e[39m skills' and the grep never
matched (found=0, expected=5). Set NO_COLOR and strip any remaining ANSI
escapes before grepping.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@njb90 njb90 mentioned this pull request Sep 7, 2026
6 tasks
Two new CI jobs alongside discovery, both backed by scripts that run
locally too:

- links: every cloudinary.com link to a .md page or llms.txt must carry
  install_source and referrer, so docs traffic from skills is attributable.
- versions: on pull requests, any skill whose content changed against the
  base must bump metadata.version. Pure moves with identical content are
  ignored, so restructures pass; new skills pass.

CONTRIBUTING documents both rules and the local commands.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@njb90

njb90 commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator Author

@jackieros - moved the restructure to here and added the 2 ci checks for links with no referrer and version bumping.

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.

2 participants