Skip to content

fix: id is legacy, we use environment now.#128

Merged
pjcdawkins merged 1 commit into
mainfrom
mr/matthias/fixes/correct-wrong-argument
Jul 13, 2026
Merged

fix: id is legacy, we use environment now.#128
pjcdawkins merged 1 commit into
mainfrom
mr/matthias/fixes/correct-wrong-argument

Conversation

@matthiaz

Copy link
Copy Markdown
Contributor

Else you get an error like:
The "id" argument does not exist.

Example:

upsun branch --type="development" "support_ticket_branch_name" "main" --yes -vvv && git checkout main && upsun branch --type="development" "support_ticket_branch_name" "main" --yes
DEBUG Finding the project root
DEBUG Project root found: /home/matthiasvanwoensel/Platform.Sh/creakiwi/phoenix_demo
DEBUG Project "yxikwfhfdu6zq" is mapped to the current directory
DEBUG Determined CA bundle path: /etc/pki/ca-trust/extracted/pem/tls-ca-bundle.pem
DEBUG Using Docker credential helper for session storage
DEBUG Loading session
DEBUG Loaded project from cache: yxikwfhfdu6zq
DEBUG Selecting environment based on input argument
DEBUG Loaded environments from cache
Selected project: phoenix demo (yxikwfhfdu6zq)
Selected environment: main (type: production)
The environment support_ticket_branch_name already exists. Check out? [Y/n] y
DEBUG Running sub-command: environment:checkout

In ArrayInput.php line 186:
                                                                  
  [Symfony\Component\Console\Exception\InvalidArgumentException]  
  The "id" argument does not exist.                               
                                                                  

Exception trace:
  at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/vendor/symfony/console/Input/ArrayInput.php:186
 Symfony\Component\Console\Input\ArrayInput->addArgument() at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/vendor/symfony/console/Input/ArrayInput.php:125
 Symfony\Component\Console\Input\ArrayInput->parse() at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/vendor/symfony/console/Input/Input.php:53
 Symfony\Component\Console\Input\Input->bind() at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/vendor/symfony/console/Command/Command.php:295
 Symfony\Component\Console\Command\Command->run() at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/vendor/symfony/console/Application.php:1117
 Symfony\Component\Console\Application->doRunCommand() at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/src/Application.php:397
 Platformsh\Cli\Application->doRunCommand() at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/vendor/symfony/console/Application.php:356
 Symfony\Component\Console\Application->doRun() at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/vendor/symfony/console/Application.php:195
 Symfony\Component\Console\Application->run() at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/src/Service/SubCommandRunner.php:57
 Platformsh\Cli\Service\SubCommandRunner->run() at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/src/Command/Environment/EnvironmentBranchCommand.php:114
 Platformsh\Cli\Command\Environment\EnvironmentBranchCommand->execute() at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/vendor/symfony/console/Command/Command.php:341
 Symfony\Component\Console\Command\Command->run() at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/vendor/symfony/console/Application.php:1117
 Symfony\Component\Console\Application->doRunCommand() at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/src/Application.php:397
 Platformsh\Cli\Application->doRunCommand() at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/vendor/symfony/console/Application.php:356
 Symfony\Component\Console\Application->doRun() at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/vendor/symfony/console/Application.php:195
 Symfony\Component\Console\Application->run() at phar:///home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar/bin/platform:32
 require() at /home/matthiasvanwoensel/.cache/upsun-tmp/legacy-5.10.7/upsun.phar:4

upsun checkout [-h|--help] [-V|--version] [-v|vv|vvv|--verbose] [-q|--quiet] [-y|--yes] [--no-interaction] [--] <command> [<environment>]

Else you get an error like: 
`The "id" argument does not exist.`
Copilot AI review requested due to automatic review settings July 13, 2026 11:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a legacy sub-command invocation so that environment:checkout is called with the correct argument name (environment) instead of the removed/unsupported legacy name (id), preventing InvalidArgumentException: The "id" argument does not exist. in upsun branch flows.

Changes:

  • Update EnvironmentBranchCommand to pass environment (not id) when running environment:checkout via SubCommandRunner.

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

@pjcdawkins pjcdawkins merged commit a53ae44 into main Jul 13, 2026
5 checks passed
@pjcdawkins pjcdawkins deleted the mr/matthias/fixes/correct-wrong-argument branch July 13, 2026 12:32
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