Refactor error handling in stack, cache provider, and entry modules to utilize centralized message constants for improved maintainability and clarity.

reeshika-h · reeshika-h · commit 438aa00166b9 · 2025-11-24T10:47:37.000+05:30
diff --git a/src/core/cache-provider/index.js b/src/core/cache-provider/index.js
@@ -1,11 +1,13 @@
 import localstorage from './localstorage';
+import MESSAGES from './messages';
+
 const CacheProvider = {};
 
 CacheProvider.providers = function (provider) {
   if (provider) {
     return localstorage;
   } else {
-    console.error('Missing cache provider. Provide a valid provider and try again.');
+    console.error(MESSAGES.CACHE_PROVIDER_MISSING);
   }
 };
 
diff --git a/src/core/lib/utils.js b/src/core/lib/utils.js
@@ -1,5 +1,6 @@
 import Request from './request';
 import Result from '../modules/result';
+import MESSAGES from '../messages';
 
 /**
  * @method addSpread
@@ -28,7 +29,7 @@ export function transform (type) {
           this._query[type].BASE = query;
           return this;
         } else {
-          console.error('Invalid parameters. Expected a string or an array of field names.');
+          console.error(MESSAGES.TRANSFORM_INVALID_SINGLE_PARAM);
         }
         break;
       case 2:
@@ -38,11 +39,11 @@ export function transform (type) {
           this._query[type][arguments[0]] = query;
           return this;
         } else {
-          console.error('Invalid parameters. Expected first parameter as a string (reference field UID) and second parameter as a string or an array of field names.');
+          console.error(MESSAGES.TRANSFORM_INVALID_DOUBLE_PARAM);
         }
         break;
       default:
-        console.error('Invalid parameters. Provide either one parameter (field name or array) or two parameters (reference field UID and field name or array).');
+        console.error(MESSAGES.TRANSFORM_INVALID_PARAM_COUNT);
     }
   };
 }
diff --git a/src/core/messages.js b/src/core/messages.js
@@ -0,0 +1,35 @@
+/**
+ * Centralized error and informational messages for Contentstack JavaScript SDK
+ * @module messages
+ */
+
+const MESSAGES = {
+  // Cache Provider Messages
+  CACHE_PROVIDER_MISSING: 'Missing cache provider. Provide a valid provider and try again.',
+  CACHE_POLICY_INVALID: 'Invalid cache policy. Provide a valid policy value and try again.',
+
+  // Stack Initialization Messages
+  STACK_INVALID_PARAMS_OBJECT: 'Invalid parameters. The specified API Key, Delivery Token, or Environment Name is invalid.',
+  STACK_INVALID_PARAMS_STRING: 'Invalid string parameters. Provide valid API Key, Delivery Token, and Environment Name.',
+  STACK_INVALID_PARAMS_GENERIC: 'Invalid parameters. Provide valid parameters to initialize the Contentstack javascript-SDK Stack.',
+  STACK_OBSOLETE_FUNCTION: "WARNING! Obsolete function called. Function 'Contentstack.Stack(api_key, delivery_token, environment)' has been deprecated, please use 'Contentstack.Stack({api_key, delivery_token, environment, region, branch, fetchOptions})' function instead!",
+
+  // Entry Messages
+  ENTRY_UID_REQUIRED: "Entry UID required. Provide an entry UID. e.g. .Entry('entry_uid')",
+  ENTRY_INCLUDE_OWNER_DEPRECATED: 'The includeOwner function is deprecated. This functionality is no longer supported. Please remove this method from your code.',
+  ENTRY_INVALID_ARGUMENT: 'Invalid argument. Argument should be a String or an Array.',
+  ENTRY_LANGUAGE_INVALID: 'Invalid language code. Argument should be a String.',
+  ENTRY_ADD_QUERY_INVALID: 'Invalid query parameters. First argument should be a String.',
+  ENTRY_ADD_PARAM_INVALID: 'Invalid parameters. Both key and value should be strings.',
+
+  // Query/Transform Messages (only/except)
+  TRANSFORM_INVALID_SINGLE_PARAM: 'Invalid parameters. Expected a string or an array of field names.',
+  TRANSFORM_INVALID_DOUBLE_PARAM: 'Invalid parameters. Expected first parameter as a string (reference field UID) and second parameter as a string or an array of field names.',
+  TRANSFORM_INVALID_PARAM_COUNT: 'Invalid parameters. Provide either one parameter (field name or array) or two parameters (reference field UID and field name or array).',
+
+  // Request/Error Messages
+  REQUEST_ERROR_OCCURRED: (error) => `An error occurred: ${error}`,
+};
+
+export default MESSAGES;
+
diff --git a/src/core/modules/entry.js b/src/core/modules/entry.js
@@ -1,4 +1,5 @@
 import * as Utils from '../lib/utils';
