Skip to content

chore: remove deprecated @crossmint/client-sdk-walletconnect package#1916

Draft
alfonso-paella wants to merge 1 commit into
mainfrom
devin/1781186601-deprecate-walletconnect
Draft

chore: remove deprecated @crossmint/client-sdk-walletconnect package#1916
alfonso-paella wants to merge 1 commit into
mainfrom
devin/1781186601-deprecate-walletconnect

Conversation

@alfonso-paella

Copy link
Copy Markdown
Contributor

Description

Removes the @crossmint/client-sdk-walletconnect package entirely from the SDK repo. The package has security vulnerabilities in its dependencies and the maintenance burden is not justified by usage.

Evidence for removal (gathered in this analysis):

  • ~570 npm downloads/month (spiky pattern suggesting CI/bots, vs 102k for common-sdk-base)
  • Zero production API hits in Datadog over 30 days
  • Zero external dependents on all of public GitHub
  • No internal consumers — crossbit-main's /wc and /api/walletConnect use @walletconnect/* protocol libs directly
  • Prior removal attempt in Sept 2025 by Pablo Egido & Julio (never merged)

Confirmed with: Alberto Elias, Maximiliano Schultheis, and Alfonso in #proj-walletconnect-deprecation.

Follow-up: Run npm deprecate @crossmint/client-sdk-walletconnect \"This package is deprecated and no longer maintained.\" to mark the existing published versions as deprecated on the npm registry.

Test plan

  • Verified pnpm install succeeds after removal (lockfile updated, 460 dependency lines removed)
  • Verified pnpm lint passes with no new errors
  • No other package in the repo imports or depends on this package

Package updates

No changeset added — this is a full package removal (not a version bump). The npm deprecation notice should be applied separately via CLI after merge.

Requested by: @alfonso-paella

The package has been confirmed unused:
- ~570 npm downloads/month (spiky, likely CI/bots)
- Zero production API hits in Datadog
- No internal consumers in SDK repo or crossbit-main
- No external dependents on public GitHub
- Prior removal attempt in Sept 2025 (never merged)

crossbit-main's /wc and /api/walletConnect use @walletconnect/* protocol
libs directly and are unaffected by this removal.

Confirmed with Alberto Elias, Maximiliano Schultheis, and Alfonso.

Co-Authored-By: Alfonso <alfonso@paella.dev>
@devin-ai-integration

Copy link
Copy Markdown
Contributor
Original prompt from Alfonso

im thinking on deprecating packages/client/walletconnect in the SDK

can you help me make this decision?

  1. check on datadog if it has usage
  2. check fi theres any public download stats for it, as well as whether we use it on our other repos
  3. check on the code the main maintainers, and give me the list so i can ask them

@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@changeset-bot

changeset-bot Bot commented Jun 11, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: e000112

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

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