bashly 0.7.9 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2ef923a576e9bb9e16ec9f78c96c6008772c77d3bfc9520299fc2dd3cd7f3dd3
-  data.tar.gz: 593393d64ca6e63216e0e974c580db50db754fe430d319115aef8610c47d39aa
+  metadata.gz: 4be359964fc41fae59f83eae85c594a4e0edec41e0cc5752fec25850b7b7ca43
+  data.tar.gz: 62023932e59ae5b9856a9617f9148a7e28bfab5bca5cebca98d3fefbf401ae21
 SHA512:
-  metadata.gz: 2b0ffb33c0c11b423cb59d594362a0c2c071a685a2ad7d4837cbdc3bb1d6a494bdcce8ff1cb3d19cfb0078ac5f167912a9d2f93c44f173d2821b415fe9cbd92e
-  data.tar.gz: c6b6989f3c75226641847feee4e4e8a73326da2a006bbbd52ce7544c7c4ffb3bf1f4a659a15f0b70f14590b1fe0392062099f6ced43134b2a3d81000d5a350f9
+  metadata.gz: a0e222c6ead53cfef4597653d9ce51b6e9fffd1a3556acd72eff5447b9a203371a0097cd42f4058405cd473e7f0c55f4f9a448d45d3b1af73898b74a79706df4
+  data.tar.gz: 8362eff89fccaad8c9a1f370270e33426d0089bbe131eec4fc55f586413afaa556caa81e62389daefb400941122db96fa1f8b4e04d3780a4d45ea88060dfdc1e

data/lib/bashly/commands/add.rb

@@ -3,14 +3,15 @@ module Bashly
     class Add < Base
       help "Add extra features and customization to your script"
 
-      usage "bashly add strings [--force]"
-      usage "bashly add lib [--force]"
-      usage "bashly add config [--force]"
       usage "bashly add colors [--force]"
-      usage "bashly add yaml [--force]"
-      usage "bashly add validations [--force]"
-      usage "bashly add test [--force]"
       usage "bashly add comp FORMAT [OUTPUT --force]"
+      usage "bashly add config [--force]"
+      usage "bashly add lib [--force]"
+      usage "bashly add settings [--force]"
+      usage "bashly add strings [--force]"
+      usage "bashly add test [--force]"
+      usage "bashly add validations [--force]"
+      usage "bashly add yaml [--force]"
       usage "bashly add (-h|--help)"
 
       option "-f --force", "Overwrite existing files"
@@ -18,14 +19,15 @@ module Bashly
       param "FORMAT", "Output format, can be one of:\n  function : generate a function file to be included in your script.\n  script   : generate a standalone bash completions script.\n  yaml     : generate a yaml compatible with completely."
       param "OUTPUT", "For the 'comp function' command: Name of the generated function.\nFor the 'comp script' or 'comp yaml' commands: path to output file.\nIn all cases, this is optional and will have sensible defaults."
 
-      command "strings", "Copy an additional configuration file to your project, allowing you to customize all the tips and error strings."
-      command "lib", "Create the additional lib directory for additional user scripts. All *.sh scripts in this folder will be included in the final bash script."
-      command "config", "Add standard functions for handling INI files to the lib directory."
       command "colors", "Add standard functions for printing colorful and formatted text to the lib directory."
-      command "yaml", "Add standard functions for reading YAML files to the lib directory."
-      command "validations", "Add argument validation functions to the lib directory."
-      command "test", "Add approval testing."
       command "comp", "Generate a bash completions script or function."
+      command "config", "Add standard functions for handling INI files to the lib directory."
+      command "lib", "Create the additional lib directory for additional user scripts. All *.sh scripts in this folder will be included in the final bash script."
+      command "settings", "Copy a sample settings.yml file to your project, allowing you to customize some bashly options."
+      command "strings", "Copy an additional configuration file to your project, allowing you to customize all the tips and error strings."
+      command "test", "Add approval testing."
+      command "validations", "Add argument validation functions to the lib directory."
+      command "yaml", "Add standard functions for reading YAML files to the lib directory."
 
       example "bashly add strings --force"
       example "bashly add comp function"
@@ -34,45 +36,51 @@ module Bashly
       environment "BASHLY_SOURCE_DIR", "The path containing the bashly configuration and source files [default: src]"
       environment "BASHLY_LIB_DIR", "The path to use for creating the library files, relative to the source dir [default: lib]"
 
-      def strings_command
-        add_lib 'strings'
+      attr_reader :skip_src_check
+
+      def colors_command
+        add_lib 'colors'
       end
 
-      def lib_command
-        add_lib 'lib'
+      def comp_command
+        format = args['FORMAT']
+        output = args['OUTPUT']
+
+        case format
+        when "script"   then add_lib 'completions_script', output
+        when "function" then add_lib 'completions', output
+        when "yaml"     then add_lib 'completions_yaml', output
+        else            raise Error, "Unrecognized format: #{format}"
+        end
       end
 
       def config_command
         add_lib 'config'
       end
 
-      def colors_command
-        add_lib 'colors'
+      def settings_command
+        @skip_src_check = true
+        add_lib 'settings'
       end
 
