fix(cjs): make transformers overwritable

fix #590
privatenumber · Jun 15, 2024 · c22fa7d · c22fa7d
1 parent 6e117c2
commit c22fa7d
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 0 deletions.
diff --git a/src/cjs/api/module-extensions.ts b/src/cjs/api/module-extensions.ts
@@ -120,6 +120,7 @@ export const createExtensions = (
 			 * If not, only expose if it's not namespaced
 			 */
 			enumerable: descriptor?.enumerable || !namespace,
+			writable: true,
 		});
 	}
 

diff --git a/tests/specs/api.ts b/tests/specs/api.ts
@@ -79,6 +79,21 @@ export default testSuite(({ describe }, node: NodeApis) => {
 				expect(stdout).toContain('index.ts:3:22');
 			});
 
+			test('loader overwritable from Module', async () => {
+				await using fixture = await createFixture({
+					'index.mjs': `
+					import Module from 'node:module';
+					const _require = Module.createRequire(import.meta.url);
+					_require.extensions['.ts'] = () => {};
+					`,
+				});
+
+				await execaNode(fixture.getPath('index.mjs'), {
+					nodePath: node.path,
+					nodeOptions: ['--require', tsxCjsPath],
+				});
+			});
+
 			test('register / unregister', async () => {
 				await using fixture = await createFixture({
 					'register.cjs': `