chore: move agent skills to api7/agent-skills - #62
Conversation
Skill content now lives in https://github.com/api7/agent-skills (skills/a6). test/skills validates that checkout via SKILLS_DIR; install.sh pulls from the new repository; recommended install is: npx skills add api7/agent-skills --skill a6
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (50)
💤 Files with no reviewable changes (41)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe PR moves the a6 skill to ChangesAgent skill externalization
Priority: ➖ Normal Estimated code review effort: 3 (Moderate) | ~25 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to The external skill migration, installation path, validation setup, and documentation updates are consistent across the changed workflows. 🚥 Pre-merge checks | ✅ 5 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (5 passed)
Full details: E2e Test Quality ReviewExplanation Blocking issue: the changed validation is not an E2E test. Resolution Add an E2E job or test that checks out
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Summary
The a6 agent skills moved to api7/agent-skills (
skills/a6/), published at https://skills.sh/api7/agent-skills/a6. This repository keeps the CLI and the test that guarantees the skill examples match it.Removed
skills/(40 flat skill directories) andscripts/validate-skills.shChanged
test/skills: reads the skill fromSKILLS_DIR(default../agent-skills/skills/a6; skips with a hint if absent, fails if explicitly set but missing). ValidatesSKILL.mdplus everyreferences/**/*.md; all other assertions unchanged..github/workflows/ci.yml: checks outapi7/agent-skillsand runsmake test-skillsagainst it; adds a dailyscheduleso drift between this CLI and the references surfaces within 24h.Makefile:test-skillsexportsSKILLS_DIR;validate-skillsremoved.install.sh: no-Node fallback now pulls theapi7/agent-skillstarball and copiesskills/a6only. Recommended install:npx skills add api7/agent-skills --skill a6.docs/skills.md,README.md,AGENTS.md: point at the new repository..github/PULL_REQUEST_TEMPLATE.md: ordering rule — add a command → merge CLI first, then the reference; remove/rename → merge the reference change first, then the CLI.Verification
go test ./test/skills -count=1withSKILLS_DIRpointing at api7/agent-skills: PASS — everya6 …invocation in the 40 references is valid on the current CLI.go build ./... && go vet ./...clean;gofmt -l .clean.🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
a6agent skill from the dedicated agent-skills repository.a6skill with supported installation methods.Documentation
Chores