fix: TS support for eslint, type defs for utils

.eslintrc.cjs

@@ -12,6 +12,27 @@
 
 module.exports = {
   root: true,
-  extends: '@adobe/helix',
-  plugins: ['import'],
+  parser: '@typescript-eslint/parser',
+  parserOptions: {
+    ecmaVersion: 2020,
+    sourceType: 'module',
+  },
+  extends: [
+    '@adobe/helix',
+    'plugin:@typescript-eslint/recommended',
+  ],
+  plugins: [
+    'import',
+    '@typescript-eslint',
+  ],
+  overrides: [
+    {
+      files: ['*.ts'],
+      rules: {},
+    },
+    {
+      files: ['*.js', '*.cjs'],
+      rules: {},
+    },
+  ],
 };

package.json

@@ -31,6 +31,8 @@
     "@semantic-release/changelog": "6.0.3",
     "@semantic-release/git": "10.0.1",
     "@semantic-release/npm": "9.0.2",
+    "@typescript-eslint/eslint-plugin": "6.13.1",
+    "@typescript-eslint/parser": "6.13.1",
     "ajv": "8.12.0",
     "c8": "8.0.1",
     "eslint": "8.54.0",
@@ -41,7 +43,8 @@
     "mocha-multi-reporters": "1.5.1",
     "nock": "13.3.8",
     "semantic-release": "19.0.5",
-    "semantic-release-monorepo": "7.0.5"
+    "semantic-release-monorepo": "7.0.5",
+    "typescript": "5.3.2"
   },
   "lint-staged": {
     "*.js": "eslint"

packages/spacecat-shared-utils/src/functions.js

@@ -49,21 +49,21 @@ function isNumber(value) {
 /**
  * Checks if the given parameter is an object and not an array or null.
  *
- * @param {*} obj - The object to check.
+ * @param {*} value - The value to check.
  * @returns {boolean} True if the parameter is an object, false otherwise.
  */
-function isObject(obj) {
-  return !Array.isArray(obj) && obj !== null && typeof obj === 'object';
+function isObject(value) {
+  return !Array.isArray(value) && value !== null && typeof value === 'object';
 }
 
 /**
  * Determines if the given parameter is a string.
  *
- * @param {*} str - The string to check.
+ * @param {*} value - The value to check.
  * @returns {boolean} True if the parameter is a string, false otherwise.
  */
-function isString(str) {
-  return (!!str || str === '') && typeof str === 'string';
+function isString(value) {
+  return (!!value || value === '') && typeof value === 'string';
 }
 
 /**
@@ -73,17 +73,17 @@ function isString(str) {
  * @returns {boolean} True if the string is not empty, false otherwise.
  */
 function hasText(str) {
-  return !!str && typeof str === 'string';
+  return !!str && isString(str);
 }
 
 /**
  * Checks whether the given object is a valid JavaScript Date.
  *
- * @param {*} obj - The object to check.
+ * @param {*} value - The value to check.
  * @returns {boolean} True if the given object is a valid Date object, false otherwise.
  */
-function isValidDate(obj) {
-  return obj instanceof Date && !Number.isNaN(obj.getTime());
+function isValidDate(value) {
+  return value instanceof Date && !Number.isNaN(value.getTime());
 }
 
 /**