Fix transformation issue of 'require' (#26)

* Fix transformation issue of 'require'

* Remove added devDependencies for test execution

Author: MartinKolbAtWork

packages/vite-plugin-commonjs/__tests__/transform.spec.ts

@@ -28,6 +28,18 @@ test('transform require', () => {
     expect(result.code).toMatch(/import \* as .+ from \'@\/page\/login';/);
 });
 
+test('require function on object', () => {
+  let code = `myObj.require("react");`
+  let result = transformRequire(code, 'main.ts');
+  expect(result.code).toMatch(`myObj.require("react");`);
+});
+
+test('require as part of function name', () => {
+  let code = `myrequire("react");`
+  let result = transformRequire(code, 'main.ts');
+  expect(result.code).toMatch(`myrequire("react");`);
+});
+
 test('require in comments', () => {
     //singleline comments
     let code = ` const a=0; // the hook will be setup by require("react").`

packages/vite-plugin-commonjs/src/lib.ts

@@ -1,5 +1,5 @@
 const commonJSRegex: RegExp = /\b(module\.exports|exports\.\w+|exports\s*=\s*|exports\s*\[.*\]\s*=\s*)/;
-const requireRegex: RegExp = /_{0,2}require\s*\(\s*(["'].*?["'])\s*\)/g;
+const requireRegex: RegExp = /(?<!\.)\b_{0,2}require\s*\(\s*(["'].*?["'])\s*\)/g;
 const IMPORT_STRING_PREFIX: String = "__require_for_vite";
 const multilineCommentsRegex = /\/\*(.|[\r\n])*?\*\//gm
 const singleCommentsRegex = /([^\:])\/\/.*/g