mcdev 7.7.0 → 7.7.1

package/.github/ISSUE_TEMPLATE/bug.yml

@@ -39,6 +39,7 @@ body:
           label: Version
           description: What version of our software are you running? (mcdev --version)
           options:
+              - 7.7.1
               - 7.7.0
               - 7.6.3
               - 7.6.2

package/boilerplate/files/eslint.config.js

@@ -1,16 +1,18 @@
 import eslintPluginPrettierRecommended from 'eslint-plugin-prettier/recommended';
+import eslintPluginUnicorn from 'eslint-plugin-unicorn';
+import globals from 'globals';
 import jsdoc from 'eslint-plugin-jsdoc';
 import js from '@eslint/js';
 import sfmcSsjs from 'eslint-config-ssjs';
-import globals from 'globals';
 
 export default [
     {
         ignores: ['deploy/**/*', 'docs/**/*', 'logs/**/*', 'node_modules/**/*', 'template/**/*']
     },
-    js.configs['recommended'],
+    js.configs.recommended,
     eslintPluginPrettierRecommended,
     jsdoc.configs['flat/recommended'],
+    eslintPluginUnicorn.configs['flat/recommended'],
     {
         plugins: { jsdoc },
         rules: {
@@ -66,7 +68,7 @@ export default [
         }
     },
     {
-        files: ['.mcdev-validations.js'],
+        files: ['lib/**.js', '.mcdev-validations.js'],
 
         languageOptions: {
             globals: {
@@ -76,8 +78,101 @@ export default [
             ecmaVersion: 2022,
             sourceType: 'module'
         },
+        settings: {
+            jsdoc: {
+                mode: 'typescript',
+
+                preferredTypes: {
+                    array: 'Array',
+                    'array.<>': '[]',
+                    'Array.<>': '[]',
+                    'array<>': '[]',
+                    'Array<>': '[]',
+                    Object: 'object',
+                    'object.<>': 'Object.<>',
+                    'object<>': 'Object.<>',
+                    'Object<>': 'Object.<>',
+                    set: 'Set',
+                    'set.<>': 'Set.<>',
+                    'set<>': 'Set.<>',
+                    'Set<>': 'Set.<>',
+                    promise: 'Promise',
+                    'promise.<>': 'Promise.<>',
+                    'promise<>': 'Promise.<>',
+                    'Promise<>': 'Promise.<>'
+                }
+            }
+        },
 
         rules: {
+            'logical-assignment-operators': ['error', 'always'],
+            'unicorn/better-regex': 'off',
+
+            'unicorn/catch-error-name': [
+                'error',
+                {
+                    name: 'ex'
+                }
+            ],
+
+            'unicorn/explicit-length-check': 'off',
+            'unicorn/no-null': 'off',
+            'unicorn/prefer-module': 'off',
+            'unicorn/prevent-abbreviations': 'off',
+            'unicorn/filename-case': 'off',
+            'unicorn/no-array-callback-reference': 'off',
+            'unicorn/no-array-reduce': 'off',
+            'unicorn/no-await-expression-member': 'off',
+            'unicorn/no-hex-escape': 'off',
+            'unicorn/no-nested-ternary': 'off',
+            'unicorn/no-static-only-class': 'off',
+            'unicorn/no-unused-properties': 'warn',
+            'unicorn/numeric-separators-style': 'off',
+            'unicorn/prefer-array-some': 'off',
+            'unicorn/prefer-set-has': 'off',
+            'unicorn/prefer-spread': 'off',
+            'unicorn/prefer-string-replace-all': 'error',
+            'arrow-body-style': ['error', 'as-needed'],
+            curly: 'error',
+
+            'jsdoc/check-line-alignment': 2,
+
+            'jsdoc/require-jsdoc': [
+                'warn',
+                {
+                    require: {
+                        FunctionDeclaration: true,
+                        MethodDefinition: true,
+                        ClassDeclaration: true,
+                        ArrowFunctionExpression: false,
+                        FunctionExpression: true
+                    }
+                }
+            ],
+
+            'jsdoc/require-param-type': 'error',
+
+            'jsdoc/tag-lines': [
+                'warn',
+                'any',
+                {
+                    startLines: 1
+                }
+            ],
+
+            'jsdoc/no-undefined-types': 'off',
+            'jsdoc/valid-types': 'off',
+
+            'spaced-comment': [
+                'warn',
+                'always',
+                {
+                    block: {
+                        exceptions: ['*'],
+                        balanced: true
+                    }
+                }
+            ],
             'no-var': 'error',
             'prefer-const': 'error',
             'prettier/prettier': 'warn'

package/boilerplate/npm-dependencies.json

@@ -4,6 +4,7 @@
     "eslint-config-ssjs",
     "eslint-plugin-jsdoc",
     "eslint-plugin-prettier",
+    "eslint-plugin-unicorn",
     "eslint",
     "globals",
     "prettier-plugin-sql",

package/lib/Builder.js

@@ -112,8 +112,8 @@ saved
      */
     static async buildTemplate(businessUnit, selectedType, keyArr, marketArr) {
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            return null;
+        if (!properties) {
+            return;
         }
         if (!Util._isValidType(selectedType)) {
             return;
@@ -198,8 +198,8 @@ saved
      */
     static async buildDefinition(businessUnit, selectedType, nameArr, marketArr) {
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            return null;
+        if (!properties) {
+            return;
         }
         if (!Util._isValidType(selectedType)) {
             return;
@@ -241,8 +241,8 @@ saved
      */
     static async buildDefinitionBulk(listName, type, nameArr) {
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            return null;
+        if (!properties) {
+            return;
         }
         try {
             Util.verifyMarketList(listName, properties);
@@ -301,6 +301,9 @@ saved
      */
     static async purgeDeployFolderList(listName) {
         const properties = await config.getProperties();
+        if (!properties) {
+            return;
+        }
         for (const businessUnit in properties.marketList[listName]) {
             if (businessUnit === 'description') {
                 // skip, it's just a metadata on this list and not a BU
@@ -320,6 +323,9 @@ saved
      */
     static async purgeDeployFolder(businessUnit) {
         const properties = await config.getProperties();
+        if (!properties) {
+            return;
+        }
         if (!Util.isValidBU(properties, businessUnit, true)) {
             throw new Error(`'${businessUnit}' does not exist.`);
         }

package/lib/Deployer.js

@@ -71,8 +71,8 @@ class Deployer {
         /** @type {Object.<string, MultiMetadataTypeMap>} */
         const buMultiMetadataTypeMap = {};
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            return null;
+        if (!properties) {
+            return;
         }
         const deployDirBak = properties.directories.deploy;
         if (Util.OPTIONS.fromRetrieve) {

package/lib/cli.js

@@ -1372,7 +1372,7 @@ yargs(hideBin(process.argv))
     .recommendCommands()
     .wrap(yargs(hideBin(process.argv)).terminalWidth())
     .epilog(
-        'Copyright 2024. Accenture. Get support at https://github.com/Accenture/sfmc-devtools/issues'
+        'Copyright 2025. Accenture. Get support at https://github.com/Accenture/sfmc-devtools/issues'
     )
     .help().argv;
 

package/lib/index.js

@@ -158,8 +158,8 @@ class Mcdev {
         Util.startLogger();
         Util.logger.info('Create Delta Package ::');
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            return null;
+        if (!properties) {
+            return;
         }
         if (argv.commitrange) {
             Util.logger.warn(
@@ -189,9 +189,10 @@ class Mcdev {
     static async selectTypes() {
         Util.startLogger();
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            return null;
+        if (!properties) {
+            return;
         }
+
         await Cli.selectTypes(properties);
     }
 
@@ -207,11 +208,11 @@ class Mcdev {
      */
     static async upgrade() {
         Util.startLogger();
-        const properties = await config.getProperties();
+        const properties = await config.getProperties(false, true);
         if (!properties) {
-            Util.logger.error('No config found. Please run mcdev init');
-            return false;
+            return;
         }
+
         if ((await InitGit.initGitRepo()).status === 'error') {
             return false;
         }
@@ -229,9 +230,7 @@ class Mcdev {
         }
         Util.OPTIONS._welcomeMessageShown = true;
 
-        const color = Util.isRunViaVSCodeExtension
-            ? { reset: '', bgWhite: '', fgBlue: '' }
-            : Util.color;
+        const color = Util.color;
         /* eslint-disable no-console */
         if (process.env['USERDNSDOMAIN'] === 'DIR.SVC.ACCENTURE.COM') {
             // Accenture internal message
@@ -270,9 +269,8 @@ class Mcdev {
         Util.startLogger();
         Util.logger.info('mcdev:: Retrieve');
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            // return null here to avoid seeing 2 error messages for the same issue
-            return null;
+        if (!properties) {
+            return;
         }
 
         // assume a list was passed in and check each entry's validity
@@ -398,9 +396,10 @@ class Mcdev {
         selectedTypesArr = structuredClone(selectedTypesArr);
 
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            return null;
+        if (!properties) {
+            return;
         }
+
         const buObject = await Cli.getCredentialObject(
             properties,
             cred === null ? null : cred + '/' + bu,
@@ -559,9 +558,10 @@ class Mcdev {
         Util.startLogger();
         Util.logger.info('mcdev:: Load BUs');
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            return null;
+        if (!properties) {
+            return;
         }
+
         const buObject = await Cli.getCredentialObject(properties, credentialsName, true);
         if (buObject !== null) {
             BuHelper.refreshBUProperties(properties, buObject.credential);
@@ -579,9 +579,10 @@ class Mcdev {
         Util.startLogger();
         Util.logger.info('mcdev:: Document');
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            return null;
+        if (!properties) {
+            return;
         }
+
         if (type && !MetadataTypeInfo[type]) {
             Util.logger.error(`:: '${type}' is not a valid metadata type`);
             return;
@@ -640,9 +641,10 @@ class Mcdev {
             }
         }
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            return null;
+        if (!properties) {
+            return;
         }
+
         const buObject = await Cli.getCredentialObject(properties, businessUnit);
         if (!buObject) {
             return;
@@ -711,9 +713,10 @@ class Mcdev {
             return;
         }
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            return null;
+        if (!properties) {
+            return;
         }
+
         const buObject = await Cli.getCredentialObject(properties, businessUnit);
         if (buObject !== null) {
             try {
@@ -760,9 +763,10 @@ class Mcdev {
         Util.startLogger();
         Util.logger.info('mcdev:: describe SOAP');
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            return null;
+        if (!properties) {
+            return;
         }
+
         const credential = Object.keys(properties.credentials)[0];
         businessUnit ||=
             credential + '/' + Object.keys(properties.credentials[credential].businessUnits)[0];
@@ -807,9 +811,10 @@ class Mcdev {
     static async badKeys(businessUnit) {
         Util.startLogger();
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            return null;
+        if (!properties) {
+            return;
         }
+
         const buObject = await Cli.getCredentialObject(properties, businessUnit);
         if (buObject !== null) {
             Util.logger.info('Gathering list of Name<>External Key mismatches (bad keys)');
@@ -884,9 +889,10 @@ class Mcdev {
             'mcdev:: [DEPRECATED] Retrieve as Template [DEPRECATED] - use "retrieve" + "buildTemplate" instead'
         );
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            return null;
+        if (!properties) {
+            return;
         }
+
         if (!Util._isValidType(selectedType)) {
             return;
         }
@@ -924,6 +930,10 @@ class Mcdev {
         }
         const initialAssetNumber = typeKeyList['asset']?.length || 0;
         const properties = await config.getProperties();
+        if (!properties) {
+            return;
+        }
+
         const buObject = await Cli.getCredentialObject(properties, businessUnit);
         Util.logger.info(
             'Searching for additional dependencies that were linked via ContentBlockByKey, ContentBlockByName and ContentBlockById'
@@ -1032,6 +1042,9 @@ class Mcdev {
         const initiallySelectedTypesArr = Object.keys(typeKeyList);
 
         const properties = await config.getProperties();
+        if (!properties) {
+            return;
+        }
         const buObject = await Cli.getCredentialObject(properties, businessUnit);
         for (const type of initiallySelectedTypesArr) {
             MetadataTypeInfo[type].properties = properties;
@@ -1138,6 +1151,9 @@ class Mcdev {
 
         // redirect templates to temporary folder when executed via build()
         const properties = await config.getProperties();
+        if (!properties) {
+            return;
+        }
         const templateDirBackup = properties.directories.template;
         properties.directories.template = '.mcdev/template/';
 
@@ -1145,7 +1161,6 @@ class Mcdev {
         await this.buildTemplate(businessUnitTemplate, typeKeyCombo, null, marketTemplate);
 
         if (typeof Util.OPTIONS.purge !== 'boolean') {
-            const properties = await config.getProperties();
             // deploy folder is in targets for definition creation
             // recommend to purge their content first
             Util.OPTIONS.purge = await confirm({
@@ -1190,6 +1205,9 @@ class Mcdev {
         Util.startLogger();
         Util.logger.info('mcdev:: Build Template from retrieved files');
         const properties = await config.getProperties();
+        if (!properties) {
+            return;
+        }
         const buObject = await Cli.getCredentialObject(properties, businessUnit);
         if (!Util.checkMarketList(marketArr, properties)) {
             return;
@@ -1317,6 +1335,9 @@ class Mcdev {
         Util.startLogger();
         Util.logger.info('mcdev:: Build Definition from Template');
         const properties = await config.getProperties();
+        if (!properties) {
+            return;
+        }
         if (!Util.checkMarketList(marketArr, properties)) {
             return;
         }
@@ -1331,7 +1352,7 @@ class Mcdev {
 
         if (Util.OPTIONS.purge) {
             const buObject = await Cli.getCredentialObject(properties, businessUnit);
-            Builder.purgeDeployFolder(buObject.credential + '/' + buObject.businessUnit);
+            await Builder.purgeDeployFolder(buObject.credential + '/' + buObject.businessUnit);
         } else {
             Util.logger.info(` ☇ skipping purge of folder`);
         }
@@ -1365,6 +1386,9 @@ class Mcdev {
         Util.logger.info('mcdev:: Build Definition from Template Bulk');
 
         const properties = await config.getProperties();
+        if (!properties) {
+            return;
+        }
         try {
             Util.verifyMarketList(listName, properties);
         } catch (ex) {
@@ -1380,7 +1404,7 @@ class Mcdev {
             return;
         }
         if (Util.OPTIONS.purge) {
-            Builder.purgeDeployFolderList(listName);
+            await Builder.purgeDeployFolderList(listName);
         } else {
             Util.logger.info(` ☇ skipping purge of folder`);
         }
@@ -1406,8 +1430,8 @@ class Mcdev {
         Util.startLogger();
         Util.logger.info('mcdev:: getFilesToCommit');
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            return null;
+        if (!properties) {
+            return;
         }
         if (!Util._isValidType(selectedType)) {
             return;
@@ -1539,6 +1563,9 @@ class Mcdev {
         this.setOptions({ referenceFrom, referenceTo });
 
         const properties = await config.getProperties();
+        if (!properties) {
+            return;
+        }
         if (!Util.isValidBU(properties, businessUnit)) {
             return;
         }
@@ -1596,6 +1623,9 @@ class Mcdev {
      */
     static async fixKeys(businessUnit, selectedTypes, keys) {
         const properties = await config.getProperties();
+        if (!properties) {
+            return;
+        }
         // make sure validation rules dont keep us from fixing the keys
         this.setOptions({ skipValidation: true });
         let reRetrieveAll = false;
@@ -1797,9 +1827,8 @@ class Mcdev {
             return resultObj;
         }
         const properties = await config.getProperties();
-        if (!(await config.checkProperties(properties))) {
-            // return null here to avoid seeing 2 error messages for the same issue
-            return resultObj;
+        if (!properties) {
+            return;
         }
         for (const selectedType of selectedTypesArr || Object.keys(selectedTypesObj)) {
             Util.logger.info(`mcdev:: ${methodName} ${selectedType}`);
@@ -1924,6 +1953,9 @@ class Mcdev {
      */
     static async #runOnBU(methodName, cred, bu, type, keyArr) {
         const properties = await config.getProperties();
+        if (!properties) {
+            return;
+        }
         const resultArr = [];
         const buObject = await Cli.getCredentialObject(
             properties,
@@ -1983,6 +2015,9 @@ class Mcdev {
      */
     static async #retrieveKeysWithLike(selectedType, buObject) {
         const properties = await config.getProperties();
+        if (!properties) {
+            return;
+        }
 
         // cache depenencies
         const deployOrder = Util.getMetadataHierachy([selectedType]);
@@ -2050,6 +2085,9 @@ class Mcdev {
      */
     static async #fixKeys(cred, bu, type, keyArr) {
         const properties = await config.getProperties();
+        if (!properties) {
+            return;
+        }
         let actuallyFixedKeys = [];
         const resultArr = [];
 
@@ -2133,6 +2171,9 @@ class Mcdev {
      */
     static async #replaceCbReference(cred, bu, type, keyArr) {
         const properties = await config.getProperties();
+        if (!properties) {
+            return;
+        }
         let updatedKeys = [];
         const resultArr = [];
 

package/lib/metadataTypes/DataExtension.js

@@ -402,7 +402,7 @@ class DataExtension extends MetadataType {
         }
 
         // find all shared data extensions
-        if (!this.deployedSharedKeys.length) {
+        if (!this.deployedSharedKeys?.length) {
             Util.logger.debug(
                 `Skipping fixShared logic because no Shared Data Extensions were updated`
             );

package/lib/metadataTypes/Journey.js

@@ -290,25 +290,42 @@ class Journey extends MetadataType {
                 );
                 // break;
             }
-            default: {
-                // Quicksend, Transactional dont have versions
-                const response = await super.deleteByKeyREST(
+            case 'Quicksend': {
+                // Quicksend doesnt have versions
+                const isDeleted = await super.deleteByKeyREST(
+                    '/interaction/v1/interactions/' + id,
+                    key,
+                    false
+                );
+                return isDeleted;
+            }
+            case 'Transactional': {
+                //  Transactional dont have versions
+                const isDeleted = await super.deleteByKeyREST(
                     '/interaction/v1/interactions/' + id,
                     key,
                     false
                 );
-                if (response && cachedJourney.definitionType === 'Transactional') {
+                const transactionalEmailKey =
+                    cachedJourney.activities[0]?.configurationArguments?.triggeredSendKey;
+                if (isDeleted) {
                     const msg = [];
                     if (cachedJourney.activities[0]?.configurationArguments?.triggeredSendKey) {
                         msg.push(
                             `transactionalEmail "${cachedJourney.activities[0].configurationArguments.triggeredSendKey}"`
                         );
                     }
-                    if (msg.length) {
-                        Util.logger.info(` - Remember to also delete linked ${msg.join(' and ')}`);
-                    }
                 }
-                return response;
+                if (isDeleted && transactionalEmailKey) {
+                    Util.logger.info(
+                        ` - deleted ${TransactionalEmail.definition.type}: ${transactionalEmailKey} (SFMC auto-deletes the related transactionalEmail of ${this.definition.type} ${key})`
+                    );
+                    TransactionalEmail.buObject = this.buObject;
+                    TransactionalEmail.properties = this.properties;
+                    TransactionalEmail.client = this.client;
+                    TransactionalEmail.postDeleteTasks(transactionalEmailKey);
+                }
+                return isDeleted;
             }
         }
     }
@@ -416,6 +433,10 @@ class Journey extends MetadataType {
      */
     static async postRetrieveTasks(metadata) {
         // folder
+        if (metadata.r__folder_Path && Util.OPTIONS.publish) {
+            // if we re-retrieve this as part of deploy with --publish then the cached version will already have been processed
+            return metadata;
+        }
         super.setFolderPath(metadata);
 
         switch (metadata.definitionType) {
@@ -2288,6 +2309,7 @@ class Journey extends MetadataType {
                             toBeRetrievedTypes.map((item) => item + 's').join(', ')
                     );
                     await retriever.retrieve(toBeRetrievedTypes, eventTransEmailCombo);
+                    // TODO find r__automation_key in events and retrieve these automations as well
                 }
             }
         } catch (ex) {

package/lib/metadataTypes/MetadataType.js

@@ -2857,17 +2857,17 @@ class MetadataType {
      * Gets executed before deploying metadata
      *
      * @param {'retrieve'|'buildDefinition'|'deploy'} method used to select the right config
-     * @param {MetadataTypeItem | CodeExtractItem} item a single metadata item
+     * @param {MetadataTypeItem | CodeExtractItem} originalItem a single metadata item
      * @param {string} targetDir folder where files for deployment are stored
      * @returns {Promise.<MetadataTypeItem | CodeExtractItem>} Promise of a single metadata item
      */
-    static async validation(method, item, targetDir) {
+    static async validation(method, originalItem, targetDir) {
         if (!this.properties.options?.validation?.[method]) {
-            return item;
+            return originalItem;
         }
         // if the fix parameter was set, allow validation rules to override the metadata
-        const originalItem = item;
-        item = Util.OPTIONS.fix ? item : structuredClone(item);
+        /** @type {MetadataTypeItem | CodeExtractItem} */
+        let item = Util.OPTIONS.fix ? originalItem : structuredClone(originalItem);
         /** @type {MetadataTypeItem} */
         const metadataItem = item.json && item.codeArr ? item.json : item;
         const codeArr = item.codeArr || null;
@@ -2902,36 +2902,43 @@ class MetadataType {
 
         // run validation rules
         const warnings = [];
+        let fixed = false;
         for (const rule of Object.keys(validationRules)) {
             if (
                 validationConfig[rule] &&
                 validationConfig[rule] !== 'off' &&
                 !this.definition.skipValidation?.[rule]
             ) {
-                const passed = await validationRules[rule].passed();
+                const mode = Util.OPTIONS.fix ? 'fix' : validationConfig[rule];
+                const passed = await validationRules[rule].passed(mode);
+                if (mode === 'fix') {
+                    // potentially fixed really
+                    fixed = true;
+                }
                 if (!passed) {
-                    if (!Util.OPTIONS.skipValidation && validationConfig[rule] === 'error') {
-                        if (!Util.OPTIONS.fix || passed === false) {
-                            throw new Error(validationRules[rule].failedMsg);
-                        } else if (Util.OPTIONS.fix && passed === null) {
-                            // ensure this item no longer exists outside of the validation rule
-                            item = null;
-                            warnings.push(
-                                ` ☇ filtered item via validation rule --fix: ` +
-                                    validationRules[rule].failedMsg
-                            );
-                        }
-                    } else if (Util.OPTIONS.skipValidation || validationConfig[rule] === 'warn') {
-                        if (!Util.OPTIONS.fix || passed === false) {
+                    if (mode === 'fix') {
+                        if (passed === false) {
                             warnings.push(validationRules[rule].failedMsg);
-                        } else if (Util.OPTIONS.fix && passed === null) {
+                        } else if (passed === null) {
                             // ensure this item no longer exists outside of the validation rule
                             item = null;
                             warnings.push(
-                                ` ☇ filtered item via validation rule --fix: ` +
+                                ` ☇ filtered item via validation rule fix: ` +
                                     validationRules[rule].failedMsg
                             );
+                            break;
                         }
+                    } else if (
+                        !Util.OPTIONS.skipValidation &&
+                        validationConfig[rule] === 'error' &&
+                        passed === false
+                    ) {
+                        throw new Error(validationRules[rule].failedMsg);
+                    } else if (
+                        (Util.OPTIONS.skipValidation || validationConfig[rule] === 'warn') &&
+                        passed === false
+                    ) {
+                        warnings.push(validationRules[rule].failedMsg);
                     }
                 }
             }
@@ -2943,7 +2950,7 @@ class MetadataType {
             );
         }
         // only return "fixed" item if --fix is set, otherwise return the original item
-        return Util.OPTIONS.fix ? item : originalItem;
+        return Util.OPTIONS.fix || fixed ? item : originalItem;
     }
 }
 

package/lib/metadataTypes/definitions/Journey.definition.js

@@ -990,6 +990,12 @@ export default {
             retrieving: true,
             template: true,
         },
+        'triggers[].metaData.scheduleState': {
+            isCreateable: true,
+            isUpdateable: true,
+            retrieving: true,
+            template: true,
+        },
         version: {
             isCreateable: false,
             isUpdateable: true,

package/lib/util/config.js

@@ -46,6 +46,9 @@ const config = {
         if (await File.pathExists(Util.configFileName)) {
             try {
                 config.properties = await File.readJSON(Util.configFileName);
+                if (!isInit && !(await this.checkProperties(config.properties, silent))) {
+                    return;
+                }
             } catch (ex) {
                 Util.logger.error(`${ex.code}: ${ex.message}`);
                 return;
@@ -94,6 +97,9 @@ const config = {
                 );
                 return;
             }
+        } else if (!silent && !isInit) {
+            Util.logger.error(`Could not find ${Util.configFileName} in ${process.cwd()}.`);
+            Util.logger.error(`Run 'mcdev init' to initialize your project.\n`);
         }
         return config.properties;
     },

package/lib/util/devops.js

@@ -450,7 +450,7 @@ const DevOps = {
             // if --purge was defined and there is more than one source-target mapping, execute the purge up front to avoid deleting the package that was created by a prior mapping in this same run
             for (const sourceMlName of sourceMarketListArr) {
                 /** @type {string} */
-                Builder.purgeDeployFolderList(sourceTargetMapping[sourceMlName]);
+                await Builder.purgeDeployFolderList(sourceTargetMapping[sourceMlName]);
             }
             Util.OPTIONS.purge = false;
         }

package/lib/util/file.js

@@ -196,7 +196,7 @@ const File = {
         let formatted;
         const properties = await config.getProperties();
         if (
-            (properties.options.formatOnSave && Util.OPTIONS.format === undefined) ||
+            (properties?.options?.formatOnSave && Util.OPTIONS.format === undefined) ||
             Util.OPTIONS.format
         ) {
             formatted =
@@ -239,7 +239,7 @@ const File = {
     beautify_beautyAmp: async function (content, formatHTML = true) {
         const properties = await config.getProperties();
         if (
-            (properties.options.formatOnSave && Util.OPTIONS.format === undefined) ||
+            (properties?.options?.formatOnSave && Util.OPTIONS.format === undefined) ||
             Util.OPTIONS.format
         ) {
             // logs trough console only for the moment.
@@ -338,7 +338,7 @@ const File = {
 
             formatted = await prettier.format(content, FileFs.prettierConfig);
         } catch (ex) {
-            if (properties.options.formatErrorLog) {
+            if (properties?.options?.formatErrorLog) {
                 // save prettier errror into log file
                 // Note: we have to filter color codes from prettier's error message before saving it to file
                 /* eslint-disable no-control-regex */

package/lib/util/util.js

@@ -39,9 +39,6 @@ const __dirname = path.dirname(fileURLToPath(import.meta.url));
  * Util that contains logger and simple util methods
  */
 export const Util = {
-    isRunViaVSCodeExtension:
-        process.env.VSCODE_AMD_ENTRYPOINT === 'vs/workbench/api/node/extensionHostProcess' || // run via VSCode extension
-        process.env.VSCODE_CRASH_REPORTER_PROCESS_TYPE === 'extensionHost',
     authFileName: '.mcdev-auth.json',
     boilerplateDirectory: '../../boilerplate',
     configFileName: '.mcdevrc.json',
@@ -359,8 +356,8 @@ export const Util = {
         //   debug: 5,
         //   silly: 6
         // }
+
         if (
-            this.isRunViaVSCodeExtension || // run via VSCode extension
             process.env.FORK_PROCESS_ID || // run via Git-Fork
             process.env.PATH.toLowerCase().includes('sourcetree') // run via Atlassian SourceTree
         ) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "mcdev",
-    "version": "7.7.0",
+    "version": "7.7.1",
     "description": "Accenture Salesforce Marketing Cloud DevTools",
     "author": "Accenture: joern.berkefeld, douglas.midgley, robert.zimmermann, maciej.barnas",
     "license": "MIT",

package/test/mockRoot/.mcdevrc.json

@@ -155,5 +155,5 @@
             "verification"
         ]
     },
-    "version": "7.7.0"
+    "version": "7.7.1"
 }

package/test/resourceFactory.js

@@ -12,30 +12,7 @@ const projectRoot = projectRootHelper.join(path.sep) + path.sep;
 const parser = new XMLParser();
 const attributeParser = new XMLParser({ ignoreAttributes: false });
 /** @type {typeof Util.color} */
-let color;
-
-/* eslint-disable unicorn/prefer-ternary */
-if (Util.isRunViaVSCodeExtension) {
-    // when we execute the test in a VSCode extension host, we don't want CLI color codes.
-    // @ts-expect-error hacky way to get rid of colors - ts doesn't appreciate the hack
-    color = new Proxy(
-        {},
-        {
-            /**
-             * catch-all for color
-             *
-             * @returns {string} empty string
-             */
-            get() {
-                return '';
-            },
-        }
-    );
-} else {
-    // test is executed directly in a command prompt. Use colors.
-    color = Util.color;
-}
-/* eslint-enable unicorn/prefer-ternary */
+const color = Util.color;
 
 export const tWarn = `${color.bgYellow}${color.fgBlack}TEST-WARNING${color.reset}`;
 export const tError = `${color.bgRed}${color.fgBlack}TEST-ERROR${color.reset}`;