fix: faster/more robust commonjs playground plugin (#1381)

Handle some edge cases around default exports, a especially weird code snippet of a specific very popular but abandoned library, and bail in more cases for perf

Author: dummdidumm

packages/repl/src/lib/workers/bundler/index.ts

@@ -76,6 +76,8 @@ self.addEventListener('message', async (event: MessageEvent<BundleMessageData>)
 						can_use_experimental_async
 					);
 
+					console.log('[bundle worker result]', result);
+
 					if (JSON.stringify(result.error) === JSON.stringify(ABORT)) return;
 					if (result && uid === current_id) postMessage(result);
 				});

packages/repl/src/lib/workers/bundler/plugins/commonjs.ts

@@ -12,7 +12,14 @@ const plugin: Plugin = {
 	name: 'commonjs',
 
 	transform: (code, id) => {
-		if (!/\b(require|module|exports)\b/.test(code)) return;
+		if (
+			id.endsWith('.mjs') ||
+			id.endsWith('.esm.js') ||
+			code.startsWith('import ') ||
+			!/\b(require|module|exports)\b/.test(code)
+		) {
+			return;
+		}
 
 		try {
 			const ast = parse(code, {
@@ -40,9 +47,12 @@ const plugin: Plugin = {
 					if (node.left.object.type !== 'Identifier' || node.left.object.name !== 'exports') return;
 					if (node.left.computed || node.left.property.type !== 'Identifier') return;
 
-					exports.push(
-						`export const ${node.left.property.name} = module.exports.${node.left.property.name};`
-					);
+					// Default is a special case (and would result in invalid syntax) and kinda fucked up: https://github.com/evanw/esbuild/issues/1719#issuecomment-953470495
+					if (node.left.property.name !== 'default') {
+						exports.push(
+							`export const ${node.left.property.name} = module.exports.${node.left.property.name};`
+						);
+					}
 
 					context.next();
 				}
@@ -53,6 +63,11 @@ const plugin: Plugin = {
 				.map((id, i) => `'${id}': __repl_${i}`)
 				.join(', ')} };`;
 
+			// Special case https://github.com/mathiasbynens/CSS.escape/issues/12
+			if (id.includes('css.escape')) {
+				code = code.replace("typeof global != 'undefined' ? global : this", 'globalThis');
+			}
+
 			const transformed = [
 				imports,
 				lookup,