Skip to content

Remove auto-generated comment from SKILL.md files#6

Merged
nelsonwittwer merged 2 commits intomainfrom
nw/fix-skill-md-frontmatter
Apr 7, 2026
Merged

Remove auto-generated comment from SKILL.md files#6
nelsonwittwer merged 2 commits intomainfrom
nw/fix-skill-md-frontmatter

Conversation

@nelsonwittwer
Copy link
Copy Markdown
Collaborator

@nelsonwittwer nelsonwittwer commented Apr 6, 2026

Summary

  • All 16 SKILL.md files had an HTML comment (<!-- AUTO-GENERATED -->) before the opening --- YAML frontmatter marker
  • The npx skills add CLI requires --- to be the very first line; the comment caused every skill to fail parsing with "No valid skills found"
  • Removed the comment entirely -- this repo will close all PRs as a clue to internal/external contributors they are making changes in the wrong place.

Test plan

  • Run pnpx skills add Shopify/shopify-ai-toolkit --list after merge and confirm all 16 skills are listed

Note for the generation pipeline: The generate_agent_skills script in shopify-dev-tools should also be updated to not emit this comment.

🤖 Generated with Claude Code

The skills CLI (npx skills add) requires YAML frontmatter to start on
line 1. The HTML comment before --- caused all 16 skills to be skipped
with "No valid skills found". Moved the comment into the Markdown body
(after closing ---) so `npx skills add Shopify/shopify-ai-toolkit` works.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

Thanks for your interest but we don't accept pull requests. Any pull requests will be automatically closed.

If you have feedback or suggestions, please share them in the Shopify Developer Community.

If you are a Shopify employee, please reach out to Dev Tools: Dev Experience team at help-dev-platform.

@github-actions github-actions bot closed this Apr 6, 2026
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@nelsonwittwer nelsonwittwer changed the title Move auto-generated comment after YAML frontmatter in SKILL.md files Remove auto-generated comment from SKILL.md files Apr 6, 2026
@nelsonwittwer nelsonwittwer reopened this Apr 6, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

Thanks for your interest but we don't accept pull requests. Any pull requests will be automatically closed.

If you have feedback or suggestions, please share them in the Shopify Developer Community.

If you are a Shopify employee, please reach out to Dev Tools: Dev Experience team at help-dev-platform.

@github-actions github-actions bot closed this Apr 6, 2026
@nelsonwittwer nelsonwittwer reopened this Apr 7, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

Thanks for your interest but we don't accept pull requests. Any pull requests will be automatically closed.

If you have feedback or suggestions, please share them in the Shopify Developer Community.

If you are a Shopify employee, please reach out to Dev Tools: Dev Experience team at help-dev-platform.

@github-actions github-actions bot closed this Apr 7, 2026
@nelsonwittwer nelsonwittwer reopened this Apr 7, 2026
@nelsonwittwer nelsonwittwer merged commit f182c31 into main Apr 7, 2026
2 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

Thanks for your interest but we don't accept pull requests. Any pull requests will be automatically closed.

If you have feedback or suggestions, please share them in the Shopify Developer Community.

If you are a Shopify employee, please reach out to Dev Tools: Dev Experience team at help-dev-platform.

@nelsonwittwer nelsonwittwer deleted the nw/fix-skill-md-frontmatter branch April 7, 2026 15:35
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.

1 participant