Skip to content

fix: @W-22134404@ delete fragment skill#212

Merged
hsinghbisht-sfdc merged 2 commits intoforcedotcom:mainfrom
sowmya-sriram:remove-fragment-skill
Apr 22, 2026
Merged

fix: @W-22134404@ delete fragment skill#212
hsinghbisht-sfdc merged 2 commits intoforcedotcom:mainfrom
sowmya-sriram:remove-fragment-skill

Conversation

@sowmya-sriram
Copy link
Copy Markdown
Contributor

@sowmya-sriram sowmya-sriram commented Apr 20, 2026

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

@W-22134404@

What changed

Deleted fragment skill

Why

Widget/fragment generation use case is currently only for CLT renditions and the needed guidance will be added a reference to CLT skill.

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)
@@ -1,117 +0,0 @@
---
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why are you deleting the skill?

Copy link
Copy Markdown
Contributor

@hsinghbisht-sfdc hsinghbisht-sfdc Apr 20, 2026

Choose a reason for hiding this comment

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

I got justification from @AndrewLPetersonSF offline.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Just an FYI, I had already added "Why" in PR description.

@@ -1,117 +0,0 @@
---
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

PR title should be prefixed with feat: or fix: to trigger the release.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

please use fix: as prefix and validate action is still failing. Check other merged PRs for your reference.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

@sowmya-sriram sowmya-sriram changed the title @W-22134404 delete fragment skill Apr 20, 2026
@sowmya-sriram sowmya-sriram changed the title @W-22134404 feat: delete fragment skill Apr 21, 2026
@hsinghbisht-sfdc hsinghbisht-sfdc merged commit c818135 into forcedotcom:main Apr 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants