Skip to content

Specify credential file#81

Merged
danhill-stripe merged 5 commits intomainfrom
danhill/specify-credential-file
May 8, 2026
Merged

Specify credential file#81
danhill-stripe merged 5 commits intomainfrom
danhill/specify-credential-file

Conversation

@danhill-stripe
Copy link
Copy Markdown
Contributor

Allow clients to specify the path to read/write auth credentials with --auth path/to/file.json

@danhill-stripe danhill-stripe requested a review from a team as a code owner May 7, 2026 14:48
Copy link
Copy Markdown
Contributor

@raubrey-stripe raubrey-stripe left a comment

Choose a reason for hiding this comment

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

Overall I think this looks fine, probably want to merge tomorrow + bake + manually test before a new release!

Comment thread packages/sdk/src/utils/storage.ts
Comment thread CLAUDE.md Outdated
Comment thread README.md Outdated
@raubrey-stripe raubrey-stripe mentioned this pull request May 8, 2026
Merge new shipping-address, user-info commands and DISPLAY_DELAY_MS
constant from main while preserving authStorage prop threading from
the --auth feature branch.

Committed-By-Agent: claude
Make requireAuth a factory and requireAuthGuard accept optional storage
so the --auth flag actually controls which credential file is checked.

Committed-By-Agent: claude
@danhill-stripe danhill-stripe merged commit 281b61c into main May 8, 2026
5 checks passed
@danhill-stripe danhill-stripe deleted the danhill/specify-credential-file branch May 8, 2026 16:33
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.

2 participants