Skip to content

Harden skill validation and contributor workflows#1

Open
dknauss wants to merge 2 commits intojdevalk:mainfrom
dknauss:codex/upstream-friendly
Open

Harden skill validation and contributor workflows#1
dknauss wants to merge 2 commits intojdevalk:mainfrom
dknauss:codex/upstream-friendly

Conversation

@dknauss
Copy link

@dknauss dknauss commented Mar 21, 2026

Summary

This PR tightens repository validation and adds baseline contributor workflows without changing the repo's public positioning.

What changed

  • Hardened skill validation:
    • fail when frontmatter is missing a closing ---
    • require name: to match the containing directory
    • verify referenced support files exist
    • validate bundled evals/evals.json
  • Expanded validation triggers so changes in references/, evals/, and test-cases/ also run validation
  • Added workflow_dispatch to the markdown/link/skill validation workflows for manual runs. (This gives the maintainer a manual recovery and test path without needing a dummy commit.)
  • Added baseline contributor files and templates:
    • CONTRIBUTING.md
    • SECURITY.md
    • CODE_OF_CONDUCT.md
    • SUPPORT.md
    • PULL_REQUEST_TEMPLATE.md
    • issue forms .github/ISSUE_TEMPLATE
    • Dependabot config for GitHub Actions

Why

The current validator only checks for an opening frontmatter fence and the presence of two keys inside whatever text follows. In practice, that means malformed frontmatter and missing linked support files can ship undetected. This PR raises the floor on correctness for the skill pack and gives contributors a clearer submission path.

Validation

  • bash .github/scripts/validate-skills.sh

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