allow default as string with repeatable

Author: DannyBen

examples/repeatable-arg/README.md

@@ -36,8 +36,8 @@ args:
   # Setting unique to true will ignore non-unique repeating values.
   unique: true
 
-  # Setting default value(s) for a repeatable argument should be done in an
-  # array form.
+  # Setting default value(s) for a repeatable argument may be done in an array
+  # form (or a string form if it is a single default value only).
   default:
   - file1
   - file2

examples/repeatable-arg/src/bashly.yml

@@ -15,8 +15,8 @@ args:
   # Setting unique to true will ignore non-unique repeating values.
   unique: true
 
-  # Setting default value(s) for a repeatable argument should be done in an
-  # array form.
+  # Setting default value(s) for a repeatable argument may be done in an array
+  # form (or a string form if it is a single default value only).
   default:
   - file1
   - file2

examples/repeatable-flag/README.md

@@ -44,8 +44,8 @@ flags:
   # Setting this to true will ignore repeating arguments that are not unique.
   unique: true
 
-  # Setting default value(s) for a repeatable flag argument should be done in
-  # an array form.
+  # Setting default value(s) for a repeatable flag argument may be done in an
+  # array form (or a string form if it is a single default value only).
   default:
   - file one
   - file-two

examples/repeatable-flag/src/bashly.yml

@@ -23,8 +23,8 @@ flags:
   # Setting this to true will ignore repeating arguments that are not unique.
   unique: true
 
-  # Setting default value(s) for a repeatable flag argument should be done in
-  # an array form.
+  # Setting default value(s) for a repeatable flag argument may be done in an
+  # array form (or a string form if it is a single default value only).
   default:
   - file one
   - file-two

lib/bashly/config_validator.rb

@@ -110,10 +110,6 @@ def assert_arg(key, value)
       if value['default'].is_a? Array
         assert value['repeatable'], "#{key}.default array does not make sense without nub`repeatable`"
       end
-
-      if value['repeatable'] && value['default']
-        assert value['default'].is_a?(Array), "#{key}.default must be an array when using nub`repeatable`"
-      end
     end
 
     def assert_flag(key, value)
@@ -163,10 +159,6 @@ def assert_flag(key, value)
       if value['default'].is_a? Array
         assert value['repeatable'], "#{key}.default array does not make sense without nub`repeatable`"
       end
-
-      if value['repeatable'] && value['default']
-        assert value['default'].is_a?(Array), "#{key}.default must be an array when using nub`repeatable`"
-      end
     end
 
     def assert_env_var(key, value)

lib/bashly/script/argument.rb

@@ -12,7 +12,13 @@ def option_keys
       end
 
       def default_string
-        default.is_a?(Array) ? Shellwords.shelljoin(default) : default
+        if default.is_a?(Array)
+          Shellwords.shelljoin default
+        elsif default.is_a?(String) && repeatable
+          Shellwords.shellescape default
+        else
+          default
+        end
       end
 
       def usage_string

lib/bashly/script/flag.rb

@@ -23,7 +23,13 @@ def aliases
       end
 
       def default_string
-        default.is_a?(Array) ? Shellwords.shelljoin(default) : default
+        if default.is_a?(Array)
+          Shellwords.shelljoin default
+        elsif default.is_a?(String) && repeatable
+          Shellwords.shellescape default
+        else
+          default
+        end
       end
 
       def name

spec/approvals/validations/arg_default_string_with_repeatable

@@ -13,14 +13,6 @@
     - spaced file
     - other-file
 
-:arg_default_string_with_repeatable:
-  name: invalid
-  help: default cannot be a string with repeatable
-  args:
-  - name: path
-    repeatable: true
-    default: spaced file
-
 :arg_nested_name:
   name: invalid
   help: last nested arg does not have a name
@@ -284,15 +276,6 @@
     - spaced file
     - other-file
 
-:flag_default_string_with_repeatable:
-  name: invalid
-  help: default cannot be a string with repeatable
-  flags:
-  - long: --path
-    arg: location
-    repeatable: true
-    default: spaced file
-
 :flag_allowed_without_arg:
   name: invalid
   help: flag must have an arg when using allowed