Merge pull request #339 from DannyBen/update/libraries

Refactor libraries

Author: DannyBen

examples/completions/README.md

@@ -7,7 +7,7 @@ This example was generated with:
 ```bash
 $ bashly init
 # ... now edit src/bashly.yml to match the example ...
-$ bashly add comp function
+$ bashly add completions
 $ bashly generate
 # ... now edit src/completions_command.sh ...
 $ bashly generate
@@ -112,7 +112,7 @@ commands:
 ```bash
 # Call the `send_completions` function which was added by running:
 #
-#   $ bashly add comp function
+#   $ bashly add completions
 #
 # Users can now enable bash completion for this script by running:
 #

examples/completions/src/completions_command.sh

@@ -1,6 +1,6 @@
 # Call the `send_completions` function which was added by running:
 #
-#   $ bashly add comp function
+#   $ bashly add completions
 #
 # Users can now enable bash completion for this script by running:
 #

examples/completions/test.sh

@@ -2,7 +2,7 @@
 
 set -x
 
-bashly add comp function --force
+bashly add completions --force
 bashly generate
 
 ### Try Me ###

lib/bashly/commands/add.rb

@@ -3,116 +3,56 @@ module Commands
     class Add < Base
       help 'Add extra features and customization to your script'
 
-      usage 'bashly add colors [--force]'
-      usage 'bashly add comp FORMAT [OUTPUT --force]'
-      usage 'bashly add config [--force]'
-      usage 'bashly add help [--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 LIBRARY [ARGS...] [--force]'
+      usage 'bashly add --list'
       usage 'bashly add (-h|--help)'
 
       option '-f --force', 'Overwrite existing files'
-
-      param 'FORMAT', <<~USAGE
-        Output format, can be one of:
-          function : generate a function file to be included in your script.
-          script   : generate a standalone bash completions script.
-          yaml     : generate a yaml compatible with completely.
-      USAGE
-
-      param 'OUTPUT', <<~USAGE
-        For the 'comp function' command: Name of the generated function.
-        For the 'comp script' or 'comp yaml' commands: path to output file.
-        In all cases, this is optional and will have sensible defaults.
-      USAGE
-
-      command 'colors', 'Add standard functions for printing colorful and formatted text to the lib directory.'
-      command 'comp', 'Generate a bash completions script or function.'
-      command 'config', 'Add standard functions for handling INI files to the lib directory.'
-      command 'help', 'Add a help command, in addition to the standard --help flag.'
-      command 'lib', <<~USAGE
-        Create the lib directory for any additional user scripts.
-        All *.sh scripts in this directory will be included in the final bash script.
-        Note that if you configured a different partials_extension, then the extensions of the files in this directory need to match.
-      USAGE
-
-      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'
-      example 'bashly add comp script completions.bash'
+      option '-l --list', 'Show available libraries'
 
       attr_reader :skip_src_check
 
-      def colors_command
-        add_lib 'colors'
-      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}"
+      def run
+        if args['--list']
+          show_list
+        else
+          add_lib args['LIBRARY']
         end
       end
 
-      def config_command
-        add_lib 'config'
-      end
-
-      def lib_command
-        add_lib 'lib'
-      end
-
-      def settings_command
-        @skip_src_check = true
-        add_lib 'settings'
-      end
+    private
 
-      def strings_command
-        add_lib 'strings'
+      def lib_source
+        @lib_source ||= Bashly::LibrarySource.new
       end
 
-      def test_command
-        add_lib 'test'
+      def show_list
+        lib_source.config.each do |key, config|
+          usage = key
+          usage += " #{config['usage']}" if config['usage']
+          say "g`bashly add #{usage}`"
+          say word_wrap("  #{config['help']}")
+          say ''
+        end
       end
 
-      def help_command
-        add_lib 'help'
-      end
+      def add_lib(name)
+        library = lib_source.libraries[name.to_sym]
+        raise "Unknown library: g`#{name}`\nRun m`bashly add --list` to see available libraries" unless library
 
-      def validations_command
-        add_lib 'validations'
-      end
+        library.args = args['ARGS']
+        @skip_src_check = lib_source.config.dig name, 'skip_src_check'
 
-      def yaml_command
-        add_lib 'yaml'
+        add_library_files library
       end
 
-    private
-
-      def add_lib(name, *args)
-        library = Bashly::Library.new name, *args
+      def add_library_files(library)
         files_created = 0
         library.files.each do |file|
           created = safe_write file[:path], file[:content]
           files_created += 1 if created
         end
+
         message = library.post_install_message
         say "\n#{message}" if message && files_created.positive?
       end

lib/bashly/commands/generate.rb

@@ -92,16 +92,19 @@ def generated_files
       end
 
       def upgrade(existing_file, library_name, *args)
-        if Library.exist? library_name
-          upgrade! existing_file, library_name, *args
+        source = Bashly::LibrarySource.new
+        library = source.libraries[library_name.to_sym]
+
+        if library
+          library.args = args
+          upgrade! existing_file, library
         else
           quiet_say "r`warning` not upgrading c`#{existing_file}`, " \
             "unknown library '#{library_name}'"
         end
       end
 
-      def upgrade!(existing_file, library_name, *args)
-        library = Bashly::Library.new library_name, *args
+      def upgrade!(existing_file, library)
         file = library.find_file existing_file
 
         if file

lib/bashly/docs/command.yml

@@ -75,7 +75,7 @@ command.commands:
       help: Register a local repository
 
 command.completions:
-  help: Specify a list of additional completion suggestions when used in conjunction with `bashly add comp`.
+  help: Specify a list of additional completion suggestions when used in conjunction with `bashly add completions`.
   url: https://bashly.dannyb.co/configuration/command/#completions
   example: |-
     commands:

lib/bashly/docs/flag.yml

@@ -46,7 +46,7 @@ flag.arg:
       help: Clone using SSH
 
 flag.completions:
-  help: Specify a list of additional completion suggestions when used in conjunction with `bashly add comp`. Must be accompanied by `arg`.
+  help: Specify a list of additional completion suggestions when used in conjunction with `bashly add completions`. Must be accompanied by `arg`.
   url: https://bashly.dannyb.co/configuration/flag/#completions
   example: |-
     flags: