@contentstack/cli-migration 0.1.1-beta.3 → 1.0.0

package/src/modules/migration.js

@@ -1,17 +1,17 @@
-'use strict'
+'use strict';
 
-const {map: _map, getCallsite, constants, safePromise} = require('../utils')
-const Listr = require('listr')
-const {waterfall} = require('async')
-const {requests} = constants
+const { map: _map, getCallsite, constants, safePromise } = require('../utils');
+const Listr = require('listr');
+const { waterfall } = require('async');
+const { requests } = constants;
 
 // Properties
-const {getMapInstance, set, get} = _map
+const { getMapInstance, set, get } = _map;
 
-const ContentType = require('./content-types')
+const ContentType = require('./content-types');
 
 // Merge all classes containing migration methods into a single class
-const _Migration = _Class => class extends _Class { }
+const _Migration = (_Class) => class extends _Class {};
 
 /**
  * Migration class
@@ -41,66 +41,65 @@ class Migration extends _Migration(ContentType) {
    * migration.addTask(task)
    */
   addTask(taskDescription) {
-    const {title, failMessage, successMessage} = taskDescription
-    let {tasks, task} = taskDescription
-    const callsite = getCallsite()
-    const mapInstance = getMapInstance()
+    const { title, failMessage, successMessage } = taskDescription;
+    let { tasks, task } = taskDescription;
+    const callsite = getCallsite();
+    const mapInstance = getMapInstance();
     // eslint-disable-next-line no-warning-comments
     // TODO: Make it better to accept only single task
-    if (tasks && !Array.isArray(tasks))
-      tasks = [tasks]
+    if (tasks && !Array.isArray(tasks)) tasks = [tasks];
     if (task && !Array.isArray(task)) {
-      tasks = [task]
+      tasks = [task];
     }
-    this.contentTypeService.base.dispatch(callsite, null, null, tasks)
-    let _requests = get(requests, mapInstance)
+    this.contentTypeService.base.dispatch(callsite, null, null, tasks);
+    let _requests = get(requests, mapInstance);
     const req = {
       title: title,
       failedTitle: failMessage || `Failed to execute task: ${title}`,
       successTitle: successMessage || `Successfully executed task: ${title}`,
       tasks,
-    }
-    _requests.push(req)
-    set(requests, mapInstance, _requests)
+    };
+    _requests.push(req);
+    set(requests, mapInstance, _requests);
   }
 
   async run() {
-    const mapInstance = getMapInstance()
-    let _requests = get(requests, mapInstance)
+    const mapInstance = getMapInstance();
+    let _requests = get(requests, mapInstance);
     // Make calls from here
-    const tasks = await this.getTasks(_requests)
-    const listr = new Listr(tasks)
-    await listr.run().catch(error => {
-      this.handleErrors(error)
+    const tasks = await this.getTasks(_requests);
+    const listr = new Listr(tasks);
+    await listr.run().catch((error) => {
+      this.handleErrors(error);
       // When the process is child, send error message to parent
-      if (process.send) process.send({errorOccurred: true})
-    })
+      if (process.send) process.send({ errorOccurred: true });
+    });
   }
 
