feature(connector_sdk): adding copilot cli support#4
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a GitHub Copilot CLI plugin variant of the existing Connector SDK agent tooling, and wires it into the repo’s plugin-sync and documentation so the same canonical workflows/skills/tools can be used from Copilot CLI.
Changes:
- Extend
scripts/sync-plugins.shto generate/sync acopilot/plugin directory alongside existing agent targets. - Add the Copilot CLI plugin tree (
copilot/) including agents, skills, commands, hooks, and secure config tooling. - Update top-level README and add Copilot marketplace metadata under
.github/plugin/.
Reviewed changes
Copilot reviewed 23 out of 23 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/sync-plugins.sh | Adds Copilot as a sync target for agents/skills/tools/reference docs. |
| README.md | Documents Copilot CLI install and updates repository layout notes. |
| copilot/tools/run_connector.py | Secure runner to decrypt config in-memory and pass to fivetran debug via named pipe. |
| copilot/tools/requirements.txt | Dependency list for Copilot plugin tooling. |
| copilot/tools/enter_configuration.py | Terminal-only credential entry + encryption helper writing to configuration.json. |
| copilot/tools/deploy_connector.py | Deploy helper that discovers destination via REST API and passes config via named pipe. |
| copilot/skills/test-connector/SKILL.md | Copilot skill definition for local testing flow. |
| copilot/skills/evaluate-connector/SKILL.md | Copilot skill definition for static connector evaluation. |
| copilot/skills/deploy-connector/SKILL.md | Copilot skill definition for deploy flow. |
| copilot/skills/build-connector/SKILL.md | Copilot skill definition for end-to-end connector generation flow. |
| copilot/sdk-reference.md | Generated SDK reference copy for Copilot plugin. |
| copilot/README.md | Copilot-specific tutorial and usage documentation. |
| copilot/native-connectors.md | Generated native connector catalog copy for Copilot plugin. |
| copilot/hooks/hooks.json | Copilot hook to remind users to run tests after edits. |
| copilot/commands/test-connector.md | Copilot slash-command entrypoint for test flow. |
| copilot/commands/evaluate-connector.md | Copilot slash-command entrypoint for evaluation flow. |
| copilot/commands/deploy-connector.md | Copilot slash-command entrypoint for deploy flow. |
| copilot/commands/build-connector.md | Copilot slash-command entrypoint for build flow. |
| copilot/agents/connector-validator.md | Generated validator agent for Copilot. |
| copilot/agents/connector-generator.md | Generated generator agent for Copilot. |
| copilot/agents/connector-fixer.md | Generated fixer agent for Copilot. |
| copilot/AGENTS.md | Copilot routing + credential security policy for the plugin. |
| .github/plugin/marketplace.json | Copilot marketplace manifest pointing to the copilot/ plugin. |
Comments suppressed due to low confidence (1)
README.md:79
- The usage table omits
/fivetran-connector-sdk:evaluate-connector, even though the plugin exposes it for Claude/Gemini/Copilot (and it’s listed elsewhere in this repo). Adding it here makes the install/usage docs consistent.
| Command (Claude Code / Gemini CLI / Copilot CLI) | Codex CLI | Purpose |
|---|---|---|
| `/fivetran-connector-sdk:build-connector` | `$build_connector` | Research an API and generate a new connector |
| `/fivetran-connector-sdk:test-connector` | `$test_connector` | Run and validate an existing connector locally |
| `/fivetran-connector-sdk:deploy-connector` | `$deploy_connector` | Deploy a connector to your Fivetran account |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
fivetran-rishabhghosh
previously approved these changes
Jun 3, 2026
35e35f3 to
0727c11
Compare
fivetran-rishabhghosh
approved these changes
Jun 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes