- Refactor inspect_args handling of environment variables

Author: DannyBen

examples/command-aliases/README.md

@@ -147,6 +147,9 @@ Examples:
 args:
 - ${args[source]} = somefile
 
+environment variables:
+- $API_KEY = 
+
 
 ````
 
@@ -215,6 +218,9 @@ Examples:
 args:
 - ${args[source]} = somefile
 
+environment variables:
+- $API_KEY = 
+
 
 ````
 

examples/commands/README.md

@@ -185,6 +185,10 @@ args:
 - ${args[source]} = sourcefile
 - ${args[target]} = targetfile
 
+environment variables:
+- $API_KEY = 
+- $DEFAULT_TARGET_LOCATION = 
+
 
 ````
 
@@ -235,6 +239,9 @@ args:
 - ${args[source]} = sourcefile
 - ${args[--user]} = username
 
+environment variables:
+- $API_KEY = 
+
 
 ````
 

examples/environment-variables/README.md

@@ -46,8 +46,16 @@ commands:
 
   # Using the `default: value` option will cause the value to variable to be 
   # set if it is not provided by the user.
+  - name: region
+    help: Cloud region
+    default: us-east-2
+
+  # Using the `allowed: [value1, value2]` option will halt the script's
+  # execution with a friendly error message, unless the variable matches one
+  # of the defined values.
   - name: environment
     help: One of development, production or test
+    allowed: [development, production, testing]
     default: development
 ````
 
@@ -59,11 +67,6 @@ echo "# code for 'cli verify' goes here"
 echo "# you can edit it freely and regenerate (it will not be overwritten)"
 inspect_args
 
-echo "environment:"
-echo "- API_KEY=${API_KEY:-}"
-echo "- ENVIRONMENT=${ENVIRONMENT:-}"
-echo "- MY_SECRET=${MY_SECRET:-}"
-
 ````
 
 
@@ -133,8 +136,13 @@ Environment Variables:
   MY_SECRET (required)
     Your secret
 
+  REGION
+    Cloud region
+    Default: us-east-2
+
   ENVIRONMENT
     One of development, production or test
+    Allowed: development, production, testing
     Default: development
 
 
@@ -156,10 +164,12 @@ missing required environment variable: MY_SECRET
 # code for 'cli verify' goes here
 # you can edit it freely and regenerate (it will not be overwritten)
 args: none
-environment:
-- API_KEY=
-- ENVIRONMENT=development
-- MY_SECRET=there is no spoon
+
+environment variables:
+- $API_KEY = 
+- $ENVIRONMENT = development
+- $MY_SECRET = there is no spoon
+- $REGION = us-east-2
 
 
 ````
@@ -171,10 +181,12 @@ environment:
 # code for 'cli verify' goes here
 # you can edit it freely and regenerate (it will not be overwritten)
 args: none
-environment:
-- API_KEY=
-- ENVIRONMENT=production
-- MY_SECRET=safe-with-me
+
+environment variables:
+- $API_KEY = 
+- $ENVIRONMENT = production
+- $MY_SECRET = safe-with-me
+- $REGION = us-east-2
 
 
 ````

examples/environment-variables/src/bashly.yml

@@ -23,6 +23,14 @@ commands:
 
   # Using the `default: value` option will cause the value to variable to be 
   # set if it is not provided by the user.
+  - name: region
+    help: Cloud region
+    default: us-east-2
+
+  # Using the `allowed: [value1, value2]` option will halt the script's
+  # execution with a friendly error message, unless the variable matches one
+  # of the defined values.
   - name: environment
     help: One of development, production or test
+    allowed: [development, production, testing]
     default: development

examples/environment-variables/src/verify_command.sh

@@ -2,8 +2,3 @@ echo "# this file is located in 'src/verify_command.sh'"
 echo "# code for 'cli verify' goes here"
 echo "# you can edit it freely and regenerate (it will not be overwritten)"
 inspect_args
-
-echo "environment:"
-echo "- API_KEY=${API_KEY:-}"
-echo "- ENVIRONMENT=${ENVIRONMENT:-}"
-echo "- MY_SECRET=${MY_SECRET:-}"

examples/multiline/README.md

@@ -142,6 +142,9 @@ Examples:
 # you can edit it freely and regenerate (it will not be overwritten)
 args: none
 
+environment variables:
+- $MULTI_VITAMIN = 
+
 
 ````
 

lib/bashly/views/command/environment_variables_filter.gtx

@@ -1,24 +1,26 @@
-if default_environment_variables.any? || required_environment_variables.any? ||
-  whitelisted_environment_variables.any?
-
-  = view_marker 
+if environment_variables.any?
+  = view_marker
   = render(:environment_variables_default)
 
-  if required_environment_variables.any?
-    required_environment_variables.each do |env_var|
-      > if [[ -z "${<%= env_var.name.upcase %>:-}" ]]; then
-      >   printf "{{ strings[:missing_required_environment_variable] % { var: env_var.name.upcase } }}\n" >&2
-      >   exit 1
-      > fi
-    end
+  environment_variables.each do |env_var|
+    > env_var_names+=("{{ env_var.name.upcase }}")
+  end
+end
+
+if required_environment_variables.any?
+  required_environment_variables.each do |env_var|
+    > if [[ -z "${<%= env_var.name.upcase %>:-}" ]]; then
+    >   printf "{{ strings[:missing_required_environment_variable] % { var: env_var.name.upcase } }}\n" >&2
+    >   exit 1
+    > fi
   end
+end
 
-  if whitelisted_environment_variables.any?
-    whitelisted_environment_variables.each do |env_var|
-      > if [[ -n "${<%= env_var.name.upcase %>:-}" ]] && [[ ! ${<%= env_var.name.upcase %>:-} =~ ^({{ env_var.allowed.join '|' }})$ ]]; then
-      >   printf "%s\n" "{{ strings[:disallowed_environment_variable] % { name: env_var.name.upcase, allowed: env_var.allowed.join(', ') } }}" >&2
-      >   exit 1
-      > fi
-    end
+if whitelisted_environment_variables.any?
+  whitelisted_environment_variables.each do |env_var|
+    > if [[ -n "${<%= env_var.name.upcase %>:-}" ]] && [[ ! ${<%= env_var.name.upcase %>:-} =~ ^({{ env_var.allowed.join '|' }})$ ]]; then
+    >   printf "%s\n" "{{ strings[:disallowed_environment_variable] % { name: env_var.name.upcase, allowed: env_var.allowed.join(', ') } }}" >&2
+    >   exit 1
+    > fi
   end
-end
+end

lib/bashly/views/command/inspect_args.gtx

@@ -4,7 +4,9 @@
 >   if ((${#args[@]})); then
 >     readarray -t sorted_keys < <(printf '%s\n' "${!args[@]}" | sort)
 >     echo args:
->     for k in "${sorted_keys[@]}"; do echo "- \${args[$k]} = ${args[$k]}"; done
+>     for k in "${sorted_keys[@]}"; do
+>       echo "- \${args[$k]} = ${args[$k]}"
+>     done
 >   else
 >     echo args: none
 >   fi
@@ -22,16 +24,18 @@
 >     readarray -t sorted_keys < <(printf '%s\n' "${!deps[@]}" | sort)
 >     echo
 >     echo deps:
->     for k in "${sorted_keys[@]}"; do echo "- \${deps[$k]} = ${deps[$k]}"; done
+>     for k in "${sorted_keys[@]}"; do
+>       echo "- \${deps[$k]} = ${deps[$k]}"
+>     done
 >   fi
 >
-if environment_variables.any?
-  >   echo
-  >   echo environment variables:
-  environment_variables.each do |env_var|
-    >   echo "- \${{ env_var.name.upcase }} = ${<%= env_var.name.upcase %>:-unset}"
-  end
-  >
-end
+>   if ((${#env_var_names[@]})); then
+>     readarray -t sorted_names < <(printf '%s\n' "${env_var_names[@]}" | sort)
+>     echo
+>     echo "environment variables:"
+>     for k in "${sorted_names[@]}"; do
+>       echo "- \$$k = ${!k:-}"
+>     done
+>   fi
 > }
 > 

lib/bashly/views/command/run.gtx

@@ -4,6 +4,7 @@
 >   declare -A args=()
 >   declare -A deps=()
 >   declare -a other_args=()
+>   declare -a env_var_names=()
 >   declare -a input=()
 >   normalize_input "$@"
 >   parse_requirements "${input[@]}"

spec/approvals/examples/command-aliases

@@ -71,7 +71,7 @@ args:
 - ${args[source]} = somefile
 
 environment variables:
-- $API_KEY = unset
+- $API_KEY = 
 + ./cli upload --help
 cli upload - Upload a file
 
@@ -124,4 +124,4 @@ args:
 - ${args[source]} = somefile
 
 environment variables:
-- $API_KEY = unset
+- $API_KEY =