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

package/src/commands/cm/{migration.js → stacks/migration.js}

@@ -1,33 +1,25 @@
 /* eslint-disable no-unused-expressions */
 /* eslint-disable no-warning-comments */
 /* eslint-disable camelcase */
-"use strict";
+'use strict';
 
 // Dependencies
-const Listr = require("listr");
-const { resolve, extname } = require("path");
-const { Command, flags } = require("@contentstack/cli-command");
-const { waterfall } = require("async");
-const { Parser } = require("../../modules");
-const { ActionList } = require("../../actions");
-const fs = require("fs");
-const chalk = require("chalk");
+const Listr = require('listr');
+const { resolve, extname } = require('path');
+const { Command, flags } = require('@contentstack/cli-command');
+const { waterfall } = require('async');
+const { Parser } = require('../../../modules');
+const { ActionList } = require('../../../actions');
+const fs = require('fs');
+const chalk = require('chalk');
+const { configHandler } = require('@contentstack/cli-utilities');
+const { printFlagDeprecation } = require('@contentstack/cli-utilities');
 
-const {
-  ApiError,
-  SchemaValidator,
-  MigrationError,
-  FieldValidator,
-} = require("../../validators");
+const { ApiError, SchemaValidator, MigrationError, FieldValidator } = require('../../../validators');
 
 // Utils
-const {
-  map: _map,
-  constants,
-  safePromise,
-  errorHelper,
-} = require("../../utils");
-const { success } = require("../../utils/logger");
+const { map: _map, constants, safePromise, errorHelper } = require('../../../utils');
+const { success } = require('../../../utils/logger');
 
 // Properties
 const { get, set, getMapInstance, resetMapInstance } = _map;
@@ -44,41 +36,50 @@ const {
 
 class MigrationCommand extends Command {
   static examples = [
-    "$ csdx cm:migration -A -n <migration/script/file/path> -k <api-key>",
-    "$ csdx cm:migration -A -n <migration/script/file/path> -k <api-key> -B <target branch name>",
-    "$ csdx cm:migration --config <key1>:<value1> <key2>:<value2> ... -n <migration/script/file/path>",
-    "$ csdx cm:migration --config-file <path/to/json/config/file> -n <migration/script/file/path>",
-    "$ csdx cm:migration --multi -n <migration/scripts/dir/path> ",
-    "$ csdx cm:migration -a -n <migration/script/file/path> -k <api-key>",
+    '$ csdx cm:migration --file-path <migration/script/file/path> -k <api-key>',
+    '$ csdx cm:migration --file-path <migration/script/file/path> -k <api-key> --branch <target branch name>',
+    '$ csdx cm:migration --config <key1>:<value1> <key2>:<value2> ... --file-path <migration/script/file/path>',
+    '$ csdx cm:migration --config-file <path/to/json/config/file> --file-path <migration/script/file/path>',
+    '$ csdx cm:migration --multiple --file-path <migration/scripts/dir/path> ',
+    '$ csdx cm:migration --alias --file-path <migration/script/file/path> -k <api-key>',
   ];
 
   async run() {
     // TODO: filePath validation required.
     const migrationCommandFlags = this.parse(MigrationCommand).flags;
-    const { filePath, multi, branch } = migrationCommandFlags;
-    const authtoken = migrationCommandFlags.authtoken;
-    const apiKey = migrationCommandFlags["api-key"];
-    const alias = migrationCommandFlags["management-token-alias"];
-    const config = migrationCommandFlags["config"];
+    const { branch } = migrationCommandFlags;
+    const filePath = migrationCommandFlags['file-path'] || migrationCommandFlags.filePath;
+    const multi = migrationCommandFlags.multiple || migrationCommandFlags.multi;
+    const authtoken = configHandler.get('authtoken');
+    const apiKey = migrationCommandFlags['api-key'] || migrationCommandFlags['stack-api-key'];
+    const alias = migrationCommandFlags['alias'] || migrationCommandFlags['management-token-alias'];
+    const config = migrationCommandFlags['config'];
+
+    if (!authtoken && !alias) {
+      this.log(
+        "AuthToken is not present in local drive, Hence use 'csdx auth:login' command for login or provide management token alias",
+      );
+      this.exit();
+    }
 
     if (!filePath) {
-      this.log("Please provide the migration script file path, use -n or --filePath flag");
+      this.log('Please provide the migration script file path, use --file-path flag');
       this.exit();
     }
 
     // Reset map instance
     const mapInstance = getMapInstance();
     resetMapInstance(mapInstance);
-    if (migrationCommandFlags["config-file"]) {
-      set("config-path", mapInstance, migrationCommandFlags["config-file"]);
+    if (migrationCommandFlags['config-file']) {
+      set('config-path', mapInstance, migrationCommandFlags['config-file']);
     }
 
     if (Array.isArray(config) && config.length > 0) {
       let configObj = config.reduce((a, v) => {
-        let objArr = v.split(":");
+        let objArr = v.split(':');
         return { ...a, [objArr[0]]: objArr[1] };
       }, {});
-      set("config", mapInstance, configObj);
+      set('config', mapInstance, configObj);
     }
 
     let stackSDKInstance;
@@ -104,9 +105,7 @@ class MigrationCommand extends Command {
           });
         }
       }
-    }
-
-    if (authtoken) {
+    } else if (authtoken) {
       set(AUTH_TOKEN, mapInstance, authtoken);
       set(API_KEY, mapInstance, apiKey);
       this.managementAPIClient = { authtoken: this.authToken };
@@ -162,6 +161,7 @@ class MigrationCommand extends Command {
       requests.splice(0, requests.length);
     } catch (error) {
       // errorHandler(null, null, null, error)
+      this.log(error);
     }
   }
 
