saas-1368 hide inherited filename option (#270)

* make crudFilenameOption non-global + add crudFilenameOption where it was used by inheritance

* return draftlog to Output + make root commands by default require auth + describe choices for date format option

* update version

Author: yaroslav-codefresh

lib/interface/cli/Command.js

@@ -10,9 +10,8 @@ class Command {
     constructor(command) {
         this.root = command.root;
         delete command.root;
-        if (command.hasOwnProperty('requiresAuthentication')) {
-            this.requiresAuthentication = command.requiresAuthentication;
-        }
+
+        this.requiresAuthentication = command.hasOwnProperty('requiresAuthentication') ? command.requiresAuthentication : true;
         this.betaCommand = command.betaCommand || false;
         this.onPremCommand = command.onPremCommand || false;
 

lib/interface/cli/commands/composition/create.cmd.js

@@ -6,6 +6,7 @@ const { prepareKeyValueCompostionFromCLIEnvOption, printError } = require('../..
 const { composition } = require('../../../../logic').api;
 const fs = require('fs');
 const createRoot = require('../root/create.cmd');
+const { crudFilenameOption } = require('../../helpers/general');
 
 
 const command = new Command({
@@ -18,6 +19,7 @@ const command = new Command({
         title: 'Create Composition',
     },
     builder: (yargs) => {
+        crudFilenameOption(yargs);
         return yargs
             .positional('name', {
                 describe: 'Name of composition',

lib/interface/cli/commands/composition/delete.cmd.js

@@ -4,6 +4,7 @@ const CFError = require('cf-errors');
 const _ = require('lodash');
 const { composition } = require('../../../../logic').api;
 const deleteRoot = require('../root/delete.cmd');
+const { crudFilenameOption } = require('../../helpers/general');
 
 
 const command = new Command({
@@ -16,6 +17,7 @@ const command = new Command({
         title: 'Delete Composition',
     },
     builder: (yargs) => {
+        crudFilenameOption(yargs);
         return yargs
             .positional('name', {
                 describe: 'Composition name',

lib/interface/cli/commands/composition/replace.cmd.js

@@ -4,6 +4,7 @@ const CFError = require('cf-errors');
 const _ = require('lodash');
 const { composition } = require('../../../../logic').api;
 const replaceRoot = require('../root/replace.cmd');
+const { crudFilenameOption } = require('../../helpers/general');
 
 
 const command = new Command({
@@ -16,6 +17,7 @@ const command = new Command({
         title: 'Replace Composition',
     },
     builder: (yargs) => {
+        crudFilenameOption(yargs);
         return yargs;
     },
     handler: async (argv) => {

lib/interface/cli/commands/context/apply.cmd.js

@@ -4,6 +4,7 @@ const CFError = require('cf-errors');
 const _ = require('lodash');
 const { context } = require('../../../../logic').api;
 const applyRoot = require('../root/apply.cmd');
+const { crudFilenameOption } = require('../../helpers/general');
 
 
 const command = new Command({
@@ -18,6 +19,7 @@ const command = new Command({
         weight: 90,
     },
     builder: (yargs) => {
+        crudFilenameOption(yargs);
         return yargs
             .example('codefresh apply context -f ./context.yml', 'Apply changes to a context');
     },

lib/interface/cli/commands/context/delete.cmd.js

@@ -4,6 +4,7 @@ const CFError = require('cf-errors');
 const _ = require('lodash');
 const { context } = require('../../../../logic').api;
 const deleteRoot = require('../root/delete.cmd');
+const { crudFilenameOption } = require('../../helpers/general');
 
 const command = new Command({
     command: 'context [name]',
@@ -15,6 +16,7 @@ const command = new Command({
         title: 'Delete Context',
     },
     builder: (yargs) => {
+        crudFilenameOption(yargs);
         return yargs
             .positional('name', {
                 describe: 'Name of context',

lib/interface/cli/commands/root/get.cmd.js

@@ -26,7 +26,7 @@ const get = new Command({
                 group: 'Output Options',
             })
             .option('date-format', {
-                describe: 'Date format: default, date, datetime, date-diff or custom',
+                describe: 'Provide predefined or custom date format. Predefined options: ["default", "date", "datetime", "date-diff"]',
                 alias: 'df',
                 group: 'Output Options',
             })

lib/interface/cli/commands/runtimeEnvironments/apply.cmd.js

@@ -4,6 +4,7 @@ const CFError = require('cf-errors');
 const _ = require('lodash');
 const { runtimeEnvironments } = require('../../../../logic').api;
 const applyRoot = require('../root/apply.cmd');
+const { crudFilenameOption } = require('../../helpers/general');
 
 
 const command = new Command({
@@ -18,6 +19,7 @@ const command = new Command({
         weight: 90,
     },
     builder: (yargs) => {
+        crudFilenameOption(yargs);
         return yargs
             .positional('name', {
                 describe: 'Runtime environments name',

lib/interface/cli/commands/systemRuntimeEnvironments/apply.cmd.js

@@ -4,6 +4,7 @@ const CFError = require('cf-errors');
 const _ = require('lodash');
 const { runtimeEnvironments } = require('../../../../logic').api;
 const applyRoot = require('../root/apply.cmd');
+const { crudFilenameOption } = require('../../helpers/general');
 
 
 const command = new Command({
@@ -18,6 +19,7 @@ const command = new Command({
         weight: 90,
     },
     builder: (yargs) => {
+        crudFilenameOption(yargs);
         return yargs
             .positional('name', {
                 describe: 'Runtime environments name',

lib/interface/cli/helpers/general.js

@@ -102,6 +102,7 @@ const crudFilenameOption = (yargs, options = {}) => {
     const filenameOption = {
         alias: options.alias || 'f',
         describe: options.describe || 'Filename or directory of spec files use to create the resource',
+        global: options.global || false,
     };
     if (options.group) {
         filenameOption.group = options.group;