Add env push force confirmation flag#3831
Merged
Merged
Conversation
Contributor
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced Jul 2, 2026
gonzaloriestra
force-pushed
the
gdt-hydrogen-init-agent-flags
branch
from
3c106a6 to
e64ef52
Compare
gonzaloriestra
force-pushed
the
gdt-hydrogen-env-push-agent-confirm
branch
from
af9c736 to
b8922b7
Compare
Contributor
|
Oxygen deployed a preview of your
Learn more about Hydrogen's GitHub integration. |
gonzaloriestra
changed the base branch from
gdt-hydrogen-init-agent-flags
to
graphite-base/3831
gonzaloriestra
force-pushed
the
graphite-base/3831
branch
from
e64ef52 to
bccd91a
Compare
gonzaloriestra
force-pushed
the
gdt-hydrogen-env-push-agent-confirm
branch
from
b8922b7 to
020210c
Compare
gonzaloriestra
force-pushed
the
gdt-hydrogen-env-push-agent-confirm
branch
from
020210c to
3841045
Compare
gonzaloriestra
changed the title
lucyxiang
approved these changes
Jul 3, 2026
graygilmore
reviewed
Jul 3, 2026
gonzaloriestra
force-pushed
the
gdt-hydrogen-env-push-agent-confirm
branch
from
3841045 to
3f36df5
Compare
graygilmore
approved these changes
Jul 7, 2026
Add --force and --yes to hydrogen env push so agents can push an already-computed environment diff without answering the confirmation prompt.\n\nThe command still fetches environments, filters read-only and secret variables, detects no-op pushes, and shows the existing prompt by default. When either confirmation flag is set, it skips only the final diff confirmation and proceeds to push the local variables.\n\nTests cover both flag spellings and assert that the prompt is skipped while the push mutation receives the expected variables. The Oclif manifest and changeset are updated for the new flags.\n\nCloses shop/issues-develop#22875
gonzaloriestra
force-pushed
the
gdt-hydrogen-env-push-agent-confirm
branch
from
3f36df5 to
30d29d6
Compare
Contributor
Author
Merge activity
|
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
3 participants
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.
WHY are these changes introduced?
Fixes https://github.com/shop/issues-develop/issues/22875
shopify hydrogen env pushshowed a diff confirmation prompt that agents could not skip.WHAT is this pull request doing?
Adds
--forcetohydrogen env pushso environment variable pushes can proceed without the diff confirmation prompt.Also, adds
--dry-run(can't be used with force) to preview the diff without doing any change.HOW to test your changes?
Use a linked Hydrogen project with a changed local
.envfile:The push command should skip the environment variable diff confirmation prompt.