Skip to content

Update skills name validation check#69

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

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

Copy link
Copy Markdown
Contributor

@gbockus-sf gbockus-sf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

@hsinghbisht-sfdc
Copy link
Copy Markdown
Contributor Author

All existing skills are not following the gerund form especially agentforce*, hence closing it for now.

@hsinghbisht-sfdc hsinghbisht-sfdc deleted the hsinghbisht/validate-skills-name branch April 8, 2026 10:26
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.

3 participants