Skip to content

Publish shopify-dev-tools to Cloudsmith#8152

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

Publish shopify-dev-tools to Cloudsmith#8152
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
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant