Skip to content

CEXT-6421: Publish previews in npm#566

Merged
obarcelonap merged 6 commits into
mainfrom
previews-in-npm
Jul 14, 2026
Merged

CEXT-6421: Publish previews in npm#566
obarcelonap merged 6 commits into
mainfrom
previews-in-npm

Conversation

@obarcelonap

Copy link
Copy Markdown
Member

Description

Preview (alpha, per-PR /snapshot) and internal (beta) package snapshots now publish to the public npm registry instead of Artifactory, reusing the same npm token and publishConfig that stable releases already use. Also drops the now-unused release-channel branching in the Slack announcement, defaults the npm registry URL in the shared setup action, and fixes a leftover pre-release notice that incorrectly said beta snapshots weren't publicly available.

Related Issue

N/A

Motivation and Context

Artifactory-hosted snapshots caused two recurring problems:

  • Consumers testing a preview from another public GitHub repo couldn't install alpha/beta packages at all, since Artifactory isn't reachable without internal credentials.
  • Anyone who did have access needed to keep tweaking their local .npmrc to point at Artifactory, which was a poor developer experience for something as routine as trying out a preview build.

Publishing snapshots to the public npm registry removes both frictions, at the cost of alpha/beta versions being publicly visible (mitigated via the existing deprecate workflow if one needs to be pulled back).

How Has This Been Tested?

  • pnpm test for the affected scripts package (40/40 passing).
  • pnpm typecheck across the whole monorepo.
  • pnpm lint (biome) on all changed files.
  • Manual review of the updated workflow YAML for the preview, internal, and public publish flows.

Screenshots (if appropriate):

N/A

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have read the DEVELOPMENT document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@changeset-bot

changeset-bot Bot commented Jul 10, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: fe8c819

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions github-actions Bot added without-changeset The PR does not contain a Changeset file scripts Includes changes in `scripts` labels Jul 10, 2026
Comment thread .github/actions/setup-release/action.yml Outdated
The merge duplicated content: PR #567 moved the snapshot/release/back-sync
sections from DEVELOPMENT.md into the new MAINTAINERS.md, while PR #566
independently reworded that same content in DEVELOPMENT.md (Artifactory ->
public npm registry). Git's line-based merge kept both instead of following
the move, so DEVELOPMENT.md ended up with a stale duplicate and
MAINTAINERS.md kept the outdated Artifactory wording.
@obarcelonap obarcelonap merged commit 9712633 into main Jul 14, 2026
5 checks passed
@obarcelonap obarcelonap deleted the previews-in-npm branch July 14, 2026 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scripts Includes changes in `scripts` without-changeset The PR does not contain a Changeset file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants