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 install flags); no application or native wallet logic is modified in the diff.

Overview
This PR switches the repo from Yarn to npm by checking in npm’s install metadata and local install policy.

A new package-lock.json (lockfile v3) pins the tree for react-native-zano@0.2.8, including the existing runtime dep cleaners and the current ESLint/TypeScript/Husky dev toolchain.

A new .npmrc sets legacy-peer-deps=true (looser peer resolution during install) and ignore-scripts=true (skips lifecycle scripts on npm install).

Reviewed by Cursor Bugbot for commit f4ec5ec. 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