markdown_exec 1.8.1 → 1.8.4

data/lib/link_history.rb

@@ -0,0 +1,131 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+# encoding=utf-8
+module MarkdownExec
+  class LinkState
+    attr_accessor :block_name, :document_filename,
+                  :inherited_block_names, :inherited_lines
+
+    # Initialize the LinkState with keyword arguments for each attribute.
+    # @param block_name [String, nil] the name of the block.
+    # @param document_filename [String, nil] the filename of the document.
+    # @param inherited_block_names [Array<String>, nil] the names of the inherited blocks.
+    # @param inherited_lines [Array<String>, nil] the inherited lines of code.
+    def initialize(block_name: nil, document_filename: nil,
+                   inherited_block_names: [], inherited_lines: nil)
+      @block_name = block_name
+      @document_filename = document_filename
+      @inherited_block_names = inherited_block_names
+      @inherited_lines = inherited_lines
+    end
+
+    # Creates an empty LinkState instance.
+    # @return [LinkState] an instance with all attributes set to their default values.
+    def self.empty
+      new
+    end
+
+    # Custom equality method to compare LinkState objects.
+    # @param other [LinkState] the other LinkState object to compare with.
+    # @return [Boolean] true if the objects are equal, false otherwise.
+    def ==(other)
+      other.class == self.class &&
+        other.block_name == block_name &&
+        other.document_filename == document_filename &&
+        other.inherited_block_names == inherited_block_names &&
+        other.inherited_lines == inherited_lines
+    end
+  end
+
+  class LinkHistory
+    def initialize
+      @history = []
+    end
+
+    # Peeks at the most recent LinkState, returns an empty LinkState if stack is empty.
+    def peek
+      @history.last || LinkState.empty
+    end
+
+    # Pops the most recent LinkState off the stack, returns an empty LinkState if stack is empty.
+    def pop
+      @history.pop || LinkState.empty
+    end
+
+    def prior_state_exist?
+      peek.document_filename.present?
+    end
+
+    # Pushes a LinkState onto the stack.
+    def push(link_state)
+      @history.push(link_state)
+    end
+  end
+end
+
+if $PROGRAM_NAME == __FILE__
+  require 'bundler/setup'
+  Bundler.require(:default)
+
+  require 'minitest/autorun'
+  require 'mocha/minitest'
+
+  module MarkdownExec
+    class TestLinkHistory < Minitest::Test
+      def setup
+        @link_history = LinkHistory.new
+        @link_state1 = LinkState.new(block_name: 'block1', document_filename: 'document1.txt',
+                                     inherited_lines: ['code1.rb'])
+        @link_state2 = LinkState.new(block_name: 'block2', document_filename: 'document2.txt',
+                                     inherited_lines: ['code2.rb'])
+      end
+
+      def test_push
+        @link_history.push(@link_state1)
+        assert_equal @link_state1, @link_history.peek
+      end
+
+      def test_pop
+        @link_history.push(@link_state1)
+        @link_history.push(@link_state2)
+        assert_equal @link_state2, @link_history.pop
+        assert_equal @link_state1, @link_history.peek
+      end
+
+      def test_peek_empty
+        assert_equal LinkState.empty, @link_history.peek
+      end
+
+      def test_pop_empty
+        assert_equal LinkState.empty, @link_history.pop
+      end
+    end
+  end
+end
+
+__END__
+
+To generate the Ruby classes `LinkState` and `LinkHistory` with their current features and specifications, including the custom `==` method for object comparison and the implementation to handle empty states, you can use the following prompt:
+
+---
+
+Create Ruby classes `LinkState` and `LinkHistory` with the following specifications:
+
+1. **Class `LinkState`**:
+    - Attributes: `block_name`, `document_filename`, `inherited_lines`.
+    - Initialize with optional parameters for each attribute, defaulting to `nil`.
+    - Include a class method `empty` that creates an instance with all attributes set to `nil`.
+    - Implement a custom `==` method for comparing instances based on attribute values.
+
+2. **Class `LinkHistory`**:
+    - Use an array to manage a stack of `LinkState` instances.
+    - Implement the following methods:
+        - `push`: Adds a `LinkState` instance to the top of the stack.
+        - `pop`: Removes and returns the most recent `LinkState` from the stack, or returns an empty `LinkState` if the stack is empty.
+        - `peek`: Returns the most recent `LinkState` without removing it from the stack, or an empty `LinkState` if the stack is empty.
+
+3. **Testing**:
+    - Write Minitest test cases to validate each method in `LinkHistory`. Test scenarios should include pushing and popping `LinkState` instances, and handling an empty `LinkHistory`.
+
+The goal is to create a robust and efficient implementation of these classes, ensuring proper handling of stack operations and object comparisons.

data/lib/markdown_exec/version.rb

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

data/lib/markdown_exec.rb

@@ -41,8 +41,6 @@ tap_config envvar: MarkdownExec::TAP_DEBUG
 $stderr.sync = true
 $stdout.sync = true
 
-MDE_HISTORY_ENV_NAME = 'MDE_MENU_HISTORY'
-
 # custom error: file specified is missing
 #
 class FileMissingError < StandardError; end
@@ -172,26 +170,6 @@ module MarkdownExec
       }
     end
 
-    def clear_required_file
-      ENV['MDE_LINK_REQUIRED_FILE'] = ''
-    end
-
-    # # Deletes a required temporary file specified by an environment variable.
-    # # The function checks if the file exists before attempting to delete it.
-    # # Clears the environment variable after deletion.
-    # #
-    # def delete_required_temp_file
-    #   temp_blocks_file_path = ENV.fetch('MDE_LINK_REQUIRED_FILE', nil)
-
-    #   return if temp_blocks_file_path.nil? || temp_blocks_file_path.empty?
-
-    #   FileUtils.rm_f(temp_blocks_file_path)
-
-    #   clear_required_file
-    # rescue StandardError
-    #   error_handler('delete_required_temp_file')
-    # end
-
     public
 
     ## Determines the correct filename to use for searching files
@@ -222,7 +200,7 @@ module MarkdownExec
     # Reports and executes block logic
     def execute_block_logic(files)
       @options[:filename] = select_document_if_multiple(files)
-      @options.select_approve_and_execute_block
+      @options.select_execute_bash_and_special_blocks
     end
 
     ## Executes the block specified in the options
