Snapshotting CJS exports

[bmeck]

Dunno if this one has a sane solution without punching require('module').Module.prototype.load pretty hard...

https://github.com/nodejs/node-eps/blob/master/002-es-modules.md#491-es-will-not-honor-reassigning-moduleexports-after-evaluation

// entry.js

require('@std/esm');
require('./mutates');
global.incr();
require('./a')

// a.mjs
import assert from 'assert';
import v from './mutates';
assert.strictEqual(v, 1);

// mutates.js
global.incr = () => module.exports++;
module.exports = 1;

[jdalton]

Thanks @bmeck!

We'll need to add a test for this for sure.

[jdalton]

Ok checking back, we seem to snapshot appropriately when running within the hook. The example presented above is running in the bridge file which is before rules can be imposed. I'll be adding a round of unit tests over the next couple of days to solidify support.

[bmeck]

Different example not using the bridge file:

// a.mjs
import b from './b';
console.log('a', b); // prints 1
import('./c');

// b.js
module.exports = 1;
process.nextTick(()=>module.exports++);

// c.mjs
import b from './b';
console.log('c', b); // prints 2

[jdalton]

Thanks! That's super interesting!