-      def yaml_command
-        add_lib 'yaml'
+      def strings_command
+        add_lib 'strings'
       end
 
-      def validations_command
-        add_lib 'validations'
+      def lib_command
+        add_lib 'lib'
       end
 
       def test_command
         add_lib 'test'
       end
 
-      def comp_command
-        format = args['FORMAT']
-        output = args['OUTPUT']
-
-        case format
-        when "script"   then add_lib 'completions_script', output
-        when "function" then add_lib 'completions', output
-        when "yaml"     then add_lib 'completions_yaml', output
-        else            raise Error, "Unrecognized format: #{format}"
-        end
+      def yaml_command
+        add_lib 'yaml'
+      end
 
+      def validations_command
+        add_lib 'validations'
       end
 
     private
@@ -89,7 +97,7 @@ module Bashly
       end
 
       def safe_write(path, content)
-        if !Dir.exist? Settings.source_dir
+        if !skip_src_check and !Dir.exist? Settings.source_dir
           raise InitError, "Directory !txtgrn!#{Settings.source_dir}!txtrst! does not exist\nRun !txtpur!bashly init!txtrst! first"
         end
 

data/lib/bashly/commands/base.rb

@@ -6,10 +6,28 @@ module Bashly
     class Base < MisterBin::Command
       include AssetHelper
 
+      def config
+        @config ||= Config.new "#{Settings.source_dir}/bashly.yml"
+      end
+
+      def config_validator
+        @config_validator ||= ConfigValidator.new config
+      end
+
       def validate_config
-        config = Config.new "#{Settings.source_dir}/bashly.yml"
-        validator = ConfigValidator.new config
-        validator.validate
+        config_validator.validate
+      end
+
+      def with_valid_config
+        validate_config
+        yield
+        show_deprecations
+      end
+
+      def show_deprecations
+        return if config_validator.deprecations.empty? or ENV['BASHLY_HIDE_DEPRECATIONS']
+        messages = "\n" + config_validator.deprecations.map(&:message).join("\n\n") + "\n\n"
+        say! messages
       end
     end
   end

data/lib/bashly/commands/generate.rb

@@ -16,6 +16,7 @@ module Bashly
       environment "BASHLY_TARGET_DIR", "The path to use for creating the bash script [default: .]"
       environment "BASHLY_LIB_DIR", "The path to use for upgrading library files, relative to the source dir [default: lib]"
       environment "BASHLY_STRICT", "When not empty, enable bash strict mode (set -euo pipefail)"
+      environment "BASHLY_TAB_INDENT", "When not empty, the generated script will use tab indentation instead of spaces (every 2 leading spaces will be converted to a tab character)"
       environment "BASHLY_ENV", <<~EOF
         Set to 'production' or 'development':
         - production    generate a smaller script, without file markers
@@ -28,13 +29,12 @@ module Bashly
       example "bashly generate --wrap my_function"
 
       def run
-        validate_config
-        ENV['BASHLY_ENV'] = args['--env'] if args['--env']
-        quiet_say "creating !txtgrn!production!txtrst! version" if Bashly.production?
-        create_user_files
-        upgrade_libs if args['--upgrade']
-        create_master_script
-        quiet_say "run !txtpur!#{master_script_path} --help!txtrst! to test your bash script"
+        with_valid_config do
+          Settings.env = args['--env'] if args['--env']
+          quiet_say "creating !txtgrn!production!txtrst! version" if Settings.production?
+          generate_all_files
+          quiet_say "run !txtpur!#{master_script_path} --help!txtrst! to test your bash script"
+        end
       end
 
     private
@@ -43,6 +43,12 @@ module Bashly
         say message unless args['--quiet']
       end
 
+      def generate_all_files
+        create_user_files
+        upgrade_libs if args['--upgrade']
+        create_master_script
+      end
+
       def upgrade_libs
         generated_files.each do |file|
           content = File.read file
@@ -114,7 +120,7 @@ module Bashly
       end
 
       def create_master_script
-        File.write master_script_path, script.code
+        File.write master_script_path, script.code(tab_indent: Settings.tab_indent)
         FileUtils.chmod "+x", master_script_path
         quiet_say "!txtgrn!created!txtrst! #{master_script_path}"
       end
@@ -127,10 +133,6 @@ module Bashly
         "#{Settings.target_dir}/#{command.name}"
       end
 
-      def config
-        @config ||= Config.new "#{Settings.source_dir}/bashly.yml"
-      end
-
       def command
         @command ||= Script::Command.new config
       end

data/lib/bashly/commands/preview.rb

@@ -9,10 +9,11 @@ module Bashly
       environment "BASHLY_SOURCE_DIR", "The path containing the bashly configuration and source files [default: src]"
 
       def run
-        config = Config.new "#{Settings.source_dir}/bashly.yml"
-        command = Script::Command.new(config)
-        script = Script::Wrapper.new command
-        puts script.code
+        with_valid_config do
+          command = Script::Command.new config
+          script = Script::Wrapper.new command
+          puts script.code
+        end
       end
     end
   end

data/lib/bashly/commands/validate.rb

