@knocklabs/cli 0.1.0-rc.4 → 0.1.3

package/dist/commands/translation/get.js

@@ -0,0 +1,128 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", {
+    value: true
+});
+Object.defineProperty(exports, "default", {
+    enumerable: true,
+    get: ()=>TranslationGet
+});
+const _core = require("@oclif/core");
+const _baseCommand = /*#__PURE__*/ _interopRequireDefault(require("../../lib/base-command"));
+const _date = require("../../lib/helpers/date");
+const _request = require("../../lib/helpers/request");
+const _translation = /*#__PURE__*/ _interopRequireWildcard(require("../../lib/marshal/translation"));
+function _interopRequireDefault(obj) {
+    return obj && obj.__esModule ? obj : {
+        default: obj
+    };
+}
+function _getRequireWildcardCache(nodeInterop) {
+    if (typeof WeakMap !== "function") return null;
+    var cacheBabelInterop = new WeakMap();
+    var cacheNodeInterop = new WeakMap();
+    return (_getRequireWildcardCache = function(nodeInterop) {
+        return nodeInterop ? cacheNodeInterop : cacheBabelInterop;
+    })(nodeInterop);
+}
+function _interopRequireWildcard(obj, nodeInterop) {
+    if (!nodeInterop && obj && obj.__esModule) {
+        return obj;
+    }
+    if (obj === null || typeof obj !== "object" && typeof obj !== "function") {
+        return {
+            default: obj
+        };
+    }
+    var cache = _getRequireWildcardCache(nodeInterop);
+    if (cache && cache.has(obj)) {
+        return cache.get(obj);
+    }
+    var newObj = {};
+    var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor;
+    for(var key in obj){
+        if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) {
+            var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null;
+            if (desc && (desc.get || desc.set)) {
+                Object.defineProperty(newObj, key, desc);
+            } else {
+                newObj[key] = obj[key];
+            }
+        }
+    }
+    newObj.default = obj;
+    if (cache) {
+        cache.set(obj, newObj);
+    }
+    return newObj;
+}
+class TranslationGet extends _baseCommand.default {
+    async run() {
+        const { args , flags  } = this.props;
+        const parsedRef = _translation.parseTranslationRef(args.translationRef);
+        if (!parsedRef) {
+            return this.error(`Invalid translation ref \`${args.translationRef}\`, use valid <locale> or <namespace>.<locale> for namespaced translations`);
+        }
+        const resp = await (0, _request.withSpinner)(()=>this.apiV1.getTranslation(this.props, parsedRef));
+        if (flags.json) return resp.data;
+        this.render(resp.data);
+    }
+    render(translation) {
+        const { translationRef  } = this.props.args;
+        const { environment: env , "hide-uncommitted-changes": commitedOnly  } = this.props.flags;
+        const qualifier = env === "development" && !commitedOnly ? "(including uncommitted)" : "";
+        this.log(`‣ Showing translation \`${translationRef}\` in \`${env}\` environment ${qualifier}\n`);
+        /*
+     * Translation table
+     */ const rows = [
+            {
+                key: "Language",
+                value: _translation.formatLanguage(translation)
+            },
+            {
+                key: "Locale",
+                value: translation.locale_code
+            },
+            {
+                key: "Namespace",
+                value: translation.namespace || "-"
+            },
+            {
+                key: "Updated at",
+                value: (0, _date.formatDate)(translation.updated_at)
+            },
+            {
+                key: "Created at",
+                value: (0, _date.formatDate)(translation.created_at)
+            }
+        ];
+        _core.ux.table(rows, {
+            key: {
+                header: "Translation",
+                minWidth: 24
+            },
+            value: {
+                header: "",
+                minWidth: 16
+            }
+        });
+        this.log("");
+        _core.ux.styledJSON(JSON.parse(translation.content));
+    }
+}
+TranslationGet.summary = "Display a single translation from an environment.";
+TranslationGet.flags = {
+    environment: _core.Flags.string({
+        default: "development",
+        summary: "The environment to use."
+    }),
+    "hide-uncommitted-changes": _core.Flags.boolean({
+        summary: "Hide any uncommitted changes."
+    })
+};
+TranslationGet.args = {
+    translationRef: _core.Args.string({
+        description: _translation.translationRefDescription,
+        required: true
+    })
+};
+TranslationGet.enableJsonFlag = true;

