core-services-sdk 1.3.20 → 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.20",
+  "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",
@@ -37,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

@@ -83,7 +83,9 @@ const tryGetJsonResponse = async (response) => {
     jsonText = await getTextResponse(response)
     return tryConvertJsonResponse(jsonText)
   } catch (error) {
-    if (!jsonText) throw error
+    if (!jsonText) {
+      throw error
+    }
     return jsonText
   }
 }
@@ -100,7 +102,9 @@ const tryGetXmlResponse = async (response) => {
     xmlText = await getTextResponse(response)
     return await parseStringPromise(xmlText)
   } catch (error) {
-    if (!xmlText) throw error
+    if (!xmlText) {
+      throw error
+    }
     return xmlText
   }
 }

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/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()
   })