@@ -10,7 +10,13 @@ module Bashly
 
       def run
         validate_config
-        say "!txtgrn!OK"
+        show_deprecations
+        deprecations = config_validator.deprecations
+        if deprecations.empty?
+          say "!txtgrn!OK"
+        else
+          say "!txtred!WARNING!txtrst! Found #{deprecations.count} deprecations"
+        end
       end
     end
   end

data/lib/bashly/concerns/command_scopes.rb

@@ -1,6 +1,21 @@
 module Bashly
   # This is a `Command` concern responsible for providing additional scopes.
   module CommandScopes
+    # Returns an array of all full names (including aliases and aliases of
+    # parents)
+    def all_full_names
+      if parent_command
+        parent_command.all_full_names.product(aliases).map { |a| a.join ' ' }
+      else
+        aliases
+      end
+    end
+
+    # Returns a full list of the Command names and aliases combined
+    def command_aliases
+      commands.map(&:aliases).flatten
+    end
+
     # Returns only the names of the Commands
     def command_names
       commands.map &:name

data/lib/bashly/concerns/completions.rb

@@ -1,51 +1,76 @@
 require 'completely'
 
 module Bashly
-  # This is a `Command` concern responsible for providing bash completion data
+  # This is a `Command` and `Flag` concern responsible for providing bash
+  # completion data
   module Completions
-    def completion_data(with_version: true)
-      result = { full_name => completion_words(with_version: with_version) }
-      
-      commands.each do |command|
-        result.merge! command.completion_data(with_version: false)
-      end
+    module Flag
+      def completion_data(command_full_name)
+        result = {}
+        comps = allowed || completions
 
-      result
-    end
+        if comps
+          aliases.each do |name|
+            result["#{command_full_name}*#{name}"] = comps
+          end
+        end
 
-    def completion_script
-      completion_generator.script
+        result
+      end
     end
 
-    def completion_function(name = nil)
-      completion_generator.wrapper_function(name)
-    end
+    module Command
+      def completion_data(with_version: true)
+        result = {}
 
-  private
+        all_full_names.each do |name|
+          result[name] = completion_words(with_version: with_version)
+          flags.each do |flag|
+            result.merge! flag.completion_data(name)
+          end
+        end
+        
+        commands.each do |command|
+          result.merge! command.completion_data(with_version: false)
+        end
 
-    def completion_generator
-      Completely::Completions.new(completion_data)
-    end
+        result
+      end
 
-    def completion_flag_names
-      flags.map(&:name) + flags.map(&:short)
-    end
+      def completion_script
+        completion_generator.script
+      end
 
-    def completion_allowed_args
-      flags.map(&:allowed).flatten + args.map(&:allowed).flatten
-    end
+      def completion_function(name = nil)
+        completion_generator.wrapper_function(name)
+      end
 
-    def completion_words(with_version: false)
-      trivial_flags = %w[--help -h]
-      trivial_flags += %w[--version -v] if with_version
-      all = (
-        command_names + trivial_flags +
-        completion_flag_names + completion_allowed_args
-      )
+    private
 
-      all += completions if completions
-      all.compact.uniq.sort
-    end
+      def completion_generator
+        Completely::Completions.new(completion_data)
+      end
+
+      def completion_flag_names
+        flags.map(&:name) + flags.map(&:short)
+      end
+
+      def completion_allowed_args
+        args.map(&:allowed).flatten
+      end
+
+      def completion_words(with_version: false)
+        trivial_flags = %w[--help -h]
+        trivial_flags += %w[--version -v] if with_version
+        all = (
+          command_aliases + trivial_flags + 
+          completion_flag_names + completion_allowed_args
+        )
 
+        all += completions if completions
+        all.compact.uniq.sort
+      end
+
+    end
   end
-end
+end

data/lib/bashly/concerns/renderable.rb

@@ -15,7 +15,7 @@ module Bashly
 
     def view_marker(id = nil)
       id ||= ":#{caller_locations.first.path}"
-      "# #{id}" unless Bashly.production?
+      "# #{id}" unless Settings.production?
     end
 
   private

data/lib/bashly/concerns/validation_helpers.rb

