@oclif/core 1.3.4 → 1.4.0

package/CHANGELOG.md

@@ -2,6 +2,27 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+## [1.4.0](https://github.com/oclif/core/compare/v1.3.6...v1.4.0) (2022-03-01)
+
+
+### Features
+
+* make global flags settable ([#385](https://github.com/oclif/core/issues/385)) ([e14061c](https://github.com/oclif/core/commit/e14061ca7e6a4c288eb50e0e9954b38e042682df))
+
+### [1.3.6](https://github.com/oclif/core/compare/v1.3.5...v1.3.6) (2022-02-28)
+
+
+### Bug Fixes
+
+* parsing the default is wrong types ([ba08723](https://github.com/oclif/core/commit/ba087237773e6f4b3649d03dc88f693a22681de9))
+
+### [1.3.5](https://github.com/oclif/core/compare/v1.3.4...v1.3.5) (2022-02-25)
+
+
+### Bug Fixes
+
+* print valid flag values in error message when using `exactlyOne` ([#349](https://github.com/oclif/core/issues/349)) ([ddcaeb2](https://github.com/oclif/core/commit/ddcaeb2f9b690d9b92dd0ac4937b6399f606adfa))
+
 ### [1.3.4](https://github.com/oclif/core/compare/v1.3.3...v1.3.4) (2022-02-11)
 
 

package/lib/command.d.ts

@@ -63,9 +63,11 @@ export default abstract class Command {
      * @param {Interfaces.LoadOptions} opts options
      */
     static run: Interfaces.Command.Class['run'];
-    private static globalFlags;
+    protected static _globalFlags: Interfaces.FlagInput<any>;
+    static get globalFlags(): Interfaces.FlagInput<any>;
+    static set globalFlags(flags: Interfaces.FlagInput<any>);
     /** A hash of flags for the command */
-    private static _flags;
+    protected static _flags: Interfaces.FlagInput<any>;
     static get flags(): Interfaces.FlagInput<any>;
     static set flags(flags: Interfaces.FlagInput<any>);
     id: string | undefined;

package/lib/command.js

@@ -17,6 +17,12 @@ process.stdout.on('error', (err) => {
         return;
     throw err;
 });
+const jsonFlag = {
+    json: Flags.boolean({
+        description: 'Format output as json.',
+        helpGroup: 'GLOBAL',
+    }),
+};
 /**
  * An abstract class which acts as the base for each command
  * in your project.
@@ -33,11 +39,20 @@ class Command {
             this.debug = () => { };
         }
     }
+    static get globalFlags() {
+        return this._globalFlags;
+    }
+    static set globalFlags(flags) {
+        this._globalFlags = this.enableJsonFlag ?
+            Object.assign({}, jsonFlag, this.globalFlags, flags) :
+            Object.assign({}, this.globalFlags, flags);
+    }
     static get flags() {
         return this._flags;
     }
     static set flags(flags) {
-        this._flags = this.enableJsonFlag ? Object.assign({}, Command.globalFlags, flags) : flags;
+        this.globalFlags = {};
+        this._flags = Object.assign({}, this.globalFlags, flags);
     }
     get ctor() {
         return this.constructor;
@@ -169,9 +184,3 @@ Command.run = async function (argv, opts) {
     const cmd = new this(argv, config);
     return cmd._run(argv);
 };
-Command.globalFlags = {
-    json: Flags.boolean({
-        description: 'Format output as json.',
-        helpGroup: 'GLOBAL',
-    }),
-};

package/lib/parser/parse.js

@@ -181,11 +181,7 @@ class Parser {
                 this.metaData.flags[k] = { setFromDefault: true };
                 // eslint-disable-next-line no-await-in-loop
                 const defaultValue = (typeof flag.default === 'function' ? await flag.default({ options: flag, flags, ...this.context }) : flag.default);
-                const parsedValue = flag.type === 'option' && flag.parse ?
-                    // eslint-disable-next-line no-await-in-loop
-                    await flag.parse(defaultValue, this.context) :
-                    defaultValue;
-                flags[k] = parsedValue;
+                flags[k] = defaultValue;
             }
         }
         return flags;

package/lib/parser/validate.js

@@ -30,6 +30,7 @@ function validate(parse) {
         }
     }
     function validateAcrossFlags(flag) {
+        var _a;
         const intersection = Object.entries(parse.input.flags)
             .map(entry => entry[0]) // array of flag names
             .filter(flagName => parse.output.flags[flagName] !== undefined) // with values
@@ -37,8 +38,8 @@ function validate(parse) {
         if (intersection.length === 0) {
             // the command's exactlyOne may or may not include itself, so we'll use Set to add + de-dupe
             throw new errors_1.CLIError(`Exactly one of the following must be provided: ${[
-                ...new Set(...flag.exactlyOne || [], flag.name),
-            ].join(',')}`);
+                ...new Set((_a = flag.exactlyOne) === null || _a === void 0 ? void 0 : _a.map(flag => `--${flag}`)),
+            ].join(', ')}`);
         }
     }
     function validateFlags() {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@oclif/core",
   "description": "base library for oclif CLIs",
-  "version": "1.3.4",
+  "version": "1.4.0",
   "author": "Salesforce",
   "bugs": "https://github.com/oclif/core/issues",
   "dependencies": {