fix: validate installed dependencies only if they exists

Author: jotanarciso

lib/build/dispatcher/dispatcher.js

@@ -17,6 +17,7 @@ import {
   presets,
   getExportedFunctionBody,
   getPackageManager,
+  getProjectJsonFile,
   relocateImportsAndRequires,
 } from '#utils';
 import { Messages } from '#constants';
@@ -201,6 +202,37 @@ async function folderExistsInProject(folder) {
     return Promise.resolve(false);
   }
 }
+/**
+ * Check if node_modules dir exists
+ */
+async function checkNodeModulesDir() {
+  let projectJson;
+  try {
+    projectJson = getProjectJsonFile('package.json');
+  } catch (error) {
+    if (error.code === 'ENOENT') {
+      return;
+    }
+
+    feedback.prebuild.error(error);
+    process.exit(1);
+  }
+
+  if (
+    projectJson &&
+    (projectJson.dependencies || projectJson.devDependencies)
+  ) {
+    const pkgManager = await getPackageManager();
+    const existNodeModules = await folderExistsInProject('node_modules');
+
+    if (!existNodeModules) {
+      feedback.prebuild.error(
+        Messages.build.error.install_dependencies_failed(pkgManager),
+      );
+      process.exit(1);
+    }
+  }
+}
 
 /**
  * Class representing a Dispatcher for build operations.
@@ -231,15 +263,7 @@ class Dispatcher {
    * Run the build process.
    */
   run = async () => {
-    // Check install depenedencies
-    const pckManager = await getPackageManager();
-    const existNodeModules = await folderExistsInProject('node_modules');
-    if (!existNodeModules) {
-      feedback.prebuild.error(
-        Messages.build.error.install_dependencies_failed(pckManager),
-      );
-      process.exit(1);
-    }
+    await checkNodeModulesDir();
 
     // Load Context based on preset
     const { entryContent, prebuild, config } = await loadBuildContext(

lib/utils/generateTimestamp/generateTimestamp.utils.test.js

@@ -1,4 +1,4 @@
-import generateTimestamp from './index.js';
+import generateTimestamp from './generateTimestamp.utils.js';
 
 describe('generateTimestamp utils', () => {
   test('Should generate a timestamp string in the format "YYYYMMDDHHmmss".', async () => {

lib/utils/getExportedFunctionBody/getExportedFunctionBody.utils.js

@@ -1,4 +1,4 @@
-import parser from '@babel/parser';
+import * as parser from '@babel/parser';
 import _traverse from '@babel/traverse';
 import _generate from '@babel/generator';
 import * as t from '@babel/types';

lib/utils/getExportedFunctionBody/getExportedFunctionBody.utils.test.js

@@ -1,4 +1,4 @@
-import getExportedFunctionBody from '../lib/utils/getExportedFunctionBody/getExportedFunctionBody.utils.js';
+import getExportedFunctionBody from './getExportedFunctionBody.utils.js';
 
 describe('getExportedFunctionBody', () => {
   it('should extract the body from a default exported function declaration', () => {
@@ -12,7 +12,9 @@ describe('getExportedFunctionBody', () => {
       return 'Hello, world!';
     `;
 
-    expect(getExportedFunctionBody(inputCode)).toBe(expectedOutput);
+    expect(getExportedFunctionBody(inputCode).replace(/\s/g, '')).toBe(
+      expectedOutput.replace(/\s/g, ''),
+    );
   });
 
   it('should extract the body from a default exported anonymous function', () => {
@@ -26,7 +28,9 @@ describe('getExportedFunctionBody', () => {
       return 'Hello, world!';
     `;
 
-    expect(getExportedFunctionBody(inputCode)).toBe(expectedOutput);
+    expect(getExportedFunctionBody(inputCode).replace(/\s/g, '')).toBe(
+      expectedOutput.replace(/\s/g, ''),
+    );
   });
 
   it('should extract the body from a default exported arrow function', () => {
@@ -40,7 +44,9 @@ describe('getExportedFunctionBody', () => {
       return 'Hello, world!';
     `;
 
-    expect(getExportedFunctionBody(inputCode)).toBe(expectedOutput);
+    expect(getExportedFunctionBody(inputCode).replace(/\s/g, '')).toBe(
+      expectedOutput.replace(/\s/g, ''),
+    );
   });
 
   it('should extract the body from a default exported function assigned to a variable', () => {
@@ -55,7 +61,9 @@ describe('getExportedFunctionBody', () => {
       return 'Hello, world!';
     `;
 
-    expect(getExportedFunctionBody(inputCode)).toBe(expectedOutput);
+    expect(getExportedFunctionBody(inputCode).replace(/\s/g, '')).toBe(
+      expectedOutput.replace(/\s/g, ''),
+    );
   });
 
   it('should throw an error if there are no default exports', () => {
@@ -65,6 +73,8 @@ describe('getExportedFunctionBody', () => {
       }
     `;
 
-    expect(() => getExportedFunctionBody(inputCode)).toThrow('No default exports were found for entrypoint in the provided code.');
+    expect(() => getExportedFunctionBody(inputCode)).toThrow(
+      'No default exports were found for entrypoint in the provided code.',
+    );
   });
-});
+});