Skip to content

feat: add GitHub Copilot CLI support - #28

Merged
gerfru merged 1 commit into
masterfrom
feat/copilot-cli-support
Aug 13, 2026
Merged

feat: add GitHub Copilot CLI support#28
gerfru merged 1 commit into
masterfrom
feat/copilot-cli-support

Conversation

@gerfru

@gerfru gerfru commented Aug 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • This plugin now works as a GitHub Copilot CLI plugin/marketplace in addition to Claude Code — same repo, same skills, no duplicated tree. Verified live against both tools (see test log in review notes below).
  • meta-install gained --target claude|copilot-cli, --copilot-native, and --standalone-skills for installing rules/skills into Copilot CLI projects that don't use the plugin/marketplace mechanism.
  • Fixed a real bug along the way: 3 files had unquoted single-line description: values containing a colon+space, which breaks YAML parsing under Copilot CLI's stricter frontmatter parser (Claude Code tolerated it). This silently prevented review-app, and the meta-install/meta-create-skill commands, from loading under Copilot CLI.
  • Skill name: frontmatter changed from dev:<skill> to dev-<skill> (Copilot CLI rejects : in skill names). Verified live this has zero effect on Claude Code, whose /dev:<skill> slash-invocation is driven by commands/<skill>.md (plugin name + command filename), never by this field.
  • ${CLAUDE_PLUGIN_ROOT} (Claude Code-only template variable) replaced with plain relative paths in the 4 SKILL.md files that used it; it remains fine in commands/*.md, which is Claude Code-only and not read by Copilot CLI.
  • validate-skills.sh gained checks to catch both classes of bug (stray ${CLAUDE_PLUGIN_ROOT}, unsafe description: colons, missing/present skills field in plugin.json) automatically going forward.
  • Corrected a pre-existing stale "24 skills" / "26 skills" count to the actual 27, across marketplace.json, plugin.json, CLAUDE.md, and README.md.

Test plan

  • bash scripts/validate-skills.sh — all checks green
  • diff -r claude/ plugins/dev/rules/ — rule mirror still in sync
  • Installed via copilot plugin install dev@gerald-dev-best-practices from a local marketplace source — Skills (29/29 enabled), no YAML parse errors, all 27 skills including the previously-missing meta-install/meta-create-skill present
  • Verified in Claude Code (this session, local plugin cache synced) that /dev:design-api still resolves correctly after the name: field rename
  • Reviewer: reinstall via claude plugin install dev@gerald-dev-best-practices in a fresh project as a final regression check before merge

🤖 Generated with Claude Code

Makes this plugin dual-compatible with Claude Code and GitHub Copilot
CLI, which converged on a near-identical marketplace/plugin/skill
architecture. Verified live against both tools.

- meta-install: new --target claude|copilot-cli, --copilot-native, and
  --standalone-skills modes
- skill name: frontmatter changed dev:<skill> -> dev-<skill> (Copilot
  CLI rejects colons in skill names; confirmed no effect on Claude
  Code, whose /dev:<skill> invocation comes from commands/<skill>.md)
- fixed 3 files with unquoted description: values containing ": ",
  which broke YAML parsing under Copilot CLI's stricter parser
- replaced ${CLAUDE_PLUGIN_ROOT} (Claude-only) with relative paths in
  4 SKILL.md files
- validate-skills.sh: new Copilot CLI compatibility checks
- corrected stale skill counts (24/26 -> 27) across manifests and docs

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@gerfru
gerfru merged commit 67f44cb into master Aug 13, 2026
5 checks passed
@gerfru
gerfru deleted the feat/copilot-cli-support branch August 13, 2026 12:07
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