@@ -0,0 +1,75 @@
+module Bashly
+  # This is a `ConfigValidator` concern responsible for providing basic
+  # assertion methods.
+  module ValidationHelpers
+
+    def deprecations
+      @deprecations ||= []
+    end
+
+  protected
+
+    def assert(valid, message)
+      raise ConfigurationError, message unless valid
+    end
+
+    def refute(invalid, message)
+      assert !invalid, message
+    end
+
+    def deprecate(key, **options)
+      deprecations.push Deprecation.new(key, **options)
+    end
+
+    def assert_string(key, value)
+      assert value.is_a?(String), "#{key} must be a string"
+    end
+
+    def assert_optional_string(key, value)
+      assert_string key, value if value
+    end
+
+    def assert_boolean(key, value)
+      assert [true, false, nil].include?(value), "#{key} must be a boolean" 
+    end
+
+    def assert_array(key, value, of: nil)
+      return unless value
+      assert value.is_a?(Array), "#{key} must be an array"
+      if of
+        value.each_with_index do |val, i|
+          send "assert_#{of}".to_sym, "#{key}[#{i}]", val
+        end
+      end
+    end
+
+    def assert_hash(key, value, whitelist = nil)
+      assert value.is_a?(Hash), "#{key} must be a hash"
+      
+      if whitelist
+        invalid_keys = value.keys.map(&:to_sym) - whitelist
+        assert invalid_keys.empty?, "#{key} contains invalid options: #{invalid_keys.join(', ')}"
+      end
+    end
+
+    def assert_uniq(key, value, array_keys)
+      return unless value
+      array_keys = [array_keys] unless array_keys.is_a? Array
+      list = []
+      array_keys.each do |array_key|
+        list += value.map { |c| c[array_key] }.compact.flatten
+      end
+
+      nonuniqs = list.nonuniq
+      assert nonuniqs.empty?, "#{key} contains non-unique elements (#{nonuniqs.join ', '}) in #{array_keys.join ' or '}"
+    end
+
+    def assert_string_or_array(key, value)
+      return unless value
+      assert [Array, String].include?(value.class),
+        "#{key} must be a string or an array"
+
+      assert_array key, value, of: :string if value.is_a? Array
+    end
+  end
+end

data/lib/bashly/config_validator.rb

@@ -1,5 +1,7 @@
 module Bashly
   class ConfigValidator
+    include ValidationHelpers
+
     attr_reader :data
 
     def initialize(data)
@@ -12,45 +14,6 @@ module Bashly
 
   private
 
-    def assert(valid, message)
-      raise ConfigurationError, message unless valid
-    end
-
-    def refute(invalid, message)
-      assert !invalid, message
-    end
-
-    def assert_string(key, value)
-      assert value.is_a?(String), "#{key} must be a string"
-    end
-
-    def assert_optional_string(key, value)
-      assert_string key, value if value
-    end
-
-    def assert_boolean(key, value)
-      assert [true, false, nil].include?(value), "#{key} must be a boolean" 
-    end
-
-    def assert_array(key, value, of: nil)
-      return unless value
-      assert value.is_a?(Array), "#{key} must be an array"
-      if of
-        value.each_with_index do |val, i|
-          send "assert_#{of}".to_sym, "#{key}[#{i}]", val
-        end
-      end
-    end
-
-    def assert_hash(key, value, whitelist = nil)
-      assert value.is_a?(Hash), "#{key} must be a hash"
-      
-      if whitelist
-        invalid_keys = value.keys.map(&:to_sym) - whitelist
-        assert invalid_keys.empty?, "#{key} contains invalid options: #{invalid_keys.join(', ')}"
-      end
-    end
-
     def assert_version(key, value)
       return unless value
       assert [String, Integer, Float].include?(value.class),
@@ -98,6 +61,8 @@ module Bashly
       assert_hash key, value, Script::Flag.option_keys
       assert value['short'] || value['long'], "#{key} must have at least one of long or short name"
 
+      refute value['allowed'] && value['completions'], "#{key} cannot have both allowed and completions"
+
       assert_optional_string "#{key}.long", value['long']
       assert_optional_string "#{key}.short", value['short']
       assert_optional_string "#{key}.help", value['help']
@@ -109,6 +74,7 @@ module Bashly
       assert_boolean "#{key}.required", value['required']
       assert_array "#{key}.allowed", value['allowed'], of: :string
       assert_array "#{key}.conflicts", value['conflicts'], of: :string
+      assert_array "#{key}.completions", value['completions'], of: :string
 
       assert value['long'].match(/^--[a-zA-Z0-9_\-]+$/), "#{key}.long must be in the form of '--name'" if value['long']
       assert value['short'].match(/^-[a-zA-Z0-9]$/), "#{key}.short must be in the form of '-n'" if value['short']
@@ -123,6 +89,10 @@ module Bashly
       if value['allowed']
         assert value['arg'], "#{key}.allowed does not make sense without arg"
       end
+
+      if value['completions']
+        assert value['arg'], "#{key}.completions does not make sense without arg"
+      end
     end
 
     def assert_env_var(key, value)
@@ -138,9 +108,8 @@ module Bashly
 
       refute value['commands'] && value['args'], "#{key} cannot have both commands and args"
       refute value['commands'] && value['flags'], "#{key} cannot have both commands and flags"
-      
+
       assert_string "#{key}.name", value['name']
-      assert_optional_string "#{key}.short", value['short']
       assert_optional_string "#{key}.help", value['help']
       assert_optional_string "#{key}.footer", value['footer']
       assert_optional_string "#{key}.group", value['group']
@@ -150,6 +119,7 @@ module Bashly
       assert_boolean "#{key}.default", value['default']
       assert_version "#{key}.version", value['version']
       assert_catch_all "#{key}.catch_all", value['catch_all']
+      assert_string_or_array "#{key}.alias", value['alias']
       assert_extensible "#{key}.extensible", value['extensible']
       
       assert_array "#{key}.args", value['args'], of: :arg
