Skip to content

Conversation

@dependabot
Copy link

@dependabot dependabot bot commented on behalf of github Nov 9, 2022

Bumps @metamask/controllers from 26.0.0 to 33.0.0.

Release notes

Sourced from @​metamask/controllers's releases.

33.0.0

Added

  • Add clear function for NotificationController (#943)

Changed

  • BREAKING: Rename "Collectible" to "NFT" (#934)
    • Rename CollectiblesController to NftController, and rename all methods, controller properties, types, constants, and state properties to use the term "NFT" instead of "collectible".
    • Rename CollectibleDetectionController to NftDetectionController, and rename all methods, controller properties, types, constants, and state properties to use the term "NFT" instead of "collectible".
    • Rename AssetsContractController method getERC721CollectibleTokenId to getERC721NftTokenId.
    • Rename assetsUtil method compareCollectiblesMetadata to compareNftMetadata.
    • Rename constant ASSET_TYPES.COLLECTIBLE to ASSET_TYPES.NFT
    • Rename PreferencesController state property useCollectibleDetection to useNftDetection
    • Rename PreferenceController method setUseCollectibleDetection to setUseNftDetection

Fixed

  • Fix NetworkController lookupNetwork deadlock (#948)
  • Fix invalid rpcTarget and nickname network state after switching from custom network to Infura network using setProviderType (#947)

32.0.2

Fixed

  • Fix issue introduced in v32.0.1 where EthQuery is not instantiated correctly on GasFeeController when constructed (#936)

32.0.1

Fixed

  • Make both callback and messaging system options for TokenListController & GasFeeController when listening to NetworkController (#932)
  • Add error handling when eth_estimateGas fails in TransactionController (#920)

32.0.0

Added

  • Add isOutOfDate to the PhishingController, for checking whether the phishing lists are out-of-date. (#929)

Changed

  • BREAKING: Make the PhishingController test synchronous (#929)
    • Rather than being polled regularly (as it was pre-v31), or updated on fetch (as in v31), the phishing lists are no longer updated automatically at all. You will need to manually call isOutOfDate to check whether they are out-of-date, and then call updatePhishingLists to update them.
  • BREAKING: Migrate NetworkController to BaseControllerV2 (#903)
    • There is no more "controller configuration" with BaseControllerV2. Instead configuration is set by constructor parameters.
    • The constructor accepts a single "options" object rather than separate parameters. The initial state is now passed as part of this options object, via the state property.
    • The constructor must be passed the infuraProjectId and messenger options. The messenger should be a restricted controller messenger.
    • Controller subscriptions must be registered through the controller messenger, rather than through the controller directly.
    • A providerChange event has been added. This is dispatched when the provider instance variable is updated.
      • When there is a network switch, this is dispatched after the network has been fully initialized.
    • A new getEthQuery action has been added, for obtaining an EthQuery instance that is already setup with the current provider
      • Note that it will stop working when the provider changes, so you should call this again each time the providerChange event is dispatched.

Fixed

  • Prevent parallel phishing configuration updates (#930)
  • Fix issues with empty responses in ERC20Standard (#927)
    • Calling getTokenSymbol and getTokenDecimals on contracts that return empty values (0x) for symbol() and decimals() will now throw an error.

31.2.0

... (truncated)

Changelog

Sourced from @​metamask/controllers's changelog.

[33.0.0]

Added

  • Add clear function for NotificationController (#943)

Changed

  • BREAKING: Rename "Collectible" to "NFT" (#934)
    • Rename CollectiblesController to NftController, and rename all methods, controller properties, types, constants, and state properties to use the term "NFT" instead of "collectible".
    • Rename CollectibleDetectionController to NftDetectionController, and rename all methods, controller properties, types, constants, and state properties to use the term "NFT" instead of "collectible".
    • Rename AssetsContractController method getERC721CollectibleTokenId to getERC721NftTokenId.
    • Rename assetsUtil method compareCollectiblesMetadata to compareNftMetadata.
    • Rename constant ASSET_TYPES.COLLECTIBLE to ASSET_TYPES.NFT
    • Rename PreferencesController state property useCollectibleDetection to useNftDetection
    • Rename PreferenceController method setUseCollectibleDetection to setUseNftDetection

Fixed

  • Fix NetworkController lookupNetwork deadlock (#948)
  • Fix invalid rpcTarget and nickname network state after switching from custom network to Infura network using setProviderType (#947)

[32.0.2]

Fixed

  • Fix issue introduced in v32.0.1 where EthQuery is not instantiated correctly on GasFeeController when constructed (#936)

[32.0.1]

Fixed

  • Make both callback and messaging system options for TokenListController & GasFeeController when listening to NetworkController (#932)
  • Add error handling when eth_estimateGas fails in TransactionController (#920)

[32.0.0]

Added

  • Add isOutOfDate to the PhishingController, for checking whether the phishing lists are out-of-date. (#929)

Changed

  • BREAKING: Make the PhishingController test synchronous (#929)
    • Rather than being polled regularly (as it was pre-v31), or updated on fetch (as in v31), the phishing lists are no longer updated automatically at all. You will need to manually call isOutOfDate to check whether they are out-of-date, and then call updatePhishingLists to update them.
  • BREAKING: Migrate NetworkController to BaseControllerV2 (#903)
    • There is no more "controller configuration" with BaseControllerV2. Instead configuration is set by constructor parameters.
    • The constructor accepts a single "options" object rather than separate parameters. The initial state is now passed as part of this options object, via the state property.
    • The constructor must be passed the infuraProjectId and messenger options. The messenger should be a restricted controller messenger.
    • Controller subscriptions must be registered through the controller messenger, rather than through the controller directly.
    • A providerChange event has been added. This is dispatched when the provider instance variable is updated.
      • When there is a network switch, this is dispatched after the network has been fully initialized.
    • A new getEthQuery action has been added, for obtaining an EthQuery instance that is already setup with the current provider
      • Note that it will stop working when the provider changes, so you should call this again each time the providerChange event is dispatched.

Fixed

  • Prevent parallel phishing configuration updates (#930)
  • Fix issues with empty responses in ERC20Standard (#927)
    • Calling getTokenSymbol and getTokenDecimals on contracts that return empty values (0x) for symbol() and decimals() will now throw an error.

[31.2.0]

... (truncated)

Commits
Maintainer changes

This version was pushed to npm by metamaskbot, a new releaser for @​metamask/controllers since your current version.


Dependabot compatibility score

You can trigger a rebase of this PR by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will 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 version will 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 dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Note
Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

@dependabot dependabot bot requested review from a team and meppsilon as code owners November 9, 2022 06:09
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Nov 9, 2022
@socket-security
Copy link

socket-security bot commented Nov 9, 2022

🚨 Potential security issues detected. Learn more about Socket for GitHub ↗︎

To accept the risk, merge this PR and you will not be notified again.

Alert Package NoteSource
New author npm/async@2.6.4
New author npm/istanbul-lib-report@3.0.1
New author npm/istanbul-lib-source-maps@4.0.1
New author npm/anymatch@3.1.3
New author npm/fb-watchman@2.0.2
Network access npm/https-proxy-agent@5.0.1
Network access npm/https-proxy-agent@5.0.1
New author npm/asn1@0.2.6
Network access npm/http-proxy-agent@5.0.0
Network access npm/http-proxy-agent@5.0.0
New author npm/for-each@0.3.3
New author npm/react-is@18.2.0
New author npm/tr46@3.0.0
Network access npm/jsdom@20.0.3
Network access npm/jsdom@20.0.3
Network access npm/jsdom@20.0.3
New author npm/querystringify@2.2.0
New author npm/fast-diff@1.3.0
New author npm/http-signature@1.3.6
New author npm/jsonify@0.0.1
  • New Author: ljharb
  • Previous Author:
New author npm/npm-normalize-package-bin@3.0.1
New author npm/write-file-atomic@5.0.1
New author npm/which@3.0.1
New author npm/exponential-backoff@3.1.1
New author npm/rlp@2.2.7
Network access npm/socks@2.7.1
Network access npm/agentkeepalive@4.5.0
Network access npm/agentkeepalive@4.5.0
New author npm/@npmcli/fs@2.1.2
Deprecated npm/@npmcli/move-file@2.0.1
  • Reason: This functionality has been moved to @npmcli/fs
New author npm/are-we-there-yet@3.0.1
Network access npm/make-fetch-happen@10.2.1
Network access npm/make-fetch-happen@10.2.1
Network access npm/make-fetch-happen@10.2.1
Network access npm/make-fetch-happen@10.2.1
Network access npm/make-fetch-happen@10.2.1
Network access npm/make-fetch-happen@10.2.1
Network access npm/make-fetch-happen@10.2.1
Network access npm/minipass-fetch@2.1.2
Network access npm/minipass-fetch@2.1.2
Network access npm/minipass-fetch@2.1.2
New author npm/nopt@6.0.0
New author npm/npmlog@6.0.2
Network access npm/socks-proxy-agent@7.0.0
Network access npm/socks-proxy-agent@7.0.0
New author npm/@npmcli/node-gyp@3.0.0
New author npm/read-cmd-shim@4.0.0
New author npm/@metamask/obs-store@7.0.0
New author npm/ethereumjs-wallet@1.0.2
New author npm/@keystonehq/bc-ur-registry@0.5.5
New author npm/create-jest@29.7.0
New author npm/unique-slug@3.0.0
New author npm/assert@2.1.0
Network access npm/@cypress/request@3.0.1
Network access npm/@cypress/request@3.0.1
Network access npm/@cypress/request@3.0.1
New author npm/cmd-shim@6.0.2
Network access npm/web3-providers-http@4.1.0

@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/main/metamask/controllers-33.0.0 branch from a10bfaa to 4f3678c Compare November 18, 2022 16:37
@dependabot dependabot bot requested a review from dan437 as a code owner November 18, 2022 16:37
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/main/metamask/controllers-33.0.0 branch 2 times, most recently from 4b30668 to 155fa49 Compare November 21, 2022 13:53
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/main/metamask/controllers-33.0.0 branch 2 times, most recently from 544cb97 to c216dc3 Compare December 1, 2022 20:45
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/main/metamask/controllers-33.0.0 branch 2 times, most recently from 8e6375d to ee62fa5 Compare December 12, 2022 09:13
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/main/metamask/controllers-33.0.0 branch 2 times, most recently from bc0bf3b to f6b328c Compare December 21, 2022 14:41
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/main/metamask/controllers-33.0.0 branch from f6b328c to 1edb8f3 Compare January 9, 2023 14:22
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/main/metamask/controllers-33.0.0 branch from 1edb8f3 to 73a4d28 Compare January 24, 2023 12:44
@legobeat
Copy link
Contributor

@dependabot rebase

@dependabot @github
Copy link
Author

dependabot bot commented on behalf of github Jul 14, 2023

Looks like this PR is already up-to-date with main! If you'd still like to recreate it from scratch, overwriting any edits, you can request @dependabot recreate.

@legobeat
Copy link
Contributor

@dependabot recreate

@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/main/metamask/controllers-33.0.0 branch from 73a4d28 to 4b2e541 Compare July 14, 2023 22:31
@legobeat
Copy link
Contributor

legobeat commented Jul 14, 2023

@dan437 @meppsilon @sethkfman PTAL. This needs a bump of min nodejs version.

Updating @metamask/controllers andsiblings will be necessary to resolve some security issues.

This repo is currently set to private such that I don't seem to be able to clone or create pull requests right now.

@socket-security
Copy link

socket-security bot commented Jul 14, 2023

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@babel/code-frame@7.23.5 Transitive: environment +10 191 kB nicolo-ribaudo
npm/@babel/core@7.23.9 environment, filesystem, unsafe Transitive: shell +52 10.8 MB nicolo-ribaudo
npm/@babel/generator@7.23.6 Transitive: environment +10 3.38 MB nicolo-ribaudo
npm/@babel/parser@7.23.9 None 0 1.88 MB nicolo-ribaudo
npm/@babel/template@7.23.9 Transitive: environment +15 4.59 MB nicolo-ribaudo
npm/@babel/types@7.23.9 environment +3 2.49 MB nicolo-ribaudo
npm/@eslint-community/regexpp@4.10.0 None 0 431 kB eslint-community-bot
npm/@ethereumjs/common@2.6.5 Transitive: environment, filesystem +43 11.6 MB holgerd77
npm/@ethereumjs/tx@3.5.2 Transitive: environment, filesystem +44 12 MB holgerd77
npm/@jridgewell/trace-mapping@0.3.22 None +2 265 kB jridgewell
npm/@lavamoat/allow-scripts@3.0.1 environment, filesystem, shell Transitive: network +121 5.58 MB naugtur
npm/@metamask/auto-changelog@2.6.1 Transitive: environment, filesystem, shell +33 1.19 MB gudahtt
npm/@metamask/controllers@33.0.0 network Transitive: environment, eval, filesystem, shell +326 70.8 MB metamaskbot
npm/@metamask/eslint-config-jest@12.1.0 Transitive: environment, eval, filesystem, network, shell, unsafe +461 91.1 MB metamaskbot
npm/@metamask/eslint-config-nodejs@12.1.0 Transitive: environment, eval, filesystem, network, shell, unsafe +199 31.7 MB metamaskbot
npm/@metamask/eslint-config-typescript@12.1.0 Transitive: environment, eval, filesystem, network, shell, unsafe +223 108 MB metamaskbot
npm/@metamask/eslint-config@12.2.0 Transitive: environment, eval, filesystem, network, shell, unsafe +191 30.1 MB metamaskbot
npm/@metamask/ethjs-query@0.5.2 Transitive: environment, eval, filesystem +13 4.26 MB lgbot
npm/@nodelib/fs.stat@2.0.5 filesystem 0 11.8 kB mrmlnc
npm/@nodelib/fs.walk@1.2.8 Transitive: filesystem +6 127 kB mrmlnc
npm/@types/babel__traverse@7.20.5 Transitive: environment +4 2.58 MB types
npm/@types/istanbul-lib-coverage@2.0.6 None 0 5.45 kB types
npm/@types/node@20.11.16 None +1 4.06 MB types
npm/@typescript-eslint/eslint-plugin@5.62.0 Transitive: environment, eval, filesystem, shell, unsafe +127 48.3 MB jameshenry
npm/@typescript-eslint/parser@5.62.0 Transitive: environment, eval, filesystem, shell, unsafe +119 45.1 MB jameshenry
npm/@typescript-eslint/utils@5.62.0 Transitive: environment, eval, filesystem, shell, unsafe +123 45.7 MB jameshenry
npm/acorn-walk@8.3.2 None 0 52.4 kB marijn
npm/acorn@8.11.3 None 0 531 kB marijn
npm/ansi-regex@5.0.1 None 0 5.61 kB qix
npm/array-buffer-byte-length@1.0.1 Transitive: eval +12 245 kB ljharb
npm/async-mutex@0.3.2 None +1 134 kB dirtyhairy
npm/async@2.6.4 None 0 541 kB hargasinski
npm/available-typed-arrays@1.0.6 None 0 20.4 kB ljharb
npm/bignumber.js@9.1.2 None 0 351 kB mikemcl
npm/bn.js@5.2.1 None 0 99 kB fanatid
npm/call-bind@1.0.6 Transitive: eval +10 214 kB ljharb
npm/crc-32@1.2.2 None 0 31 kB sheetjs
npm/define-data-property@1.1.2 Transitive: eval +8 173 kB ljharb
npm/define-properties@1.2.1 Transitive: eval +10 212 kB ljharb
npm/es-abstract@1.22.3 Transitive: eval +57 3.26 MB ljharb
npm/es-errors@1.3.0 None 0 12.3 kB ljharb
npm/eslint-config-prettier@8.10.0 Transitive: environment, eval, filesystem, shell, unsafe +96 10.7 MB lydell
npm/eslint-plugin-import@2.26.0 filesystem, unsafe Transitive: environment, eval, network, shell +174 15.9 MB ljharb
npm/eslint-plugin-jest@27.6.3 filesystem Transitive: environment, eval, network, shell, unsafe +372 72 MB simenb
npm/eslint-plugin-jsdoc@43.2.0 Transitive: environment, eval, filesystem, shell, unsafe +105 13.3 MB gajus
npm/eslint-plugin-n@15.7.0 filesystem Transitive: environment, eval, shell, unsafe +110 12.6 MB weiran.zsd
npm/eslint-plugin-prettier@4.2.1 Transitive: environment, eval, filesystem, shell, unsafe +99 22.1 MB jounqin
npm/eslint-plugin-promise@6.1.1 Transitive: environment, eval, filesystem, shell, unsafe +96 10.8 MB eslint-community-bot
npm/eslint-visitor-keys@3.4.3 None 0 32.3 kB eslintbot
npm/eslint@8.56.0 environment, filesystem Transitive: eval, shell, unsafe +95 10.7 MB eslintbot
npm/espree@9.6.1 None +3 662 kB eslintbot
npm/esquery@1.5.0 None +1 1.04 MB michaelficarra
npm/estraverse@5.3.0 None 0 37.1 kB michaelficarra
npm/ethereumjs-util@7.1.5 Transitive: environment, filesystem +41 8.99 MB holgerd77
npm/get-intrinsic@1.2.4 eval +5 124 kB ljharb
npm/glob@7.2.3 filesystem Transitive: environment +10 145 kB isaacs
npm/graceful-fs@4.2.11 environment, filesystem 0 32.5 kB isaacs
npm/has-bigints@1.0.2 None 0 12.8 kB ljharb
npm/has-property-descriptors@1.0.1 Transitive: eval +6 135 kB ljharb
npm/has-symbols@1.0.3 None 0 20.6 kB ljharb
npm/has-tostringtag@1.0.2 None +1 38.2 kB ljharb
npm/https-proxy-agent@5.0.1 network Transitive: environment +2 75.2 kB tootallnate
npm/iconv-lite@0.6.3 None +1 391 kB ashtuchkin
npm/ignore@5.3.1 None 0 51.5 kB kael
npm/is-array-buffer@3.0.4 Transitive: eval +11 231 kB ljharb
npm/is-callable@1.2.7 None 0 28.9 kB ljharb
npm/is-core-module@2.13.1 None +2 72.5 kB ljharb
npm/is-glob@4.0.3 None +1 19.8 kB phated
npm/is-string@1.0.7 None +2 57.3 kB ljharb
npm/is-symbol@1.0.4 None +1 42.6 kB ljharb
npm/is-typed-array@1.1.13 Transitive: eval +16 358 kB ljharb
npm/istanbul-lib-coverage@3.2.2 None 0 34.4 kB oss-bot
npm/jest-environment-jsdom@29.7.0 Transitive: environment, eval, filesystem, network, shell, unsafe +100 14.5 MB simenb
npm/jest-util@29.7.0 environment Transitive: filesystem +18 4.88 MB simenb
npm/jest@29.7.0 Transitive: environment, eval, filesystem, network, shell, unsafe +287 56.7 MB simenb
npm/mime-types@2.1.35 None +1 224 kB dougwilson
npm/minimist@1.2.8 None 0 54.5 kB ljharb
npm/minipass@3.3.6 None 0 48.1 kB isaacs
npm/node-fetch@2.7.0 network +6 891 kB node-fetch-bot
npm/npm-normalize-package-bin@3.0.1 None 0 3.58 kB npm-cli-ops
npm/p-limit@3.1.0 None +1 13.8 kB sindresorhus
npm/parse5@7.1.2 None +1 1.11 MB feedic
npm/picomatch@2.3.1 None 0 90 kB mrmlnc
npm/prettier@2.8.8 environment, filesystem, unsafe 0 11.2 MB prettier-bot
npm/punycode@2.3.1 None 0 33.5 kB google-wombot
npm/resolve@1.22.8 environment, filesystem +5 232 kB ljharb
npm/rlp@2.2.7 None +1 162 kB ralxz
npm/secp256k1@4.0.3 Transitive: environment, filesystem +9 2.9 MB fanatid
npm/semver@7.6.0 None +1 110 kB npm-cli-ops
npm/signal-exit@3.0.7 None 0 9.96 kB isaacs
npm/string-width@4.2.3 None +4 68 kB sindresorhus
npm/strip-ansi@6.0.1 None +1 9.64 kB sindresorhus
npm/tar@6.2.0 environment, filesystem +7 354 kB isaacs
npm/tough-cookie@4.1.3 None +6 690 kB awaterma
npm/ts-jest@29.1.2 environment, filesystem, unsafe Transitive: eval, network, shell +290 57.1 MB kul
npm/ts-node@10.9.2 environment, filesystem, unsafe +19 38.1 MB blakeembrey
npm/tslib@2.6.2 None 0 84 kB typescript-bot
npm/typescript@5.3.3 None 0 32 MB typescript-bot
npm/web3-core@4.3.2 Transitive: environment, eval, filesystem, network +53 13.4 MB luu-alex
npm/web3-errors@1.1.4 None +1 694 kB jdevcs
npm/web3-eth-accounts@4.1.1 Transitive: environment, eval +34 7.45 MB luu-alex
npm/web3-eth-contract@4.2.0 Transitive: environment, eval, filesystem, network +61 48.1 MB luu-alex
npm/web3-eth@4.4.0 Transitive: environment, eval, filesystem, network +60 47.7 MB luu-alex
npm/web3-rpc-methods@1.1.4 Transitive: environment, eval, filesystem, network +54 13.5 MB luu-alex
npm/web3-types@1.3.1 None 0 290 kB jdevcs
npm/web3-utils@4.1.1 Transitive: environment, eval +31 6.01 MB luu-alex
npm/web3-validator@2.0.4 Transitive: environment, eval +30 5.48 MB luu-alex
npm/web3@4.4.0 Transitive: environment, eval, filesystem, network +65 52.2 MB luu-alex
npm/which-typed-array@1.1.14 Transitive: eval +15 335 kB ljharb
npm/yargs-parser@21.1.1 environment, filesystem 0 128 kB oss-bot

🚮 Removed packages: npm/@babel/code-frame@7.12.11, npm/@babel/compat-data@7.23.2, npm/@babel/core@7.23.2, npm/@babel/helper-compilation-targets@7.22.15, npm/@babel/highlight@7.22.20, npm/@babel/parser@7.23.0, npm/@babel/runtime@7.23.2, npm/@babel/template@7.22.15, npm/@babel/traverse@7.23.2, npm/@babel/types@7.23.0, npm/@ethereumjs/common@2.4.0, npm/@ethereumjs/tx@3.3.0, npm/@ethersproject/abi@5.4.0, npm/@ethersproject/abstract-provider@5.4.0, npm/@ethersproject/abstract-signer@5.4.0, npm/@ethersproject/address@5.4.0, npm/@ethersproject/base64@5.4.0, npm/@ethersproject/basex@5.4.0, npm/@ethersproject/bignumber@5.4.0, npm/@ethersproject/bytes@5.4.0, npm/@ethersproject/constants@5.4.0, npm/@ethersproject/hash@5.4.0, npm/@ethersproject/hdnode@5.4.0, npm/@ethersproject/json-wallets@5.4.0, npm/@ethersproject/keccak256@5.4.0, npm/@ethersproject/logger@5.4.0, npm/@ethersproject/networks@5.4.1, npm/@ethersproject/pbkdf2@5.4.0, npm/@ethersproject/properties@5.4.0, npm/@ethersproject/random@5.4.0, npm/@ethersproject/rlp@5.4.0, npm/@ethersproject/sha2@5.4.0, npm/@ethersproject/signing-key@5.4.0, npm/@ethersproject/strings@5.4.0, npm/@ethersproject/transactions@5.4.0, npm/@ethersproject/web@5.4.0, npm/@ethersproject/wordlists@5.4.0, npm/@jridgewell/trace-mapping@0.3.19, npm/@lavamoat/allow-scripts@1.0.6, npm/@metamask/auto-changelog@2.5.0, npm/@metamask/controllers@26.0.0, npm/@metamask/eslint-config-jest@8.0.0, npm/@metamask/eslint-config-nodejs@8.0.0, npm/@metamask/eslint-config-typescript@8.0.0, npm/@metamask/eslint-config@8.0.0, npm/@nodelib/fs.stat@2.0.4, npm/@types/babel__core@7.1.14, npm/@types/babel__traverse@7.11.1, npm/@types/fetch-mock@7.3.3, npm/@types/istanbul-lib-coverage@2.0.3, npm/@types/node@16.6.2, npm/@typescript-eslint/eslint-plugin@4.22.0, npm/@typescript-eslint/experimental-utils@4.22.0, npm/@typescript-eslint/parser@4.22.0, npm/abab@2.0.5, npm/acorn@7.4.1, npm/async-mutex@0.3.1, npm/async@2.6.3, npm/bignumber.js@9.0.1, npm/bip39@2.6.0, npm/browserslist@4.22.1, npm/buffer-from@1.1.1, npm/call-bind@1.0.2, npm/deep-is@0.1.3, npm/es-abstract@1.18.0, npm/eslint-config-prettier@8.2.0, npm/eslint-plugin-import@2.22.1, npm/eslint-plugin-jest@24.3.5, npm/eslint-plugin-prettier@3.4.0, npm/eslint@7.32.0, npm/espree@7.3.1, npm/ethereumjs-util@7.1.0, npm/ethjs-query@0.3.8, npm/extend-shallow@3.0.2, npm/fetch-mock@9.11.0, npm/get-intrinsic@1.1.1, npm/get-value@2.0.6, npm/glob@7.1.6, npm/globals@13.11.0, npm/graceful-fs@4.2.8, npm/has-symbols@1.0.2, npm/is-callable@1.2.3, npm/is-descriptor@1.0.2, npm/is-extendable@0.1.1, npm/is-glob@4.0.1, npm/is-plain-object@2.0.4, npm/is-symbol@1.0.3, npm/isobject@3.0.1, npm/jest-util@26.6.2, npm/jest@26.6.3, npm/kind-of@6.0.3, npm/mime-types@2.1.30, npm/minimist@1.2.7, npm/node-fetch@2.6.7, npm/normalize-package-data@2.5.0, npm/picomatch@2.2.3, npm/prettier@2.2.1, npm/psl@1.8.0, npm/punycode@2.1.1, npm/regex-not@1.0.2, npm/regexpp@3.1.0, npm/request@2.88.2, npm/resolve@1.20.0, npm/rlp@2.2.6, npm/semver@7.3.5, npm/set-value@2.0.1, npm/signal-exit@3.0.3, npm/source-map-support@0.5.19, npm/split-string@3.1.0, npm/string-width@4.2.2, npm/strip-ansi@6.0.0, npm/to-regex@3.0.2, npm/ts-jest@26.5.5, npm/ts-node@9.1.1, npm/tslib@2.2.0, npm/typescript@4.2.4, npm/walker@1.0.7, npm/word-wrap@1.2.5, npm/yargs-parser@20.2.9

View full report↗︎

@legobeat
Copy link
Contributor

@nikoferro

@legobeat
Copy link
Contributor

@dan437 Minimum Node.js version has been increased to 16 (#173), which should allow bumping here.

@legobeat
Copy link
Contributor

@dependabot recreate

Bumps [@metamask/controllers](https://github.com/MetaMask/controllers) from 26.0.0 to 33.0.0.
- [Release notes](https://github.com/MetaMask/controllers/releases)
- [Changelog](https://github.com/MetaMask/controllers/blob/main/CHANGELOG.md)
- [Commits](MetaMask/core@v26.0.0...v33.0.0)

---
updated-dependencies:
- dependency-name: "@metamask/controllers"
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/main/metamask/controllers-33.0.0 branch from 4b2e541 to e5768ae Compare November 22, 2023 11:55
davibroc
davibroc previously approved these changes Dec 6, 2023
Copy link

@davibroc davibroc left a comment

Choose a reason for hiding this comment

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

I am testing it now with mobile

@legobeat
Copy link
Contributor

legobeat commented Dec 6, 2023

I am testing it now with mobile

The types do not pass build, though?
https://github.com/MetaMask/swaps-controller/actions/runs/7062382351/job/19226072611

$ tsc --project .
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(8,81): error TS1005: ',' expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(8,90): error TS1109: Expression expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(8,96): error TS1005: ')' expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(8,114): error TS1005: ';' expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(8,130): error TS1005: ';' expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(8,131): error TS1128: Declaration or statement expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(8,137): error TS1109: Expression expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(15,5): error TS1128: Declaration or statement expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(15,29): error TS1005: ',' expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(15,68): error TS1109: Expression expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(15,80): error TS1005: ';' expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(22,5): error TS1128: Declaration or statement expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(22,35): error TS1005: ',' expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(22,59): error TS1109: Expression expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(22,71): error TS1005: ';' expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(32,5): error TS1128: Declaration or statement expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(32,34): error TS1005: ',' expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(32,60): error TS1011: An element access expression should take an argument.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(32,73): error TS1109: Expression expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(32,85): error TS1005: ';' expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(33,1): error TS1128: Declaration or statement expected.
Error: node_modules/web3-types/lib/commonjs/eth_contract_types.d.ts(109,5): error TS1131: Property or signature expected.
Error: node_modules/web3-types/lib/commonjs/eth_contract_types.d.ts(109,9): error TS1005: ';' expected.
Error: node_modules/web3-types/lib/commonjs/eth_contract_types.d.ts(109,24): error TS1005: ';' expected.
Error: node_modules/web3-types/lib/commonjs/eth_contract_types.d.ts(110,9): error TS1005: ';' expected.
Error: node_modules/web3-types/lib/commonjs/eth_contract_types.d.ts(110,28): error TS1005: ',' expected.
Error: node_modules/web3-types/lib/commonjs/eth_contract_types.d.ts(124,13): error TS1109: Expression expected.
Error: node_modules/web3-types/lib/commonjs/eth_contract_types.d.ts(128,26): error TS1109: Expression expected.
Error: node_modules/web3-types/lib/commonjs/eth_contract_types.d.ts(132,18): error TS1109: Expression expected.
Error: node_modules/web3-types/lib/commonjs/eth_contract_types.d.ts(133,1): error TS1128: Declaration or statement expected.
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Error: Process completed with exit code 2.

@davibroc
Copy link

davibroc commented Dec 7, 2023

I am testing it now with mobile

The types do not pass build, though? https://github.com/MetaMask/swaps-controller/actions/runs/7062382351/job/19226072611

$ tsc --project .
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(8,81): error TS1005: ',' expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(8,90): error TS1109: Expression expected.

I am able to build the PR locally with this command yarn && yarn build && yarn pack --filename <file-path>

From the mobile repo I then add the package with yarn add <file-path> but when I run yarn setup I get the following error Error: Patch file found for package bip39 which is not present at node_modules/bip39

Any idea why?

@legobeat
Copy link
Contributor

legobeat commented Dec 7, 2023

I am testing it now with mobile

The types do not pass build, though? https://github.com/MetaMask/swaps-controller/actions/runs/7062382351/job/19226072611

$ tsc --project .
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(8,81): error TS1005: ',' expected.
Error: node_modules/web3-eth-accounts/lib/commonjs/tx/transactionFactory.d.ts(8,90): error TS1109: Expression expected.

I am able to build the PR locally with this command yarn && yarn build && yarn pack --filename <file-path>

From the mobile repo I then add the package with yarn add <file-path> but when I run yarn setup I get the following error Error: Patch file found for package bip39 which is not present at node_modules/bip39

Any idea why?

Hm, the build fails locally for me with the same errors as in CI (typescript errors in web3-eth-acounts).
Have you tried wiping your node_modules? Can you post build output?

Re the bip39 thing: It's trying to patch a package which is no longer present (as eth-keyring-controller@^6.2.1 is no longer pulled in after this upgrade. you should be able to observe this from the yarn.lock diff). So in general it's either one of that patch needing to be ported, or no longer being necessary.

@klejeune
Copy link
Contributor

klejeune commented Dec 7, 2023

Typing issues have been fixed by upgrading the Typescript version. Jest has some issues running the tests though.

@nikoferro
Copy link

Currently working on this, had to update dependencies since we werent able to run the linter with the new rules that are extended from @metamask/eslint-config. Last commit fixes all the linting errors after the updates. Still need to fix jest

@nikoferro
Copy link

@dan437 @davibroc hey guys, this should be ready to test and merge if everything goes well, we got some socket security alerts, but i think those are safe to ignore. can you review this PR?

@nikoferro nikoferro requested a review from davibroc February 1, 2024 09:22
dan437
dan437 previously approved these changes Feb 1, 2024
Copy link
Contributor

@dan437 dan437 left a comment

Choose a reason for hiding this comment

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

Code-wise it looks good. Let's wait for a QA approval before merging.

…ndabot/npm_and_yarn/main/metamask/controllers-33.0.0
@nikoferro nikoferro merged commit 3e6db87 into main Feb 15, 2024
@nikoferro nikoferro deleted the dependabot/npm_and_yarn/main/metamask/controllers-33.0.0 branch February 15, 2024 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants