Skip to content

feat(copilot): add skills, MCP, and plugin support - #166

Merged
gricha merged 13 commits into
getsentry:mainfrom
mishamilovidov:milovido-adobe-copilot-adapter-adoption
Sep 9, 2026
Merged

gricha merged 13 commits into
getsentry:mainfrom
mishamilovidov:milovido-adobe-copilot-adapter-adoption

Conversation

@mishamilovidov

@mishamilovidov mishamilovidov commented Aug 21, 2026 •

Copy link
Copy Markdown
Contributor

Add GitHub Copilot CLI as a thin adapter for dotagents' existing portable surfaces: project and global skills, project and global MCP configuration, and canonical Agent Plugins marketplaces. Copilot consumes the same installed skills and plugin bundles as the other harnesses; dotagents adds only Copilot's native paths, config reconciliation, and marketplace registration.

MCP reconciliation preserves unmanaged servers and both Copilot-supported document shapes, safely shares .mcp.json with Claude, repairs restrictive global config modes, and rejects non-regular config paths. Plugin handling follows Copilot's native marketplace and manifest precedence, canonicalizes supported alternate manifest locators, and fails fast on ambiguity or Copilot-native agents, commands, hooks, LSP, and executable extensions. Those guards keep this adapter limited to skills, MCP, and canonical plugin installation rather than expanding dotagents' portable component model.

Native no-auth QA with Copilot CLI 1.0.83 covered project and global discovery, sync repair, marketplace browsing and installation, live plugin skill and MCP inventory, .github/plugin/plugin.json canonicalization, and representative real plugin sources. The full repository suite passes with 1,131 tests, and the documentation build is clean.

Fixes #167

@vercel

vercel Bot commented Aug 21, 2026

Copy link
Copy Markdown

@mishamilovidov is attempting to deploy a commit to the Sentry Team on Vercel.

A member of the Team first needs to authorize it.

Add user and project MCP configuration for GitHub Copilot CLI and Desktop while preserving unmanaged native entries. Reuse Claude-compatible project serialization and protect the user MCP file with mode 0600 on POSIX systems.
@mishamilovidov
mishamilovidov force-pushed the milovido-adobe-copilot-adapter-adoption branch from c738fbe to 056e9db Compare August 21, 2026 23:31
@mishamilovidov
mishamilovidov marked this pull request as ready for review August 21, 2026 23:31
@github-actions github-actions Bot added the risk: high PR risk score: high label Aug 21, 2026
Preserve fallback MCP entries when Claude and Copilot converge on `.mcp.json`. Recover if a mode check races with external file deletion.
Propagate preferred-file write errors instead of reporting them as fallback read failures. Recreate JSONC state if the config file disappears during reconciliation.
@mishamilovidov

This comment was marked as outdated.

Generate a native Copilot marketplace from portable plugin bundles so Copilot can install their skills without the Pi target. Preserve managed-output ownership, pruning, and unchanged-write behavior across project and global scopes.
Integrate upstream changes through getsentry#180 while preserving the Copilot adapter behavior and tests.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit dd76614. Configure here.

Use Copilot 1.0.83 native commands to prove local marketplace registration.
Verify live skill discovery and both plugin MCP resources without relying on copied caches.
@mishamilovidov
mishamilovidov force-pushed the milovido-adobe-copilot-adapter-adoption branch from 635c874 to c366a28 Compare September 2, 2026 21:53
@gricha gricha changed the title feat: add github copilot target feat(copilot): add skills, MCP, and plugin support Sep 9, 2026
Comment thread packages/dotagents/src/plugins/store.ts Resolved
@gricha
gricha merged commit 60254b3 into getsentry:main Sep 9, 2026
12 of 13 checks passed
mishamilovidov added a commit to mishamilovidov/dotagents that referenced this pull request Sep 9, 2026
Reconcile the marketplace-name migration with Copilot plugin support (getsentry#166) and project-path containment (getsentry#182). The Copilot marketplace generation call site is corrected to match the two-argument pathMarketplace signature after the merge.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: high PR risk score: high

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support GitHub Copilot CLI and Desktop

2 participants