Skip to content

Publish shopify-dev-tools to Cloudsmith - #8152

Closed
durga256 wants to merge 1 commit into
test-shopify-dev-tools-cifrom
publish-shopify-dev-tools-cloudsmith
Closed

durga256 wants to merge 1 commit into
test-shopify-dev-tools-cifrom
publish-shopify-dev-tools-cloudsmith

Conversation

@durga256

@durga256 durga256 commented Jul 22, 2026 •

Copy link
Copy Markdown
Contributor

WHY are these changes introduced?

PR #8151 moves @shopify/shopify-dev-tools into this monorepo, but the package still needs a private release path before ownership can move away from Shopify/ai-toolkit-source.

This PR adds that release path while keeping the package private, avoiding public npm, and allowing the old publisher to remain operational during migration.

WHAT is this pull request doing?

  • Adds a dedicated Shopify Build pipeline that builds and packs @shopify/shopify-dev-tools, then publishes its tarball to Shopify's internal Cloudsmith node repository.
  • Adds an idempotent release job for main pushes and manual shopify-dev-tools retries.
  • Compares the package version with HEAD~1, checks Cloudsmith only after a version bump or manual retry, and triggers Buildkite only when that exact version is missing.
  • Uses a new cli-shopify-dev-tools-publish-package pipeline slug so the existing ai-toolkit-source publisher can coexist until the migration is verified.
  • Documents the package's independent Changesets versioning and private-registry release process.
  • Keeps private: true and publishConfig.access: restricted; no public npm publishing is introduced.

No changeset is included because this changes release infrastructure rather than package behavior or versioned contents.

How to test your changes?

Validated locally with:

  • YAML parsing for the GitHub Actions and Shopify Build configurations
  • node --check bin/resolve-package-publish-state.js
  • Synthetic resolver scenarios covering unchanged pushes, bumped/missing versions, already-published versions, manual retries, and missing credentials
  • pnpm exec prettier --check for the changed source/configuration files
  • pnpm knip
  • pnpm nx lint workspace --skip-nx-cache
  • pnpm nx lint shopify-dev-tools --skip-nx-cache
  • pnpm nx type-check shopify-dev-tools --skip-nx-cache
  • pnpm nx build shopify-dev-tools --skip-nx-cache
  • pnpm --filter @shopify/shopify-dev-tools pack (approximately 20 MB tarball)

Post-release steps

  • Register the new Shopify Build pipeline:
    @spy build setup cli-shopify-dev-tools-publish-package repo=Shopify/cli pr=false
  • Provision or verify BUILDKITE_API_TOKEN and CLOUDSMITH_NPM_TOKEN_RO for the Release workflow.
  • Keep the old Shopify/ai-toolkit-source publisher enabled until the full migration is merged and verified.

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 (not applicable to release infrastructure)
  • This is not a user-facing package change, so no changeset is required

durga256 commented Jul 22, 2026 •

Copy link
Copy Markdown
Contributor Author

@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 22, 2026
@durga256 durga256 added the includes-post-release-steps PRs including this label require additional steps after releasing label Jul 22, 2026
@durga256
durga256 marked this pull request as ready for review July 22, 2026 16:17
@durga256
durga256 requested a review from a team as a code owner July 22, 2026 16:17
@durga256
durga256 marked this pull request as draft July 22, 2026 16:52
@durga256
durga256 changed the base branch from move-shopify-dev-tools to graphite-base/8152 July 22, 2026 17:15
@durga256
durga256 force-pushed the publish-shopify-dev-tools-cloudsmith branch from f5e37df to 874099a Compare July 22, 2026 17:15
@durga256
durga256 force-pushed the graphite-base/8152 branch from 7d68cdd to 2933c1b Compare July 22, 2026 17:15
@durga256
durga256 changed the base branch from graphite-base/8152 to test-shopify-dev-tools-ci July 22, 2026 17:15
@durga256
durga256 force-pushed the test-shopify-dev-tools-ci branch from 2933c1b to d97a272 Compare July 22, 2026 18:45
@durga256
durga256 force-pushed the publish-shopify-dev-tools-cloudsmith branch from 874099a to 21a4398 Compare July 22, 2026 18:45
Assisted-By: devx/8aa8112b-1966-49eb-8fd0-855a849a29f9
@durga256
durga256 force-pushed the publish-shopify-dev-tools-cloudsmith branch from 21a4398 to 8124667 Compare July 22, 2026 18:52
@durga256
durga256 force-pushed the test-shopify-dev-tools-ci branch from d97a272 to 24291a5 Compare July 22, 2026 18:52
@github-actions

Copy link
Copy Markdown
Contributor

This PR seems inactive. If it's still relevant, please add a comment saying so. Otherwise, take no action.
→ If there's no activity within a week, then a bot will automatically close this.
Thanks for helping to improve Shopify's dev tooling and experience.

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

Labels

includes-post-release-steps PRs including this label require additional steps after releasing no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. no-pr-activity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant