@contrail/cli 1.0.31 → 1.0.33

package/.eslintrc.js

@@ -0,0 +1,53 @@
+module.exports = {
+  env: {
+    browser: true,
+    es2021: true,
+    node: true
+
+  },
+  extends: [
+    'eslint:recommended',
+    'plugin:@typescript-eslint/eslint-recommended',
+    'plugin:@typescript-eslint/recommended'
+  ],
+  parser: '@typescript-eslint/parser',
+  parserOptions: {
+    ecmaVersion: 'latest',
+    sourceType: 'module'
+  },
+  plugins: [
+    '@typescript-eslint',
+    'unused-imports'
+  ],
+  rules: {
+    indent: [
+      'error',
+      2
+    ],
+    'linebreak-style': [
+      'error',
+      'unix'
+    ],
+    'quote-props': ['error', 'as-needed'],
+    quotes: [
+      'error',
+      'single'
+    ],
+    semi: [
+      'error',
+      'always'
+    ],
+    'no-multiple-empty-lines': ['error', { max: 1, maxEOF: 0 }],
+    'no-trailing-spaces': 'error',
+    'no-mixed-spaces-and-tabs': ['error', 'smart-tabs'],
+    '@typescript-eslint/no-empty-function': ['error', { allow: ['arrowFunctions'] }],
+    '@typescript-eslint/no-explicit-any': 'off',
+    '@typescript-eslint/no-var-requires': 'warn',
+    'no-unused-vars': 'off',
+    'unused-imports/no-unused-imports': 'error',
+    'unused-imports/no-unused-vars': [
+      'warn',
+      { vars: 'all', varsIgnorePattern: '^_', args: 'after-used', argsIgnorePattern: '^_' }
+    ]
+  }
+};

package/lib/apps/deploy-app/actions-deployer.js

