Skip to content

Regenerate command tree snapshot#8094

Merged
isaacroldan merged 1 commit into
mainfrom
maintenance-regenerate-command-snapshot
Jul 16, 2026
Merged

Regenerate command tree snapshot#8094
isaacroldan merged 1 commit into
mainfrom
maintenance-regenerate-command-snapshot

Conversation

@alfonso-noriega

@alfonso-noriega alfonso-noriega commented Jul 15, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

WHY are these changes introduced?

The E2E command tree snapshot is stale after new shopify store subcommands were added. This causes shopify commands --tree matches snapshot to fail in PR CI.

WHAT is this pull request doing?

Regenerates packages/e2e/data/snapshots/commands.txt from latest origin/main so it includes the current store command tree.

How to test your changes?

pnpm nx run cli:build --skip-nx-cache
pnpm test:regenerate-snapshots
pnpm --dir packages/e2e exec playwright test tests/commands.spec.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
  • This change is generated test snapshot maintenance; no changeset is needed.

@alfonso-noriega
Regenerate command tree snapshot
83787a9 
Assisted-By: devx/d9c9b8a1-2521-4fdd-af59-d77381c6f0ee
@alfonso-noriega alfonso-noriega changed the title Add non-interactive app dev prompt bypasses Regenerate command tree snapshot Jul 15, 2026
@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 Jul 15, 2026
@alfonso-noriega Graphite App

alfonso-noriega commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

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

@NathanPJF NathanPJF mentioned this pull request Jul 15, 2026
Merged
@NathanPJF
NathanPJF marked this pull request as ready for review July 15, 2026 20:50
@NathanPJF
NathanPJF requested a review from a team as a code owner July 15, 2026 20:50
NathanPJF
NathanPJF approved these changes Jul 15, 2026
View reviewed changes

@NathanPJF NathanPJF left a comment

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.

LGTM - I see that these were added in e64abd3#diff-e2825d1d03042adbe26fcc68c3c42d4bb6b85db6517de3b5e64b9994a4a02cb2 to main, but the test suite wasn't updated.

@isaacroldan
isaacroldan added this pull request to the merge queue Jul 16, 2026
Merged via the queue into main with commit dc844e0 Jul 16, 2026
30 checks passed
@isaacroldan
isaacroldan deleted the maintenance-regenerate-command-snapshot branch July 16, 2026 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@isaacroldan isaacroldan isaacroldan approved these changes
@NathanPJF NathanPJF NathanPJF approved these changes

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alfonso-noriega @isaacroldan @NathanPJF