@knocklabs/cli 0.1.0-rc.4 → 0.1.3

package/dist/commands/workflow/pull.js

@@ -78,7 +78,7 @@ class WorkflowPull extends _baseCommand.default {
         if (dirContext.exists) {
             this.log(`‣ Found \`${dirContext.key}\` at ${dirContext.abspath}`);
         } else {
-            const prompt = `Create a new workflow directory \`${dirContext.key}\`?`;
+            const prompt = `Create a new workflow directory \`${dirContext.key}\` at ${dirContext.abspath}?`;
             const input = flags.force || await (0, _ux.promptToConfirm)(prompt);
             if (!input) return;
         }
@@ -171,22 +171,30 @@ class WorkflowPull extends _baseCommand.default {
         }, workflows) : workflows;
     }
 }
+WorkflowPull.summary = "Pull one or more workflows from an environment into a local file system.";
 WorkflowPull.flags = {
     environment: _core.Flags.string({
-        default: "development"
+        default: "development",
+        summary: "The environment to use."
+    }),
+    all: _core.Flags.boolean({
+        summary: "Whether to pull all workflows from the specified environment."
     }),
-    all: _core.Flags.boolean(),
     "workflows-dir": _flag.dirPath({
+        summary: "The target directory path to pull all workflows 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."
+    })
 };
-WorkflowPull.args = [
-    {
-        name: "workflowKey",
+WorkflowPull.args = {
+    workflowKey: _core.Args.string({
         required: false
-    }
-];
+    })
+};

package/dist/commands/workflow/push.js

@@ -6,15 +6,17 @@ Object.defineProperty(exports, "default", {
     enumerable: true,
     get: ()=>WorkflowPush
 });
-const _nodePath = /*#__PURE__*/ _interopRequireWildcard(require("node:path"));
 const _core = require("@oclif/core");
 const _baseCommand = /*#__PURE__*/ _interopRequireDefault(require("../../lib/base-command"));
 const _const = require("../../lib/helpers/const");
 const _error = require("../../lib/helpers/error");
+const _flag = /*#__PURE__*/ _interopRequireWildcard(require("../../lib/helpers/flag"));
 const _object = require("../../lib/helpers/object");
 const _request = require("../../lib/helpers/request");
+const _string = require("../../lib/helpers/string");
+const _ux = require("../../lib/helpers/ux");
 const _workflow = /*#__PURE__*/ _interopRequireWildcard(require("../../lib/marshal/workflow"));
