Skip to content

Convert from yarn to npm#13

Open
peachbits wants to merge 2 commits into
mainfrom
convert-to-npm
Open

Convert from yarn to npm#13
peachbits wants to merge 2 commits into
mainfrom
convert-to-npm

Conversation

@peachbits
Copy link
Copy Markdown
Contributor

@peachbits peachbits commented May 25, 2026

Note

Low Risk
Tooling-only change (lockfile and npm config); no runtime or wallet logic is modified, though ignore-scripts skips postinstall scripts which can affect native/build steps if relied on.

Overview
This PR migrates the repo from Yarn to npm by adding a full package-lock.json (lockfile v3) for react-native-zano@0.2.8 and introducing .npmrc with legacy-peer-deps=true and ignore-scripts=true so installs resolve peer conflicts and skip lifecycle scripts during dependency setup.

There is no application or library source change in the diff—only install/lockfile tooling.

Reviewed by Cursor Bugbot for commit a383905. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit f4ec5ec. Configure here.

Comment thread .npmrc
@@ -0,0 +1,2 @@
legacy-peer-deps=true
ignore-scripts=true
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Install scripts skip lib build

High Severity

Setting ignore-scripts=true stops npm install / npm ci from running the root prepare script, which is what compiles TypeScript into lib/. That output is listed in .gitignore and is the package main entry, so a normal install leaves the library without a loadable build until someone runs prepare by hand.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit f4ec5ec. Configure here.

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