refactor: Refactor configuration loading class using jssert

Author: arpan404

src/core/config.ts

@@ -1,8 +1,7 @@
 import fs from 'node:fs';
 import path from 'node:path';
 
-import Jrror, { JoorError } from '@/core/error';
-import logger from '@/helpers/joorLogger';
+import { handleError, jssert } from '@/core/error';
 import validateConfig from '@/helpers/validateConfig';
 import JOOR_CONFIG from '@/types/config';
 
@@ -26,16 +25,12 @@ class Configuration {
    */
   private async loadConfig(): Promise<void> {
     // Check if the configuration data is already loaded
-    if (Configuration.configData !== null) {
-      throw new Jrror({
-        code: 'config-loaded-already',
-        docsPath: '/configuration',
-        message:
-          'The configuration data is already loaded. Attempting to load it again is not recommended',
-        type: 'warn',
-      });
-    }
-
+    jssert(
+      Configuration.configData === null,
+      'Configuration data is already loaded. Attempting to load it again is not recommended.',
+      '/configuration',
+      'warn'
+    );
     try {
       // Default config file name is joor.config.js or else fallback to joor.config.ts
       let configFile = 'joor.config.js';
@@ -44,29 +39,21 @@ class Configuration {
         configFile = 'joor.config.ts';
       }
 
-      if (!fs.existsSync(path.resolve(process.cwd(), configFile))) {
-        throw new Jrror({
-          code: 'config-file-missing',
-          docsPath: '/configuration',
-          message:
-            'The configuration file (joor.config.js or joor.config.ts) is missing in the root directory.',
-          type: 'error',
-        });
-      }
+      // Check if the configuration file exists
+      jssert(
+        fs.existsSync(path.resolve(process.cwd(), configFile)),
+        'The configuration file (joor.config.js or joor.config.ts) is missing in the root directory.',
+        '/configuration',
+        'error'
+      );
 
       const configPath = path.resolve(process.cwd(), configFile);
       // Dynamically import the configuration file
       const configData = (await import(configPath)).config as JOOR_CONFIG;
       Configuration.configData = validateConfig(configData);
-      Configuration.configData = configData;
       this.setConfigToEnv();
     } catch (error) {
-      throw new Jrror({
-        code: 'config-load-failed',
-        message: `Error occured while loading the configuration file. ${error}`,
-        type: 'panic',
-        docsPath: '/configuration',
-      });
+      handleError(error);
     }
   }
 
@@ -106,11 +93,7 @@ class Configuration {
       try {
         await this.loadConfig();
       } catch (error: unknown) {
-        if (error instanceof Jrror || error instanceof JoorError) {
-          error.handle();
-        } else {
-          logger.error(error);
-        }
+        handleError(error);
       }
     }
 

src/core/error.ts

@@ -122,8 +122,8 @@ class JoorError extends Error {
               Error Code: ${this.errorCode}
               Message: ${this.message}
               ${marker.greenBright(
-      'For more information, visit:'
-    )} ${marker.bgGreenBright.whiteBright(docLink)}
+                'For more information, visit:'
+              )} ${marker.bgGreenBright.whiteBright(docLink)}
               `;
   }
 }
@@ -163,22 +163,24 @@ class Jrror extends JoorError {
       // Throws error code joor-e1 if errorData is not provided, e2 if code is not provided, e3 if message is not provided, e4 if type is not provided
       throw new Jrror({
         message: `Instance of Jrror has been created without passing required data. 
-              Missing: ${!errorData
-            ? 'errorData'
-            : !errorData.code
-              ? 'error code'
-              : !errorData.message
-                ? 'message'
-                : 'type'
-          }`,
-        code: `jrror-${!errorData
+              Missing: ${
+                !errorData
+                  ? 'errorData'
+                  : !errorData.code
+                    ? 'error code'
+                    : !errorData.message
+                      ? 'message'
+                      : 'type'
+              }`,
+        code: `jrror-${
+          !errorData
             ? 'e1'
             : !errorData.code
               ? 'e2'
               : !errorData.message
                 ? 'e3'
                 : 'e4'
-          }`,
+        }`,
         type: 'error',
       });
     }
@@ -196,7 +198,7 @@ class Jrror extends JoorError {
  * If the error is not an instance of Jrror or JoorError, it logs the error.
  *
  * @param {unknown} error - The error to handle.
- * 
+ *
  * Meant to reduce code duplication while handling errors in the codebase.
  */
 function handleError(error: unknown): void {
@@ -208,23 +210,30 @@ function handleError(error: unknown): void {
 }
 
 /**
-* Implicitly asserts that a condition is true. If the condition is false, it throws an error with the provided message and documentation path.
-* Alternative to `assert(condition, message)` from the `node:assert` module.
-* 
-* For naming convention, `jssert` is used to avoid confusion with the `assert` function from the `node:assert` module.
-* 
-* @param {boolean} condition - The condition to assert.
-* @param {string} message - The error message to throw if the assertion fails.
-* @param {string} docsPath - The documentation path for the error.
-*/
-function jssert(condition: boolean, message: string, docsPath: string = "/assertion"): asserts condition {
+ * Implicitly asserts that a condition is true. If the condition is false, it throws an error with the provided message and documentation path.
+ * Alternative to `assert(condition, message)` from the `node:assert` module.
+ *
+ * For naming convention, `jssert` is used to avoid confusion with the `assert` function from the `node:assert` module.
+ *
+ * @param {boolean} condition - The condition to assert.
+ * @param {string} message - The error message to throw if the assertion fails.
+ * @param {string} docsPath - The documentation path for the error.
+ * @param {JOOR_ERROR['type']} type - The type of error to throw ["warn" | "error" | panic]
+ *
+ */
+function jssert(
+  condition: boolean,
+  message: string,
+  docsPath: string = '/assertion',
+  type: JOOR_ERROR['type'] = 'error'
+): asserts condition {
   if (!condition) {
     throw new Jrror({
       code: 'assertion-failed',
       message,
-      type: 'error',
+      type: type,
       docsPath: docsPath,
-    })
+    });
   }
 }