Make cache file be regular .js file about memoize-fs

[borisdiakur]

Yes, I’m ok with using fs-extra.

[tunnckoCore]

Or, I'm guessing that if we make a simple check if there is astBody: true to put function's ast there instead of silently ignore.

[tunnckoCore]

Some other ways are https://github.com/luin/serialize and https://www.npmjs.com/package/serializr

[tunnckoCore]

can confirm that with node-serialize everything works :) We should not worry about the "security implications" since we are in the fs and most probably there won't have (end-)user input ability given by the developer anyway.

serializr package wants some schema and stuff and is too much for our case.

[tunnckoCore]

Want PR? We also can allow passing serialize and deserialize as options.

- JSON.parse(resultString).data
+ return deserialize(resultString).data

and

const meriyah = require('meriyah')
+ const {serialize, unserialize: deserialize} = require('node-serialize')

[borisdiakur]

Hey @tunnckoCore, I like the idea of having the ability to use a custom serializer, or choosing one using an option. Maybe could list one as a peer dependency. PRs welcome 😉

[tunnckoCore]

Yea, i gone for the options method in my patch too, I'll PR.

Are you okay including fs-extra instead of fs in that PR too? It fixes other problems (including one if you use fs.mkdir recursive: true as I suggested) when you have high parallelism (i'm running memoize-fs inside jest-runner which runner runs hundreds of files).