Reconsider use of json-joy dependency about memfs HOT 7 CLOSED

I'd be happy to accept PRs moving us off json-joy

from memfs.

I did make a branch for this but realized that the problem here (and with json-joy) is that they're doing too much. It seems like this package should be split up into a core memfs just for node and separate packages for the experimental FSA adapters (since only those use the extra dependencies).

I realize that's a big ask so I understand if that's out of scope of the project.

from memfs.

@thatsmydoing I don't disagree but that's also a much bigger piece of work and not strictly my call alone to make.

In the meantime though do you think it might be possible to move off json-joy?

from memfs.

alternatively, we could try to make json-joy an optional peer dependency 🤔

from memfs.

@thatsmydoing I'm just released v4.8.0-next.1 which makes both dependencies optional

derp nope ok we need this for core

from memfs.

Oh, sorry just got back to this. Thanks for doing this. Which part of core memfs used json-joy?

from memfs.

src/print/index.ts iirc

from memfs.