@fonoster/ctl 0.21.0 → 0.22.1

package/dist/commands/applications/update.js

@@ -78,7 +78,7 @@ class Update extends AuthenticatedCommand_1.AuthenticatedCommand {
             if (flags["from-file"]) {
                 try {
                     const client = yield this.createSdkClient();
-                    yield (0, createOrUpdateApplication_1.createOrUpdateApplication)(client, flags["from-file"], ref);
+                    yield (0, createOrUpdateApplication_1.createOrUpdateApplication)(client, flags["from-file"], ref, true);
                     this.log("Done!");
                     return;
                 }
@@ -87,9 +87,12 @@ class Update extends AuthenticatedCommand_1.AuthenticatedCommand {
                     return;
                 }
             }
+            if (!ref) {
+                this.error("An Application ref is required. Pass it as the positional argument or use --from-file with a `ref` field.");
+            }
             const client = yield this.createSdkClient();
             const applications = new SDK.Applications(client);
-            const applicationFromDB = yield applications.getApplication(args.ref);
+            const applicationFromDB = yield applications.getApplication(ref);
             if (!applicationFromDB) {
                 this.error("Application not found.");
             }
@@ -208,8 +211,7 @@ Update.description = "modify the configuration of an Application";
 Update.examples = ["<%= config.bin %> <%= command.id %>"];
 Update.args = {
     ref: core_1.Args.string({
-        description: "the Application to update",
-        required: true
+        description: "the Application to update (optional when `ref` is set in --from-file)"
     })
 };
 Update.flags = {

package/dist/utils/createOrUpdateApplication.d.ts

@@ -1,2 +1,2 @@
 import * as SDK from "@fonoster/sdk";
-export declare function createOrUpdateApplication(client: SDK.Client, filePath: string, appRef?: string): Promise<import("@fonoster/types").BaseApiObject>;
+export declare function createOrUpdateApplication(client: SDK.Client, filePath: string, appRef?: string, isUpdate?: boolean): Promise<import("@fonoster/types").BaseApiObject>;

package/dist/utils/createOrUpdateApplication.js

@@ -68,8 +68,8 @@ const fs_1 = require("fs");
 const path_1 = __importDefault(require("path"));
 const SDK = __importStar(require("@fonoster/sdk"));
 const js_yaml_1 = require("js-yaml");
-function createOrUpdateApplication(client, filePath, appRef) {
-    return __awaiter(this, void 0, void 0, function* () {
+function createOrUpdateApplication(client_1, filePath_1, appRef_1) {
+    return __awaiter(this, arguments, void 0, function* (client, filePath, appRef, isUpdate = false) {
         var _a, _b, _c;
         const fileContent = (0, fs_1.readFileSync)(filePath, "utf8");
         const fileExt = path_1.default.extname(filePath).toLowerCase();
@@ -86,12 +86,17 @@ function createOrUpdateApplication(client, filePath, appRef) {
         const applications = new SDK.Applications(client);
         delete config.testCases;
         (_c = (_b = (_a = config.intelligence) === null || _a === void 0 ? void 0 : _a.config) === null || _b === void 0 ? void 0 : _b.languageModel) === null || _c === void 0 ? true : delete _c.apiKey;
-        if (appRef) {
-            const updateConfig = Object.assign({ ref: appRef }, config);
+        if (isUpdate) {
+            // The positional appRef takes precedence over any `ref` in the file; fall
+            // back to the file's `ref` when the positional is omitted.
+            const ref = appRef !== null && appRef !== void 0 ? appRef : config.ref;
+            if (!ref) {
+                throw new Error("An Application ref is required to update. Provide it as the positional argument or include `ref` in the file.");
+            }
+            delete config.ref;
+            const updateConfig = Object.assign(Object.assign({}, config), { ref });
             return applications.updateApplication(updateConfig);
         }
-        else {
-            return applications.createApplication(config);
-        }
+        return applications.createApplication(config);
     });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fonoster/ctl",
-  "version": "0.21.0",
+  "version": "0.22.1",
   "description": "Fonoster Control Tool",
   "author": "Pedro Sanders <psanders@fonoster.com>",
   "homepage": "https://github.com/fonoster/fonoster#readme",
@@ -30,7 +30,7 @@
   "bugs": {
     "url": "https://github.com/fonoster/fonoster/issues"
   },
-  "gitHead": "34139b12d44bb6c59eb5cc9f1083fbcfaa61b76e",
+  "gitHead": "643925587e6ba287fb36414c5d09fbac6db47ca1",
   "bin": {
     "fonoster": "./bin/run.js"
   },
@@ -51,7 +51,7 @@
     }
   },
   "dependencies": {
-    "@fonoster/sdk": "^0.21.0",
+    "@fonoster/sdk": "^0.22.0",
     "@inquirer/prompts": "^7.1.0",
     "@oclif/core": "^4.0.34",
     "@oclif/plugin-warn-if-update-available": "^3.1.28",