Skip to content

Convert to npm#454

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

Convert to npm#454
peachbits wants to merge 2 commits into
masterfrom
convert-to-npm

Conversation

@peachbits

@peachbits peachbits commented May 26, 2026

Copy link
Copy Markdown
Contributor

Note

Low Risk
Tooling and documentation only; no runtime swap plugin logic changes. CI (e.g. Jenkins) may still need updating if it still invokes Yarn outside this diff.

Overview
This PR moves the repo from Yarn to npm as the documented and supported package manager.

Tooling: Adds .npmrc with legacy-peer-deps=true, removes .yarnrc, and stops ignoring package-lock.json in .gitignore so the npm lockfile is committed.

Docs and scripts: README, plugin docs, and convert.sh now use npm install, npm run prepare, npm run start, npm run mapctl, and related npm scripts instead of yarn / yarn updot.

Note: The README “Adding Your Exchange” block shows npm run npm install prepare, which looks like a mistaken merge of npm install and npm run prepare.

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

@peachbits peachbits force-pushed the convert-to-npm branch 2 times, most recently from af93a63 to 22e4535 Compare June 2, 2026 17:55
@peachbits peachbits changed the title Convert yarn → npm + Socket security upgrades + @edge.app fork aliases Convert to npm Jun 2, 2026

@j0ntz j0ntz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approving — no open review threads and no failing checks; the conversion looks clean here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale comment

Comment thread .npmrc
convert-to-npm added 2 commits June 10, 2026 23:29
Ran `socket fix --all --range-style preserve --minimum-release-age 7d`.
Pinned ethers to 5.7.0 (exact) afterward — socket fix wanted to bump
ethers v5 → v6 (a breaking API change) and we cannot adopt v6 yet.
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.

2 participants