Create skills registry in AGENTS.md with validation hook#1532
Create skills registry in AGENTS.md with validation hook#1532nathanjmcdougall merged 8 commits intomainfrom
Conversation
- Update AGENTS.md with a complete skills registry table listing all skills and their descriptions - Create hooks/check-skills-documented script that verifies all skill directory names in .agents/skills/ are mentioned in AGENTS.md - Add check-skills-documented as a local prek hook in .pre-commit-config.yaml at priority 0 Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com> Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/8968f481-6345-4852-af05-448495a66340
Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com> Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/8968f481-6345-4852-af05-448495a66340
hooks/check-skills-documented
Outdated
There was a problem hiding this comment.
I'd want this in Python, not bash.
There was a problem hiding this comment.
@copilot please re-write according to the priciples in the usethis-prek-hook-bespoke-create skill
There was a problem hiding this comment.
Rewrote as a Python script following the usethis-prek-hook-bespoke-create skill conventions: pure Python with pathlib, errors to stderr, success to stdout, raise SystemExit(main()) pattern, and wired via uv run --frozen --offline. Also added 2 newly-created skills (usethis-prek-hook-bespoke-create, usethis-python-functions) to the registry. See commit ee508ba.
|
@copilot There's been a couple of new skills added onto |
- Replace bash script with Python script (hooks/check-skills-documented.py) - Follow usethis-prek-hook-bespoke-create skill conventions: pure Python, errors to stderr, success to stdout, raise SystemExit - Update .pre-commit-config.yaml entry to use uv run --frozen --offline - Add two newly-created skills to the registry: usethis-prek-hook-bespoke-create and usethis-python-functions Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com> Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/0911b302-1e44-4ea9-8b34-d57ad7e0e2f9
Both |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
@copilot Please disable |
…-registry-in-agents-md
…ffected-find to registry, merge main Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com> Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/bc26d8b8-cf79-44f7-a8e3-a1f6770b64c4
Added |
Agents were ignoring available skills because AGENTS.md didn't enumerate them. Added a skills registry and a prek hook to keep it from rotting.
AGENTS.md: Added a skills registry table listing all 17 skills with descriptions. Fixed.agent/skills→.agents/skillstypo.hooks/check-skills-documented.py: Python script following theusethis-prek-hook-bespoke-createskill conventions that fails if any.agents/skills/*/directory name isn't mentioned inAGENTS.md. Uses pure Python with pathlib, errors to stderr, success to stdout..pre-commit-config.yaml: Wired up the hook as a local prek hook at priority 0 viauv run --frozen --offline.pyproject.toml: Addedlint.per-file-ignores."hooks/**" = [ "D", "INP001" ]to disable docstring and implicit namespace package ruff rules for hook scripts.When a new skill is added without updating AGENTS.md, the hook produces:
Original prompt
AGENTS.md#1531📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.