@iobroker/js-controller-cli 4.0.0-alpha.9-20211115-5dac659e → 4.0.3

package/lib/cli/cliProcess.js

@@ -8,14 +8,13 @@ const dbTools = require('@iobroker/js-controller-common-db').tools;
 const deepClone = require('deep-clone');
 const { EXIT_CODES } = require('@iobroker/js-controller-common');
 
-const { getObjectFrom, getInstanceName, normalizeAdapterName, enumInstances} = require('./cliTools.js');
+const { getObjectFrom, getInstanceName, normalizeAdapterName, enumInstances } = require('./cliTools.js');
 
 // The root of this project. Change this when moving code to another directory
 const rootDir = tools.getControllerDir();
 const killAllScriptPath = path.join(rootDir, 'killall.sh');
 
 module.exports = class CLIProcess extends CLICommand {
-
     /** @param {import('./cliCommand').CLICommandOptions} options */
     constructor(options) {
         super(options);
@@ -62,7 +61,7 @@ module.exports = class CLIProcess extends CLICommand {
      */
     stop(args) {
         const adapterName = normalizeAdapterName(args[0]);
-        if (adapterName === undefined ) {
+        if (adapterName === undefined) {
             this.stopJSController();
         } else if (adapterName === 'all') {
             this.setAllAdaptersEnabled(false);
@@ -99,7 +98,7 @@ module.exports = class CLIProcess extends CLICommand {
      * @param {boolean} [restartIfRunning=false] Whether running instances should be restarted
      */
     setAdapterEnabled(adapter, enabled, restartIfRunning) {
-        const {callback, dbConnect} = this.options;
+        const { callback, dbConnect } = this.options;
         dbConnect(async objects => {
             // Due to the many return locations we cannot simply chain the promises here
             // Use the pre-Node8 async/await pattern
@@ -108,7 +107,10 @@ module.exports = class CLIProcess extends CLICommand {
                 const adapterInstances = await enumInstances(objects, adapter);
                 // If there are multiple instances for this adapter, ask the user to specify which one
                 if (adapterInstances.length > 1) {
-                    CLI.error.specifyInstance(adapter, adapterInstances.map(obj => obj._id.substring('system.adapter.'.length)));
+                    CLI.error.specifyInstance(
+                        adapter,
+                        adapterInstances.map(obj => obj._id.substring('system.adapter.'.length))
+                    );
                     return void callback(EXIT_CODES.INVALID_ADAPTER_ID);
                 } else if (adapterInstances.length === 0) {
                     CLI.error.noInstancesFound(adapter);
@@ -162,14 +164,14 @@ module.exports = class CLIProcess extends CLICommand {
         const daemon = setupDaemonize();
         // On non-Windows OSes start KILLALL script
         // to make sure nothing keeps running
-        if (!require('os').platform().match(/^win/)) {
+        if (!require('os').platform().startsWith('win')) {
             daemon.on('stopped', () => {
                 let data = '';
                 if (fs.existsSync(killAllScriptPath)) {
                     fs.chmodSync(killAllScriptPath, '777');
                     const child = require('child_process').spawn(killAllScriptPath, [], { windowsHide: true });
-                    child.stdout.on('data', _data => data += _data.toString().replace(/\n/g, ''));
-                    child.stderr.on('data', _data => data += _data.toString().replace(/\n/g, ''));
+                    child.stdout.on('data', _data => (data += _data.toString().replace(/\n/g, '')));
+                    child.stderr.on('data', _data => (data += _data.toString().replace(/\n/g, '')));
                     child.on('exit', exitCode => {
                         console.log('Exit code for "killall.sh": ' + exitCode);
                         return void callback();
@@ -185,9 +187,7 @@ module.exports = class CLIProcess extends CLICommand {
     /** Restarts the JS controller */
     restartJSController() {
         const daemon = setupDaemonize();
-        daemon
-            .on('stopped', () => daemon.start())
-            .on('notrunning', () => daemon.start());
+        daemon.on('stopped', () => daemon.start()).on('notrunning', () => daemon.start());
         daemon.stop();
     }
 
@@ -207,7 +207,10 @@ module.exports = class CLIProcess extends CLICommand {
                     const alive = hostAlive ? hostAlive.val : false;
                     CLI.success.controllerStatus(alive);
                     console.log();
-                    if (!dbTools.isLocalStatesDbServer(config.states.type, config.states.host) && !dbTools.isLocalObjectsDbServer(config.objects.type, config.objects.host)) {
+                    if (
+                        !dbTools.isLocalStatesDbServer(config.states.type, config.states.host) &&
+                        !dbTools.isLocalObjectsDbServer(config.objects.type, config.objects.host)
+                    ) {
                         CLI.success.systemStatus(!isOffline);
                     }
 
@@ -232,7 +235,10 @@ module.exports = class CLIProcess extends CLICommand {
                     const adapterInstances = await enumInstances(_objects, adapterName);
                     // If there are multiple instances of this adapter, ask the user to specify which one
                     if (adapterInstances.length > 1) {
-                        CLI.error.specifyInstance(adapterName, adapterInstances.map(obj => obj._id.substring('system.adapter.'.length)));
+                        CLI.error.specifyInstance(
+                            adapterName,
+                            adapterInstances.map(obj => obj._id.substring('system.adapter.'.length))
+                        );
                         return void callback(EXIT_CODES.INVALID_ADAPTER_ID);
                     } else if (adapterInstances.length === 0) {
                         CLI.error.noInstancesFound(adapterName);
@@ -341,7 +347,9 @@ function setupDaemonize() {
         }
     } catch {
         console.warn('Cannot read memoryLimitMB');
-        console.warn(`May be config file does not exist.\nPlease call "${tools.appName} setup first" to initialize the settings.`);
+        console.warn(
+            `May be config file does not exist.\nPlease call "${tools.appName} setup first" to initialize the settings.`
+        );
     }
     const startObj = {
         main: path.join(rootDir, 'controller.js'),

package/lib/cli/cliStates.js

@@ -1,14 +1,13 @@
 'use strict';
 const { tools } = require('@iobroker/js-controller-common');
-const CLI               = require('./messages.js');
-const CLICommand        = require('./cliCommand.js');
-const {formatValue}     = require('./cliTools');
+const CLI = require('./messages.js');
+const CLICommand = require('./cliCommand.js');
+const { formatValue } = require('./cliTools');
 const ALIAS_STARTS_WITH = 'alias.';
 
 /** Command iobroker state ... */
 module.exports = class CLIStates extends CLICommand {
-
-    /** @param {import('./cliCommand').CLICommandOptions} options */
+    /** @param {CLICommandOptions} options */
     constructor(options) {
         super(options);
     }
@@ -33,19 +32,16 @@ module.exports = class CLIStates extends CLICommand {
                         return 'null';
                     }
 
-                    return [
-                        obj.val,
-                        obj.ack,
-                        obj.from,
-                        obj.ts,
-                        obj.lc
-                    ].map(line => formatValue(line)).join('\n');
+                    return [obj.val, obj.ack, obj.from, obj.ts, obj.lc].map(line => formatValue(line)).join('\n');
                 };
                 return this.get_(args, resultTransform);
             case 'getValue':
             case 'getvalue':
-                resultTransform = obj => obj ? formatValue(obj.val, pretty) : 'null';
+                resultTransform = obj => (obj ? formatValue(obj.val, pretty) : 'null');
                 return this.get_(args, resultTransform);
+            case 'getBinary':
+            case 'getbinary':
+                return this._getBinary(args);
             case 'set':
                 return this.set_(args);
             case 'chmod':
@@ -55,6 +51,10 @@ module.exports = class CLIStates extends CLICommand {
             case 'delete':
             case 'del':
                 return this.delete(args);
+            case 'getDBVersion':
+                return this.getDBVersion(args);
+            case 'setDBVersion':
+                return this.setDBVersion();
             default:
                 CLI.error.unknownCommand('state', command);
                 showHelp();
@@ -62,32 +62,133 @@ module.exports = class CLIStates extends CLICommand {
         }
     }
 
+    /**
+     * Get the protocol version
+     */
+    getDBVersion() {
+        const { callback, dbConnect } = this.options;
+        dbConnect(async (objects, states) => {
+            const version = await states.getProtocolVersion();
+            console.log(`Current States DB protocol version: ${version}`);
+            return void callback();
+        });
+    }
+
+    /**
+     * Set protocol version
+     */
+    setDBVersion() {
+        const { callback, dbConnect } = this.options;
+        dbConnect(async (objects, states) => {
+            const rl = require('readline-sync');
+
+            let answer = rl.question('Changing the protocol version will restart all hosts! Continue? [N/y]', {
+                limit: /^(yes|y|n|no)$/i,
+                defaultInput: 'no'
+            });
+
+            answer = answer.toLowerCase();
+
+            if (answer !== 'y' && answer !== 'yes') {
+                console.log('Protocol version has not been changed!');
+                return void callback();
+            }
+
+            try {
+                await states.setProtocolVersion(this.options.version);
+            } catch (e) {
+                console.error(`Cannot update protocol version: ${e.message}`);
+                return void callback(1);
+            }
+            console.log(`States DB protocol updated to version ${this.options.version}`);
+            return void callback();
+        });
+    }
+
+    /**
+     * Checks if state is a binary state
+     * @param {string} id id of the state
+     * @param {object} objects the objects db
+     * @param {ioBroker.OtherObject?} obj cached object
+     * @return {Promise<boolean>}
+     * @private
+     */
+    async _isBinary(id, objects, obj) {
+        obj = obj || (await objects.getObjectAsync(id));
+
+        return !!(obj && (obj.binary || (obj.common && obj.common.type === 'file')));
+    }
+
+    /**
+     * Get and show binary state
+     *
+     * @param {any[]} args
+     * @private
+     */
+    _getBinary(args) {
+        const { callback, dbConnect } = this.options;
+        const id = args[1];
+        dbConnect(async (objects, states) => {
+            try {
+                /** @type Buffer | null */
+                const state = await states.getBinaryState(id);
+
+                if (!state) {
+                    CLI.error.stateNotFound(id);
+                    return void callback(1);
+                }
+
+                if (Buffer.isBuffer(state)) {
+                    console.log(state.toString(this.options.encoding));
+                    return void callback();
+                } else {
+                    CLI.error.stateNotBinary(id);
+                    return void callback(1);
+                }
+            } catch (e) {
+                CLI.error.unknown(e);
+                return void callback(1);
+            }
+        });
+    }
+
     /**
      * Returns the value of a state
      * @param {any[]} args
      * @param {(input: any) => any} resultTransform
      */
     get_(args, resultTransform) {
-        const {callback, dbConnect} = this.options;
+        const { callback, dbConnect } = this.options;
         const id = args[1];
 
-        dbConnect((objects, states) => {
+        dbConnect(async (objects, states) => {
             if (id.startsWith(ALIAS_STARTS_WITH)) {
                 objects.getObject(id, (err, targetObj) => {
                     // alias
                     if (targetObj && targetObj.common && targetObj.common.alias && targetObj.common.alias.id) {
-                        const aliasId = typeof targetObj.common.alias.id.read === 'string' ? targetObj.common.alias.id.read : targetObj.common.alias.id;
-                        objects.getObject(aliasId, (err, sourceObj) => {
-                            // write target
-                            states.getState(aliasId, (err, state) => {
-                                if (err || !targetObj) {
-                                    CLI.error.unknown(err);
+                        const aliasId =
+                            typeof targetObj.common.alias.id.read === 'string'
+                                ? targetObj.common.alias.id.read
+                                : targetObj.common.alias.id;
+                        objects.getObject(aliasId, async (err, sourceObj) => {
+                            // read target
+                            try {
+                                if (await this._isBinary(aliasId, objects, targetObj)) {
+                                    CLI.error.stateBinaryGetUnsupported(aliasId);
+                                    return void callback(1);
+                                }
+
+                                const state = await states.getStateAsync(aliasId);
+                                if (!state) {
+                                    CLI.error.stateNotFound(id);
                                 } else {
                                     tools.formatAliasValue(sourceObj.common, targetObj.common, state, console, '');
                                     console.log(resultTransform(state));
                                 }
-                                return void callback();
-                            });
+                            } catch (e) {
+                                CLI.error.unknown(e);
+                            }
+                            return void callback();
                         });
                     } else {
                         CLI.error.unknown(err || `Alias ${id} has no target`);
@@ -95,14 +196,21 @@ module.exports = class CLIStates extends CLICommand {
                     }
                 });
             } else {
-                states.getState(id, (err, state) => {
-                    if (err || !state) {
-                        CLI.error.stateNotFound(id, err);
+                try {
+                    if (await this._isBinary(id, objects)) {
+                        CLI.error.stateBinaryGetUnsupported(id);
+                        return void callback(1);
+                    }
+                    const state = await states.getStateAsync(id);
+                    if (!state) {
+                        CLI.error.stateNotFound(id);
                     } else {
                         console.log(resultTransform(state));
                     }
-                    return void callback();
-                });
+                } catch (e) {
+                    CLI.error.unknown(e);
+                }
+                return void callback();
             }
         });
     }
@@ -111,7 +219,7 @@ module.exports = class CLIStates extends CLICommand {
      * @param {any[]} args
      */
     set_(args) {
-        const {callback, dbConnect, showHelp} = this.options;
+        const { callback, dbConnect, showHelp } = this.options;
         // eslint-disable-next-line prefer-const
         let [id, val, ack] = /** @type {[string, any, any]} */ (args.slice(1));
         const force = args.includes('--force') || args.includes('-f');
@@ -127,13 +235,20 @@ module.exports = class CLIStates extends CLICommand {
         }
 
         dbConnect((objects, states) => {
-            const newVal = ack === undefined ?  {val, ack: false} : {val, ack: !!ack};
+            const newVal = ack === undefined ? { val, ack: false } : { val, ack: !!ack };
 
             if (id.startsWith(ALIAS_STARTS_WITH)) {
-                objects.getObject(id, (err, obj) => {
+                objects.getObject(id, async (err, obj) => {
+                    if (await this._isBinary(id, objects, obj)) {
+                        CLI.error.stateBinarySetUnsupported(id);
+                        return void callback(1);
+                    }
                     // alias
                     if (obj && obj.common && obj.common.alias && obj.common.alias.id) {
-                        const aliasId = typeof obj.common.alias.id.write === 'string' ? obj.common.alias.id.write : obj.common.alias.id;
+                        const aliasId =
+                            typeof obj.common.alias.id.write === 'string'
+                                ? obj.common.alias.id.write
+                                : obj.common.alias.id;
 
                         objects.getObject(aliasId, (err, targetObj) => {
                             if (err) {
@@ -152,20 +267,28 @@ module.exports = class CLIStates extends CLICommand {
                                     newVal.val = parseFloat(newVal.val);
                                 } else if (obj.common.type === 'boolean') {
                                     newVal.val = newVal.val.toString();
-                                    newVal.val = newVal.val === 'true' || newVal.val === '1' || newVal.val === 'ON' || newVal.val === 'on';
+                                    newVal.val =
+                                        newVal.val === 'true' ||
+                                        newVal.val === '1' ||
+                                        newVal.val === 'ON' ||
+                                        newVal.val === 'on';
                                 }
                             }
 
                             // write target
-                            states.setState(aliasId, tools.formatAliasValue(obj.common, targetObj.common, newVal, console, ''), err => {
-                                if (err) {
-                                    CLI.error.unknown(err);
-                                    return void callback(1); // ?
-                                } else {
-                                    CLI.success.stateUpdated(id, val, !!ack);
-                                    return void callback();
+                            states.setState(
+                                aliasId,
+                                tools.formatAliasValue(obj.common, targetObj.common, newVal, console, ''),
+                                err => {
+                                    if (err) {
+                                        CLI.error.unknown(err);
+                                        return void callback(1); // ?
+                                    } else {
+                                        CLI.success.stateUpdated(id, val, !!ack);
+                                        return void callback();
+                                    }
                                 }
-                            });
+                            );
                         });
                     } else {
                         CLI.error.unknown(`Alias ${id} has no target`);
@@ -173,11 +296,17 @@ module.exports = class CLIStates extends CLICommand {
                     }
                 });
             } else {
-                objects.getObject(id, (err, obj) => {
+                objects.getObject(id, async (err, obj) => {
                     if (err) {
                         CLI.error.unknown(err);
                         return void callback(1); // access error
                     }
+
+                    if (await this._isBinary(id, objects, obj)) {
+                        CLI.error.stateBinarySetUnsupported(id);
+                        return void callback(1);
+                    }
+
                     if (!obj && !force) {
                         CLI.error.objectNotFound(id, 'null');
                         return void callback(1); // object not exists
@@ -190,7 +319,11 @@ module.exports = class CLIStates extends CLICommand {
                             newVal.val = parseFloat(newVal.val);
                         } else if (obj.common.type === 'boolean') {
                             newVal.val = newVal.val.toString();
-                            newVal.val = newVal.val === 'true' || newVal.val === '1' || newVal.val === 'ON' || newVal.val === 'on';
+                            newVal.val =
+                                newVal.val === 'true' ||
+                                newVal.val === '1' ||
+                                newVal.val === 'ON' ||
+                                newVal.val === 'on';
                         }
                     }
 
@@ -213,7 +346,7 @@ module.exports = class CLIStates extends CLICommand {
      * @param {any[]} args
      */
     delete(args) {
-        const {callback, dbConnect} = this.options;
+        const { callback, dbConnect } = this.options;
         /** @type {string} */
         const id = args[1];
         if (!id) {
@@ -230,6 +363,7 @@ module.exports = class CLIStates extends CLICommand {
                     CLI.success.stateDeleted(id);
                     return void callback();
                 }
-            }));
+            })
+        );
     }
 };

package/lib/cli/cliTools.js

@@ -9,9 +9,7 @@ const { tools } = require('@iobroker/js-controller-common');
 function formatValue(val, pretty) {
     // Only use JSON.stringify if we need it (for objects and arrays)
     const needsStringify = tools.isObject(val) || tools.isArray(val);
-    const output = !needsStringify ? val
-        : pretty ? JSON.stringify(val, null, 2)
-            : JSON.stringify(val);
+    const output = !needsStringify ? val : pretty ? JSON.stringify(val, null, 2) : JSON.stringify(val);
     return output;
 }
 
@@ -53,6 +51,7 @@ function validateAdapterOrInstanceIdentifier(name) {
  * Ensures that the given string contains a valid identifier for
  * an adapter (without instance number) or instance (with instance number)
  * @param {string} name
+ * @return {{name: string, instance:string|null, version: string|null, nameWithVersion:string}|null}
  */
 function splitAdapterOrInstanceIdentifierWithVersion(name) {
     const res = name.match(/^([a-z0-9\-_]+)\.?(\d+)?@?([a-z0-9\-_.]*)?$/);
@@ -108,7 +107,7 @@ function enumHosts(objects) {
 function enumObjects(objects, type, startkey) {
     return new Promise((resolve, reject) => {
         const endkey = startkey + '\u9999';
-        objects.getObjectView('system', type, {startkey, endkey}, null, (err, res) => {
+        objects.getObjectView('system', type, { startkey, endkey }, null, (err, res) => {
             if (err) {
                 return reject(err);
             }

package/lib/cli/messages.js

@@ -2,49 +2,39 @@
 const { tools } = require('@iobroker/js-controller-common');
 
 const errorMessages = Object.freeze({
-    stateNotFound: (
-        /** @type {string} */ stateID,
-        /** @type {string?} */ error
-    ) => `The state ${stateID} was not found!` + (error ? ` Reason: ${error}` : ''),
-    objectNotFound: (
-        /** @type {string} */ objectID,
-        /** @type {string?} */ error
-    ) => `The object ${objectID} was not found!` + (error ? ` Reason: ${error}` : ''),
-    cannotUpdateObject: (
-        /** @type {string} */ objectID,
-        /** @type {string?} */ error
-    ) => `The object ${objectID} could not be updated!` + (error ? ` Reason: ${error}` : ''),
-    cannotDeleteObject: (
-        /** @type {string} */ objectID,
-        /** @type {string?} */ error
-    ) => `The object ${objectID} cannot be deleted!` + (error ? ` Reason: ${error}` : ''),
-    cannotDeleteObjectFromEnums: (
-        /** @type {string} */ objectID,
-        /** @type {string?} */ error
-    ) => `The object ${objectID} could not be deleted from enums!` + (error ? ` Reason: ${error}` : ''),
-    cannotChangeObject: (
-        /** @type {string} */ objectID,
-        /** @type {string?} */ error
-    ) => `The object ${objectID} cannot be changed!` + (error ? ` Reason: ${error}` : ''),
-    objectPropertyNotFound: (
-        /** @type {string} */ objectID,
-        /** @type {string} */ propPath
-    ) => `The requested property "${propPath}" or one of its parents was not found in the object "${objectID}"!`,
+    stateNotFound: (/** @type {string} */ stateID, /** @type {string?} */ error) =>
+        `The state ${stateID} was not found!` + (error ? ` Reason: ${error}` : ''),
+    stateNotBinary: (/** @type {string} */ stateId) => `State "${stateId}" is not binary.`,
+    stateBinaryGetUnsupported: (/** @type {string} */ stateId) =>
+        `State "${stateId}" is a binary state, please use getBinary.`,
+    stateBinarySetUnsupported: (/** @type {string} */ stateId) =>
+        `State "${stateId}" is a binary state and cannot be set via cli.`,
+    objectNotFound: (/** @type {string} */ objectID, /** @type {string?} */ error) =>
+        `The object ${objectID} was not found!` + (error ? ` Reason: ${error}` : ''),
+    cannotUpdateObject: (/** @type {string} */ objectID, /** @type {string?} */ error) =>
+        `The object ${objectID} could not be updated!` + (error ? ` Reason: ${error}` : ''),
+    cannotDeleteObject: (/** @type {string} */ objectID, /** @type {string?} */ error) =>
+        `The object ${objectID} cannot be deleted!` + (error ? ` Reason: ${error}` : ''),
+    cannotDeleteObjectFromEnums: (/** @type {string} */ objectID, /** @type {string?} */ error) =>
+        `The object ${objectID} could not be deleted from enums!` + (error ? ` Reason: ${error}` : ''),
+    cannotChangeObject: (/** @type {string} */ objectID, /** @type {string?} */ error) =>
+        `The object ${objectID} cannot be changed!` + (error ? ` Reason: ${error}` : ''),
+    objectPropertyNotFound: (/** @type {string} */ objectID, /** @type {string} */ propPath) =>
+        `The requested property "${propPath}" or one of its parents was not found in the object "${objectID}"!`,
     invalidPropertyOrValue: () => `The property path or value is not valid. Please make sure the value is valid JSON.`,
     invalidJSONValue: () => `The given value is not valid JSON.`,
 
-    unknownCommand: (
-        /** @type {string} */ prefix,
-        /** @type {string} */ command
-    ) => `Unknown command "${prefix} ${command}"!`,
-    requiredArgumentMissing: (
-        /** @type {string} */ argName,
-        /** @type {string?} */ exampleCommand
-    ) => `The required argument "${argName}" is missing!` + (exampleCommand ? ` Example: "${exampleCommand}"` : ''),
+    unknownCommand: (/** @type {string} */ prefix, /** @type {string} */ command) =>
+        `Unknown command "${prefix} ${command}"!`,
+    requiredArgumentMissing: (/** @type {string} */ argName, /** @type {string?} */ exampleCommand) =>
+        `The required argument "${argName}" is missing!` + (exampleCommand ? ` Example: "${exampleCommand}"` : ''),
 
     noInstancesFound: adapter => `Cannot find any instances of "${adapter}"!`,
     invalidInstance: instance => `The instance "${instance}" does not exist!`,
-    specifyInstance: (/** @type {string} */ adapter, /** @type {Array<string>?} */adapterInstances) => `The adapter "${adapter}" has multiple instances! Please specify which one should be started: "${adapterInstances.join('", "')}".`,
+    specifyInstance: (/** @type {string} */ adapter, /** @type {Array<string>?} */ adapterInstances) =>
+        `The adapter "${adapter}" has multiple instances! Please specify which one should be started: "${adapterInstances.join(
+            '", "'
+        )}".`,
     adapterDirNotFound: adapter => `Cannot find the installation dir for adapter "${adapter}"!`,
     mainFileNotFound: adapter => `Cannot find the main file for adapter "${adapter}"!`,
     cannotLoadIoPackage: adapter => `Cannot load the io-package.json file for adapter "${adapter}"!`,
@@ -53,8 +43,10 @@ const errorMessages = Object.freeze({
         /** @type {string} */ wrongPrefix,
         /** @type {string} */ command,
         /** @type {string?} */ correctPrefix
-    ) => `The command ${command} is not intended to be used with ${wrongPrefix}!`
-    + correctPrefix ? `Please use "${correctPrefix} ${command}" instead.` : '',
+    ) =>
+        `The command ${command} is not intended to be used with ${wrongPrefix}!` + correctPrefix
+            ? `Please use "${correctPrefix} ${command}" instead.`
+            : '',
     unknown: err => `An unknown error occurred: ${err}`,
 
     cannotChangeRunningSystem: () => `Cannot execute changes on running system. Stop ${tools.appName} first.`,
@@ -63,47 +55,46 @@ const errorMessages = Object.freeze({
     hostDoesNotExist: hostname => `The host "${hostname}" does not exist!`,
     dontRemoveCurrentHost: hostname => `Cannot remove host "${hostname}" from itself!`,
 
-    pluginNotDefined: (pluginName, host, instance) => `The plugin "${pluginName}" does not exist for ${host ? `host "${host}"` : `instance "${instance}"`}!`,
+    pluginNotDefined: (pluginName, host, instance) =>
+        `The plugin "${pluginName}" does not exist for ${host ? `host "${host}"` : `instance "${instance}"`}!`,
 
-    cert: certName => `Certificate "${certName}" not found or error parsing certificate information.`
+    cert: certName => `Certificate "${certName}" not found or error parsing certificate information.`,
+    instanceAlreadyRunning: instance => `The instance "${instance}" is already running, please stop instance first.`
 });
 
 const successMessages = Object.freeze({
     objectUpdated: objectID => `The object "${objectID}" was updated successfully.`,
     objectDeleted: objectID => `The object "${objectID}" was deleted.`,
-    stateUpdated: (
-        /** @type {string} */ stateID,
-        /** @type {any} */ value,
-        /** @type {boolean | undefined} */ ack
-    ) => `The state "${stateID}" was set to "${value}"${ack === undefined ? '' : ` with flag ack=${ack}`}`,
+    stateUpdated: (/** @type {string} */ stateID, /** @type {any} */ value, /** @type {boolean | undefined} */ ack) =>
+        `The state "${stateID}" was set to "${value}"${ack === undefined ? '' : ` with flag ack=${ack}`}`,
     stateDeleted: stateID => `The state "${stateID}" was deleted.`,
     adapterStarted: adapter => `The adapter "${adapter}" was started.`,
     adapterStopped: adapter => `The adapter "${adapter}" was stopped.`,
     adapterRestarted: adapter => `The adapter "${adapter}" was restarted.`,
-    systemStatus: isRunning => isRunning ? `At least one ${tools.appName} host is running.` : `No ${tools.appName} host is running.`,
+    systemStatus: isRunning =>
+        isRunning ? `At least one ${tools.appName} host is running.` : `No ${tools.appName} host is running.`,
     controllerStatus: isRunning => `${tools.appName} is ${isRunning ? '' : 'not '}running on this host.`,
-    messageSent: (
-        /** @type {string} */ adapter,
-        /** @type {string} */ command,
-        /** @type {any} */ message
-    ) => `The command "${command}" was sent to "${adapter}" with the message "${message}".`,
-    hostRenamed: (
-        /** @type {string} */ from,
-        /** @type {string} */ to
-    ) => `Host "${from}" successfully renamed to "${to}".`,
-    instanceHostChanged: (
-        /** @type {string} */ instance,
-        /** @type {string} */ from,
-        /** @type {string} */ to
-    ) => `The host for instance "${instance}" was changed from "${from}" to "${to}".`,
+    messageSent: (/** @type {string} */ adapter, /** @type {string} */ command, /** @type {any} */ message) =>
+        `The command "${command}" was sent to "${adapter}" with the message "${message}".`,
+    hostRenamed: (/** @type {string} */ from, /** @type {string} */ to) =>
+        `Host "${from}" successfully renamed to "${to}".`,
+    instanceHostChanged: (/** @type {string} */ instance, /** @type {string} */ from, /** @type {string} */ to) =>
+        `The host for instance "${instance}" was changed from "${from}" to "${to}".`,
     hostDeleted: hostname => `The host "${hostname}" was deleted.`,
 
-    pluginEnabledOrDisabled: (pluginName, host, instance, status) => `The plugin "${pluginName}" was successfully ${status ? 'enabled' : 'disabled'} for ${host ? `host "${host}"` : `instance "${instance}"`}.`,
-    pluginStatus: (pluginName, host, instance, status) => `The plugin "${pluginName}" is ${status ? 'enabled' : 'disabled'} for ${host ? `host "${host}"` : `instance "${instance}"`}.`
+    pluginEnabledOrDisabled: (pluginName, host, instance, status) =>
+        `The plugin "${pluginName}" was successfully ${status ? 'enabled' : 'disabled'} for ${
+            host ? `host "${host}"` : `instance "${instance}"`
+        }.`,
+    pluginStatus: (pluginName, host, instance, status) =>
+        `The plugin "${pluginName}" is ${status ? 'enabled' : 'disabled'} for ${
+            host ? `host "${host}"` : `instance "${instance}"`
+        }.`
 });
 
 const warnings = Object.freeze({
-    noInstancesFoundOnHost: hostname => hostname ? `No instances found for host "${hostname}"!` : `No instances found!`
+    noInstancesFoundOnHost: hostname =>
+        hostname ? `No instances found for host "${hostname}"!` : `No instances found!`
 });
 
 // Capsule success messages in console.log