Skip to content

fix(deps): drop three dependencies the package never imports - #532

Open
emmanuel-musau wants to merge 1 commit into
IntersectMBO:mainfrom
emmanuel-musau:fix/drop-unused-runtime-deps
Open

emmanuel-musau wants to merge 1 commit into
IntersectMBO:mainfrom
emmanuel-musau:fix/drop-unused-runtime-deps

Conversation

@emmanuel-musau

Copy link
Copy Markdown
Collaborator

@effect/platform-node is declared in packages/evolution but imported by none of its 222 source files, and because it pins @effect/cluster, @effect/rpc and @effect/sql as peer dependencies, every consumer auto-installed that whole subtree plus @effect/experimental, @effect/workflow, @effect/platform-node-shared and @parcel/watcher with its thirteen prebuilt native binaries; bip39 is used by one test only and moves to devDependencies (@scure/bip39 is the runtime implementation), and @types/bip39 is a deprecated stub whose own npm metadata says bip39 ships its own types. Measured on a fresh npm install of the packed tarball, this takes the published package from 55 resolved dependencies and 89 MB to 23 and 72 MB, with no source or public API change; build, type-check, lint and all 1151 tests pass, and @evolution-sdk/devnet is unaffected since it declares @effect/platform-node in its own peer dependencies. Relates to #449.

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.

1 participant