-const _runContext = require("../../lib/run-context");
+const _validate = /*#__PURE__*/ _interopRequireDefault(require("./validate"));
 function _interopRequireDefault(obj) {
     return obj && obj.__esModule ? obj : {
         default: obj
@@ -61,57 +63,60 @@ function _interopRequireWildcard(obj, nodeInterop) {
 }
 class WorkflowPush extends _baseCommand.default {
     async run() {
-        // 1. Retrieve the target workflow directory context.
-        const dirContext = await this.getWorkflowDirContext();
-        this.log(`‣ Reading \`${dirContext.key}\` at ${dirContext.abspath}`);
-        // 2. Read the workflow.json with its template files.
-        const [workflow, errors] = await _workflow.readWorkflowDir(dirContext, {
+        const { flags  } = this.props;
+        // 1. First read all workflow directories found for the given command.
+        const target = await _workflow.ensureValidCommandTarget(this.props, this.runContext);
+        const [workflows, readErrors] = await _workflow.readAllForCommandTarget(target, {
             withExtractedFiles: true
         });
-        if (errors.length > 0) {
-            this.error(`Found the following errors in \`${dirContext.key}\` ${_workflow.WORKFLOW_JSON}\n\n` + (0, _error.formatErrors)(errors));
+        if (readErrors.length > 0) {
+            this.error((0, _error.formatErrors)(readErrors, {
+                prependBy: "\n\n"
+            }));
         }
-        // 3. Push up the compiled workflow data.
-        const resp = await (0, _request.withSpinner)(()=>{
+        if (workflows.length === 0) {
+            this.error(`No workflow directories found in ${target.context.abspath}`);
+        }
+        // 2. Then validate them all ahead of pushing them.
+        _ux.spinner.start(`‣ Validating`);
+        const apiErrors = await _validate.default.validateAll(this.apiV1, this.props, workflows);
+        if (apiErrors.length > 0) {
+            this.error((0, _error.formatErrors)(apiErrors, {
+                prependBy: "\n\n"
+            }));
+        }
+        _ux.spinner.stop();
+        // 3. Finally push up each workflow, abort on the first error.
+        _ux.spinner.start(`‣ Pushing`);
+        for (const workflow of workflows){
             const props = (0, _object.merge)(this.props, {
-                args: {
-                    workflowKey: dirContext.key
-                },
                 flags: {
                     annotate: true
                 }
             });
-            return this.apiV1.upsertWorkflow(props, workflow);
-        });
-        // 4. Update the workflow directory with the successfully pushed workflow
-        // payload from the server.
-        await _workflow.writeWorkflowDirFromData(dirContext, resp.data.workflow);
-        this.log(`‣ Successfully pushed \`${dirContext.key}\`, and updated ${dirContext.abspath}`);
-    }
-    async getWorkflowDirContext() {
-        const { workflowKey  } = this.props.args;
-        const { resourceDir , cwd: runCwd  } = this.runContext;
-        if (resourceDir) {
-            const target = {
-                commandId: _baseCommand.default.id,
-                type: "workflow",
-                key: workflowKey
-            };
-            return (0, _runContext.ensureResourceDirForTarget)(resourceDir, target);
-        }
-        if (workflowKey) {
-            const dirPath = _nodePath.resolve(runCwd, workflowKey);
-            const exists = await _workflow.isWorkflowDir(dirPath);
-            return exists ? {
-                type: "workflow",
-                key: workflowKey,
-                abspath: dirPath,
-                exists
-            } : this.error(`Cannot locate a workflow directory for \`${workflowKey}\``);
+            // eslint-disable-next-line no-await-in-loop
+            const resp = await this.apiV1.upsertWorkflow(props, {
+                ...workflow.content,
+                key: workflow.key
+            });
+            if ((0, _request.isSuccessResp)(resp)) {
+                // Update the workflow directory with the successfully pushed workflow
+                // payload from the server.
+                // eslint-disable-next-line no-await-in-loop
+                await _workflow.writeWorkflowDirFromData(workflow, resp.data.workflow);
+                continue;
+            }
+            const error = new _error.SourceError((0, _request.formatErrorRespMessage)(resp), _workflow.workflowJsonPath(workflow), "ApiError");
+            this.error((0, _error.formatError)(error));
         }
-        return this.error("Missing 1 required arg:\nworkflowKey");
+        _ux.spinner.stop();
+        // 4. Display a success message.
+        const workflowKeys = workflows.map((w)=>w.key);
+        const actioned = flags.commit ? "pushed and committed" : "pushed";
+        this.log(`‣ Successfully ${actioned} ${workflows.length} workflow(s):\n` + (0, _string.indentString)(workflowKeys.join("\n"), 4));
     }
 }
+WorkflowPush.summary = "Push one or more workflows from a local file system to Knock.";
 WorkflowPush.flags = {
     environment: _core.Flags.string({
         summary: "Pushing a workflow is only allowed in the development environment",
@@ -120,6 +125,15 @@ WorkflowPush.flags = {
             _const.KnockEnv.Development
         ]
     }),
+    all: _core.Flags.boolean({
+        summary: "Whether to push all workflows from the target directory."
+    }),
+    "workflows-dir": _flag.dirPath({
+        summary: "The target directory path to find all workflows to push.",
+        dependsOn: [
+            "all"
+        ]
+    }),
     commit: _core.Flags.boolean({
         summary: "Push and commit the workflow(s) at the same time"
     }),
@@ -131,9 +145,8 @@ WorkflowPush.flags = {
         ]
     })
 };
-WorkflowPush.args = [
-    {
-        name: "workflowKey",
+WorkflowPush.args = {
+    workflowKey: _core.Args.string({
         required: false
-    }
-];
+    })
+};

package/dist/commands/workflow/run.js

@@ -0,0 +1,58 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", {
+    value: true
+});
+Object.defineProperty(exports, "default", {
+    enumerable: true,
+    get: ()=>WorkflowRun
+});
+const _core = require("@oclif/core");
+const _baseCommand = /*#__PURE__*/ _interopRequireDefault(require("../../lib/base-command"));
+const _flag = require("../../lib/helpers/flag");
+const _request = require("../../lib/helpers/request");
+const _string = require("../../lib/helpers/string");
+function _interopRequireDefault(obj) {
+    return obj && obj.__esModule ? obj : {
+        default: obj
+    };
+}
+class WorkflowRun extends _baseCommand.default {
+    async run() {
+        const { args , flags  } = this.props;
+        const resp = await (0, _request.withSpinner)(()=>this.apiV1.runWorkflow(this.props), {
+            action: "‣ Running"
+        });
+        this.log(`‣ Successfully ran \`${args.workflowKey}\` workflow in \`${flags.environment}\` environment`);
+        this.log((0, _string.indentString)(`Workflow run id: ${resp.data.workflow_run_id}`), 4);
+    }
+}
+WorkflowRun.summary = "Test run a workflow using the latest version from Knock, or a local workflow directory.";
+WorkflowRun.flags = {
+    environment: _core.Flags.string({
+        default: "development",
+        summary: "The environment in which to run the workflow"
+    }),
+    recipients: _core.Flags.string({
+        required: true,
+        aliases: [
+            "recipient"
+        ],
+        summary: "One or more recipient ids for this workflow run, separated by comma.",
+        multiple: true,
+        delimiter: ","
+    }),
+    actor: _core.Flags.string({
+        summary: "An actor id for the workflow run."
+    }),
+    tenant: _core.Flags.string({
+        summary: "A tenant id for the workflow run."
+    }),
+    data: (0, _flag.jsonStr)({
+        summary: "A JSON string of the data for this workflow"
+    })
+};
+WorkflowRun.args = {
+    workflowKey: _core.Args.string({
+        required: true
+    })
+};

package/dist/commands/workflow/validate.js

@@ -6,15 +6,15 @@ Object.defineProperty(exports, "default", {
     enumerable: true,
     get: ()=>WorkflowValidate
 });
-const _nodePath = /*#__PURE__*/ _interopRequireWildcard(require("node:path"));
 const _core = require("@oclif/core");
 const _baseCommand = /*#__PURE__*/ _interopRequireDefault(require("../../lib/base-command"));
 const _const = require("../../lib/helpers/const");
 const _error = require("../../lib/helpers/error");
-const _object = require("../../lib/helpers/object");
+const _flag = /*#__PURE__*/ _interopRequireWildcard(require("../../lib/helpers/flag"));
 const _request = require("../../lib/helpers/request");
+const _string = require("../../lib/helpers/string");
+const _ux = require("../../lib/helpers/ux");
 const _workflow = /*#__PURE__*/ _interopRequireWildcard(require("../../lib/marshal/workflow"));
-const _runContext = require("../../lib/run-context");
 function _interopRequireDefault(obj) {
     return obj && obj.__esModule ? obj : {
         default: obj
@@ -61,53 +61,49 @@ function _interopRequireWildcard(obj, nodeInterop) {
 }
 class WorkflowValidate extends _baseCommand.default {
     async run() {
-        // 1. Retrieve the target workflow directory context.
-        const dirContext = await this.getWorkflowDirContext();
-        this.log(`‣ Reading \`${dirContext.key}\` at ${dirContext.abspath}`);
-        // 2. Read the workflow.json with its template files.
-        const [workflow, errors] = await _workflow.readWorkflowDir(dirContext, {
+        // 1. Read all workflow directories found for the given command.
+        const target = await _workflow.ensureValidCommandTarget(this.props, this.runContext);
+        const [workflows, readErrors] = await _workflow.readAllForCommandTarget(target, {
             withExtractedFiles: true
         });
-        if (errors.length > 0) {
-            this.error(`Found the following errors in \`${dirContext.key}\` ${_workflow.WORKFLOW_JSON}\n\n` + (0, _error.formatErrors)(errors));
+        if (readErrors.length > 0) {
+            this.error((0, _error.formatErrors)(readErrors, {
+                prependBy: "\n\n"
+            }));
         }
-        // 3. Validate the compiled workflow data.
-        await (0, _request.withSpinner)(()=>{
-            const props = (0, _object.merge)(this.props, {
-                args: {
-                    workflowKey: dirContext.key
-                }
-            });
-            return this.apiV1.validateWorkflow(props, workflow);
-        }, {
-            action: "‣ Validating"
-        });
-        this.log(`‣ Successfully validated \`${dirContext.key}\``);
-    }
-    async getWorkflowDirContext() {
-        const { workflowKey  } = this.props.args;
-        const { resourceDir , cwd: runCwd  } = this.runContext;
-        if (resourceDir) {
-            const target = {
-                commandId: _baseCommand.default.id,
-                type: "workflow",
-                key: workflowKey
-            };
-            return (0, _runContext.ensureResourceDirForTarget)(resourceDir, target);
+        if (workflows.length === 0) {
+            this.error(`No workflow directories found in ${target.context.abspath}`);
         }
-        if (workflowKey) {
-            const dirPath = _nodePath.resolve(runCwd, workflowKey);
-            const exists = await _workflow.isWorkflowDir(dirPath);
-            return exists ? {
-                type: "workflow",
-                key: workflowKey,
-                abspath: dirPath,
-                exists
-            } : this.error(`Cannot locate a workflow directory for \`${workflowKey}\``);
+        // 2. Validate each workflow data.
+        _ux.spinner.start(`‣ Validating`);
+        const apiErrors = await WorkflowValidate.validateAll(this.apiV1, this.props, workflows);
+        if (apiErrors.length > 0) {
+            this.error((0, _error.formatErrors)(apiErrors, {
+                prependBy: "\n\n"
+            }));
         }
-        return this.error("Missing 1 required arg:\nworkflowKey");
+        _ux.spinner.stop();
+        // 3. Display a success message.
+        const workflowKeys = workflows.map((w)=>w.key);
+        this.log(`‣ Successfully validated ${workflows.length} workflow(s):\n` + (0, _string.indentString)(workflowKeys.join("\n"), 4));
+    }
+    static async validateAll(api, props, workflows) {
+        // TODO: Throw an error if a non validation error (e.g. authentication error)
+        // instead of printing out same error messages repeatedly.
+        const errorPromises = workflows.map(async (workflow)=>{
+            const resp = await api.validateWorkflow(props, {
+                ...workflow.content,
+                key: workflow.key
+            });
+            if ((0, _request.isSuccessResp)(resp)) return;
+            const error = new _error.SourceError((0, _request.formatErrorRespMessage)(resp), _workflow.workflowJsonPath(workflow), "ApiError");
+            return error;
+        });
+        const errors = (await Promise.all(errorPromises)).filter((e)=>Boolean(e));
+        return errors;
     }
 }
+WorkflowValidate.summary = "Validate one or more workflows from a local file system.";
 WorkflowValidate.flags = {
     environment: _core.Flags.string({
         summary: "Validating a workflow is only done in the development environment",
@@ -115,11 +111,19 @@ WorkflowValidate.flags = {
         options: [
             _const.KnockEnv.Development
         ]
+    }),
+    all: _core.Flags.boolean({
+        summary: "Whether to validate all workflows from the target directory."
+    }),
+    "workflows-dir": _flag.dirPath({
+        summary: "The target directory path to find all workflows to validate.",
+        dependsOn: [
+            "all"
+        ]
     })
 };
-WorkflowValidate.args = [
-    {
-        name: "workflowKey",
+WorkflowValidate.args = {
+    workflowKey: _core.Args.string({
         required: false
-    }
-];
+    })
+};

package/dist/lib/api-v1.js

@@ -20,6 +20,9 @@ class ApiV1 {
     async ping() {
         return this.get("/ping");
     }
+    async whoami() {
+        return this.get("/whoami");
+    }
     // By resources: Workflows
     async listWorkflows({ flags  }) {
         const params = (0, _object.prune)({
@@ -42,7 +45,7 @@ class ApiV1 {
             params
         });
     }
-    async upsertWorkflow({ args , flags  }, workflow) {
+    async upsertWorkflow({ flags  }, workflow) {
         const params = (0, _object.prune)({
             environment: flags.environment,
             annotate: flags.annotate,
@@ -52,18 +55,18 @@ class ApiV1 {
         const data = {
             workflow
         };
-        return this.put(`/workflows/${args.workflowKey}`, data, {
+        return this.put(`/workflows/${workflow.key}`, data, {
             params
         });
     }
-    async validateWorkflow({ args , flags  }, workflow) {
+    async validateWorkflow({ flags  }, workflow) {
         const params = (0, _object.prune)({
             environment: flags.environment
         });
         const data = {
             workflow
         };
-        return this.put(`/workflows/${args.workflowKey}/validate`, data, {
+        return this.put(`/workflows/${workflow.key}/validate`, data, {
             params
         });
     }
@@ -76,6 +79,18 @@ class ApiV1 {
             params
         });
     }
+    async runWorkflow({ args , flags  }) {
+        const params = (0, _object.prune)({
+            environment: flags.environment,
+            recipients: flags.recipients,
+            tenant: flags.tenant,
+            data: flags.data,
+            actor: flags.actor
+        });
+        return this.put(`/workflows/${args.workflowKey}/run`, {}, {
+            params
+        });
+    }
     // By resources: Commits
     async commitAllChanges({ flags  }) {
         const params = (0, _object.prune)({
@@ -95,16 +110,28 @@ class ApiV1 {
         });
     }
     // By resources: Translations
-    async listTranslations({ flags  }) {
+    async listTranslations({ flags  }, filters = {}) {
         const params = (0, _object.prune)({
             environment: flags.environment,
             hide_uncommitted_changes: flags["hide-uncommitted-changes"],
+            locale_code: filters.localeCode,
+            namespace: filters.namespace,
             ...(0, _page.toPageParams)(flags)
         });
         return this.get("/translations", {
             params
         });
     }
+    async getTranslation({ flags  }, translation) {
+        const params = (0, _object.prune)({
+            environment: flags.environment,
+            hide_uncommitted_changes: flags["hide-uncommitted-changes"],
+            namespace: translation.namespace
+        });
+        return this.get(`/translations/${translation.localeCode}`, {
+            params
+        });
+    }
     async upsertTranslation({ flags  }, translation) {
         const params = (0, _object.prune)({
             environment: flags.environment,

package/dist/lib/base-command.js

@@ -60,21 +60,25 @@ class BaseCommand extends _core.Command {
         // 1. Load user's config from the config dir, as available.
         await _userConfig.default.load(this.config.configDir);
         // 2. Parse flags and args, must come after the user config load.
-        this.props = await this.parse(this.constructor);
+        const { args , flags  } = await this.parse(this.ctor);
+        this.props = {
+            args: args,
+            flags: flags
+        };
         // 3. Instantiate a knock api client.
         this.apiV1 = new _apiV1.default(this.props.flags, this.config);
         // 4. Load the run context of the invoked command.
         this.runContext = await _runContext.load(this.id);
     }
 }
-// Global flags are inherited by any command that extends BaseCommand.
-BaseCommand.globalFlags = {
+// Base flags are inherited by any command that extends BaseCommand.
+BaseCommand.baseFlags = {
     // Evaluated in the following precedence:
     // - service token flag passed into the command
     // - if not provided, fall back to env variable
     // - if not available, fall back to user config
     "service-token": _core.Flags.string({
-        summary: "The service token to authenticate with",
+        summary: "The service token to authenticate with.",
         required: true,
         multiple: false,
         env: "KNOCK_SERVICE_TOKEN",

package/dist/lib/helpers/flag.js

@@ -10,7 +10,8 @@ function _export(target, all) {
 }
 _export(exports, {
     booleanStr: ()=>booleanStr,
-    dirPath: ()=>dirPath
+    dirPath: ()=>dirPath,
+    jsonStr: ()=>jsonStr
 });
 const _nodePath = /*#__PURE__*/ _interopRequireWildcard(require("node:path"));
 const _core = require("@oclif/core");
@@ -74,3 +75,13 @@ const dirPath = _core.Flags.custom({
         };
     }
 });
+const jsonStr = _core.Flags.custom({
+    parse: async (input)=>{
+        try {
+            const data = JSON.parse(input);
+            return data;
+        } catch  {
+            throw new Error(`${input} is not a valid JSON string.`);
+        }
+    }
+});

package/dist/lib/helpers/page.js

@@ -28,9 +28,14 @@ function _interopRequireDefault(obj) {
 }
 const MAX_PAGINATION_LIMIT = 100;
 const pageFlags = {
-    after: _core.Flags.string(),
-    before: _core.Flags.string(),
+    after: _core.Flags.string({
+        summary: "The cursor after which to fetch the next page."
+    }),
+    before: _core.Flags.string({
+        summary: "The cursor before which to fetch the previous page."
+    }),
     limit: _core.Flags.integer({
+        summary: "The total number of entries to fetch per page.",
         max: MAX_PAGINATION_LIMIT
     })
 };

package/dist/lib/helpers/request.js

@@ -38,7 +38,7 @@ const withSpinner = async (requestFn, opts = {})=>{
     // Error out before the action stop so the spinner can update accordingly.
     if (ensureSuccess && !isSuccessResp(resp)) {
         const message = formatErrorRespMessage(resp);
-        _core.CliUx.ux.error(new _error.ApiError(message));
+        _core.ux.error(new _error.ApiError(message));
     }
     _ux.spinner.stop();
     return resp;

package/dist/lib/helpers/ux.js

@@ -34,9 +34,9 @@ const promptToConfirm = async (message)=>{
 };
 const spinner = {
     start (action) {
-        if (!_const.isTestEnv) _core.CliUx.ux.action.start(action);
+        if (!_const.isTestEnv) _core.ux.action.start(action);
     },
     stop () {
-        _core.CliUx.ux.action.stop();
+        _core.ux.action.stop();
     }
 };

package/dist/lib/marshal/translation/helpers.js

@@ -9,6 +9,9 @@ function _export(target, all) {
     });
 }
 _export(exports, {
+    translationRefDescription: ()=>translationRefDescription,
+    formatLanguage: ()=>formatLanguage,
+    formatRef: ()=>formatRef,
     isValidLocale: ()=>isValidLocale,
     isTranslationDir: ()=>isTranslationDir,
     buildTranslationFileCtx: ()=>buildTranslationFileCtx,
@@ -65,6 +68,16 @@ function _interopRequireWildcard(obj, nodeInterop) {
     }
     return newObj;
 }
+const translationRefDescription = `
+Translation ref is a identifier string that refers to a unique translation.
+If a translation has no namespace, it is the same as the locale, e.g. \`en\`.
+If namespaced, it is formatted as namespace.locale, e.g. \`admin.en\`.
+`.trim();
+const formatLanguage = (translation)=>{
+    const lang = _localeCodes.default.getByTag(translation.locale_code);
+    return lang.location ? `${lang.name}, ${lang.location}` : lang.name;
+};
+const formatRef = ({ locale_code , namespace  })=>namespace ? `${namespace}.${locale_code}` : locale_code;
 const isValidLocale = (localeCode)=>Boolean(_localeCodes.default.getByTag(localeCode));
 const isTranslationDir = (dirPath)=>{
     const locale = _nodePath.basename(dirPath);
@@ -87,7 +100,8 @@ const parseTranslationRef = (reference)=>{
     // Locale code only (e.g. `en`)
     if (strings.length === 1) {
         return {
-            localeCode: strings[0]
+            localeCode: strings[0],
+            namespace: undefined
         };
     }
     // Locale code prefixed with a namespace (e.g. `admin.en`)
@@ -105,16 +119,16 @@ const ensureValidCommandTarget = async (props, runContext)=>{
     const { commandId , resourceDir: resourceDirCtx , cwd: runCwd  } = runContext;
     // Error, trying to run the command not in a translation directory.
     if (resourceDirCtx && resourceDirCtx.type !== "translation") {
-        return _core.CliUx.ux.error(`Cannot run ${commandId} inside a ${resourceDirCtx.type} directory`);
+        return _core.ux.error(`Cannot run ${commandId} inside a ${resourceDirCtx.type} directory`);
     }
     // Error, got neither the translationRef arg nor the --all flag.
     if (!args.translationRef && !flags.all) {
-        _core.CliUx.ux.error("At least one of translation ref arg or --all flag must be given");
+        _core.ux.error("At least one of translation ref arg or --all flag must be given");
     }
     // No translationRef arg, which means --all flag is used.
     if (!args.translationRef) {
         // Targeting all translation files in the current locale directory.
-        if (resourceDirCtx) {
+        if (resourceDirCtx && !flags["translations-dir"]) {
             return {
                 type: "translationDir",
                 context: resourceDirCtx
@@ -135,12 +149,12 @@ const ensureValidCommandTarget = async (props, runContext)=>{
     // From this point on, we have translationRef so parse and validate the format.
     const parsedRef = parseTranslationRef(args.translationRef);
     if (!parsedRef) {
-        return _core.CliUx.ux.error(`Invalid translationRef arg \`${args.translationRef}\`, use valid <locale> or <namespace>.<locale> for namespaced translations`);
+        return _core.ux.error(`Invalid translation ref \`${args.translationRef}\`, use valid <locale> or <namespace>.<locale> for namespaced translations`);
     }
     const { localeCode , namespace  } = parsedRef;
     // If we are in the translation dir, make sure the locale matches.
     if (resourceDirCtx && resourceDirCtx.key !== localeCode) {
-        return _core.CliUx.ux.error(`Cannot run ${commandId} with \`${args.translationRef}\` inside a ${resourceDirCtx.key} directory`);
+        return _core.ux.error(`Cannot run ${commandId} with \`${args.translationRef}\` inside a ${resourceDirCtx.key} directory`);
     }
     const targetDirPath = resourceDirCtx ? resourceDirCtx.abspath : _nodePath.resolve(runCwd, localeCode);
     // Got translationRef arg but no --all flag, which means target only a single
@@ -155,7 +169,7 @@ const ensureValidCommandTarget = async (props, runContext)=>{
     // From this point on, we have both translationRef and --all flag used
     // together, so make sure we are targeting a non-namespaced top-level locale.
     if (namespace) {
-        return _core.CliUx.ux.error(`Cannot use --all with a namespaced translation \`${args.translationRef}\``);
+        return _core.ux.error(`Cannot use --all with a namespaced translation \`${args.translationRef}\``);
     }
     const translationDirCtx = {
         type: "translation",