Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Dependency updates #1965

Open
19 of 23 tasks
Tracked by #121 ...
SgtPooki opened this issue Jul 6, 2022 · 2 comments · Fixed by ipfs/kubo#10241
Open
19 of 23 tasks
Tracked by #121 ...

chore: Dependency updates #1965

SgtPooki opened this issue Jul 6, 2022 · 2 comments · Fixed by ipfs/kubo#10241
Assignees
Labels
effort/months Epic exp/expert Having worked on the specific codebase is important need/analysis Needs further analysis before proceeding P0 Critical: Tackled by core team ASAP topic/dependencies Topic dependencies

Comments

@SgtPooki
Copy link
Member

SgtPooki commented Jul 6, 2022

This issue is intended to track & discuss any and all issues related to old dependencies:

Issues tracking dependency work

Core dependency updates needed (priority)

Other work

Nice to have but not required

Items blocked by outdated deps in one way or another

Issues blocking dependency updates


Investigating dependencies

npm outdated --all > npm-outdated.raw.log gives us this: npm-outdated.raw.log

Then we want to remove duplicates, because npm outdated doesn't do that for us for some reason (echo "$(head -n 1 npm-outdated.raw.log && tail -n +2 npm-outdated.raw.log | sort | uniq -d)" > npm-outdated.uniq.log): npm-outdated.uniq.log
cat npm-outdated.raw.log | awk '{print $6}' | sort | uniq -c | sort > most-impactful-dependencies.log

To see which dependencies are mentioned the most in the depended by column, we can run cat npm-outdated.uniq.log | awk '{print $6}' | sort | uniq -c | sort > most-impactful-dependencies.log to get: most-impactful-dependencies.log

and then to get the top 10 most impactful dependency updates, tail -n 10 most-impactful-dependencies.log:

  8 ipfs-http-client
  9 @babel/core
  9 @storybook/theming
  9 css-loader
 10 jest-cli
 11 ipfs-core-utils
 11 ipfs-utils
 11 react-dev-utils
 15 @storybook/components
 51 @babel/preset-env

So which of those versions do we 'want'?

> tail -n 10 most-impactful-dependencies.log | awk '{print $2}' | xargs -n1 -I% sh -c 'cat npm-outdated.uniq.log | grep "^%"'
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/helper-compilation-targets
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/helper-create-class-features-plugin
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/plugin-proposal-class-properties
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/plugin-proposal-dynamic-import
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/plugin-proposal-export-namespace-from
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/plugin-proposal-logical-assignment-operators
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/plugin-proposal-nullish-coalescing-operator
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/plugin-proposal-numeric-separator
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/plugin-proposal-optional-chaining
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/plugin-proposal-private-methods
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/plugin-syntax-flow
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/plugin-syntax-typescript
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/plugin-transform-flow-strip-types
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/plugin-transform-modules-commonjs
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/plugin-transform-typescript
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @babel/preset-typescript
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @jest/transform
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              @svgr/plugin-jsx
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              istanbul-lib-instrument
@babel/core                                                            7.15.5               7.18.6               7.18.6  node_modules/@babel/core                                                                                                              jest-snapshot
@storybook/theming                                                     5.3.21               5.3.21                6.5.9  node_modules/@storybook/theming                                                                                                       @storybook/components
ipfs-core-utils                                                        0.10.5               0.10.5               0.15.1  node_modules/ipfs-core-utils                                                                                                          ipfs-http-client
@babel/preset-env                                                     7.12.11               7.18.6               7.18.6  node_modules/@babel/preset-env                                                                                                        @svgr/webpack

And why are they there? tail -n 10 most-impactful-dependencies.log | awk '{print $2}' | xargs -n1 -I% sh -c 'npm explain %' > explain.txt: explain.txt