+import MESSAGES from '../messages';
 
 /**
  * @class
@@ -83,7 +84,7 @@ export default class Entry {
         this.queryCachePolicy = policy;
       }
     } else {
-      if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'Kindly provide the valid policy');
+      if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.CACHE_POLICY_INVALID);
     }
     return this;
   }
@@ -135,7 +136,7 @@ export default class Entry {
       }
       return this;
     } else {
-      if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'Argument should be a String or an Array.');
+      if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.ENTRY_INVALID_ARGUMENT);
     }
   }
 
@@ -160,7 +161,7 @@ export default class Entry {
       this._query.locale = language_code;
       return this;
     } else {
-      if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'Argument should be a String.');
+      if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.ENTRY_LANGUAGE_INVALID);
     }
   }
 
@@ -179,7 +180,7 @@ export default class Entry {
       this._query[key] = value;
       return this;
     } else {
-      if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'First argument should be a String.');
+      if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.ENTRY_ADD_QUERY_INVALID);
     }
   }
 
@@ -294,7 +295,7 @@ export default class Entry {
      * @instance
      */
   includeOwner () {
-    console.warn('The includeOwner function is deprecated. This functionality is no longer supported. Please remove this method from your code.');
+    console.warn(MESSAGES.ENTRY_INCLUDE_OWNER_DEPRECATED);
     this._query.include_owner = true;
     return this;
   }
@@ -338,7 +339,7 @@ export default class Entry {
       this._query[key] = value;
       return this;
     } else {
-      if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'Kindly provide valid parameters.');
+      if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.ENTRY_ADD_PARAM_INVALID);
     }
   }
 
@@ -391,7 +392,7 @@ export default class Entry {
       const options = Utils.mergeDeep(this.fetchOptions, fetchOptions);
       return Utils.sendRequest(Utils.mergeDeep({}, this), options);
     } else {
-      if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', "Kindly provide an entry uid. e.g. .Entry('asset_uid')");
+      if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.ENTRY_UID_REQUIRED);
     }
   }
 }
diff --git a/src/core/stack.js b/src/core/stack.js
@@ -6,6 +6,7 @@ import Query from './modules/query';
 import Taxonomy from './modules/taxonomy';
 import Request from './lib/request';
 import CacheProvider from './cache-provider/index';
+import MESSAGES from './messages';
 const errorRetry = [408, 429];
 
 /**
@@ -118,10 +119,10 @@ export default class Stack {
           this.environment = stack_arguments[0].environment;
           return this;
         } else {
-          if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'Kindly provide valid object parameters. The specified API Key, Delivery Token, or Environment Name is invalid.');
+          if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.STACK_INVALID_PARAMS_OBJECT);
         }
       case 3:
-        if (this.fetchOptions.debug) this.fetchOptions.logHandler('warning', "WARNING! Obsolete function called. Function 'Contentstack.Stack(api_key, delivery_token, environment)' has been deprecated, please use 'Contentstack.Stack({api_key, delivery_token, environment, region, branch, fetchOptions})' function instead!");
+        if (this.fetchOptions.debug) this.fetchOptions.logHandler('warning', MESSAGES.STACK_OBSOLETE_FUNCTION);
         if (typeof stack_arguments[0] === 'string' && typeof stack_arguments[1] === 'string' && typeof stack_arguments[2] === 'string') {
           this.headers = {
             api_key: stack_arguments[0],
@@ -130,18 +131,18 @@ export default class Stack {
           this.environment = stack_arguments[2];
           return this;
         } else {
-          if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'Kindly provide valid string parameters.');
+          if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.STACK_INVALID_PARAMS_STRING);
         }
       case 4:
-        if (this.fetchOptions.debug) this.fetchOptions.logHandler('warning', "WARNING! Obsolete function called. Function 'Contentstack.Stack(api_key, delivery_token, environment)' has been deprecated, please use 'Contentstack.Stack({api_key, delivery_token, environment, region, branch, fetchOptions})' function instead!");
+        if (this.fetchOptions.debug) this.fetchOptions.logHandler('warning', MESSAGES.STACK_OBSOLETE_FUNCTION);
         if (typeof stack_arguments[0] === 'string' && typeof stack_arguments[1] === 'string' && typeof stack_arguments[2] === 'string') {
           this.headers = {
             api_key: stack_arguments[0],
             access_token: stack_arguments[1]
           };
           this.environment = stack_arguments[2];
         } else {
-          if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'Kindly provide valid string parameters.');
+          if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.STACK_INVALID_PARAMS_STRING);
         }
         if (stack_arguments[3]) {
           if (typeof stack_arguments[3] === 'string' && stack_arguments[3] !== undefined && stack_arguments[3] !== 'us') {
@@ -152,15 +153,15 @@ export default class Stack {
         }
         return this;
       case 5:
-        if (this.fetchOptions.debug) this.fetchOptions.logHandler('warning', "WARNING! Obsolete function called. Function 'Contentstack.Stack(api_key, delivery_token, environment)' has been deprecated, please use 'Contentstack.Stack({api_key, delivery_token, environment, region, branch, fetchOptions})' function instead!");
+        if (this.fetchOptions.debug) this.fetchOptions.logHandler('warning', MESSAGES.STACK_OBSOLETE_FUNCTION);
         if (typeof stack_arguments[0] === 'string' && typeof stack_arguments[1] === 'string' && typeof stack_arguments[2] === 'string') {
           this.headers = {
             api_key: stack_arguments[0],
             access_token: stack_arguments[1]
           };
           this.environment = stack_arguments[2];
         } else {
-          if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'Kindly provide valid string parameters.');
+          if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.STACK_INVALID_PARAMS_STRING);
         }
 
         if (stack_arguments[3]) {
@@ -175,7 +176,7 @@ export default class Stack {
         }
         return this;
       default:
-        if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'Kindly provide valid parameters to initialize the Contentstack javascript-SDK Stack.');
+        if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.STACK_INVALID_PARAMS_GENERIC);
     }
   }
 
@@ -240,7 +241,7 @@ export default class Stack {
         this.queryCachePolicy = policy;
       }
     } else {
-      if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'Kindly provide the valid policy');
+      if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.CACHE_POLICY_INVALID);
     }
     return this;
   }
@@ -426,7 +427,7 @@ export default class Stack {
      *    .then(function(result) {
      *      // 'result' is a single contentType information.
      *     }).catch((error) => {
-     *        console.error('An error occurred:', error.message || error)
+     *        console.error(MESSAGES.REQUEST_ERROR_OCCURRED(error.message || error))
      *  });
      * @returns {promise}
      * @instance

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`import * as Utils from '../lib/utils';`
	`2`	`+import MESSAGES from '../messages';`
`2`	`3`
`3`	`4`	`/**`
`4`	`5`	`* @class`
`@@ -83,7 +84,7 @@ export default class Entry {`
`83`	`84`	`this.queryCachePolicy = policy;`
`84`	`85`	`}`
`85`	`86`	`} else {`
`86`		`- if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'Kindly provide the valid policy');`
	`87`	`+ if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.CACHE_POLICY_INVALID);`
`87`	`88`	`}`
`88`	`89`	`return this;`
`89`	`90`	`}`
`@@ -135,7 +136,7 @@ export default class Entry {`
`135`	`136`	`}`
`136`	`137`	`return this;`
`137`	`138`	`} else {`
`138`		`- if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'Argument should be a String or an Array.');`
	`139`	`+ if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.ENTRY_INVALID_ARGUMENT);`
`139`	`140`	`}`
`140`	`141`	`}`
`141`	`142`
`@@ -160,7 +161,7 @@ export default class Entry {`
`160`	`161`	`this._query.locale = language_code;`
`161`	`162`	`return this;`
`162`	`163`	`} else {`
`163`		`- if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'Argument should be a String.');`
	`164`	`+ if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.ENTRY_LANGUAGE_INVALID);`
`164`	`165`	`}`
`165`	`166`	`}`
`166`	`167`
`@@ -179,7 +180,7 @@ export default class Entry {`
`179`	`180`	`this._query[key] = value;`
`180`	`181`	`return this;`
`181`	`182`	`} else {`
`182`		`- if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'First argument should be a String.');`
	`183`	`+ if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.ENTRY_ADD_QUERY_INVALID);`
`183`	`184`	`}`
`184`	`185`	`}`
`185`	`186`
`@@ -294,7 +295,7 @@ export default class Entry {`
`294`	`295`	`* @instance`
`295`	`296`	`*/`
`296`	`297`	`includeOwner () {`
`297`		`- console.warn('The includeOwner function is deprecated. This functionality is no longer supported. Please remove this method from your code.');`
	`298`	`+ console.warn(MESSAGES.ENTRY_INCLUDE_OWNER_DEPRECATED);`
`298`	`299`	`this._query.include_owner = true;`
`299`	`300`	`return this;`
`300`	`301`	`}`
`@@ -338,7 +339,7 @@ export default class Entry {`
`338`	`339`	`this._query[key] = value;`
`339`	`340`	`return this;`
`340`	`341`	`} else {`
`341`		`- if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', 'Kindly provide valid parameters.');`
	`342`	`+ if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.ENTRY_ADD_PARAM_INVALID);`
`342`	`343`	`}`
`343`	`344`	`}`
`344`	`345`
`@@ -391,7 +392,7 @@ export default class Entry {`
`391`	`392`	`const options = Utils.mergeDeep(this.fetchOptions, fetchOptions);`
`392`	`393`	`return Utils.sendRequest(Utils.mergeDeep({}, this), options);`
`393`	`394`	`} else {`
`394`		`- if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', "Kindly provide an entry uid. e.g. .Entry('asset_uid')");`
	`395`	`+ if (this.fetchOptions.debug) this.fetchOptions.logHandler('error', MESSAGES.ENTRY_UID_REQUIRED);`
`395`	`396`	`}`
`396`	`397`	`}`
`397`	`398`	`}`