deps(deps-dev): bump bitcoinjs-lib from 6.1.7 to 7.0.1#736
deps(deps-dev): bump bitcoinjs-lib from 6.1.7 to 7.0.1#736dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [bitcoinjs-lib](https://github.com/bitcoinjs/bitcoinjs-lib) from 6.1.7 to 7.0.1. - [Changelog](https://github.com/bitcoinjs/bitcoinjs-lib/blob/master/CHANGELOG.md) - [Commits](bitcoinjs/bitcoinjs-lib@v6.1.7...v7.0.1) --- updated-dependencies: - dependency-name: bitcoinjs-lib dependency-version: 7.0.1 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6b6a9fd16e
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "bip32": "^4.0.0", | ||
| "bip39": "^3.1.0", | ||
| "bitcoinjs-lib": "6.1.7", | ||
| "bitcoinjs-lib": "7.0.1", |
There was a problem hiding this comment.
Avoid upgrading bitcoinjs-lib before bigint migration
Bumping bitcoinjs-lib to 7.0.1 introduces a breaking runtime API change (Transaction output values are now bigint), but this repo’s BTC test helpers still build transactions with numeric values (e.g. _start_tx_builder in packages/sdk/src/__test__/utils/helpers.ts calls tx.addOutput(..., value) and tx.addOutput(..., changeValue) where both are numbers). With this version bump, those code paths will throw during BTC signing/address tests instead of producing reference transactions, so CI/test workflows that exercise these helpers will regress until values are converted to bigint.
Useful? React with 👍 / 👎.
Bumps bitcoinjs-lib from 6.1.7 to 7.0.1.
Changelog
Sourced from bitcoinjs-lib's changelog.
Commits
dc7f0677.0.13619e35Bump valibote4280e2Merge pull request #2248 from landabaso/feat/restore-hash-helpersdc1dbc6Merge branch 'master' into feat/restore-hash-helpers99bb080Merge pull request #2297 from landabaso/feat/p2ms-20-keys48d1e53feat(crypto): restore ripemd160, sha1 and sha256 hash helpers16f1bacp2ms: support up to 20 keys13aea8cRemoved unnecessary check for tapInternalKey for signature validations of tap...c151d53Merge pull request #2264 from bitcoinjs/bump-ver18b1c227.0.0Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)