That's super hard to understand, so let's aggregate again... cat explain.txt | grep 'from the root project' | awk '{$1=$1};1' | sort | uniq -c | sort:

   1 dev @babel/core@"^7.15.5" from the root project
   1 dev @storybook/addon-links@"^5.3.19" from the root project
   1 dev @storybook/addons@"^5.3.19" from the root project
   1 ipld-explorer-components@"^2.4.1" from the root project
   2 dev @svgr/cli@"^5.4.0" from the root project
   4 ipfs-http-client@"49.0.2" from the root project
   5 dev @storybook/addon-a11y@"^5.3.19" from the root project
   6 dev @storybook/addon-actions@"^5.3.19" from the root project
   6 dev @storybook/addon-knobs@"^5.3.19" from the root project
  29 dev @playwright/test@"^1.12.1" from the root project
 106 dev ipfs@"0.58.3" from the root project
 117 dev ipfsd-ctl@"^7.2.0" from the root project
 190 dev jest@"^26.6.3" from the root project
 213 dev @storybook/react@"^5.3.19" from the root project
 301 dev jest-playwright-preset@"^1.7.0" from the root project
 754 react-scripts@"^4.0.3" from the root project

Breakdown of the above:

  1. List all outdated npm modules
  2. find the outdated modules that are mentioned the most
  3. find and count the npm modules that depend on the most-mentioned out-of-date modules.
    • These are the "most impactful dependencies"
  4. get npm to tell us why these most impactful dependencies are in our repo.
  5. filter through step4 above to tell us which module we actually depend on is at fault for all the above
  6. find the most-mentioned module from step5 that we depend on
    • The module with the most entries should be the first target for our upgrade plan.

So what does this mean?

Using react-scripts as an example: there are 754 listings of react-scripts being the ROOT CAUSE. react-scripts is the package we depend on, that is listed in the npm-explain output, when trying to determine why out-of-date modules are listed in our dependency graph.

Why did you do all this?

I attempted to upgrade some (see main...SgtPooki:ipfs-webui:fix/node-14-windows) to resolve the build errors showing at https://github.com/SgtPooki/ipfs-webui/actions/runs/2605014389, (see #1961) and was consistently blocked by npm because of semver conflicts of wanted packages. In order to move forward.. since there is no npm-dependency-graph-resolver upgrade tool, I figured I would need all of this data.

@SgtPooki SgtPooki added the need/triage Needs initial labeling and prioritization label Jul 6, 2022
@SgtPooki SgtPooki self-assigned this Jul 7, 2022
@SgtPooki SgtPooki added exp/expert Having worked on the specific codebase is important P0 Critical: Tackled by core team ASAP topic/dependencies Topic dependencies need/analysis Needs further analysis before proceeding effort/days Estimated to take multiple days, but less than a week and removed need/triage Needs initial labeling and prioritization labels Jul 20, 2022
@SgtPooki SgtPooki pinned this issue Jul 21, 2022
@SgtPooki
Copy link
Member Author

@lidel Can we chat about what this story looks like by tying ourselves specifically to Kubo? e.g. Pulling out ipfs-http-client entirely from webui, and replacing with https://github.com/ipfs/js-kubo-rpc-client.

@SgtPooki
Copy link
Member Author

SgtPooki commented Oct 8, 2022

on the latest main:

> cat npm-install.log | sort | uniq | grep 'npm WARN deprecated' | wc -l
      52
> npm i &> npm-install.log

> cat npm-install.log | perl -pe 's/npm WARN deprecated //g' | perl -pe 's/(^[^:]+)@[^:]+/$1/g' | sort | uniq | perl -ne 'print if /ipfs|cid|multi|ipld|libp2p/'
cids: This module has been superseded by the multiformats module
interface-ipld-format: This module has been superseded by the multiformats module
ipfs-block-service: This module has been merged into ipfs
ipld-dag-cbor: This module has been superseded by @ipld/dag-cbor and multiformats
ipld-dag-pb: This module has been superseded by @ipld/dag-pb and multiformats
ipld-ethereum: This module is no longer maintained
ipld-git: This module is no longer maintained
ipld-raw: This module has been superseded by the multiformats module
ipld: Please use the multiformats module instead
multibase: This module has been superseded by the multiformats module
multicodec: This module has been superseded by the multiformats module
multihashing-async: This module has been superseded by the multiformats module

All of these packages listed need to be removed/updated prior to us being able to safely&sanely keep up with new kubo feature development.

cat npm-install.log | perl -pe 's/npm WARN deprecated //g' | perl -pe 's/(^[^:]+)@[^:]+/$1/g' | sort | uniq | perl -ne 'print if /ipfs|cid|multi|ipld|libp2p/' | awk -F: '{ print $1 }' | xargs -tn1 npm explain &> npm-explain.log

npm-explain.log

Callouts

