@oclif/core 4.6.0 → 4.7.1

package/lib/command.d.ts

@@ -73,6 +73,7 @@ export declare abstract class Command {
     static usage: string | string[] | undefined;
     protected debug: (...args: any[]) => void;
     id: string | undefined;
+    parsed: boolean;
     constructor(argv: string[], config: Config);
     /**
      * instantiate and run the command

package/lib/command.js

@@ -128,6 +128,7 @@ class Command {
     static usage;
     debug;
     id;
+    parsed = false;
     constructor(argv, config) {
         this.argv = argv;
         this.config = config;
@@ -189,6 +190,11 @@ class Command {
         }
         if (result && this.jsonEnabled())
             this.logJson(this.toSuccessJson(result));
+        if (!this.parsed && !(0, util_2.isProd)()) {
+            process.emitWarning(`Command ${this.id} did not parse its arguments. Did you forget to call 'this.parse'?`, {
+                code: 'UnparsedCommand',
+            });
+        }
         return result;
     }
     async catch(err) {
@@ -273,6 +279,7 @@ class Command {
         this.argv = [...argvToParse];
         const results = await Parser.parse(argvToParse, opts);
         this.warnIfFlagDeprecated(results.flags ?? {});
+        this.parsed = true;
         return results;
     }
     toErrorJson(err) {

package/lib/parser/parse.js

@@ -75,6 +75,7 @@ const validateOptions = (flag, input) => {
         throw new errors_1.FlagInvalidOptionError(flag, input);
     return input;
 };
+const NEGATION = '--no-';
 class Parser {
     input;
     argv;
@@ -299,9 +300,16 @@ class Parser {
             if (tokenLength) {
                 // boolean
                 if (fws.inputFlag.flag.type === 'boolean' && (0, util_1.last)(fws.tokens)?.input) {
+                    const doesNotContainNegation = (i) => {
+                        const possibleNegations = [i.inputFlag.name, ...(i.inputFlag.flag.aliases ?? [])].map((n) => `${NEGATION}${n}`);
+                        const input = (0, util_1.last)(i.tokens)?.input;
+                        if (!input)
+                            return true;
+                        return !possibleNegations.includes(input);
+                    };
                     return {
                         ...fws,
-                        valueFunction: async (i) => parseFlagOrThrowError((0, util_1.last)(i.tokens)?.input !== `--no-${i.inputFlag.name}`, i.inputFlag.flag, this.context, (0, util_1.last)(i.tokens)),
+                        valueFunction: async (i) => parseFlagOrThrowError(doesNotContainNegation(i), i.inputFlag.flag, this.context, (0, util_1.last)(i.tokens)),
                     };
                 }
                 // multiple with custom delimiter
@@ -455,10 +463,14 @@ class Parser {
         if (this.flagAliases[name]) {
             return this.flagAliases[name].name;
         }
-        if (arg.startsWith('--no-')) {
-            const flag = this.booleanFlags[arg.slice(5)];
+        if (arg.startsWith(NEGATION)) {
+            const flag = this.booleanFlags[arg.slice(NEGATION.length)];
             if (flag && flag.allowNo)
                 return flag.name;
+            const flagAlias = this.flagAliases[arg.slice(NEGATION.length)];
+            if (flagAlias && flagAlias.type === 'boolean' && flagAlias.allowNo) {
+                return flagAlias.name;
+            }
         }
     }
     findShortFlag([_, char]) {

package/package.json

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