Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refactors the Upsun “skills” documentation by removing the legacy using-upsun skill content and introducing a new consolidated upsun skill with a smaller reference set, plus a minor Cursor plugin manifest tweak.
Changes:
- Remove the
plugins/upsun/skills/using-upsun/skill and its reference docs. - Add a new
plugins/upsun/skills/upsun/skill entrypoint and a.upsun/config.yamlreference template page. - Minor formatting/punctuation update in the Cursor plugin manifest.
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| plugins/upsun/skills/using-upsun/references/troubleshooting.md | Deleted legacy troubleshooting reference doc. |
| plugins/upsun/skills/using-upsun/references/services-databases.md | Deleted legacy services/databases reference doc. |
| plugins/upsun/skills/using-upsun/references/resources-scaling.md | Deleted legacy resources/scaling reference doc. |
| plugins/upsun/skills/using-upsun/references/projects-organizations.md | Deleted legacy projects/organizations reference doc. |
| plugins/upsun/skills/using-upsun/references/integration-variables.md | Deleted legacy integrations/variables reference doc. |
| plugins/upsun/skills/using-upsun/references/environments.md | Deleted legacy environments reference doc. |
| plugins/upsun/skills/using-upsun/references/development-tools.md | Deleted legacy development tools reference doc. |
| plugins/upsun/skills/using-upsun/references/deployments.md | Deleted legacy deployments reference doc. |
| plugins/upsun/skills/using-upsun/references/backups.md | Deleted legacy backups reference doc. |
| plugins/upsun/skills/using-upsun/references/access-security.md | Deleted legacy access/security reference doc. |
| plugins/upsun/skills/using-upsun/references/COMMAND-INDEX.md | Deleted legacy command index reference doc. |
| plugins/upsun/skills/using-upsun/SKILL.md | Deleted legacy using-upsun skill entrypoint. |
| plugins/upsun/skills/upsun/references/config.md | Added .upsun/config.yaml reference templates and common type values. |
| plugins/upsun/skills/upsun/SKILL.md | Added new consolidated Upsun skill entrypoint and workflows. |
| plugins/upsun/.cursor-plugin/plugin.json | Minor manifest description punctuation change. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
pjcdawkins
reviewed
Apr 8, 2026
pjcdawkins
reviewed
Apr 8, 2026
|
|
||
| ## Safety rules | ||
|
|
||
| - `backup:restore` → create a safety backup of current state first, then confirm |
Collaborator
There was a problem hiding this comment.
not sure what this means?
Contributor
Author
There was a problem hiding this comment.
removed it as it was just in the old one but doesnt make sense.
pjcdawkins
reviewed
Apr 8, 2026
pjcdawkins
added a commit
that referenced
this pull request
Apr 9, 2026
SKILL.md: - Add "How Upsun works" section covering core concepts (environments, build vs runtime, relationship env vars, .environment file) - Fix deployment strategy: explain rolling vs stopstart with correct CLI flags (--deploy-strategy, --strategy) - Improve env var section: env: prefix, --visible-build, project vs environment level - Clarify backup trade-offs (standard consistency vs live downtime) - Fix resources: container profiles, --resources-init, horizontal scaling - Add source integration warning in Deploy workflow - Add project:set-remote fallback in Step 1 - Remove disk from YAML config (resources are API-managed) - Consolidate safety rules, remove --browser rule - Address review comments from PR #14 (lines 178, 184, 188) references/config.md: - Add hook phases and .environment file documentation - Add build.flavor: none and set -ex to all examples - Consolidate near-duplicate YAML templates into one plus incremental "Adding services" snippet - Add monorepo example with source.root - Remove disk key from examples - Add PHP-specific notes Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
pjcdawkins
added a commit
that referenced
this pull request
Apr 10, 2026
SKILL.md: - Add "How Upsun works" section covering core concepts (environments, build vs runtime, relationship env vars, .environment file) - Fix deployment strategy: explain rolling vs stopstart with correct CLI flags (--deploy-strategy, --strategy) - Improve env var section: env: prefix, --visible-build, project vs environment level - Clarify backup trade-offs (standard consistency vs live downtime) - Fix resources: container profiles, --resources-init, horizontal scaling - Add source integration warning in Deploy workflow - Add project:set-remote fallback in Step 1 - Remove disk from YAML config (resources are API-managed) - Consolidate safety rules, remove --browser rule - Address review comments from PR #14 (lines 178, 184, 188) references/config.md: - Add hook phases and .environment file documentation - Add build.flavor: none and set -ex to all examples - Consolidate near-duplicate YAML templates into one plus incremental "Adding services" snippet - Add monorepo example with source.root - Remove disk key from examples - Add PHP-specific notes Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
pjcdawkins
added a commit
that referenced
this pull request
Apr 10, 2026
SKILL.md: - Add "How Upsun works" section covering core concepts (environments, build vs runtime, relationship env vars, .environment file) - Fix deployment strategy: explain rolling vs stopstart with correct CLI flags (--deploy-strategy, --strategy) - Improve env var section: env: prefix, --visible-build, project vs environment level - Clarify backup trade-offs (standard consistency vs live downtime) - Fix resources: container profiles, --resources-init, horizontal scaling - Add source integration warning in Deploy workflow - Add project:set-remote fallback in Step 1 - Remove disk from YAML config (resources are API-managed) - Consolidate safety rules, remove --browser rule - Address review comments from PR #14 (lines 178, 184, 188) references/config.md: - Add hook phases and .environment file documentation - Add build.flavor: none and set -ex to all examples - Consolidate near-duplicate YAML templates into one plus incremental "Adding services" snippet - Add monorepo example with source.root - Remove disk key from examples - Add PHP-specific notes Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
SKILL.md: - Add "How Upsun works" section (environments, build vs runtime, relationship env vars, .environment file) - Fix deployment strategy: rolling vs stopstart with correct CLI flags - Improve env vars: env: prefix, --visible-build, project vs env level, auto-deploy behavior - Clarify backup trade-offs (standard consistency vs live downtime) - Fix resources: container profiles, --resources-init, horizontal scaling - Add source integration warning in Deploy workflow - Add project:set-remote fallback in Step 1 - Remove disk from YAML config (resources are API-managed) - Broaden allowed-tools for read-only CLI commands - Consolidate safety rules references/config.md: - Add hook phases and .environment file documentation - Add build.flavor: none and set -ex to all examples - Consolidate near-duplicate YAML templates - Add monorepo example with source.root - Remove disk key, add PHP-specific notes Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- upsun push works with source integrations (pushes to the source repo), but advanced git-push options are unavailable - Clarify deploy vs redeploy: deploy is for staged changes, redeploy re-deploys current state (e.g. after project-level variable changes) - Explain env:deploy:type manual for batching variable changes Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
pjcdawkins
requested changes
Apr 20, 2026
Collaborator
pjcdawkins
left a comment
pjcdawkins
left a comment
There was a problem hiding this comment.
Suggesting changes in #16
* fix: address review comments on skill refactor - SKILL.md: switch allowed-tools frontmatter to comma separation so read-only patterns are parsed as separate entries. - SKILL.md: correct deploy strategy default (stopstart, not rolling); rolling is opt-in for zero-downtime deploys. - SKILL.md: fix env:deploy:type syntax — it's a command, not a property set via `set`. - SKILL.md: clarify resources:set uses numeric CPU sizes; container profile (HIGH_CPU/BALANCED/HIGH_MEMORY/HIGHER_MEMORY) is set in .upsun/config.yaml, not via CLI. - SKILL.md: update install commands to github.com/upsun/cli (CLI repo moved from platformsh/cli) and note Alpine/Debian/RPM packages. - SKILL.md: qualify MCP tool names with mcp__upsun__ prefix. - README.md: replace stale `using-upsun` references with `upsun`; simplify skill documentation section now that there is one reference file instead of eleven. Update install commands to upsun/cli. - run-evals.yml: rename step to "Install upsun skill"; update CLI installer URL; drop redundant PATH export ($HOME/.local/bin is on PATH by default on ubuntu-latest runners). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * Update plugins/upsun/skills/upsun/SKILL.md Co-authored-by: Ganeshdip Dumbare <ganeshdip.dumbare@gmail.com> --------- Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Co-authored-by: Ganeshdip Dumbare <ganeshdip.dumbare@gmail.com>
pjcdawkins
approved these changes
Apr 21, 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.
Uh oh!
There was an error while loading. Please reload this page.