markdown_exec 3.0.8 → 3.0.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fd2355b28f935804fb135a650cc4260276be2bed20a8ed33cf9d188e01734466
-  data.tar.gz: 9e001b6252ebc4f012408e8a43373ab49b474d72982fc9bbc653c93e6d39c497
+  metadata.gz: db0df1de92fc6f0f9e015825c75de74529867c8e128ea1fa8042aeed6ce7a632
+  data.tar.gz: 2c3d84281cb92393d86b3920e365e5567823022cbbb8b0fc9121780160decc46
 SHA512:
-  metadata.gz: a17c1e662569ae1b0f34ecbb63571120c3d56bc2efdde29651240ec6d6d07e0f24e361ae9c95d34725f5c9f6a1c2d6f18fc9aad9971bdba3e2b2f850f14418d5
-  data.tar.gz: 95184776dfde2e6579aa18e0893a9f54180c2b641d65263f0d47f5533512611f3b92c9012959bb18c0c263ca3f50dccc42e4f69651f46a4af878d02784ce40ba
+  metadata.gz: dc269ea3ef8706322f031617df8b47fcaa9e14cbffc586bdf834e69c4d277120e0f9b7922257e7c68f332aa3ab188c9a79cd70f42fa8e1619d2d5beffac6cf8c
+  data.tar.gz: 042c2709076ce1eb8c201029f5404fab11933dea6f3d068616e2082bffed08a3baf6550fa5f6e9feaa7e3248797720df239a8e33e02a2e16385af4611d0d7a71

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    markdown_exec (3.0.8)
+    markdown_exec (3.0.9)
       clipboard (~> 1.3.6)
       open3 (~> 0.1.1)
       optparse (~> 0.1.1)

data/bats/cli.bats

@@ -36,6 +36,18 @@ load 'test_helper'
    ' ARG1: 37'
 }
 
+@test 'vars in link block are appended to inherited lines - local link' {
+  spec_mde_xansi_dname_doc_blocks_expect docs/dev/requiring-blocks.md \
+   '[link-local-block-with-vars]' \
+   '* Exit_# [link-local-block-with-vars]_ARG1="37"_block: echo-ARG1_  file: docs/dev/linked-file.md_  vars:_    ARG1: arg1-from-link-file_block: output_arguments_  vars:_    ARG1: 37_block: missing_ARG1=37_output_arguments'
+}
+
+@test 'vars in link block are appended to inherited lines - external file' {
+  spec_mde_xansi_dname_doc_blocks_expect docs/dev/requiring-blocks.md \
+   '[link-file-block-with-vars]' \
+   '* Exit_# [link-file-block-with-vars]_ARG1="arg1-from-link-file"_echo-ARG1'
+}
+
 # the last block is a link block, so menu is displayed
 @test 'link block setting an environment variable requires a bash block' {
   BATS_OUTPUT_FILTER=A

data/docs/dev/linked-file.md

@@ -1,3 +1,6 @@
 ```bash :echo-ARG1
 echo "ARG1: $ARG1"
+```
+```opts :(document_opts)
+menu_with_inherited_lines: true
 ```

data/docs/dev/requiring-blocks.md

@@ -21,4 +21,5 @@ echo "ARG1: $ARG1"
 @import bats-document-configuration.md
 ```opts :(document_opts)
 menu_with_exit: true
+menu_with_inherited_lines: true
 ```

data/lib/evaluate_shell_expressions.rb

@@ -45,7 +45,6 @@ def evaluate_shell_expressions(initial_code, expressions, shell: '/bin/bash',
   end
 
   result_hash
-
 rescue StandardError
   ww $@, $!, caller.deref
   ww initial_code, expressions

data/lib/fcb.rb

@@ -23,6 +23,7 @@ def parse_yaml_of_ux_block(
     default: export['default'],
     echo: export['echo'],
     exec: export['exec'],
+    force: export['force'],
     init: export['init'],
     menu_format: export['format'] || export['menu_format'] || menu_format,
     name: name,
@@ -156,20 +157,26 @@ module MarkdownExec
       @attrs[:id] = id
 
       if @attrs[:type] == BlockType::UX
-        case data = YAML.load(@attrs[:body].join("\n"))
-        when Hash
-          export = parse_yaml_of_ux_block(
-            data,
-            menu_format: menu_format,
-            prompt: prompt
-          )
-
-          @attrs[:center] = table_center
-          oname = @attrs[:oname] = format(export.menu_format, export.to_h)
-          @attrs[:readonly] = export.readonly
-        else
-          # triggered by an empty or non-YAML block
-          return NullResult.new(message: 'Invalid YAML', data: data)
+        begin
+          case data = YAML.load(@attrs[:body].join("\n"))
+          when Hash
+            export = parse_yaml_of_ux_block(
+              data,
+              menu_format: menu_format,
+              prompt: prompt
+            )
+
+            @attrs[:center] = table_center
+            oname = @attrs[:oname] = format(export.menu_format, export.to_h)
+            @attrs[:readonly] = export.readonly
+          else
+            # triggered by an empty or non-YAML block
+            return NullResult.new(message: 'Invalid YAML', data: data)
+          end
+        rescue StandardError
+          ww $@, $!, caller.deref
+          ww @attrs[:body], data, export
+          raise StandardError, $!
         end
       end
 

data/lib/hash_delegator.rb

@@ -954,7 +954,7 @@ module MarkdownExec
                 blocks << fcb unless result.failure?
               end
             rescue StandardError
-              # ww $@, $!
+              ww $@, $!, caller.deref
               HashDelegator.error_handler('blocks_from_nested_files',
                                           { abort: true })
             end
@@ -978,7 +978,7 @@ module MarkdownExec
       end
       OpenStruct.new(blocks: blocks, results: results)
     rescue StandardError
-      # ww $@, $!
+      ww $@, $!, caller.deref
       HashDelegator.error_handler('blocks_from_nested_files')
     end
 
@@ -994,9 +994,6 @@ module MarkdownExec
       initial_code_required: false,
       occurrence_expressions: nil
     )
-ww commands
-ww link_state
-# binding.irb####
       evaluate_shell_expressions(
         (link_state&.inherited_lines_block || ''),
         commands,
@@ -1113,11 +1110,9 @@ ww link_state
       selected, mdoc, inherited_code: nil, force: true, only_default: false,
       silent:
     )
-ww selected
       ret_command_result = nil
       exit_prompt = @delegate_object[:prompt_filespec_back]
 
-ww \
       required = mdoc.collect_recursively_required_code(
         anyname: selected.pub_name,
         label_format_above: @delegate_object[:shell_code_label_format_above],
@@ -1171,19 +1166,8 @@ ww \
           return command_result_w_e_t_nl if command_result_w_e_t_nl.failure?
 
           command_result_w_e_t_nl.new_lines =
-            command_result_w_e_t_nl.new_lines.map do |name_force|
-              transformed = if command_result_w_e_t_nl.transformable
-                              transform_export_value(name_force[:text], export)
-                            else
-                              name_force[:text]
-
-                            end
-              EnvInterface.set(name_force[:name], transformed)
-
-              code_line_safe_assign(
-                name_force[:name], transformed, force: name_force[:force]
-              )
-            end
+            process_command_result_lines(command_result_w_e_t_nl, export,
+                                         required_lines)
           required_lines.concat(command_result_w_e_t_nl.new_lines)
           ret_command_result = command_result_w_e_t_nl
         else
@@ -1193,22 +1177,19 @@ ww \
 
       ret_command_result || CommandResult.new(stdout: required_lines)
     rescue StandardError
-      ww $@, $!
+      ww $@, $!, caller.deref
       HashDelegator.error_handler('code_from_ux_block_to_set_environment_variables')
     end
 
-    def env_set(name, value)
-      EnvInterface.set(name, value)
-    end
-
     # sets ENV
     def code_from_vars_block_to_set_environment_variables(selected)
       code_lines = []
+      # code_lines << '# code_from_vars_block_to_set_environment_variables'
       case data = YAML.load(selected.body.join("\n"))
       when Hash
         data.each do |key, value|
           EnvInterface.set(key, value.to_s)
-          code_lines.push "#{key}=#{Shellwords.escape(value)}"
+          code_lines << assign_key_value_in_bash(key, value)
 
           next unless @delegate_object[:menu_vars_set_format].present?
 
@@ -1706,10 +1687,6 @@ ww \
       @prior_execution_block = nil
     end
 
-    def selected_shell(shell_name)
-      shell_name.empty? ? shell : shell_name
-    end
-
     # Determines the state of a selected block in the menu based
     #  on the selected option.
     # It categorizes the selected option into either EXIT, BACK,
@@ -1932,6 +1909,10 @@ ww \
       result_text
     end
 
+    def env_set(name, value)
+      EnvInterface.set(name, value)
+    end
+
     # Execute a code block after approval and provide user interaction options.
     #
     # This method displays required code blocks, asks for user approval, and
@@ -2176,10 +2157,10 @@ ww \
       # load key and values from link block into current environment
       #
       if link_block_data[LinkKeys::VARS]
-        code_lines.push BashCommentFormatter.format_comment(selected.pub_name)
-        (link_block_data[LinkKeys::VARS] || []).each do |(key, value)|
+        code_lines << BashCommentFormatter.format_comment(selected.pub_name)
+        link_block_data[LinkKeys::VARS].each do |key, value|
+          code_lines << assign_key_value_in_bash(key, value)
           EnvInterface.set(key, value.to_s)
-          code_lines.push(assign_key_value_in_bash(key, value))
         end
       end
 
@@ -2202,7 +2183,7 @@ ww \
       if link_block_data.fetch(LinkKeys::EVAL,
                                false) || link_block_data.fetch(LinkKeys::EXEC,
                                                                false)
-        code_lines = link_block_data_eval(
+        code_lines += link_block_data_eval(
           link_state, code_lines, selected, link_block_data,
           block_source: block_source,
           shell: @delegate_object[:block_type_default]
@@ -2540,7 +2521,6 @@ ww \
     end
 
     def expand_references!(fcb, link_state)
-ww fcb, link_state, caller.deref
       expand_variable_references!(
         blocks: [fcb],
         echo_formatter: method(:format_echo_command),
@@ -2583,22 +2563,15 @@ ww fcb, link_state, caller.deref
       link_state:,
       pattern:
     )
-ww blocks####
-ww link_state
-# binding.irb
       variable_counts, occurrence_expressions = count_named_group_occurrences(
         blocks, pattern, group_name: group_name
       )
       return if variable_counts.nil? || variable_counts == {}
-ww [variable_counts, occurrence_expressions]
 
-ww \
       echo_commands = generate_echo_commands(
         variable_counts, formatter: echo_formatter
       )
 
-ww link_state
-ww \
       replacements = build_replacement_dictionary(
         echo_commands, link_state,
         initial_code_required: initial_code_required,
@@ -2608,11 +2581,9 @@ ww \
       return if replacements.nil?
       if replacements == EvaluateShellExpression::StatusFail
         # happens on first processing of blocks before requirements are met
-        ww "EvaluateShellExpression::StatusFail", echo_commands, link_state
         return
       end
 
-ww \
       expand_blocks_with_replacements(blocks, replacements)
       # no return
     end
@@ -3015,7 +2986,11 @@ ww \
     # convert single items to arrays
     def join_array_of_arrays(*args)
       args.map do |item|
-        item.nil? ? nil : (item.is_a?(Array) ? item : [item])
+        if item.nil?
+          nil
+        else
+          (item.is_a?(Array) ? item : [item])
+        end
       end.compact.flatten(1)
     end
 
@@ -3639,13 +3614,14 @@ ww \
 
         exportable = export ? export.exportable : false
         new_lines = []
+        # new_lines << { comment: 'output_from_adhoc_bash_script_file' }
         if export
           new_lines << "#{export.name}="
           export_value = output
           ### transform?
 
           command_result, exportable, new_lines = export_echo_with_code(
-            ["#{export.name}=#{Shellwords.escape(export_value)}"],
+            [assign_key_value_in_bash(export.name, export_value)],
             export,
             force: force,
             silent: silent,
@@ -3811,6 +3787,35 @@ ww \
 
     # private
 
+    def process_command_result_lines(command_result_w_e_t_nl, export,
+                                     required_lines)
+      command_result_w_e_t_nl.new_lines.map do |name_force|
+        comment = name_force.fetch(:comment, '')
+        name = name_force.fetch(:name, '')
+        if !name.empty?
+          transformed = if command_result_w_e_t_nl.transformable
+                          transform_export_value(name_force[:text], export)
+                        else
+                          name_force[:text]
+                        end
+
+          # store the transformed value in ENV
+          EnvInterface.set(name, transformed)
+
+          set = code_line_safe_assign(
+            name, transformed, force: name_force[:force]
+          )
+
+          comment.empty? ? set : "#{set} # #{comment}"
+        else
+          "# #{comment}" if comment.empty?
+        end
+      end
+    rescue StandardError
+      ww $@, $!, caller.deref
+      raise StandardError, $!
+    end
+
     def process_string_array(arr, begin_pattern: nil, end_pattern: nil, scan1: nil,
                              format1: nil, name: '')
       in_block = !begin_pattern.present?
@@ -4440,6 +4445,10 @@ ww \
       selected
     end
 
+    def selected_shell(shell_name)
+      shell_name.empty? ? shell : shell_name
+    end
+
     def shell
       @delegate_object[:shell]
     end
@@ -4679,7 +4688,7 @@ ww \
     )
       command_result = nil
       exportable = true
-      force = true
+      force = export.force.nil? ? true : export.force
       new_lines = []
       silent = false
       transformable = true
@@ -4729,7 +4738,7 @@ ww \
         else
           export_init = menu_from_list_with_back(export.allow)
           command_result, exportable, new_lines = export_echo_with_code(
-            ["#{export.name}=#{Shellwords.escape(export_init)}"],
+            [assign_key_value_in_bash(export.name, export_init)],
             export,
             force: force,
             silent: silent,
@@ -4764,7 +4773,12 @@ ww \
           transformable = false
         end
 
-        command_result = CommandResult.new(stdout: output)
+        if exportable
+          EnvInterface.set(export.name, output)
+          new_lines << { name: export.name, force: force,
+                         text: output }
+          command_result = CommandResult.new(stdout: output)
+        end
 
       when :exec, UxActSource::EXEC
         command_result, exportable, new_lines = output_from_adhoc_bash_script_file(
@@ -4794,7 +4808,7 @@ ww \
     )
       command_result = nil
       exportable = true
-      force = false
+      force = export.force.nil? ? false : export.force
       new_lines = []
       silent = true
       transformable = true
@@ -4819,7 +4833,7 @@ ww \
           )
           export_init = cr_echo.stdout.split("\n").first
           command_result, exportable, new_lines = export_echo_with_code(
-            ["#{export.name}=#{Shellwords.escape(export_init)}"],
+            [assign_key_value_in_bash(export.name, export_init)],
             export,
             force: force,
             silent: silent,
@@ -4835,7 +4849,7 @@ ww \
           unless command_result.failure?
             export_init = command_result.stdout.split("\n").first
             command_result, exportable, new_lines = export_echo_with_code(
-              ["#{export.name}=#{Shellwords.escape(export_init)}"],
+              [assign_key_value_in_bash(export.name, export_init)],
               export,
               force: force,
               silent: silent,
@@ -4847,7 +4861,7 @@ ww \
           # first item from 'allow' list
           export_init = export.allow.first
           command_result, exportable, new_lines = export_echo_with_code(
-            ["#{export.name}=#{Shellwords.escape(export_init)}"],
+            [assign_key_value_in_bash(export.name, export_init)],
             export,
             force: force,
             silent: silent,
@@ -4880,7 +4894,7 @@ ww \
       else
         export_init = export.init.to_s
         command_result, exportable, new_lines = export_echo_with_code(
-          ["#{export.name}=#{Shellwords.escape(export_init)}"],
+          [assign_key_value_in_bash(export.name, export_init)],
           export,
           force: force,
           silent: silent,
@@ -5337,7 +5351,6 @@ ww \
         mdoc_menu_and_blocks_from_nested_files(
           @dml_link_state, source_id: source_id
         )
-ww @dml_link_state####
       dump_delobj(@dml_blocks_in_file, @dml_menu_blocks, @dml_link_state)
     end
 
@@ -5884,7 +5897,7 @@ module MarkdownExec
         mdoc: @mdoc, selected: @selected, block_source: {}
       )
 
-      assert_equal ['code line', 'key=value'], result
+      assert_equal ['code line', 'key="value"'], result
     end
   end
 

data/lib/markdown_exec/version.rb

@@ -7,5 +7,5 @@ module MarkdownExec
   BIN_NAME = 'mde'
   GEM_NAME = 'markdown_exec'
   TAP_DEBUG = 'MDE_DEBUG'
-  VERSION = '3.0.8'
+  VERSION = '3.0.9'
 end

data/lib/mdoc.rb

@@ -178,11 +178,12 @@ module MarkdownExec
                 collect_block_code_stdout(fcb)
               elsif [BlockType::OPTS].include? fcb.type
                 fcb.body # entire body is returned to requesing block
+
               elsif [BlockType::LINK,
                      BlockType::LOAD,
                      BlockType::UX,
                      BlockType::VARS].include? fcb.type
-                nil
+                nil # Vars for all types are collected later
               elsif fcb[:chrome] # for Link blocks like History
                 nil
               elsif fcb.type == BlockType::PORT

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: markdown_exec
 version: !ruby/object:Gem::Version
-  version: 3.0.8
+  version: 3.0.9
 platform: ruby
 authors:
 - Fareed Stevenson
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-06-11 00:00:00.000000000 Z
+date: 2025-06-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: clipboard