core-services-sdk 1.3.19 → 1.3.21

package/.eslintrc.json

@@ -0,0 +1,14 @@
+{
+  "env": {
+    "es2021": true,
+    "node": true
+  },
+  "extends": ["eslint:recommended", "plugin:prettier/recommended"],
+  "plugins": ["prettier"],
+  "rules": {
+    "prettier/prettier": "error",
+    "no-unused-vars": "warn",
+    "no-console": "off"
+  },
+  "ignores": ["node_modules", "dist", ".next"]
+}

package/.vscode/settings.json

@@ -12,5 +12,10 @@
   },
   "javascript.preferences.importModuleSpecifierEnding": "js",
   "js/ts.implicitProjectConfig.checkJs": true,
-  "javascript.validate.enable": true
+  "javascript.validate.enable": true,
+  "eslint.validate": ["javascript"],
+
+  "editor.codeActionsOnSave": {
+    "source.fixAll.eslint": true
+  }
 }

package/eslint.config.js

@@ -0,0 +1,21 @@
+import eslintPluginPrettier from 'eslint-plugin-prettier'
+
+/** @type {import("eslint").Linter.FlatConfig} */
+export default [
+  {
+    files: ['**/*.js'],
+    languageOptions: {
+      sourceType: 'module',
+      ecmaVersion: 'latest',
+    },
+    plugins: {
+      prettier: eslintPluginPrettier,
+    },
+    rules: {
+      'no-console': 'off',
+      'no-unused-vars': 'warn',
+      'curly': ['error', 'all'],
+      'prettier/prettier': 'error',
+    },
+  },
+]

package/package.json

@@ -1,10 +1,14 @@
 {
   "name": "core-services-sdk",
-  "version": "1.3.19",
+  "version": "1.3.21",
   "main": "src/index.js",
   "type": "module",
   "scripts": {
-    "test": "vitest run --coverage"
+    "lint": "eslint .",
+    "lint:fix": "eslint . --fix",
+    "test": "vitest run --coverage",
+    "format": "prettier --write .",
+    "bump": "node ./scripts/bump-version.js"
   },
   "repository": {
     "type": "git",
@@ -28,7 +32,6 @@
     "google-libphonenumber": "^3.2.42",
     "http-status": "^2.1.0",
     "mongodb": "^6.18.0",
-    "node-fetch": "^3.3.2",
     "nodemailer": "^7.0.5",
     "pino": "^9.7.0",
     "uuid": "^11.1.0",
@@ -38,6 +41,10 @@
     "@vitest/coverage-v8": "^3.2.4",
     "path": "^0.12.7",
     "url": "^0.11.4",
-    "vitest": "^3.2.4"
+    "vitest": "^3.2.4",
+    "eslint": "^9.30.0",
+    "eslint-config-prettier": "^10.1.5",
+    "eslint-plugin-prettier": "^5.5.1",
+    "prettier": "^3.6.2"
   }
-}
+}

package/scripts/bump-version.js

@@ -0,0 +1,68 @@
+import fs from 'fs'
+import path from 'path'
+import { execSync } from 'child_process'
+
+const type = process.argv[2] // "major" | "minor" | "patch" | "-major" | "-minor" | "-patch"
+
+if (!['major', 'minor', 'patch', '-major', '-minor', '-patch'].includes(type)) {
+  console.error(`Usage: npm run bump <major|minor|patch|-major|-minor|-patch>`)
+  process.exit(1)
+}
+
+const pkgPath = path.resolve(process.cwd(), 'package.json')
+const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf-8'))
+
+let [major, minor, patch] = pkg.version.split('.').map(Number)
+
+const bump = () => {
+  switch (type) {
+    case 'major':
+      return [major + 1, 0, 0]
+    case 'minor':
+      return [major, minor + 1, 0]
+    case 'patch':
+      return [major, minor, patch + 1]
+    case '-major':
+      if (major === 0) {
+        throw new Error('Cannot decrement major below 0')
+      }
+      return [major - 1, 0, 0]
+    case '-minor':
+      if (minor === 0) {
+        if (major === 0) {
+          throw new Error('Cannot decrement minor below 0.0')
+        }
+        return [major - 1, 0, 0]
+      }
+      return [major, minor - 1, 0]
+    case '-patch':
+      if (patch === 0) {
+        if (minor === 0) {
+          if (major === 0) {
+            throw new Error('Cannot decrement below 0.0.0')
+          }
+          return [major - 1, 0, 0]
+        }
+        return [major, minor - 1, 0]
+      }
+      return [major, minor, patch - 1]
+  }
+}
+
+const [newMajor, newMinor, newPatch] = bump()
+const newVersion = `${newMajor}.${newMinor}.${newPatch}`
+
+pkg.version = newVersion
+fs.writeFileSync(pkgPath, JSON.stringify(pkg, null, 2) + '\n')
+
+console.log(`✔ Updated version to ${newVersion}`)
+
+// update package-lock.json if exists
+if (fs.existsSync('./package-lock.json')) {
+  try {
+    execSync('npm install --package-lock-only', { stdio: 'inherit' })
+    console.log('✔ package-lock.json updated')
+  } catch (err) {
+    console.error('Failed to update package-lock.json', err)
+  }
+}

package/src/core/normalize-phone-number.js

@@ -7,10 +7,9 @@ import * as raw from 'google-libphonenumber'
 export function getLib() {
   // Prefer direct (CJS-style or ESM w/ named), else default
   // e.g. raw.PhoneNumberUtil OR raw.default.PhoneNumberUtil
-  // eslint-disable-next-line no-unused-vars
   /** @type {any} */
   const anyRaw = raw
-  const lib = anyRaw.PhoneNumberUtil ? anyRaw : anyRaw.default ?? anyRaw
+  const lib = anyRaw.PhoneNumberUtil ? anyRaw : (anyRaw.default ?? anyRaw)
 
   if (!lib || !lib.PhoneNumberUtil || !lib.PhoneNumberFormat) {
     throw new Error('google-libphonenumber failed to load (exports not found)')
@@ -71,7 +70,9 @@ export function normalizePhoneOrThrowIntl(input) {
   try {
     const util = phoneUtil()
     const parsed = util.parseAndKeepRawInput(clean(input))
-    if (!util.isValidNumber(parsed)) throw new Error('x')
+    if (!util.isValidNumber(parsed)) {
+      throw new Error('Phone number failed validation')
+    }
     return toResult(parsed)
   } catch (e) {
     const err = new Error('Invalid phone number')
@@ -91,7 +92,9 @@ export function normalizePhoneOrThrowWithRegion(input, defaultRegion) {
   try {
     const util = phoneUtil()
     const parsed = util.parseAndKeepRawInput(clean(input), defaultRegion)
-    if (!util.isValidNumber(parsed)) throw new Error('x')
+    if (!util.isValidNumber(parsed)) {
+      throw new Error('Phone number failed validation')
+    }
     return toResult(parsed)
   } catch (e) {
     const err = new Error('Invalid phone number')

package/src/core/regex-utils.js

@@ -16,8 +16,12 @@
  */
 
 export const isValidRegex = (pattern) => {
-  if (pattern instanceof RegExp) return true
-  if (typeof pattern !== 'string') return false
+  if (pattern instanceof RegExp) {
+    return true
+  }
+  if (typeof pattern !== 'string') {
+    return false
+  }
   try {
     new RegExp(pattern)
     return true

package/src/http/http.js

@@ -1,11 +1,12 @@
 /**
- * A lightweight HTTP client wrapper around `node-fetch` supporting JSON, XML, and plain text responses.
- * Provides simplified helper methods for GET, POST, PUT, PATCH, and DELETE with automatic error handling.
+ * A lightweight HTTP client wrapper around Node.js native fetch
+ * supporting JSON, XML, and plain text responses.
+ * Provides simplified helper methods for GET, POST, PUT, PATCH, and DELETE
+ * with automatic error handling.
  *
  * @module http
  */
 
-import fetch from 'node-fetch'
 import httpStatus from 'http-status'
 import { parseStringPromise } from 'xml2js'
 
@@ -20,26 +21,27 @@ const JSON_HEADER = {
 /**
  * Checks if the HTTP status is considered successful (2xx).
  *
- * @param {import('node-fetch').Response} response
+ * @param {Response} response
  * @returns {boolean}
  */
 const isOkStatus = ({ status }) =>
   status >= httpStatus.OK && status < httpStatus.MULTIPLE_CHOICES
 
 /**
- * @param {import('node-fetch').Response} response
- * @returns {Promise<import('node-fetch').Response>}
+ * Ensures the response has a successful status, otherwise throws HttpError.
+ *
+ * @param {Response} response
+ * @returns {Promise<Response>}
  */
 const checkStatus = async (response) => {
   if (!isOkStatus(response)) {
     const text = await response.text()
-    const info = tryConvertJsonResponse(text)
     const { status, statusText } = response
-
     throw new HttpError({
       code: status,
       httpStatusCode: status,
       httpStatusText: statusText,
+      extendInfo: { text },
     })
   }
   return response
@@ -48,19 +50,17 @@ const checkStatus = async (response) => {
 /**
  * Reads the raw text from a fetch response.
  *
- * @param {import('node-fetch').Response} response
- * @returns {Promise<string>} The plain text body.
+ * @param {Response} response
+ * @returns {Promise<string>}
  */
-const getTextResponse = async (response) => {
-  return await response.text()
-}
+const getTextResponse = async (response) => response.text()
 
 /**
  * Attempts to parse a JSON string.
  *
- * @param {string} responseText - A raw JSON string.
- * @returns {Object} Parsed JSON object.
- * @throws {SyntaxError} If the string is not valid JSON.
+ * @param {string} responseText
+ * @returns {Object}
+ * @throws {SyntaxError} If not valid JSON.
  */
 const tryConvertJsonResponse = (responseText) => {
   try {
@@ -72,10 +72,10 @@ const tryConvertJsonResponse = (responseText) => {
 }
 
 /**
- * Attempts to extract a JSON object from a fetch response.
+ * Extracts JSON from a response, or returns raw text on failure.
  *
- * @param {import('node-fetch').Response} response
- * @returns {Promise<Object|string>} Parsed JSON or raw string if parsing fails.
+ * @param {Response} response
+ * @returns {Promise<Object|string>}
  */
 const tryGetJsonResponse = async (response) => {
   let jsonText
@@ -83,16 +83,18 @@ const tryGetJsonResponse = async (response) => {
     jsonText = await getTextResponse(response)
     return tryConvertJsonResponse(jsonText)
   } catch (error) {
-    if (!jsonText) throw error
+    if (!jsonText) {
+      throw error
+    }
     return jsonText
   }
 }
 
 /**
- * Attempts to extract an XML object from a fetch response.
+ * Extracts XML from a response, or returns raw text on failure.
  *
- * @param {import('node-fetch').Response} response
- * @returns {Promise<Object|string>} Parsed XML object or raw string if parsing fails.
+ * @param {Response} response
+ * @returns {Promise<Object|string>}
  */
 const tryGetXmlResponse = async (response) => {
   let xmlText
@@ -100,17 +102,19 @@ const tryGetXmlResponse = async (response) => {
     xmlText = await getTextResponse(response)
     return await parseStringPromise(xmlText)
   } catch (error) {
-    if (!xmlText) throw error
+    if (!xmlText) {
+      throw error
+    }
     return xmlText
   }
 }
 
 /**
- * Extracts and parses the fetch response body based on expected type.
+ * Parses the fetch response based on expected type.
  *
- * @param {import('node-fetch').Response} response
- * @param {string} responseType - Expected type: 'json', 'xml', or 'text'.
- * @returns {Promise<any>} The parsed response payload.
+ * @param {Response} response
+ * @param {string} responseType
+ * @returns {Promise<any>}
  */
 const getResponsePayload = async (response, responseType) => {
   switch (responseType) {
@@ -119,149 +123,100 @@ const getResponsePayload = async (response, responseType) => {
     case ResponseType.xml:
       return tryGetXmlResponse(response)
     default:
-    case ResponseType.text:
       return getTextResponse(response)
   }
 }
 
 /**
  * Sends an HTTP GET request.
- *
- * @param {Object} params
- * @param {string} params.url - Target URL.
- * @param {Object} [params.headers] - Optional request headers.
- * @param {string} [params.credentials='include'] - Credential policy.
- * @param {string} [params.expectedType='json'] - Expected response format.
- * @returns {Promise<any>} Parsed response data.
  */
 export const get = async ({
   url,
   headers = {},
-  credentials = 'include',
   expectedType = ResponseType.json,
 }) => {
-  /** @type {import('node-fetch').Response} */
   const response = await fetch(url, {
     method: HTTP_METHODS.GET,
     headers: { ...JSON_HEADER, ...headers },
-    ...(credentials ? { credentials } : {}),
   })
-
   await checkStatus(response)
-  return await getResponsePayload(response, expectedType)
+  return getResponsePayload(response, expectedType)
 }
 
 /**
  * Sends an HTTP POST request.
- *
- * @param {Object} params
- * @param {string} params.url - Target URL.
- * @param {Object} params.body - Request body (will be JSON.stringify-ed).
- * @param {Object} [params.headers] - Optional request headers.
- * @param {string} [params.credentials='include'] - Credential policy.
- * @param {string} [params.expectedType='json'] - Expected response format.
- * @returns {Promise<any>} Parsed response data.
  */
 export const post = async ({
   url,
   body,
-  headers,
-  credentials = 'include',
+  headers = {},
   expectedType = ResponseType.json,
 }) => {
   const response = await fetch(url, {
     method: HTTP_METHODS.POST,
     headers: { ...JSON_HEADER, ...headers },
     body: JSON.stringify(body),
-    ...(credentials ? { credentials } : {}),
   })
   await checkStatus(response)
-  return await getResponsePayload(response, expectedType)
+  return getResponsePayload(response, expectedType)
 }
 
 /**
  * Sends an HTTP PUT request.
- *
- * @param {Object} params
- * @param {string} params.url
- * @param {Object} params.body
- * @param {Object} [params.headers]
- * @param {string} [params.credentials='include']
- * @param {string} [params.expectedType='json']
- * @returns {Promise<any>} Parsed response data.
  */
 export const put = async ({
   url,
   body,
   headers = {},
-  credentials = 'include',
   expectedType = ResponseType.json,
 }) => {
   const response = await fetch(url, {
     method: HTTP_METHODS.PUT,
     headers: { ...JSON_HEADER, ...headers },
     body: JSON.stringify(body),
-    ...(credentials ? { credentials } : {}),
   })
   await checkStatus(response)
-  return await getResponsePayload(response, expectedType)
+  return getResponsePayload(response, expectedType)
 }
 
 /**
  * Sends an HTTP PATCH request.
- *
- * @param {Object} params - Same as `post`.
- * @returns {Promise<any>} Parsed response data.
  */
 export const patch = async ({
   url,
   body,
-  headers,
-  credentials = 'include',
+  headers = {},
   expectedType = ResponseType.json,
 }) => {
   const response = await fetch(url, {
     method: HTTP_METHODS.PATCH,
     headers: { ...JSON_HEADER, ...headers },
     body: JSON.stringify(body),
-    ...(credentials ? { credentials } : {}),
   })
   await checkStatus(response)
-  return await getResponsePayload(response, expectedType)
+  return getResponsePayload(response, expectedType)
 }
 
 /**
  * Sends an HTTP DELETE request.
- *
- * @param {Object} params - Same as `post`, but body is optional.
- * @returns {Promise<any>} Parsed response data.
  */
 export const deleteApi = async ({
   url,
   body,
-  headers,
-  credentials = 'include',
+  headers = {},
   expectedType = ResponseType.json,
 }) => {
   const response = await fetch(url, {
     method: HTTP_METHODS.DELETE,
     headers: { ...JSON_HEADER, ...headers },
     ...(body ? { body: JSON.stringify(body) } : {}),
-    ...(credentials ? { credentials } : {}),
   })
   await checkStatus(response)
-  return await getResponsePayload(response, expectedType)
+  return getResponsePayload(response, expectedType)
 }
 
 /**
- * Consolidated HTTP client with method shortcuts.
- *
- * @namespace http
- * @property {Function} get - HTTP GET method.
- * @property {Function} post - HTTP POST method.
- * @property {Function} put - HTTP PUT method.
- * @property {Function} patch - HTTP PATCH method.
- * @property {Function} deleteApi - HTTP DELETE method.
+ * Consolidated HTTP client.
  */
 export const http = {
   get,

package/src/mongodb/validate-mongo-uri.js

@@ -26,7 +26,7 @@ export function isValidMongoUri(uri) {
     const hasHostname = Boolean(parsed.hostname)
 
     return isValidProtocol && hasHostname
-  } catch (e) {
+  } catch {
     return false
   }
 }

package/src/rabbit-mq/rabbit-mq.js

@@ -75,7 +75,9 @@ export const subscribeToQueue = async ({
     !!prefetch && (await channel.prefetch(prefetch))
 
     channel.consume(queue, async (msgInfo) => {
-      if (!msgInfo) return
+      if (!msgInfo) {
+        return
+      }
 
       try {
         const { msgId, data } = parseMessage(msgInfo)

package/tests/http/http-method.unit.test.js

@@ -20,7 +20,7 @@ describe('HTTP_METHODS constant', () => {
     // Attempt to mutate and verify it fails
     try {
       HTTP_METHODS.NEW = 'NEW'
-    } catch (err) {
+    } catch {
       // Ignore in strict mode
     }
 

package/tests/http/http.unit.test.js

@@ -1,20 +1,13 @@
 import { vi, it, expect, describe, afterEach } from 'vitest'
 
-import fetch from 'node-fetch'
 import { http } from '../../src/http/http.js'
 import { HttpError } from '../../src/http/HttpError.js'
 import { HTTP_METHODS } from '../../src/http/http-method.js'
 import { ResponseType } from '../../src/http/responseType.js'
 
-vi.mock('node-fetch', async () => {
-  const actual = await vi.importActual('node-fetch')
-  return {
-    ...actual,
-    default: vi.fn(),
-  }
-})
-
-const mockFetch = /** @type {typeof fetch} */ (fetch)
+// Mock the global fetch
+const mockFetch = vi.fn()
+global.fetch = mockFetch
 
 const createMockResponse = ({
   body = '',
@@ -23,22 +16,22 @@ const createMockResponse = ({
   headers = {},
 } = {}) => {
   return {
-    ok: status >= 200 && status < 300,
     status,
     statusText,
     text: vi.fn().mockResolvedValue(body),
-    headers: { get: vi.fn().mockImplementation((k) => headers[k]) },
+    headers: {
+      get: vi.fn().mockImplementation((k) => headers[k]),
+    },
   }
 }
 
-describe('http client', () => {
+describe('http client (native fetch)', () => {
   afterEach(() => {
     vi.clearAllMocks()
   })
 
   describe('GET', () => {
     it('should return parsed JSON response', async () => {
-      // @ts-ignore
       mockFetch.mockResolvedValueOnce(
         createMockResponse({ body: JSON.stringify({ hello: 'world' }) }),
       )
@@ -52,7 +45,6 @@ describe('http client', () => {
     })
 
     it('should throw HttpError on non-2xx status', async () => {
-      // @ts-ignore
       mockFetch.mockResolvedValueOnce(
         createMockResponse({
           status: 404,
@@ -69,7 +61,6 @@ describe('http client', () => {
 
   describe('POST', () => {
     it('should send a JSON body and return parsed JSON', async () => {
-      // @ts-ignore
       mockFetch.mockResolvedValueOnce(
         createMockResponse({ body: JSON.stringify({ ok: true }) }),
       )
@@ -92,7 +83,6 @@ describe('http client', () => {
 
   describe('PUT', () => {
     it('should send a PUT request and return JSON', async () => {
-      // @ts-ignore
       mockFetch.mockResolvedValueOnce(
         createMockResponse({ body: JSON.stringify({ updated: true }) }),
       )
@@ -108,7 +98,6 @@ describe('http client', () => {
 
   describe('PATCH', () => {
     it('should send a PATCH request and return JSON', async () => {
-      // @ts-ignore
       mockFetch.mockResolvedValueOnce(
         createMockResponse({ body: JSON.stringify({ patched: true }) }),
       )
@@ -124,7 +113,6 @@ describe('http client', () => {
 
   describe('DELETE', () => {
     it('should send a DELETE request with body and return JSON', async () => {
-      // @ts-ignore
       mockFetch.mockResolvedValueOnce(
         createMockResponse({ body: JSON.stringify({ deleted: true }) }),
       )
@@ -140,7 +128,6 @@ describe('http client', () => {
 
   describe('ResponseType', () => {
     it('should return text if expectedType is text', async () => {
-      // @ts-ignore
       mockFetch.mockResolvedValueOnce(createMockResponse({ body: 'hello' }))
 
       const result = await http.get({
@@ -153,7 +140,6 @@ describe('http client', () => {
 
     it('should return XML as parsed object if expectedType is xml', async () => {
       const xml = `<note><to>User</to><from>ChatGPT</from></note>`
-      // @ts-ignore
       mockFetch.mockResolvedValueOnce(createMockResponse({ body: xml }))
 
       const result = await http.get({

package/tests/ids/prefixes.unit.test.js

@@ -35,7 +35,7 @@ describe('ID_PREFIXES', () => {
     // Try to mutate
     try {
       ID_PREFIXES.USER = 'hacked'
-    } catch (_) {
+    } catch {
       // Ignore error in strict mode
     }
 

package/tests/templates/template-loader.unit.test.js

@@ -1,4 +1,3 @@
-import dot from 'dot'
 import * as fs from 'fs/promises'
 import { describe, it, expect, vi, beforeEach } from 'vitest'
 
@@ -7,9 +6,6 @@ import { loadTemplates } from '../../src/templates/template-loader.js'
 vi.mock('fs/promises')
 
 describe('loadTemplates', () => {
-  // @ts-ignore
-  const mockCompile = vi.spyOn(dot, 'compile')
-
   beforeEach(() => {
     vi.resetAllMocks()
   })