Skip to content

feat: public skills repository framework - #1

Merged
faction23 merged 17 commits into
mainfrom
feature/skills-repo-framework
May 13, 2026
Merged

faction23 merged 17 commits into
mainfrom
feature/skills-repo-framework

Conversation

@faction23

Copy link
Copy Markdown
Contributor

Description

Closes: https://github.com/gravityforms/backlog/issues/7488

  • Set up gravityforms/gravityskills as a pure content repo for distributing AI agent skills via GitHub Releases and Claude Code plugin marketplace
  • Shell-based validation and packaging scripts — no Node.js, pnpm, or build tooling required
  • Placeholder skill following the Agent Skills spec from agentskills.io
  • CI workflow validates skills and verifies zip integrity on every push/PR
  • Release workflow generates per-skill zips and attaches them to GitHub Releases on v* tag push
  • Claude Code plugin manifest at .claude-plugin/marketplace.json
  • @gravity/skills CLI was initially scaffolded here but moved to the Gravity Monorepo — this repo is public-facing content only

Testing instructions

  1. Clone the branch
  2. Run ./scripts/validate-skills.sh — should output "Validated 1 skill."
  3. Run ./scripts/pack-skills.sh — should create dist/skills/placeholder-skill.zip
  4. Run unzip -t dist/skills/placeholder-skill.zip — should show no errors
  5. Review CI and release workflows in .github/workflows/

Screenshots

N/A

Checklist:

  • I've checked the error log to ensure my code doesn't throw any errors.
  • My code follows the WordPress coding standards and inline documentation standards.
  • My code follows the accessibility standards.
  • This PR has a related issue.
  • This PR has a related test.
  • I have considered what will happen when an existing user updates to this branch.
  • I have added a "Documentation Needed" label to this PR (if necessary).

faction23 added 16 commits May 13, 2026 06:15
Files were committed before .gitignore rule was in place.
The .gitignore entry already exists — this commit just
removes the tracked files so they stay local-only.
…d README (#7488)

The public skills repo should contain only skills content.
The CLI package now lives in gravitysuite where internal
tooling belongs. Also adds AGENTS.md and public-facing README.
Remove package.json, pnpm-workspace.yaml, pnpm-lock.yaml, .nvmrc,
and TypeScript scripts/tests. Replace with shell scripts for
validation and packaging. No build tooling required — just bash
and zip, matching Vercel's agent-skills repo pattern.
Plugin marketplace is for richer packages (MCP + skills + hooks),
not pure skill distribution. A proper GF Claude plugin will live
in its own repo when the abilities API work lands.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Bootstraps gravityskills as a public, content-only repository for distributing AI agent "skills" (per the agentskills.io spec) via GitHub Releases and a Claude Code plugin marketplace. No Node/pnpm tooling lives here — validation and packaging are done with shell scripts, and a placeholder skill is included to exercise the end-to-end pipeline.

Changes:

  • Add shell scripts to validate skill frontmatter and pack each skill into a zip (scripts/validate-skills.sh, scripts/pack-skills.sh).
  • Add CI and Release GitHub workflows that validate skills on push/PR and publish per-skill zips on v* tags.
  • Add a placeholder skill, top-level README.md, AGENTS.md, GPL-2.0 LICENSE, and an updated .gitignore.

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 10 comments.

Show a summary per file
File Description
skills/placeholder-skill/SKILL.md Placeholder skill with frontmatter; references nonexistent pnpm commands.
skills/placeholder-skill/references/README.md Sub-directory marker to verify zip nesting.
scripts/validate-skills.sh Bash-based SKILL.md validation (frontmatter, name regex, dup detection).
scripts/pack-skills.sh Bash zip packaging into dist/skills/.
README.md Public-facing project README and contributor guidelines.
LICENSE GPL-2.0 license text.
AGENTS.md Internal contributor/agent notes and commit conventions.
.gitignore Trimmed and refocused on this content-only repo.
.github/workflows/release.yml Tag-triggered release workflow that publishes zips.
.github/workflows/ci.yml Push/PR workflow validating, packing, and unzip-testing skills.
Comments suppressed due to low confidence (1)

AGENTS.md:46

  • Same hard-coded issue reference appears in the commit format example. If/when this repo accepts unrelated contributions, (#7488) will be wrong; recommend showing a generic placeholder like (#<issue>) instead.
Format: `<type>: <subject> (#7488)` — present tense, imperative mood, no period.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread skills/placeholder-skill/SKILL.md Outdated
Comment thread scripts/pack-skills.sh Outdated
Comment thread scripts/validate-skills.sh
Comment thread README.md Outdated
Comment thread scripts/validate-skills.sh Outdated
Comment thread scripts/validate-skills.sh Outdated
Comment thread scripts/pack-skills.sh Outdated
Comment thread .github/workflows/release.yml Outdated
Comment thread .github/workflows/release.yml
Comment thread AGENTS.md Outdated
- Replace pnpm refs with shell script paths in placeholder SKILL.md
- Fix **.tgz glob pattern in pack-skills.sh
- Replace bc dependency with pure bash arithmetic
- Fix frontmatter closing delimiter detection in validate-skills.sh
- Allow digits in skill names per Agent Skills spec
- Document single-line description restriction
- Tighten release tag pattern to v[0-9]*
- Add zip integrity verification to release workflow
- Remove hardcoded issue ref from commit convention
- Gate CLI install behind coming-soon note in README
@faction23
faction23 merged commit ccd6afc into main May 13, 2026
1 check passed
@faction23
faction23 deleted the feature/skills-repo-framework branch May 13, 2026 19:37
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.

2 participants