@manyos/smileconnect-api 1.73.0 → 1.74.2

package/.claude/settings.local.json

@@ -3,7 +3,13 @@
     "allow": [
       "Bash(cat:*)",
       "Bash(find:*)",
-      "Bash(xargs ls:*)"
+      "Bash(xargs ls:*)",
+      "Bash(npm view *)",
+      "Bash(npm pack *)",
+      "WebFetch(domain:unpkg.com)",
+      "WebFetch(domain:www.npmjs.com)",
+      "Bash(node *)",
+      "Bash(grep -E '^\\(p-queue|proper-lockfile|async-mutex|p-limit\\)$')"
     ]
   }
 }

package/Dockerfile

@@ -1,7 +1,7 @@
 ### STAGE 1: Build ###
 
 # We label our stage as 'builder'
-FROM node:21-bullseye as builder
+FROM node:22-bullseye as builder
 ARG NPM_TOKEN
 COPY package.json package-lock.json ./
 

package/controller/scriptController.js

@@ -153,7 +153,7 @@ async function renameScript(scriptIdOld, scriptIdNew) {
             })
             scriptDefOps[usage.scriptType] = scriptDefNew
         }
-        setClient(usage.client, clientConfig)
+        await setClient(usage.client, clientConfig)
     }
     return true;
 }

package/docs/releases.md

@@ -1,6 +1,15 @@
 # Release Notes
 
 ## API
+### 1.74.2 - 20.05.26
+Improve client config validation to avoid duplicate client configs.
+
+### 1.74.1 - 12.05.26
+Fix issue with dynamic openapi spec for worklogs.
+
+### 1.74.0 - 29.04.26
+Update node.js to v22 and replace request-native with native fetch in adapter.
+
 ### 1.73.0 - 28.04.26
 Add detectMime flag to attachments & attachments to dynamic openapi spec.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@manyos/smileconnect-api",