@@ -244,17 +222,17 @@ module MarkdownExec
 
       simple_commands = {
         doc_glob: -> { @fout.fout options[:md_filename_glob] },
-        list_blocks: -> { list_blocks },
+        # list_blocks: -> { list_blocks },
+        list_default_env: -> { @fout.fout_list list_default_env },
         list_default_yaml: -> { @fout.fout_list list_default_yaml },
         list_docs: -> { @fout.fout_list files },
-        list_default_env: -> { @fout.fout_list list_default_env },
-        list_recent_output: lambda {
+        list_recent_output: -> {
                               @fout.fout_list list_recent_output(
                                 @options[:saved_stdout_folder],
                                 @options[:saved_stdout_glob], @options[:list_count]
                               )
                             },
-        list_recent_scripts: lambda {
+        list_recent_scripts: -> {
                                @fout.fout_list list_recent_scripts(
                                  options[:saved_script_folder],
                                  options[:saved_script_glob], options[:list_count]
@@ -358,19 +336,26 @@ module MarkdownExec
     def lambda_for_procname(procname, options)
       case procname
       when 'debug'
-        ->(value) {
-          tap_config value: value
-        }
+        ->(value) { tap_config value: value }
       when 'exit'
         ->(_) { exit }
-
       when 'find'
         ->(value) {
-          # initialize_and_parse_cli_options
-          @fout.fout "Searching in: " \
-           "#{HashDelegator.new(@options).string_send_color(@options[:path], :menu_chrome_color)}"
+          @fout.fout 'Searching in: ' \
+                     "#{HashDelegator.new(@options).string_send_color(@options[:path],
+                                                                      :menu_chrome_color)}"
           searcher = DirectorySearcher.new(value, [@options[:path]])
 
+          @fout.fout 'In file contents'
+          hash = searcher.search_in_file_contents
+          hash.each.with_index do |(key, v2), i1|
+            @fout.fout format('- %3.d: %s', i1 + 1, key)
+            @fout.fout AnsiFormatter.new(options).format_and_highlight_array(
+              v2.map { |nl| format('=%4.d: %s', nl.index, nl.line) },
+              highlight: [value]
+            )
+          end
+
           @fout.fout 'In directory names'
           @fout.fout AnsiFormatter.new(options).format_and_highlight_array(
             searcher.search_in_directory_names, highlight: [value]
@@ -381,27 +366,15 @@ module MarkdownExec
             searcher.search_in_file_names, highlight: [value]
           ).join("\n")
 
-          @fout.fout 'In file contents'
-          hash = searcher.search_in_file_contents
-          hash.each.with_index do |(key, v2), i1|
-            @fout.fout format('- %3.d: %s', i1 + 1, key)
-            @fout.fout AnsiFormatter.new(options).format_and_highlight_array(
-              v2.map { |nl| format('=%4.d: %s', nl.index, nl.line) },
-              highlight: [value]
-            )
-          end
           exit
         }
-
       when 'help'
         ->(_) {
           @fout.fout menu_help
           exit
         }
-      # when %w[who what where why how which when whom]
       when 'how'
         ->(value) {
-          # value = 'color'
           @fout.fout(list_default_yaml.select { |line| line.include? value })
           exit
         }
@@ -432,7 +405,7 @@ module MarkdownExec
       end
     end
 
-    def list_blocks; end
+    # def list_blocks; end
 
     def list_default_env
       menu_iter do |item|
@@ -551,10 +524,8 @@ module MarkdownExec
     public
 
     def run
-      clear_required_file
       initialize_and_parse_cli_options
       execute_block_with_error_handling
-      @options.delete_required_temp_file
     rescue StandardError
       error_handler('run')
     end
@@ -568,7 +539,7 @@ module MarkdownExec
 
       saved_name_split filename
       @options[:save_executed_script] = false
-      @options.select_approve_and_execute_block
+      @options.select_execute_bash_and_special_blocks
     rescue StandardError
       error_handler('run_last_script')
     end
@@ -635,7 +606,7 @@ module MarkdownExec
 
       saved_name_split(filename)
 
-      @options.select_approve_and_execute_block ### ({ save_executed_script: false })
+      @options.select_execute_bash_and_special_blocks ### ({ save_executed_script: false })
     end
 
     public

data/lib/mdoc.rb

@@ -3,8 +3,8 @@
 
 # encoding=utf-8
 
-require_relative 'filter'
 require_relative 'block_types'
+require_relative 'filter'
 
 module MarkdownExec
   ##
@@ -75,7 +75,6 @@ module MarkdownExec
         raise "Named code block `#{name}` not found. (@#{__LINE__})"
       end
 
-      # all_dependency_names = [name_block.oname] + recursively_required(name_block[:reqs])
       dependencies = collect_dependencies(name_block[:oname])
       all_dependency_names = collect_unique_names(dependencies).push(name_block[:oname]).uniq
       unmet_dependencies = all_dependency_names.dup
@@ -91,10 +90,11 @@ module MarkdownExec
         else
           []
         end + [fcb]
-      end.flatten(1) #.tap { rbp }
-      { all_dependency_names: all_dependency_names, 
-        blocks: blocks, 
-        dependencies: dependencies, 
+      end.flatten(1)
+
+      { all_dependency_names: all_dependency_names,
+        blocks: blocks,
+        dependencies: dependencies,
         unmet_dependencies: unmet_dependencies }
     end
 
@@ -103,26 +103,37 @@ module MarkdownExec
     # @param name [String] The name of the code block to start the collection from.
     # @return [Array<String>] An array of strings containing the collected code blocks.
     #
-    def collect_recursively_required_code(name, label_body: true, label_format_above: nil, label_format_below: nil)
+    def collect_recursively_required_code(name, label_body: true, label_format_above: nil,
+                                          label_format_below: nil, block_source:)
       block_search = collect_recursively_required_blocks(name)
-      block_search.merge({ code: collect_wrapped_blocks(block_search[:blocks]).map do |fcb|
-        if fcb[:cann]
-          collect_block_code_cann(fcb)
-        elsif fcb[:stdout]
-          collect_block_code_stdout(fcb)
-        elsif [BlockType::LINK, BlockType::OPTS,
-               BlockType::VARS].include? fcb[:shell]
-          nil
-        elsif fcb[:shell] == BlockType::PORT
-          collect_block_code_shell(fcb)
-        elsif label_body
-          [label_format_above && format(label_format_above, { name: fcb[:oname] })] + 
-           fcb[:body] + 
-           [label_format_below && format(label_format_below, { name: fcb[:oname] })]
-        else # raw body
-          fcb[:body]
-        end
-      end.compact.flatten(1).compact })
+      if block_search[:blocks]
+        blocks = collect_wrapped_blocks(block_search[:blocks])
+        block_search.merge(
+          { block_names: blocks.map { |block| block[:oname] },
+            code: blocks.map do |fcb|
+              if fcb[:cann]
+                collect_block_code_cann(fcb)
+              elsif fcb[:stdout]
+                collect_block_code_stdout(fcb)
+              elsif [BlockType::LINK, BlockType::OPTS,
+                     BlockType::VARS].include? fcb[:shell]
+                nil
+              elsif fcb[:shell] == BlockType::PORT
+                collect_block_code_shell(fcb)
+              elsif label_body
+                [label_format_above && format(label_format_above,
+                                              block_source.merge({ block_name: fcb[:oname] }))] +
+                 fcb[:body] +
+                 [label_format_below && format(label_format_below,
+                                               block_source.merge({ block_name: fcb[:oname] }))]
+              else # raw body
+                fcb[:body]
+              end
+            end.compact.flatten(1).compact }
+        )
+      else
+        block_search.merge({ block_names: [], code: [] })
+      end
     end
 
     def collect_unique_names(hash)
@@ -223,7 +234,6 @@ module MarkdownExec
       end
     end
 
-    # if tr   ue
     # Recursively fetches required code blocks for a given list of requirements.
     #
     # @param reqs [Array<String>] An array of requirements to start the recursion from.
@@ -247,7 +257,6 @@ module MarkdownExec
       memo
     end
 
-    # else
     # Recursively fetches required code blocks for a given list of requirements.
     #
     # @param source [String] The name of the code block to start the recursion from.
@@ -273,8 +282,6 @@ module MarkdownExec
       memo
     end
 
-    # end
-
     # Recursively collects dependencies of a given source.
     # @param source [String] The name of the initial source block.
     # @param memo [Hash] A memoization hash to store resolved dependencies.
@@ -283,11 +290,10 @@ module MarkdownExec
       return memo unless source
 
       if (block = get_block_by_anyname(source)).nil? || block.keys.empty?
-        if true
-          return memo
-        else
-          raise "Named code block `#{source}` not found. (@#{__LINE__})"
-        end
+        return memo if true
+
+        raise "Named code block `#{source}` not found. (@#{__LINE__})"
+
       end
 
       return memo unless block[:reqs]

data/lib/menu.src.yml

@@ -96,6 +96,7 @@
   :default: false
   :description: Dump BlocksInFile (stage 1)
   :env_var: MDE_DUMP_BLOCKS_IN_FILE
+  :long_name: dump-blocks-in-file
   :opt_name: dump_blocks_in_file
   :procname: val_as_bool
 
@@ -103,6 +104,7 @@
   :default: false
   :description: Dump MenuBlocks (stage 2)
   :env_var: MDE_DUMP_MENU_BLOCKS
+  :long_name: dump-menu-blocks
   :opt_name: dump_menu_blocks
   :procname: val_as_bool
 
@@ -110,10 +112,11 @@
   :default: false
   :description: Dump selected block
   :env_var: MDE_DUMP_SELECTED_BLOCK
+  :long_name: dump-selected-block
   :opt_name: dump_selected_block
   :procname: val_as_bool
 
-- :default: fg_rgbh_E0_E0_20 # Y
+- :default: fg_rgbh_ff_00_7f
   :description: Color of exception detail
   :env_var: MDE_EXCEPTION_COLOR_DETAIL
   :opt_name: exception_color_detail
@@ -125,7 +128,7 @@
   :opt_name: exception_format_detail
   :procname: val_as_str
 
-- :default: fg_rgbh_E0_20_20 # R
+- :default: fg_rgbh_ff_00_00
   :description: Color of exception name
   :env_var: MDE_EXCEPTION_COLOR_NAME
   :opt_name: exception_color_name
@@ -158,7 +161,7 @@
   :opt_name: exclude_expect_blocks
   :procname: val_as_bool
 
-- :default: fg_rgbh_20_80_80
+- :default: fg_rgbh_7f_ff_00
   :description: execution_report_preview_frame_color
   :env_var: MDE_EXECUTION_REPORT_PREVIEW_FRAME_COLOR
   :opt_name: execution_report_preview_frame_color
@@ -320,7 +323,7 @@
   :opt_name: menu_back_at_top
   :procname: val_as_bool
 
-- :default: fg_rgbh_40_c0_f0
+- :default: fg_rgbh_00_c0_c0
   :description: Color of menu bash
   :env_var: MDE_MENU_BASH_COLOR
   :opt_name: menu_bash_color
@@ -340,7 +343,7 @@
   :opt_name: menu_blocks_with_headings
   :procname: val_as_bool
 
-- :default: fg_rgbh_80_80_20
+- :default: fg_rgbh_40_c0_c0
   :description: Color of menu chrome
   :env_var: MDE_MENU_CHROME_COLOR
   :opt_name: menu_chrome_color
@@ -352,7 +355,7 @@
   :opt_name: menu_chrome_format
   :procname: val_as_str
 
-- :default: fg_rgbh_20_98_80
+- :default: fg_rgbh_80_d0_c0
   :description: Color of menu divider
   :env_var: MDE_MENU_DIVIDER_COLOR
   :opt_name: menu_divider_color
@@ -417,7 +420,7 @@
   :opt_name: menu_initial_divider
   :procname: val_as_str
 
-- :default: fg_rgbh_20_E0_20 # G
+- :default: fg_rgbh_e0_e0_20
   :description: Color of menu link
   :env_var: MDE_MENU_LINK_COLOR
   :opt_name: menu_link_color
@@ -429,7 +432,7 @@
   :opt_name: menu_link_format
   :procname: val_as_str
 
-- :default: fg_rgbh_40_a0_a0
+- :default: fg_rgbh_b0_b0_b0
   :description: Color of menu note
   :env_var: MDE_MENU_NOTE_COLOR
   :opt_name: menu_note_color
@@ -463,13 +466,13 @@
   :opt_name: menu_option_exit_name
   :procname: val_as_str
 
-- :default: fg_rgbh_E0_60_E0 # R
+- :default: fg_rgbh_ff_00_ff
   :description: Color of menu opts
   :env_var: MDE_MENU_OPTS_COLOR
   :opt_name: menu_opts_color
   :procname: val_as_str
 
-- :default: fg_rgbh_E0_20_20
+- :default: fg_rgbh_7f_00_ff
   :description: Color of menu opts
   :env_var: MDE_MENU_OPTS_SET_COLOR
   :opt_name: menu_opts_set_color
@@ -481,7 +484,7 @@
   :opt_name: menu_opts_set_format
   :procname: val_as_str
 
-- :default: fg_rgbh_A0_20_D0 # V
+- :default: fg_rgbh_ff_ff_ff
   :description: Color of menu task
   :env_var: MDE_MENU_TASK_COLOR
   :opt_name: menu_task_color
@@ -505,13 +508,13 @@
   :opt_name: menu_task_symbol
   :procname: val_as_str
 
-- :default: fg_rgbh_E0_80_20
+- :default: fg_rgbh_ff_a0_ff
   :description: Color of menu vars
   :env_var: MDE_MENU_VARS_COLOR
   :opt_name: menu_vars_color
   :procname: val_as_str
 
-- :default: fg_rgbh_E0_80_20
+- :default: fg_rgbh_00_ff_ff
   :description: Color of menu vars
   :env_var: MDE_MENU_VARS_SET_COLOR
   :opt_name: menu_vars_set_color
@@ -558,13 +561,13 @@
   :opt_name: output_execution_label_format
   :procname: val_as_str
 
-- :default: fg_rgbh_20_D8_80
+- :default: fg_rgbh_00_ff_00
   :description: Color of output_execution_label_name
   :env_var: MDE_OUTPUT_EXECUTION_LABEL_NAME_COLOR
   :opt_name: output_execution_label_name_color
   :procname: val_as_str
 
-- :default: fg_rgbh_20_E0_80
+- :default: fg_rgbh_00_ff_00
   :description: Color of output_execution_label_value
   :env_var: MDE_OUTPUT_EXECUTION_LABEL_VALUE_COLOR
   :opt_name: output_execution_label_value_color
@@ -626,7 +629,7 @@
   :opt_name: prompt_approve_block
   :procname: val_as_str
 
-- :default: fg_rgbh_20_E8_80
+- :default: fg_rgbh_00_ff_00
   :description: Color of prompt after script execution
   :env_var: MDE_PROMPT_COLOR_AFTER_SCRIPT_EXECUTION
   :opt_name: prompt_color_after_script_execution
@@ -768,7 +771,7 @@
   :opt_name: saved_stdout_glob
   :procname: val_as_str
 
-- :default: fg_rgbh_20_80_80
+- :default: fg_rgbh_00_ff_7f
   :description: script_execution_frame_color
   :env_var: MDE_SCRIPT_EXECUTION_FRAME_COLOR
   :opt_name: script_execution_frame_color
@@ -786,7 +789,7 @@
   :opt_name: script_execution_tail
   :procname: val_as_str
 
-- :default: fg_rgbh_20_80_80
+- :default: fg_rgbh_7f_ff_00
   :description: Color of output divider
   :env_var: MDE_OUTPUT_DIVIDER_COLOR
   :opt_name: script_preview_frame_color
@@ -846,13 +849,13 @@
   :opt_name: shell
   :procname: val_as_str
 
-- :default: "#  -^-  +%{name}  -^-"
+- :default: "#  -^-  +%{block_name}  -o-  %{document_filename}  -^-"
   :description: shell_code_label_format_above
   :env_var: MDE_SHELL_CODE_LABEL_FORMAT_ABOVE
   :opt_name: shell_code_label_format_above
   :procname: val_as_str
 
-- :default: "#  -v-  +%{name}  -v-"
+- :default: "#  -v-  +%{block_name}  -v-"
   :description: shell_code_label_format_below
   :env_var: MDE_SHELL_CODE_LABEL_FORMAT_BELOW
   :opt_name: shell_code_label_format_below
@@ -877,7 +880,7 @@
   :procname: version
   :short_name: v
 
-- :default: fg_rgbh_E0_E0_20 # Y
+- :default: fg_rgbh_ff_7f_00
   :description: Color of warning message
   :env_var: MDE_WARNING_COLOR
   :opt_name: warning_color