Skip to content

Refactor npm release workflow#19

Merged
NodeByteLTD merged 1 commit into
mainfrom
develop
May 11, 2026
Merged

Refactor npm release workflow#19
NodeByteLTD merged 1 commit into
mainfrom
develop

Conversation

@NodeByteLTD
Copy link
Copy Markdown
Owner

@NodeByteLTD NodeByteLTD commented May 11, 2026

Removed npm update step and simplified publish command.

Summary

Describe what changed and why.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Chore / maintenance

Testing

List what you ran:

  • pnpm lint
  • pnpm build
  • Relevant tests for changed area

Commands/results:

Paste test output or brief notes here.

Screenshots / Evidence (if applicable)

Add screenshots, recordings, or logs.

Checklist

  • I linked related issue(s)
  • I updated docs where needed
  • I updated changelog for user-facing changes
  • I removed secrets from code, logs, and screenshots

Summary by CodeRabbit

  • Chores
    • Improved the release process by streamlining build and publish steps for better reliability and consistency during releases.

Review Change Stack

Removed npm update step and simplified publish command.
@NodeByteLTD NodeByteLTD merged commit 4fb4aaf into main May 11, 2026
12 of 13 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 11, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 582f5a39-0c5b-4760-b9f9-bc1060ef6e32

📥 Commits

Reviewing files that changed from the base of the PR and between f4a1c3d and 8f8ad47.

📒 Files selected for processing (1)
  • .github/workflows/npm-release.yml

Walkthrough

The npm release workflow was corrected for a display name typo, refactored to use explicit dependency installation, and simplified to always publish with provenance enabled without prior NODE_AUTH_TOKEN-based conditional branching.

Changes

npm Release Workflow

Layer / File(s) Summary
Workflow metadata and permissions
.github/workflows/npm-release.yml
Workflow display name corrected from "Relase Node SDK" to "Release Node SDK". OIDC permission id-token: write retained while removing inline explanatory comment.
Release publish flow
.github/workflows/npm-release.yml
Dependency installation added as explicit step with pnpm install --frozen-lockfile. Publish step simplified to always run npm publish --provenance after build, removing prior conditional NODE_AUTH_TOKEN handling.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch develop

Comment @coderabbitai help to get the list of available commands and usage tips.

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