improving detection for cjs. fixes issue #9

index.js

@@ -80,10 +80,8 @@ function extract(src) {
         });
         throw new Error('Common JS script detected');
     };
-    context.module = {
-        exports: Object.create(null)
-    };
-    context.exports = context.module.exports;
+    context.exports = Object.create(null);
+    context.module = context;
 
 
     // executing the JavaScript source into a new context to avoid leaking
@@ -98,11 +96,10 @@ function extract(src) {
         }
     } finally {
         // very dummy detection process for CommonJS modules
-        if (typeof context.module.exports === 'function' ||
-                typeof context.exports === 'function' ||
-                Object.keys(context.module.exports).length > 0 ||
+        if (typeof context.exports === 'function' ||
+                typeof context.exports === 'string' ||
+                typeof context.exports === 'number' ||
                 Object.keys(context.exports).length > 0 ||
-                Object.getPrototypeOf(context.module.exports) ||
                 Object.getPrototypeOf(context.exports)) {
             mods.push({type: 'cjs'});
         }

test/fixtures/cjs-export-number.js

@@ -0,0 +1 @@
+module.exports = 2;

test/fixtures/cjs-export-regex.js

@@ -0,0 +1 @@
+module.exports = new RegExp('abc');

test/fixtures/cjs-export-string.js

@@ -0,0 +1 @@
+exports = 'something';