Skip to content

fix: update deps to build under pnpm link - #869

Draft
antigremlin wants to merge 2 commits into
mainfrom
andrey/pnpm-link-fix
Draft

fix: update deps to build under pnpm link#869
antigremlin wants to merge 2 commits into
mainfrom
andrey/pnpm-link-fix

Conversation

@antigremlin

Copy link
Copy Markdown
Contributor

These are draft fixes to the build system to allow building under pnpm link. We will try to avoid pnpm link in general but it will take more work and most of the fixes are correct even without it (except that the starknet dependency is questionable).

@antigremlin antigremlin added the DO NOT MERGE Just don't label Dec 18, 2025
@vercel

vercel Bot commented Dec 18, 2025

Copy link
Copy Markdown

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

Project Deployment Review Updated (UTC)
hyperlane-warp-template Ready Ready Preview, Comment Dec 18, 2025 2:51pm
4 Skipped Deployments
Project Deployment Review Updated (UTC)
injective-bridge Ignored Ignored Dec 18, 2025 2:51pm
nexus-bridge Ignored Ignored Dec 18, 2025 2:51pm
ousdt-bridge Ignored Ignored Dec 18, 2025 2:51pm
trump-bridge Ignored Ignored Dec 18, 2025 2:51pm

Comment thread package.json
"react": "^18.3.1",
"react-dom": "^18.3.1",
"react-toastify": "^10.0.6",
"starknet": "7.6.4",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Starknet fails to resolve as a peer dependency under pnpm link because of a major version conflict.

Comment thread package.json
"starknet": "7.6.4",
"starknetkit": "2.6.1",
"viem": "^2.21.41",
"viem": "^2.21.45",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is to align versions with the monorepo

Comment thread package.json
"viem": "^2.21.45",
"wagmi": "^2.12.26",
"zod": "3.21.4",
"zod": "3.25.76",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is to align versions with the monorepo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DO NOT MERGE Just don't

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant