Disallow multiple repeatable args, and a repeatable arg that is not the last one

lib/bashly/config_validator.rb

@@ -207,6 +207,16 @@ def assert_command(key, value)
  refute repeatable_arg, "#{key}.catch_all makes no sense with repeatable arg (#{repeatable_arg})"
  end
 
+ if value['args']
+ repeatable_args = value['args'].count { |a| a['repeatable'] }
+ assert repeatable_args < 2, "#{key}.args cannot have more than one repeatable args"
+
+ if repeatable_args == 1
+ assert value['args'].last['repeatable'],
+ "#{key}.args cannot contain a repeatable arg unless it is the last one"
+ end
+ end
+
  if value['expose']
  assert value['commands'], "#{key}.expose makes no sense without nub`commands`"
  end

spec/approvals/validations/arg_repeatable_more_than_one

@@ -0,0 +1 @@
+#<Bashly::ConfigurationError: root.args cannot have more than one repeatable args>

spec/approvals/validations/arg_repeatable_not_last

@@ -0,0 +1 @@
+#<Bashly::ConfigurationError: root.args cannot contain a repeatable arg unless it is the last one>

spec/fixtures/script/validations.yml

@@ -19,6 +19,29 @@
  - name: ok
  - help: there is no name
 
+:arg_repeatable_more_than_one:
+ name: invalid
+ help: can only have one repeatable arg
+ args:
+ - name: source
+ required: true
+ repeatable: true
+ help: URL to download from
+ - name: target
+ repeatable: true
+ help: Target filename
+
+:arg_repeatable_not_last:
+ name: invalid
+ help: can the last arg can be repeatable
+ args:
+ - name: source
+ required: true
+ repeatable: true
+ help: URL to download from
+ - name: target
+ help: Target filename
+
 :command_catch_all_type:
  name: invalid
  help: catch_all must be boolean, string, or hash