@adobe/aio-cli-plugin-app 14.3.2 → 14.4.0

package/oclif.manifest.json

@@ -18,6 +18,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "actions": {
           "description": "[default: true] Build actions if any",
           "exclusive": [
@@ -106,6 +112,12 @@
           "name": "version",
           "allowNo": false,
           "type": "boolean"
+        },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
         }
       },
       "hasDynamicHelp": false,
@@ -147,6 +159,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "import": {
           "char": "i",
           "description": "Import an Adobe I/O Developer Console configuration file",
@@ -189,6 +207,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "actions": {
           "description": "[default: true] Deploy actions if any",
           "exclusive": [
@@ -342,6 +366,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "cdn": {
           "description": "Display CDN based action URLs",
           "name": "cdn",
@@ -402,6 +432,12 @@
           "name": "version",
           "allowNo": false,
           "type": "boolean"
+        },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
         }
       },
       "hasDynamicHelp": false,
@@ -437,6 +473,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "json": {
           "char": "j",
           "description": "Output json",
@@ -516,6 +558,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "install": {
           "description": "[default: true] Run npm installation after files are created",
           "name": "install",
@@ -704,6 +752,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "allow-scripts": {
           "description": "Allow post and preinstall scripts during npm install/npm ci",
           "name": "allow-scripts",
@@ -759,6 +813,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "limit": {
           "char": "l",
           "description": "Limit number of activations to fetch logs from ( 1-50 )",
@@ -856,6 +916,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "lock-file": {
           "description": "Include the package-lock.json file in the packaged app",
           "name": "lock-file",
@@ -905,6 +971,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "serve": {
           "description": "[default: true] Start frontend server (experimental)",
           "name": "serve",
@@ -965,6 +1037,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "extension": {
           "char": "e",
           "description": "the extension(s) to test",
@@ -1039,6 +1117,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "actions": {
           "description": "[default: true] Undeploy actions if any",
           "name": "actions",
@@ -1120,6 +1204,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "overwrite": {
           "description": "Overwrite any .aio and .env files during import of the Adobe Developer Console configuration file",
           "exclusive": [
@@ -1244,6 +1334,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "install": {
           "description": "[default: true] Run npm installation after files are created",
           "name": "install",
@@ -1299,6 +1395,12 @@
           "name": "version",
           "allowNo": false,
           "type": "boolean"
+        },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
         }
       },
       "hasDynamicHelp": false,
@@ -1337,6 +1439,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "install": {
           "description": "[default: true] Run npm installation after files are created",
           "name": "install",
@@ -1396,6 +1504,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "install": {
           "description": "[default: true] Run npm installation after files are created",
           "name": "install",
@@ -1451,6 +1565,12 @@
           "name": "version",
           "allowNo": false,
           "type": "boolean"
+        },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
         }
       },
       "hasDynamicHelp": false,
@@ -1489,6 +1609,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "use-jwt": {
           "description": "if the config has both jwt and OAuth Server to Server Credentials (while migrating), prefer the JWT credentials",
           "name": "use-jwt",
@@ -1530,6 +1656,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "install": {
           "description": "[default: true] Run npm installation after files are created",
           "name": "install",
@@ -1588,6 +1720,12 @@
           "name": "version",
           "allowNo": false,
           "type": "boolean"
+        },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
         }
       },
       "hasDynamicHelp": false,
@@ -1633,6 +1771,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "yes": {
           "char": "y",
           "description": "Skip questions, and use all default values",
@@ -1675,6 +1819,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "yes": {
           "char": "y",
           "description": "Skip questions, and use all default values",
@@ -1720,6 +1870,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "yes": {
           "char": "y",
           "description": "Skip questions, and use all default values",
@@ -1775,6 +1931,12 @@
           "name": "version",
           "allowNo": false,
           "type": "boolean"
+        },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
         }
       },
       "hasDynamicHelp": false,
@@ -1813,6 +1975,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "use-jwt": {
           "description": "if the config has both jwt and OAuth Server to Server Credentials (while migrating), prefer the JWT credentials",
           "name": "use-jwt",
@@ -1854,6 +2022,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "yes": {
           "char": "y",
           "description": "Skip questions, and use all default values",
@@ -1899,6 +2073,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "json": {
           "char": "j",
           "description": "Output json",
@@ -1951,6 +2131,12 @@
           "allowNo": false,
           "type": "boolean"
         },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
+        },
         "json": {
           "char": "j",
           "description": "Output json",
@@ -1999,6 +2185,12 @@
           "name": "version",
           "allowNo": false,
           "type": "boolean"
+        },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
         }
       },
       "hasDynamicHelp": false,
@@ -2036,6 +2228,12 @@
           "name": "version",
           "allowNo": false,
           "type": "boolean"
+        },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
         }
       },
       "hasDynamicHelp": false,
@@ -2075,6 +2273,12 @@
           "name": "version",
           "allowNo": false,
           "type": "boolean"
+        },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
         }
       },
       "hasDynamicHelp": false,
