Skip to content

prefer cli, if not then mcp#29

Merged
pjcdawkins merged 9 commits intomainfrom
prefer-mcp-if-cli-not-available
May 4, 2026
Merged

prefer cli, if not then mcp#29
pjcdawkins merged 9 commits intomainfrom
prefer-mcp-if-cli-not-available

Conversation

@ganeshdipdumbare
Copy link
Copy Markdown
Contributor

  • prefer cli but if not available then use MCP

Copilot AI review requested due to automatic review settings April 30, 2026 08:01
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Upsun assistant skill documentation to prefer using the Upsun CLI when available, and fall back to Upsun MCP (or console) when the CLI isn’t reachable.

Changes:

  • Added an explicit “tooling availability” detection step and mode selection (CLI/MCP/console).
  • Reworked first-time setup and common workflows to present CLI vs MCP vs console actions.
  • Documented CLI-only operations and added MCP equivalents in workflow and confirmation guidance.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread plugins/upsun/skills/upsun/SKILL.md Outdated
Comment thread plugins/upsun/skills/upsun/SKILL.md Outdated
Comment thread plugins/upsun/skills/upsun/SKILL.md Outdated
Comment thread plugins/upsun/skills/upsun/SKILL.md Outdated
Comment thread plugins/upsun/skills/upsun/SKILL.md Outdated
ganeshdipdumbare and others added 5 commits April 30, 2026 15:01
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Comment thread plugins/upsun/skills/upsun/SKILL.md Outdated
**Before doing anything**, silently determine which tools are reachable:

1. Try `upsun --version` — if it succeeds, **CLI is available**.
2. Try `mcp__upsun__list-project` — if it succeeds, **MCP is available**.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think agent harnesses may already be told which MCPs are available?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I guess it can be simplified. I just added one liner for tooling preference now and removed all the other changes.

@pjcdawkins pjcdawkins merged commit 4abd4f9 into main May 4, 2026
1 check passed
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.

3 participants