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.

@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label May 25, 2026
@gonzaloriestra gonzaloriestra force-pushed the cli-interactivity/app-dev-flags branch from 970e05e to ea50335 Compare May 26, 2026 14:53
@gonzaloriestra gonzaloriestra force-pushed the cli-interactivity/theme-profile-token branch from 9b530e1 to b73773b Compare May 26, 2026 14:53
@gonzaloriestra gonzaloriestra force-pushed the cli-interactivity/theme-profile-token branch from b73773b to 8f6ca1c Compare May 26, 2026 15:22
@gonzaloriestra gonzaloriestra force-pushed the cli-interactivity/app-dev-flags branch from ea50335 to 3948ce1 Compare May 26, 2026 15:22
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