@@ -2113,6 +2317,12 @@
           "name": "version",
           "allowNo": false,
           "type": "boolean"
+        },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
         }
       },
       "hasDynamicHelp": false,
@@ -2152,6 +2362,12 @@
           "name": "version",
           "allowNo": false,
           "type": "boolean"
+        },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
         }
       },
       "hasDynamicHelp": false,
@@ -2191,6 +2407,12 @@
           "name": "version",
           "allowNo": false,
           "type": "boolean"
+        },
+        "config-validation": {
+          "description": "[default: true] Validate the app configuration file(s) before continuing.",
+          "name": "config-validation",
+          "allowNo": true,
+          "type": "boolean"
         }
       },
       "hasDynamicHelp": false,
@@ -2212,5 +2434,5 @@
       ]
     }
   },
-  "version": "14.3.2"
+  "version": "14.4.0"
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@adobe/aio-cli-plugin-app",
   "description": "Create, Build and Deploy Adobe I/O Applications",
-  "version": "14.3.2",
+  "version": "14.4.0",
   "author": "Adobe Inc.",
   "bugs": "https://github.com/adobe/aio-cli-plugin-app/issues",
   "dependencies": {

package/src/BaseCommand.js

@@ -66,7 +66,7 @@ class BaseCommand extends Command {
   }
 
   async getAppExtConfigs (flags, options = {}) {
-    const all = (await this.getFullConfig(options)).all
+    const all = (await this.getFullConfig(options, flags)).all
 
     // default case: no flags, return all
     let ret = all
@@ -96,41 +96,41 @@ class BaseCommand extends Command {
     return ret
   }
 
-  async getRuntimeManifestConfigFile (implName) {
+  async getRuntimeManifestConfigFile (implName, flags) {
     let configKey
     if (implName === APPLICATION_CONFIG_KEY) {
       configKey = APPLICATION_CONFIG_KEY
     } else {
       configKey = `${EXTENSIONS_CONFIG_KEY}.${implName}`
     }
-    let configData = await this.getConfigFileForKey(`${configKey}.runtimeManifest`)
+    let configData = await this.getConfigFileForKey(`${configKey}.runtimeManifest`, flags)
     if (!configData.file) {
       // first action manifest is not defined
-      configData = await this.getConfigFileForKey(`${configKey}`)
+      configData = await this.getConfigFileForKey(`${configKey}`, flags)
       configData.key = configData.key + '.runtimeManifest'
     }
     return configData
   }
 
-  async getEventsConfigFile (implName) {
+  async getEventsConfigFile (implName, flags) {
     let configKey
     if (implName === APPLICATION_CONFIG_KEY) {
       configKey = APPLICATION_CONFIG_KEY
     } else {
       configKey = `${EXTENSIONS_CONFIG_KEY}.${implName}`
     }
-    let configData = await this.getConfigFileForKey(`${configKey}.events`)
+    let configData = await this.getConfigFileForKey(`${configKey}.events`, flags)
     if (!configData.file) {
       // first events manifest is not defined
-      configData = await this.getConfigFileForKey(`${configKey}`)
+      configData = await this.getConfigFileForKey(`${configKey}`, flags)
       configData.key = configData.key + '.events'
     }
     return configData
   }
 
-  async getConfigFileForKey (fullKey) {
+  async getConfigFileForKey (fullKey, flags = {}) {
     // NOTE: the index returns undefined if the key is loaded from a legacy configuration file
-    const fullConfig = await this.getFullConfig()
+    const fullConfig = await this.getFullConfig({}, flags)
     // full key like 'extensions.dx/excshell/1.runtimeManifest'
     // returns { key: relKey, file: configFile}
     const configData = fullConfig.includeIndex[fullKey]
@@ -142,12 +142,12 @@ class BaseCommand extends Command {
     return configData || {}
   }
 
-  async getFullConfig (options = {}) {
-    // validate appConfig defaults to false for now
-    const validateAppConfig = options.validateAppConfig === true
+  async getFullConfig (options = {}, flags = {}) {
+    // validate appConfig defaults to true unless flag is explicitly set to off
+    const validateAppConfig = flags['config-validation'] !== false
+    aioLogger.debug(`validateAppConfig=${validateAppConfig}`)
 
     if (!this.appConfig) {
-      // this will explicitly set validateAppConfig=false if not set
       this.appConfig = await appConfig.load({ ...options, validateAppConfig })
     }
     return this.appConfig
@@ -191,7 +191,12 @@ class BaseCommand extends Command {
 
 BaseCommand.flags = {
   verbose: Flags.boolean({ char: 'v', description: 'Verbose output' }),
-  version: Flags.boolean({ description: 'Show version' })
+  version: Flags.boolean({ description: 'Show version' }),
+  'config-validation': Flags.boolean({
+    description: '[default: true] Validate the app configuration file(s) before continuing.',
+    default: true,
+    allowNo: true
+  })
 }
 
 BaseCommand.args = {}

package/src/commands/app/add/action.js

@@ -32,7 +32,7 @@ class AddActionCommand extends TemplatesCommand {
     const config = entries[0][1]
 
     const actionFolder = path.relative(config.root, config.actions.src)
-    const configData = await this.getRuntimeManifestConfigFile(configName)
+    const configData = await this.getRuntimeManifestConfigFile(configName, flags)
 
     const projectOrgId = aioConfigLoader.get('project.org.id')
     if (!projectOrgId) {

package/src/commands/app/add/event.js

@@ -31,8 +31,8 @@ class AddEventCommand extends TemplatesCommand {
     const configName = entries[0][0]
     const config = entries[0][1]
     const actionFolder = path.relative(config.root, config.actions.src)
-    const runtimeManifestData = await this.getRuntimeManifestConfigFile(configName)
-    const eventsData = await this.getEventsConfigFile(configName)
+    const runtimeManifestData = await this.getRuntimeManifestConfigFile(configName, flags)
+    const eventsData = await this.getEventsConfigFile(configName, flags)
     const templateOptions = {
       'skip-prompt': false,
       'action-folder': actionFolder,

package/src/commands/app/add/extension.js

@@ -24,7 +24,7 @@ class AddExtensionCommand extends TemplatesCommand {
       this.error('--extension= must also be provided when using --yes')
     }
 
-    const fullConfig = await this.getFullConfig({ allowNoImpl: true })
+    const fullConfig = await this.getFullConfig({ allowNoImpl: true }, flags)
     const alreadyImplemented = fullConfig.implements
 
     if (flags.extension) {

package/src/commands/app/add/web-assets.js

@@ -19,7 +19,7 @@ class AddWebAssetsCommand extends TemplatesCommand {
     const { flags } = await this.parse(AddWebAssetsCommand)
     aioLogger.debug(`add web-assets with flags: ${JSON.stringify(flags)}`)
 
-    const projectName = (await this.getFullConfig()).packagejson.name
+    const projectName = (await this.getFullConfig({}, flags)).packagejson.name
     // guaranteed to have at least one, otherwise would throw in config load or in matching the ext name
     const entries = Object.entries(await this.getAppExtConfigs(flags))
     if (entries.length > 1) {

package/src/commands/app/config/get/log-forwarding/errors.js

@@ -16,8 +16,9 @@ const { setRuntimeApiHostAndAuthHandler } = require('../../../../../lib/auth-hel
 
 class ErrorsCommand extends BaseCommand {
   async run () {
+    const { flags } = await this.parse(ErrorsCommand)
     const spinner = ora()
-    const lf = await this.getLogForwarding()
+    const lf = await this.getLogForwarding(flags)
     spinner.start('Checking for errors...')
     const res = await lf.getErrors()
     const destinationMessage = res.configured_forwarder !== undefined
@@ -30,8 +31,8 @@ class ErrorsCommand extends BaseCommand {
     }
   }
 
-  async getLogForwarding () {
-    let aioConfig = (await this.getFullConfig()).aio
+  async getLogForwarding (flags) {
+    let aioConfig = (await this.getFullConfig({}, flags)).aio
     aioConfig = setRuntimeApiHostAndAuthHandler(aioConfig)
 
     const runtimeConfig = aioConfig.runtime
@@ -46,5 +47,8 @@ class ErrorsCommand extends BaseCommand {
 
 ErrorsCommand.description = 'Get log forwarding errors'
 ErrorsCommand.aliases = ['app:config:get:log-forwarding:errors', 'app:config:get:lf:errors']
+ErrorsCommand.flags = {
+  ...BaseCommand.flags
+}
 
 module.exports = ErrorsCommand

package/src/commands/app/config/get/log-forwarding.js

@@ -15,7 +15,8 @@ const { setRuntimeApiHostAndAuthHandler } = require('../../../../lib/auth-helper
 
 class LogForwardingCommand extends BaseCommand {
   async run () {
-    let aioConfig = (await this.getFullConfig()).aio
+    const { flags } = await this.parse(LogForwardingCommand)
+    let aioConfig = (await this.getFullConfig({}, flags)).aio
     aioConfig = setRuntimeApiHostAndAuthHandler(aioConfig)
     const lf = await LogForwarding.init(aioConfig)
 
@@ -50,4 +51,8 @@ class LogForwardingCommand extends BaseCommand {
 LogForwardingCommand.description = 'Get log forwarding destination configuration'
 LogForwardingCommand.aliases = ['app:config:get:log-forwarding', 'app:config:get:lf']
 
+LogForwardingCommand.flags = {
+  ...BaseCommand.flags
+}
+
 module.exports = LogForwardingCommand