-  async getTasks(requests) {
-    const _tasks = []
-    const results = []
-    for (let i = 0; i < requests.length; i++) {
-      let reqObj = requests[i]
-      const {title, failedTitle, successTitle, tasks} = reqObj
+  async getTasks(_requests) {
+    const _tasks = [];
+    const results = [];
+    for (let i = 0; i < _requests.length; i++) {
+      let reqObj = _requests[i];
+      const { title, failedTitle, successTitle, tasks } = reqObj;
       const task = {
         title: title,
-        task: async (ctx, task) => {
-          const [err, result] = await safePromise(waterfall(tasks))
+        task: async (ctx, _task) => {
+          const [err, result] = await safePromise(waterfall(tasks));
           if (err) {
-            ctx.error = true
-            task.title = failedTitle
-            throw err
+            ctx.error = true;
+            _task.title = failedTitle;
+            throw err;
           }
-          result && results.push(result)
-          task.title = successTitle
-          return result
+          result && results.push(result);
+          _task.title = successTitle;
+          return result;
         },
-      }
-      _tasks.push(task)
+      };
+      _tasks.push(task);
     }
-    return _tasks
+    return _tasks;
   }
 }
 
-module.exports = Migration
+module.exports = Migration;

package/src/modules/parser.js

@@ -1,92 +1,105 @@
-'use strict'
+'use strict';
 
-const Migration = require('./migration')
+const Migration = require('./migration');
 
-const {
-  CreateContentTypeValidator,
-  EditContentTypeValidator,
-  _TypeError,
-  FieldValidator,
-} = require('../validators')
+const { CreateContentTypeValidator, EditContentTypeValidator, _TypeError, FieldValidator } = require('../validators');
 // eslint-disable-next-line no-warning-comments
 // TODO: Need a better way to combine classes
-const Base = require('./base')
+const Base = require('./base');
 
-const {ActionList} = require('../actions')
+const { ActionList } = require('../actions');
 // Utils
-const {map: _map, constants, fsHelper} = require('../utils')
+const { map: _map, constants, fsHelper } = require('../utils');
 // map properties
-const {getMapInstance, get} = _map
+const { getMapInstance, get } = _map;
 // Constants
-const {actionMapper, MANAGEMENT_SDK, MANAGEMENT_TOKEN, AUTH_TOKEN, API_KEY, BRANCH, MANAGEMENT_CLIENT, SOURCE_BRANCH} = constants
+const {
+  actionMapper,
+  MANAGEMENT_SDK,
+  MANAGEMENT_TOKEN,
+  AUTH_TOKEN,
+  API_KEY,
+  BRANCH,
+  MANAGEMENT_CLIENT,
+  SOURCE_BRANCH,
+} = constants;
 
 class Parser {
   async getMigrationParser(migrationFunc) {
-    const migration = new Migration()
-    const mapInstance = getMapInstance()
-    const parseResult = {}
-    const typeErrors = []
+    const migration = new Migration();
+    const mapInstance = getMapInstance();
+    const parseResult = {};
+    let typeErrors = [];
     // migrations
     try {
-      const stackSDKInstance = get(MANAGEMENT_SDK, mapInstance)
-      const managementToken = get(MANAGEMENT_TOKEN, mapInstance)
-      const authToken = get(AUTH_TOKEN, mapInstance)
-      const apiKey = get(API_KEY, mapInstance)
-      const branch = get(BRANCH, mapInstance)
-      const managementAPIClient = get(MANAGEMENT_CLIENT, mapInstance)
-      const externalConfigPath = get("config-path", mapInstance)
-      const externalConfig = get("config", mapInstance)
-      let externalFileConfig
-      if (typeof externalConfigPath == "string") {
+      const stackSDKInstance = get(MANAGEMENT_SDK, mapInstance);
+      const managementToken = get(MANAGEMENT_TOKEN, mapInstance);
+      const authToken = get(AUTH_TOKEN, mapInstance);
+      const apiKey = get(API_KEY, mapInstance);
+      const branch = get(BRANCH, mapInstance);
+      const managementAPIClient = get(MANAGEMENT_CLIENT, mapInstance);
+      const externalConfigPath = get('config-path', mapInstance);
+      const externalConfig = get('config', mapInstance);
+      let externalFileConfig;
+      if (typeof externalConfigPath == 'string') {
         externalFileConfig = await fsHelper.readJSONFile(externalConfigPath);
       }
       const config = Object.assign({}, externalFileConfig, externalConfig);
-      await migrationFunc({migration, stackSDKInstance, managementAPIClient, managementToken, authToken, apiKey, branch, config})
+      await migrationFunc({
+        migration,
+        stackSDKInstance,
+        managementAPIClient,
+        managementToken,
+        authToken,
+        apiKey,
+        branch,
+        config,
+      });
     } catch (error) {
       if (error instanceof TypeError) {
         if (error.message.includes('is not a function')) {
-          const base = new Base()
+          const base = new Base();
           // eslint-disable-next-line
           const [, filename, line] = error.stack.match(/\/([\/\w-_\.]+\.js):(\d*):(\d*)/);
           const callsite = {
             getFileName: () => `/${filename}`,
             getLineNumber: () => line,
-          }
-          const errMsgString = error.message.split(' ')
-          const typeErrorFirstStr = errMsgString[0].split('.')
-          const typeErrorFunction = typeErrorFirstStr[typeErrorFirstStr.length - 1]
-          typeErrors.push(typeErrorFunction)
-          base.dispatch(callsite, null, {typeErrors}, 'typeError')
+          };
+          const errMsgString = error.message.split(' ');
+          const typeErrorFirstStr = errMsgString[0].split('.');
+          const typeErrorFunction = typeErrorFirstStr[typeErrorFirstStr.length - 1];
+          typeErrors.push(typeErrorFunction);
+          base.dispatch(callsite, null, { typeErrors }, 'typeError');
         }
       } else {
-        console.log(error)
+        console.log(error);
         // eslint-disable-next-line
-        const [, filename, line] = error.stack.match(/\/([\/\w-_\.]+\.js):(\d*):(\d*)/)
+        const [, filename, line] = error.stack.match(/\/([\/\w-_\.]+\.js):(\d*):(\d*)/);
         const callsite = {
           getFileName: () => `/${filename}`,
           getLineNumber: () => line,
-        }
-        const base = new Base()
-        let typeErrors = [error]
-        base.dispatch(callsite, null, {typeErrors}, 'typeError')
+        };
+        const base = new Base();
+        typeErrors = [error];
+        base.dispatch(callsite, null, { typeErrors }, 'typeError');
       }
     }
-    const actions = get(actionMapper, mapInstance)
-    const actionList = new ActionList(actions)
+    const actions = get(actionMapper, mapInstance);
+    const actionList = new ActionList(actions);
 
-    actionList.addValidators(new CreateContentTypeValidator())
-    actionList.addValidators(new FieldValidator())
-    actionList.addValidators(new _TypeError())
-    actionList.addValidators(new EditContentTypeValidator())
+    actionList.addValidators(new CreateContentTypeValidator());
+    actionList.addValidators(new FieldValidator());
+    actionList.addValidators(new _TypeError());
+    actionList.addValidators(new EditContentTypeValidator());
 
-    const hasErrors = actionList.validate()
+    const hasErrors = actionList.validate();
 
     if (hasErrors.length > 0) {
-      parseResult.hasErrors = hasErrors
-      return parseResult
+      parseResult.hasErrors = hasErrors;
+      return parseResult;
     }
-    return parseResult
+    return parseResult;
   }
 }
 
-module.exports = Parser
+module.exports = Parser;