Skip to content

👷 [RUM-15055] fix release / publish pipeline issues from v0.1.2#87

Merged
bcaudan merged 2 commits into
mainfrom
bcaudan/fix-npm-publish-5
Apr 7, 2026
Merged

👷 [RUM-15055] fix release / publish pipeline issues from v0.1.2#87
bcaudan merged 2 commits into
mainfrom
bcaudan/fix-npm-publish-5

Conversation

@bcaudan

@bcaudan bcaudan commented Apr 7, 2026

Copy link
Copy Markdown
Collaborator

Motivation

Two issues discovered after the v0.1.2 release:

  1. npm publish 404: CI publishes were failing because an org-level NODE_AUTH_TOKEN secret seems to be present, preventing the OIDC Trusted Publishers exchange from running.
  2. Changelog generation: git describe --tags was failing to find the last release because release tags are created on the release branch and are not ancestors of main. This caused the changelog to include all commits instead of only those since the last release. Additionally, squash-merged release PRs produce titles like v0.1.1 (#80) which the regex didn't match.

Changes

  • Clear NODE_AUTH_TOKEN in the publish and dry-run steps so npm falls through to the OIDC Trusted Publishers exchange instead of using the invalid org secret
  • Replace git describe --tags with a log scan to find the last release commit, making it resilient to tags not being ancestors of main
  • Update RELEASE_COMMIT_RE to match squash-merged PR titles (e.g. v0.1.1 (#80))

Test instructions

  1. Trigger the publish workflow with dry_run: true from a tag — the OIDC check step should succeed without falling back to the org token
  2. Run node scripts/generate-changelog.ts --dry-run locally and verify the changelog only includes commits since the last release

Checklist

  • Tested locally (playground)
  • Added unit tests for this change.
  • Added e2e/integration tests for this change.
  • Updated related documentation.

bcaudan added 2 commits April 7, 2026 15:22
- scan git log instead of `git describe` to find last release commit (tags aren't ancestors of main)
- update RELEASE_COMMIT_RE to match squash-merged PR titles like `v0.1.1 (#80)`
@bcaudan bcaudan changed the title 👷[RUM-15055] fix changelog generation 👷 [RUM-15055] fix release / publish pipeline issues from v0.1.2 Apr 7, 2026
@bcaudan bcaudan force-pushed the bcaudan/fix-npm-publish-5 branch from fe52767 to ecf65ee Compare April 7, 2026 15:37
@bcaudan bcaudan marked this pull request as ready for review April 7, 2026 15:38
@bcaudan bcaudan requested a review from a team as a code owner April 7, 2026 15:38
@bcaudan bcaudan merged commit 1f1634c into main Apr 7, 2026
12 of 14 checks passed
@bcaudan bcaudan deleted the bcaudan/fix-npm-publish-5 branch April 7, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants