Skip to content

feat: add ygg skill install/uninstall for AI coding agents#1

Merged
joch merged 7 commits into
mainfrom
add-skill
Mar 27, 2026
Merged

feat: add ygg skill install/uninstall for AI coding agents#1
joch merged 7 commits into
mainfrom
add-skill

Conversation

@joch
Copy link
Copy Markdown
Owner

@joch joch commented Mar 27, 2026

Summary

  • Add ygg skill install / ygg skill uninstall commands that manage the ygg agent skill for Claude Code and Codex
  • Skill file is embedded in the binary via go:embed so it works for Homebrew/binary installs
  • Installs to ~/.claude/skills/ygg/ (Claude Code) and ~/.agents/skills/ygg/ (Codex)
  • Shows "already installed, updating..." message when overwriting

Test plan

  • Unit tests for install/uninstall logic (4 tests passing)
  • ygg skill install creates skill files in both locations
  • ygg skill install twice shows update message
  • ygg skill uninstall removes both directories
  • ygg skill uninstall when not installed shows info message
  • Error propagation returns non-zero exit code on failure
  • All existing tests still pass

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 40a379dc4c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread internal/skill/skill.go Outdated
@joch joch merged commit a022d51 into main Mar 27, 2026
8 checks passed
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