-  "version": "1.73.0",
+  "version": "1.74.2",
   "description": "A proxy and abstraction layer for BMCs IT Service Management Suite",
   "main": "app.js",
   "scripts": {
@@ -14,7 +14,7 @@
     "node": ">=0.10.3"
   },
   "dependencies": {
-    "@manyos/adapter-foundation": "^1.1.2",
+    "@manyos/adapter-foundation": "^2.0.0",
     "@manyos/logger": "^1.3.0",
     "bunyan": "^1.8.15",
     "bunyan-express-serializer": "^1.0.0",

package/routes/appConfigRoutes.js

@@ -317,7 +317,7 @@ module.exports = (function() {
 
     routes.post('/clients', isAuthorizedAdmin,
         checkSchema(clientConfigSchema),
-        function (req, res, next) {
+        async function (req, res, next) {
             const origData = JSON.parse(JSON.stringify(req.body));
             setEventData(
                 req,
@@ -336,7 +336,7 @@ module.exports = (function() {
                 try {
                     const clientConfig = req.body.data;
                     const clientName = clientConfig.name;
-                    config.setClient(clientName, clientConfig)
+                    await config.setClient(clientName, clientConfig)
                     const savedClientConfig = config.getClient(clientName);
                     req.result = {
                         "data": savedClientConfig
@@ -351,7 +351,7 @@ module.exports = (function() {
 
     routes.put('/clients/:id', isAuthorizedAdmin,
         checkSchema(clientConfigSchema),
-        function (req, res, next) {
+        async function (req, res, next) {
             const id = req.params.id;
             const origData = JSON.parse(JSON.stringify(req.body));
             setEventData(
@@ -370,7 +370,7 @@ module.exports = (function() {
                 try {
                     const clientConfig = req.body.data;
                     const clientName = clientConfig.name;
-                    config.setClient(id, clientConfig);
+                    await config.setClient(id, clientConfig);
                     const savedClientConfig = config.getClient(clientName);
                     req.result = {
                         "data": savedClientConfig

package/util/config.js

@@ -1,11 +1,14 @@
 const fs = require('fs');
 const fsPromises = require('fs').promises;
+const crypto = require('crypto');
+const pLimit = require('p-limit');
 require('dotenv').config();
 
 const path = require('path');
 const log = require('@manyos/logger').setupLog('SMILEconnect', path.basename(__filename));
 
 const clientConfigFile = 'conf/clients.json';
+const clientConfigWriteLimit = pLimit(1);
 const arquery = require('../util/arquery');
 const mappingUtil = require('../util/mappingUtil');
 const CacheService = require ('../util/cache.service');
@@ -123,17 +126,31 @@ function setMappings(mapping) {
     return true
 }
 
-function setClient(clientName, clientConfig) {
+async function setClient(clientName, clientConfig) {
     log.debug('start to update config in file', clientConfigFile);
     log.debug('set client data', clientName, clientConfig);
-    let config = JSON.parse(fs.readFileSync(clientConfigFile)) || {};
-    config = config.filter(function( obj ) {
-        return obj.name !== clientName;
+    if (!clientConfig || typeof clientConfig !== 'object' || !clientConfig.name) {
+        throw new Error('setClient: clientConfig.name is required');
+    }
+    return clientConfigWriteLimit(async () => {
+        let config;
+        try {
+            const raw = await fsPromises.readFile(clientConfigFile, 'utf8');
+            config = JSON.parse(raw);
+        } catch (e) {
+            log.warn('setClient: cannot parse existing clients.json, starting empty', e.message);
+            config = [];
+        }
+        if (!Array.isArray(config)) config = [];
+        const namesToRemove = new Set([clientName, clientConfig.name].filter(Boolean));
+        config = config.filter(obj => obj && !namesToRemove.has(obj.name));
+        config.push(clientConfig);
+        log.debug('new config size', config.length);
+        const tmp = `${clientConfigFile}.tmp.${process.pid}.${crypto.randomBytes(4).toString('hex')}`;
+        await fsPromises.writeFile(tmp, JSON.stringify(config, null, 2));
+        await fsPromises.rename(tmp, clientConfigFile);
+        return true;
     });
-    config.push(clientConfig)
-    log.debug('new config', config);
-    fs.writeFileSync(clientConfigFile, JSON.stringify(config, null, 2));
-    return true
 }
 
 function deleteClient(clientName) {
@@ -243,7 +260,7 @@ async function checkCustomFormMapping() {
     }
 }
 
-function checkClientConfig(client) {
+async function checkClientConfig(client) {
     let updateRequired = false;
     const clientKeys = [
         'cmdbobject',
@@ -361,25 +378,25 @@ function checkClientConfig(client) {
         log.warn(`check config for client ${client.name} done. Update required.`);
         client.config = clientConfig;
         //save
-        setClient(client.name, client);
+        await setClient(client.name, client);
         log.warn(`check config for client ${client.name} done. Update done.`);
     }
     log.info(`check config for client ${client.name} done. No update required.`);
 }
 
-function checkClientConfigs() {
+async function checkClientConfigs() {
     log.info('check client configs');
     const clientConfigs = getClients();
-    clientConfigs.forEach(clientConfig => {
-        checkClientConfig(clientConfig);
-    });
+    for (const clientConfig of clientConfigs) {
+        await checkClientConfig(clientConfig);
+    }
     log.info('check client configs done');
 }
 
 async function checkConfig() {
     log.info('check config');
     checkMapping();
-    checkClientConfigs();
+    await checkClientConfigs();
     await checkCustomFormMapping();
     log.info('check config done');
 }
@@ -434,7 +451,7 @@ async function getDesignPackage(clientId) {
                 const mapping = getDeprecatedMappingAsCustom(config.requestTypeWorkLog)
                 schemas[config.requestTypeWorkLog] = await getObjectSchema(clientConfig[config.requestTypeWorkLog].fields, mapping, config.forms.workLog)
                 schemas[config.requestTypeWorkLog + '_create_update'] = await getObjectSchema(clientConfig[config.requestTypeWorkLog].fields, mapping, config.forms.workLog, true)
-                paths = {...paths, ... await getPathDef(config.assocTicketType, config.baseURI + '/{ticketId}/worklogs', config.requestTypeWorkLog, true, true)}
+                paths = {...paths, ... await getPathDef(`${config.assocTicketType} Worklog`, config.baseURI + '/{ticketId}/worklogs', config.requestTypeWorkLog, true, true)}
                 paths = {...paths, ... await getAttachmentPathDef(config.assocTicketType, config.baseURI + '/{ticketId}/worklogs')}
             }
             if (config.requestTemplate && clientConfig[config.requestTemplate].fields  && clientConfig[config.requestTemplate].fields.length > 0) {