Skip to content

Allow theme profile with Theme Access tokens#7633

Draft
gonzaloriestra wants to merge 1 commit into
cli-interactivity/app-dev-flagsfrom
cli-interactivity/theme-profile-token
Draft

Allow theme profile with Theme Access tokens#7633
gonzaloriestra wants to merge 1 commit into
cli-interactivity/app-dev-flagsfrom
cli-interactivity/theme-profile-token

Conversation

@gonzaloriestra
Copy link
Copy Markdown
Contributor

@gonzaloriestra gonzaloriestra commented May 25, 2026

WHY are these changes introduced?

The interactivity audit found that theme profile required browser OAuth even when a Theme Access token was available for non-interactive runs.

WHAT is this pull request doing?

  • Allows theme profile to use Theme Access tokens.
  • Keeps the existing rejection for raw Admin API tokens with clearer guidance.

How to test your changes?

  • pnpm vitest run packages/theme/src/cli/services/profile.test.ts

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes
  • I've considered analytics changes to measure impact
  • The change is user-facing, so I've added a changelog entry with pnpm changeset add

Copy link
Copy Markdown
Contributor Author

gonzaloriestra commented May 25, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant