chore: remove 8 more unused dependencies#668
Open
rickstaa wants to merge 2 commits into
Open
Conversation
Drop 20 direct dependencies in package.json with zero imports anywhere in source. Verified via scripted sweep across all *.ts/tsx/js/mjs/cjs files plus manual confirmation that none are peer deps, build-time codegen, or implicit-via-config (MDX, Apollo Server endpoint, etc.). Removed by group: GraphQL Tools — only @graphql-tools/wrap is actually imported (in apollo/createSchema.ts): - @graphql-tools/delegate, @graphql-tools/links, @graphql-tools/schema, @graphql-tools/stitch, graphql-tools (umbrella) MDX — no .mdx files in repo, no MDX wiring in next.config.js: - @mdx-js/loader, @mdx-js/react, @next/mdx Apollo Server stack — no GraphQL endpoint code references them; AS3 is EOL: - apollo-server-micro, apollo-link, apollo-link-http, micro-cors UI/utility leftovers from earlier refactors: - @reach/dialog, @wagmi/core (transitive via wagmi), gsap, lightweight-charts, match-sorter, parse-domain, react-confetti, react-number-format Lockfile shrinks by 2,203 lines, reducing supply-chain attack surface and CVE alert noise. Closes #662 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Follow-up to #663. Drop direct deps with zero source imports: - dotenv-cli (kept dotenv — used by `pnpm codegen`) - typechain, @typechain/ethers-v5 — no script, no generated output - baseline-browser-mapping — no references - ts-node — Jest 30 loads jest.config.ts natively - @types/change-case, @types/react-icons, @types/recharts — packages ship their own types Lockfile shrinks by 284 lines. Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR continues the dependency cleanup work (follow-up to #663) by removing additional direct dependencies from package.json that have no remaining references in source code or build/test configs, reducing maintenance surface and transitive security noise.
Changes:
- Remove 8 unused direct dependencies from
dependencies/devDependencies(including legacy Apollo Server/link packages, TypeChain tooling,dotenv-cli, and obsolete@types/*packages). - Regenerate
pnpm-lock.yamlto reflect the updated direct dependency set and pruned transitive graph.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| package.json | Drops unused direct deps from runtime and dev/tooling sections. |
| pnpm-lock.yaml | Updates lockfile to match dependency removals and prunes now-unreachable transitive packages. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Follow-up to #663. Drops 8 more direct dependencies with zero references in source or build configs. Cumulative cleanup is now 32 of 104 (~31%).
What's removed and why
Runtime
dotenv-cli— distinct fromdotenv(kept;pnpm codegenuses-r dotenv/config).Type generation (no longer used)
typechain,@typechain/ethers-v5— notypechainscript, no config, no generated output directory.Build / tooling leftovers
baseline-browser-mapping— zero references; not wired intonext.config.jsor any browserslist config.ts-node— Jest 30 loadsjest.config.tsnatively (verified by runningpnpm testafter removal).Obsolete
@types/*(packages now ship their own declarations)@types/change-case—change-casev4+ ships types@types/react-icons—react-iconsships types@types/recharts—rechartsv2+ ships types (the DT package only covered v1)What's intentionally not removed
@stitches/react— declared transitively by@livepeer/design-system; safe to drop but verifying separately as a follow-up.@emotion/react,@emotion/styled— MUI v7 peers.Verification
pnpm typecheckpassespnpm lintpassespnpm format:checkpassespnpm test— 84/84 tests pass;jest.config.tsloads withoutts-node(one suite fails on missing env vars, unrelated)Depends on
Stacked on #663 (
chore/remove-orphan-deps). Targetmainonce #663 lands; rebase if needed.