Skip to content

Comments

Fix setup skill frontmatter validation and add testing milestone#3

Open
oreganoflakesgit wants to merge 2 commits intoagenticnotetaking:mainfrom
oreganoflakesgit:setup-frontmatter-validation
Open

Fix setup skill frontmatter validation and add testing milestone#3
oreganoflakesgit wants to merge 2 commits intoagenticnotetaking:mainfrom
oreganoflakesgit:setup-frontmatter-validation

Conversation

@oreganoflakesgit
Copy link
Contributor

Summary

  • add validate-setup.sh to catch malformed generated skill frontmatter
  • enforce required shared frontmatter keys
  • update /setup instructions for frontmatter validation
  • add testing milestone coverage for setup frontmatter checks

Closes #2

@arscontexta
Copy link
Contributor

thanks for reporting this and providing a fix, will take a look tomorrow, appreciate the help

Copy link

@ajmeese7 ajmeese7 left a comment

Choose a reason for hiding this comment

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

Looks good to me -- syntax is solid, no new vulns // malicious code.

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.

Setup can generate skill frontmatter as markdown table instead of YAML

3 participants