MetaMask / MetaMask/metamask-sdk
`@metamask/sdk-communication-layer` pulls in old `date-fns` v2
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 338
- Forks
- 260
- PR merge metrics
- No merged PRs in 30d
Description
# Feature Request
Upgrade `date-fns` from v2 to v4 in `@metamask/sdk-communication-layer` , see: https://github.com/MetaMask/metamask-sdk/blob/main/packages/sdk-communication-layer/package.json#L51
## Why it is needed
`date-fns` v2 is 25MB in size in `node_modules` and doesn't deduplicate with modern projects using e.g. `date-fns` v4
## Other
`@metamask/sdk-communication-layer` generally appears like its dependencies are not maintained well, so it should probably be modernized as a whole.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at packages/sdk-communication-layer/package.json around the date-fns entry near line 51, then inspect how that package uses date-fns for v4 compatibility. Update the dependency and verify the package checks; done means the communication layer uses date-fns v4 without pulling in the old v2 dependency.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- api
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 62/100