Skip to content

Document agent skill packaging, add AGENTS.md and package-skill skill - #870

Open
danbarr wants to merge 2 commits into
mainfrom
docs-skill-packaging
Open

Document agent skill packaging, add AGENTS.md and package-skill skill#870
danbarr wants to merge 2 commits into
mainfrom
docs-skill-packaging

Conversation

@danbarr

@danbarr danbarr commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • README, CONTRIBUTING, and docs/ covered MCP servers exclusively, with no mention of agent skills despite 150+ of them already vendored under skills/. Adds docs/adding-skills.md and cross-links it from README, docs/index.md, and CONTRIBUTING.md; extends docs/security.md with the skill-scanner equivalent of the existing MCP scanning section.
  • Adds a root AGENTS.md (there was none).
  • Adds a package-skill skill under .claude/skills/ mirroring package-mcp-server, grounded in the actual workflow used to package the 20 HashiCorp skills earlier in this branch's history (naming/collision guidance, spec.yaml authoring, scan/allowlist triage, versioning policy). Also reachable agent-agnostically via a .agents/skills symlink, since these skills aren't Claude Code-specific.
  • Ran package-skill through skill-creator for validation and description-triggering optimization. Structure passed validation as-is; the automated description-optimization loop found no description that clearly beat the original on held-out queries (a known limitation of the eval harness for a repo-scoped skill whose triggering depends on skills//spec.yaml context it doesn't have), so only a small negative-trigger clause was added to reduce confusion with package-mcp-server.

Test plan

  • All new/edited markdown links verified (no broken relative links introduced)
  • python3 scripts/quick_validate.py .claude/skills/package-skill passes
  • Symlink .agents/skills -> ../.claude/skills recorded correctly in git (mode 120000) and resolves

README, CONTRIBUTING, and docs/ covered MCP servers exclusively, with
no mention of agent skills despite 150+ of them already vendored under
skills/. Add docs/adding-skills.md, cross-link it from README/index/
CONTRIBUTING, and extend docs/security.md with the skill-scanner
equivalent of the existing MCP scanning section.

Also add a root AGENTS.md (there was none), and a package-skill skill
mirroring package-mcp-server, so future skill-packaging work in this
repo has the same guided workflow MCP servers already have. It lives
under .claude/skills/ but is also reachable agent-agnostically via a
.agents/skills symlink, since these skills aren't Claude Code-specific.

Signed-off-by: Dan Barr <6922515+danbarr@users.noreply.github.com>
@danbarr
danbarr requested a review from a team as a code owner August 11, 2026 18:42
@danbarr
danbarr enabled auto-merge (squash) August 11, 2026 19:36
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