Of those 52 deprecated packages, 12 are core packages.
Of those 12 core packages, 7 require replacing fully with the multiformats package.
Of those 12 core packages, 1 was merged into ipfs.
Of those 12 core packages, 2 are no longer maintained
Of those 12 core packages, 2 require replacing with a mixture of @ipld/pkg + multiformats.

@SgtPooki SgtPooki added Epic effort/months and removed effort/days Estimated to take multiple days, but less than a week labels Oct 10, 2022
@SgtPooki SgtPooki assigned whizzzkid and unassigned SgtPooki Nov 22, 2022
@tinytb tinytb added this to the Dependencies update milestone Nov 22, 2022
SgtPooki added a commit that referenced this issue Dec 1, 2023
* feat: use @multiformats/multiaddr
* deps: update multiaddr-to-uri
* deps: update ipld-explorer-components

This also updates ipld-explorer-components to 5.2.0 which will allow us to remove a lot of old deps (part of #1965)
ipfs-gui-bot pushed a commit that referenced this issue Dec 2, 2023
## [4.2.0](v4.1.1...v4.2.0) (2023-12-02)

 CID `bafybeidf7cpkwsjkq6xs3r6fbbxghbugilx3jtezbza7gua3k5wjixpmba`

 ---

### Features

* peers table can be filtered ([#2181](#2181)) ([37b5880](37b5880))
* **pinning:** add Scaleway pinning service ([#2132](#2132)) ([0cc3b04](0cc3b04))

### Bug Fixes

* error loading scaleway template URL ([7a3388e](7a3388e))
* replace estuary with web3 for pinning ([#2182](#2182)) ([e2fc7c8](e2fc7c8))
* status page supports webrtc ([#2180](#2180)) ([ff75d4e](ff75d4e)), closes [#1965](#1965)
* switch to new dnslink updater ([#2175](#2175)) ([a63f48a](a63f48a))

### Trivial Changes

* Pull transifex translations ([#2162](#2162)) ([c0f8f54](c0f8f54))
* pull transifex translations ([#2177](#2177)) ([7f377ff](7f377ff))
* update release transifex links ([74cce69](74cce69))
SgtPooki added a commit to ipfs/kubo that referenced this issue Dec 2, 2023
## [4.2.0](ipfs/ipfs-webui@v4.1.1...v4.2.0) (2023-12-02)


 CID `bafybeidf7cpkwsjkq6xs3r6fbbxghbugilx3jtezbza7gua3k5wjixpmba`

 --- 



### Features

* peers table can be filtered ([#2181](ipfs/ipfs-webui#2181)) ([37b5880](ipfs/ipfs-webui@37b5880))
* **pinning:** add Scaleway pinning service ([#2132](ipfs/ipfs-webui#2132)) ([0cc3b04](ipfs/ipfs-webui@0cc3b04))


### Bug Fixes

* error loading scaleway template URL ([7a3388e](ipfs/ipfs-webui@7a3388e))
* replace estuary with web3 for pinning ([#2182](ipfs/ipfs-webui#2182)) ([e2fc7c8](ipfs/ipfs-webui@e2fc7c8))
* status page supports webrtc ([#2180](ipfs/ipfs-webui#2180)) ([ff75d4e](ipfs/ipfs-webui@ff75d4e)), closes [#1965](ipfs/ipfs-webui#1965)
* switch to new dnslink updater ([#2175](ipfs/ipfs-webui#2175)) ([a63f48a](ipfs/ipfs-webui@a63f48a))


### Trivial Changes

* Pull transifex translations ([#2162](ipfs/ipfs-webui#2162)) ([c0f8f54](ipfs/ipfs-webui@c0f8f54))
* pull transifex translations ([#2177](ipfs/ipfs-webui#2177)) ([7f377ff](ipfs/ipfs-webui@7f377ff))
* update release transifex links ([74cce69](ipfs/ipfs-webui@74cce69))
@SgtPooki SgtPooki reopened this Dec 6, 2023
@SgtPooki SgtPooki self-assigned this Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
effort/months Epic exp/expert Having worked on the specific codebase is important need/analysis Needs further analysis before proceeding P0 Critical: Tackled by core team ASAP topic/dependencies Topic dependencies
Projects
No open projects
Status: 🥞 Todo
Development

Successfully merging a pull request may close this issue.

3 participants