Skip to content

upgrade to ethers v6#86

Merged
libotony merged 6 commits intomasterfrom
fix/upgrade-to-ethers-v6
Mar 23, 2026
Merged

upgrade to ethers v6#86
libotony merged 6 commits intomasterfrom
fix/upgrade-to-ethers-v6

Conversation

@libotony
Copy link
Copy Markdown
Member

No description provided.

Replace @vechain/ethers@4.0.27-5 (critical GHSA-vjh7-7g9h-fjfh via elliptic@6.5.4)
with ethers@^6.13.0 across all four dependent files:

- src/abi.ts: replace Coder subclass with AbiCoder + processDecoded post-processor
  (handles address→lowercase, BigInt→string, recursive tuple/array unwrapping)
- src/mnemonic.ts: Mnemonic.fromEntropy / Mnemonic.isValidMnemonic
- src/hdnode.ts: HDNodeWallet.fromPhrase / fromExtendedKey / deriveChild + encodeBase58
- src/keystore.ts: encryptKeystoreJson / decryptKeystoreJson / computeAddress
- tests/abi.test.ts: add 3 encodeParameters/decodeParameters coverage tests;
  relax one error-message assertion that was tied to v4 typo
- package.json: typescript ^3→^5, @types/node ^10→^18 (TS5 Uint8Array compat)
- Replace tslint with eslint@9 + @typescript-eslint: removes deprecated
  tslint and migrates rules (quotes, semi, max-len for src only)
- Upgrade mocha 5 → 11, nyc 14 → 18 to eliminate critical/high vulnerabilities
- Add npm overrides for serialize-javascript@^7.0.4 (mocha transitive dep)
- Upgrade @types/node ^10 → ^18, @types/mocha ^5 → ^10
- Auto-fix quote style across src/ and tests/ to match eslint config
- Result: 23 vulnerabilities (4 critical, 11 high) → 3 low
@coveralls
Copy link
Copy Markdown

coveralls commented Mar 20, 2026

Coverage Status

coverage: 97.283%. first build
when pulling 689ee70 on fix/upgrade-to-ethers-v6
into 150e07e on master.

@libotony libotony marked this pull request as ready for review March 23, 2026 08:24
@libotony libotony merged commit 6961b75 into master Mar 23, 2026
8 checks passed
@libotony libotony deleted the fix/upgrade-to-ethers-v6 branch March 23, 2026 08:25
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