@@ -170,16 +170,16 @@ class MigrationCommand extends Command {
     const resolvedMigrationPath = resolve(filePath);
     try {
       const files = fs.readdirSync(resolvedMigrationPath);
-      for (let index = 0; index < files.length; index++) {
-        const file = files[index];
-        if (extname(file) === ".js") {
+      for (const element of files) {
+        const file = element;
+        if (extname(file) === '.js') {
           success(chalk`{white Executing file:} {grey {bold ${file}}}`);
           // eslint-disable-next-line no-await-in-loop
           await this.execSingleFile(resolve(filePath, file), mapInstance);
         }
       }
     } catch (error) {
-      error(error);
+      errorHelper(error);
     }
   }
 
@@ -187,8 +187,8 @@ class MigrationCommand extends Command {
     const _tasks = [];
     const results = [];
 
-    for (let i = 0; i < requests.length; i++) {
-      let reqObj = requests[i];
+    for (const element of requests) {
+      let reqObj = element;
       const { title, failedTitle, successTitle, tasks } = reqObj;
       const task = {
         title: title,
@@ -224,48 +224,76 @@ class MigrationCommand extends Command {
   }
 }
 
-MigrationCommand.description = "Contentstack migration script.";
+MigrationCommand.description = 'Contentstack migration script.';
 
 MigrationCommand.flags = {
-  "api-key": flags.string({
-    char: "k",
-    description: "With this flag add the API key of your stack.",
-    dependsOn: ["authtoken"],
-    exclusive: ["management-token-alias"],
+  'api-key': flags.string({
+    char: 'k',
+    description: 'With this flag add the API key of your stack.',
+    dependsOn: ['authtoken'],
+    exclusive: ['alias'],
+    parse: printFlagDeprecation(['--api-key'], ['-k', '--stack-api-key']),
+    hidden: true,
+  }),
+  'stack-api-key': flags.string({
+    char: 'k',
+    description: 'With this flag add the API key of your stack.',
+    dependsOn: ['authtoken'],
+    exclusive: ['alias'],
   }),
   authtoken: flags.boolean({
-    char: "A",
+    char: 'A',
     description:
-      "Use this flag to use the auth token of the current session. After logging in CLI, an auth token is generated for each new session.",
-    dependsOn: ["api-key"],
-    exclusive: ["management-token-alias"],
+      'Use this flag to use the auth token of the current session. After logging in CLI, an auth token is generated for each new session.',
+    dependsOn: ['api-key'],
+    exclusive: ['alias'],
+    parse: printFlagDeprecation(['-A', '--authtoken']),
+    hidden: true,
+  }),
+  alias: flags.string({
+    char: 'a',
+    description: 'Use this flag to add the management token alias.',
+    exclusive: ['authtoken'],
+  }),
+  'management-token-alias': flags.string({
+    description: 'alias of the management token',
+    exclusive: ['authtoken'],
+    hidden: true,
+    parse: printFlagDeprecation(['--management-token-alias'], ['-a', '--alias']),
   }),
-  "management-token-alias": flags.string({
-    char: "a",
-    description: "Use this flag to add the management token alias.",
-    exclusive: ["authtoken"],
-  }), // Add a better description
   filePath: flags.string({
-    char: "n",
-    description:
-      "Use this flag to provide the path of the file of the migration script provided by the user.",
+    char: 'n',
+    description: 'Use this flag to provide the path of the file of the migration script provided by the user.',
+    parse: printFlagDeprecation(['-n', '--filePath'], ['--file-path']),
+    hidden: true,
+  }),
+  'file-path': flags.string({
+    description: 'Use this flag to provide the path of the file of the migration script provided by the user.',
   }),
   branch: flags.string({
-    char: "B",
-    description:
-      "Use this flag to add the branch name where you want to perform the migration.",
+    char: 'B',
+    description: 'Use this flag to add the branch name where you want to perform the migration.',
+    parse: printFlagDeprecation(['-B'], ['--branch']),
   }),
-  "config-file": flags.string({
-    description: "[optional] Path of the JSON configuration file",
+  'config-file': flags.string({
+    description: '[optional] Path of the JSON configuration file',
   }),
   config: flags.string({
-    description: "[optional] inline configuration, <key1>:<value1>",
+    description: '[optional] inline configuration, <key1>:<value1>',
     multiple: true,
   }),
   multi: flags.boolean({
-    description:
-      "This flag helps you to migrate multiple content files in a single instance.",
-  }), // Add a better description
+    description: 'This flag helps you to migrate multiple content files in a single instance.',
+    parse: printFlagDeprecation(['--multi'], ['--multiple']),
+    hidden: true,
+  }),
+  multiple: flags.boolean({
+    description: 'This flag helps you to migrate multiple content files in a single instance.',
+  }),
 };
 
+MigrationCommand.aliases = ['cm:migration'];
+
+MigrationCommand.usage = 'cm:stacks:migration [-k <value>] [-a <value>] [--file-path <value>] [--branch <value>] [--config-file <value>] [--config <value>] [--multiple]';
+
 module.exports = MigrationCommand;

package/src/config/api-config.js

@@ -1,17 +1,16 @@
 /* eslint-disable camelcase */
-"use strict";
+'use strict';
 
 const { CONTENTSTACK_API_KEY, CONTENTSTACK_AUTHTOKEN } = process.env;
-const { version } = require("../../package.json");
+const { version } = require('../../package.json');
 module.exports = {
-  hostname: "dev9-app.contentstack.com",
-  // hostname: 'stag-app.contentstack.com',
-  version: "/v3",
-  method: "GET", // Default Http method
+  hostname: 'api.contentstack.io',
+  version: '/v3',
+  method: 'GET', // Default Http method
   headers: {
-    "Content-Type": "application/json",
-    "Content-Length": null,
-    "X-User-Agent": `@contentstack-migration/v${version}`,
+    'Content-Type': 'application/json',
+    'Content-Length': null,
+    'X-User-Agent': `@contentstack-migration/v${version}`,
     authtoken: CONTENTSTACK_AUTHTOKEN,
     api_key: CONTENTSTACK_API_KEY,
     // management_token: CONTENTSTACK_MANAGEMENT_TOKEN

package/src/config/default-options.js

@@ -1,7 +1,7 @@
 /* eslint-disable camelcase */
-'use strict'
+'use strict';
 
 module.exports = {
   is_page: true,
   singleton: false,
-}
+};

package/src/config/index.js

@@ -1,7 +1,7 @@
-'use strict'
+'use strict';
 
 module.exports = {
   apiConfig: require('./api-config'),
   defaultOptions: require('./default-options'),
   masterLocale: require('./master-locale'),
-}
+};

package/src/config/master-locale.js

@@ -1,5 +1,5 @@
 /* eslint-disable camelcase */
-'use strict'
+'use strict';
 
 module.exports = {
   master_locale: {
@@ -7,4 +7,4 @@ module.exports = {
     name: 'English - United States',
     code: 'en-us',
   },
-}
+};

package/src/modules/base.js

@@ -1,12 +1,12 @@
-'use strict'
+'use strict';
 
 // Utils
-const {map: _map, constants} = require('../utils')
+const { map: _map, constants } = require('../utils');
 // Actions
-const {actionCreators} = require('../actions')
+const { actionCreators } = require('../actions');
 // Utils properties
-const {getMapInstance, get} = _map
-const {actionMapper} = constants
+const { getMapInstance, get } = _map;
+const { actionMapper } = constants;
 
 /**
  * Base class for module classes
@@ -15,9 +15,9 @@ const {actionMapper} = constants
  */
 class Base {
   constructor(id, action) {
-    this.id = id
-    this.action = action
-    this.actions = []
+    this.id = id;
+    this.action = action;
+    this.actions = [];
   }
 
   /**
@@ -26,14 +26,14 @@ class Base {
    * @returns {Base} current instance of inherited class
    */
   title(value) {
-    const mapInstance = getMapInstance()
-    const {id, action} = this
+    const mapInstance = getMapInstance();
+    const { id, action } = this;
 
-    const contentType = get(id, mapInstance)
+    const contentType = get(id, mapInstance);
 
-    contentType[action].content_type.title = value
+    contentType[action].content_type.title = value;
 
-    return this
+    return this;
   }
 
   /**
@@ -42,11 +42,11 @@ class Base {
    * @returns {Base} current instance of inherited class
    */
   description(value) {
-    const mapInstance = getMapInstance()
-    const {id, action} = this
-    const contentType = get(id, mapInstance)
-    contentType[action].content_type.description = value
-    return this
+    const mapInstance = getMapInstance();
+    const { id, action } = this;
+    const contentType = get(id, mapInstance);
+    contentType[action].content_type.description = value;
+    return this;
   }
 
   /**
@@ -55,14 +55,14 @@ class Base {
    * @returns {Base} current instance of inherited class
    */
   force(value) {
-    const mapInstance = getMapInstance()
-    const {id, action} = this
+    const mapInstance = getMapInstance();
+    const { id, action } = this;
 
-    const contentType = get(id, mapInstance)
+    const contentType = get(id, mapInstance);
 
-    contentType[action].content_type.force = value
+    contentType[action].content_type.force = value;
 
-    return this
+    return this;
   }
 
   /**
@@ -75,21 +75,21 @@ class Base {
    */
   dispatch(callsite, id, opts, method) {
     if (!id && !opts) {
-      let mapInstance = getMapInstance()
-      let actions = get(actionMapper, mapInstance) // Returns an array if empty
-      let action = actionCreators.customTasks(callsite, opts)
-      actions.push(action)
+      let mapInstance = getMapInstance();
+      let actions = get(actionMapper, mapInstance); // Returns an array if empty
+      let action = actionCreators.customTasks(callsite, opts);
+      actions.push(action);
     } else {
-      let mapInstance = getMapInstance()
-      let actions = get(actionMapper, mapInstance) // Returns an array if empty
-      let action = actionCreators.contentType[method](callsite, id, {...opts, id})
-      actions.push(action)
+      let mapInstance = getMapInstance();
+      let actions = get(actionMapper, mapInstance); // Returns an array if empty
+      let action = actionCreators.contentType[method](callsite, id, { ...opts, id });
+      actions.push(action);
     }
   }
 
   getActions() {
-    return this.actions
+    return this.actions;
   }
 }
 
-module.exports = Base
+module.exports = Base;