fix(npm-publish): Resolve publishing pipeline conflicts and CI issues#4
Merged
Conversation
lemyskaman
commented
Jun 17, 2026
lemyskaman
commented
Member
- Remove OIDC from release.yml, add NODE_AUTH_TOKEN for NPM_TOKEN auth
- Delete conflicting release-please.yml and .release-please-manifest.json
- Move test-matrix to PR-only, add HUSKY=0 to npm ci steps
- Remove Node 20 from matrix (unsupported by engine>=22)
- Add AI tool directories to .gitignore"
- Remove OIDC from release.yml, add NODE_AUTH_TOKEN for NPM_TOKEN auth - Delete conflicting release-please.yml and .release-please-manifest.json - Move test-matrix to PR-only, add HUSKY=0 to npm ci steps - Remove Node 20 from matrix (unsupported by engine>=22) - Add AI tool directories to .gitignore
lemyskaman
force-pushed
the
feat/publish-to-npm
branch
from
ac78644 to
990088f
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
This PR focuses on stabilizing the GitHub Actions publishing pipeline and CI by removing conflicting release automation and aligning Node versions with the package’s Node >=22 engine requirement.
Changes:
- Switch release workflow away from OIDC and toward token-based npm auth; simplify Node setup and disable Husky during CI installs.
- Remove Release Please workflow/config to avoid pipeline conflicts.
- Limit the Node test matrix to PRs only and drop Node 20 from the matrix.
Reviewed changes
Copilot reviewed 5 out of 7 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/release.yml |
Updates release pipeline (Node version, auth method, Husky behavior). |
.github/workflows/ci.yml |
Adjusts CI behavior (PR-only test matrix, remove Node 20, disable Husky on install). |
.github/workflows/release-please.yml |
Removes conflicting Release Please workflow. |
.release-please-manifest.json |
Removes Release Please manifest config. |
.gitignore |
Ignores local AI/editor tooling directories and working notes. |
CHANGELOG.md |
Reformats changelog list items (but currently includes duplicate entries/typos). |
node_modules/.vite/vitest/results.json |
Removes a generated test artifact from the repo. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.