Using memfs to replace webpack/memory-fs

[dai-shi]

Hi,
I'm trying to use memfs as well as unionfs and linkfs with webpack replacing memory-fs.

So far, I had two issues, which are actually not directly related to memfs though.

• unionfs doesn't pass mkdirp, although memfs implements it.
  webpack requires mkdirp and my workaround is to patch it:

ufs1.mkdirp = (...args) => ufs1.asyncMethod('mkdirp', args);

• when using with linkfs (node's fs behind it), unionfs fails with unsupported mkdirp, instead of calling back an exception which is an expected behavior.
  it seems like a bug in https://github.com/streamich/unionfs/blob/5580150f2faf88276bff205b84ae70684823c30e/src/union.ts#L77

if (func) func.apply(fs, args);

or something like this helps my case.

I could try preparing a PR, but I'd leave it for now as you might have a better design choice.

Thanks for your great work!

[streamich]

Hi,

Thank you for finding that bug.

Regarding mkdirp, it is not a standard method and the idea behind memfs is to be a drop-in replacement for fs module. I am not keen on adding non-standard methods, in general.

However, mkdirp is implemented in memfs (and memory-fs as well) and apparently many people use that, so I included it in the list as an exception, unionfs should now pass it through:

streamich/fs-monkey@980c0d0

I've released a new version that should fix your issues:

npm i unionfs@2.0.7

Please tell me if it works.

[dai-shi]

@streamich Thanks for the instant fix. It's working. 😄