package/dist/commands/translation/list.js

@@ -7,17 +7,56 @@ Object.defineProperty(exports, "default", {
     get: ()=>TranslationList
 });
 const _core = require("@oclif/core");
-const _localeCodes = /*#__PURE__*/ _interopRequireDefault(require("locale-codes"));
 const _baseCommand = /*#__PURE__*/ _interopRequireDefault(require("../../lib/base-command"));
 const _date = require("../../lib/helpers/date");
 const _object = require("../../lib/helpers/object");
 const _page = require("../../lib/helpers/page");
 const _request = require("../../lib/helpers/request");
+const _translation = /*#__PURE__*/ _interopRequireWildcard(require("../../lib/marshal/translation"));
 function _interopRequireDefault(obj) {
     return obj && obj.__esModule ? obj : {
         default: obj
     };
 }
+function _getRequireWildcardCache(nodeInterop) {
+    if (typeof WeakMap !== "function") return null;
+    var cacheBabelInterop = new WeakMap();
+    var cacheNodeInterop = new WeakMap();
+    return (_getRequireWildcardCache = function(nodeInterop) {
+        return nodeInterop ? cacheNodeInterop : cacheBabelInterop;
+    })(nodeInterop);
+}
+function _interopRequireWildcard(obj, nodeInterop) {
+    if (!nodeInterop && obj && obj.__esModule) {
+        return obj;
+    }
+    if (obj === null || typeof obj !== "object" && typeof obj !== "function") {
+        return {
+            default: obj
+        };
+    }
+    var cache = _getRequireWildcardCache(nodeInterop);
+    if (cache && cache.has(obj)) {
+        return cache.get(obj);
+    }
+    var newObj = {};
+    var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor;
+    for(var key in obj){
+        if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) {
+            var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null;
+            if (desc && (desc.get || desc.set)) {
+                Object.defineProperty(newObj, key, desc);
+            } else {
+                newObj[key] = obj[key];
+            }
+        }
+    }
+    newObj.default = obj;
+    if (cache) {
+        cache.set(obj, newObj);
+    }
+    return newObj;
+}
 class TranslationList extends _baseCommand.default {
     async run() {
         const resp = await this.request();
@@ -40,16 +79,17 @@ class TranslationList extends _baseCommand.default {
         this.log(`‣ Showing ${entries.length} translations in \`${env}\` environment ${qualifier}\n`);
         /*
      * Translations list table
-     */ _core.CliUx.ux.table(entries, {
+     */ _core.ux.table(entries, {
+            ref: {
+                header: "Ref",
+                get: (entry)=>_translation.formatRef(entry)
+            },
             language_name: {
                 header: "Language",
-                get: (entry)=>{
-                    const language = _localeCodes.default.getByTag(entry.locale_code);
-                    return language.location ? `${language.name}, ${language.location}` : language.name;
-                }
+                get: (entry)=>_translation.formatLanguage(entry)
             },
             locale_code: {
-                header: "Locale code"
+                header: "Locale"
             },
             namespace: {
                 header: "Namespace"
@@ -57,6 +97,10 @@ class TranslationList extends _baseCommand.default {
             updated_at: {
                 header: "Updated at",
                 get: (entry)=>(0, _date.formatDate)(entry.updated_at)
+            },
+            created_at: {
+                header: "Created at",
+                get: (entry)=>(0, _date.formatDate)(entry.created_at)
             }
         });
         return this.prompt(data);
@@ -72,11 +116,15 @@ class TranslationList extends _baseCommand.default {
         }
     }
 }
+TranslationList.summary = "Display all translations for an environment.";
 TranslationList.flags = {
     environment: _core.Flags.string({
-        default: "development"
+        default: "development",
+        summary: "The environment to use."
+    }),
+    "hide-uncommitted-changes": _core.Flags.boolean({
+        summary: "Hide any uncommitted changes."
     }),
-    "hide-uncommitted-changes": _core.Flags.boolean(),
     ..._page.pageFlags
 };
 TranslationList.enableJsonFlag = true;

package/dist/commands/translation/pull.js

@@ -61,40 +61,76 @@ function _interopRequireWildcard(obj, nodeInterop) {
 }
 class TranslationPull extends _baseCommand.default {
     async run() {
+        const target = await _translation.ensureValidCommandTarget(this.props, this.runContext);
+        switch(target.type){
+            case "translationFile":
+                return this.pullOneTranslation(target.context);
+            case "translationDir":
+                return this.pullAllTranslationsForLocale(target.context);
+            case "translationsIndexDir":
+                return this.pullAllTranslations(target.context);
+            default:
+                throw new Error(`Invalid translation command target: ${target}`);
+        }
+    }
+    /*
+   * Pull a single translation (using TranslationFileContext)
+   */ async pullOneTranslation(targetCtx) {
         const { flags  } = this.props;
-        // TODO MKD: Enable pulling a single translation or group of translations for locale
-        return flags.all ? this.pullAllTranslations() : this.error("Must use --all to pull all translations");
+        if (targetCtx.exists) {
+            this.log(`‣ Found \`${targetCtx.ref}\` at ${targetCtx.abspath}`);
+        } else {
+            const prompt = `Create a new translation file \`${targetCtx.ref}\` at ${targetCtx.abspath}?`;
+            const input = flags.force || await (0, _ux.promptToConfirm)(prompt);
+            if (!input) return;
+        }
+        const resp = await (0, _request.withSpinner)(()=>this.apiV1.getTranslation(this.props, targetCtx));
+        await _translation.writeTranslationFile(targetCtx, resp.data);
+        const actioned = targetCtx.exists ? "updated" : "created";
+        this.log(`‣ Successfully ${actioned} \`${targetCtx.ref}\` at ${targetCtx.abspath}`);
     }
     /*
-   * Pull all translations
-   */ async pullAllTranslations() {
+   * Pull all translations for a locale (using TranslationDirContext)
+   */ async pullAllTranslationsForLocale(targetCtx) {
         const { flags  } = this.props;
-        // TODO: In the future we should default to the knock project config first
-        // if present, before defaulting to the cwd.
-        const defaultToCwd = {
-            abspath: this.runContext.cwd,
-            exists: true
+        const prompt = targetCtx.exists ? `Pull latest \`${targetCtx.key}\` translations into ${targetCtx.abspath}?\n  This will overwrite the contents of this directory.` : `Create a new \`${targetCtx.key}\` translations directory at ${targetCtx.abspath}?`;
+        const input = flags.force || await (0, _ux.promptToConfirm)(prompt);
+        if (!input) return;
+        // Fetch all translations for a given locale then write them to the local
+        // file system.
+        _ux.spinner.start(`‣ Loading`);
+        const filters = {
+            localeCode: targetCtx.key
         };
-        const targetDirCtx = flags["translations-dir"] || defaultToCwd;
-        const prompt = targetDirCtx.exists ? `Pull latest translations into ${targetDirCtx.abspath}?\n  This will overwrite the contents of this directory.` : `Create a new translations directory at ${targetDirCtx.abspath}?`;
+        const translations = await this.listAllTranslations(filters);
+        await _translation.writeTranslationFiles(targetCtx, translations);
+        _ux.spinner.stop();
+        const actioned = targetCtx.exists ? "updated" : "created";
+        this.log(`‣ Successfully ${actioned} the \`${targetCtx.key}\` translations directory at ${targetCtx.abspath}`);
+    }
+    /*
+   * Pull all translations (using DirContext)
+   */ async pullAllTranslations(targetCtx) {
+        const { flags  } = this.props;
+        const prompt = targetCtx.exists ? `Pull latest translations into ${targetCtx.abspath}?\n  This will overwrite the contents of this directory.` : `Create a new translations directory at ${targetCtx.abspath}?`;
         const input = flags.force || await (0, _ux.promptToConfirm)(prompt);
         if (!input) return;
         // Fetch all translations then write them to the local file system.
         _ux.spinner.start(`‣ Loading`);
         const translations = await this.listAllTranslations();
-        await _translation.writeTranslationsIndexDir(targetDirCtx, translations);
+        await _translation.writeTranslationFiles(targetCtx, translations);
         _ux.spinner.stop();
-        const action = targetDirCtx.exists ? "updated" : "created";
-        this.log(`‣ Successfully ${action} the translations directory at ${targetDirCtx.abspath}`);
+        const action = targetCtx.exists ? "updated" : "created";
+        this.log(`‣ Successfully ${action} the translations directory at ${targetCtx.abspath}`);
     }
-    async listAllTranslations(pageParams = {}, translationsFetchedSoFar = []) {
+    async listAllTranslations(filters = {}, pageParams = {}, translationsFetchedSoFar = []) {
         const props = (0, _object.merge)(this.props, {
             flags: {
                 ...pageParams,
                 limit: _page.MAX_PAGINATION_LIMIT
             }
         });
-        const resp = await this.apiV1.listTranslations(props);
+        const resp = await this.apiV1.listTranslations(props, filters);
         if (!(0, _request.isSuccessResp)(resp)) {
             const message = (0, _request.formatErrorRespMessage)(resp);
             this.error(new _error.ApiError(message));
@@ -104,21 +140,36 @@ class TranslationPull extends _baseCommand.default {
             ...translationsFetchedSoFar,
             ...entries
         ];
-        return pageInfo.after ? this.listAllTranslations({
+        return pageInfo.after ? this.listAllTranslations(filters, {
             after: pageInfo.after
         }, translations) : translations;
     }
 }
+TranslationPull.summary = "Pull one or more translations from an environment into a local file system.";
 TranslationPull.flags = {
     environment: _core.Flags.string({
-        default: "development"
+        default: "development",
+        summary: "The environment to use."
+    }),
+    all: _core.Flags.boolean({
+        summary: "Whether to pull all translations from the specified environment."
     }),
-    all: _core.Flags.boolean(),
     "translations-dir": _flag.dirPath({
+        summary: "The target directory path to pull all translations into.",
         dependsOn: [
             "all"
         ]
     }),
-    "hide-uncommitted-changes": _core.Flags.boolean(),
-    force: _core.Flags.boolean()
+    "hide-uncommitted-changes": _core.Flags.boolean({
+        summary: "Hide any uncommitted changes."
+    }),
+    force: _core.Flags.boolean({
+        summary: "Remove the confirmation prompt."
+    })
+};
+TranslationPull.args = {
+    translationRef: _core.Args.string({
+        description: _translation.translationRefDescription,
+        required: false
+    })
 };

package/dist/commands/translation/push.js

@@ -62,9 +62,10 @@ function _interopRequireWildcard(obj, nodeInterop) {
 }
 class TranslationPush extends _baseCommand.default {
     async run() {
-        // First read all translation files found for the given command.
+        const { flags  } = this.props;
+        // 1. First read all translation files found for the given command.
         const target = await _translation.ensureValidCommandTarget(this.props, this.runContext);
-        const [translations, readErrors] = await _translation.readTranslationFilesForCommandTarget(target);
+        const [translations, readErrors] = await _translation.readAllForCommandTarget(target);
         if (readErrors.length > 0) {
             this.error((0, _error.formatErrors)(readErrors, {
                 prependBy: "\n\n"
@@ -73,13 +74,17 @@ class TranslationPush extends _baseCommand.default {
         if (translations.length === 0) {
             this.error(`No translation files found in ${target.context.abspath}`);
         }
-        // Then validate them all ahead of pushing them.
+        // 2. Then validate them all ahead of pushing them.
         _ux.spinner.start(`‣ Validating`);
         const apiErrors = await _validate.default.validateAll(this.apiV1, this.props, translations);
         if (apiErrors.length > 0) {
-            this.error((0, _error.formatErrors)(apiErrors));
+            this.error((0, _error.formatErrors)(apiErrors, {
+                prependBy: "\n\n"
+            }));
         }
-        // Finally push up each translation file, abort on the first error.
+        _ux.spinner.stop();
+        // 3. Finally push up each translation file, abort on the first error.
+        _ux.spinner.start(`‣ Pushing`);
         for (const translation of translations){
             // eslint-disable-next-line no-await-in-loop
             const resp = await this.apiV1.upsertTranslation(this.props, {
@@ -93,10 +98,13 @@ class TranslationPush extends _baseCommand.default {
             this.error((0, _error.formatError)(error));
         }
         _ux.spinner.stop();
+        // 4. Display a success message.
         const handledRefs = translations.map((t)=>t.ref);
-        this.log(`‣ Successfully pushed ${translations.length} translation(s):\n` + (0, _string.indentString)(handledRefs.join("\n"), 4));
+        const actioned = flags.commit ? "pushed and committed" : "pushed";
+        this.log(`‣ Successfully ${actioned} ${translations.length} translation(s):\n` + (0, _string.indentString)(handledRefs.join("\n"), 4));
     }
 }
+TranslationPush.summary = "Push one or more translations from a local file system to Knock.";
 TranslationPush.flags = {
     environment: _core.Flags.string({
         summary: "Pushing a translation is only allowed in the development environment",
@@ -105,8 +113,11 @@ TranslationPush.flags = {
             _const.KnockEnv.Development
         ]
     }),
-    all: _core.Flags.boolean(),
+    all: _core.Flags.boolean({
+        summary: "Whether to push all translations from the target directory."
+    }),
     "translations-dir": _flag.dirPath({
+        summary: "The target directory path to find all translations to push.",
         dependsOn: [
             "all"
         ]
@@ -122,9 +133,9 @@ TranslationPush.flags = {
         ]
     })
 };
-TranslationPush.args = [
-    {
-        name: "translationRef",
+TranslationPush.args = {
+    translationRef: _core.Args.string({
+        description: _translation.translationRefDescription,
         required: false
-    }
-];
+    })
+};

package/dist/commands/translation/validate.js

@@ -62,7 +62,7 @@ function _interopRequireWildcard(obj, nodeInterop) {
 class TranslationValidate extends _baseCommand.default {
     async run() {
         const target = await _translation.ensureValidCommandTarget(this.props, this.runContext);
-        const [translations, readErrors] = await _translation.readTranslationFilesForCommandTarget(target);
+        const [translations, readErrors] = await _translation.readAllForCommandTarget(target);
         if (readErrors.length > 0) {
             this.error((0, _error.formatErrors)(readErrors, {
                 prependBy: "\n\n"
@@ -99,24 +99,28 @@ class TranslationValidate extends _baseCommand.default {
         return errors;
     }
 }
+TranslationValidate.summary = "Validate one or more translations from a local file system.";
 TranslationValidate.flags = {
     environment: _core.Flags.string({
-        summary: "Validating a workflow is only done in the development environment",
+        summary: "Validating a translation is only done in the development environment",
         default: _const.KnockEnv.Development,
         options: [
             _const.KnockEnv.Development
         ]
     }),
-    all: _core.Flags.boolean(),
+    all: _core.Flags.boolean({
+        summary: "Whether to validate all translations from the target directory."
+    }),
     "translations-dir": _flag.dirPath({
+        summary: "The target directory path to find all translations to validate.",
         dependsOn: [
             "all"
         ]
     })
 };
-TranslationValidate.args = [
-    {
-        name: "translationRef",
+TranslationValidate.args = {
+    translationRef: _core.Args.string({
+        description: _translation.translationRefDescription,
         required: false
-    }
-];
+    })
+};

package/dist/commands/whoami.js

@@ -0,0 +1,31 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", {
+    value: true
+});
+Object.defineProperty(exports, "default", {
+    enumerable: true,
+    get: ()=>Whoami
+});
+const _baseCommand = /*#__PURE__*/ _interopRequireDefault(require("../lib/base-command"));
+const _request = require("../lib/helpers/request");
+const _string = require("../lib/helpers/string");
+function _interopRequireDefault(obj) {
+    return obj && obj.__esModule ? obj : {
+        default: obj
+    };
+}
+class Whoami extends _baseCommand.default {
+    async run() {
+        const resp = await (0, _request.withSpinner)(()=>this.apiV1.whoami());
+        const { flags  } = this.props;
+        if (flags.json) return resp.data;
+        this.log(`‣ Successfully verified the provided service token:`);
+        const info = [
+            `Account name: ${resp.data.account_name}`,
+            `Service token name: ${resp.data.service_token_name}`
+        ];
+        this.log((0, _string.indentString)(info.join("\n"), 4));
+    }
+}
+Whoami.summary = "Verify the provided service token.";
+Whoami.enableJsonFlag = true;

package/dist/commands/workflow/activate.js

@@ -35,20 +35,31 @@ class WorkflowActivate extends _baseCommand.default {
         this.log(`‣ Successfully ${actioned} \`${args.workflowKey}\` workflow in \`${flags.environment}\` environment`);
     }
 }
+WorkflowActivate.summary = "Activate or deactivate a workflow in a given environment.";
+WorkflowActivate.description = `
+This immediately enables or disables a workflow in a given environment without
+needing to go through environment promotion.
+
+By default, this command activates a given workflow. Pass in the --status flag
+with \`false\` in order to deactivate it.
+`.trim();
 WorkflowActivate.flags = {
     // Do not default to any env for this command, since this action runs
     // directly in each environment outside the commit and promote flow.
     environment: _core.Flags.string({
-        required: true
+        required: true,
+        summary: "The environment to use."
     }),
     status: (0, _flag.booleanStr)({
-        default: true
+        default: true,
+        summary: "The workflow active status to set."
     }),
-    force: _core.Flags.boolean()
+    force: _core.Flags.boolean({
+        summary: "Remove the confirmation prompt."
+    })
 };
-WorkflowActivate.args = [
-    {
-        name: "workflowKey",
+WorkflowActivate.args = {
+    workflowKey: _core.Args.string({
         required: true
-    }
-];
+    })
+};

package/dist/commands/workflow/get.js

@@ -102,7 +102,7 @@ class WorkflowGet extends _baseCommand.default {
                 value: (0, _date.formatDateTime)(workflow.updated_at)
             }
         ];
-        _core.CliUx.ux.table(rows, {
+        _core.ux.table(rows, {
             key: {
                 header: "Workflow",
                 minWidth: 24
@@ -113,8 +113,9 @@ class WorkflowGet extends _baseCommand.default {
             }
         });
         this.log("");
+        // Leading space is there intentionally to align the left padding.
         if (workflow.steps.length === 0) {
-            return _core.CliUx.ux.log(" This workflow has no steps to display.");
+            return _core.ux.log(" This workflow has no steps to display.");
         }
         /*
      * Workflow steps table
@@ -122,7 +123,7 @@ class WorkflowGet extends _baseCommand.default {
                 ...step,
                 index
             }));
-        _core.CliUx.ux.table(steps, {
+        _core.ux.table(steps, {
             index: {
                 header: "Steps",
                 get: (step)=>step.index + 1
@@ -146,16 +147,19 @@ class WorkflowGet extends _baseCommand.default {
         });
     }
 }
+WorkflowGet.summary = "Display a single workflow from an environment.";
 WorkflowGet.flags = {
     environment: _core.Flags.string({
-        default: "development"
+        default: "development",
+        summary: "The environment to use."
     }),
-    "hide-uncommitted-changes": _core.Flags.boolean()
+    "hide-uncommitted-changes": _core.Flags.boolean({
+        summary: "Hide any uncommitted changes."
+    })
 };
-WorkflowGet.args = [
-    {
-        name: "workflowKey",
+WorkflowGet.args = {
+    workflowKey: _core.Args.string({
         required: true
-    }
-];
+    })
+};
 WorkflowGet.enableJsonFlag = true;

package/dist/commands/workflow/list.js

@@ -79,7 +79,7 @@ class WorkflowList extends _baseCommand.default {
         this.log(`‣ Showing ${entries.length} workflows in \`${env}\` environment ${qualifier}\n`);
         /*
      * Workflows list table
-     */ _core.CliUx.ux.table(entries, {
+     */ _core.ux.table(entries, {
             key: {
                 header: "Key"
             },
@@ -118,11 +118,15 @@ class WorkflowList extends _baseCommand.default {
         }
     }
 }
+WorkflowList.summary = "Display all workflows for an environment.";
 WorkflowList.flags = {
     environment: _core.Flags.string({
-        default: "development"
+        default: "development",
+        summary: "The environment to use."
+    }),
+    "hide-uncommitted-changes": _core.Flags.boolean({
+        summary: "Hide any uncommitted changes."
     }),
-    "hide-uncommitted-changes": _core.Flags.boolean(),
     ..._page.pageFlags
 };
 WorkflowList.enableJsonFlag = true;

package/dist/commands/workflow/new.js

@@ -125,11 +125,10 @@ WorkflowNew.flags = {
     }),
     force: _core.Flags.boolean()
 };
-WorkflowNew.args = [
-    {
-        name: "workflowKey",
+WorkflowNew.args = {
+    workflowKey: _core.Args.string({
         required: true
-    }
-];
+    })
+};
 // TODO(KNO-3072): Unhide after we move the generator logic to the backend.
 WorkflowNew.hidden = true;