Skip to content

Drop unused Hardhat/npm stack to close Dependabot alerts - #57

Merged
vm06007 merged 1 commit into
mainfrom
chore/drop-unused-hardhat-npm-stack
Aug 21, 2026
Merged

Drop unused Hardhat/npm stack to close Dependabot alerts#57
vm06007 merged 1 commit into
mainfrom
chore/drop-unused-hardhat-npm-stack

Conversation

@vm06007

@vm06007 vm06007 commented Aug 21, 2026

Copy link
Copy Markdown
Member

Summary

  • This repo is Foundry-only (no hardhat.config, no JS tests). The leftover Hardhat 2 / ethers 5 / waffle / web3 1 tree in package-lock.json was the source of the open Dependabot alerts (bn.js, elliptic, tar, yargs-parser, qs, tmp, web3-core-subscriptions, and ~25 others).
  • Removed those unused npm dependencies and regenerated an empty lockfile. Foundry CI no longer runs npm install; foundry.toml now looks only at lib/ (git submodules).

Test plan

  • Confirm Foundry workflows pass (build sizes + gas report) without npm install
  • After merge, check Dependabot alerts auto-close (expect the 37 open npm alerts to go to 0)
  • forge build --sizes and forge test still work locally with submodules checked out

Made with Cursor

This repo is Foundry-only (no Hardhat config or JS tests), so the ethers/web3/waffle tree in package-lock.json can go. CI now skips npm install and Forge looks only at lib/.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vm06007
vm06007 merged commit 1a7ec46 into main Aug 21, 2026
0 of 2 checks passed
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