Skip to content

Add GitHub Copilot CLI adapter (closes #94)#105

Open
madeleinesong wants to merge 5 commits intomainfrom
feat/copilot-cli-adapter
Open

Add GitHub Copilot CLI adapter (closes #94)#105
madeleinesong wants to merge 5 commits intomainfrom
feat/copilot-cli-adapter

Conversation

@madeleinesong
Copy link
Copy Markdown
Collaborator

Summary

  • Adds copilot-cli runtime adapter for GitHub Copilot CLI support
  • Implements the full adapter interface: install, uninstall, tool name translation, config management
  • Follows the established adapter pattern (modeled on OpenCode adapter)

Closes #94

Changes

  • New: src/gpd/adapters/copilot_cli.py — CopilotCliAdapter class
  • Updated: src/gpd/adapters/runtime_catalog.json — copilot-cli runtime entry
  • New: tests/adapters/test_copilot_cli.py — adapter tests
  • New: docs/github-copilot-cli.md — quickstart guide
  • Updated: README.md, docs/README.md, docs/macos.md, docs/windows.md, docs/linux.md — add copilot-cli to runtime lists
  • Updated: tests/repo_graph_contract.json, tests/README.md — repo contracts

Test plan

  • pytest tests/adapters/test_copilot_cli.py passes (40 tests)
  • pytest tests/adapters/ passes (750 passed, no regressions)
  • pytest tests/ passes (3814 passed, 5 skipped)

Limitations

  • Copilot CLI's exact tool surface and config format may evolve; adapter uses reasonable defaults based on available documentation
  • MCP server configuration follows the pattern used by other adapters
  • Permission sync is a no-op since Copilot CLI does not expose a documented approval control

- Remove unused imports replace_placeholders, strip_sub_tags (F401)
- Remove unused variable uninstall_result (F841)
- Fix import sort order in test_copilot_cli.py (I001)
- Bump adapters scope count 9→10 in repo_graph_contract.json and README.md
@SergioHC95 SergioHC95 self-requested a review April 15, 2026 01:09
@SergioHC95 SergioHC95 closed this Apr 15, 2026
@SergioHC95 SergioHC95 reopened this Apr 15, 2026
@SergioHC95
Copy link
Copy Markdown
Collaborator

@madeleinesong approve and ready for merge when deconflicted and tests pass

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.

[Feature]: could you please add support to github copilot CLI?

2 participants