@@ -161,8 +131,18 @@ module Bashly
       assert_array "#{key}.environment_variables", value['environment_variables'], of: :env_var
       assert_array "#{key}.examples", value['examples'], of: :string
 
+      assert_uniq "#{key}.commands", value['commands'], ['name', 'alias']
+      assert_uniq "#{key}.flags", value['flags'], 'long'
+      assert_uniq "#{key}.flags", value['flags'], 'short'
+      assert_uniq "#{key}.args", value['args'], 'name'
+
+      if value['catch_all'] and value['args']
+        repeatable_arg = value['args'].select { |a| a['repeatable'] }.first&.dig 'name'
+        refute repeatable_arg, "#{key}.catch_all makes no sense with repeatable arg (#{repeatable_arg})"
+      end
+
       if key == "root"
-        refute value['short'], "#{key}.short makes no sense"
+        refute value['alias'], "#{key}.alias makes no sense"
         refute value['group'], "#{key}.group makes no sense"
         refute value['default'], "#{key}.default makes no sense"
         refute value['private'], "#{key}.private makes no sense"
@@ -170,6 +150,11 @@ module Bashly
         refute value['version'], "#{key}.version makes no sense"
         refute value['extensible'], "#{key}.extensible makes no sense"
       end
+
+      # DEPRECATION 0.8.0
+      if value['short']
+        deprecate "#{key}.short", replacement: "alias", reference: "https://github.com/DannyBen/bashly/pull/220"
+      end
     end
   end
 end

data/lib/bashly/deprecation.rb

@@ -0,0 +1,25 @@
+module Bashly
+  class Deprecation
+    attr_reader :old, :replacement, :reference
+
+    def initialize(old, replacement: nil, reference: nil)
+      @old, @replacement, @reference = old, replacement, reference
+    end
+
+    def message
+      result = ["Deprecation Warning:", "!txtred!#{old}!txtrst! is deprecated"]
+      result.push "use !txtgrn!#{replacement}!txtrst! instead" if replacement
+      result.push "see !undblu!#{reference}!txtrst!" if reference
+      
+      result.map { |line| "!txtred!▐!txtrst! #{line}"}.join("\n")
+    end
+
+    def to_h
+      {
+        old: old,
+        replacement: replacement,
+        reference: reference
+      }
+    end
+  end
+end

data/lib/bashly/extensions/array.rb

@@ -4,4 +4,9 @@ class Array
     indentation = " " * offset
     map { |line| "#{indentation}#{line}" }
   end
+
+  def nonuniq
+    tally.select { |key, count| count > 1 }.keys
+  end
+
 end

data/lib/bashly/extensions/string.rb

@@ -31,4 +31,10 @@ class String
     split(/^---\s*/).last
   end
 
