Skip to content

TRAC-1250: Add missing storefront_api scope to device login flow#3116

Merged
jordanarldt merged 1 commit into
canaryfrom
TRAC-1250
Jul 17, 2026
Merged

TRAC-1250: Add missing storefront_api scope to device login flow#3116
jordanarldt merged 1 commit into
canaryfrom
TRAC-1250

Conversation

@jordanarldt

@jordanarldt jordanarldt commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Jira: TRAC-1193

What/Why?

The device login flow's OAuth scope list was missing store_storefront_api, so tokens issued via catalyst auth didn't grant access to the storefront API. Adds that single scope to the existing list.

Rollout/Rollback

Standard release; no migration or flag required. Revert the commit to roll back.

Testing

Ran catalyst auth login locally and confirmed the device authorization request includes store_storefront_api in the requested scopes, and that CLI flows depending on the storefront API no longer fail with a 401/403.

@changeset-bot

changeset-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: b951e28

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@jordanarldt
jordanarldt marked this pull request as ready for review July 16, 2026 22:52
@jordanarldt
jordanarldt requested a review from a team as a code owner July 16, 2026 22:52
@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
catalyst Ready Ready Preview, Comment Jul 17, 2026 3:34pm

Request Review

@github-actions

github-actions Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Unlighthouse Performance Comparison — Vercel

Comparing PR preview deployment Unlighthouse scores vs production Unlighthouse scores.

Summary Score

Aggregate score across all categories as reported by Unlighthouse.

Prod Desktop Prod Mobile Preview Desktop Preview Mobile
Score 90 93 92 94

Category Scores

Category Prod Desktop Prod Mobile Preview Desktop Preview Mobile
Performance 71 83 71 83
Accessibility 95 92 95 92
Best Practices 100 100 100 100
SEO 88 100 88 100

Core Web Vitals

Metric Prod Desktop Prod Mobile Preview Desktop Preview Mobile
LCP 5.0 s 4.5 s 5.6 s 4.6 s
CLS 0.037 0 0.037 0
FCP 1.2 s 1.1 s 1.1 s 1.1 s
TBT 0 ms 0 ms 0 ms 0 ms
Max Potential FID 50 ms 50 ms 40 ms 30 ms
Time to Interactive 5.0 s 4.6 s 5.6 s 4.7 s

Full Unlighthouse report →

@github-actions

Copy link
Copy Markdown
Contributor

Bundle Size Report

Comparing against baseline from 064d1ee (2026-07-17).

No bundle size changes detected.

@jordanarldt
jordanarldt added this pull request to the merge queue Jul 17, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 17, 2026
@jordanarldt
jordanarldt added this pull request to the merge queue Jul 17, 2026
Merged via the queue into canary with commit 3cbfcb3 Jul 17, 2026
25 of 26 checks passed
@jordanarldt
jordanarldt deleted the TRAC-1250 branch July 17, 2026 16:58
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