Skip to content

Store auth/execute UX improvements#7151

Merged
isaacroldan merged 1 commit intomainfrom
dlm-store-execute-ux-improvements
Apr 2, 2026
Merged

Store auth/execute UX improvements#7151
isaacroldan merged 1 commit intomainfrom
dlm-store-execute-ux-improvements

Conversation

@dmerand
Copy link
Copy Markdown
Contributor

@dmerand dmerand commented Apr 1, 2026

What

Improve the shopify store auth and shopify store execute UX around authentication, verification, and recovery.

This changes the store auth output to follow the existing shopify auth style more closely, makes auth and re-auth commands easier to follow from error states, and updates the help text to describe store auth as store-command auth rather than store execute-specific auth.

Why

These changes follow the feedback collected in the Store execute friction doc:
https://docs.google.com/document/d/1Hwo9kFlIrEitsvS7i27q179_TiMbE9YNRZmn-j77G7s/edit?tab=t.0

The current flow has a few sources of friction:

  • store auth output still reads like a bespoke flow instead of matching the CLI's auth conventions
  • long auth and re-auth commands are hard to scan in wrapped error messages
  • the help text is narrower than reviewer intent and ties store auth too closely to store execute
  • the post-auth verification step is present, but the guidance can be clearer

How

  • switch store auth happy-path output to login-style completion lines
  • keep the auth URL as fallback-only, but render it as a hyperlink-capable link when shown
  • separate the post-auth verification command onto its own line
  • move missing-auth and re-auth commands into structured error next steps
  • update generated command docs/help output for the revised wording

Testing

  • Run shopify store auth --store <store> --scopes read_products
  • Confirm the auth flow prints login-style success output and a follow-up verification command
  • Simulate missing stored auth and confirm the recovery command is shown in the error next steps
  • Simulate browser-open failure and confirm the fallback auth URL is shown as a clickable link when supported

Copy link
Copy Markdown
Contributor Author

dmerand commented Apr 1, 2026

@dmerand dmerand force-pushed the dlm-store-execute-ux-improvements branch from b179a5b to 30ad26e Compare April 1, 2026 23:08
@dmerand dmerand marked this pull request as ready for review April 1, 2026 23:17
@dmerand dmerand requested review from a team as code owners April 1, 2026 23:17
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

We detected some changes at packages/*/src and there are no updates in the .changeset.
If the changes are user-facing, run pnpm changeset add to track your changes and include them in the next release CHANGELOG.

Caution

DO NOT create changesets for features which you do not wish to be included in the public changelog of the next CLI release.

@dmerand dmerand force-pushed the dlm-store-execute-ux-improvements branch from 30ad26e to 97a6535 Compare April 1, 2026 23:34
@dmerand dmerand force-pushed the dlm-store-execute-ux-improvements branch from 97a6535 to 09f967a Compare April 1, 2026 23:42
Copy link
Copy Markdown
Contributor Author

dmerand commented Apr 1, 2026

/snapit

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

🫰✨ Thanks @dmerand! Your snapshot has been published to npm.

Test the snapshot by installing your package globally:

npm i -g --@shopify:registry=https://registry.npmjs.org @shopify/cli@0.0.0-snapshot-20260401235335

Caution

After installing, validate the version by running shopify version in your terminal.
If the versions don't match, you might have multiple global instances installed.
Use which shopify to find out which one you are running and uninstall it.

return new AbortError(
`No stored app authentication found for ${store}.`,
'To create stored auth for this store, run:',
storeAuthCommandNextSteps(store, '<comma-separated-scopes>'),
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.

couldn't we pass the scopes here and build the full command?

Base automatically changed from dlm-store-execute-canonicalize-store to main April 2, 2026 08:25
@isaacroldan isaacroldan added this pull request to the merge queue Apr 2, 2026
Merged via the queue into main with commit 2477fe9 Apr 2, 2026
25 checks passed
@isaacroldan isaacroldan deleted the dlm-store-execute-ux-improvements branch April 2, 2026 08:46
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