Fix npm install by reverting version bump and removing npm upgrade step#83
Merged
Fix npm install by reverting version bump and removing npm upgrade step#83
Conversation
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.
Summary
This PR addresses a broken npm install issue caused by the publish workflow and an incorrect version bump. Two targeted changes are made to restore proper build and publish behavior:
package.jsonversion from0.2.41back to0.2.40— the version was prematurely or incorrectly bumped, likely causing conflicts or failed installs for consumers of the package.npm upgradestep from the GitHub Actions publish workflow (.github/workflows/publish.yml) — this step was modifying dependency versions during CI, which can introduce non-deterministic builds and was likely the root cause of the install failures.Root Cause
The publish workflow included an
npm upgradestep that mutated dependency versions at publish time, potentially causing version mismatches, lockfile inconsistencies, or pulling in breaking transitive dependencies. Combined with the premature version bump to0.2.41, this resulted in a broken install experience for downstream consumers.Changes
package.json0.2.41→0.2.40.github/workflows/publish.ymlKey Improvements
npm installfor all consumers of the packageBreaking Changes
None. This is a corrective fix that restores the previously working state. The version is reverted to
0.2.40, so no new public version is introduced.Testing Notes for Reviewers
npm installcompletes successfully in a clean environment after these changes0.2.40installs correctly:npm install <package-name>@0.2.40package-lock.json(it should remain unchanged by this PR)Browser Compatibility Considerations
No impact — these changes are limited to CI/CD configuration and package metadata. No application code, runtime behavior, or front-end assets are modified.
🤖 Generated with Claude Code
Branch Info:
bugfix/fix-npm-installmainCo-Authored-By: Claude noreply@anthropic.com