Skip to content

feat: Support GitHub Copilot#65

Merged
matz3 merged 7 commits into
mainfrom
support-github-copilot
Jun 3, 2026
Merged

feat: Support GitHub Copilot#65
matz3 merged 7 commits into
mainfrom
support-github-copilot

Conversation

@matz3
Copy link
Copy Markdown
Member

@matz3 matz3 commented Jun 3, 2026

matz3 added 6 commits June 3, 2026 13:29
Claude Code also supports the root `plugin.json` file.
Although this seems to have worked with Claude Code, it is generally
required to place each skill in a separate sub-directory with the same
name as the skill.
Add CI step that fails when any registered copy of plugins/<name>/plugin.json
differs from the root file. Currently checks .github/plugin/plugin.json,
required by github/awesome-copilot external-plugin smoke test.
Copy each plugins/<name>/plugin.json to <name>/.github/plugin/plugin.json,
required by github/awesome-copilot external-plugin install verification.
Sync with root enforced by CI step added in previous commit.
The github/awesome-copilot marketplace's external-plugin validator
rejects keywords containing spaces or uppercase characters; only
lowercase letters, digits, and hyphens are allowed (max 30 chars,
max 10 entries per plugin).

Replace 'api documentation', 'development guidelines', and
'best practices' with their hyphenated equivalents so the plugin
passes intake validation when listed in plugins/external.json.

Reference:
- awesome-copilot/eng/external-plugin-validation.mjs (regex /^[a-z0-9-]+$/)
- awesome-copilot/CONTRIBUTING.md - 'Adding External Plugins'
  ('keywords as lowercase hyphenated tags')
@matz3 matz3 force-pushed the support-github-copilot branch from dd4fa2c to 9417901 Compare June 3, 2026 12:43
@matz3 matz3 marked this pull request as ready for review June 3, 2026 12:44
@matz3 matz3 requested a review from a team June 3, 2026 12:44
RandomByte
RandomByte previously approved these changes Jun 3, 2026
Align with the plugin spec, which requires a top-level "mcpServers"
wrapper around server entries. Both Claude Code and Copilot CLI
tolerated the flat shape, but the spec-compliant form is safer for
strict validators down the line.
@matz3 matz3 force-pushed the support-github-copilot branch from 0f07140 to 5ddbb2b Compare June 3, 2026 13:26
@matz3 matz3 merged commit 44a72cd into main Jun 3, 2026
7 checks passed
@matz3 matz3 deleted the support-github-copilot branch June 3, 2026 13:33
@openui5bot openui5bot mentioned this pull request Jun 3, 2026
kineticjs pushed a commit to kineticjs/plugins-coding-agents that referenced this pull request Jun 3, 2026
Preparation to support GitHub Copilot via [Awesome GitHub Copilot](https://awesome-copilot.github.com/).

See:
- UI5#38
- github/awesome-copilot#1889
- github/awesome-copilot#1890

JIRA: CPOUI5FOUNDATION-1240


* refactor: Remove duplicate plugin.json

Claude Code also supports the root `plugin.json` file.

* refactor(ui5-typescript-conversion): Move skill to sub-dir

Although this seems to have worked with Claude Code, it is generally
required to place each skill in a separate sub-directory with the same
name as the skill.

* refactor(ui5): Do not list skills individually

* ci: Verify plugin.json copies stay in sync with root

Add CI step that fails when any registered copy of plugins/<name>/plugin.json
differs from the root file. Currently checks .github/plugin/plugin.json,
required by github/awesome-copilot external-plugin smoke test.

* ci: Add plugin.json copies for awesome-copilot smoke test

Copy each plugins/<name>/plugin.json to <name>/.github/plugin/plugin.json,
required by github/awesome-copilot external-plugin install verification.
Sync with root enforced by CI step added in previous commit.

* fix(ui5): Normalize keywords to lowercase-hyphenated form

The github/awesome-copilot marketplace's external-plugin validator
rejects keywords containing spaces or uppercase characters; only
lowercase letters, digits, and hyphens are allowed (max 30 chars,
max 10 entries per plugin).

Replace 'api documentation', 'development guidelines', and
'best practices' with their hyphenated equivalents so the plugin
passes intake validation when listed in plugins/external.json.

Reference:
- awesome-copilot/eng/external-plugin-validation.mjs (regex /^[a-z0-9-]+$/)
- awesome-copilot/CONTRIBUTING.md - 'Adding External Plugins'
  ('keywords as lowercase hyphenated tags')

* fix(ui5): Wrap MCP server config in mcpServers key

Align with the plugin spec, which requires a top-level "mcpServers"
wrapper around server entries. Both Claude Code and Copilot CLI
tolerated the flat shape, but the spec-compliant form is safer for
strict validators down the line.
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