@madgex/fert 6.0.2 → 6.1.0

package/README.md

@@ -212,7 +212,7 @@ Run for 1 service from the service folder:
 | `--target`  | Where to publish the `dist` directory assets.<br/>`dev` \| `production` (`string`) |
 | `--dry-run` | Dry run, dont actually upload anything                                             |
 
-Send all files and directories created in the `dist` directory to either development or production versions of the Asset Store API. Files uploaded to the Asset Store API are available via a CloudFront-based CDN.
+Send all files and directories created in the `dist` directory to either development or production versions of the Asset Store API. Files uploaded to the Asset Store API are available via a CloudFront-based CDN
 
 **Outline of implementation**
 

package/bin/cli.js

@@ -47,6 +47,7 @@ const run = () => {
       'Download known configs from the Configuration API for environment "dev" or "production"',
     )
     .option('--query [configName]', 'Describe known keys for a config, omit to list all known configs')
+    .option('--skip-key-delete', 'Skip deleting keys not present in local config files on publish')
     .action((...args) => configsCommand(...args));
 
   cli

package/bin/commands/configs.js

@@ -93,6 +93,7 @@ export async function handlePublish({ workingDir, clientPropertyId }, api, optio
       workingDir,
       clientPropertyId,
       environment: options.publish,
+      skipKeyDelete: options.skipKeyDelete,
     });
   } catch (err) {
     log.debug(err);

package/bin/utils/configs.js

@@ -124,10 +124,16 @@ export async function loadLocalConfigs(workingDir) {
  * @param {string} options.workingDir - required, Working dir for configs - should usually be 'root' where fert.config.js is
  * @param {string} options.clientPropertyId - required, clientPropertyId
  * @param {string} options.environment - default=production, clientPropertyId
+ * @param {boolean} [options.skipKeyDelete] - Whether to skip deleting keys not present in local files during publish
  *
  * @returns {Promise<boolean>} - Returns a promise that resolves to true if the update is successful.
  */
-export async function updateProjectConfigs({ clientPropertyId, workingDir, environment = 'production' } = {}) {
+export async function updateProjectConfigs({
+  clientPropertyId,
+  workingDir,
+  environment = 'production',
+  skipKeyDelete = false,
+} = {}) {
   Hoek.assert(clientPropertyId, 'clientPropertyId is required');
   Hoek.assert(workingDir, 'workingDir is required');
 
@@ -149,7 +155,12 @@ export async function updateProjectConfigs({ clientPropertyId, workingDir, envir
     const { toRemove, toSet } = collateConfigs(api, localConfigs);
 
     // Will throw on first failure
-    await Promise.all([setConfigs(api, toSet), removeConfigs(api, toRemove)]);
+    if (skipKeyDelete) {
+      log.warn('Skipping deleting config keys - can no longer ensure config parity!');
+      await setConfigs(api, toSet); // only set
+    } else {
+      await Promise.all([setConfigs(api, toSet), removeConfigs(api, toRemove)]);
+    }
 
     const { host } = new URL(api.options.apiUrl);
     log.info(``);

package/bin/utils/logging.js

@@ -13,6 +13,9 @@ export const log = {
   error: (...args) => {
     console.log(chalk.red.bold('✖'), ...args);
   },
+  warn: (...args) => {
+    console.log(chalk.yellow.bold('⚠'), ...args);
+  },
   debug: (...args) => {
     debugLog(...args);
   },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@madgex/fert",
-  "version": "6.0.2",
+  "version": "6.1.0",
   "description": "Tool to help build the V6 branding",
   "bin": {
     "fert": "./bin/cli.js"