Skip to content

Update skills name validation check#69

Open
hsinghbisht-sfdc wants to merge 1 commit intomainfrom
hsinghbisht/validate-skills-name
Open

Update skills name validation check#69
hsinghbisht-sfdc wants to merge 1 commit intomainfrom
hsinghbisht/validate-skills-name

Conversation

@hsinghbisht-sfdc
Copy link
Copy Markdown
Contributor

References: Contributing guide · Skill authoring guide · Agent Skills spec

What changed

Updating the validate-skills PR to enforce skill names with the gerund form.

Why

We need a consistent naming pattern, and as more people are adding skills, I think now is a good time to set a strict check on the naming so teams follow the correct naming pattern from the get go.

Notes


Skills

Manual checklist

Description quality

  • Describes what the skill does and the expected output
  • Includes relevant Salesforce domain keywords (Apex, LWC, SOQL, metadata types, etc.)
  • Trigger phrases are specific enough for Vibes to select this skill reliably

Instructions

  • Clear goal statement
  • Step-by-step workflow
  • Validation rules for generated output
  • Defined output / artifact

Context efficiency

  • Core instructions are concise — supporting material lives in templates/, examples/, or docs/ subdirectories
  • No unnecessary background explanation in the body

Automated checks

Enforced by CI (npm run validate:skills) per the Agent Skills spec:

  • Directory is one level deep, named in kebab-case (max 64 chars), contains SKILL.md
  • Frontmatter name matches directory name; description is present, ≥ 20 words, ≤ 1024 characters, and includes trigger language
  • Body is non-empty and under 500 lines
  • Name uses gerund form ⚠ (warning — does not block merge)

@mohanraj-r
Copy link
Copy Markdown
Contributor

@hsinghbisht-sfdc need to rename the existing skills for the validate-skills check to pass in the PR

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