ourzora / zora-protocol

Monorepo for Zora Protocol (contracts & sdks)
MIT License
113 stars 70 forks source link

Can't install `@zoralabs/protocol-sdk` with yarn 4.x #413

Closed JoshNavi closed 1 month ago

JoshNavi commented 4 months ago

Reproductions:

With yarn v4

~~> yarn add @zoralabs/protocol-sdk
➤ YN0000: · Yarn 4.2.2
➤ YN0000: ┌ Resolution step
➤ YN0001: │ TypeError: @zoralabs/mints-deployments@npm:*: Cannot convert undefined or null to object
    at Function.entries (<anonymous>)
    at b1t (/Users/josh/.cache/node/corepack/yarn/4.2.2/yarn.js:677:20583)
    at /Users/josh/.cache/node/corepack/yarn/4.2.2/yarn.js:677:19098
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async I1 (/Users/josh/.cache/node/corepack/yarn/4.2.2/yarn.js:147:33381)
    at async wb (/Users/josh/.cache/node/corepack/yarn/4.2.2/yarn.js:147:31578)
    at async Object.N4 (/Users/josh/.cache/node/corepack/yarn/4.2.2/yarn.js:147:33665)
    at async Em (/Users/josh/.cache/node/corepack/yarn/4.2.2/yarn.js:677:21070)
    at async /Users/josh/.cache/node/corepack/yarn/4.2.2/yarn.js:677:18746
    at async S1t (/Users/josh/.cache/node/corepack/yarn/4.2.2/yarn.js:677:18700)
➤ YN0000: └ Completed in 3s 127ms
➤ YN0000: · Failed with errors in 3s 144ms

With npm v10: was able to install

~~> npm i --save @zoralabs/protocol-sdk

added 12 packages, and audited 13 packages in 3s

8 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities
iainnash commented 4 months ago

Looking

iainnash commented 4 months ago

We have a fix that's coming out today.

oveddan commented 4 months ago

should be fixed with @zoralabs/protocol-sdk version 0.5.12

could you please try installing the latest version?

iainnash commented 4 months ago

Fix is in:

> yarn
➤ YN0088: A new stable version of Yarn is available: 4.2.2!
➤ YN0088: Upgrade now by running yarn set version 4.2.2

➤ YN0087: Migrated your project to the latest Yarn version 🚀

➤ YN0000: · Yarn 4.2.1
➤ YN0000: ┌ Resolution step
➤ YN0085: │ + @zoralabs/protocol-sdk@npm:0.5.13, @zoralabs/protocol-deployments@npm:0.1.10
➤ YN0000: └ Completed
➤ YN0000: ┌ Post-resolution validation
➤ YN0002: │ demo-protocol-sdk@workspace:. doesn't provide viem (p2a449), requested by @zoralabs/protocol-sdk.
➤ YN0086: │ Some peer dependencies are incorrectly met; run yarn explain peer-requirements <hash> for details, where <hash> is the six-letter p-prefixed code.
➤ YN0000: └ Completed
➤ YN0000: ┌ Fetch step
➤ YN0013: │ 2 packages were added to the project (+ 7.58 MiB).
➤ YN0000: └ Completed in 0s 336ms
➤ YN0000: ┌ Link step
➤ YN0000: └ Completed
➤ YN0000: · Done with warnings in 0s 403ms
> yarn --version
4.2.1
JoshNavi commented 4 months ago

Confirming it works! Thank you!

kyhyco commented 3 months ago

Hello folks!

Running into issue with this again. Here is an example repo of not working - https://github.com/kyhyco/for-zoralabs

with yarn v4, yarn add @zorolabs/protocol-sdk is looking for @zoralabs/mints-deployments.

❯ yarn add @zoralabs/protocol-sdk
➤ YN0000: · Yarn 4.3.0
➤ YN0000: ┌ Resolution step
➤ YN0001: │ TypeError: @zoralabs/mints-deployments@npm:*: Cannot convert undefined or null to object
    at Function.entries (<anonymous>)
    at k1t (/Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:677:20583)
    at /Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:677:19098
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async B1 (/Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:147:33381)
    at async Ib (/Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:147:31578)
    at async Object.N4 (/Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:147:33665)
    at async ym (/Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:677:21070)
    at async /Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:677:18746
    at async x1t (/Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:677:18700)
➤ YN0000: └ Completed
➤ YN0000: · Failed with errors in 0s 196ms

Strangely enough installing using yarn v1 and npm works fine.

iainnash commented 3 months ago

We’re working on a fix for this. Unfortunately it’s a pretty large change and taking longer than we want.

On Mon, Jun 17, 2024 at 23:15 Ky Lee @.***> wrote:

Hello folks!

Running into issue with this again. Here is an example repo of not working

with yarn v4, yarn add @zorolabs/protocol-sdk is looking for @zoralabs/mints-deployments.

❯ yarn add @zoralabs/protocol-sdk ➤ YN0000: · Yarn 4.3.0 ➤ YN0000: ┌ Resolution step ➤ YN0001: │ TypeError: @@.**:: Cannot convert undefined or null to object at Function.entries () at k1t (/Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:677:20583) at /Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:677:19098 at process.processTicksAndRejections (node:internal/process/task_queues:95:5) at async B1 (/Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:147:33381) at async Ib (/Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:147:31578) at async Object.N4 (/Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:147:33665) at async ym (/Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:677:21070) at async /Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:677:18746 at async x1t (/Users/ky.lee/tmp/test-zora2/.yarn/releases/yarn-4.3.0.cjs:677:18700) ➤ YN0000: └ Completed ➤ YN0000: · Failed with errors in 0s 196ms

Strangely enough installing using yarn v1 and npm works fine.

— Reply to this email directly, view it on GitHub https://github.com/ourzora/zora-protocol/issues/413#issuecomment-2174885418, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGMCOGPQG4RFF7ZKVEELE3ZH6Q6NAVCNFSM6AAAAABHUYW7JSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZUHA4DKNBRHA . You are receiving this because you commented.Message ID: @.***>

iainnash commented 1 month ago

Hello this is now fixed :). Let us know if you run into any further issues.

kyhyco commented 1 month ago

confirmed that it works!