@@ -21,14 +21,21 @@ export class ActionsDeployer {
         const existingActions = await getActionsForApp(app);
         // UPSERT / DELETE BASED ON ACTION IDENTIFIER
         console.log('%s \tDeploying actions.', chalk.yellow.bold('ACTION'));
+        let appActionIdentifiers = [];
         if (appDeclaration.actions) {
-            const localActions = Object.keys(appDeclaration.actions);
-            if ((localActions === null || localActions === void 0 ? void 0 : localActions.length) > 0) {
-                for (const actionKey of localActions) {
+            const localActionKeys = Object.keys(appDeclaration.actions);
+            appActionIdentifiers = localActionKeys.map(x => getActionDeclarationIdentifier(x, app));
+            if ((localActionKeys === null || localActionKeys === void 0 ? void 0 : localActionKeys.length) > 0) {
+                for (const actionKey of localActionKeys) {
                     await this.deployAction(actionKey, appDeclaration.actions[actionKey], app, existingActions);
                 }
             }
         }
+        const toDelete = existingActions.filter(x => !appActionIdentifiers.includes(x.identifier));
+        for (const action of toDelete) {
+            console.log(`%s \tDeleting action: ${action.identifier}`, chalk.yellow.bold('ACTION'));
+            await deleteAction(action.id);
+        }
     }
     async deployAction(actionKey, actionDeclaration, app, existingActions) {
         console.log(`%s \tDeploying action: ${actionKey}`, chalk.yellow.bold('ACTION'));
@@ -52,6 +59,7 @@ export class ActionsDeployer {
             ownerKey: `app:${app.id}`,
             public: 'public' in actionDeclaration ? actionDeclaration.public : app.public,
             configProperties,
+            async: actionDeclaration.async,
         };
         const functionString = await this.getFunction(actionDeclaration);
         if (functionString) {
@@ -76,6 +84,7 @@ export class ActionsDeployer {
             name: actionDeclaration.name,
             public: 'public' in actionDeclaration ? actionDeclaration.public : app.public,
             configProperties,
+            async: actionDeclaration.async,
         };
         const functionString = await this.getFunction(actionDeclaration);
         if (functionString) {

package/lib/apps/deploy-app/app-deployer.js

@@ -8,11 +8,11 @@ export class AppDeployer {
     async deploy(appDeclaration) {
         let app = await this.getExistingApp(appDeclaration);
         if (!app) {
-            console.log(`%s      Creating new app.`, chalk.yellow.bold('APP'));
+            console.log('%s      Creating new app.', chalk.yellow.bold('APP'));
             app = await this.createNewApp(appDeclaration);
         }
         else {
-            console.log(`%s      Updating existing app.`, chalk.yellow.bold('APP'));
+            console.log('%s      Updating existing app.', chalk.yellow.bold('APP'));
             app = await this.updateExistingApp(appDeclaration, app);
         }
         await new ActionsDeployer().deployActions(appDeclaration, app);
@@ -57,7 +57,6 @@ export class AppDeployer {
             keywords: appDeclaration.keywords,
             configProperties,
             triggerKeyDefinitions,
-            isDockerBased: appDeclaration.isDockerBased,
             dockerImageRef: appDeclaration.dockerImageRef,
         };
         const app = await new Entities().create({ entityName: 'app', object: newApp });

package/lib/apps/deploy-app/app-template-parsers.js

@@ -25,7 +25,7 @@ export class ConfigPropertiesParser {
             const config = {
                 label: declaration.label,
                 slug: configKey,
-                propertyType: declaration.propertyType,
+                propertyType: declaration.propertyType, // TODO should support options
             };
             if (declaration.options) {
                 config.options = declaration.options;

package/lib/apps/deploy-app/workflow-definition-deployer.js

@@ -53,7 +53,7 @@ export class WorkflowDefinitionDeployer {
     }
     async populateActionIds(workflowDefinition, app) {
         var _a;
-        let availableAppActions = await getActionsForApp(app);
+        const availableAppActions = await getActionsForApp(app);
         if (!((_a = workflowDefinition.workflowDefinition) === null || _a === void 0 ? void 0 : _a.paths)) {
             return;
         }

package/lib/auth/login.js

@@ -8,13 +8,13 @@ const CONFIGURATION = {
     clientId: process.env.CONTRAIL_COGNITO_CLIENT_ID || '2oubek96sue5rsncovtqhs3scm',
 };
 export async function authUser() {
-    console.debug("authorizing user");
+    console.debug('authorizing user');
     const config = await getConfig();
     if (config &&
         config.apiUserToken &&
         config.apiGateway &&
         config.orgSlug) {
-        console.debug("populating auth config from fs config");
+        console.debug('populating auth config from fs config');
         Auth.config = {
             apiUserToken: config.apiUserToken,
             apiGateway: config.apiGateway,
@@ -25,7 +25,7 @@ export async function authUser() {
             return await refreshCurrentUserOrg();
         }
         catch (e) {
-            console.log("Token expired. Requesting auth credentials...");
+            console.log('Token expired. Requesting auth credentials...');
         }
     }
     const questions = [];
@@ -120,11 +120,11 @@ async function setAuthContext(user, params) {
     return refreshCurrentUserOrg();
 }
 async function refreshCurrentUserOrg() {
-    console.debug("refreshing current user org");
+    console.debug('refreshing current user org');
     const currentUser = await Request.request('/users/current', {
         method: 'GET',
         headers: {
-            'Accept': 'application/json',
+            Accept: 'application/json',
             'Content-Type': 'application/json',
         },
     });

package/lib/cli.js

@@ -62,7 +62,7 @@ export async function cli(args) {
                     break;
                 }
                 case 'uploadTypes': {
-                    uploadTypes(args);
+                    uploadTypes();
                     break;
                 }
                 case 'uploadOptionSets': {
@@ -204,6 +204,7 @@ const helpSections = [
         content: [
             'load                         Load colors from a CSV. Headers (name, hex, red,',
             '.                            green, blue) are required.',
+            // 'delete                       Delete all colors.',
         ],
     },
     {

package/lib/colors/color-loader.js

@@ -7,7 +7,7 @@ export async function deleteColors() {
     try {
         await authUser();
         const colors = await getExistingColors();
-        console.log("Deleting %s colors", colors.length);
+        console.log('Deleting %s colors', colors.length);
         for (const color of colors) {
             await deleteColor(color);
         }
@@ -21,10 +21,10 @@ export async function deleteColors() {
 export async function loadColors() {
     try {
         await authUser();
-        console.log(`%s Getting Existing Colors`, chalk.yellow.bold('COLORS'));
-        console.log(`%s Uploading`, chalk.yellow.bold('COLORS'));
+        console.log('%s Getting Existing Colors', chalk.yellow.bold('COLORS'));
+        console.log('%s Uploading', chalk.yellow.bold('COLORS'));
         await uploadColors();
-        console.log(`%s Uploaded`, chalk.yellow.bold('COLORS'));
+        console.log('%s Uploaded', chalk.yellow.bold('COLORS'));
     }
     catch (e) {
         console.log(`%s Loading Colors Failed. ${e}`, chalk.red.bold('FAIL'));
@@ -33,7 +33,7 @@ export async function loadColors() {
     console.log('%s Loading Colors Complete.', chalk.green.bold('COMPLETE'));
 }
 async function getLocalColors() {
-    console.log("loading colors from local file");
+    console.log('loading colors from local file');
     const TARGET_DIR = process.cwd();
     try {
         const separator = getSeparatorByPlatform();
@@ -45,10 +45,10 @@ async function getLocalColors() {
     }
 }
 async function getExistingColors() {
-    console.log(`%s Getting Existing Colors`, chalk.yellow.bold('COLORS'));
+    console.log('%s Getting Existing Colors', chalk.yellow.bold('COLORS'));
     try {
         const result = await new Entities().get({ entityName: 'color', take: 3000 }); //, useSearch: false }); Not in EntitiesClientGetOptions yet
-        console.log("Found %s existing colors", result.length);
+        console.log('Found %s existing colors', result.length);
         return result;
     }
     catch (e) {
@@ -69,7 +69,7 @@ async function uploadColors() {
     const existingNamesValues = existingColors.filter(x => x.name).map(x => x.name);
     for (const color of colors) {
         if (existingNamesValues.includes(color.name)) {
-            console.log(`%s SKIPPING %s (%s)`, chalk.yellow.bold('COLORS'), color.hex, color.name);
+            console.log('%s SKIPPING %s (%s)', chalk.yellow.bold('COLORS'), color.hex, color.name);
             continue;
         }
         if (color.hex.length > 1 && color.hex[0] === '#') {

package/lib/config/config.js

@@ -9,7 +9,7 @@ export async function initializeVibeIQConfig() {
         await fsPromises.mkdir(configDirectoryLocation, { recursive: true });
     }
     catch (e) {
-        console.error("could not initialize vibeiq directory");
+        console.error('could not initialize vibeiq directory');
         throw e;
     }
     try {
@@ -48,7 +48,7 @@ export async function writeVibeIQConfig(config) {
     }
 }
 export async function getConfig() {
-    console.debug("getting config from file system");
+    console.debug('getting config from file system');
     return await readVibeIQConfig();
 }
 export async function deleteConfigs(configNames) {

package/lib/interfaces.d.ts

@@ -21,7 +21,6 @@ export interface AppDeclaration {
     workflows?: {
         [key: string]: WorkflowDefinition | WorkflowTemplateDeclaration;
     };
-    isDockerBased?: boolean;
     dockerImageRef?: string;
 }
 export interface ConfigPropertyElements {
@@ -38,6 +37,7 @@ export interface AppActionDeclaration {
     name: string;
     description?: string;
     actionConfiguration?: ConfigPropertyElements;
+    async?: boolean;
 }
 export interface AppTriggerDeclaration {
     key: string;
@@ -116,7 +116,7 @@ export interface EventWorkflowActionDefinition {
     ownerKey?: string;
     public?: boolean;
     configProperties?: ConfigProperty[];
-    isDockerBased?: boolean;
+    async?: boolean;
 }
 export declare class TriggerKeyDefinition {
     name: string;

package/lib/types/interfaces.js

@@ -0,0 +1 @@
+export {};

package/lib/types/option-set-merger.js

@@ -9,11 +9,11 @@ export async function MergeOptionSets(args) {
         if ((args === null || args === void 0 ? void 0 : args.length) < 4) {
             throw new Error('File name must be included.');
         }
-        console.log(`%s Importing local file`, chalk.yellow.bold('OPTION SET MERGE'));
+        console.log('%s Importing local file', chalk.yellow.bold('OPTION SET MERGE'));
         const fileLocation = args[3];
         console.log('Using File: ', fileLocation);
         await mergeOptionSets(fileLocation);
-        console.log(`%s Merged`, chalk.yellow.bold('OPTION SET MERGE'));
+        console.log('%s Merged', chalk.yellow.bold('OPTION SET MERGE'));
     }
     catch (e) {
         console.log(`%s Merge Failed. ${e}`, chalk.red.bold('FAIL'));

package/lib/types/option-set-parser.js

@@ -9,11 +9,11 @@ export async function ParseOptionSetsLocalFiles(args) {
         if ((args === null || args === void 0 ? void 0 : args.length) < 4) {
             throw new Error('File name must be included.');
         }
-        console.log(`%s Importing local file`, chalk.yellow.bold('OPTION SET PARSE'));
+        console.log('%s Importing local file', chalk.yellow.bold('OPTION SET PARSE'));
         const fileLocation = args[3];
         console.log('Using File: ', fileLocation);
         await parseOptionSetsLocalFiles(fileLocation);
-        console.log(`%s Parsed`, chalk.yellow.bold('OPTION SET PARSE'));
+        console.log('%s Parsed', chalk.yellow.bold('OPTION SET PARSE'));
     }
     catch (e) {
         console.log(`%s PARSE Failed. ${e}`, chalk.red.bold('FAIL'));

package/lib/types/option-set-uploader.js

@@ -8,9 +8,9 @@ export async function UploadOptionSets(args) {
     try {
         const userConfig = await authUser();
         console.log(`%s Getting all types for orgId: ${userConfig.orgId}`, chalk.blue.bold('START'));
-        console.log(`%s Uploading`, chalk.yellow.bold('OPTION SETS'));
+        console.log('%s Uploading', chalk.yellow.bold('OPTION SETS'));
         await uploadOptionSets();
-        console.log(`%s Uploaded`, chalk.yellow.bold('OPTION SETS'));
+        console.log('%s Uploaded', chalk.yellow.bold('OPTION SETS'));
     }
     catch (e) {
         console.log(`%s Types Upload Failed. ${e}`, chalk.red.bold('FAIL'));

package/lib/types/parse-types.js

@@ -12,11 +12,11 @@ export async function ParseTypesFromLocalFiles(args) {
         if ((args === null || args === void 0 ? void 0 : args.length) < 4) {
             throw new Error('File names must be included.');
         }
-        console.log(`%s Parsing`, chalk.yellow.bold('TYPES AND OPTION SETS'));
+        console.log('%s Parsing', chalk.yellow.bold('TYPES AND OPTION SETS'));
         const fileLocations = args.slice(3);
         console.log('Using File Location(s): ', fileLocations);
         await parseTypesFromLocalFiles(fileLocations);
-        console.log(`%s Parsed`, chalk.yellow.bold('TYPES AND OPTION SETS'));
+        console.log('%s Parsed', chalk.yellow.bold('TYPES AND OPTION SETS'));
     }
     catch (e) {
         console.log(`%s Parsing Failed. ${e}`, chalk.red.bold('FAIL'));
@@ -30,11 +30,11 @@ export async function ParseTypesFromFiles(args) {
         if ((args === null || args === void 0 ? void 0 : args.length) < 4) {
             throw new Error('File ids must be included.');
         }
-        console.log(`%s Parsing`, chalk.yellow.bold('TYPES AND OPTION SETS'));
+        console.log('%s Parsing', chalk.yellow.bold('TYPES AND OPTION SETS'));
         const fileIds = args.slice(3);
         console.log('Using FileId(s): ', fileIds);
         await parseTypesFromFiles(fileIds);
-        console.log(`%s Parsed`, chalk.yellow.bold('TYPES AND OPTION SETS'));
+        console.log('%s Parsed', chalk.yellow.bold('TYPES AND OPTION SETS'));
     }
     catch (e) {
         console.log(`%s Parsing Failed. ${e}`, chalk.red.bold('FAIL'));
@@ -48,13 +48,13 @@ export async function ParseTypesFromTemplateFromLocalFiles(args) {
         if ((args === null || args === void 0 ? void 0 : args.length) < 5) {
             throw new Error('File names must be included.');
         }
-        console.log(`%s Parsing`, chalk.yellow.bold('TYPES AND OPTION SETS'));
+        console.log('%s Parsing', chalk.yellow.bold('TYPES AND OPTION SETS'));
         const templateFileLocation = args[3];
         const dataFileLocation = args[4];
         console.log('Using Template File Location: ', templateFileLocation);
         console.log('Using Data File Location: ', dataFileLocation);
         await parseTypesFromTemplateFromLocalFiles(templateFileLocation, dataFileLocation);
-        console.log(`%s Parsed`, chalk.yellow.bold('TYPES AND OPTION SETS'));
+        console.log('%s Parsed', chalk.yellow.bold('TYPES AND OPTION SETS'));
     }
     catch (e) {
         console.log(`%s Parsing Failed. ${e}`, chalk.red.bold('FAIL'));

package/lib/types/types-extractor.js

@@ -8,19 +8,19 @@ export async function getTypes(args) {
     try {
         const userConfig = await authUser();
         console.log(`%s Getting all types for orgId: ${userConfig.orgId}`, chalk.blue.bold('START'));
-        console.log(`%s Retrieving`, chalk.yellow.bold('OPTION SETS'));
+        console.log('%s Retrieving', chalk.yellow.bold('OPTION SETS'));
         const optionSets = await new Entities().get({
             entityName: 'type-property-option-set',
         });
         await getOptionSets(optionSets);
-        console.log(`%s Retrieved`, chalk.yellow.bold('OPTION SETS'));
-        console.log(`%s Retrieving`, chalk.yellow.bold('TYPES'));
+        console.log('%s Retrieved', chalk.yellow.bold('OPTION SETS'));
+        console.log('%s Retrieving', chalk.yellow.bold('TYPES'));
         const reducedTypes = await getReducedTypes(optionSets);
         const typesHolder = {
             types: reducedTypes,
         };
         saveData(typesHolder);
-        console.log(`%s Retrieved`, chalk.yellow.bold('TYPES'));
+        console.log('%s Retrieved', chalk.yellow.bold('TYPES'));
     }
     catch (e) {
         console.log(`%s Types Retrieval Failed. ${e}`, chalk.red.bold('FAIL'));

package/lib/types/types-properties-merger.js

@@ -10,13 +10,13 @@ export async function MergeTypeProperties(args) {
         if ((args === null || args === void 0 ? void 0 : args.length) < 5) {
             throw new Error('File name must be included along with a type path.');
         }
-        console.log(`%s Importing local file`, chalk.yellow.bold('TYPE PROPERTIES MERGE'));
+        console.log('%s Importing local file', chalk.yellow.bold('TYPE PROPERTIES MERGE'));
         const fileLocation = args[3];
         console.log('Using File: ', fileLocation);
         const typePaths = args.slice(4);
         console.log('Using TypePaths: ', typePaths);
         await mergeTypeProperties(fileLocation, typePaths);
-        console.log(`%s Merged`, chalk.yellow.bold('TYPE PROPERTIES MERGE'));
+        console.log('%s Merged', chalk.yellow.bold('TYPE PROPERTIES MERGE'));
     }
     catch (e) {
         console.log(`%s Merge Failed. ${e}`, chalk.red.bold('FAIL'));

package/lib/types/types-uploader.d.ts

@@ -1 +1 @@
-export declare function uploadTypes(args: any): Promise<void>;
+export declare function uploadTypes(): Promise<void>;

package/lib/types/types-uploader.js

@@ -4,14 +4,14 @@ import fs from 'fs';
 import { authUser } from '../auth/login';
 import { getSeparatorByPlatform } from '../common';
 import * as _ from 'lodash';
-const yaml = require('js-yaml');
-export async function uploadTypes(args) {
+import yaml from 'js-yaml';
+export async function uploadTypes() {
     try {
         const userConfig = await authUser();
         console.log(`%s Getting all types for orgId: ${userConfig.orgId}`, chalk.blue.bold('START'));
-        console.log(`%s Uploading`, chalk.yellow.bold('TYPES'));
+        console.log('%s Uploading', chalk.yellow.bold('TYPES'));
         await uploadReducedTypes();
-        console.log(`%s Uploaded`, chalk.yellow.bold('TYPES'));
+        console.log('%s Uploaded', chalk.yellow.bold('TYPES'));
     }
     catch (e) {
         console.log(`%s Types Upload Failed. ${e}`, chalk.red.bold('FAIL'));
@@ -146,7 +146,7 @@ async function uploadReducedTypeProperties(type, currentPropertyTypes, newProper
                         validationFunction: newPropertyType.validationFunction,
                     }),
                     headers: {
-                        'Accept': 'application/json',
+                        Accept: 'application/json',
                         'Content-Type': 'application/json',
                     },
                 });
@@ -186,7 +186,7 @@ async function uploadReducedTypeProperties(type, currentPropertyTypes, newProper
                 await Request.request(`/types/${type.id}/type-properties/${currentPropertyType.id}`, {
                     method: 'DELETE',
                     headers: {
-                        'Accept': 'application/json',
+                        Accept: 'application/json',
                         'Content-Type': 'application/json',
                     },
                 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contrail/cli",
-  "version": "1.0.31",
+  "version": "1.0.33",
   "description": "VibeIQ's Contrail Platform CLI.",
   "main": "lib/index.js",
   "bin": {
@@ -17,14 +17,14 @@
   "scripts": {
     "build": "tsc",
     "format": "prettier --write \"src/**/*.ts\" \"src/**/*.js\"",
-    "lint": "tslint -p tsconfig.json",
+    "lint": "eslint . --fix",
     "test": "jest --watch"
   },
   "author": "",
   "license": "ISC",
   "dependencies": {
     "@contrail/sdk": "^1.0.154",
-    "@contrail/types": "^2.0.69",
+    "@contrail/types": "^2.0.70",
     "@contrail/util": "^1.0.17",
     "amazon-cognito-identity-js": "^5.2.3",
     "arg": "^4.0.1",
@@ -32,6 +32,7 @@
     "chalk": "^4.0.0",
     "command-line-usage": "^6.1.1",
     "csvtojson": "^2.0.10",
+    "eslint-plugin-unused-imports": "^2.0.0",
     "esm": "^3.2.25",
     "execa": "^5.0.0",
     "inquirer": "^8.2.0",
@@ -45,12 +46,13 @@
     "@types/jest": "^23.3.14",
     "@types/lodash": "^4.14.178",
     "@types/node": "^14.6.4",
+    "@typescript-eslint/eslint-plugin": "^5.32.0",
+    "@typescript-eslint/parser": "^5.32.0",
+    "eslint": "^8.21.0",
     "jest": "^29.3.1",
     "prettier": "^1.19.1",
     "ts-jest": "^29.0.3",
-    "tslint": "^5.11.0",
-    "tslint-config-prettier": "^1.18.0",
-    "typescript": "^3.0.1"
+    "typescript": "^4.9.3"
   },
   "jest": {
     "moduleFileExtensions": [

package/src/apps/deploy-app/actions-deployer.ts

@@ -6,7 +6,6 @@ import { getSeparatorByPlatform } from '../../common';
 
 import fs from 'fs';
 
-
 export async function getActionsForApp(app: App) {
   return await new Entities().get({
     entityName: 'event-workflow-action-definition',
@@ -14,16 +13,16 @@ export async function getActionsForApp(app: App) {
   });
 }
 
-
 export function getActionDeclarationIdentifier(actionKey, app: App) {
   return app.identifier + ':' + actionKey;
 }
 
-export async function deleteAction(actionId): Promise<any>{
+export async function deleteAction(actionId): Promise<any> {
   return await new Entities().delete({ entityName: 'event-workflow-action-definition', id: actionId });
 }
 
 export class ActionsDeployer {
+
   async deployActions(appDeclaration: AppDeclaration, app: App) {
 
     // FETCH ALL ACTIONS FROM THE API.
@@ -31,16 +30,23 @@ export class ActionsDeployer {
 
     // UPSERT / DELETE BASED ON ACTION IDENTIFIER
     console.log('%s \tDeploying actions.', chalk.yellow.bold('ACTION'));
-
+    let appActionIdentifiers = [];
     if (appDeclaration.actions) {
-      const localActions = Object.keys(appDeclaration.actions);
-      if (localActions?.length > 0) {
-        for (const actionKey of localActions) {
+      const localActionKeys = Object.keys(appDeclaration.actions);
+      appActionIdentifiers = localActionKeys.map(x => getActionDeclarationIdentifier(x, app));
+      if (localActionKeys?.length > 0) {
+        for (const actionKey of localActionKeys) {
           await this.deployAction(actionKey, appDeclaration.actions[actionKey], app, existingActions);
         }
       }
     }
+    const toDelete = existingActions.filter(x => !appActionIdentifiers.includes(x.identifier));
+    for (const action of toDelete) {
+      console.log(`%s \tDeleting action: ${action.identifier}`, chalk.yellow.bold('ACTION'));
+      await deleteAction(action.id);
+    }
   }
+
   async deployAction(actionKey: string, actionDeclaration: AppActionDeclaration, app: App, existingActions: Array<any>) {
     console.log(`%s \tDeploying action: ${actionKey}`, chalk.yellow.bold('ACTION'));
 
@@ -63,8 +69,9 @@ export class ActionsDeployer {
       identifier: getActionDeclarationIdentifier(actionKey, app),
       name: actionDeclaration.name,
       ownerKey: `app:${app.id}`,
-      public: 'public' in actionDeclaration? actionDeclaration.public: app.public,
+      public: 'public' in actionDeclaration ? actionDeclaration.public : app.public,
       configProperties,
+      async: actionDeclaration.async,
     };
 
     const functionString = await this.getFunction(actionDeclaration);
@@ -94,8 +101,9 @@ export class ActionsDeployer {
 
     const updates: EventWorkflowActionDefinition = {
       name: actionDeclaration.name,
-      public: 'public' in actionDeclaration? actionDeclaration.public: app.public,
+      public: 'public' in actionDeclaration ? actionDeclaration.public : app.public,
       configProperties,
+      async: actionDeclaration.async,
     };
 
     const functionString = await this.getFunction(actionDeclaration);
@@ -114,7 +122,6 @@ export class ActionsDeployer {
     return actionDefinition;
   }
 
-
   async getFunction(actionDeclaration: AppActionDeclaration): Promise<string> {
     if (!actionDeclaration.handler) {
       return;

package/src/apps/deploy-app/app-deployer.ts

@@ -10,10 +10,10 @@ export class AppDeployer {
   public async deploy(appDeclaration: AppDeclaration) {
     let app = await this.getExistingApp(appDeclaration);
     if (!app) {
-      console.log(`%s      Creating new app.`, chalk.yellow.bold('APP'));
+      console.log('%s      Creating new app.', chalk.yellow.bold('APP'));
       app = await this.createNewApp(appDeclaration);
     } else {
-      console.log(`%s      Updating existing app.`, chalk.yellow.bold('APP'));
+      console.log('%s      Updating existing app.', chalk.yellow.bold('APP'));
       app = await this.updateExistingApp(appDeclaration, app);
     }
     await new ActionsDeployer().deployActions(appDeclaration, app);
@@ -65,7 +65,6 @@ export class AppDeployer {
       keywords: appDeclaration.keywords,
       configProperties,
       triggerKeyDefinitions,
-      isDockerBased: appDeclaration.isDockerBased,
       dockerImageRef: appDeclaration.dockerImageRef,
     };
     const app = await new Entities().create({ entityName: 'app', object: newApp });
@@ -105,7 +104,7 @@ export class AppDeployer {
   public async remove(appDeclaration: AppDeclaration) {
     let app = await this.getExistingApp(appDeclaration);
     if (app) {
-      const actions = await getActionsForApp(app)
+      const actions = await getActionsForApp(app);
       for(const action of actions)
       {
         console.log(`%s      Deleting action: ${action.identifier}`, chalk.yellow.bold('APP'));