+  def expand_tabs(tabstop = 2)
+    gsub(/^( {#{tabstop}}+)/) do
+      "\t" * ($1.size / tabstop)
+    end
+  end
+
 end

data/lib/bashly/libraries.yml

@@ -3,21 +3,25 @@ colors:
     - source: "templates/lib/colors.sh"
       target: "%{user_lib_dir}/colors.sh"
 
+completions: :CompletionsFunction
+completions_script: :CompletionsScript
+completions_yaml: :CompletionsYAML
+
 config:
   files:
     - source: "templates/lib/config.sh"
       target: "%{user_lib_dir}/config.sh"
 
-yaml:
-  files:
-    - source: "templates/lib/yaml.sh"
-      target: "%{user_lib_dir}/yaml.sh"
-
 lib:
   files:
     - source: "templates/lib/sample_function.sh"
       target: "%{user_lib_dir}/sample_function.sh"
 
+settings:
+  files:
+    - source: "templates/settings.yml"
+      target: "settings.yml"
+
 strings:
   files:
     - source: "templates/strings.yml"
@@ -37,7 +41,6 @@ test:
 
     Docs: !undblu!https://github.com/DannyBen/approvals.bash
 
-
 validations:
   files:
     - source: "templates/lib/validations/validate_dir_exists.sh"
@@ -49,6 +52,7 @@ validations:
     - source: "templates/lib/validations/validate_not_empty.sh"
       target: "%{user_lib_dir}/validations/validate_not_empty.sh"
 
-completions: :CompletionsFunction
-completions_script: :CompletionsScript
-completions_yaml: :CompletionsYAML
+yaml:
+  files:
+    - source: "templates/lib/yaml.sh"
+      target: "%{user_lib_dir}/yaml.sh"

data/lib/bashly/script/command.rb

@@ -1,18 +1,20 @@
 module Bashly
   module Script
     class Command < Base
-      include Completions
+      include Completions::Command
       include CommandScopes
 
       class << self
         def option_keys
           @option_keys ||= %i[
-            args catch_all commands completions
+            alias args catch_all commands completions
             default dependencies environment_variables examples
             extensible filename filters flags
             footer group help name
-            private short version
+            private version
+            short
           ]
+          # DEPRECATION 0.8.0
         end
       end
 
@@ -27,7 +29,15 @@ module Bashly
 
       # Returns all the possible aliases for this command
       def aliases
-        short ? [name, short] : [name]
+        [name] + alt
+      end
+
+      # Returns an array of alternative aliases if any
+      def alt
+        # DEPRECATION 0.8.0
+        options['alias'] ||= options['short']
+        return [] unless options["alias"]
+        options['alias'].is_a?(String) ? [options['alias']] : options['alias']
       end
 
       # Returns an array of Arguments
@@ -40,7 +50,7 @@ module Bashly
 
       # Returns a string suitable to be a headline
       def caption_string
-        help ? "#{full_name} - #{summary}" : full_name
+        help.empty? ? full_name : "#{full_name} - #{summary}"
       end
 
       def catch_all
@@ -52,6 +62,7 @@ module Bashly
         return [] unless options["commands"]
         options["commands"].map do |options|
           options['parents'] = parents + [name]
+          options['parent_command'] = self
           Command.new options
         end
       end
@@ -94,15 +105,21 @@ module Bashly
       # If the file is not found, returns a string with a hint.
       def load_user_file(file, placeholder: true)
         path = "#{Settings.source_dir}/#{file}"
-        default_content = placeholder ? "echo \"error: cannot load file\"" : ''
 
         content = if File.exist? path
           File.read(path).remove_front_matter
-        else 
-          default_content
+        elsif placeholder
+          %q[echo "error: cannot load file"]
+        else
+          ''
         end
 
-        Bashly.production? ? content : "#{view_marker path}\n#{content}"
+        Settings.production? ? content : "#{view_marker path}\n#{content}"
+      end
+
+      # Returns the Command instance of the direct parent
+      def parent_command
+        options['parent_command']
       end
 
       # Returns an array of all parents. For example, the command 
@@ -111,7 +128,12 @@ module Bashly
         options['parents'] || []
       end
 
-      # Returns trus if this is the root command (no parents)
+      # Returns true if one of the args is repeatable
+      def repeatable_arg_exist?
+        args.select(&:repeatable).any?
+      end
+
+      # Returns true if this is the root command (no parents)
       def root_command?
         parents.empty?
       end

data/lib/bashly/script/flag.rb

@@ -1,11 +1,13 @@
 module Bashly
   module Script
     class Flag < Base
+      include Completions::Flag
+
       class << self
         def option_keys
           @option_keys ||= %i[
-            allowed arg conflicts default help long repeatable required
-            short validate
+            allowed arg completions conflicts default help long repeatable
+            required short validate
           ]
         end
       end

data/lib/bashly/script/wrapper.rb

@@ -9,18 +9,22 @@ module Bashly
         @command, @function_name = command, function_name
       end
 
-      def code
-        if function_name
-          result = [header, render('wrapper')].join "\n"
+      def code(tab_indent: false)
+        tab_indent ? base_code.expand_tabs : base_code
+      end
+
+    private
+
+      def base_code
+        result = if function_name
+          [header, render('wrapper')]
         else
-          result = [header, body].join "\n"
+          [header, body]
         end
 
-        result.lint
+        result.join("\n").lint
       end
 
-    private
-
       def header
         @header ||= header!
       end

data/lib/bashly/settings.rb

@@ -1,27 +1,56 @@
 module Bashly
   class Settings
     class << self
-      attr_writer :source_dir, :target_dir, :lib_dir, :strict
+      attr_writer :source_dir, :target_dir, :lib_dir, :strict, :tab_indent
 
       def source_dir
-        @source_dir ||= ENV['BASHLY_SOURCE_DIR'] || 'src'
+        @source_dir ||= get :source_dir, 'src'
       end
 
       def target_dir
-        @target_dir ||= ENV['BASHLY_TARGET_DIR'] || '.'
+        @target_dir ||= get :target_dir, '.'
       end
 
       def lib_dir
-        @lib_dir ||= ENV['BASHLY_LIB_DIR'] || 'lib'
+        @lib_dir ||= get :lib_dir, 'lib'
       end
 
       def strict
-        @strict ||= ENV['BASHLY_STRICT']
+        @strict ||= get :strict
+      end
+
+      def tab_indent
+        @tab_indent ||= get :tab_indent
+      end
+
+      def env
+        @env ||= get(:env, :development)&.to_sym
+      end
+
+      def env=(value)
+        @env = value&.to_sym
+      end
+
+      def production?
+        env == :production
       end
 
       def full_lib_dir
         "#{source_dir}/#{lib_dir}"
       end
+
+    private
+
+      def get(key, default = nil)
+        ENV["BASHLY_#{key.upcase}"] || user_settings[key.to_s] || default
+      end
+
+      def user_settings
+        @user_settings ||= begin
+          File.exist?('settings.yml') ? Config.new('settings.yml') : {}
+        end
+      end
+
     end
   end
 end

data/lib/bashly/templates/bashly.yml

@@ -8,7 +8,7 @@ environment_variables:
 
 commands:
 - name: download
-  short: d
+  alias: d
   help: Download a file
 
   args:
@@ -32,7 +32,7 @@ commands:
     help: Set the default location to download to
 
 - name: upload
-  short: u
+  alias: u
   help: Upload a file
   args:
   - name: source

data/lib/bashly/templates/settings.yml

@@ -0,0 +1,24 @@
+# All settings are optional (with their default values provided below), and
+# can also be set with an environment variable with the same name, capitalized
+# and prefixed by `BASHLY_` - for example: BASHLY_SOURCE_DIR
+
+# The path containing the bashly configuration and source files
+source_dir: src
+
+# The path to use for creating the bash script
+target_dir: .
+
+# The path to use for upgrading library files, relative to the source dir
+lib_dir: lib
+
+# When true, enable bash strict mode (set -euo pipefail)
+strict: false
+
+# When true, the generated script will use tab indentation instead of spaces
+# (every 2 leading spaces will be converted to a tab character)
+tab_indent: false
+
+# Set to 'production' or 'development':
+# - production    generate a smaller script, without file markers
+# - development   generate with file markers
+env: development

data/lib/bashly/templates/strings.yml

@@ -11,7 +11,7 @@ environment_variables: "Environment Variables:"
 group: "%{group} Commands:"
 
 # Usage helpers
-command_shortcut: "Shortcut: %{short}"
+command_alias: "Alias: %{alias}"
 default_command_summary: "%{summary} (default)"
 required: "(required)"
 repeatable: "(repeatable)"

data/lib/bashly/version.rb

@@ -1,3 +1,3 @@
 module Bashly
-  VERSION = "0.7.9"
+  VERSION = "0.8.1"
 end

data/lib/bashly/views/command/parse_requirements_case.erb

@@ -1,38 +1,8 @@
 <%= view_marker %>
-% repeatable_arg = false
-% if args.any?
-% condition = "if"
-% args.each do |arg|
-<%= condition %> [[ -z ${args[<%= arg.name %>]+x} ]]; then
-<%= arg.render(:validations).indent 2 %>
-% if arg.repeatable
-% repeatable_arg = true
-  args[<%= arg.name %>]="\"$1\""
-  shift
-else
-  args[<%= arg.name %>]="${args[<%= arg.name %>]} \"$1\""
-  shift
-% else
-  args[<%= arg.name %>]=$1
-  shift
-% end
-% condition = "elif"
-% end
-% if !repeatable_arg
-else
-% end
 % if catch_all.enabled?
-  other_args+=("$1")
-  shift
-% elsif !repeatable_arg
-  printf "<%= strings[:invalid_argument] %>\n" "$key"
-  exit 1
-% end
-fi
-% elsif catch_all.enabled?
-  other_args+=("$1")
-  shift
-% elsif !repeatable_arg
-printf "<%= strings[:invalid_argument] %>\n" "$key"
-exit 1
+<%= render(:parse_requirements_case_catch_all) %>
+% elsif repeatable_arg_exist?
+<%= render(:parse_requirements_case_repeatable) %>
+% else
+<%= render(:parse_requirements_case_simple) %>
 % end

data/lib/bashly/views/command/parse_requirements_case_catch_all.erb

@@ -0,0 +1,18 @@
+<%= view_marker %>
+% if args.any?
+% condition = "if"
+% args.each do |arg|
+<%= condition %> [[ -z ${args[<%= arg.name %>]+x} ]]; then
+<%= arg.render(:validations).indent 2 %>
+  args[<%= arg.name %>]=$1
+  shift
+% condition = "elif"
+% end
+else
+  other_args+=("$1")
+  shift
+fi
+% else
+other_args+=("$1")
+shift
+% end

data/lib/bashly/views/command/parse_requirements_case_repeatable.erb

@@ -0,0 +1,18 @@
+<%= view_marker %>
+% condition = "if"
+% args.each do |arg|
+<%= condition %> [[ -z ${args[<%= arg.name %>]+x} ]]; then
+<%= arg.render(:validations).indent 2 %>
+% if arg.repeatable
+  args[<%= arg.name %>]="\"$1\""
+  shift
+else
+  args[<%= arg.name %>]="${args[<%= arg.name %>]} \"$1\""
+  shift
+% else
+  args[<%= arg.name %>]=$1
+  shift
+% end
+% condition = "elif"
+% end
+fi

data/lib/bashly/views/command/parse_requirements_case_simple.erb

@@ -0,0 +1,18 @@
+<%= view_marker %>
+% if args.any?
+% condition = "if"
+% args.each do |arg|
+<%= condition %> [[ -z ${args[<%= arg.name %>]+x} ]]; then
+<%= arg.render(:validations).indent 2 %>
+  args[<%= arg.name %>]=$1
+  shift
+% condition = "elif"
+% end
+else
+  printf "<%= strings[:invalid_argument] %>\n" "$key"
+  exit 1
+fi
+% else
+printf "<%= strings[:invalid_argument] %>\n" "$key"
+exit 1
+% end

data/lib/bashly/views/command/usage.erb

@@ -15,8 +15,8 @@
     echo 
   fi
 
-  <%- if short -%>
-  printf "<%= strings[:command_shortcut] % { short: short } %>\n"
+  <%- if alt&.any? -%>
+  printf "<%= strings[:command_alias] % { alias: alt.join(', ') } %>\n"
   echo
   <%- end -%>
 

data/lib/bashly/views/flag/case.erb

@@ -1,30 +1,4 @@
 <%= view_marker %>
 <%= aliases.join " | " %> )
 <%= render(:conflicts).indent 2 %>
-% if arg
-  if [[ -n ${2+x} ]]; then
-<%= render(:validations).indent 4 %>
-% if repeatable
-    if [[ -z ${args[<%= name %>]+x} ]]; then
-      args[<%= name %>]="\"$2\""
-    else
-      args[<%= name %>]="${args[<%= name %>]} \"$2\""
-    fi
-% else
-    args[<%= name %>]="$2"
-% end
-    shift
-    shift
-  else
-    printf "%s\n" "<%= strings[:flag_requires_an_argument] % { name: name, usage: usage_string } %>"
-    exit 1
-  fi
-% else
-% if repeatable
-  (( args[<%= name %>]+=1 ))
-% else
-  args[<%= name %>]=1
-% end
-  shift
-% end
-  ;;
+<%= render(arg ? :case_arg : :case_no_arg).indent 2 %>

data/lib/bashly/views/flag/case_arg.erb

@@ -0,0 +1,19 @@
+<%= view_marker %>
+if [[ -n ${2+x} ]]; then
+<%= render(:validations).indent 2 %>
+% if repeatable
+  if [[ -z ${args[<%= name %>]+x} ]]; then
+    args[<%= name %>]="\"$2\""
+  else
+    args[<%= name %>]="${args[<%= name %>]} \"$2\""
+  fi
+% else
+  args[<%= name %>]="$2"
+% end
+  shift
+  shift
+else
+  printf "%s\n" "<%= strings[:flag_requires_an_argument] % { name: name, usage: usage_string } %>"
+  exit 1
+fi
+;;

data/lib/bashly/views/flag/case_no_arg.erb

@@ -0,0 +1,8 @@
+<%= view_marker %>
+% if repeatable
+(( args[<%= name %>]+=1 ))
+% else
+args[<%= name %>]=1
+% end
+shift
+;;

data/lib/bashly.rb

@@ -15,15 +15,3 @@ requires 'bashly/script/base'
 requires 'bashly/commands/base'
 requires 'bashly/libraries/base'
 requires 'bashly'
-
-module Bashly
-  class << self
-    def env
-      ENV['BASHLY_ENV']&.to_sym
-    end
-
-    def production?
-      env == :production
-    end
-  end
-end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bashly
 version: !ruby/object:Gem::Version
-  version: 0.7.9
+  version: 0.8.1
 platform: ruby
 authors:
 - Danny Ben Shitrit
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-04-02 00:00:00.000000000 Z
+date: 2022-05-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: colsole
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.6'
+        version: '0.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.6'
+        version: '0.7'
 - !ruby/object:Gem::Dependency
   name: completely
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.3'
+        version: 0.4.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.3'
+        version: 0.4.1
 - !ruby/object:Gem::Dependency
   name: mister_bin
   requirement: !ruby/object:Gem::Requirement
@@ -58,14 +58,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: '0.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: '0.2'
 description: Generate bash command line tools using YAML configuration
 email: db@dannyben.com
 executables:
@@ -87,8 +87,10 @@ files:
 - lib/bashly/concerns/command_scopes.rb
 - lib/bashly/concerns/completions.rb
 - lib/bashly/concerns/renderable.rb
+- lib/bashly/concerns/validation_helpers.rb
 - lib/bashly/config.rb
 - lib/bashly/config_validator.rb
+- lib/bashly/deprecation.rb
 - lib/bashly/exceptions.rb
 - lib/bashly/extensions/array.rb
 - lib/bashly/extensions/file.rb
@@ -121,6 +123,7 @@ files:
 - lib/bashly/templates/lib/validations/validate_not_empty.sh
 - lib/bashly/templates/lib/yaml.sh
 - lib/bashly/templates/minimal.yml
+- lib/bashly/templates/settings.yml
 - lib/bashly/templates/strings.yml
 - lib/bashly/templates/test/approvals.bash
 - lib/bashly/version.rb
@@ -145,6 +148,9 @@ files:
 - lib/bashly/views/command/normalize_input.erb
 - lib/bashly/views/command/parse_requirements.erb
 - lib/bashly/views/command/parse_requirements_case.erb
+- lib/bashly/views/command/parse_requirements_case_catch_all.erb
+- lib/bashly/views/command/parse_requirements_case_repeatable.erb
+- lib/bashly/views/command/parse_requirements_case_simple.erb
 - lib/bashly/views/command/parse_requirements_while.erb
 - lib/bashly/views/command/required_args_filter.erb
 - lib/bashly/views/command/required_flags_filter.erb
@@ -163,6 +169,8 @@ files:
 - lib/bashly/views/command/whitelist_filter.erb
 - lib/bashly/views/environment_variable/usage.erb
 - lib/bashly/views/flag/case.erb
+- lib/bashly/views/flag/case_arg.erb
+- lib/bashly/views/flag/case_no_arg.erb
 - lib/bashly/views/flag/conflicts.erb
 - lib/bashly/views/flag/usage.erb
 - lib/bashly/views/flag/validations.erb