chore: resolution for ws library

tommasini commented 3 months ago

Description

This PR resolves ws library of version 8.17.1 Currently we have multiple dependencies using ws library, and some of them are using version 7, making this upgrade a possible breaking change More details of ws version 8 breaking changes here

├─┬ @ethersproject/providers@5.7.2
│ └── ws@7.4.6
├─┬ @metamask/swaps-controller@9.0.0
│ └─┬ web3@4.9.0
│   └─┬ web3-providers-ws@4.0.7
│     ├─┬ isomorphic-ws@5.0.0
│     │ └── ws@8.17.0
│     └── ws@8.17.0
├─┬ @walletconnect/client@1.8.0
│ └─┬ @walletconnect/core@1.8.0
│   └─┬ @walletconnect/socket-transport@1.8.0
│     └── ws@7.5.3
├─┬ @walletconnect/core@2.13.0
│ └─┬ @walletconnect/jsonrpc-ws-connection@1.0.14
│   └── ws@7.5.9 deduped
├─┬ @wdio/browserstack-service@7.32.4
│ └─┬ webdriverio@7.32.4
│   ├─┬ devtools@7.32.0
│   │ └─┬ puppeteer-core@13.1.3
│   │   └── ws@8.2.3
│   └─┬ puppeteer-core@13.7.0
│     └── ws@8.5.0
├─┬ appium@1.22.3
│ ├─┬ appium-android-driver@4.54.0
│ │ └── ws@8.17.0
│ ├─┬ appium-base-driver@7.11.3
│ │ ├─┬ webdriverio@6.12.1
│ │ │ ├─┬ @types/puppeteer-core@5.4.0
│ │ │ │ └─┬ @types/puppeteer@7.0.4
│ │ │ │   └─┬ puppeteer@21.3.1
│ │ │ │     └─┬ puppeteer-core@21.3.1
│ │ │ │       └── ws@8.14.1
│ │ │ └─┬ puppeteer-core@5.5.0
│ │ │   └── ws@7.5.9
│ │ └── ws@8.17.0
│ ├─┬ appium-flutter-driver@0.0.36
│ │ └─┬ rpc-websockets@7.5.0
│ │   └── ws@8.17.0
│ ├─┬ appium-ios-driver@4.8.3
│ │ └─┬ appium-remote-debugger@5.7.0
│ │   └─┬ appium-base-driver@4.5.1
│ │     └── ws@7.5.9 deduped
│ ├─┬ appium-tizen-driver@1.1.1-beta.5
│ │ └─┬ appium-base-driver@4.5.1
│ │   └── ws@7.5.9 deduped
│ └─┬ appium-xcuitest-driver@3.62.0
│   └── ws@8.17.0
├─┬ detox@20.20.3
│ └── ws@7.5.9
├─┬ metro-config@0.73.10
│ └─┬ metro@0.73.10
│   ├─┬ metro-inspector-proxy@0.73.10
│   │ └── ws@7.5.9 deduped
│   └── ws@7.5.9 deduped
├─┬ react-native@0.71.15 invalid: "^0.56.0" from node_modules/react-native-jazzicon
│ ├─┬ @react-native-community/cli@10.2.6
│ │ └─┬ @react-native-community/cli-server-api@10.1.1
│ │   └── ws@7.5.9 deduped
│ ├─┬ react-devtools-core@4.28.4
│ │ └── ws@7.5.9 deduped
│ └── ws@6.2.2
└─┬ socket.io-client@4.6.1
  └─┬ engine.io-client@6.4.0
    └── ws@8.11.0

Related issues

Fixes:

Manual testing steps

Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

[ ] I’ve followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
[ ] I've completed the PR template to the best of my ability
[ ] I’ve included tests if applicable
[ ] I’ve documented my code using JSDoc format if applicable
[ ] I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

[ ] I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
[ ] I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.