ac-support-connector 1.0.9 → 1.0.11

package/.github/CODEOWNERS

@@ -0,0 +1,9 @@
+# Code Owners
+#
+# This file defines who is responsible for code in this repository.
+# Reviews from code owners are automatically requested for pull requests.
+#
+# More info: https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners
+
+# Default owner for everything in the repo
+* @AdmiralCloud/ac-maintainer-backend

package/.github/workflows/node.js.yml

@@ -16,7 +16,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [18.x, 20.x, 22.x]
+        node-version: [20.x, 22.x]
         # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
         redis-version: [6]
 

package/CHANGELOG.md

@@ -1,4 +1,27 @@
 
+## [1.0.11](https://github.com/admiralcloud/ac-support-connector/compare/v1.0.10..v1.0.11) (2026-02-24 19:50:42)
+
+
+### Bug Fix
+
+* **Misc:** Package updates | MP | [b6607b26ec283654cf2dcffa02c8719d1cc284b6](https://github.com/admiralcloud/ac-support-connector/commit/b6607b26ec283654cf2dcffa02c8719d1cc284b6)    
+Package updates incl ESLint 9 -> 10 and lint fixes  
+Related issues:
+### Chores
+
+* **Misc:** Remove Node 18 from tests | MP | [d08361704454ba261573ef9156584e6732450537](https://github.com/admiralcloud/ac-support-connector/commit/d08361704454ba261573ef9156584e6732450537)    
+Remove Node 18 from tests  
+Related issues:
+
+## [1.0.10](https://github.com/admiralcloud/ac-support-connector/compare/v1.0.9..v1.0.10) (2026-02-03 08:35:38)
+
+
+### Bug Fix
+
+* **Connector:** Package updates | MP | [a5c71a2b5941e2c82e2c078ce5d594dd0b8c12f7](https://github.com/admiralcloud/ac-support-connector/commit/a5c71a2b5941e2c82e2c078ce5d594dd0b8c12f7)    
+Package updates  
+Related issues:
+
 ## [1.0.9](https://github.com/admiralcloud/ac-support-connector/compare/v1.0.8..v1.0.9) (2026-01-27 13:26:39)
 
 

package/Makefile

@@ -2,10 +2,10 @@ MOCHA_OPTS= --slow 0 -A
 REPORTER = spec
 
 lint-fix:
-	./node_modules/.bin/eslint --fix index.js test/test.js
+	./node_modules/.bin/eslint --fix
 
 lint-check:
-	./node_modules/.bin/eslint index.js test/test.js
+	./node_modules/.bin/eslint
 
 commit:
 	@node ./node_modules/ac-semantic-release/lib/commit.js

package/eslint.config.js

@@ -1,30 +1,34 @@
-const globals = require('globals');
+const globals = require('globals')
 
-module.exports = {
-  ignores: [
-    'config/env/**'
-  ],
-  languageOptions: {
-    ecmaVersion: 2022,
-    sourceType: 'module',
-    globals: {
-      ...globals.commonjs,
-      ...globals.es6,
-      ...globals.node,
-      expect: 'readonly',
-      describe: 'readonly',
-      it: 'readonly'
+module.exports = [
+  {
+    files: ['index.js', 'test/test.js'],
+    languageOptions: {
+      ecmaVersion: 2022,
+      sourceType: 'module',
+      globals: {
+        ...globals.commonjs,
+        ...globals.es2015,
+        ...globals.node,
+        expect: 'readonly',
+        describe: 'readonly',
+        it: 'readonly'
+      }
+    },
+    rules: {
+      'no-const-assign': 'error',
+      'space-before-function-paren': 'off',
+      'no-extra-semi': 'off',
+      'object-curly-spacing': ['error', 'always'],
+      'brace-style': ['error', 'stroustrup', { allowSingleLine: true }],
+      'block-spacing': 'error',
+      'no-useless-escape': 'off',
+      'no-console': ['warn', { allow: ['warn', 'error'] }],
+      'no-unused-vars': 'error',
+      'eqeqeq': 'error',
+      'no-var': 'error',
+      'curly': 'error',
+      'prefer-const': ['error', { ignoreReadBeforeAssign: true }]
     }
-  },
-  rules: {
-    'no-const-assign': 'error', // Ensure this rule is enabled
-    'space-before-function-paren': 'off',
-    'no-extra-semi': 'off',
-    'object-curly-spacing': ['error', 'always'],
-    'brace-style': ['error', 'stroustrup', { allowSingleLine: true }],
-    'no-useless-escape': 'off',
-    'standard/no-callback-literal': 'off',
-    'new-cap': 'off',
-    'no-console': ['warn', { allow: ['warn', 'error'] }]
   }
-};
+]

package/index.js

@@ -33,7 +33,7 @@ module.exports = {
           fifo: true
         }]
       }
-      if (profile) sqsParams.profile = profile // Optional AWS profile, see below
+      if (profile) { sqsParams.profile = profile } // Optional AWS profile, see below
       if (bucket) {
         sqsParams.useS3 = { 
           enabled: true,
@@ -71,11 +71,11 @@ module.exports = {
   createMessage: async function({ subject, text, message = [], attachments = [], block, platforms, reporter, messageGroupId }) {
     // check block time in Redis, fallback to memory
     if (block) {
-      if (block < 1) block = 1 // make sure to use reasonable value!
+      if (block < 1) { block = 1 } // make sure to use reasonable value!
       const key = `${process.env.NODE_ENV}:supportConnector:${this.serviceName}:${Buffer.from(subject).toString('base64')}`
       if (this.redisInstance) {
         // OK if set for the first time, null if existing
-        let response = await this.redisInstance.set(key, 1, 'ex', block, 'nx')
+        const response = await this.redisInstance.set(key, 1, 'ex', block, 'nx')
         if (response !== 'OK') {
           return// { statusCode: 423, message: 'keyBlocked' }
         }
@@ -91,11 +91,11 @@ module.exports = {
     }
 
     // message is an array of objects, if it is empty and just text is set, add it to the array
-    if (message.length === 0) message.push({ type: 'text', content: text })
+    if (message.length === 0) { message.push({ type: 'text', content: text }) }
 
     // service is also part of message array
     message.push({ type: 'service', content: this.serviceName })
-    if (this.instanceId) message.push({ type: 'instanceId', content: this.instanceId })
+    if (this.instanceId) { message.push({ type: 'instanceId', content: this.instanceId }) }
 
     const messagePayload = {
       platforms,
@@ -103,7 +103,7 @@ module.exports = {
       reporter,
       message,
     }
-    if (attachments.length > 0) messagePayload.attachments = attachments
+    if (attachments.length > 0) { messagePayload.attachments = attachments }
 
 
     if (process.env.NODE_ENV === 'test') {
@@ -111,7 +111,7 @@ module.exports = {
     }
     try {
       const response = await this.acsqs.sendSQSMessage({ name: this.sqsQueue, message: JSON.stringify(messagePayload), messageGroupId: (messageGroupId || this.serviceName) })
-      if (this.debugMode) console.warn(response)
+      if (this.debugMode) { console.warn(response) }
       return response
     }
     catch(e) {

package/package.json

@@ -3,18 +3,19 @@
   "author": "Mark Poepping (https://www.admiralcloud.com)",
   "license": "MIT",
   "repository": "admiralcloud/ac-support-connector",
-  "version": "1.0.9",
+  "version": "1.0.11",
   "dependencies": {
-    "@aws-sdk/client-sts": "^3.975.0",
-    "ac-sqs": "^3.3.3",
+    "@aws-sdk/client-sts": "^3.996.0",
+    "ac-sqs": "^4.0.4",
     "node-cache": "^5.1.2"
   },
   "devDependencies": {
     "ac-semantic-release": "^0.4.10",
     "chai": "^4.5.0",
-    "eslint": "^9.39.2",
-    "expect": "^29.7.0",
-    "ioredis": "^5.9.2",
+    "eslint": "^10.0.2",
+    "expect": "^30.2.0",
+    "globals": "^17.3.0",
+    "ioredis": "^5.9.3",
     "mocha": "^11.7.5"
   },
   "scripts": {
@@ -24,7 +25,8 @@
     "node": ">=20.0.0"
   },
   "resolutions": {
-    "mocha/diff": "^8.0.3"
+    "mocha/diff": "^8.0.3",
+    "minimatch": "^10.2.1"
   },
   "packageManager": "yarn@1.22.22+sha512.a6b2f7906b721bba3d67d4aff083df04dad64c399707841b7acf00f6b133b7ac24255f2652fa22ae3534329dc6180534e98d17432037ff6fd140556e2bb3137e"
 }

package/test/test.js

@@ -14,11 +14,11 @@ function timeout(ms) {
 
 describe('Basic tests', () => {
   it('Test properties', async() => {
-    let params = {
+    const params = {
       subject: 'Failed',
       text: 'This failed'
     }
-    let response = await supCon.createMessage(params)
+    const response = await supCon.createMessage(params)
     expect(response).to.have.property('subject', params.subject)
     expect(response).to.have.property('message').that.is.an('array').with.lengthOf.at.least(1);
     expect(response.message[0]).to.have.property('type', 'text')
@@ -26,7 +26,7 @@ describe('Basic tests', () => {
   })
 
   it('Check truncated payload > 256KB - should not throw an error', async() => {
-    let params = {
+    const params = {
       subject: 'Failed',
       message: [{
         type: 'text',
@@ -34,7 +34,7 @@ describe('Basic tests', () => {
       }]
     }
 
-    let response = await supCon.createMessage(params)
+    const response = await supCon.createMessage(params)
     expect(response).to.have.property('subject', params.subject)
     expect(response).to.have.property('message').that.is.an('array').with.lengthOf.at.least(1);
     expect(response.message[0]).to.have.property('type', 'text')
@@ -52,24 +52,24 @@ describe('Block with Redis', () => {
   })
 
   it('Send message - should work', async() => {
-    let params = {
+    const params = {
       subject: 'Failed',
       text: 'This failed',
       level: 'warn',
       block: 5
     }
-    let response = await supCon.createMessage(params)
+    const response = await supCon.createMessage(params)
     expect(response).to.have.property('subject', params.subject)
   })
 
   it('Send message - should be blocked', async() => {
-    let params = {
+    const params = {
       subject: 'Failed',
       text: 'This failed',
       level: 'warn',
       block: 5
     }
-    let response = await supCon.createMessage(params)
+    const response = await supCon.createMessage(params)
     expect(response).to.be.undefined
   })
 
@@ -78,13 +78,13 @@ describe('Block with Redis', () => {
   }).timeout(10000)
 
   it('Send message - should work again', async() => {
-    let params = {
+    const params = {
       subject: 'Failed',
       text: 'This failed',
       level: 'warn',
       block: 5
     }
-    let response = await supCon.createMessage(params)
+    const response = await supCon.createMessage(params)
     expect(response).to.have.property('subject', params.subject)
   })  
 })
@@ -99,24 +99,24 @@ describe('Block with Memory', () => {
   })
   
   it('Send message - should work', async() => {
-    let params = {
+    const params = {
       subject: 'Failed',
       text: 'This failed',
       level: 'warn',
       block: 5
     }
-    let response = await supCon.createMessage(params)
+    const response = await supCon.createMessage(params)
     expect(response).to.have.property('subject', params.subject)
   })
 
   it('Send message - should be blocked', async() => {
-    let params = {
+    const params = {
       subject: 'Failed',
       text: 'This failed',
       level: 'warn',
       block: 5
     }
-    let response = await supCon.createMessage(params)
+    const response = await supCon.createMessage(params)
     expect(response).to.be.undefined
   })
 
@@ -126,13 +126,13 @@ describe('Block with Memory', () => {
   }).timeout(10000)
 
   it('Send message - should work again', async() => {
-    let params = {
+    const params = {
       subject: 'Failed',
       text: 'This failed',
       level: 'warn',
       block: 5
     }
-    let response = await supCon.createMessage(params)
+    const response = await supCon.createMessage(params)
     expect(response).to.have.property('subject', params.subject)
   })