Skip to content

feat(repotype): skill-best-practices lint rule#1

Merged
ianderrington merged 1 commit into
ian-devfrom
skill-best-practices-rule
Jun 22, 2026
Merged

feat(repotype): skill-best-practices lint rule#1
ianderrington merged 1 commit into
ian-devfrom
skill-best-practices-rule

Conversation

@ianderrington

Copy link
Copy Markdown
Contributor

Adds the skill-best-practices ValidatorAdapter enforcing Agent Skill authoring rules (mgechev/skills-best-practices) on skills/*/SKILL.md: non-empty, frontmatter present, name==dir + valid charset, description present + <=1024 chars, <=500 lines (warn). 14/14 unit tests + tsc pass.

🤖 Generated with Claude Code

Adds the skill-best-practices adapter enforcing Agent Skill authoring
best-practices (per github.com/mgechev/skills-best-practices) on every
skills/<id>/SKILL.md manifest:

- non-empty file (error)
- has YAML frontmatter (error)
- name present, equals parent dir, matches ^[a-z0-9]+(-[a-z0-9]+)*$ (error)
- description present and <= 1024 chars (error)
- <= 500 lines (warning)

Registered in createDefaultEngine so the runner discovers it. Mirrors the
board-yaml-completeness / board-story-completeness adapter pattern. Adds a
14-case vitest mirroring existing adapter tests.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ianderrington
ianderrington merged commit b0972fd into ian-dev Jun 22, 2026
@ianderrington
ianderrington deleted the skill-best-practices-rule branch June 22, 2026 17:53
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