Skip to content

feat: add GitHub Copilot CLI platform support#277

Open
JiaqiZhang-Dev wants to merge 4 commits into
nextlevelbuilder:mainfrom
JiaqiZhang-Dev:copilot/add-copilot-cli-support
Open

feat: add GitHub Copilot CLI platform support#277
JiaqiZhang-Dev wants to merge 4 commits into
nextlevelbuilder:mainfrom
JiaqiZhang-Dev:copilot/add-copilot-cli-support

Conversation

@JiaqiZhang-Dev

@JiaqiZhang-Dev JiaqiZhang-Dev commented Apr 24, 2026

Copy link
Copy Markdown

The skill supports GitHub Copilot in VS Code (.github/prompts/) but not the GitHub Copilot CLI. This adds copilot-cli as a new platform type, installing to .github/skills/ui-ux-pro-max/.

Changes

  • Platform config: New copilot-cli.json in both src/ui-ux-pro-max/templates/platforms/ and cli/assets/templates/platforms/
  • Type registration (cli/src/types/index.ts): Added copilot-cli to AIType, AI_TYPES, and AI_FOLDERS (maps to ['.github'])
  • Detection (cli/src/utils/detect.ts): Auto-detects via .github/copilot-instructions.md
  • Template mapping (cli/src/utils/template.ts): Added AI_TO_PLATFORM entry
  • README: Added to install examples and supported platforms list
uipro init --ai copilot-cli   # GitHub Copilot CLI

Tests

image

Copilot AI and others added 4 commits April 24, 2026 03:57
Add 'copilot-cli' as a new platform type for the GitHub Copilot
coding agent. This installs the skill to .github/skills/ui-ux-pro-max/
(separate from VS Code Copilot's .github/prompts/ path).

Changes:
- New platform config copilot-cli.json (src + cli assets)
- Add copilot-cli to AIType, AI_TYPES, AI_FOLDERS
- Add detection via .github/copilot-instructions.md
- Add AI_TO_PLATFORM mapping
- Update README with new platform

Agent-Logs-Url: https://github.com/JiaqiZhang-Dev/ui-ux-pro-max-skill/sessions/837ef0ab-1be3-4164-b4d9-0d7b3b3d8fd8

Co-authored-by: JiaqiZhang-Dev <194873822+JiaqiZhang-Dev@users.noreply.github.com>
…port

feat: add GitHub Copilot CLI platform support
@JiaqiZhang-Dev JiaqiZhang-Dev changed the title Copilot/add copilot cli support feat: add GitHub Copilot CLI platform support Apr 24, 2026
@JiaqiZhang-Dev JiaqiZhang-Dev marked this pull request as ready for review April 24, 2026 05:08

@mrgoonie mrgoonie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Summary: Copilot CLI support is a useful platform addition and the PR updates both canonical src/ templates and packaged CLI assets, but the proposed install path is not aligned with GitHub Copilot custom instruction conventions and will likely not be auto-loaded by Copilot.

Risk level: Medium

Mandatory gates:

  • Duplicate/prior implementation: overlap checked — #328 covers VS Code Copilot prompt-file behavior, but no existing Copilot CLI platform PR implements this copilot-cli target.
  • Project standards: issue found — source and CLI asset copies are both added, but platform detection/install semantics need to match the target assistant.
  • Strategic necessity: clear value — first-class Copilot CLI support would reduce manual install friction for GitHub Copilot users.
  • CI/checks: locally blocked — this environment has no bun, so npm --prefix cli run build cannot complete; the screenshot only proves a skill-like command is loaded, not that GitHub Copilot CLI will consume .github/skills.

Findings:

  • Important: The new platform installs SKILL.md under .github/skills/ui-ux-pro-max/, but the detection logic keys off .github/copilot-instructions.md, and the repository's existing Copilot support uses .github/prompts/. I don't see evidence in the PR that GitHub Copilot CLI auto-loads arbitrary .github/skills/*/SKILL.md directories. Please verify the documented Copilot CLI/custom-instructions location and adjust the platform config accordingly, or include a link/test showing .github/skills/ui-ux-pro-max/SKILL.md is actually discovered by Copilot CLI.
  • Important: detectAIType() will detect both copilot and copilot-cli whenever .github/copilot-instructions.md exists, because it first pushes copilot for any .github directory and then pushes copilot-cli. That makes the suggested type become all, so a Copilot CLI project can get all-platform install behavior instead of the specific copilot-cli target. Please make the detection precedence unambiguous.

Verdict: REQUEST_CHANGES

@mrgoonie mrgoonie added agent:github-maintain Processed by github-maintain automation pr:reviewed PR reviewed by maintain workflow pr:changes-requested Maintain review requested changes labels Jun 22, 2026
@mrgoonie mrgoonie mentioned this pull request Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent:github-maintain Processed by github-maintain automation pr:changes-requested Maintain review requested changes pr:reviewed PR reviewed by maintain workflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants