Skip to content

Add environment argument to aws:pull and aws:status#150

Merged
jonathonbyrdziak merged 1 commit into
masterfrom
fix/aws-pull-environment-argument
Mar 24, 2026
Merged

Add environment argument to aws:pull and aws:status#150
jonathonbyrdziak merged 1 commit into
masterfrom
fix/aws-pull-environment-argument

Conversation

@jonathonbyrdziak

Copy link
Copy Markdown
Contributor

Summary

  • aws:pull and aws:status now accept an optional environment argument, matching aws:push
  • Without it, behavior is unchanged (uses current config branch)
  • With it: protocol aws:pull production --stdout pulls from protocol/{project}/production
  • Added secretNameOverride parameter to pullSecret(), describeSecret(), and getSecretKeys() helpers

Test plan

  • protocol aws:pull production --stdout — should pull production secrets
  • protocol aws:pull --stdout — should still use current branch (backward compat)
  • protocol aws:status production — should show production secret metadata
  • protocol aws:push production — should still work as before

🤖 Generated with Claude Code

aws:push already lets you pick an environment, but aws:pull and aws:status
hardcoded the secret name from the current config branch. On local machines
where the branch is localhost-byrdziak, this meant you couldn't pull/check
production secrets.

Now all three commands accept an optional environment argument:
  protocol aws:pull production --stdout
  protocol aws:status production

Also adds secretNameOverride parameter to pullSecret, describeSecret, and
getSecretKeys helpers for consistency.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jonathonbyrdziak
jonathonbyrdziak merged commit f7c38ef into master Mar 24, 2026
5 of 8 checks passed
@jonathonbyrdziak
jonathonbyrdziak deleted the fix/aws-pull-environment-argument branch March 24, 2026 00:51
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.

1 participant