Skip to content

STA-31: remove incompatible walletconnect logger override and enforce Yarn 4 on Vercel - #54

Merged
gertsio merged 2 commits into
mainfrom
codex/STA-31-fix-walletconnect-warnings
Feb 23, 2026
Merged

STA-31: remove incompatible walletconnect logger override and enforce Yarn 4 on Vercel#54
gertsio merged 2 commits into
mainfrom
codex/STA-31-fix-walletconnect-warnings

Conversation

@gertsio

@gertsio gertsio commented Feb 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Remove forced @walletconnect/logger resolution override from package.json.
  • Regenerate yarn.lock so @walletconnect/logger@2.1.2 is resolved where requested.
  • Add vercel.json install command to enforce Corepack + Yarn 4 install path on Vercel.

Validation

  • yarn install (Yarn 4.12.0) no longer shows the incompatible resolution warning: Resolution field "@walletconnect/logger@3.0.0" is incompatible with requested version "@walletconnect/logger@2.1.2".
  • yarn check currently fails on pre-existing TypeScript errors in src/app/(app)/dashboard/setup/components/setup-wizard.tsx from main baseline (unrelated to this diff).
  • vercel build in this fresh worktree requires project settings pull first (vercel pull --yes), so preview validation is expected via PR deployment.

Linear

  • STA-31

Greptile Summary

Fixed incompatible WalletConnect logger resolution and enforced Yarn 4 runtime on Vercel. The PR removes the forced @walletconnect/logger@3.0.0 override that was causing version conflicts with dependencies requesting 2.1.2, regenerates the lockfile to properly resolve 2.1.2, and adds a vercel.json config to ensure Vercel uses Corepack + Yarn 4.12.0 during deployments.

Key Changes:

  • Removed @walletconnect/logger@3.0.0 from resolutions in package.json - this was overriding transitive dependencies that specifically require 2.1.2
  • Added vercel.json with custom install command using Corepack to activate Yarn 4.12.0 and --immutable flag for lockfile validation
  • Regenerated yarn.lock now includes the proper @walletconnect/logger@2.1.2 entry with its dependencies (@walletconnect/safe-json@^1.0.2, pino@7.11.0)

Validation Notes:
The PR description mentions that yarn check fails on pre-existing TypeScript errors in setup-wizard.tsx from the main branch, which is unrelated to this change. Preview validation is expected via Vercel PR deployment.

Confidence Score: 5/5

  • Safe to merge - fixes known version incompatibility and enforces consistent build environment
  • All three changes are low-risk infrastructure fixes: removing an incompatible version override, regenerating the lockfile to match actual dependency requirements, and standardizing the Vercel build environment to match local development (Yarn 4.12.0). No application logic changed.
  • No files require special attention

Important Files Changed

Filename Overview
package.json Removed incompatible @walletconnect/logger@3.0.0 resolution override that was forcing version 3.0.0 when dependencies require 2.1.2
vercel.json Added custom install command to enforce Corepack and Yarn 4.12.0 on Vercel deployments with immutable lockfile validation
yarn.lock Regenerated to include proper @walletconnect/logger@2.1.2 resolution, removing the incompatibility warning

Last reviewed commit: ff831f4

@vercel

vercel Bot commented Feb 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
startupchain Ready Ready Preview, Comment Feb 23, 2026 5:30am

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@gertsio
gertsio merged commit 148a307 into main Feb 23, 2026
5 checks passed
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