mcdev 3.1.1 → 4.0.0

package/lib/util/init.config.js

@@ -1,10 +1,13 @@
 'use strict';
 
+const TYPE = require('../../types/mcdev.d');
 const Cli = require('./cli');
 const File = require('./file');
+const config = require('./config');
 const Util = require('./util');
 const inquirer = require('inquirer');
-const path = require('path');
+const path = require('node:path');
+const semver = require('semver');
 
 /**
  * CLI helper class
@@ -13,8 +16,9 @@ const path = require('path');
 const Init = {
     /**
      * helper method for this.upgradeProject that upgrades project config if needed
-     * @param {Object} properties config file's json
-     * @returns {Promise<Boolean>} returns true if worked without errors
+     *
+     * @param {TYPE.Mcdevrc} properties config file's json
+     * @returns {Promise.<boolean>} returns true if worked without errors
      */
     async fixMcdevConfig(properties) {
         if (!properties) {
@@ -26,10 +30,10 @@ const Init = {
 
         const upgradeMsgs = [`Upgrading existing ${Util.configFileName}:`];
 
-        const missingFields = Util.checkProperties(properties, true);
+        const missingFields = await config.checkProperties(properties, true);
+        const defaultProps = await config.getDefaultProperties();
         if (missingFields.length) {
-            const defaultProps = Util.getDefaultProperties();
-            missingFields.forEach((fieldName) => {
+            for (const fieldName of missingFields) {
                 switch (fieldName) {
                     case 'marketList':
                         if (properties.marketBulk) {
@@ -41,18 +45,35 @@ const Init = {
                             this._updateLeaf(properties, defaultProps, fieldName);
                         }
                         break;
-                    case 'directories.dataExtension':
+                    case 'directories.docs':
+                        if (properties.directories.badKeys) {
+                            delete properties.directories.badKeys;
+                            upgradeMsgs.push(`- ✋ removed 'directories.badKeys'`);
+                        }
+                        if (properties.directories.dataExtension) {
+                            File.removeSync(properties.directories.dataExtension);
+                            delete properties.directories.dataExtension;
+                            upgradeMsgs.push(`- ✋ removed 'directories.dataExtension'`);
+                        }
+                        if (properties.directories.deltaPackage) {
+                            delete properties.directories.deltaPackage;
+                            upgradeMsgs.push(`- ✋ removed 'directories.deltaPackage'`);
+                        }
                         if (properties.directories.dataextension) {
-                            upgradeMsgs.push(
-                                `- ✔️  converted 'directories.dataextension' to '${fieldName}'`
-                            );
-                            properties.directories.dataExtension =
-                                properties.directories.dataextension;
                             delete properties.directories.dataextension;
-                        } else {
-                            upgradeMsgs.push(`- ✔️  added '${fieldName}'`);
-                            this._updateLeaf(properties, defaultProps, fieldName);
+                            upgradeMsgs.push(`- ✋ removed 'directories.dataextension'`);
+                        }
+                        if (properties.directories.roles) {
+                            delete properties.directories.roles;
+                            upgradeMsgs.push(`- ✋ removed 'directories.roles'`);
                         }
+                        if (properties.directories.users) {
+                            delete properties.directories.users;
+                            upgradeMsgs.push(`- ✋ removed 'directories.users'`);
+                        }
+
+                        this._updateLeaf(properties, defaultProps, fieldName);
+                        upgradeMsgs.push(`- ✔️  added '${fieldName}'`);
                         break;
                     case 'metaDataTypes.documentOnRetrieve':
                         if (!properties.options.documentOnRetrieve) {
@@ -90,19 +111,22 @@ const Init = {
                         break;
                     case 'version':
                         // do nothing other than ensure we re-save the config (with the new version)
-                        delete properties.catalystFullVersion;
-                        delete properties.catalystVersion;
-
                         upgradeMsgs.push(`- ✔️  version updated`);
                         break;
                     default:
                         this._updateLeaf(properties, defaultProps, fieldName);
                         upgradeMsgs.push(`- ✔️  added '${fieldName}'`);
                 }
-            });
+            }
             updateConfigNeeded = true;
         }
 
+        // ensure we document dataExtensions and automations on retrieve as they should now be in the retrieve folder
+        this._updateLeaf(properties, defaultProps, 'metaDataTypes.documentOnRetrieve');
+        upgradeMsgs.push(
+            `- ✔️  updated 'metaDataTypes.documentOnRetrieve' to include all available types`
+        );
+
         // check if metaDataTypes.retrieve is set to default values and if not, launch selectTypes
         const defaultRetrieveArr = Util.getRetrieveTypeChoices();
         let reselectDefaultRetrieve = false;
@@ -118,6 +142,18 @@ const Init = {
             updateConfigNeeded = true;
         }
 
+        // move to version 4 uses integers for MIDs
+        for (const cred in properties.credentials) {
+            properties.credentials[cred].eid = Number.parseInt(properties.credentials[cred].eid);
+            for (const bu in properties.credentials[cred].businessUnits) {
+                properties.credentials[cred].businessUnits[bu] = Number.parseInt(
+                    properties.credentials[cred].businessUnits[bu]
+                );
+            }
+            updateConfigNeeded = true;
+            upgradeMsgs.push(`- ✔️  updated Business Unit format (${cred})`);
+        }
+
         // update config
         if (updateConfigNeeded) {
             for (const msg of upgradeMsgs) {
@@ -131,16 +167,16 @@ const Init = {
                 Util.logger.warn('');
                 if (toBeAddedTypes.length) {
                     Util.logger.warn('Adding types:');
-                    toBeAddedTypes.forEach((type) => {
+                    for (const type of toBeAddedTypes) {
                         Util.logger.warn(` - ${type}`);
-                    });
+                    }
                     Util.logger.warn('');
                 }
                 if (toBeRemovedTypes.length) {
                     Util.logger.warn('Removing types:');
-                    toBeRemovedTypes.forEach((type) => {
+                    for (const type of toBeRemovedTypes) {
                         Util.logger.warn(` - ${type}`);
-                    });
+                    }
                     Util.logger.warn('');
                 }
                 await Cli.selectTypes(properties, defaultRetrieveArr);
@@ -154,21 +190,19 @@ const Init = {
 
         return true;
     },
+
     /**
      * handles creation/update of all config file from the boilerplate
-     * @returns {Promise<Boolean>} status of config file creation
+     *
+     * @param {string} versionBeforeUpgrade 'x.y.z'
+     * @returns {Promise.<boolean>} status of config file creation
      */
-    async createIdeConfigFiles() {
+    async createIdeConfigFiles(versionBeforeUpgrade) {
         Util.logger.info('Checking configuration files (existing files will not be changed):');
         const creationLog = [];
-
-        if (!File.existsSync('deploy/')) {
-            File.mkdirpSync('deploy/');
-        }
-
-        if (!File.existsSync('src/cloudPages')) {
-            File.mkdirpSync('src/cloudPages');
-        }
+        await File.ensureDir('deploy/');
+        await File.ensureDir('src/cloudPages');
+        const relevantForcedUpdates = await this._getForcedUpdateList(versionBeforeUpgrade);
 
         // copy in .gitignore (cant be retrieved via npm install directly)
         const gitignoreFileName = path.resolve(
@@ -176,32 +210,39 @@ const Init = {
             Util.boilerplateDirectory,
             'gitignore-template'
         );
-        if (!File.existsSync(gitignoreFileName)) {
+        if (!(await File.pathExists(gitignoreFileName))) {
             Util.logger.debug(`Dependency file not found in ${gitignoreFileName}`);
             return false;
         } else {
-            const fileContent = File.readFileSync(gitignoreFileName, 'utf8');
+            const fileContent = await File.readFile(gitignoreFileName, 'utf8');
             creationLog.push(
-                await this._createIdeConfigFile(['.' + path.sep, '', '.gitignore'], fileContent)
+                await this._createIdeConfigFile(
+                    ['.' + path.sep, '', '.gitignore'],
+                    relevantForcedUpdates,
+                    fileContent
+                )
             );
         }
 
         // load file list from boilerplate dir and initiate copy process
         const boilerPlateFilesPath = path.resolve(__dirname, Util.boilerplateDirectory, 'files');
-        const directories = File.readDirectoriesSync(boilerPlateFilesPath, 10, false);
-
+        // ! do not switch to readDirectories before merging the two custom methods. Their logic is different!
+        const directories = await File.readDirectoriesSync(boilerPlateFilesPath, 10, false);
         for (const subdir of directories) {
             // walk thru the root of our boilerplate-files  directory and all sub folders
             const curDir = path.join(boilerPlateFilesPath, subdir);
-            for (const file of File.readdirSync(curDir)) {
+            for (const file of await File.readdir(curDir)) {
                 // read all files in these directories
-                if (!File.lstatSync(path.join(curDir, file)).isDirectory()) {
+                if (!(await File.lstat(path.join(curDir, file))).isDirectory()) {
                     // filter entries that are actually folders
                     const fileArr = file.split('.');
                     const ext = '.' + fileArr.pop();
                     // awaiting the result here due to interactive optional overwrite
                     creationLog.push(
-                        await this._createIdeConfigFile([subdir + path.sep, fileArr.join('.'), ext])
+                        await this._createIdeConfigFile(
+                            [subdir + path.sep, fileArr.join('.'), ext],
+                            relevantForcedUpdates
+                        )
                     );
                 }
             }
@@ -220,9 +261,10 @@ const Init = {
     },
     /**
      * recursive helper for _fixMcdevConfig that adds missing settings
-     * @param {Object} propertiersCur current sub-object of project settings
-     * @param {Object} defaultPropsCur current sub-object of default settings
-     * @param {String} fieldName dot-concatenated object-path that needs adding
+     *
+     * @param {object} propertiersCur current sub-object of project settings
+     * @param {object} defaultPropsCur current sub-object of default settings
+     * @param {string} fieldName dot-concatenated object-path that needs adding
      * @returns {void}
      */
     _updateLeaf(propertiersCur, defaultPropsCur, fieldName) {
@@ -242,13 +284,45 @@ const Init = {
             propertiersCur[fieldName] = defaultPropsCur[fieldName];
         }
     },
+    /**
+     * returns list of files that need to be updated
+     *
+     * @param {string} projectVersion version found in config file of the current project
+     * @returns {Promise.<string[]>} relevant files with path that need to be updated
+     */
+    async _getForcedUpdateList(projectVersion) {
+        // list of files that absolutely need to get overwritten, no questions asked, when upgrading from a version lower than the given.
+        let forceIdeConfigUpdate;
+        const relevantForcedUpdates = [];
+        if (await File.pathExists(Util.configFileName)) {
+            forceIdeConfigUpdate = File.readJsonSync(
+                path.resolve(__dirname, Util.boilerplateDirectory, 'forcedUpdates.json')
+            );
+            // return all if no project version was found or only changes from "newer" versions otherwise
+            for (const element of forceIdeConfigUpdate) {
+                if (!projectVersion || semver.gt(element.version, projectVersion)) {
+                    relevantForcedUpdates.push(
+                        // adapt it for local file systems
+                        ...element.files.map((item) => path.normalize(item))
+                    );
+                } else {
+                    continue;
+                }
+            }
+        }
+
+        return relevantForcedUpdates;
+    },
     /**
      * handles creation/update of one config file from the boilerplate at a time
-     * @param {String[]} fileNameArr 0: path, 1: filename, 2: extension with dot
-     * @param {String} [fileContent] in case we cannot copy files 1:1 this can be used to pass in content
-     * @returns {Promise<Boolean>} install successful or error occured
+     *
+     * @param {string[]} fileNameArr 0: path, 1: filename, 2: extension with dot
+     * @param {string[]} relevantForcedUpdates if fileNameArr is in this list we require an override
+     * @param {string} [boilerplateFileContent] in case we cannot copy files 1:1 this can be used to pass in content
+     * @returns {Promise.<boolean>} install successful or error occured
      */
-    async _createIdeConfigFile(fileNameArr, fileContent) {
+    async _createIdeConfigFile(fileNameArr, relevantForcedUpdates, boilerplateFileContent) {
+        let update = false;
         const fileName = fileNameArr.join('');
         const boilerplateFileName = path.resolve(
             __dirname,
@@ -256,41 +330,93 @@ const Init = {
             'files',
             fileName
         );
-        if (File.existsSync(fileName)) {
-            Util.logger.info(`- ✋  ${fileName} already existing`);
-            const questions = [
-                {
-                    type: 'confirm',
-                    name: 'overrideFile',
-                    message: 'Would you like to override it?',
-                    default: false,
-                },
-            ];
-            const responses = await new Promise((resolve) => {
-                inquirer.prompt(questions).then((answers) => {
-                    resolve(answers);
-                });
-            });
-            if (!responses.overrideFile) {
+        boilerplateFileContent =
+            boilerplateFileContent || (await File.readFile(boilerplateFileName, 'utf8'));
+
+        if (await File.pathExists(fileName)) {
+            const existingFileContent = await File.readFile(fileName, 'utf8');
+            if (existingFileContent === boilerplateFileContent) {
+                Util.logger.info(`- ✔️  ${fileName} found. No update needed`);
                 return true;
             }
+            if (relevantForcedUpdates.includes(path.normalize(fileName))) {
+                Util.logger.info(
+                    `- ✋  ${fileName} found but an update is required. Commencing with override:`
+                );
+            } else {
+                Util.logger.info(
+                    `- ✋  ${fileName} found with differences to the new standard version. We recommend updating it.`
+                );
+                if (!Util.skipInteraction) {
+                    const responses = await inquirer.prompt([
+                        {
+                            type: 'confirm',
+                            name: 'overrideFile',
+                            message: 'Would you like to update (override) it?',
+                            default: true,
+                        },
+                    ]);
+                    if (!responses.overrideFile) {
+                        // skip override without error
+                        return true;
+                    }
+                }
+            }
+            update = true;
+
+            // ensure our update is not leading to data loss in case config files were not versioned correctly by the user
+            await File.rename(fileName, fileName + '.BAK');
         }
-        fileContent = fileContent || File.readFileSync(boilerplateFileName, 'utf8');
         const saveStatus = await File.writeToFile(
             fileNameArr[0],
             fileNameArr[1],
-            fileNameArr[2].substr(1),
-            fileContent
+            fileNameArr[2].slice(1),
+            boilerplateFileContent
         );
 
         if (saveStatus) {
-            Util.logger.info(`- ✔️  ${fileName} created`);
+            Util.logger.info(
+                `- ✔️  ${fileName} ${
+                    update
+                        ? `updated (we created a backup of the old file under ${fileName + '.BAK'})`
+                        : 'created'
+                }`
+            );
             return true;
         } else {
-            Util.logger.warn(`- ❌  ${fileName} creation failed`);
+            Util.logger.warn(`- ❌  ${fileName} ${update ? 'update' : 'creation'} failed`);
             return false;
         }
     },
+    /**
+     * helper method for this.upgradeProject that upgrades project config if needed
+     *
+     * @returns {Promise.<boolean>} returns true if worked without errors
+     */
+    async upgradeAuthFile() {
+        if (await File.pathExists(Util.authFileName)) {
+            const existingAuth = await File.readJSON(Util.authFileName);
+            // if has credentials key then is old format
+            if (existingAuth.credentials) {
+                const newAuth = {};
+                for (const cred in existingAuth.credentials) {
+                    newAuth[cred] = {
+                        client_id: existingAuth.credentials[cred].clientId,
+                        client_secret: existingAuth.credentials[cred].clientSecret,
+                        auth_url: `https://${existingAuth.credentials[cred].tenant}.auth.marketingcloudapis.com/`,
+                        account_id: Number.parseInt(existingAuth.credentials[cred].eid),
+                    };
+                }
+                await File.writeJSONToFile(
+                    './',
+                    Util.authFileName.replace(/(.json)+$/, ''),
+                    newAuth
+                );
+                Util.logger.info(`- ✔️  upgraded credential file`);
+            }
+        }
+        return true;
+    },
 };
 
 module.exports = Init;

package/lib/util/init.git.js

@@ -1,10 +1,10 @@
 'use strict';
-
+const TYPE = require('../../types/mcdev.d');
 const File = require('./file');
 const inquirer = require('inquirer');
 const Util = require('./util');
 const commandExists = require('command-exists');
-const git = require('simple-git/promise')();
+const git = require('simple-git')();
 
 /**
  * CLI helper class
@@ -13,9 +13,9 @@ const git = require('simple-git/promise')();
 const Init = {
     /**
      * check if git repo exists and otherwise create one
-     * @param {Object} [skipInteraction] signals what to insert automatically for things usually asked via wizard
-     * @param {String} skipInteraction.gitRemoteUrl URL of Git remote server
-     * @returns {Promise<{status:String, repoName:String}>} success flag
+     *
+     * @param {TYPE.skipInteraction} [skipInteraction] signals what to insert automatically for things usually asked via wizard
+     * @returns {Promise.<{status: string, repoName: string}>} success flag
      */
     async initGitRepo(skipInteraction) {
         const result = { status: null, repoName: null };
@@ -23,13 +23,13 @@ const Init = {
         if (!commandExists.sync('git')) {
             Util.logger.error('Git installation not found.');
             Util.logger.error(
-                'Please follow our tutorial on installing Git: https://go.accenture.com/mcdevdocs'
+                'Please follow our tutorial on installing Git: https://github.com/Accenture/sfmc-devtools#212-install-the-git-command-line'
             );
             result.status = 'error';
             return result;
         }
         // 3. test if in git repo
-        const gitRepoFoundInCWD = File.existsSync('.git');
+        const gitRepoFoundInCWD = await File.pathExists('.git');
         let newRepoInitialized = null;
         if (gitRepoFoundInCWD) {
             Util.logger.info(`✔️  Git repository found`);
@@ -37,7 +37,7 @@ const Init = {
         } else {
             Util.logger.warn('No Git repository found. Initializing git:');
             Util.execSync('git', ['init']);
-            if (File.existsSync('.git')) {
+            if (await File.pathExists('.git')) {
                 newRepoInitialized = true;
             } else {
                 Util.logger.error(
@@ -50,7 +50,7 @@ const Init = {
         Util.logger.info('Ensuring long file paths are not causing issues with git:');
         try {
             Util.execSync('git', ['config', '--local', 'core.longpaths', 'true']);
-        } catch (ex) {
+        } catch {
             Util.logger.warn(
                 `Updating your git config failed. We recommend running the above command manually yourself to avoid issues.`
             );
@@ -58,7 +58,7 @@ const Init = {
         Util.logger.info('Ensuring checkout (git pull) as-is and commit Unix-style line endings:');
         try {
             Util.execSync('git', ['config', '--local', 'core.autocrlf', 'input']);
-        } catch (ex) {
+        } catch {
             Util.logger.warn(
                 `Updating your git config failed. We recommend running the above command manually yourself to avoid issues.`
             );
@@ -77,7 +77,8 @@ const Init = {
     },
     /**
      * offer to push the new repo straight to the server
-     * @param {Boolean|Object} [skipInteraction] signals what to insert automatically for things usually asked via wizard
+     *
+     * @param {boolean | TYPE.skipInteraction} [skipInteraction] signals what to insert automatically for things usually asked via wizard
      * @returns {void}
      */
     async gitPush(skipInteraction) {
@@ -101,19 +102,14 @@ const Init = {
                 );
                 let responses;
                 if (!skipInteraction) {
-                    const questions = [
+                    responses = await inquirer.prompt([
                         {
                             type: 'confirm',
                             name: 'gitPush',
                             message: `Would you like to 'push' your backup to the remote Git repo?`,
                             default: true,
                         },
-                    ];
-                    responses = await new Promise((resolve) => {
-                        inquirer.prompt(questions).then((answers) => {
-                            resolve(answers);
-                        });
-                    });
+                    ]);
                 }
                 if (skipInteraction || responses.gitPush) {
                     Util.execSync('git', ['push', '-u', 'origin', 'master']);
@@ -127,34 +123,30 @@ const Init = {
     },
     /**
      * offers to add the git remote origin
-     * @param {Object} [skipInteraction] signals what to insert automatically for things usually asked via wizard
-     * @param {String} skipInteraction.gitRemoteUrl URL of Git remote server
-     * @returns {String} repo name (optionally)
+     *
+     * @param {TYPE.skipInteraction} [skipInteraction] signals what to insert automatically for things usually asked via wizard
+     * @returns {string} repo name (optionally)
      */
     async _addGitRemote(skipInteraction) {
         // #1 ask if the user wants to do it now
         let responses;
         if (!skipInteraction) {
-            const questions = [
+            responses = await inquirer.prompt([
                 {
                     type: 'confirm',
                     name: 'gitOriginKnown',
                     message: `Do you know the remote/clone URL of your Git repo (starts with ssh:// or http:// and ends on '.git')?`,
                     default: true,
                 },
-            ];
-            responses = await new Promise((resolve) => {
-                inquirer.prompt(questions).then((answers) => {
-                    resolve(answers);
-                });
-            });
+            ]);
         }
         if (skipInteraction || responses.gitOriginKnown) {
             // #2 if yes, guide the user to input the right url
+            /* eslint-disable unicorn/prefer-ternary */
             if (skipInteraction) {
                 responses = skipInteraction;
             } else {
-                const questions = [
+                responses = await inquirer.prompt([
                     {
                         type: 'input',
                         name: 'gitRemoteUrl',
@@ -173,13 +165,10 @@ const Init = {
                             }
                         },
                     },
-                ];
-                responses = await new Promise((resolve) => {
-                    inquirer.prompt(questions).then((answers) => {
-                        resolve(answers);
-                    });
-                });
+                ]);
             }
+            /* eslint-enable unicorn/prefer-ternary */
+
             responses.gitRemoteUrl = responses.gitRemoteUrl.trim();
             Util.execSync('git', ['remote', 'add', 'origin', responses.gitRemoteUrl]);
             return responses.gitRemoteUrl.split('/').pop().split('.')[0];
@@ -187,7 +176,8 @@ const Init = {
     },
     /**
      * checks global config and ask to config the user info and then store it locally
-     * @param {Object|Boolean} [skipInteraction] signals what to insert automatically for things usually asked via wizard
+     *
+     * @param {boolean | TYPE.skipInteraction} [skipInteraction] signals what to insert automatically for things usually asked via wizard
      * @returns {void}
      */
     async _updateGitConfigUser(skipInteraction) {
@@ -196,19 +186,20 @@ const Init = {
             `Please confirm your Git user name & email. It should be in the format 'FirstName LastName' and 'your.email@accenture.com'. The current (potentially wrong) values are provided as default. If correct, confirm with ENTER, otherwise please update:`
         );
         let responses;
+        /* eslint-disable unicorn/prefer-ternary */
         if (skipInteraction) {
             responses = {
                 name: gitUser['user.name'],
                 email: gitUser['user.email'],
             };
         } else {
-            const questions = [
+            responses = await inquirer.prompt([
                 {
                     type: 'input',
                     name: 'name',
                     message: 'Git user.name',
                     default: gitUser['user.name'] || null,
-                    // eslint-disable-next-line require-jsdoc
+                    // eslint-disable-next-line jsdoc/require-jsdoc
                     validate: function (value) {
                         if (
                             !value ||
@@ -226,7 +217,7 @@ const Init = {
                     name: 'email',
                     message: 'Git user.email',
                     default: gitUser['user.email'] || null,
-                    // eslint-disable-next-line require-jsdoc
+                    // eslint-disable-next-line jsdoc/require-jsdoc
                     validate: function (value) {
                         value = value.trim();
                         const regex =
@@ -237,24 +228,28 @@ const Init = {
                         return true;
                     },
                 },
-            ];
-            responses = await new Promise((resolve) => {
-                inquirer.prompt(questions).then((answers) => {
-                    resolve(answers);
-                });
-            });
+            ]);
         }
+        /* eslint-enable unicorn/prefer-ternary */
+
         if (responses.name && responses.email) {
             // name can contain spaces - wrap it in quotes
             const name = `"${responses.name.trim()}"`;
             const email = responses.email.trim();
-            Util.execSync('git', ['config', '--local', 'user.name', name]);
-            Util.execSync('git', ['config', '--local', 'user.email', email]);
+            try {
+                Util.execSync('git', ['config', '--local', 'user.name', name]);
+                Util.execSync('git', ['config', '--local', 'user.email', email]);
+            } catch (ex) {
+                // if project folder is not a git folder then using --local will lead to a fatal error
+                Util.logger.warn('- Could not update git user name and email');
+                Util.logger.debug(ex.message);
+            }
         }
     },
     /**
      * retrieves the global user.name and user.email values
-     * @returns {Promise<{'user.name': String, 'user.email': String}>} user.name and user.email
+     *
+     * @returns {Promise.<{'user.name': string, 'user.email': string}>} user.name and user.email
      */
     async _getGitConfigUser() {
         const gitConfigs = await git.listConfig();
@@ -262,14 +257,14 @@ const Init = {
         delete gitConfigs.values['.git/config'];
         const result = {};
 
-        Object.keys(gitConfigs.values).forEach((file) => {
+        for (const file of Object.keys(gitConfigs.values)) {
             if (gitConfigs.values[file]['user.name']) {
                 result['user.name'] = gitConfigs.values[file]['user.name'];
             }
             if (gitConfigs.values[file]['user.email']) {
                 result['user.email'] = gitConfigs.values[file]['user.email'];
             }
-        });
+        }
         if (!result['user.name'] || !result['user.email']) {
             return null;
         }