Skip to content

Conversation

@slorber
Copy link

@slorber slorber commented Jan 22, 2026

Recently, our Docusaurus E2E CI started to fail on Yarn Berry with PnP linker: facebook/docusaurus#11681

Job logs: https://github.com/facebook/docusaurus/actions/runs/21246255725/job/61136186782?pr=11681

Error:  Error: @jsonjoy.com/fs-node tried to access @jsonjoy.com/fs-snapshot, but it isn't declared in its dependencies; this makes the require call ambiguous and unsound.

Required package: @jsonjoy.com/fs-snapshot
Required by: @jsonjoy.com/fs-node@virtual:fc57f29978701b0380cf34b4829d515bc5ec75e6b58ee2fb1bd72225778da5b959326c5bab0085ce264437f5e8a4f00a12b95b40fdc82131fc496c7331e426f3#npm:4.56.9 (via /home/runner/work/docusaurus/test-website/.yarn/__virtual__/@jsonjoy.com-fs-node-virtual-9537c3ce10/4/.yarn/berry/cache/@jsonjoy.com-fs-node-npm-4.56.9-49734c8568-10c0.zip/node_modules/@jsonjoy.com/fs-node/lib/)

Require stack:
- /home/runner/work/docusaurus/test-website/.yarn/__virtual__/@jsonjoy.com-fs-node-virtual-9537c3ce10/4/.yarn/berry/cache/@jsonjoy.com-fs-node-npm-4.56.9-49734c8568-10c0.zip/node_modules/@jsonjoy.com/fs-node/lib/volume.js

It looks to me that volume.ts depends on fs-snapshot, so you'd rather add the dependency in package.json:

import * as fsSnapshot from '@jsonjoy.com/fs-snapshot';

Copy link
Owner

@streamich streamich left a comment

Choose a reason for hiding this comment

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

Thank you!

@streamich
Copy link
Owner

Merged in e142b9b

@streamich streamich closed this Jan 23, 2026
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.

2 participants