Skip to content

chore: publish npm via DK-CICD instead of NPM_TOKEN - #58

Open
caroolcanelas2 wants to merge 1 commit into
masterfrom
chore/update-npm-publish-pipeline
Open

chore: publish npm via DK-CICD instead of NPM_TOKEN#58
caroolcanelas2 wants to merge 1 commit into
masterfrom
chore/update-npm-publish-pipeline

Conversation

@caroolcanelas2

Copy link
Copy Markdown
Contributor

This pull request updates the CI/CD and npm publishing workflows to modernize and improve the release process. The main changes are the removal of legacy GitHub Actions workflows, the introduction of a new workflow leveraging AWS CodeArtifact and GitHub Trusted Publishing, and the integration of this workflow into the DK-CICD pipeline.

Publishing and CI/CD workflow updates:

  • Added a new workflow, publish-npm.yml, to publish packages from AWS CodeArtifact to npm using GitHub Trusted Publishing and environment-based tagging (production or beta). This workflow uses modern Node.js versions and improved credential management.
  • Integrated the new publish-npm.yml workflow into the DK-CICD pipeline via the npm-publish-v1 step in .vtex/deployment.yaml, enabling publishing on tag push events.

Removal of legacy workflows:

  • Removed the old CI workflow (ci.yml) that handled linting and testing across platforms.
  • Removed legacy npm publishing workflows for prerelease (publish-prerelease-npm.yml) and stable releases (publish-stable-npm.yml), which relied on NPM_TOKEN and older Node.js versions. [1] [2]

Documentation:

  • Updated the CHANGELOG.md to reflect the switch to DK-CICD and GitHub Trusted Publishing for npm releases.The GitHub Actions NPM_TOKEN secret is expired, so tag publishes fail. Switch to the same npm-publish-v1 + Trusted Publishing flow used by node-vtex-api.

The GitHub Actions NPM_TOKEN secret is expired, so tag publishes fail.
Switch to the same npm-publish-v1 + Trusted Publishing flow used by node-vtex-api.

Co-authored-by: Cursor <cursoragent@cursor.com>
@sonar-workflows

Copy link
Copy Markdown

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.

1 participant